Merge pull request #17393 from eileenmcnaughton/to

[REF] Code simplification & unit test on suppressed emails in task

Merge pull request #17391 from mattwire/createEmailActivityclarity

NFC Add a bit more clarity to createEmailActivity function

Merge pull request #17388 from seamuslee001/domain_tokens_message_template_admin_interface

Add in domain tokens onto the Message Template admin interface

Merge pull request #17253 from mattwire/utf8convertblocksize

utf8mb4 - If strict mode enabled query will fail if KEY_BLOCK_SIZE is not 0

Merge pull request #17178 from mattwire/simplifyrecurtemplates

Simplify recurring contribution template inheritance

Merge pull request #17398 from eileenmcnaughton/mem_recur

[REF] Pass an array of correct params to the function to create a recurring contribution.

[REF] Pass an array of correct params to the function to create a recurring contribution.

This is getting away from 'passing whatever params we have' to 'passing the right params', which makes it easier
to see what can and can't be altered.

Note that from https://issues.civicrm.org/jira/browse/CRM-17636 we can see that is_email_receipt is not set
for payment purposes but contributionRecur purposes and since it's not a propertyBag param we can leave it off
here

Merge pull request #17395 from seamuslee001/update_DAO

Update DAO and L10n Schema file

Update DAO and L10n Schema file

Merge pull request #17394 from colemanw/api4CustomJoins

APIv4 - support custom entity joins

Merge pull request #17363 from eileenmcnaughton/price_field

Api4: Add Price api entities

[REF] Code simplification & unit test on suppressed emails in task

Merge pull request #17118 from lcdservices/dev-core-1722

dev/core#1722 Allow for first name and last name to be searched on

Merge pull request #17244 from totten/master-preboot

dev/translation#39 - Fix `core:install --lang`. Realign boot plugins.

Merge pull request #17392 from eileenmcnaughton/ct

Remove unused 'reset' param, other NFC tidy up:

Merge pull request #17364 from eileenmcnaughton/price_pseudo

Fix pricefield pseudoconstant.

PriceFieldValue - Make name and label required in DB

Fix notice and redundant code in PriceFieldValue BAO

Add Price api entities

Api4 entities for Price, PriceField & PriceFieldValue.

All 3 entities are accessed in the PriceFieldTest

Merge pull request #17390 from mattwire/api3activitydatetimedefault

Set default using spec for activity_date_time

Merge pull request #17386 from seamuslee001/dev_core_1775

dev/core#1775 Fix fee amount reversals when doing a change of financi…

Remove unused 'reset' param, other NFC tidy up:

APIv4 - Fix explicit joins to custom entities

Merge pull request #17387 from eileenmcnaughton/ct

Use apiv4, cache infra for basicTypes

APIv4 - Improve joins test coverage

Use apiv4, cache infra for basicTypes

I came across this as using an old weird pattern accessing the DAO. On looking it made most
sense just to switch to an apiv4 call. The caching was not convoluted but effectively
used an array cache, if loaded, and the array or Redis or sql cache if not. This
is the same as using the 'metadata' or 'contactTypes' cache so I switched to the latter.

I was inclined towards the former as I lean towards thinking this metadata all belongs in one - but there
is already another function in the same class going to the latter.

Add a bit more clarity to createEmailActivity function

Set default using spec for activity_date_time

Merge pull request #17228 from mattwire/fixmultilingualoptiongroups

Fix adding languages in multilingual

Merge pull request #17308 from colemanw/BAOcleanup

Various BAO create/add cleanups to use writeRecord()

Merge pull request #17348 from eileenmcnaughton/payex

Remove instantiation of transaction from payment express ipn class

Add in domain tokens onto the Message Template admin interface

Merge pull request #17381 from eileenmcnaughton/dao

Remove null code

Merge pull request #17384 from eileenmcnaughton/abort

Remove validation bypass

Merge pull request #17383 from eileenmcnaughton/ids

[NFC] Remove some more of the old  cvs blocks

dev/core#1775 Fix fee amount reversals when doing a change of financial type

