[REF] clean up merge array.
authoreileen <emcnaughton@wikimedia.org>
Wed, 27 Nov 2019 06:56:55 +0000 (19:56 +1300)
committereileen <emcnaughton@wikimedia.org>
Wed, 27 Nov 2019 08:27:30 +0000 (21:27 +1300)
commit0b743fb1745128866ff2d9e8815f2a1106656ef1
tree64963091b8b3cfcb34fc0efa6eb237196872f2da
parent00e7a58a8f507a2454102f765c13d203b8a7a68b
[REF] clean up merge array.

This code has been doing my head in because it is just like 'construct complex array & then another
complex array & iterate them together in nasty ways.

However, after another round of misery I'm pretty sure that this cleanup works. Basically the whole removed
section of code amounts to 'the array built in the first pass takes precedence over the one in the
second pass'. By using a property & only setting the array values when not-yet-set we can
allow the second pass to not clobber the first while populating it rather than as follow up wrangling
CRM/Export/BAO/ExportProcessor.php