[REF] export code simplification
authoreileen <emcnaughton@wikimedia.org>
Tue, 9 Jul 2019 04:43:26 +0000 (16:43 +1200)
committereileen <emcnaughton@wikimedia.org>
Tue, 9 Jul 2019 04:49:06 +0000 (16:49 +1200)
commit12a6fdef6678def37e39e076951ce0c7336b28c7
treef1b357953c72a40ca05ff9404429996d22f2cf85
parent7a10f07be874f22f7eb8f9e3f11c0b5f90d5612f
[REF] export code simplification

On review I find the value ['exportOption'] is a parameter used at the form later to determine whether to
export 'Primary fields only' (ahem) or the user should select an array of fields. By the time the form is submitted we
either have an array of selected fields or NULL denoting that there is no selection (& hence the defaults should be used)
and the selection at the form layer is redundant. However it is still being referenced in one place (and hence the tests
are passing it in to avoid an enotice). This fixes that one place & removes it from the tests. It also extracts part of
a test to move towards making it easier to alter the signature of exportComponents without messing with so many places in
the code
CRM/Export/BAO/Export.php
tests/phpunit/CRM/Export/BAO/ExportTest.php