Devs,

After checking Fabien's commit (r11644) on plugin activation in
symfony 1.2, I have a few remarks:

 - The compat 10 plugin should enable all plugins by default -
otherwise, the compatibility is broken

 - If I have to enable/disable 10 plugins, I'd like the
`ProjectConfiguration::enablePlugin()` to accept more than one
argument, so that I can write

    $this->enablePlugin('sfDoctrinePlugin', 'sfDoctrineGuardPlugin',
'sfDoctrineSimpleCMSPlugin');

instead of

    $this->enablePlugin('sfDoctrinePlugin');
    $this->enablePlugin('sfDoctrineGuardPlugin');
    $this->enablePlugin('sfDoctrineSimpleCMSPlugin');

Because of that, you might want to add an 's' to the two method names.

 - Shouldn't the disabled plugin classes be removed from the autoload?

 - I though that `create_function()` was banned due to memory leaks.
Is it an acceptable practice now?

François

--~--~---------~--~----~------------~-------~--~----~
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