[NFC] Minor code tidy up

Declare Exceptions, use strict comparison

Merge pull request #16321 from colemanw/CheckBox

dev/core#1535 - Fix missing address is_primary field

dev/core#1535 - Fix missing address is_primary field

Merge pull request #16317 from eileenmcnaughton/duplicate

Remove duplicate line of code

Merge pull request #16319 from civicrm/5.22

5.22

Merge pull request #16313 from eileenmcnaughton/522

dev/core#1217 Fix for failure to respect event_id (or any?) criteria in smart groups

Remove duplicate line of code

Merge pull request #16310 from eileenmcnaughton/mem

[NFC] code cleanup

Merge pull request #16309 from eileenmcnaughton/gn

[NFC] Code cleanup

dev/core#1217 Fix for failure to respect event_id (or any?) criteria in smart groups

From what I can tell it's possible to create smart groups from Advanced search & other contact searchs & Membership Search & Event search.

Membership search actually just leverages advanced search but Event Search smart group has it's own code & reloads into Event search.

This has been flawed for months but the current iteration is the criteria are not respected. On digging I found the decision was made in
contact search to store the parsed QueryParams (in [field_name, 'IN', [1,2]] format not the formValues. I think this was probably a mistake.

However, by adjusting the event code to do the same & ensuring the defaults are set it starts working again

Merge pull request #16308 from eileenmcnaughton/search_ref

[REF] Remove apparent copy & paste code.

Merge pull request #16302 from colemanw/wildcards

Api4 - Support wildcard * in select clause

Merge pull request #16305 from colemanw/formatOutput

Api4 - format output consistently across get/create/update.

[NFC] code cleanup

Declare Exceptions, use strict, use single quotes

[NFC] Code cleanup

Throw exception rather than fatal
- declare exceptions
- single quotes
- strict comparisons

[REF] Remove apparent copy & paste code.

In code familiarisation to tackle dev#core/1217 I looked at this var & concluded that the var is set in 9
places but in 8 of those it is never used. In the last place (CRM_Contact_Form_Search) it is used to
determine whether to build the pre-next cache. I swapped it over to just doing a getButton without any
change - I stepped through in advanced search, search builder, custom search, basic search it still enters the cache lines of code. In Contribution, event search it doesn't (before & after).

Part of just makingg code semi-readable

Merge pull request #16306 from seamuslee001/dev_mail_59

dev/mailing#59 Update the version of zetacomponents/mail package to b…

dev/mailing#59 Update the version of zetacomponents/mail package to 1.9.0 to be one that potentially will support XAUTH2 for google purposes

Merge pull request #16304 from mattwire/case_removeunusedcustom

Remove unused code in Case/Form/Activity postProcess

Api4 - format output consistently across get/create/update.

Previously output from a DAO-based Get would be run through an unserializer/type-converter.
This applies that same conversion to DAO-based create/update ops.

Merge pull request #16299 from mattwire/scheduledjobs_doc

Update docs link for edit scheduled job parameters

Remove unused code in Case/Form/Activity postProcess

Api4 - Support wildcard * in select clause

Merge pull request #16300 from colemanw/contactTypeFields

Api4 - Filter returned fields by contact type

Merge pull request #16303 from MikeyMJCO/patch-3

Fix minor typo in help text.

Api4 - Filter returned fields by contact type

Merge pull request #15857 from demeritcowboy/clear-dyn-folder

dev/core#1395 - Clear the asset builder cache when clearing cache

Fix minor typo in help text.

Merge pull request #16230 from aydun/1511_event_show_location

dev/core#1511 Expose option to show event location on configuration tab

Merge pull request #16301 from eileenmcnaughton/cust_clean

[NFC] code cleanup

Merge pull request #16113 from colemanw/meta

Add metadata to is_primary fields

Merge pull request #15891 from mecachisenros/order-create-isskiplineitem

Enforce isSkipLineItem for membership payment entity in Order.create

Merge pull request #16266 from jitendrapurohit/core-1517

dev/core#1517 - Permission error on event info page for anonymous users

[NFC] code cleanup

Swap out some fatals  for exceptions, use single quotes,  strict comparison

Merge pull request #15759 from tunbola/active-option-values-for-custom-group

Fetch Only Active Custom Group Extend Values

Update docs link for edit scheduled job parameters

Add help text for Show Location option

Merge pull request #16296 from seamuslee001/master

5.22

Merge pull request #16273 from demeritcowboy/d8-check-group-roles

dev/drupal#89 Drupal 8 version of checkGroupRole

Merge in 5.22

Merge pull request #16292 from eileenmcnaughton/522

dev/user-interface#13 fix regression where option to email is not longer displayed

Merge pull request #16293 from eileenmcnaughton/part_search

Enotice fix

Merge pull request #16290 from eileenmcnaughton/cont

[NFC] Minor code cleanup

Enotice fix

dev/user-interface#13 fix regression where option to email is not longer displayed.

The api parameter here was removed, seemingly unintentionally. It is picked up on by the script when it checks for email so
needs re-instating

Merge pull request #15314 from jitendrapurohit/dev-1255

dev/core#1255 - fix display of email address on pay later contribution

[NFC] Minor code cleanup

Merge pull request #16246 from eileenmcnaughton/dedupe2

Check field to carry over custom data by default when the other contact has none

Merge pull request #16288 from civicrm/5.22

5.22 to master

Merge pull request #16287 from MegaphoneJon/core-1524-rc

fix created_date/modified_date relative date filter searches

Merge pull request #16216 from demeritcowboy/unicode-mailing-labels

dev/core#1502 Make unicode font dejavusans the default for mailing labels

Merge pull request #16281 from eileenmcnaughton/part_pay_wip

Basic extraction of paid event processing

fix created_date/modified_date relative date filter searches

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

Basic extraction of paid event processing

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

drupal 8 version of checkGroupRoles

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

Add metadata to is_primary fields

Rewrite auto-renew message to make it more accessible

update test to match changes elsewhere

Update contributor-key.yml

Fix auto renew text

Clear asset cache when clearing cache.

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

dev/core#1517 - Permission error on event info page for anonymous users

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