Skip to content

Add an explanation about «constraints» validation #7664

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

Closed
wants to merge 2 commits into from
Closed

Add an explanation about «constraints» validation #7664

wants to merge 2 commits into from

Conversation

chindit
Copy link
Contributor

@chindit chindit commented Mar 22, 2017

Added an explanation about the use of «constraints» in Form Classes and the requirement of «ValidatorExtension» if «constraints» keyword is used.

This is currently not explained in doc, nor in Forms, Validation, Custom validation or any other page.

Added an explanation about the use of «constraints» in Form Classes and the requirement of «ValidatorExtension» if «constraints» keyword is used.
Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

👍 @chindit thanks for your contribution.

I made some minor rewords to match the style of the rest of the docs (for example using array() instead of [ ])

Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

👍

@xabbuh
Copy link
Member

xabbuh commented Apr 15, 2017

Thank you @chindit.

xabbuh added a commit that referenced this pull request Apr 15, 2017
…t, javiereguiluz)

This PR was submitted for the 3.2 branch but it was merged into the 2.7 branch instead (closes #7664).

Discussion
----------

Add an explanation about «constraints» validation

Added an explanation about the use of «constraints» in Form Classes and the requirement of «ValidatorExtension» if «constraints» keyword is used.

This is currently not explained in doc, nor in Forms, Validation, Custom validation or any other page.

Commits
-------

c9f1b95 Rewords and minor fixes
cb6b2e6 Add an explanation about «constraints» validation
xabbuh added a commit that referenced this pull request Apr 15, 2017
@xabbuh xabbuh closed this Apr 15, 2017
xabbuh added a commit that referenced this pull request Apr 15, 2017
* 2.7: (31 commits)
  Fixed the RST syntax
  Improve example context
  [#5621] Enhancing example of using bundle config
  [#7601] minor tweak
  Update expiration.rst
  Update expiration.rst
  Update expiration.rst
  Update expiration.rst
  Minor reword and fixed the line length
  Improve specification explanation
  [#7664] minor wording tweak
  Rewords and minor fixes
  Add an explanation about «constraints» validation
  [#7645] enumerate ordered list items implicitly
  Adding a new article about "Creating a Bug Reproducer"
  Fixed a syntax issue
  Use backticks
  #7311 choice_value callback argument can be null
  Fixed broken links for nginx & FastCgiExternalServer
  Update dialoghelper.rst
  ...
xabbuh added a commit that referenced this pull request Apr 15, 2017
* 2.8: (46 commits)
  [#7507] fix component name
  [#7490] minor typo fix
  Added a note about redirections to absolute URLs in tests
  Added the changes suggested by reviewers
  [#7620] use generate() in PHP templates before 2.8
  Fixed the RST syntax
  Improve example context
  [#5621] Enhancing example of using bundle config
  [#7601] minor tweak
  Update expiration.rst
  Update expiration.rst
  Update expiration.rst
  Update expiration.rst
  Minor reword and fixed the line length
  Improve specification explanation
  [#7664] minor wording tweak
  Rewords and minor fixes
  Add an explanation about «constraints» validation
  [#7645] enumerate ordered list items implicitly
  Adding a new article about "Creating a Bug Reproducer"
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants