[REF] Deprecate passing a blank currecny to CRM_Utils_Money::format a…

[seamuslee001]

…nd also update function call to remove at least one usage of money_format

Overview

This updates the call for when CRM_Utils_Money::format is passed with asking for just the number to use an appropriate internal money formatting function instead of the deprecated money_format function. It also deprecates passing in ' ' a blank string as a param for $currency.

Before

$currency = ' ' not deprecated and money_format used in at least 1 place

After

$currecny = ' ' deprecated and money_format only used in one place

ping @eileenmcnaughton @mattwire

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

This seems right to me - I'm going to leave overnight in case @mattwire sees any issues but it feels like a good time to merge as the rc has just been cut

[seamuslee001]

Its been a few days and matt hasn't responded so am going to merge this

[eileenmcnaughton]

yep - I think it had it's merge-ready spell