Merge pull request #16284 from kcristiano/cleanurls-setting

Update civicrm.settings.php.template to default to CLEANURL for WP

Merge pull request #16272 from wmortada/improve-accessibility-of-auto-renew-message

dev/core#1520 Improve accessibility of membership edit form

Update civicrm.settings.php.template to default to CLEANURL for WP

Signed-off-by: Kevin Cristiano <kcristiano@kcristiano.com>

Merge pull request #16282 from agileware/CIVICRM-1417

Force reCaptcha is not working when pay later enabled

CIVICRM-1417 enable reCaptcha with pay later option

Merge pull request #16280 from seamuslee001/fintype_delete

[REF] Fix return value on deleting financial type

Merge pull request #16274 from colemanw/type

Api4 - Convert field values to correct data type

[REF] Fix return value on deleting financial type

Extend unit test to cover deleting fin type

Merge pull request #15624 from seamuslee001/payment_processor_api4

Implement PaymentProcessor and PaymentProcessorType APIv4 Entities

Implement PaymentProcessor and PaymentProcessorType APIv4 Entities

Move default values to spec as per Coleman

Fix class name in tests and fix default values on tables

Mock financial_account_id field

Fix headers and remove some unnessary code

Update api v4 code following move of the handling to the BAO and update tests as per Coleman

Merge pull request #16279 from seamuslee001/ref_payment_processor_instrument

[REF] Move handling of default payment instrument for a payment proc…

[REF] Move handling of default payment instruement for a payment processor into the create function from API and expand tests

Merge pull request #15570 from colemanw/caseRoles

Restrict case roles by group

Add caseType test

Change caseType activityAsgmtGrps to store group name instead of id

Option to limit case roles to certain groups

Merge pull request #16262 from eileenmcnaughton/add_payments

Remove never passed-in parameter from addPayments

Merge pull request #16244 from eileenmcnaughton/dedupe

Remove unused and largely unreachable code.

Merge pull request #16263 from eileenmcnaughton/ids_3

[REF] Reduce places where we pass ids into Mailing::create

Merge pull request #16256 from eileenmcnaughton/ids

[REF] Remove ids and fully deprecate passing it to Contribution::create

Merge pull request #16265 from seamuslee001/fix_flexmailer

Ensure that we always return a raw urlencoded url for extenal urls to…

Merge pull request #16275 from seamuslee001/nfC_payment_processor_type_field

[NFC] Explicitly specify that it is the Payment processor Type Name n…

Merge pull request #16278 from civicrm/5.22

5.22

Merge pull request #16277 from seamuslee001/5.22

Add release notes for 5.21.1

Add release notes for 5.21.1

Api4 - Output field values as correct data type

[NFC] Explicitly specify that it is the Payment processor Type Name not Payment Procesor name field.

Merge pull request #15829 from eileenmcnaughton/dedupe3

Add resolved to return array for get_conflicts

Merge pull request #16269 from wmortada/fix-auto-renew-text

dev/core#1519 Fix auto renew text

Merge pull request #16268 from JGaunt/patch-2

Update contributor-key.yml

Merge pull request #16270 from demeritcowboy/testtest

Update failing test to match intervening changes elsewhere

Rewrite auto-renew message to make it more accessible

update test to match changes elsewhere

Update contributor-key.yml

Fix auto renew text

Merge pull request #15306 from bhahumanists/bhahumanists-mailingID

#1253 Pass mailing id to alterMailContent hook

Merge pull request #16195 from demeritcowboy/activityview-recent-items

dev/core#1507 Fix recent items for viewing Email activities from contact activities tab

Merge pull request #16257 from colemanw/index

Add more functionality to the api4 $index param

Merge pull request #16264 from MegaphoneJon/soft-credit-hooks

Add pre/post hooks on ContributionSoft entity

Add pre/post hooks on ContributionSoft entity

Add unit test for Civi\Api4\Utils\ReflectionUtils::parseDocBlock

[REF] Remove ids and fully deprecate passing it to Contribution::create

ids is no longer preferrred & is functionally equivalent

Test fix - legacy api4 wrapper should always return id

Ensure that we always return a raw urlencoded url for extenal urls to fix Flexmailer tests

Merge pull request #15318 from jitendrapurohit/dev-1263

dev/drupal#89 - Drupal 8 - Contact Report does not load any values in…

Merge pull request #16258 from samuelsov/i18ncountriesorder

Country/State PseudoConstant not sorted according to the locale

[REF] Reduce places where we pass ids into Mailing::create

In general the goal is to stop passing the ids variable. This removes a couple of places. Note
that the value in params should be id not mailing_id

Remove never passed-in parameter from addPayments

Merge pull request #16148 from mattwire/contributionpayment_fullrefund

Set contribution status to refunded when it has been refunded

Merge pull request #16251 from eileenmcnaughton/line_clean

[REF] Remove never-passed param from getLineItems

Merge pull request #16253 from eileenmcnaughton/mem_val2

Add tax rates to metadata

Merge pull request #16247 from eileenmcnaughton/dedupe4

[REF] Extract location wrangling code.

Merge pull request #16254 from eileenmcnaughton/partial_pay

[REF] rename lineItem & lineItems variable

Merge pull request #16240 from MegaphoneJon/core-1512

core#1512: Address ID should be exportable

Merge pull request #16208 from mattwire/activitytokensfixcaseid

Fix activity.case_id token to always display the first case ID

Merge pull request #16252 from eileenmcnaughton/line_items_by

[REF] Move use of priceSetID & amount_override to where they are used

