bug #17605 [FrameworkBundle] remove default null value for asset version (xabbuh)
This PR was merged into the 2.7 branch.
Discussion
----------
[FrameworkBundle] remove default null value for asset version
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
Setting `null` as the version of a package means that it uses the empty
version strategy. However, omitting the `version` option entirely was
meant to fall back to the default version strategy. This is not possible
when the default version value is `null` as there is no way to remove
it.
Commits
-------
25f735f
remove default null value for asset version
This commit is contained in:
commit
8c0b48f6ca
@ -558,7 +558,6 @@ class Configuration implements ConfigurationInterface
|
|||||||
->fixXmlConfig('base_url')
|
->fixXmlConfig('base_url')
|
||||||
->children()
|
->children()
|
||||||
->scalarNode('version')
|
->scalarNode('version')
|
||||||
->defaultNull()
|
|
||||||
->beforeNormalization()
|
->beforeNormalization()
|
||||||
->ifTrue(function ($v) { return '' === $v; })
|
->ifTrue(function ($v) { return '' === $v; })
|
||||||
->then(function ($v) { return; })
|
->then(function ($v) { return; })
|
||||||
|
Reference in New Issue
Block a user