Duplicate function from deprecated utils onto calling classes & remove

[eileenmcnaughton]

Overview

Duplicate function from deprecated utils onto calling classes & remove

Before

Less more

After

OMG more nasty code - but it is easier to deal with

Technical Details

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]

Yikes 😱 it feels really uncomfortable to merge this and turn that into a two-headed beast. Once divided, do you have plans to conquer?
I think those functions need a @deprecated annotation at least.

[eileenmcnaughton]

@colemanw I can easily remove almost all of it from the mail format place - the other class is already overrun with functions called deprecated this & deprecated that so it's pretty implicit! Most of it isn't required for either place (althtough I'm a bit wary of adding in more cleanup PRs until I get my existing ones merged I'd do the forward one straight away)

[eileenmcnaughton]

#22594