Skip to content

[Form] Avoid a form type extension appears many times in debug:form #30396

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
Mar 7, 2019
Merged

Conversation

markitosgv
Copy link
Contributor

@markitosgv markitosgv commented Feb 27, 2019

Q A
Branch? 4.2
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #30394
License MIT

This PR fixes #30394. Avoid a form type extension appears many times in debug:form command. This is caused by new 4.2 feature getExtendedTypes().

Copy link
Member

@yceruto yceruto left a comment

Choose a reason for hiding this comment

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

👍 Congratulation for your first PR in this repo 🎉

@xabbuh xabbuh added this to the 4.2 milestone Feb 27, 2019
@OskarStark
Copy link
Contributor

Could you add a testcase for this? As before it didn’t pop up as a failure

Or is this not needed?

@nicolas-grekas
Copy link
Member

Thank you @markitosgv.

@nicolas-grekas nicolas-grekas merged commit c4be39c into symfony:4.2 Mar 7, 2019
nicolas-grekas added a commit that referenced this pull request Mar 7, 2019
…ebug:form (markitosgv)

This PR was merged into the 4.2 branch.

Discussion
----------

[Form] Avoid a form type extension appears many times in debug:form

| Q             | A
| ------------- | ---
| Branch?       | 4.2
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #30394    <!-- #-prefixed issue number(s), if any -->
| License       | MIT

This PR fixes #30394. Avoid a form type extension appears many times in debug:form command. This is caused by new 4.2 feature getExtendedTypes().

Commits
-------

c4be39c [Form] Fixes debug:form appears many times as type extensions configured with new getExtendedTypes method
@fabpot fabpot mentioned this pull request Apr 2, 2019
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.

7 participants