Merge pull request #19898 from eileenmcnaughton/teardown2

More phpunit8 prep

Merge pull request #19899 from eileenmcnaughton/tear

[PHPUNIT8] add void to more teardown()

Merge pull request #19900 from eileenmcnaughton/tear2

[PHPUNIT 8 ] Add void to more teardowns

Merge pull request #19891 from eileenmcnaughton/task1

[REF] Simplify setting of softcredit form options (minor)

Merge pull request #19897 from eileenmcnaughton/teardown

Add :void return type to tearDown on a bunch of unit tests

Merge pull request #19639 from eileenmcnaughton/test_test

[REF] Fix ContributionPage::recurringNotify to use apiv4, simplify params

Merge pull request #19896 from colemanw/api4GetActionsFix

APIv4 - Fix GetActions function to work with entityNames that don't match className

Merge pull request #19895 from eileenmcnaughton/falsefail

Skip UFField.field_name on singleValueAlter as flakey

[PHPUNIT 8 ] Add void to more teardowns

[PHPUNIT8] add void to more teardown()

More phpunit8 prep

Add :void return type to tearDown on a bunch of unit tests

APIv4 - Fix GetActions function to work with entityNames that don't match className

Merge pull request #19892 from colemanw/searchKitCreatedModified

SearchKit - Add created & modified info to SavedSearch listing

Merge pull request #19312 from seamuslee001/dev_core_1845

dev/core#1845 Change FK on civicrm_group to delete the associated gro…

Merge pull request #19885 from eileenmcnaughton/acl_role

[REF] Remove redundant toArray() function

SearchKit - Add created & modified info to SavedSearch listing

Simplify setting of softcredit form options

I checked & the empty is just an enotice fix

Remove toArray function as it does the same as the parent

Merge pull request #19881 from eileenmcnaughton/task1

[REF] Store the getSearchQueryResults so it can be re-accessed

Merge pull request #19889 from colemanw/api4EntityNameCleanup

APIv4 - Standardize generic entity name resolution

Merge pull request #19875 from colemanw/contactSummaryCleanup

Reorganize contact summary template to support non-ajax tabs

APIv4 - Standardize generic entity name resolution so that classNames do not have to equal entityNames

This will allow the addition of a CiviCase API which cannot have the className "Case" because that is a reserved php keyword

Also cleans up other entities that use `static::class` in place of entityName because it's not a good pattern to replicate.

Merge pull request #19801 from eileenmcnaughton/mem_terms

[REF] Simplify code calculating the number of membership terms

Merge pull request #19884 from eileenmcnaughton/unit8less

Updates to ExportTest

Merge pull request #19818 from eileenmcnaughton/merge_fix

Fix for dedupe error under some mysql configs

Updates to ExportTest

Test fix to avoid caching issue

Merge pull request #19882 from eileenmcnaughton/acl_role

Fold back in getGroupACLs

Merge pull request #19817 from eileenmcnaughton/merge

Tighten mysql mode in test environment

Merge pull request #19880 from demeritcowboy/crmmoney2

dev/financial#171 - Don't pass already formatted data to crmMoney on contact summary for custom fields

Fold back in getGroupACLs

Merge pull request #19879 from eileenmcnaughton/acl_role

Remove getACLRoles

[REF] Fix the getSearchQueryResults so it can be called multiple times

don't pass already formatted data to crmMoney

Merge pull request #19860 from eileenmcnaughton/task1

Move setting of _includesSoftCredits out of getQuery

Merge pull request #19868 from colemanw/searchKitFixExtraFields

SearchKit - remove extra irrelevant fields from bridge joins

Merge pull request #19878 from civicrm/5.36

5.36

Remove getACLRoles

We have cleaned up the code to clarify that
1) it returns an empty array if contact id is set and
2) it is only called with contact_id is set

Ergo it does nada

Not called from anywhere else (I made it protected last
cleanup round)

Merge pull request #19872 from eileenmcnaughton/acl_role

[REF] Fold getACLs back into the calling function

Merge pull request #19867 from colemanw/civiCaseApi

APIv4 - Add function to get api class name

Merge pull request #19743 from totten/master-upg

dev/release#16 - Allow omission of empty upgrade steps

Merge pull request #19874 from demeritcowboy/blank-assignee

[WIP] dev/core#2473 - Don't crash when assignee left blank when creating followup on email activity

Merge pull request #19877 from eileenmcnaughton/camp

[REF] Stdise campaign_id handling

Merge pull request #19876 from demeritcowboy/dont

dev/financial#171 - Don't

[REF] Stdise campaign_id handling

don't

Reorganize contact summary template to support non-ajax tabs

There was some non-functional (due to a typo) code in TabHeader.tpl to allow tabs to load from a .tpl instead of ajax.
This fixes that typo and copies the funcionality into the contact summary tabs as well.
The functionality is demonstrated here by moving the main tab into a sub-tempate.

Merge pull request #19757 from eileenmcnaughton/tax_amount

[REF] Minor simplification around assigning taxAmount to the template

Merge pull request #19800 from eileenmcnaughton/gettypes

[REF] Remove setting of unused variables

crash when assignee left blank

Merge pull request #19797 from eileenmcnaughton/super_duper

Add new Super-duper-no-permissions-apply permission

dev/core#1845 Change FK on civicrm_group to delete the associated group if a saved search is deleted

Add in delete function on Group BAO to handle deleting associated saved search if appropriate when group is deleted and move upgrade step appropriately

Add unit test and move code to the discard function

Merge pull request #19764 from JMAConsulting/core-59

core#59: Address location type search errors when using a search profile

Merge pull request #19871 from eileenmcnaughton/acl

Return from getACLRoles when contactID is not null

[REF] Fold getACLs back into the calling function

