[Ref] [Import] Remove another good intention....mapField

[eileenmcnaughton]

Overview

The mapField() function in the import parser classes looks like a good intention - in every instance it does nothing except

1. makes the code hard to read
2. returns the same constant.

It currently has to be implemented due to the abstract function - but does nothing helpful.

Before

blah blah blah

After

poof

Technical Details

I have removed the pattern from csvimporter just now - but not the civiHR classes that extend this abstract class as it doesn't matter if they DO continue to implement the function - it's just fugly

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).

[colemanw]

Code simplification makes sense.