On 22.04.2011, at 01:31, Lukas Kahwe Smith wrote:

> 
> On 20.04.2011, at 23:16, Fabien Potencier wrote:
> 
>> On 4/19/11 11:11 PM, Jeremy Mikola wrote:
>>> Was there any problem before PR535
>>> <%20https://github.com/symfony/symfony/pull/535> was introduced and
>>> container injection for ContainerAwareInterface was added to core? If
>>> not, perhaps that can simply be removed.
>> 
>> The introduction of the ContainerAwareInterface in the DIC proved the 
>> fragility of the feature. If you decide to add an interface in your 
>> configuration, you can potentially break someone else code (in a third-party 
>> bundle for instance). And of course, you cannot fix it.
>> 
>> That's why I think the removal of the feature is the only viable solution.
> 
> 
> Ok, since it seems OpenSky, the main pro interface injection camp, seems to 
> have accepted this, I will work on a PR to remove interface injection over 
> easter.


done https://github.com/symfony/symfony/pull/629

regards,
Lukas Kahwe Smith
[email protected]



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