Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Validator] Added documentation for Traverse constraint#11310

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
wouterj merged 1 commit intosymfony:3.4fromHeahDude:validator/traverse
Apr 6, 2019

Conversation

HeahDude
Copy link
Contributor

versionadded:: 2.5    The traverse constraint has been added in Symfony 2.5

x)

@HeahDudeHeahDude added Validator ⭐️ EU-FOSSA Hackathonhttps://symfony.com/blog/the-symfony-and-api-platform-hackathon-is-coming labelsApr 6, 2019
@HeahDudeHeahDude added this to the3.4 milestoneApr 6, 2019
@HeahDudeHeahDude self-assigned thisApr 6, 2019
@wouterj
Copy link
Member

Woohoo, yet another undocumented feature documented! #EUFOSSA

OskarStark and HeahDude reacted with thumbs up emoji

@wouterjwouterj merged commit86ee59e intosymfony:3.4Apr 6, 2019
wouterj added a commit that referenced this pull requestApr 6, 2019
…t (HeahDude)This PR was squashed before being merged into the 3.4 branch (closes#11310).Discussion----------[Validator] Added documentation for Traverse constraint```rstversionadded:: 2.5    The traverse constraint has been added in Symfony 2.5```x)Commits-------86ee59e [Validator] Added documentation for Traverse constraint
@HeahDudeHeahDude deleted the validator/traverse branchApril 6, 2019 14:03
+----------------+-------------------------------------------------------------------------------------+
| Options | - `payload`_ |
+----------------+-------------------------------------------------------------------------------------+
| Class | :class:`Symfony\\Bridge\\Doctrine\\Validator\\Constraints\\Traverse` |
Copy link
Contributor

Choose a reason for hiding this comment

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

HeahDude reacted with thumbs up emoji
Copy link
Contributor

Choose a reason for hiding this comment

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

Also, why isn't there any dedicated validator for this specific constraint?

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

See#11320. LikeValid those constraints are handled by the recursive context itself to define the cascade strategy, so do not need custom validator.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ooh, TIL, thanks 😄

xabbuh added a commit that referenced this pull requestApr 6, 2019
xabbuh added a commit that referenced this pull requestApr 6, 2019
* 3.4:  [#11310] fix reference  [#11325] Use correct line lengths  Link to review comments documentation
xabbuh added a commit that referenced this pull requestApr 6, 2019
* 4.2:  [#11310] fix reference
@guilliamxavier
Copy link
Contributor

Sorry to be late (and thanks for improving the docs) but it seems that it doesn't work that way... the example throws;

Symfony\Component\Validator\Exception\ConstraintDefinitionException: "Traversal was enabled for "App\Entity\Book", but this class does not implement "\Traversable"." at [...]/vendor/symfony/validator/Validator/RecursiveContextualValidator.php

HeahDude added a commit that referenced this pull requestApr 12, 2020
This PR was merged into the 3.4 branch.Discussion----------[Validator] Fixed `Traverse` constraint referenceI'm sorry I've messed up things in#11310. Now I see that I missed#12967, this fixes it.Commits-------ef5c989 [Validator] Fixed `Traverse` constraint reference
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@PierstovalPierstovalPierstoval left review comments

@wouterjwouterjwouterj requested changes

@curry684curry684curry684 requested changes

@ToflarToflarToflar approved these changes

@OskarStarkOskarStarkOskarStark approved these changes

Assignees

@HeahDudeHeahDude

Labels
⭐️ EU-FOSSA Hackathonhttps://symfony.com/blog/the-symfony-and-api-platform-hackathon-is-comingStatus: ReviewedValidator
Projects
None yet
Milestone
3.4
Development

Successfully merging this pull request may close these issues.

8 participants
@HeahDude@wouterj@guilliamxavier@Toflar@OskarStark@curry684@Pierstoval@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp