Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[DI] Prepare dropping "strict" handling in loaders#20938
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
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| $invalidBehavior = ContainerInterface::NULL_ON_INVALID_REFERENCE; | ||
| } | ||
| if ($strict =$arg->getAttribute('strict')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I would still trigger a deprecation if the attribute is there, otherwise we cannot remove it from the XSD in 4.x
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
But there is no way to create any continuous upgrade path in XML files. That's why we didn't remove "strict" from the XSD, to allow cross 2x3 bundles. Should we consider this is not required anymore? Not sure on my side (because it doesn't hurt). WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
But the same is true for the YAML format, isn't it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Well, you kept the attribute in the XSD, which is fine to allow the continuous migration. But you should now trigger the deprecation to allow the removal in 4.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
@xabbuh the YAML format does not have a schema restricting what gets written.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
@stof I meant providing a file that works with both 2.x and 3.x. If we didn't trigger the deprecation with the XML format, it means that we assumed something did set thestrict option tofalse (otherwise you wouldn't need to provide the option at all). However, in YAML that wouldn't be necessary at all.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
@xabbuh setting a reference as not strict is deprecated in 2.8 already. The option is a no-op in 3.0.
So the way to make a file work in 2.8, 3.x and 4.0 is to skip the attribute entirely and avoiding the usage of non-strict references (which are a source of bugs anyway, as it means you bypass the scope restrictions)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
fabpot commentedDec 26, 2016
Thank you@nicolas-grekas. |
…colas-grekas)This PR was merged into the 3.3-dev branch.Discussion----------[DI] Prepare dropping "strict" handling in loaders| Q | A| ------------- | ---| Branch? | master| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | yes| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR | -"strict" has been deprecated in 2.8, removed from 3.0, but loaders still read them(note that the XSD still mentions it for BC at XML level, but it has no effect.)We could also deprecate the attribute in XML, but I don't see the need since it's already ignored.In YAML, we need people to remove this trailing `=`, which is why the PR is on master with a deprecation.Commits-------243d160 [DI] Prepare dropping "strict" handling in loaders
This PR was merged into the 3.3-dev branch.Discussion----------[DI] deprecate the strict attribute| Q | A| ------------- | ---| Branch? | master| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | yes| Tests pass? | yes| Fixed tickets |#20938 (comment)| License | MIT| Doc PR |Commits-------49cb4db [DI] deprecate the strict attribute
Uh oh!
There was an error while loading.Please reload this page.
"strict" has been deprecated in 2.8, removed from 3.0, but loaders still read them
(note that the XSD still mentions it for BC at XML level, but it has no effect.)
We could also deprecate the attribute in XML, but I don't see the need since it's already ignored.
In YAML, we need people to remove this trailing
=, which is why the PR is on master with a deprecation.