[REF] Move clone template functionality from api to bao

[colemanw]

Overview

Moves code block from one place to another, for better reusability in api4.

Before

Code in api.

After

Code in bao.

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

[mlutfy]

Test fail seems unrelated?

CRM_Member_BAO_MembershipTest::testRenewMembership
Verify correct end date is calculated after membership renewal
Failed asserting that two strings are equal.

[mlutfy]

jenkins, test this please

[colemanw]

@mlutfy that looks to be a recent regression in core, I'm seeing it on a number of PRs. So other than that, this PR is passing tests.

[colemanw]

retest this please

[colemanw]

@mlutfy was that a "merge on pass" endorsement?

[mlutfy]

I don't always feel like the most qualified for this kind of PR, but considering you are the one who initially added this code in the API, and it makes sense to have it in the BAO instead, merge on pass! :)

[colemanw]

retest this please

[eileenmcnaughton]

test this please