Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Profiler][Validator] ValidatorDataCollector: use new DataCollector::getCasters() method #23517

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

Merged
merged 1 commit into from
Jul 17, 2017
Merged

[Profiler][Validator] ValidatorDataCollector: use new DataCollector::getCasters() method #23517

merged 1 commit into from
Jul 17, 2017

Conversation

ogizanagi
Copy link
Contributor

@ogizanagi ogizanagi commented Jul 15, 2017

Q A
Branch? 3.4
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #23465 (comment)
License MIT
Doc PR N/A

First commit targets 3.3; see #23516.

I didn't re-used the ConstraintViolationInterface caster used in the form collector, as it's the purpose of the validator collector to show the constraints data.

@nicolas-grekas
Copy link
Member

first commit can be removed by rebasing now

@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Jul 15, 2017
@ogizanagi
Copy link
Contributor Author

Rebased

@fabpot
Copy link
Member

fabpot commented Jul 17, 2017

Thank you @ogizanagi.

@fabpot fabpot merged commit c725a70 into symfony:3.4 Jul 17, 2017
fabpot added a commit that referenced this pull request Jul 17, 2017
…taCollector::getCasters() method (ogizanagi)

This PR was merged into the 3.4 branch.

Discussion
----------

[Profiler][Validator] ValidatorDataCollector: use new DataCollector::getCasters() method

| Q             | A
| ------------- | ---
| Branch?       | 3.4 <!-- see comment below -->
| Bug fix?      | no
| New feature?  | no <!-- don't forget updating src/**/CHANGELOG.md files -->
| BC breaks?    | no
| Deprecations? | no <!-- don't forget updating UPGRADE-*.md files -->
| Tests pass?   | yes
| Fixed tickets | #23465 (comment) <!-- #-prefixed issue number(s), if any -->
| License       | MIT
| Doc PR        | N/A

~~First commit targets 3.3; see #23516

I didn't re-used the `ConstraintViolationInterface` caster used in the form collector, as it's the purpose of the validator collector to show the constraints data.

Commits
-------

c725a70 [Profiler][Validator] ValidatorDataCollector: use new DataCollector::getCasters() method
@ogizanagi ogizanagi deleted the validator/3.4/collector_casters branch July 17, 2017 11:03
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.

4 participants