Fix "crmURL" parameters in various templates

[christianwach]

Overview

As described on CiviCRM Lab, in Smarty templates, when the q parameter is prefixed with an ampersand, for example:

{crmButton p='civicrm/contact/view/delete' q="&reset=1&delete=1&cid=$contactId" class="delete" icon="trash"}

the resulting URL contains && like this:

http://civicrm.local/wp-admin/admin.php?page=CiviCRM&q=civicrm/contact/view/delete&&reset=1&delete=1&cid=202

Before

URLs have double ampersands (&&) e.g.

http://civicrm.local/wp-admin/admin.php?page=CiviCRM&q=civicrm/contact/view/delete&&reset=1&delete=1&cid=202

After

URLs do not have double ampersands, e.g.

http://civicrm.local/wp-admin/admin.php?page=CiviCRM&q=civicrm/contact/view/delete&reset=1&delete=1&cid=202

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

[christianwach]

Related PR: #11708

[totten]

Looks good. This makes a ton of sense from r-code perspective.

For r-run, I spot-checked the changes in Contact/Summary.tpl and paymentBlock.tpl on both (a) the patched autobuild site and (b) an unpatched local site. The Summary.tpl button still works. In both cases, the expected cleanup appeared in the URL.

Tangentially, in paymentBlock.tpl on D7 with demo data (view source for http://core-12968-6mrg.test-1.civicrm.org:8001/civicrm/contribute/transact?reset=1&id=1), there was another extra & (...currency=USD&&is_back_office=...), but that's a different problem. This PR still looks like a clear cleanup.