Merge pull request #11335 from civicrm/4.7.28-rc

Merge from rc

CRM-21436: Fix exception on pay later contribution page. (#11317)

* CRM-21436: Fix exception on pay later contribution page.

* minor fix

* move test against master

* variable fix

Merge pull request #11333 from samuelsov/CRM-21470b

CRM-21470 Ensure we don't have an empty locale

Merge pull request #11329 from eileenmcnaughton/currency

Currency

Merge pull request #11022 from eileenmcnaughton/components

CRM-21218 Improve component statics flushing

Merge pull request #10740 from eileenmcnaughton/def_country

CRM-19888 fix import to prefer default country when resolving states.

Merge pull request #11332 from JMAConsulting/CRM-21486

CRM-21486: Support multiple test mail

CRM-21470 Ensure we don't have an empty locale #2

CRM-21470 Ensure we don't have an empty locale

Merge pull request #11301 from jitendrapurohit/CRM-21456

CRM-21456 : Relationship end date not set when creating resolved cases

CRM-21486: Support multiple test mail

Merge pull request #11330 from eileenmcnaughton/comm

NFC, comments only

NFC, comments only

Merge pull request #11319 from JKingsnorth/CRM-21476

CRM-21476 Rename clicks to unique clicks in mailing summary report

CRM-21365 Ensure currency param passed via api.Contribution.transact is passed to processor

Change-Id: I59a5cd16499e89fbaa54594071fd396de81dfb82

CRM-21482 improve handling for currency on form

Change-Id: I63f3f5445b39b9bd6068835ae61c3e488177ce75

Merge pull request #11328 from colemanw/CRM-21483

CRM-21483 - Move anguar module crmRouteBinder to core

CRM-21483 - Move anguar module crmRouteBinder to core

Merge branch '4.7.28-rc' into master

Merge pull request #11322 from mlutfy/4.7.28-rc-crm21481

CRM-21481: 4.7.28: call rebuildMultilingalSchema().

Merge pull request #11324 from mattwire/CRM-21214_followup_fix_crashes

CRM-21214 Fix crashes when 'null' string is passed in via API

Fix crashes when 'null' string is passed in via API

Merge pull request #11314 from omarabuhussein/CRM-21473-adding-add-note-permission

CRM-21473: Add new permission to add contact notes and prevent users without edit contact permission to edit or delete notes

Merge pull request #11326 from civicrm/4.7.28-rc

4.7.28 rc

Merge pull request #11320 from jitendrapurohit/CRM-21446

CRM-21446 - Allow case id as well as hash in inbound email processing…

Merge pull request #11321 from mlutfy/4.7.28-rc-crm21479

CRM-21479: Syntax fix for updateContributionInvoiceNumber().

CRM-21481: 4.7.28: call rebuildMultilingalSchema().

CRM-21479: Syntax fix for updateContributionInvoiceNumber().

Merge pull request #11312 from samuelsov/CRM-21470

CRM-21470 Add support for WordPress Polylang plugin

CRM-21446 - Allow case id as well as hash in inbound email processing to autofile emails on cases

Merge pull request #11300 from eileenmcnaughton/towards1

CRM-19273 extract code to get financial items and filter those already cancelled.

CRM-21476 Rename clicks to unique clicks in mailing summary report

Merge pull request #11303 from jitendrapurohit/CRM-21190

CRM-21190 - Warning message on Thankyou page on pay later membership pr…

CRM-21190 - Warning error on Thankyou page on pay later membership priceset

Merge pull request #11315 from seamuslee001/dao_participant_update

NFC rerun setup.sh -dg and regen Participant DAO

Merge pull request #11257 from mattwire/CRM-21409_contribute_pdf_letter_hooks

CRM-21409 Don't bypass hooks when updating contribution receipt/thankyou date

NFC rerun setup.sh -dg and regen Participant DAO

CRM-21473: Add new permission to add contact notes and prevent users without edit contact permission to edit or delete notes

CRM-21470 Add support for WordPress Polylang plugin

Don't bypass hooks when updating contribution receipt/thankyou date

Merge pull request #11235 from clnlf/CRM-21389-UserDashBoard-Regions

CRM-21389 Added pre and post regions to each entitys userdashboard

Merge pull request #11226 from seamuslee001/CRM-21378

CRM-21378 Ensure that email abuse reports are treated as bounce type …

Merge pull request #11216 from mydropwizard/crm-21372

CRM-21372: Fix fatal error adding profile on Drupal 8

Merge pull request #11285 from eileenmcnaughton/dupe_url

[NFC] CRM-16819 improve on CRM_Utils_Request::retrieve

CRM-21378 Ensure that email abuse reports are treated as bounce type of spam

Move upgrade steps to .29

Merge pull request #11267 from nbrettell/CRM-21383-master

CRM-21383 - Inclusion of a directive to replace the existing select f…

Merge pull request #11308 from civicrm/4.7.28-rc

4.7.28 rc

Merge pull request #11307 from spalmstr/CRM-21001

CRM-21001

Merge pull request #11258 from seamuslee001/CRM-21411

CRM-21411 Fix issue where unsubscribe group field would not show beca…

Merge pull request #10650 from jitendrapurohit/CRM-20861

CRM-20861 : Location type does not show for custom address fields

CRM-21001

CRM-21456 : Relationship end date not set when creating resolved cases

CRM-19273 fix tests to actually run.

Due to being mis-named these tests have never run. When they do run they expose issues which
are addressed in PR 10962. This commits causes the part that actually works in these
tests to run.

Minor code tidy up

CRM-19273 very minor extraction

CRM-19273 extract code to get financial items and filter those already cancelled.

This is the first of the broken-out changes in https://github.com/civicrm/civicrm-core/pull/10962 that results in
an actual functionality change. The code that retrieves the relevant financial items is separated into
it's own function. But then a php loop iterates through and eliminates any items that have
already been reversed. This is a scenario identified in the unit tests on 10962 and occurs when
the the option value has is changed and then changed again - the items from the first change
should not be re-changed.

CRM-19888 update test to reflect change.

In fact this test is a bit silly anyway as it tests the fn in a way it is never called in

CRM-19888 fix import to prefer default country when resolving states.

The example given is the abbreviation UT is used in muliple countries and if US is the default country
it should resolve to UTAH

Towards CRM-19888 extract code that resolves defaults

Merge pull request #11299 from francescbassas/patch-12

CRM-21454 Add money raised on page contribution widget ajax response

Merge pull request #11302 from totten/master-toxic-json

(NFC) .toxic.json - Update list of functions which yield toxic warnings

Add comments

(NFC) .toxic.json - Update list of functions which yield toxic warnings

The file `.toxic.json` is supposed to list egregious functions -- any change
to these functions should produce extra comments/suggestsions.

The current list is a few years old, so I've run `pdepend` and updated the list.
The list includes anything which matches one or more of these:

 * "Non Comment Lines of Code" (`ncloc`) greater than 300.
 * "[NPath](https://modess.io/npath-complexity-cyclomatic-complexity-explained/) complexity" (`npath`) greater than 1 billion.

CRM-21454 Add money raised on page contribution widget ajax response

Merge pull request #11289 from kainukempowerment/CRM-21424

CRM-21424 Set receipt date when downloading pdf receipt

Added test

Merge pull request #11296 from jmcclelland/CRM-21449

CRM-21449 - add location field to activity report.

Merge pull request #11284 from eileenmcnaughton/towards1

[NFC] code cleanup: split out 2 functions that are mostly unrelated.

Merge pull request #11268 from mattwire/CRM-21421_allow_updating_existing_casecontact

CRM-21421 Fix to allow updating an existing CaseContact record

Merge pull request #11295 from mepps/CRM-21448

CRM-21448 Add a link to contact from contribution

Merge pull request #11293 from mlutfy/crm21447

CRM-21447: Relationship Report: move variable handling to beginPostProcessCommon

Merge pull request #11297 from civicrm/4.7.28-rc

4.7.28 rc

Merge pull request #11286 from jitendrapurohit/pay-later-fix

CRM-21436 - Fix fatal error on pay later

CRM-21449 - add location field to activity report.

CRM-21448 Add a link to contact from contribution

CRM-21447: Contact Relationship Report: move variable handling to beginPostProcessCommon().

remove obsolete comment

Merge pull request #11261 from JMAConsulting/CRM-21362

CRM-21362: followup PR which fixes duplicate mailing rows for each job

fix contribution page loading on no processor

Add unit test for CaseContact.create

Merge pull request #11152 from eileenmcnaughton/api

CRM-21324 - Support 'null' on date fields in the api

[NFC] code cleanup: split out 2 functions that are mostly unrelated.

The handling of line items & financial items has been munged together in a way that gains little, adds confusion,
and blocks the financial items portion from being re-usable.

This is a code cleanup preparatory to the actual changes in pr #10962

Merge pull request #11259 from eileenmcnaughton/report

CRM-21412 Do not give fatal error on report when no fields selected

CRM-21424 Set receipt date when downloading pdf receipt

CRM-16819 improve on CRM_Utils_Request::retrieve

I'm proposing this as an improved (but not perfect) way to retrieve data from http requests.

The biggest issue IMHO with the CRM_Utils_Request::retrieve method is that the third
parameter is & - and if any of the subsequent parameters need to be passed a NULL
array must be generated. The practice of passing a specific static NULL array is known to
have caused errors due to contamination. This commit offers an alternate function without that
parameter (my suggestion is to create a new function again where we really do want  to
work but in general it seems flakey to me). I have also re-ordered the parameters as I perceive
 as being more commonly required than  (renamed from 'abort').

I have also made it so the new function will throw an exception rather than use the deprecated
fatal - as a new function I feel we can enforce this change and I have made some changes
to the error message thrown in the validate function so it returns a list of permissable
values for the  string as the existing error is hard for developers to work with, as it
can be hard to realise why 'int' is wrong (casing) or what should have been passed.

Finally I made the visibility on CRM_Utils_Retrieve::getValues protected. This function
is not called from within core & is downright unsafe as it does no validation so we should
not permit anyone to accidentally use it (I have no reason to think they have

Merge pull request #11270 from colemanw/CRM-21421

CRM-21421 - Api case update should not re-save case contacts

Minor refactor, define  as an array for readability & insertability

CRM-21412 Do not give fatal error on report when no fields selected

Merge pull request #10674 from herbdool/crm20764

CRM-20764 Allow clean URLs with CiviCRM on Backdrop

Merge pull request #11283 from mattwire/CRM-21432_recur_end_date_display

CRM-21432 Fix display of end date for recurring contributions

Merge pull request #11278 from eileenmcnaughton/towards1

[NFC] Function extraction and renaming of variable for shorter refere…

Fix display of end date for recurring contributions

[NFC] Function extraction and renaming of variable for shorter reference.

This is a function extraction towards CRM-19273. I haven't changed the logic but it is apparent that
we are changing the contribution financial type to 'the financial type of the last line
item' - which is a little dubious although it might work in practice. We should expose the field &
let people choose it

Merge pull request #9739 from nditech/CRM-19915

Fixes CRM-19915

Merge pull request #11034 from ejegg/docStrings

[NFC] Fix a bunch of docstrings

Merge pull request #11138 from mattwire/CRM-21312_bootstrap_css_fix

CRM-21312 Fix display of 'Recent Items' when using a bootstrap theme

Update sendMail return type

Merge pull request #11233 from seanmadsen/CRM-14343-import-error-limit

CRM-14343 - Remove limit of downloading only 250 errors during CSV import

Merge pull request #11255 from tschuettler/CRM-21407

CRM-21407 - Fix ISO compliance for German counties

Merge pull request #11266 from eileenmcnaughton/additional-test

[NFC] Additional test