-
Notifications
You must be signed in to change notification settings - Fork 823
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
API Deprecate renamed validators #11490
API Deprecate renamed validators #11490
Conversation
ff8253f
to
ad2d3fc
Compare
ad2d3fc
to
b37542b
Compare
b37542b
to
ee49690
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just this one small change (sorry, was my fault. I gave the wrong capitalisation in my recommended change)
src/Forms/FieldsValidator.php
Outdated
* @deprecated 5.4.0 Will be replaced with functionality inside Silverstripe\Forms\Form::validate() | ||
*/ | ||
class FieldsValidator extends Validator | ||
{ | ||
public function __construct() | ||
{ | ||
Deprecation::noticeWithNoReplacment( | ||
'5.4.0', | ||
'Will be replaced with functionality inside Silverstripe\Forms\Form::validate()', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* @deprecated 5.4.0 Will be replaced with functionality inside Silverstripe\Forms\Form::validate() | |
*/ | |
class FieldsValidator extends Validator | |
{ | |
public function __construct() | |
{ | |
Deprecation::noticeWithNoReplacment( | |
'5.4.0', | |
'Will be replaced with functionality inside Silverstripe\Forms\Form::validate()', | |
* @deprecated 5.4.0 Will be replaced with functionality inside SilverStripe\Forms\Form::validate() | |
*/ | |
class FieldsValidator extends Validator | |
{ | |
public function __construct() | |
{ | |
Deprecation::noticeWithNoReplacment( | |
'5.4.0', | |
'Will be replaced with functionality inside SilverStripe\Forms\Form::validate()', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
ee49690
to
7d25f7f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, CI failures aren't related.
Issue #10908