Remove last usage of _fields from Import_Parser

[eileenmcnaughton]

Overview

Remove last usage of _fields

Before

In just this one place we still use the old metadata

After

Functionality is unchanged - location field loads for these fields

and the function returns the same array - which is a collection of the fields with hasLocation = true

Technical Details

Over the course of the code tidy up we stopped using the complicated _fields construct which entails a whole lot of Field objects & switched to using the metadata array - however we are still building if for this one place

Comments

In fact the motivation for clearing this up is that there are some incorrect pseudoconstant refs in the contribution code & I was trying to purge incorrect refs to contributionStatus - however, the code in question can be removed rather than fixed & this change plus a couple of other removal PRs will allow that semi-sub-system to be ripped out so we can stop maintaining it

[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]

@eileenmcnaughton can you rebase

[eileenmcnaughton]

@seamuslee001 done