[NFC, test class] formatting, remove unused variables

Merge pull request #13620 from eileenmcnaughton/signuptype

[unused code cleanup] Remove unused 'signupType' url support

Merge pull request #13609 from eileenmcnaughton/payment_conf

Payment.sendconfirmation api - add further tpl variables.

Merge pull request #13534 from yashodha/dev-696

(dev/core#696) Changes to copied event phone and email reflects in or…

Merge pull request #13639 from civicrm/5.11

5.11

Merge pull request #13636 from eileenmcnaughton/strtolower

Remove another instance of 'lower'

Merge pull request #13637 from jitendrapurohit/core737

dev/core#737 - SMS not sent if 'Send Immediately' option is chosen on…

dev/core#737 - SMS not sent if 'Send Immediately' option is chosen on the last screen

Remove another instance of 'lower'

We have removed almost all of these from the code base due to the performance cost
combined with the lack of any upside. Just found another

Merge pull request #13630 from eileenmcnaughton/goodbye_mode_and_median_hello_speed

dev/core#720 Remove median & mode stats from contribution summary in order to improve performance

Merge pull request #13629 from eileenmcnaughton/mailing_job_test_use

[Test changes] Mailing job test use

Merge pull request #13543 from mlutfy/fixDefaultPDFFormat

Fix html2pdf default PDF format when multiple pdf_format are available.

Merge pull request #13631 from colemanw/entityRefs3

 EntityRef - standardize on PascalCase for entity name and fix minor bug

Merge pull request #13632 from colemanw/CRM_Report_Form_Event

[REF] Remove useless class CRM_Report_Form_Event

Remove multiple currency handling as it seems to make it format worse

Deprecate computeStats off to the one place that still uses it.

We MIGHT have reports calling it so to avoid a fatal we will not remove the function just yet

Remove median & mode stats from contribution summary in order to improve performance

Merge pull request #13579 from mattwire/contributionrecur_metadata

Contribution/ContributionRecur metadata updates for EntityForm

Remove useless class CRM_Report_Form_Event

Use PascalCase with CRM_Core_Form::addEntityRef

Although entityRef fields accept either snake_case or CamelCase, let's be consistent.

EntityRef: Fix caching filter option lists

The filters objects were getting cloned instead of passed by reference,
which prevented loadEntityRefFilterOptions() from saving the list for reuse.

EntityRef - standardize on PascalCase for entity name

Change the call used to generate recipients in unit tests

Passes in scheduled_date (as the 'real' api call would)

Part of this attempted deprecation #13627

Merge pull request #13607 from eileenmcnaughton/cont_summary_extract

dev/core#720 [REF] refactor out components of contributionSummary function

Merge pull request #13628 from colemanw/entityRefs2

Standardize format for entityRef create links

Move output formatting to the relevant functions

Remove duplicate setting of variables

Merge pull request #13624 from eileenmcnaughton/minor_pdf_utl_cleanup

[Code cleanup] Remove unused $stationery_path parameter

Merge pull request #13626 from eileenmcnaughton/cont_summary_extract1

[REF] extract cancelled stats to own function

Standardize format for entityRef create links

Merge pull request #13625 from colemanw/entityRefs

[REF] Move entityRef filters into their respective BAOs

[REF] extract cancelled stats to own function

Remove unused  parameter

Move entityRef filters into their respective BAOs

Merge pull request #13622 from eileenmcnaughton/cont_summary_extract1

[REF] extract basic soft credit stats to separate function

Merge pull request #13621 from eileenmcnaughton/default_wrong_declared

[code cleanup] Default wrong declared

Merge pull request #13619 from eileenmcnaughton/unused_pending_param

[REF] Remove unused function parameter

Merge pull request #13623 from eileenmcnaughton/comment

Remove long block of commented out code from 4 years ago

Merge pull request #13601 from alexymik/patch-1

Always load recaptcha JS over HTTPS

Merge pull request #13499 from mattwire/profilecontact_nocontactid_subtype

If a profile is used to create a contact with a subtype the contact will not have any existing subtypes

Remove long block of commented out code from 4 years ago

[REF] extract basic soft credit stats to separate function

As previously commented this array is never used.

I did a codebase grep on ->defaults & am pretty confident here....

Merge pull request #13618 from eileenmcnaughton/test_info

[Test support] Add extra output info when getsingle fails as this seems to be common  in intermittant fails

Merge pull request #13616 from eileenmcnaughton/cont_summary_extract1

[REF] extract add median to stats

Remove unused 'signupType' url support

The use of this was removed in https://github.com/civicrm/civicrm-core/pull/911
on the basis it didn't work.

This is just cleanup not done then.

(Note we shouldn't comment stuff out we should remove it as a rule)

Remove unused function parameter

The function getContributionParams was created as part of a larger refactoring/
code cleanup effort. One of those goals was to get to a point
where contributions are always created with a pending status and are later
confirmed when a payment is added.

It if only called from one place and TRUE is always passed for
so we no longer need the  parameter in this function, remove it

Add extra output info when getsingle fails as this seems to be common in intermittant fails

Merge pull request #13617 from eileenmcnaughton/no_dead_service_tests

Remove tests that no longer work due to dead service

[REF] extract add median to stats

Remove tests that no longer work due to dead service

Merge pull request #13606 from eileenmcnaughton/profile_perf

Fix (sometimes serious) performance problem on submitting profiles for specified contacts

Merge pull request #13614 from eileenmcnaughton/cont_summary_extract1

[REF] extract calculation of mode stat

Merge pull request #13615 from civicrm/5.11

5.11 to master

Merge pull request #13613 from colemanw/openCampaignWidget

Auto-open campaign widget and sort by start date

Merge pull request #13373 from MegaphoneJon/core-580-2

fixes core#580 - view all groups when appropriately permissioned

Merge pull request #13612 from colemanw/l10n.js

Move l10n.js to coreResourcesList

[REF] extract calculation of mode stat

Auto-open campaign widget and sort by start date

Move l10n.js to coreResourcesList

This change allows core resources to be added to the list AFTER l10n variables
have been initialized. This is needed by crm.menubar.js.

Another upshot of this change is that extensions will be able to add items to the list
as fully-formed urls.

Merge pull request #13425 from mfb/utf8mb4

Add install and runtime status warnings if MySQL utf8mb4 is not supported

Merge pull request #13608 from eileenmcnaughton/cont_summary_extract1

[REF] extract calculation of basic stats

Merge pull request #13587 from eileenmcnaughton/acl_location

[REF] Move addSelectWhere-like function to be located on BAO_Contribution

Merge pull request #13563 from eileenmcnaughton/fn_fix_payment_by_type

[REF] Fix silly function to do less handling of non-existent scenarios

Merge pull request #13611 from seamuslee001/exception_api

Add in Exception API to support the refactor of Dedupe Exception Page

Add in Exception API to support the refactor of Dedupe Exception Page

Add location & relevant tests to payment.sendconfrimation

Payment.sendconfirmation api - add further tpl variables.

This is a continuance of the introduction of this api

Merge pull request #13591 from mfb/683-incorrectly-encoded-state-and-country-names

fixes dev/core#683 Incorrectly encoded state and country names

[REF] extract calculation of basic stats

Merge pull request #13605 from eileenmcnaughton/cont_summary

dev/core#720 add unit test, remove legacy code style.

Fix (sometimes serious) performance problem on submitting profiles for specified contacts.

->_id is set when the contact id is known - e.g in edit, in contactlayouteditor profile
blocks. However, current logic still retrieves duplicates and then... does nothing
with them if ->_id is set. By moving the if up higher we can save the server from
doing unnecesary queries. Note that if you submit a tag profile with no other fields in it
the query is server-destroyingly bad with this

Merge pull request #13600 from eileenmcnaughton/profile_perf

[REF] extract chunk of code to a separate function

dev/core#720 add unit test, remove legacy code style.

This PR adds tests to the summaryQuery code in preparation for changes.

The one non-test change is to remove the & from summaryContribution fn.

It is only called from one place in the code & that place does not return as reference.

```
  /**
   * @return mixed
   */
  public function getSummary() {
    return $this->_query->summaryContribution($this->_context);
  }
```

Merge pull request #13432 from francescbassas/patch-16

dev/core#657 - Add filter for country on Repeat Contributions Report

Move addSelectWhere-like function to be located on BAO_Contribution using correct name.

Since the contribution api uses the BAO_Query object I think it will not call this, nor reports AFAIK
However, this should cause the v4 api to apply these.

[REF] extract chunk of code to a separate function

Merge pull request #13602 from civicrm/5.11

5.11 to master

Merge pull request #13599 from seamuslee001/5.11

Add in release notes for 5.10.2

Update 5.10.2.md

Update 5.10.2.md

Minor copy-edits:
* Punctuation to match other point-release docs
* Use consistent reverse-alpha in credits
* Add some more authors+reviewers from Github

Always load recaptcha JS over HTTPS

Merge pull request #13593 from eileenmcnaughton/acl_regression

dev/core#721 Fix regression on refreshing group_contact cache before checking acls

Merge pull request #13592 from yashodha/dev-705

(dev/core#705) Disabling Alphabetical Pager is not respected for cont…

Add in release notes for 5.10.2

Update release notes as per Eileen's comment

Add in notes on merge of dev/core#715 fix

Fix regression on refreshing group_contact cache before checking acls

We seem to be checking if the cache needs a rebuild and IF SO doing 2 actions
1) rebuilding it
2) permitting access to contacts related to it

When really it should do only 1) above - this patch changes as such

