Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
Replaced setDefaultOptions by the new configureOptions method#4786
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
Closed
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
👍 |
fabpot added a commit to symfony/symfony that referenced this pull requestJan 18, 2015
…igureOptions() (peterrehm)This PR was merged into the 2.7 branch.Discussion----------[Form] Deprecated setDefaultOptions() in favor of configureOptions()| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | yes| Tests pass? || Fixed tickets |#12782| License | MIT| Doc PR |symfony/symfony-docs#4786This tries to provide a compatible API with the depreciation of the OptionResolverInterface. I would like to have this in 2.6.2 but I think that might not be possible? To me I think we should always provide an API where you do not need to use deprecated classes.Also can you think of any way to trigger errors on the use of the deprecated setDefaultOptions() method? Since it is usually overwritten without calling the parent class this might be tricky. Maybe only in the resolver if we can check if actual options has been resolved in a call to setDefaultOptions.Commits-------3d43cae Deprecated setDefaultOptions() in favor of configureOptions()
fabpot added a commit to symfony/security-bundle that referenced this pull requestJan 18, 2015
…igureOptions() (peterrehm)This PR was merged into the 2.7 branch.Discussion----------[Form] Deprecated setDefaultOptions() in favor of configureOptions()| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | yes| Tests pass? || Fixed tickets | #12782| License | MIT| Doc PR |symfony/symfony-docs#4786This tries to provide a compatible API with the depreciation of the OptionResolverInterface. I would like to have this in 2.6.2 but I think that might not be possible? To me I think we should always provide an API where you do not need to use deprecated classes.Also can you think of any way to trigger errors on the use of the deprecated setDefaultOptions() method? Since it is usually overwritten without calling the parent class this might be tricky. Maybe only in the resolver if we can check if actual options has been resolved in a call to setDefaultOptions.Commits-------3d43cae Deprecated setDefaultOptions() in favor of configureOptions()
fabpot added a commit to symfony/form that referenced this pull requestJan 18, 2015
…igureOptions() (peterrehm)This PR was merged into the 2.7 branch.Discussion----------[Form] Deprecated setDefaultOptions() in favor of configureOptions()| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | yes| Tests pass? || Fixed tickets | #12782| License | MIT| Doc PR |symfony/symfony-docs#4786This tries to provide a compatible API with the depreciation of the OptionResolverInterface. I would like to have this in 2.6.2 but I think that might not be possible? To me I think we should always provide an API where you do not need to use deprecated classes.Also can you think of any way to trigger errors on the use of the deprecated setDefaultOptions() method? Since it is usually overwritten without calling the parent class this might be tricky. Maybe only in the resolver if we can check if actual options has been resolved in a call to setDefaultOptions.Commits-------3d43cae Deprecated setDefaultOptions() in favor of configureOptions()
@wouterj Sincesymfony/symfony#12891 is merged, you can remove the on hold label or even just merge it :) |
Great job@peterrehm ! |
fabpot added a commit to symfony/form that referenced this pull requestJan 20, 2015
…igureOptions() (peterrehm)This PR was merged into the 2.7 branch.Discussion----------[Form] Deprecated setDefaultOptions() in favor of configureOptions()| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | yes| Tests pass? || Fixed tickets | #12782| License | MIT| Doc PR |symfony/symfony-docs#4786This tries to provide a compatible API with the depreciation of the OptionResolverInterface. I would like to have this in 2.6.2 but I think that might not be possible? To me I think we should always provide an API where you do not need to use deprecated classes.Also can you think of any way to trigger errors on the use of the deprecated setDefaultOptions() method? Since it is usually overwritten without calling the parent class this might be tricky. Maybe only in the resolver if we can check if actual options has been resolved in a call to setDefaultOptions.Commits-------3d43cae Deprecated setDefaultOptions() in favor of configureOptions()
@peterrehm seems like this needs a rebase. And I assume this has to be merged into the 2.7 branch? |
I have rebased it based on 2.7. It must be merged into 2.7, you are right. |
weaverryan added a commit that referenced this pull requestJan 21, 2015
…method (peterrehm)This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes#4786).Discussion----------Replaced setDefaultOptions by the new configureOptions method| Q | A| ------------- | ---| Doc fix? | yes| New docs? | yes|no (PRsymfony/symfony#12891)| Applies to | 2.7| Fixed tickets |#4565Pending on the merge ofsymfony/symfony#12891.With this PR all references to setDefaultOptions and the deprecated OptionResolverInterface are removed in 2.7.Commits-------728205f Replaced setDefaultOptions by the new configureOptions method
weaverryan added a commit that referenced this pull requestJan 21, 2015
Ah, thanks Peter! I'mvery happy about this - great docs PR, and I'm happy to have this in the code (and the new method name is such an improvement!). I did add 2 Thanks! |
Perfect, thanks! :) |
weaverryan added a commit that referenced this pull requestJan 25, 2015
* 2.7: [#4805] Adding versionadded added documentation for the new absolute_url() and relative_path() Twig functions Remove 'acme' Update routing.rst [#4786] Adding a few versionadded's for the changed method name Replaced setDefaultOptions by the new configureOptions method Typo Update introduction.rst [Book][Translation] Added tip for routing params [varnish] be more precise about version differences Remove trailing whitespace [BestPractices] fix minor typo Remove horizontal scrollbar [Components][Debug] fix DebugClassLoader namespace Update override.rst Update override.rst [Book][Security] add back old anchors [Cookbook][Security] Hint about createToken can return null Add version added note for the debug:event-dispatcher commandConflicts:reference/twig_reference.rst
fabpot added a commit to symfony/symfony that referenced this pull requestFeb 5, 2015
…ResolverInterface (peterrehm)This PR was merged into the 3.0-dev branch.Discussion----------[Form] Remove deprecated setDefaultOptions and OptionsResolverInterface| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | yes| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR |symfony/symfony-docs#4786| Related |#12891Commits-------9b9158b Remove the deprecated OptionsResolverInterface6026781 Removed deprecated setDefaultOptions methods
fabpot added a commit to symfony/form that referenced this pull requestFeb 5, 2015
…ResolverInterface (peterrehm)This PR was merged into the 3.0-dev branch.Discussion----------[Form] Remove deprecated setDefaultOptions and OptionsResolverInterface| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | yes| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR |symfony/symfony-docs#4786| Related | #12891Commits-------9b9158b Remove the deprecated OptionsResolverInterface6026781 Removed deprecated setDefaultOptions methods
fabpot added a commit to symfony/options-resolver that referenced this pull requestFeb 5, 2015
…ResolverInterface (peterrehm)This PR was merged into the 3.0-dev branch.Discussion----------[Form] Remove deprecated setDefaultOptions and OptionsResolverInterface| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | yes| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR |symfony/symfony-docs#4786| Related | #12891Commits-------9b9158b Remove the deprecated OptionsResolverInterface6026781 Removed deprecated setDefaultOptions methods
javiereguiluz pushed a commit to javiereguiluz/symfony-docs that referenced this pull requestJun 22, 2015
fabpot added a commit to symfony/form that referenced this pull requestDec 1, 2015
…ResolverInterface (peterrehm)This PR was merged into the 3.0-dev branch.Discussion----------[Form] Remove deprecated setDefaultOptions and OptionsResolverInterface| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | yes| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR |symfony/symfony-docs#4786| Related | #12891Commits-------9b9158b Remove the deprecated OptionsResolverInterface6026781 Removed deprecated setDefaultOptions methods
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pending on the merge ofsymfony/symfony#12891.
With this PR all references to setDefaultOptions and the deprecated OptionResolverInterface are removed in 2.7.