Country/Region PseudoConstant not sorted according to the locale

core#1512: Address ID should be exportable

Improve civicrm_api4 docblock and use it for help display in Explorer

Add tax rates to metadata

This doesn't do anything by itself but it lays the foundation for a solid fix for

https://github.com/civicrm/civicrm-core/pull/15895 - ie by having minimum_fee_with_tax as part of our
generally available metadata we get out of all the complexity of doing this in the js layer - which is
what is driving that bug

Unit test for #16148

Set contribution status to refunded when it has previously been fully paid and now payments add up to 0

[REF] rename lineItem & lineItems variable

Readability fix -

=======Before========
 refers to an array of lineItems,  refers to a single lineItem

=======After
 refers to an array of line items,  refers to the single (created) line itmem

Move use of priceSetID & amount_override to where they are used

The field name is misleading since it implies it is useful in general to pass in a priceSetID - however, it has
a very specific usage & meaning & ideally we would deprecate it in favour of something more readable. For now
group the code more logically & comment it

Merge pull request #16059 from eileenmcnaughton/mem_val2

[REF] Cleanup CRM_Member_BAO_Membership::buildMembershipTypeValues

Merge pull request #16249 from eileenmcnaughton/ex_exclosed

[REF] Remove enclosed & escaped variables

Merge pull request #16248 from eileenmcnaughton/cc

[NFC] Code comments

Merge pull request #16250 from eileenmcnaughton/member

[NFC] Minor cleanup in test class

dev/core#1263 - Drupal 8 - Contact Report does not load any values in the ACL Group/Role field

[REF] Remove never-passed param from getLineItems

Parameter  was added to getLineItems in https://github.com/civicrm/civicrm-core/commit/25d0c6478ee236a4ac150e8d2b39614602ca4ae8
to support invoicing. However that function no longer calls getLineItems & has not for some time (at least a year). No other functions
pass in this parameter so we can remove it

Merge pull request #16245 from eileenmcnaughton/dedupe1

[REF] Minor code simplification in dedupe.

[NFC] Minor cleanup in test class

Code comments

Remove enclosed variable.

Given that it can only ever be a double quote parameterizing it is only confusing

[REF] Extract location wrangling code.

Note I have deliberately kept this extraction as vanilla as possible wtih the goal being to rationalise in
later iterations

Support index param as array in api explorer

civicrm_api4 - Extend functionality of index param

Passing an array allows you to return a single result column, with automatic handling of the select clause.

[REF] Minor code simplification in dedupe.

This just tweaks readability by
1) adding a variable for a value nested fairly deeply in an array
2) removing the foundField handling in favour of a ternary since
it's kinda confusing as is

Merge pull request #15475 from mecachisenros/externUrl

dev/core#1100 Add externUrl method and civicrm_alterExternUrl hook

Merge pull request #16231 from mfb/imap-batch-size

Remove patch hunk now that zetacomponents/Mail#73 has been merged

Remove unused and largely unreachable code.

The removed lines can only be reached if
a) CMS is Joomla or
b) CMS is Drupal and NULL is not equal to NULL

This seemed a little suspicious - especially since the merge screen still does display the value that should
be but isn't calculated here. After digging I found it is duplicated here
https://github.com/civicrm/civicrm-core/blob/bc77d7c0798081f999e6eca92d75fa874fd04e74/CRM/Contact/Form/Merge.php#L115
and that is the value used in the tpl here
https://github.com/civicrm/civicrm-core/blob/1188c7a8512382b1cb588b6a1ea4b13aa9587bc9/templates/CRM/Contact/Form/Merge.tpl#L217

so these lines can be safely removed as redundant. I also signposted where I think the next cleanups should happen on this
with code comments

Merge pull request #16241 from eileenmcnaughton/sess

[REF] move sessionStart functionality to System subclass

Merge pull request #16242 from civicrm/5.22

5.22 to master

Merge pull request #16239 from MegaphoneJon/mailing-60-522

mailing#60: Fix regression where `is_bulkmail` flag isn't respected

civicrm_api4 - auto-adjust select query when passing an index

Merge pull request #15778 from agh1/reminder-do-not-email-test

dev/core#1378 Scheduled reminders: do not email if do_not_email or on_hold

[REF] move sessionStart functionality to System subclass

Preparatory to https://github.com/civicrm/civicrm-core/pull/15131/files

Merge pull request #16105 from seamuslee001/dev_core_522_204

dev/core#522 CRM-19767 dev/core#204 Add Case Tokens to emails

Merge pull request #15969 from eileenmcnaughton/utfmb8

Add utf8 to utfmb8 conversion api command

Merge pull request #16234 from demeritcowboy/more-t6

dev/core#495 Make admin settings field labels translatable

Merge pull request #16235 from demeritcowboy/more-t7

dev/core#495 Make admin settings field labels translatable

Merge pull request #16236 from demeritcowboy/more-t8

dev/core#495 Make admin settings field labels translatable

Merge pull request #16237 from demeritcowboy/more-t9

dev/core#495 Make admin settings field labels translatable

Merge pull request #16233 from demeritcowboy/more-t5

dev/core#495 Make admin settings field labels translatable

Merge pull request #16232 from demeritcowboy/more-t4

dev/core#495 Make admin settings field labels translatable

mailing#60: Fix regression where multiple bulk recipient option isn't respected

test fixes

make translatable

make translatable

Merge pull request #16228 from demeritcowboy/more-t2

dev/core#495 Make admin settings field labels translatable

Merge pull request #16229 from demeritcowboy/more-t3

dev/core#495 Make admin settings field labels translatable