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.

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

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

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

Merge pull request #11277 from mihaelmladenov/NDI-14-cancel-event-participants

CRM-21426: Removed duplicate submit buttons on Cancel/Delete Event Participants

NDI-14: Removed top submit buttons on Delete Event Participants task

NDI-14: Changed label of cancel event participants submit button

Merge pull request #11273 from aydun/CRM-21422_london_timezone

CRM-21422 - Fix handling of 0 timezone offset

Code improvement as per Eileen's comment

NDI-14: Removed duplicate submit buttons on Cancel Event Participants page

Merge pull request #11272 from eileenmcnaughton/towards1

NFC code tidy up preparatory to CRM-19273

Merge pull request #11275 from civicrm/4.7.28-rc

4.7.28 rc

CRM-21304 quash form reload message when refreshing (#11122)

* CRM-21304 quash form reload message when refreshing

* CRM-21304 - Disable unsaved changes warning on dedupe form

----------------------------------------
* CRM-21304: Dedupe usablity, quash refresh message in more cases
  https://issues.civicrm.org/jira/browse/CRM-21304

Merge pull request #11072 from cividesk/CRM-21265-4.7

CRM-21265: update payment status to Failed if membership transaction…

Merge pull request #11205 from agileware/CRM-20166

CRM-20166: Making CVV always required for front-end pages.

Fix fatal error on pay later

Merge pull request #11264 from seanmadsen/CRM-21419-activity-separation

CRM-21419 Prevent users from accidentally creating a single activity when they want multiple

Merge pull request #11269 from varshith/CRM-21041_respect_default_communication_style_on_api_contact_create

CRM-21041: Respect default 'Communication Style' when creating a …

clear weird character causing warnings

Merge pull request #11254 from colemanw/CRM-21406

CRM-21406 - Standalone export form

Fix handling of 0 offset

CRM-20166: Making CVV always required for front-end contribution pages.

Adding test cases for CRM-21041

Fix syntax check and remove irrelevant Ajax params

NFC code tidy up, convert result to array more cleanly

NFC code tidy up preparatory to CRM-19273

As part of reducing https://github.com/civicrm/civicrm-core/pull/10962/files easier to parse this makes some small
code tidy ups
1) renames functions to not use the deprecated convention of leading underscore
2) renames function to getLineItemsToAlter rather than 'to add & update'
 (later changes will cause this array to include 'to_cancel' & 'to_resurrect')
3) changes the array return format for that function to be an array of requiredChanges
4) rename function _getFinancialItemsToRecord to getReverseFinancialItemsToRecord to
reflect the fact it is generating an array of reversals to perform

Avoid 'array_pop calling a function' situation

Merge pull request #11195 from jitendrapurohit/CRM-21349

CRM-21349: Increase timeout of status message after batch merge

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

CRM-21041: Respect default 'Communication Style' when creating a Contact via API

minor fix

Merge pull request #11204 from MiyaNoctem/CRM-21360-make-open-case-manageable

CRM-21360: Make 'Open Case' Activity Optional

CRM-21383 - Inclusion of a directive to replace the existing select field for loading Message templates on demand in the CiviMail compose UI

Merge pull request #11224 from colemanw/CRM-21384

CRM-21384 - Don't show deleted contacts in case relations

Merge pull request #11260 from JMAConsulting/CRM-21416

CRM-21416: Add reset link beside 'Search' button below Advance Search form

CRM-21416: Add reset link beside 'Search' button below Advance Search form

Add unit test on financial items for pending payments.

This test was supposed to demonstrate a failure but actually it demonstrates something working.

Still, good to have an additional test

Merge pull request #11265 from eileenmcnaughton/test

[NFC] Minor refactor of test to allow re-use

[NFC] Minor refactor of test to allow re-use

Change-Id: Ie0ec7ff504420aa41641abc6de4edb8cfb5a4b64

Merge pull request #11253 from christianwach/CRM-20358

CRM-20358: Fix WordPress Access Control table