On 10/28/2010 07:14 AM, Lukas Kahwe Smith wrote:
On 22.10.2010, at 11:23, Jordi Boggiano wrote:

Heya,

Following the discussion on a DoctrineUserBundle commit [1] I'd like to
propose some sort of best practice.

First have a look at the patch [2] (contains an example config) or the
complete Extension class [3].

Now what I would like to propose is that those two remapParameters*
methods I added would be added to the Symfony core Extension class, so
you could just pass it an array and parse/remap most of your config to
parameters.

This means less code to write and hopefully more standardized and
uniform config schemes.

Cheers

[1] http://github.com/knplabs/DoctrineUserBundle/commit/682c52
[2] http://github.com/Seldaek/DoctrineUserBundle/commit/074df2
[3]
http://github.com/Seldaek/DoctrineUserBundle/blob/config_change/DependencyInjection/DoctrineUserExtension.php


IMHO the remap*() methods should be merged into 
Symfony\Component\DependencyInjection\Extension\Extension
+1

--
If you want to report a vulnerability issue on symfony, please send it to 
security at symfony-project.com

You received this message because you are subscribed to the Google
Groups "symfony developers" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/symfony-devs?hl=en

Reply via email to