Prepend Child Bundle paths before the parent #9112
Conversation
|
Imo the application level Resources should be loaded first. |
|
You are right. I will update shortly. |
There was a problem hiding this comment.
The getParent() method is always there since it's a part of Symfony\Component\HttpKernel\Bundle\BundleInterface. There's no need to verify its existence.
|
And finally, would be good to cover this with tests. |
…ding. app Resources should always override child bundle resources.
|
@jakzal app Resources will now take priority. I had to sort it by Child bundles (and removed prepending) to ensure that the priority is correct. |
|
@jakzal I am not too sure how to write tests for this use case as there will be no parent/child bundles to access from the tests? |
There was a problem hiding this comment.
can't see the need of $parentBundlehere
…o parent bundle namespace for consitency
This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes #9112). Discussion ---------- Prepend Child Bundle paths before the parent This fixes #9085 Say you have AcmeDemoBundle and AppDemoBundle. AcmeDemoBundle is the parent of AppDemoBundle. If you load templates using @AcmeDemoBundle/ControllerDir/template.html.twig it means that you cannot override the template in AppDemoBundle. The patch below prepends the AppDemoBundle Resources directory to the AcmeDemo namespace. The namespace directories would not result in: ``` [AcmeDemo] => Array( [0] => [absolute-dir-here]/src/App/DemoBundle/Resources/views [1] => [absolute-dir-here]/app/Resources/AcmeDemoBundle/views [2] => [absolute-dir-here]/src/Acme/DemoBundle/Resources/views ) ``` Commits ------- 19fad88 Prepend Child Bundle paths before the parent
|
@fabpot I suggest reverting this for now as it is not BC. It forbids using constructor arguments in bundle classes |
|
👍 for reverting. I'm not sure if that patch works correctly anyway (no test case was added). |
|
can't |
|
reverted |
* 2.3: Revert "bug #9112 Prepend Child Bundle paths before the parent (trsteel88)"
* 2.4: Revert "bug #9112 Prepend Child Bundle paths before the parent (trsteel88)"
6 participants
This fixes #9085
Say you have AcmeDemoBundle and AppDemoBundle. AcmeDemoBundle is the parent of AppDemoBundle.
If you load templates using @AcmeDemoBundle/ControllerDir/template.html.twig it means that you cannot override the template in AppDemoBundle. The patch below prepends the AppDemoBundle Resources directory to the AcmeDemo namespace.
The namespace directories would not result in: