dev/core#2073 Fix use of legacy leaky method in tested code

[eileenmcnaughton]

Overview

Fix use of legacy leaky method in tested code

Before

$columnsResult = $db->query($columnsQuery);

After

$columnsResult = CRM_Core_DAO::executeQuery($columnsQuery);

Technical Details

This is too low volume to really leak but it uses the leaky legacy method and
has test cover in

CRM_Contact_Import_Form_MapFieldTest.testSubmit with data set basic_data
CRM_Contact_Import_Form_MapFieldTest.testSubmit with data set save_mapping

Comments

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[seamuslee001]

Test fails relate @eileenmcnaughton

[eileenmcnaughton]

Interesting - I didn't hit them locally - will look

[eileenmcnaughton]

Ah I did miss something off - should be fine now