Strtolower fix 'name'

[eileenmcnaughton]

Overview

Remove a further instance of performance-hurting, bug creating lower case comparison - in this case in a piece of code that is not hit by tests & which I could not hit using the UI. I added a deprecation notice to surface anywhere it is actually used

Before

strtolower + LOWER used

After

mysql handling used.

Technical Details

Per #12494

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

[eileenmcnaughton]

test this please

[eileenmcnaughton]

test this please

[seamuslee001]

This is keeping in with our understanding that removing both LOWER and the strtolower improves performance and doesn't cause regressions. Whilst this is not covered by the test coverage i think its safe to merge as we have done this in other areas with no dramas and also there is the warning that is given off if there is an issue