Merge pull request #16307 from seamuslee001/dev_mail_59

dev/mailing#59 Add in provider option IMAP_XOAUTH2 to assist with goo…

Merge pull request #16326 from wmortada/core#1537

dev/core#1537 Don't append multiple copies of the eligible related contacts text

Merge pull request #16286 from alifrumin/offlineContributor

Add "Contributor Name" to Offline Contribution Receipts

Merge pull request #16343 from colemanw/abstract

Api4 - Make abstract function abstract

Merge pull request #16346 from colemanw/entityGet

Api4 - Improve Entity::get

Merge pull request #16347 from demeritcowboy/name-label-yet-again

dev/core#1116 Wrong linebreaks viewing Inbound Emails when activity type label is changed

Merge pull request #16337 from eileenmcnaughton/ev_static

Change function buildEventFeeForm to non-static

wrong linebreaks when activity type label is changed

Merge pull request #16344 from eileenmcnaughton/ev_test

Fix unit test so price set params are valid

Merge pull request #16340 from jitendrapurohit/dev-1540

Add user friendly error message on merge error

Fix unit test so price set params are valid

Add user friendly error message on merge error

Merge pull request #16345 from eileenmcnaughton/ev_title

Simplify event title retrieval, use cache, add test

Merge pull request #16342 from JKingsnorth/patch-13

[NFC] Remove unused local variables from CRM_Contribute_Form_AdditionalPayment::processCreditCard

Api4 - Improve Entity::get

Ensure that every php file found by the scanner is_a class inheriting from AbstractEntity.
Remove the CustomValue class by virtue of its class type instead of manually remove it.
Improve performance when fetching entities by name.

Simplify event title retrieval, use cache, add test

Merge pull request #16341 from mattwire/membership_overridden

When viewing a membership show if the status is overridden

Api4 - Make abstract function abstract

Remove unused variable $now

When viewing a membership show if the status is overridden

Remove unused local variable $fields

Merge pull request #14591 from JKingsnorth/dev/core#1064

dev/core#1064 Allow personalised 'view in browser' links for mass emails

Changee buildEventFeeForm to non-static

I searched on thee full function name & also 'buildEvent' & 'feeForm' and I don't believe this is called
from outside this function. I want  to call it from tests to  fix some test set up issues butt
there are a bunch of undefined properties so I concluded I should set this up first.

Merge pull request #16315 from eileenmcnaughton/var

[REF] Remove setting of unused function.

Merge pull request #16331 from eileenmcnaughton/cont_soft

[NFC] Code formatting

Merge pull request #16329 from eileenmcnaughton/ev_one

Start to use function rather than multiple queries for event details add test

Merge pull request #16333 from eileenmcnaughton/bounce

[REF] Move the bounce to the start of the form submit

Merge pull request #16332 from seamuslee001/event_confirm_text_extend

[NFC] Extend unit test to cover for recent refactor by eileen to ensure sup…

Move the bounce to the start of the form submit

Start to use function rather than multiple queries for event details, add test

[NFC] Code formatting

Extend unit test to cover for recent refactor by eileen to ensure support for CRM-4196 remains

Merge pull request #16330 from eileenmcnaughton/process_event

[REF] Eliminate silly parameter from function signature.

Merge pull request #16018 from mattwire/manualcontributionstatus

Fix retrieving contribution_status_id for manual payment processor

Add code docs to Api4/CustomValue

Fix retrieving contribution_status_id for manual payment processor

Eliminate silly parameter from function signature.

Removes the isAdditionalAmount parameter which is only used to adjust the amount key on the value variable.

Value is reset each participant so we don't need to worry about it being 'cumulative'

Merge pull request #16316 from eileenmcnaughton/parent

Remove function that exactly matches parent class.

Merge pull request #16327 from eileenmcnaughton/pp

Extract code for isPaymentOnExistingContribution.

Extract code for isPaymentOnExistingContribution.

This allows us to make the meaning of the IF clause clearer

Merge pull request #16325 from eileenmcnaughton/ev

[REF] Extract code that assigns event variables to the tpl

Merge pull request #16314 from eileenmcnaughton/nfc

[NFC] Code cleanup

Extract code that assigns event variables to the tpl

Don't append multiple copies of the eligible related contacts text

Merge pull request #15992 from eileenmcnaughton/utf_temp

Update temp table handler to support utf8mb4 if that is the db collation

dev/mailing#59 Add in provider option IMAP_XOAUTH2 to assist with google's changes to imap access

Apply PHP Syntax fix patch and update tests

Bump up to 1.9.1 of zetacomponents/mail to fix syntax issue

Remove function that exactly matches parent class.

I copied it over the parent class fn & apart from minor comment block differences it matches

Merge pull request #15383 from michaelmcandrew/lab-1286

also skip report instance header and footer (lab-1286)

Merge pull request #16320 from mattwire/pcp_pagecleanup

NFC Convert Manage PCP page to short array syntax

[NFC] Fix some missing exceptions etc

[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

lab-1286 - skip report instance header and footer in CRM_Utils_API_HTMLInputCoder

Convert Manage PCP page to short array syntax

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

Remove setting of unused function.

The called function ONLY returns the query so nothing gained by getting a throw-away variable

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 temp table handler to support utf8mb4 if that is the db collation

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