ParameterHandler icon indicating copy to clipboard operation
ParameterHandler copied to clipboard

Add possibility of merge missing nested array values under the top-level key

Open Pym opened this issue 11 years ago • 5 comments

Original pull request https://github.com/Incenteev/ParameterHandler/pull/36 by @zeliard91

Bug fix: no Feature addition: yes Backwards compatibility break: no Tests pass: yes

With this, we can now have new multi dimensional arrays in the dist file under the top-level key parameter be merged in the generated parameter file.

The prompt is still waiting for a full YAML definition of the missing key.

Pym avatar Jan 16 '14 18:01 Pym

:+1: please merge this

luxifer avatar Jan 18 '14 21:01 luxifer

Any chances to have the PR merged?

luxifer avatar Mar 13 '14 14:03 luxifer

I will dedicated some time this weekend to review it

stof avatar Mar 13 '14 14:03 stof

this feature would be nice to have

elvisciotti avatar Feb 10 '15 12:02 elvisciotti

I think this breaks things for cases where you have a list in an argument (for instance a list of trusted proxies, even though that's the bad example for Symfony projects as trusted proxies are needed earlier than the loading of parameters)

stof avatar Dec 13 '19 14:12 stof