Fix enotices on paypal express (dev/financial/issues/14)

For background - Paypal Express was not originally supported for recurring. It kind of snuck
in the back door since it was actually not possible to block in when configuring paypal pro
and people kept adding little fixes since they assumed it was broken rather than deliberately
missing functionality. Somewhere along the line this function was probably copied & extracted
from the main function - but it still contained a bunch of lines not
required for paypal express. I took a look at what data is actually available in this function
and removed reference to unavailable fields.

This is from https://developer.paypal.com/docs/classic/api/merchant/CreateRecurringPaymentsProfile_API_Operation_NVP/ and from testing and from https://lab.civicrm.org/dev/financial/issues/14

The one are where I differed from logic from @carbar1103
@carbar1103
  is the trxn_id - I retained the profile id which I think is consistent with use for other processsors, but needs testing

Allow 'Cancel Subscription' for PayPal Express

Merge pull request #12220 from civicrm/5.2

5.2 merge to master

Merge pull request #12219 from eileenmcnaughton/case

dev/core#137 Fix Find Deleted Cases

Fix Find Deleted Cases

Merge pull request #12174 from eileenmcnaughton/matt

Towards dev/financial/#16 Paypal unreliable getting payment processor type

Remove usage of deprecated function from paypal

Merge pull request #12210 from colemanw/dev/core#117

dev/core#117 Replace useage of deprecated each() in MembershpView

dev/core#117 Replace useage of deprecated each() in MembershpView

Merge pull request #12192 from eileenmcnaughton/utils

Further additional utility function for handling odd array structure

Merge pull request #12208 from JMAConsulting/access-3-datepicker

dev/accessiblity#3 Add aria-label to datepicker element

dev/accessiblity#3 Add aria-label to datepicker element

Merge pull request #12204 from eileenmcnaughton/api_teste

[NFC] test clean up.

Merge pull request #11951 from mydropwizard/open-flash-chart-markup

Adjust creation of markup for Open Flash Chart

Merge pull request #12203 from eileenmcnaughton/api_tester

dev/core#38 Fix inappropriate limit on participant.getcount

Merge pull request #12199 from seamuslee001/case_core_117

dev/core#117 Replace deprecated each() function in CiviCase

dev/core#38 Fix inappropriate limit on participant.getcount

[NFC] test clean up.

Just some basic tidy up, removed a couple of tests that are very much covered by syntaxConformance tests

Merge pull request #12202 from seamuslee001/event_core_117

dev/core#117 Replace deprecated each in CiviEvent Participant registr…

Merge pull request #12201 from seamuslee001/report_activity_117

dev/core#117 Replace usage of deprecated each() in Activity Summary R…

Merge pull request #12196 from yashodha/CRM-12186_fix

CRM-21632 - {membership.fee} prints out in documents with 9 decimal p…

dev/core#117 Replace usage of deprecated each() in Activity Summary Report

dev/core#117 Replace deprecated each in CiviEvent Participant registration

dev/core#117 Replace deprecated each() function in CiviCase

Merge pull request #12021 from michaelmcandrew/managed-entity-deletion

More robust managed entity deletion

Added comment describing obfuscation

CRM-21632 - {membership.fee} prints out in documents with 9 decimal places

Merge pull request #12177 from eileenmcnaughton/test_monish

[NFC] code changes for export with merge household

Merge pull request #12180 from agileware/CIVICRM-832-5.0

Removing related memberships if parent membership type is changed which does not have relation type associated.

Further utility on handling odd array structure

Merge pull request #12191 from seamuslee001/dev_core_135

dev/core#135 Fix remaining non well formed numeric number problems in…

Merge pull request #12188 from larssg-wildsight/master

dev/core#43 - Update geocoding setting text to reflect need for Google API key

dev/core#135 Fix remaining non well formed numeric number problems in CRM_Batch_Form_EntryTest

Merge pull request #12190 from seamuslee001/dev_ore_135_a

dev/core#135 Fix first of 2 non numeric issues in CRM_Batch_form_entr…

Merge pull request #12189 from seamuslee001/117_report_recur

Dev/Core#117 Fix use of each() function in Recur Report

dev/core#135 Fix first of 2 non numeric issues in CRM_Batch_form_entryTest

Dev/Core#117 Fix use of each() function in Recur Report

Merge pull request #12187 from colemanw/recursive

Add utils for finding nested array items

Add utils for finding nested array items

Update GetFields.php

Update Map.setting.php

Update Mapping.tpl

Update Mapping.tpl

adding test to ensure dependent entities are uninstalled cleanly

Delete entities in the reverse order that they were created

always remove entity from civicrm_managed if we think it should be deleted

CIVICRM-832: Removing related memberships if parent membership type is changed which does not have relation type associated.

Merge pull request #12137 from eileenmcnaughton/year_fiscal

Extend fiscal year relative options to better match other periods

Merge pull request #12182 from michaelmcandrew/dev/mail#10

only add a closing quote if it is not already present

only add a closing quote if it is not already present

Merge pull request #12169 from seamuslee001/mime_mail_upgrade

Update variable name as per upgraded Mime_mail package to support PHP7.2

[NFC] stop passing around variables that are cached anyway.

[NFC] minor variable renaming.

I am working through https://github.com/civicrm/civicrm-core/pull/11870/files &
this is one very minor change that makes sense to commit in isolation

minor fix

Update variable name as per upgraded Mime_mail package to support PHP7.2

Merge pull request #12170 from seamuslee001/php72_pledge_foreach

dev/core#117 Fix use of each in CRM/Pledge