Remove validation bypass

I was trying to understand why executeQuery allows validation to be bypassed. That seems
intuitively like something that should be handled by the calling function. This is one of the few
places that calls it - but I can't see a reason for it to by pass validation....

My best guess is that a parameter could legitimately be NULL - and the group properly is the
only one that seems to be a candidate

I also note the case for passing i18nRewrite seems dubious as the table has no localisation

[NFC] Remove some more of the old  cvs blocks

Remove null code

Merge pull request #17358 from seamuslee001/d8_prevnext_cache_test_fix

[TEST] Fix test failure on Drupal 8 E2E PrevNextTest by only includin…

Merge pull request #16541 from bhahumanists/subtype-issue-991

#991 subtypes not filled in when editing a smart group

Merge pull request #17379 from colemanw/del

Don't exclude deleted contacts from ACL cache when user has permission

Merge pull request #17318 from agh1/aria-hidden-icon

Make all icons aria-hidden

Merge pull request #17378 from agh1/cleanup-feed-urls

CiviEvent Dashboard and Manage Events: clean up disused url template vars

ACL - Don't exclude deleted contacts from ACL cache when user has permission to see them

Test failures seem to be due to assumptions of the test.
Removed via Hume's guillotine

When the user has 'access deleted contacts' permission, deleted contacts
should not be excluded from the ACL cache.

CiviEvent Dashboard and Manage Events: clean up disused url template vars

Add aria-hidden to extant icons

Add aria-hidden to icon helpers

Merge pull request #17282 from agh1/calendar-icons

Replace ical and office calendar image icons

Merge pull request #17345 from eileenmcnaughton/ev_batch

Preliminary cleanup on form

Fix pricefield pseudoconstant.

The price field turns out not to be returning pseudoconstant fields for 2 reasons
1) the label field is defined but the name field is not
2) any price fields with no domain are filtered out. In my single domain db this is
all of them.

I think we can safely understand NULL domain_id as 'not restricted by domain'. For some tables
(membershipType) it is a required field but for PriceField it is not.

I would be inclined to say it should be required less often. However, where it is required thenn
the OR IS NULL should never be true

Merge pull request #17376 from artfulrobot/artfulrobot-public-status-messages

Public status messages should show correct class/icon like error

Merge pull request #17373 from eileenmcnaughton/post_simp

[REF] Minor code simplification

Merge pull request #17346 from eileenmcnaughton/trans2

[REF] Call completeOrder directly from event status update form

Public status messages should show correct class/icon like error

Merge pull request #17371 from eileenmcnaughton/too

Undo variable variable.

Merge pull request #17370 from eileenmcnaughton/to

Use non-deprecated method

Merge pull request #17368 from colemanw/api4suffix

APIv4 - Improve pseudoconstant suffix support

[REF] Minor code simiplification

On digging the only reason for the  array is to prevent duplicates.
The function  is passed do doesn't care how it's indexed so
just use php array to prevent duplicates. Reduce brain hurt

Undo variable variable.

This removes the remainder of a weird variable-varible loop.

 No actual change

Use non-deprecated methd

Merge pull request #17312 from totten/url3

Migrate CiviMail "extern" scripts to conventional routes

Merge pull request #17362 from colemanw/dev-core-1757-hide-toggle-menu-button

dev/core#1757 hide menu toggle when D7 toolbar is absent

Merge pull request #17369 from civicrm/5.26

5.26 to master

Merge pull request #17352 from christianwach/lab-wp-49-civi5-26

Remove "page=CiviCRM" query string from WordPress front-end (5.26)

APIv4 - Improve pseudoconstant suffix support

Suffixes like :color now work for get actions (but not create because they are not unique idenfitiers).
Test added for rich option lists in getfields

Merge pull request #17365 from eileenmcnaughton/pdf

Revert "[REF] Code readability changes on activity tokens."

Merge pull request #17356 from eileenmcnaughton/lang

Fix setLanguage docblock, other nfc changes

Merge pull request #17317 from colemanw/escJs

Improve js escaping in smarty {ts}

