Fix deprecation handling [intra-rc-follow up] on #13999]

[eileenmcnaughton]

Overview

The signature for CRM_Contribute_BAO_ContributionRecur::cancelRecurContribution was altered in 5.13 ( 9cfc631) to expect an array. Out of concern for sites calling the BAO directly from extensions we added deprecation handle to make it succeed naughtily

Turns out we were one of the sites naughtily using the BAO directly who needed this handling
to work - but because we passed a number in quote it didn't - this fixes

Before

Deprecation handling works for 5 but not '5'

After

Deprecation handling works for 5 and '5'

Technical Details

Comments

We don't support calling the BAO directly but somethings we kinda try to be helpful anyway

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

[seamuslee001]

change makes sense adding merge on pass