Fixed issue dev/core#2449: remove an unnecessary call to cleanMoney

[jaapjansma]

Overview

Removes an unnecessary call to CRM_Utils_Rule::cleanMoney

Before

When your locaization settings decimal separator where set to , and you had a contribution page with an amount of 12,987654321 it resulted in paying a wrong amount.

After

Through the contribution page the right amount is paid.

Technical Details

The call to CRM_Utils_Rule::cleanMoney is removed which was at a weird place. We should sanitize/validate/clean user input early.

Comments

See https://lab.civicrm.org/dev/core/-/issues/2449

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

Thanks @jaapjansma - merging this as already discussed. Per the code comment the line was wrong in terms of our expected flow but now @jaapjansma has confirmed that it does actual harm so removing is correct