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

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

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

SearchKit Toolbar - Fix conditionals, add tests

Merge pull request #27525 from larssandergreen/Remove-spaces-around-curly-braces-to-prepare-for-Smarty-3+

Remove spaces around curly braces to prepare for Smarty 3+

Remove spaces around curly braces to prepare for Smarty 3+

PHP 8.x always assign taxTerm, ppID

Merge pull request #27521 from civicrm/5.66

5.66

Merge pull request #27520 from colemanw/dfk

APIv4 - Add dfk metadata to getfields

Merge pull request #27476 from eileenmcnaughton/part_amounts

Add test cover, use token for `event.is_share`

APIv4 - Hack around bad dfk metadata in unit tests

APIv4 - Add dfk metadata to getfields

(mostly) fixes some bad guesswork in the APIv4 conformance test,
and now the dfk_entities metadata is available in getFields

Merge pull request #27487 from eileenmcnaughton/pay_later

Fix notices in text versions of event receipts

Merge pull request #27518 from demeritcowboy/bread

dev/core#4605 - Add yet another guard against failed upgrades

Merge pull request #27517 from eileenmcnaughton/custom_param

Move nasty chunk of code to where it is used

guard during upgrades

Merge pull request #27516 from eileenmcnaughton/participant82

Php8.x Fix notices when creating custom field

Move nasty chunk of code to where it is used

This code to get the customFields array is tough going. I haven't unravelled it, just moved it from being
figured out in one function & used in another such that it is figured out in the
chunk of code it is used in.

This will probably fix some php 8.2 tests because rather than all tests
hitting it only those that actually have custom fields + receipts do.

It's a bit of a fake win but it will reduce fails & make further cleanup easier

Php8.x Fix notices when creating custom field

Merge pull request #27514 from eileenmcnaughton/regen

Run regen

Run regen

Merge pull request #27510 from totten/master-event-dao

Regenerate CRM_Event_DAO_Event

Fix notices in text versions of event receipts

Pretty minor change but gets past the notices

Regenerate CRM_Event_DAO_Event

Recently changed by #27478

Merge pull request #27478 from eileenmcnaughton/carty

Move Conference slots functionality to event cart

Merge pull request #27482 from eileenmcnaughton/show_cms

Consolidate Profile Preview with other profile code

Merge pull request #27488 from eileenmcnaughton/event_notice_custom_post

php 8.x Ensure profiles are assigned to template, even when empty

Merge pull request #27497 from eileenmcnaughton/sessions_offline

Remove event cart sessions handling from offline event receipt text as gone from html

Merge pull request #27502 from eileenmcnaughton/legacy

Fix legacy custom searches to not use hacky method to access protected property on core class

Merge pull request #27509 from eileenmcnaughton/renew

php8.2 stop using yet-another undeclared property for _membershipId.

Merge pull request #27503 from eileenmcnaughton/getvar

Add deprecation annotations for getVar & setVar

Merge pull request #27508 from civicrm/5.66

5.66

Merge pull request #27507 from eileenmcnaughton/survey

Remove getVar call, access protected property in same class directly

php8.2 stop using yet-another undeclared property for _membershipId.

Instead follow our new practice of having consistent (externally supported)
functions for getting the main id variables

Merge pull request #27475 from totten/5.66-comp-status

Status Check - Ensure that components are enabled