[REF] move formatting of conflict into markConflict function
authoreileen <emcnaughton@wikimedia.org>
Mon, 1 Jul 2019 01:01:19 +0000 (13:01 +1200)
committereileen <emcnaughton@wikimedia.org>
Mon, 1 Jul 2019 22:09:04 +0000 (10:09 +1200)
commit5214f03bbc28ffb57c683660637ebfea5a0ea7e4
treec9a097ee34909c79f69e6d4a98a7a5b57f484769
parentfb820b0923ef7924689433a54468048f9c3fae87
[REF] move formatting of conflict into markConflict function

The goal here is to make this information more usable to an api caller. Currently the 'conflict' key holds text formatted
for a specific form. I was going to change this to hold an array of data that an api could use. However, the
Contact.get_merge_conflicts already has a specific api-friendly format that is keyed by 'mode'
at the top level so my revised plan is to return api-friendly data as well (without messign with the
existing form's data).

This change simply moves the decisions about what data to store & how down to the function whose responsibility
it is (markConflict) and adds a test to ensure that no change results

[REF] move formatting of conflict into markConflict function

The goal here is to make this information more usable to an api caller. Currently the 'conflict' key holds text formatted
for a specific form. I was going to change this to hold an array of data that an api could use. However, the
Contact.get_merge_conflicts already has a specific api-friendly format that is keyed by 'mode'
at the top level so my revised plan is to return api-friendly data as well (without messign with the
existing form's data).

This change simply moves the decisions about what data to store & how down to the function whose responsibility
it is (markConflict) and adds a test to ensure that no change results
CRM/Core/BAO/PrevNextCache.php
CRM/Dedupe/Merger.php
Civi/Test/Api3TestTrait.php
tests/phpunit/api/v3/JobTest.php