mazedigital / association_output

Association Output for Symphony: Inline XML
MIT License
5 stars 4 forks source link

array merge was duplicating existing output params #9

Closed jonmifsud closed 9 years ago

jonmifsud commented 9 years ago

This would cause a performance issue, as it would duplicate all the output parameters, so in my case checking what dsParamPARAMOUTPUT was returning the following:

array(
    'date',
    'article',
    'article'
)

Thus running the same snippet of code twice, additionally it was also duplicating the output parameters, as seen below.

screen shot 2015-02-26 at 08 28 09

jonmifsud commented 9 years ago

@nilshoerrmann when you have some time just pull this in, just making the array merge unique to avoid duplicates.

nilshoerrmann commented 9 years ago

Thanks, Jon!