Merge pull request #11732 from twomice/CRM-21811_optimize_search_reciprocal_relationship_group

CRM-21811: Optimize advanced search by relationship with target g…

Merge pull request #12175 from eileenmcnaughton/jamie

dev/core#35 Avoid variable leakage on recurring contribution receipts.

Merge pull request #11944 from JMAConsulting/access-3-1

(accessibility/misc-core-fixes/issues/3) Remove CMS specific print header templates that share same code

Merge pull request #12128 from eileenmcnaughton/cust_extract

Permit UI editing of custom data on other entities (here relationship_type) when enabled in an extension

Merge pull request #12155 from seamuslee001/php7_2_each_membership

dev/core#117 Replace use of Deprecated function each in CRM/Member/BA…

Merge pull request #12172 from eileenmcnaughton/report_id_filter

dev/core#126 Add support for contact id as a basic report filter.

Merge pull request #12153 from JMAConsulting/dev_core_116

dev/core#116 Search builder searches on primary addresses are producing unexpected results

dev/core#35 Avoid variable leakage on recurring contribution receipts.

This covers pcpParams (which are tested & sets up the data set to extend to honor
params).

See https://lab.civicrm.org/dev/core/issues/35

(accessibility/misc-core-fixes/issues/3) Remove CMS specific print header templates that share same code

Merge pull request #11737 from aydun/CRM-21816-relative-dates-in-search

CRM-21816 Fix Relative dates in search bug

Merge pull request #12036 from mattwire/mail_unsetcc

dev/core#85 On some servers mail() fails when 'Cc' or 'Bcc' headers are defined but empty

Merge pull request #12171 from eileenmcnaughton/paypal_ex

Towards dev/financial/#16 Paypal unreliable getting payment processor type

Add support for contact id as a basic report filter.

This is to address a declared regression https://lab.civicrm.org/dev/core/issues/126 (in 4.7.30) and also to improve flexibility

Towards dev/financial/#16 Paypal unreliable getting payment processor type

This is a partial reviewer's commit on the work Matt did in
https://github.com/civicrm/civicrm-core/pull/12007#event-1631874773

It adds the new function & new construct but I spotted an error in
the usage of the function & so want to add the usage changes more slowly
& carefully

Merge pull request #11679 from jitendrapurohit/CRM-21776

CRM-21776: Ensure from clause adds custom table when search query is …

Merge pull request #12159 from tschuettler/123-core-particpiant-import-custom-fields

dev/core#123 Retrieve existing participant custom fields for import

Merge pull request #12156 from seamuslee001/non_numeric_membership_test

Fix issue where non numeric number was encounted in running unit test…

Fix issue where non numeric number was encounted in running unit tests on php7.2

Unit Test in api_v3_MembershipTest was throwing an error on the non numeric value of tax_amount, tax_amount was a NULL string.

dev/core#117 Fix Pledge each use in CRM/Pledge

dev/core#117 Replace use of Deprecated function each in CRM/Member/BAO/Membership.php to assit in supporting php7.2

Rename Variable to be more accurate

Merge pull request #12162 from seamuslee001/group_query_count

dev/core#118 Fix issue using count on non array in Group query builder

Merge pull request #12157 from seamuslee001/pass_by_reference_mailing

Remove pass by reference for some mailing functions

Merge pull request #12166 from seamuslee001/token_each

dev/core#117 Replace deprecated each with foreach in CRM/Utils/Token.php

Merge pull request #12167 from seamuslee001/import_each

dev/core#117 Replace deprecated each with foreach in CRM/Utils/Migrat…

Merge pull request #12168 from seamuslee001/dev_core_131

dev/core#131 Add in missing UK state Monmouthshire

Merge pull request #12163 from seamuslee001/count_peldge_tests

dev/core#118 Fix Count issues with PledgePayment and Pledge BAO tests

Merge pull request #12164 from seamuslee001/region_create_function

dev/core#130 Remove deprecated create_function from RegionTest in fav…

dev/core#117 Replace deprecated each with foreach in CRM/Utils/Token.php

Merge pull request #12165 from seamuslee001/activity_query_each

dev/core#117 Replace deprecated each with foreach in CRM/Activity/BAO…

dev/core#131 Add in missing UK state Monmouthshire

Monmouthshire is in Wales

dev/core#131 Add in upgrade step to add in missing UK state province

dev/core#117 Replace deprecated each with foreach in CRM/Activity/BAO/Query.php

dev/core#117 Replace deprecated each with foreach in CRM/Utils/Migrate/Import.php

dev/core#118 Fix Count issues with PledgePayment and Pledge BAO tests

dev/core#130 Remove deprecated create_function from RegionTest in favor of an annoymous function

dev/core#118 Fix issue using count on non array in Group query builder

dev/core#123 Retrieve existing participant custom fields for import

Try removing some more passing by reference

Attempt to remove pass by reference in some mailing functions

Merge pull request #12154 from compucorp/122-fix-bug-with-action-links

dev/core#122 Wrong Action Links Shown for Reserved and Locked Option Groups

dev/core#122 Add check for DELETE flag before unsetting it

(dev/core/116) Search builder searches on primary addresses are producing unexpected results

Merge pull request #12150 from eileenmcnaughton/memdetail52

Fix Fatal error on membership detail report (with ACLs enabled) dev/core/#100

Fix Fatal error on membership detail report (with ACLs enabled) dev/core/#100

Merge pull request #11629 from JMAConsulting/CRM-21675

CRM-21675: scheduled reminders: limit to group doesn't support parent groups