-
Notifications
You must be signed in to change notification settings - Fork 347
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
Optional extra arguments for custom error messages #142
Conversation
{ | ||
$this->errors[] = array( | ||
$a = array( |
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.
please don't use such a meaningless names
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.
Hmm, it's better to use meaningless very short names for variables that are being used temporarily and locally within such a small method, but fine I'll change the name.
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.
you never know if this method will be larger in the future
Needs rebase |
@@ -7,3 +7,5 @@ coverage | |||
.settings | |||
composer.lock | |||
docs-api | |||
/.idea | |||
/composer.phar |
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.
Please move these to you global ignore
@onlinesid thanks for the rebase. Would you please squash this down to a single commit? |
…or custom error messages * Added more information to the error message of StringConstraint * Added more information to the Constraint addError * Unknown format does not generate error * Updated the test so that unknown format does not throw an error * Make messages to start with a capital letter consistently
I've squashed into a single commit now |
@Maks3w what are your thoughts on the the version number to iterate to when this is merged? |
Optional extra arguments for custom error messages
And unknown format no longer generate error