This seems to be whackamole off https://github.com/civicrm/civicrm-core/pull/12344
going too far one way & https://github.com/civicrm/civicrm-core/pull/13448
fixing but adding this new variant

Merge pull request #13561 from eileenmcnaughton/payment_conf

Add new Payment.sendconfirmation api

Update Repeat.php
Add basic contact fields and filters on Repeat Contributions Report

Merge pull request #13517 from alifrumin/contribsearchsoftcredit

Find Contributions columns/headers not aligned when "Contributions OR Soft Credits?" filter set to "Soft Credits Only"

(dev/core#705) Disabling Alphabetical Pager is not respected for contribution pages.

fixes dev/core#683 Incorrectly encoded state and country names

Add new Payment.sendconfirmation api

This adds a new api Payment.sendconfirmation intended to be an equivalent of
Contribution.sendconfirmation.

The key thing this needs to be able to do is to load up all
related information to assign to the template - this is not all covered in this
PR & until it is functionally equivalent to AdditionalPayment::sendEmail
we can't switch over but since it's new functionality we should be able to
merge & keep working on it.

Note that there is discussion on the PR this relates to (#13330) about what should
happen if 'Payment.create' api accepts 'is_email_receipt'. I think the most logical
outcome is that receipts would go out if it caused the contribution to be completed
(ie. we pass is_email_receipt into completetransaction api).

However, I don't feel like that is 'settled' yet - separating into a second api
(Payment.sendnotification) means we will have 2 generic function which can be called
from multiple places in the code rather than 2 functions tightly tied to the form layer.
I think it's OK if it is 2 not one

Merge pull request #13571 from eileenmcnaughton/book

dev/report#7 fix trxn_date on bookkeeping report

Merge pull request #13528 from jitendrapurohit/dev-684

dev/core#684 - Case Manager not updating correctly

Merge pull request #13539 from eileenmcnaughton/email_task

dev/core#469 Fix Error on action 'Email - schedule/send via CiviMail' with multiple event names filter

Merge pull request #13584 from eileenmcnaughton/user_dash_use_api

[REF} User api rather than selector for rendering contributions on user dashboard

Merge pull request #13538 from eileenmcnaughton/dedupe_birth

dev/core#397 Dedupe for Individual Birth Date Results in Error

Merge pull request #13557 from vingle/patch-5

Improves styling on Joomla! upgrade screen

Merge pull request #13573 from pradpnayak/GroupInlineEdit

dev/core/issues/714, Inline edit should be disabled if user doesn't have edit group permission

Switch userDashboard from using selector object to using the api to get contributions to display.

Note that preliminary tests for this were written & merged some time ago.

Reasons for the change are

1) readability - most devs are much more comfortable with reading the api code than the selector
2) performance - the contribute selector code is deeply unperformant - mostly due to the summary function which in
this case is somewhat mitigated by the limit of 12 but we are still doing something slow for no reason
3) test stability - the test for this turned out to have poor stability & hopefully this will help
4) preliminary cleanup - there are 2 existing PRs that attempt to add new buttons to this & moving towards
a cleaner tpl / php layer will help with those. In addition there is a serious performance issue to
address on the contribution summary function. Reducing use of that function should help with the cleanup effort