Merge pull request #27530 from eileenmcnaughton/misc

php8.x Fix notice from adding esm_loader to wrong settings form

Merge pull request #27576 from colemanw/enter

SearchKit - Save In-place edit when pressing enter

Merge pull request #27563 from eileenmcnaughton/pro

Remove unreachable code

SearchKit - Save In-place edit when pressing enter

Merge pull request #27572 from larssandergreen/Delete-unneeded-test-file

Remove unintentionally added test file

Remove unintentionally added test file

Merge pull request #27571 from civicrm/5.66

5.66

Merge pull request #27567 from eileenmcnaughton/action

Loosen action typing - doing nothing is doing something

Merge pull request #27560 from eileenmcnaughton/event_type

Stop using undefined properties to specify custom data extends values

Merge pull request #27566 from eileenmcnaughton/price

Fix weight values on PriceSet page

Merge pull request #27316 from lemniscus/oauth-session-token

Implement OAuthSessionToken

Merge pull request #27570 from jensschuppe/financialItemEntityTableCampaign

Allow `civicrm_campaign` as `entity_table` for `FinancialItem` API entity

Merge pull request #27568 from eileenmcnaughton/petition

Fix notice on 'new petition' page

Allow `civicrm_campaign` as `entity_table` for `FinancialItem` API entity

Fix notice on 'new petition' page

Loosen action typing - doing nothing is doing something

Fix weight values on PriceSet page

Remove unreachable code

The ContributionRecur::add function throws an error if there
is a problem - it doesn't return an error

update function signature to match that in parent class (recently changed)

replace deleteAll with a delete action, no "where" required

make cardinal more reliable; add a little more info to getFields

make cardinal more reliable; add a little more info to getFields

improve PHPDocs

use getEntityName instead of constant

PHPDoc for OAuthSessionToken class

Co-authored-by: colemanw <coleman@civicrm.org>

minor improvements, and add a provider for manual testing

implement OAuthSessionToken, with tests

Merge pull request #27558 from colemanw/noPaths

APIv4 - Remove function that doesn't do anything, add some type hints

Stop using undefined properties to specify custom data extends values

Merge pull request #27552 from eileenmcnaughton/custom_field_trait

Fix assigning of customGroup data to be more re-usable & use undefined properties less

APIv4 - Remove function that doesn't do anything, add some type hints

Merge pull request #27495 from eileenmcnaughton/action_notice

Cast _action to an int

Merge pull request #27555 from larssandergreen/Fix-{php}-in-hlp-test

Remove {php} in hlp test

Merge pull request #27554 from demeritcowboy/quick-safe

dev/core#4616 - Fix js syntax error on contribution page / nonfunctioning honoree toggle

Fix assigning of customGroup data to be more re-usable

Remove {php} in hlp test

Merge pull request #27535 from eileenmcnaughton/custom

Simplify CustomField::displayValue to expect an ID & have deprecated handling for anything else

Merge pull request #27532 from eileenmcnaughton/verify_1

Pass `NULL` instead of a value where function requires a parameter but does not use it

Merge pull request #27474 from eileenmcnaughton/billing_field

php8.x Remove never-used undeclared property

Merge pull request #27429 from eileenmcnaughton/ref

Fix unnecessary pass-by-ref, minor code improvements

Merge pull request #27471 from eileenmcnaughton/smartY

Fix Smarty secure mode for premiums tab

Merge pull request #27470 from eileenmcnaughton/contribution_form

PHP 8.x always assign taxTerm, ppID - backoffice contribution form

js syntax error when false

Case _action to an int

There is a lot of code comparing to action which is unsure whether it
is dealing with 2 or '2'. It should be possible to always cast to an int but
I have added deprecation just in case...

Merge pull request #27016 from eileenmcnaughton/tax

Use token for tax_amount, quick config mode

Merge pull request #27551 from civicrm/5.66

5.66

Simplify CustomField::displayValue to expect an ID & have deprecated handline for anything else

Merge pull request #27550 from seamuslee001/ca_config_update

Update CAConfig to use 23.07 version

Update CAConfig to use 23.07 version

Merge pull request #27522 from eileenmcnaughton/billing

PHP8.2 Pass variables rather than co-erce properties when caling `AbstractEditPayment:processBillingAddress`

Merge pull request #27548 from larssandergreen/Remove-templates/CRM/Price/Form/LineItem.tpl

Remove unused templates/CRM/Price/Form/LineItem.tpl

Merge pull request #27540 from eileenmcnaughton/membership_type_edit

