Merge pull request #16176 from eileenmcnaughton/mem_fix

dev/membership#21 fix regression on membership handling

New year (2020) test fixes. (Backport #16172)

We have 8 tests failing that seem to be due to 2020 starting. This fixes 2

dev/membership#21 fix regression on membership handling

This turns https://lab.civicrm.org/dev/membership/issues/21#note_29439 into a patch. As new code it seems to
have triggered an underlying issue. I also have https://github.com/civicrm/civicrm-core/pull/16139 open but
would be happy to see this hit the rc & the other have a full rc cycle.

The logic for this change makes sense as described & it is trivial. This has been a very hard bug
to reproduce so I'm unable to give full steps

Merge pull request #16170 from agh1/5.21.0-releasenotes-final

5.21.0 releasenotes final

5.21.0 release notes: last batch of edits

5.21.0 release notes - updated big contributors list for 2019

5.21.0 release notes: added late changes

Merge pull request #16160 from seamuslee001/5.21

5.20.3 Release Notes

Update 5.20.3.md

Add release-notes/5.20.3.md

Merge pull request #16155 from seamuslee001/5.21

dev/core#183 Ensure that having a rule where each field is the same w…

dev/core#183 Ensure that having a rule where each field is the same weight with the threshold being the total sum of the weights does not cause a fatal error when searching for dupes

Merge pull request #16140 from eileenmcnaughton/part_name

dev/event#25 Fix regression on billing name overwrite

dev/event#25 Fix regression on billing name overwrite

https://lab.civicrm.org/dev/event/issues/25 was caused by https://github.com/civicrm/civicrm-core/commit/b1b7f4e35bbe566d9efccac1055d311245420974
where it started overwriting the name in that function.

In digging in the code I found that it seems that as long as we call this function before the processor all is
good so we should call right before the processor rather than broader contamination.

Also note ideally the settings bag would be used.

Merge pull request #16120 from seamuslee001/dev_core_1485

dev/core#1485 Fix field name for join_date to be membership_join_date…

dev/core#1485 Fix field name for join_date to be membership_join_date in line with the DAO schema

Also include membership_join_date into the mapping as per Jamie's comments

Merge pull request #16121 from mlutfy/core1490-rc

dev/core#1490 Fix syntax error in Membership Receipt

dev/core#1490 Fix syntax error in Membership Receipt

Merge pull request #16107 from demeritcowboy/freezy-5.21

dev/core#1477 Fix fatal error when editing reserved option group settings

fix fatal error when editing reserved option group settings

Merge pull request #16052 from eileenmcnaughton/desc

Fix unreleased regression where processors using getPaymentDescription get a fatal

Merge pull request #16092 from seamuslee001/5.21

Fix Release note header

Fix Release note header

Merge pull request #16057 from eileenmcnaughton/act_perm521

dev/core#1444 Permit activities with campaigns on contact tab for con…

Merge pull request #16090 from seamuslee001/5.21

Add release-notes/5.20.2

Add release-notes/5.20.2

Fix unreleased regression where processors using getPaymentDescription get a fatal

We added the new property bag in https://github.com/civicrm/civicrm-core/pull/15697 but in the back & forth the
call to the function getDescription was retained but the function was removed. This would leave the
core processor that calls it (Paypal std) with a fatal. The known external processor using it (Omnipay)
also hits a notice because the signature changed ($params became optional). From the Omnipay
pov I think it makes sense just to stop overriding the function - hence I am sneaking a tiny
change into this to make that possible.

Note the doPayment function ensures that propertyBag is set. There is a small chance that an external processor
is fully overriding doPayment AND calling this - that would be the case for Omnipay except I'll remove
the override in conjunction with this - so I've added a few lines to ensure it is set.

Note the test would fail with the changes to Payment_Dummmy & not the changes to Core_Payment so it provides
test cover

Merge pull request #16087 from seamuslee001/deb_core_1467

dev/core#1467 - Multiple participant registration sends only p1 amoun…

dev/core#1467 - Multiple participant registration sends only p1 amount to the processor

Merge pull request #16081 from artfulrobot/payment-property-bag-add-getters

Payment PropertyBag: Add generic getter, setter methods for convenience

Payment PropertyBag: Add generic getter, setter methods for convenience

Merge pull request #16076 from alifrumin/rn5.21

5.21 Release notes: First Pass

Release Notes 5.21 - Sorting and Annotating

Merge pull request #16071 from jitendrapurohit/521rc

dev/core#1463 Fix record payment form errors

Fix record payment form errors

Merge pull request #16069 from seamuslee001/5.21

Add release-notes/5.20.1

Add release-notes/5.20.1

Merge pull request #16060 from artfulrobot/payment-property-bag-add-props

Add extra Payment PropertyBag properties

Merge pull request #16061 from eileenmcnaughton/cont_hook

dev/core#1460 move post hook to avoid errors with custom fields created in the hook

dev/core#1460 move post hook to avoid errors with custom fields created in the hook

Per dev/core#1460 a fatal error can occur if custom data is created in the post hook. In addition I note it makes sense to
include custom data before the rollback

Add extra Payment PropertyBag properties

dev/core#1444 Permit activities with campaigns on contact tab for contacts without 'administer CiviCampaign'

It seems that if an activity has a campaign it is suppressed from the activity table unless the user has 'administer CiviCampaign'.
This is inconsistent with the code to view the activity - which permits the activity to be viewed but the campaign cannot be edited
It is also inconsistent with activity search behaviour

Merge pull request #16047 from eileenmcnaughton/521

dev/membership#21 Fix regression causing transfer of membership to another contact if they paid

Merge pull request #16034 from seamuslee001/dev_core_1449

dev/core#1449 Do not dispatch pre/post hooks during upgrade

dev/membership#21 Fix regression on transferring member to another contact if they paid

dev/core#1449 Do not dispatch pre and post hooks whilst we are in an upgrade process

Add in code comments to point to new gitlab and reasoning for the change

Merge pull request #16033 from seamuslee001/dev_core_1447

dev/core#1447 Fix hard fail on upgrade due to duplicate names in the …

Add in unit test locking in the fix

Merge pull request #16036 from agh1/5.21.0-releasenotes

5.21.0 release notes: initial run

5.20.0 release notes: added boilerplate

5.20.0 release notes: raw from script

dev/core#1447 Fix hard fail on upgrade due to duplicate names in the database when trying to fix the option group name

Set version to 5.21.beta1

release-notes.md - Fix 5.19.4 TOC

release-notes/5.20.0

Update 5.19.4.md

Update 5.19.4.md

Update release notes for latest PR backported

Add in release notes for 5.19.4 drop

Also update composer.lock

Upgrade CKEditor to v4.13.0

Remove the check statement as not relevant to apiv4

Also ensure that updates etc cannot be done via get only via post as per APIv3

Ensure that API Explorer passes through necessary header

security/core#71 Prevent CSRF by ensuring that AJAX endpoint of the API Explorer can only be hit by javascript clinets

Merge pull request #16028 from civicrm/5.20

5.20

Merge pull request #16026 from eileenmcnaughton/payment_update

[NFC] code cleanup in test class.

Merge pull request #16027 from agh1/5.20.0-releasenotes-final

5.20.0 release notes: final edits

5.20.0 release notes: final edits

5.20.0 release notes: add late changes

[NFC] code cleanup in test class.

Declare exceptions, use single quotes, call static functions statically

Merge pull request #15697 from artfulrobot/payment-property-bag

Add Payment PropertyBag for payment processor method arguments

Merge pull request #16021 from mattwire/paymentinfo_amountpaid

Always display amount paid even if it is 0.00. Blank looks like it's missing data rather than nothing paid

Merge pull request #16024 from jmcclelland/pcp-contribution-status

allow PCP report to filter on contribution status

Merge pull request #16023 from seamuslee001/1435_nfc

[NFC] Fix camelCase variable name following fix for dev/core#1435

[NFC] Fix camelCase variable name following fix for dev/core#1435

Merge pull request #16019 from mattwire/contributionview_removecancelled

Remove unused cancelledStatus assign on contributionView

allow PCP report to filter on contribution status

Without this filter, incomplete contributions are included in the report
which really throws off the total.

Always display amount paid even if it is 0.00. Blank looks like it's missing data rather than nothing paid

Remove unused cancelledStatus assign

Merge pull request #16016 from seamuslee001/master

5.20

Merge in 5.20

Merge pull request #16013 from eileenmcnaughton/reg

dev/core#1435 Fix lack of filtering in Contribution tab on Membership or Participant View

Merge pull request #15989 from seamuslee001/dev_core_523

dev/core#523 get the Lybunt charts working again

dev/core#1435 Fix lack of filtering in Contribution tab on Membership or Participant View

Merge pull request #16010 from demeritcowboy/remove-recordactivityviaemail

[NFC] Remove dinosaur function

remove unused function

Merge pull request #15523 from jamie-tillman/patch-1

PayPalImpl: Add SOLUTIONTYPE parameter that triggers allowing to pay without having to create a paypal account.

Merge pull request #15971 from aydun/report#23

dev/report#23 - Fix contact sub-type 'is not one of' filtering

Merge pull request #15946 from seamuslee001/status_check_improve

Make utf8mb4 status check language more for the general consumer

Merge pull request #15974 from eileenmcnaughton/ex_clean

Make else statement more legible

Merge pull request #15908 from eileenmcnaughton/anet_setup

[Test] fix test setup on ANet tests

Merge pull request #15997 from seamuslee001/dev_core_1166

dev/core#1166 Update the name of Macedonia, Republic of to North Macedonia

Merge pull request #16008 from civicrm/5.20

5.20

Merge pull request #16007 from seamuslee001/5.20

Add release-notes/5.19.3.md

Add release-notes/5.19.3.md

Merge pull request #15970 from eileenmcnaughton/ex_easy

[REF] clean up merge array.

Merge pull request #16001 from agileware/CIVICRM-1383

Code style, remove typo, double ;; from code where it makes sense to do so

Merge pull request #15937 from seamuslee001/dev_core_1405

dev/core#1405 Strip any spaces from Option Group name parameter and r…

CIVICRM-1383 Remove typo, double ;; from CiviCRM code