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] Allow Choice Constraint Callback to be non-static#19959

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

Conversation

@patrick-mcdougle
Copy link
Contributor

QA
Branch?master
Bug fix?no
New feature?yes
BC breaks?no
Deprecations?no
Tests pass?yes
Fixed ticketsnone
LicenseMIT
Doc PRwill write if community wants this feature

Now that as of 3.0 the ExecutionContextInterface has a getObject method, the callback doesn't need to be static anymore, it can actually be on the object that's being validated. This is great because you can actually have dynamic choices and this validator will actually work. Maintainers are free to add commits to this branch.

@patrick-mcdouglepatrick-mcdougle changed the titleAllow Choice Constraint Callback to be non-static[Validator] Allow Choice Constraint Callback to be non-staticSep 17, 2016
@patrick-mcdougle
Copy link
ContributorAuthor

@webmozart Travis failure seems unrelated. What do you think?

@fabpot
Copy link
Member

👍

@ro0NL
Copy link
Contributor

Duplicate of#19745 😭

@patrick-mcdougle
Copy link
ContributorAuthor

@ro0NL 😭 indeed. I totally didn't notice that. Someone should merge that one instead.

@patrick-mcdouglepatrick-mcdougle deleted the pmcd-choice-validator-feature branchSeptember 17, 2016 19:43
@ro0NL
Copy link
Contributor

ro0NL commentedSep 17, 2016
edited
Loading

I know the feeling. And yes.. it totally feels like feature-freeze right now.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@patrick-mcdougle@fabpot@ro0NL@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp