added some notes about Bundle and Extension naming #190
Conversation
…s if the Bundle provides an Extension class
cookbook/bundles/extension.rst Outdated
There was a problem hiding this comment.
getNamespace is required in order to support an XML confg file, even without XSD
There was a problem hiding this comment.
Could you elaborate here? Are you talking about app or Bundle xml configs?
There was a problem hiding this comment.
I'm talking about app config. A bundle config file will never use the semantic config namespace but only the core namespace.
There was a problem hiding this comment.
ok .. fixed that .. and submitted a PR to add a default getNamespace() implementation:
symfony/symfony#425
| One thing to note: I am not sure the extension stuff should be in the cookbook. To me its a pretty fundamental part of Symfony2. Aside from this, if it doest stay in the cookbook, then my note about not BC guarantee's when overriding parameters/services handled by Extensions should get a prominent place in the main docs some where. |
4 participants
Mainly all the stuff mentioned in http://trac.symfony-project.org/wiki/IRCLogs20110324 plus some other minor fixes.