Not called in the civi universe

Return from getACLRoles when contactID is null

I've made this patch the least possible code wise to allow brain space to think about the query....

Basically the where is WHERE acl.entity_table='civicrm_acl_role'....

if contact_id is set it becomes AND acl.entity_table = 'civicrm_contact'

Which means that it would only ever return empty results

(once this is merged more cleanup can happen but I wanted to
make this change easy to think through)

Merge pull request #19869 from eileenmcnaughton/acl

[REF] Simplify BAO_ACL table references

Remove exposure of implied_permissions beyond class

 Add new Super-duper-no-permissions-apply permission

We discussed in the context of search kit that there are 2 competing concepts of
'administer CiviCRM'
1) like drupal user 1, can do anything
2) role that has various administrative access but acls etc still apply

In search kit we have an interest in allowing users who won't do dumb things
  the ability to expose data to people who otherwise would not have access to that data
   - e.g to create a listing of event participants to expose to anonymous users.

This effectively means we are giving people the power to create displays
that set check_permissions to FALSE. This would potentially enable people
to not just bypass ACLs applied to others but also acls applied to them.
In order words it could be a privellege escallation.

To prevent any unexpected escallation we decided that this ability
should only be given to contacts who explicitly have access to everything
anyway. There is no existing permission that does this (although
there is a perception that there is)

[REF] Simplify BAO_ACL table references

It does not appear to me, from my testing that the value in entity_table could
be something other than the actual table name (in fact I suspect
it would only ever be civicrm_acl_role - I'm working to confirm that).

This simplifies and switches to executeQuery which is multilingual aware

Merge pull request #19829 from vingle/patch-11

Add .clear class to break inline flow on social share footer

Merge pull request #19716 from jitendrapurohit/lifetime

update to lifetime membership does not clear the end date

Merge pull request #19866 from eileenmcnaughton/nfc

[NFC] Cleanup in test class

Merge pull request #19660 from totten/master-afform-token

Afform - Add support for email tokens `{afform.myFormUrl}`

Merge pull request #19858 from eileenmcnaughton/loop

Fix loop

APIv4 - Add function to get api class name

Adding the "Case" entity is a problem because it's a reserved php keyword.
So we need to be able to have entity names that don't match their classname.
This function gives us a way to get api class names without guessing from the entity name.

SearchKit - remove extra irrelevant fields from bridge joins

Merge pull request #19856 from colemanw/customFieldLabels

SearchKit - Include custom group title with custom field label

Remove never-gonna-happen todos

Share function for identical code

Use custom data trait for custom data

Remove call to deprecated function

[NFC] Cleanup in test class

Merge pull request #19864 from eileenmcnaughton/nfc

[NFC] Minor cleanup on test class

Merge pull request #19861 from eileenmcnaughton/mem_tax

[REF] Simplify a couple of variables

[NFC] Minor cleanup on test class

Fix issue with no group by sql_mode

Tighten mysql mode in test environment

At least one test fails (and the corresponding bug appears on our live site)
with the mysql modes IGNORE_SPACE, ERROR_FOR_DIVISION_BY_ZERO and STRICT_TRANS

(I think any 2 of the above is OK)

Let's throw it at jenkins & see how many friends it has

Merge pull request #19603 from JMAConsulting/issue-2226

core#2226: Scheduled Reminder fails to send if From Name includes comma

Move setting of _includesSoftCredits out of getQuery

We shouldn't be doing extra setting within the function

Merge pull request #18466 from mattwire/scheduledJobFatalErrorHandler

Remove fatalErrorHandler from scheduled jobs manager

[REF] Simplify a couple of variables

Rather than rely on these being passed around retrieve them with
a consistent function.

Note tax_amount is retrieved one other place but that is fixed in
https://github.com/civicrm/civicrm-core/pull/19757

Line item will be used less once
https://github.com/civicrm/civicrm-core/pull/19801 is merged

Merge pull request #19857 from eileenmcnaughton/task1

[REF] Extract getQueryParams

Merge pull request #19859 from civicrm/5.36

5.36

Merge pull request #19728 from totten/master-authx-guards

(dev/core#2077) AuthX - Allow optional "guards"

Merge pull request #19812 from eileenmcnaughton/recur

[REF] Clarify & simplify is_recur param

SearchKit - Include custom group title with custom field label

This makes it easier to tell custom fields apart in SearchKit when they have similar names to core or other custom fields.

This takes advantage of API4.getFields returning both a "title" and "label" attribute.
This keeps "title" the same while appending group title to "label".

Merge pull request #19825 from colemanw/api4BridgeJoinSubquery

APIv4 - Use subquery to LEFT JOIN via a bridge entity

Fix loop

It turns out that if you 'break' advanced search (in my case
I applied a patch that caused invalid sql) it enters a loop.

I really can't see why 'while' would have ever made sense here.

There is nothing to cause it to 'move along'

Extract getQueryParams

SearchKit - Add "EXCLUDE" join type, to search for entities that do not have a relationship to another entity

APIv4 - Use subquery to LEFT JOIN via a bridge entity

Bridge entities are supposed to be transparent in APIv4, but when LEFT joining,
an artifact of the double-join would give extraneous results.

APIv4 - Simplify long function by extracting part of it

Merge pull request #19828 from eileenmcnaughton/act_contact_hook

dev/core#2390 Add hook support for Activity Contact

Merge pull request #19842 from eileenmcnaughton/task

[unreleased regression] Fix export contribution to share tasktrait

Merge pull request #19853 from colemanw/api4ExplorerCssFix

Api4 Explorer - Prevent long results from breaking layout

Fix ts to E::ts

authx - If 'authx_guards' is set, then enforce them

authx - Declare setting "authx_guards"