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

info.xml - Allow PSR-0 style class-loader declarations

Before
------

If an extension defines a class in the style of `CRM_Foo_Bar`, then it
*must* use `hook_civicrm_config` and modify the `include_path`.

```php
function mymod_civicrm_config(&$config = NULL) {
  $extRoot = dirname(__FILE__) . DIRECTORY_SEPARATOR;
  $include_path = $extRoot . PATH_SEPARATOR . get_include_path();
  set_include_path($include_path);
}
```

Consequently:

* The `include_path` gets rather long.
* Setting up the extension class-loader is not sufficient for loading these classes. You must
  wait until the DB has initialized and the `hook_civicrm_config` has fired.

After
-----

An extension *may* setup classloading in `info.xml`, e.g.

```xml
<classloader>
  <psr0 prefix="CRM_" path="" />
</classloader>
```

If the `<psr0>` style is used, then it is possible to scan the extension
classes without fully booting the extension.  This is a pre-requisite for
using interfaces, annotations, or similar for event-registration.

Comments
--------

* This style closely parallels the `<psr4>` support in `info.xml`, and it parallels the semantics of `composer.json`.
* It is still valid for an extension to set `include_path` -- that works the same as ever.

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

Update flexmailer default

Switches to defaulting to flexmailer, if enabled.

This should be merged in conjunction with https://github.com/civicrm/civicrm-core/pull/19806
in order to keep those on the default in consistent state
(ie they will already be getting display-rendering for custom fields
and that will still be the case if 19806 is merged)

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"

Merge pull request #19826 from demeritcowboy/nomulticase

dev/core#2463 - Remove loop that since at least 5.23 does nothing for single cases and crashes for multiple, and for multiple recipients only uses the last activity id for 5.36+

Api4 Explorer - Prevent long results from breaking layout

Merge pull request #19852 from eileenmcnaughton/nfc_test

[NFC] Minor test cleanup

Merge pull request #19851 from demeritcowboy/ids-php74

dev/core#2366 - Failing test for php 7.4 and IDS for non-admins

[NFC] Minor test cleanup

Merge pull request #19850 from seamuslee001/mysql8_logging_test_fix

[NFC] Fix logging schema test to work on MySQL8 as it no longer outpu…

failing test for php 7.4 and IDS

[NFC] Fix logging schema test to work on MySQL8 as it no longer outputs a length for int columns

Merge pull request #19849 from seamuslee001/member_event_form_test_fixes

[NFC] Fix test failures on PHP 7.4 caused by either NULL being set fo…

Merge pull request #19847 from seamuslee001/fix_test_failures_mysql8

[NFC] Fix Test failures on MySQL 8 caused by change in output of eith…

Merge pull request #19846 from seamuslee001/fix_contribution_page_tests

[NFC] Fix PHP 7.4 errors on ContributionPage tests

Merge pull request #19845 from seamuslee001/fix_membership_type_test

[NFC] Fix Membership Type Test error on php7.4

Merge pull request #19844 from seamuslee001/php74_membership_api_fix

[REF] Fix APIv3 MembershipJob test failure on php7.4 by only doing ch…

[NFC] Fix test failures on PHP 7.4 caused by either NULL being set for the membership_type_id or Payment Processor not being set on the form

[NFC] Fix Test failures on MySQL 8 caused by change in output of either create table or Column names in queries

[NFC] Fix PHP 7.4 errors on ContributionPage tests

[NFC] Fix Membership Type Test error on php7.4

[REF] Fix APIv3 MembershipJob test failure on php7.4 by only doing checking  if it is not empty

Fix export contribution to share tasktrait

Merge pull request #19787 from demeritcowboy/cancel-button

dev/financial#170 - Cancel button on financial batch export is no longer an input so code needs updating

Merge pull request #19839 from tschuettler/core_2466

(dev/core#2466) - Drop HTML markup in exports for link custom fields

Merge pull request #19834 from eileenmcnaughton/seq

Code cleanup in bootstrap.php

Merge pull request #19835 from eileenmcnaughton/task

[REF] Minor function extraction

Drop HTML markup in exports for link custom fields

Merge pull request #19838 from eileenmcnaughton/nfc

[NFC] Minor coding standards cleanup

Merge pull request #19836 from eileenmcnaughton/legacy

Remove legacy phpunit param

[NFC] Minor coding standards cleanup

[REF] Minor function extraction

Merge pull request #19833 from eileenmcnaughton/test

Remove safe+mode setting

Remove legacy phpunit param

Merge pull request #19832 from seamuslee001/core_ext_void

[NFC] Add in Void statements on varios Core Ext Tests to future support PHP…

Merge pull request #19624 from agileware/CIVICRM-1670

Issue #2384 - Remove text about email receipts for each recurring contribution

Code cleanup in bootstrap.php

Remove safe+mode setting

This is php5.4 compatibility = not needed

Add in Void statements on varios Core Ext Tests to future support PHPUnit8

Merge pull request #19831 from colemanw/angularUpgrade

Upgrade angularJs from 1.8.0 to 1.8.2

Merge pull request #19753 from eileenmcnaughton/format

dev/core#1494 & dev/translation#65 call brickmoney number formatting, where supported

Merge pull request #19830 from kewljuice/patch-6

Fix deprecated Redis::delete() (issue #2468)

Upgrade angularJs from 1.8.0 to 1.8.2

Fix deprecated Redis::delete() (issue #2468)

Add .clear class to override left float on buttons

.crm-button has a float: left, which was added in #18880, and can in some themes cause the subsequent lines to flow back inline. This fixes that (and adds a little padding under the buttons).

Merge pull request #19827 from civicrm/5.36

5.36

dev/core#2390 Add hook support for Activity Contact

Alternative to https://github.com/civicrm/civicrm-core/pull/19623
per comments on that PR.

Add release-notes/5.35.1

(security/core#105) Joomla::authenticate() - Fix escaping

(security/core#105) Joomla::checkUserNameEmailExists - Fix mismatched escaping

This uses the escaping rule from CRM_Core_DAO to construct a query for JDatabaseDriver.
However, they use different connections and (therefore) could require different
escaping rules.

(security/core#104) CRM_Utils_System::authenticateKey - Use secure equality test

(security/core#97) PHP CLI guard is the opposite of correct

Escape api params in APIv4 Explorer

Purify PCP introductory text field

Escape information supplied by extensions to prevent XSS