Use tempTable class in creation of dedupe table.
authoreileen <emcnaughton@wikimedia.org>
Wed, 20 Mar 2019 03:43:08 +0000 (16:43 +1300)
committereileen <emcnaughton@wikimedia.org>
Wed, 20 Mar 2019 03:49:36 +0000 (16:49 +1300)
commit887c0ec28a36bbd450b263247bac983e4e1d77ef
treec02a39a549ce002f99730f4cceb26cfd225a8ffc
parenta2540ad336faf7fb0218055a485c1a702200293f
Use tempTable class in creation of dedupe table.

The is covered by unit tests ContactTest::testDuplicateTest and DedupeFinderTest::testUnsupervisedDupes

There are some others in this file but I focussed on keeping the scope to a single table replacement.

A couple of thoughts
1) I passed the indexes in as 'columns' - works, is tested so we'd know if anything broke it but it feels a bit ug
2) I think it makes sense to add a descontruct on the class to drop the temp tables
3) the longer syntax for referring to the table is a bit wordy - but harmless I guess & reformatting would
help
CRM/Dedupe/BAO/RuleGroup.php