Revert "[REF] Code readability changes on activity tokens."

Per https://docs.civicrm.org/dev/en/latest/framework/token/#token-processor
The contactId is actually documented. While I've been removing those in various places in
favour of contactID or contact_id it seems Tim has been putting them in so
I think it's safer to revert this change at this stage

This reverts commit 2ebb452b239c28e7ed2c50da69d324a8d40a7e88.

dev/core#1757 - CRM Menubar - add method to remove toggle button

Merge pull request #17355 from twomice/1608_recurring_membership_fails_paypal

Fix for dev/core#1608: correct date format for trxn_date.

Merge pull request #17332 from colemanw/api4Joins

APIv4 - Remove implicit multi joins & add explicit joins

dev/core#1757 - CRM Menubar - hide toggle menu button for non admin users

Improve js escaping in smarty {ts}

Remove "page=CiviCRM" query string from WordPress front-end

Merge pull request #17357 from seamuslee001/contrib_page_report_column

[REF] [Reports] Ensure that all contribution pages including disabled ones are return…

Merge pull request #17354 from seamuslee001/mysql8_master

[TEST] Update test to support for 8.0.19 change

Update test to support for 8.0.19 change

Update test following Eileen comments

[TEST] Fix test failure on Drupal 8 E2E PrevNextTest by only including contacts where there is a sort name

Ensure that all contribution pages including disabled ones are returned when viewing the contribution detailed report

Fix setLanguage docblock, other nfc changes

The IDE can pick up the chaining if we tell it the object is being returned

Toward fix for dev/core#1608: correct date format for trxn_date.

Merge pull request #17330 from colemanw/dao_field

[REF] Cleanup and extract string functions specific to entity names.

Preliminary cleanup on form

Merge pull request #17304 from colemanw/endCode

Switch docblock code to use markdown syntax.

Merge pull request #17349 from eileenmcnaughton/validate

[REF] Stop passing  by reference.

Merge pull request #17347 from eileenmcnaughton/status

Call completeOrder directly from contribute status update form

Merge pull request #17343 from eileenmcnaughton/js_curr

Use status bounces, not fatal, in front end contribution form base class

[REF] Stop passing  by reference.

By checking in BAOContribution::loadRelatedObjects we can see it is not modified. The other 2 functions
only pass input to that function, so can also be non-variables

Remove instantiation of transaction from payment express ipn class

This class is hopefully used, however, this change should be safe regardless as it just calls the same function
without needlessly instantiating the parent.

Note this processor is not enabled on install or recommended

Call completeOrder directly from event status update form

We should really call completetransaction but that has been left for later unravelling with the focus here being
to clarify the purpose of the transaction. We don't need to pass a transaction around unless it has DB changes not
yet committed - which is not eally the case here. There is no reason to invoke BaseIPN but more unravelling required...

Call completeOrder directly from contribute status update form

We should really call completetransaction but that has been left for later unravelling with the focus here being
to clarify the purpose of the transaction. We don't need to pass a transaction around unless it has DB changes not
yet committed - which is not eally the case here. There is no reason to invoke BaseIPN but more unravelling required...

Merge pull request #17341 from seamuslee001/fatal_exception_status_bounce

dev/core#560 Replace deprecated CRM_Core_Error::fatal with either sta…

Use status bounces, not fatal, in front end contribution form base class

Merge pull request #17342 from eileenmcnaughton/event

Remove code that still seems unnecessary.

Merge pull request #17338 from demeritcowboy/test-from-to

[TEST] dev/core#1766 - Remove flakiness from testGetFromTo tests

Remove code that still seems unnecessary.

Per the code comment we should not need to unset these fields as they should not be present if not required.

I guess the test for this is whether the fields are unset if we start off with a payment processor & then deselect
it in favour of pay later

Merge pull request #17340 from demeritcowboy/not-yet-deprecated

[NFC] Add comment that deprecated code is actually still reachable

dev/core#560 Replace depreacted CRM_Core_Error::fatal with either statusBounce or Exceptions as appropriate in CRM/Core folder update langauge as per demeritcowboy