8000 [Validator] Fixed string conversion in constraint violations by webmozart · Pull Request #10687 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[Validator] Fixed string conversion in constraint violations #10687

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.

Sign up for GitHub

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

Merged
merged 12 commits into from
Jul 30, 2014
Merged

[Validator] Fixed string conversion in constraint violations #10687

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
75e8815
[Validator] Fix constraint violation message parameterization
dnohales Apr 9, 2014
97243bc
[Validator] Fixed value-to-string conversion in constraint violations
webmozart Apr 10, 2014
fd58870
[Validator] Simplified IBAN validation algorithm
webmozart Apr 10, 2014
224e70f
[Validator] Fixed and simplified IsbnValidator
webmozart Apr 10, 2014
bff09f2
[Validator] Simplified IssnValidator
webmozart Apr 10, 2014
f329552
[Validator] Simplified and explained the LuhnValidator
webmozart Apr 10, 2014
5aa7e6d
[Validator] Added "{{ value }}" parameters where they were missing
webmozart Apr 10, 2014
cea4155
[Validator] Fixed date-to-string conversion tests to match ICU 51
webmozart May 22, 2014
71897d7
[Validator] Fixed CS
webmozart May 22, 2014
d6a783f
[Validator] Renamed valueToString() to formatValue(); added missing f…
webmozart Jul 24, 2014
08ea6d3
[Validator] Removed information from the violation output if the valu…
webmozart Jul 28, 2014
32ae95b
[Validator] Added more detailed inline documentation
webmozart Jul 30, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[Validator] Fixed CS
  • Loading branch information
@webmozart
webmozart committed Jul 24, 2014
commit 71897d7e3591d4664204da17f50c8587ccd65582
8 changes: 4 additions & 4 deletions src/Symfony/Component/Validator/ConstraintValidator.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@ protected function valueToType($value)
/**
* Returns a string representation of the value.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add some more explanations about when/why/how this is ever called? Something along the lines of:

"This method is used when a constraint error message contains the {{ value }} placeholder. By default, Symfony never includes it, but the developer can override those error messages. It's up to the developer to make sure it actually makes sense to include such string representation of submitted values in error messages."

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

*
* @param mixed $value
* @param Boolean $formatDates
* @param mixed $value
* @param bool $formatDates
*
* @return string
*/
Expand Down Expand Up @@ -100,8 +100,8 @@ protected function valueToString($value, $formatDates = false)
/**
* Returns a string representation of a list of values.
*
* @param array $values
* @param Boolean $formatDates
* @param array $values
* @param bool $formatDates
*
* @return string
*/
Expand Down
0