[php8.x[ Remove notice-causing code that seems unreachable

Remove unused templates/CRM/Price/Form/LineItem.tpl

Set variables when calling customDataBlock.tpl from MembershipType

Remove notice-causing code that seems unreachable

I really can't find any way this code could be triggered - I think it
might have been quietly broken for years...

Merge pull request #26529 from samuelsov/adminui3

civicrm_admin_ui: add Profile Copy action

civicrm_admin_ui: add Profile Copy action

Merge pull request #27546 from demeritcowboy/double-honor

Fix double fieldset for honoree section

Merge pull request #27545 from civicrm/5.66

5.66

Merge pull request #27541 from eileenmcnaughton/cid_notice

Always assign noCid when the tpl is included

Merge pull request #27539 from eileenmcnaughton/class_notice

Remove some copy & paste...

true

Merge pull request #27446 from demeritcowboy/upgrade-multi-default-566

Add missing upgrade script for multilingual defaults

Merge pull request #27544 from eileenmcnaughton/badge56

Add weights on badge layout links

Add weights on badge layout links

Merge pull request #27513 from eileenmcnaughton/mailing

dev/core#4567 Add mailing_id, is_test to civicrm_mailing_event_queue

Merge pull request #27542 from civicrm/5.66

5.66

Always assign noCid when the tpl is included

Fixes notices by moving the assign out of the function that adds the form elements

This one only matters when smarty is in secure mode:

Merge pull request #27483 from eileenmcnaughton/8_notice

Php 8.x clean up notices on Profiles

Remove some copy & paste...

Class is never in the rows array - this is copy & paste

Merge pull request #27533 from eileenmcnaughton/hash_less

CiviMail - Generate email auth-code as random string

php8.x Fix notices on displaying FieldSet

Stop using mode as magic for showing the fieldset

Loosen regex

Merge pull request #27536 from totten/master-getclass

PHP 7.x - Use get_class() instead of $object::class

Merge pull request #27537 from eileenmcnaughton/recur

Fix mistake from last week on string wrangling

Fix mistake from last week on string wrangling

This came from
https://github.com/civicrm/civicrm-core/pull/27419

PHP 7.x - Use get_class() instead of $object::class

CiviMail - Fixup edge-case chars. Use consistent function.

1. Base64 can output some funny chars that are awkward in VERP and URL contexts.
   Use strict lowercase alphanumeric.
2. The new code was being used for the test mailing - but not for real mailings. They should use same formula.

Merge pull request #27409 from seamuslee001/5.66

[REF] Fix bug caused by recent ACL changes where combination of edit …

Use random string rather than sha string

As discussed on chat
https://chat.civicrm.org/civicrm/pl/r7h6cc7xo78zjgsd8af6ff9tee

this function just generates a hard-to-guess string that is stored
in the civicrm_mailing_event_bounce table. This unique value is
used in conjunction with the unique id value from the table to
determine if it is a match but it's not ever reverse calculated
so we don't need to use a reversable function

Merge pull request #27472 from colemanw/noteForm

API - Improve ACL processing, deprecate `hook_civicrm_notePrivacy`

Merge pull request #27534 from artfulrobot/artfulrobot-fix-group-removal-count

Stop saying N contacts removed when they were already removed.

Stop saying N contacts removed when they were already removed.

Add userId and conditions to hook_civicrm_selectWhereClause signature

Note - Deprecate hook_civicrm_notePrivacy in favor of hook_civicrm_selectWhereClause

This enables note privacy to be enforced by SearchKit and the API,
and gives an upgrade path for implementations of hook_civicrm_notePrivacy to be refactored out.

DAO/APIv4 - Optimize ACL clauses

Before: no values passed to BAO::addSelectWhereClause

After: $entityName and $conditions passed.
$entityName is rarely needed, but useful for dynamic entities like ECK.
$conditions contains field/value pairs gleaned from the WHERE clause or ON clause
  (depending on how the entity was added to the query)

API/DAO - Improve permissioning sql clauses: support field names and OR

Before: selectWhereClause was limited to returning arrays of clauses which would be
joined with AND. They could not reference any other fields on the entity.

After: selectWhereClause can return sub-arrays which are joined with OR,
and can reference any field on the entity using {curly_brace} syntax.

Stricter type checking emits noisy deprecations if arrays are expected and strings are given.

DAO - addSelectWhereClause should always return an array

Activity - More accurate calculation of permittedActivityTypes

The strict array key comparison seemed wrong, array_diff_key works better.
I was hoping this would be more efficient by skipping the where clause for
privledge users, but it doesn't. Oh well at least this is more correct.

Make Note.privacy required

Stop passing now-ignored job_id to verify function

Merge pull request #27531 from eileenmcnaughton/verify

CiviMail - VERP should no longer require job_id for verification

Fix verify verp function to cope with job_id having been deleted

This allows the record to be validated based on queue_id & hash
(which is still pretty insane to brute force & allows
us to purge job records

Add mailing_id, is_test to civicrm_mailing_event_queue

Fix notice from adding esm_loader to wrong form

It is on the debugging form - but perhaps it was gonna go here for a bit

Merge pull request #27529 from civicrm/5.66

5.66 to master

Merge pull request #27528 from demeritcowboy/weight-again

Fix weights again on contribution page settings dropdown

PHP8.2 Pass variables rather than co-erce properties when caling AbstractEditPayment:processBillingAddress

fix weights again

Merge pull request #27527 from colemanw/searchToolbar

SearchKit Toolbar - Fix conditionals, add tests