Ah ok, i foreach over the $configs and apply them iteratively to a new
$mergedConfig. This means i have to call array_reverse for my algorithm to
work correctly. I will extend the pull request
https://github.com/fabpot/symfony/pull/477#issuecomment-701862 accordingly.

On Mon, 24 Jan 2011 07:34:15 +0100, Fabien Potencier
<[email protected]> wrote:
> On 1/23/11 11:26 PM, Benjamin Eberlei wrote:
>> Hello,
>>
>> I was always under the assumption that the order of the configurations
is
>> from config.yml to config_dev.yml/config_prod.yml.
>>
>> However now while testing the new merge method of DoctrineBundle i
>> realized, config_dev.yml is passed in first, then config.yml.
>>
>> Is this the desired result? It would mean that i have to call
>> array_reverse on $configs to get a meaningful override mechanism to
work.
> 
> I have just tested and the order is right.
> 
> You first get the imported file configuration, then the main 
> configuration file.
> 
> So, if you have a config_dev.xml file which includes config.xml, you 
> will have the configuration in this order: config.xml, then
config_dev.xml.
> 
> Here is the output of print_r() on the configs passed to ormLoad() for 
> the example you mention in the pull request:
> 
> Array
> (
>      [0] => Array
>          (
>              [auto_generate_proxy_classes] =>
>              [metadata_cache_driver] => apc
>          )
> 
>      [1] => Array
>          (
>              [auto_generate_proxy_classes] => 1
>          )
> 
> )
> 
> Which looks like what you expect.
> 
> Fabien
> 
>> greetings,
>> Benjamin
>>

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