Remove invalid use of crmMoney formatter

[mattwire]

Overview

crmMoney smarty function is being used to "format" an html element instead of a monetary value.

Before

HTML element passed to crmMoney.

After

Not passed to crmMoney.

Technical Details

Comments

Identified while working on brick/money where this invalid use causes CRM_Utils_Money::format to throw an exception.

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

[demeritcowboy]

Looks good. Visited contribution page amounts tab and is same before and after. Something similar came up before.

There's even a line in Money::format() that says todo deprecate handling for the html tags because .... WTF.

Test fail is that unrelated stubborn one.

[eileenmcnaughton]

Merging based on @demeritcowboy's review