Remove unnecessary pass-by-reference

Merge pull request #21891 from colemanw/initHideBoxes

Export - Fix undefined tpl var

Export - Fix undefined tpl var

This varable was a string of javascript leftover from the old export UI,
no longer used or needed by the new Angular UI.

Merge pull request #21868 from yashodha/typo

typo fixes

typo fixes

Merge pull request #21867 from civicrm/5.43

5.43 to master

Merge pull request #21866 from seamuslee001/fix_pdf_test

[REF] Fix PDF Test failure on php versions 7.4 and later

[REF] Fix PDF Test failure on php versions 7.4 and later

Merge pull request #21864 from civicrm/5.43

5.43 to master

Merge pull request #21851 from colemanw/entityTypesHookRecursionFix

Entity types hook recursion fix

Merge pull request #21856 from civicrm/5.43

5.43

Merge pull request #21845 from colemanw/relationship_cache_case_id

RelationshipCache - Add case_id column

RelationshipCache - Add case_id column

Merge pull request #21808 from braders/feature/pcp-alt-text

Add alt attribute to image on PCPInfo template.

Merge pull request #21852 from demeritcowboy/extension-name

dev/core#2876 - When installing an extension that has requirements, show the name not the key

Add alt attribute to image on PCPInfo template.

It is an accessibility best-practice that all images should have alt text.

If a description has been provided, we use that text,
otherwise an empty alt text is used (i.e. the image is treated as decorative)

Merge pull request #21838 from christianwach/lab-core-2917

Restrict "Oops" to administrative errors

show extension name not key

Merge pull request #21827 from eileenmcnaughton/sent

dev/core#2914 Fix for incorrect sent count in message

Cleanup ContactType BAO fixme and comments; use common function

This fixes the fixme in basicTypeInfo to use getAllContactTypes instead of running its
own query, and cleans up some comments.

Fix possible recursion during Entity::get()

Use plain SQL instead of APIv4 in a function that is called by APIv4 Entity.get
to avoid an infinite loop.

Merge pull request #21849 from eileenmcnaughton/testy

Fix test to use postProcess flow

Fix test to use postProcess flow

dev/core#2914 Fix for incorrect sent count in message

Merge pull request #21846 from eileenmcnaughton/nfc

Preparatory cleanup in test class

Preparatory cleanup in test class

Merge pull request #21839 from jensschuppe/api4CreateRequestEvent

Load the API kernel service prior to dispatching the `civi.api4.createRequest` event when creating an API4 request

Merge pull request #21841 from braders/feature/core-2913

Use SVG for 'Empowered by' logo

Merge pull request #21843 from braders/feature/translatable-dedupe-placeholders

Remove use of hardcoded strings in Dedupe templates

Merge pull request #21844 from demeritcowboy/repourl

dev/core#2903 - Put back missing placeholder

APIv4 - Add component metadata to Relationship.case_id field

This will hide that field when CiviCase is disabled

put back missing placeholder

Merge pull request #21842 from braders/feature/transalte-hardcoded-placeholder-strings

Makes hardcoded placeholder strings translatable

Remove use of hardcoded strings in Dedupe templates

Makes hardcoded placeholder strings translatable

Restrict "Oops" to administrative errors

Merge pull request #21836 from eileenmcnaughton/hook_toke

Fix hookTokens to be clearable outside the class

Merge pull request #21840 from braders/feature/core-2903

Remove reference to CiviCRM forum in HTTP error message

Use SVG for 'Empowered by' logo

Merge pull request #21811 from jmcclelland/urlencodesocial

ensure events with single quotes can be share via social

Remove reference to CiviCRM forum in HTTP error message

Replace with instruction to contact the site administrator for assistance.

Load the API kernel service prior to dispatching the `civi.api4.createRequest` event when creating an API4 request

Merge pull request #21837 from totten/5.43-nfc-escape

(NFC) TokenConsistencyTest - Add some examples of HTML escaping

(NFC) TokenConsistencyTest - More aggressive test cleanup

Merge pull request #21833 from eileenmcnaughton/test_cache

Stop flushing caches on enable civiCampaign

Merge pull request #21816 from eileenmcnaughton/pdf2

dev/core#2864 Remove call to getTokenDetails from Contribute pdf letter

(NFC) TokenConsistencyTest - Add some examples of HTML escaping

Fix hookTokens to be clearable outside the class

Tests can't unset the proerty in the class easily but just using the static makes it easier

Merge pull request #21832 from civicrm/5.43

5.43

Merge pull request #21807 from braders/feature/replace-spacer-img-elements

Remove use of spacer.gif img elements in CRM_Utils_Weight

Merge pull request #21830 from totten/5.43-regen

(dev/core#2836) regen.sh - Fix incorrect reference to "flexmailer". Update data file.

Remove call to getTokenDetails from Contribute pdf letter

On digging this is just a glorifed api call. The parameters loaded are only used in emailLetter for filtering

Stop flushing caches on enable civiCampaign

We should be able to get the 'right' cache clearing via the core function.

Merge pull request #21828 from colemanw/api4RecursionFix

APIv4 - Fix possible recursion during Entity::get()

Merge pull request #21831 from seamuslee001/5.43

Add release-notes/5.42.1.md

Add release-notes/5.42.1.md

Merge pull request #21826 from eileenmcnaughton/cancellabel

Fix translation regression in recurring edit workflow template

Regenerate `sql/civicrm_generated.mysql`

bin/regen.sh - Fix misnamed extension

The script tries to filter the list of extensions that wind up in the sample data-set.

The filter list targets `full_name` (ie long name), but some of the values are inaccurate.

APIv4 - Fix possible recursion during Entity::get()

Fix translation regression in recurring edit workflow template

Merge pull request #21825 from civicrm/5.43

5.43 to master

Merge pull request #21824 from eileenmcnaughton/getfields

Use preferred entity name (Website not website)

Use preferred entity name (Website not website)

Merge pull request #21823 from civicrm/5.43

5.43 to master

Merge pull request #21815 from demeritcowboy/case-token

dev/core#2895 - Failing test demonstrating case token not resolved

Merge pull request #21821 from totten/5.43-event-actres

(REF) Tokens - Populate "Event"/"Participant" without direct access to actionSearchResult

ensure events with single quotes can be share via social

Merge pull request #21804 from eileenmcnaughton/pdf

[REF] Paramter simplification

test demonstrating failure to resolve token

Merge pull request #21822 from civicrm/5.43

5.43 to master

Merge pull request #21818 from alifrumin/rn5.43

[NFC] First Pass Release notes 5.43

Merge pull request #21819 from colemanw/APIv4EntityEvent2

REF - Fix loop in APIv4 Entity.get to only load ad-hoc entities once

Merge pull request #21812 from eileenmcnaughton/gdpr_tokens

dev/core#2907 fix for gdpr clobering contact tokens

(REF) ParticipantTokens - Simplify

The previous commit passes `eventId` through the front-door of `$context`,
so we shouldn't need as much special effort to bring it in the side-door.

(REF) Tokens - Populate "Event"/"Participant" without direct access to actionSearchResult

Overview
--------

When sending a scheduled remidner for a `civicrm_participant` record, it supports
tokens for both `{participant.*}` (`$context['participantId']`) and `{event.*}`
(`$context['eventId']`). This creates a special requirement to load the `eventId`
in addition to the `participantId`.

This patch changes the way in which `eventId` is loaded.

Before
------

In `ParticipantToken`, the token-evaluation-step for `{participant.*}`
tokens has a side-effect of copying `$context['eventId']` based on reading
`actionSearchResult` (although this appears indirect).

After
-----

The action-search returns `tokenContext_eventId`, which automatically maps
to `$context['eventId']`.

This means that the `eventId` will be set before evaluation begins.

[NFC] first pass at 5.43 release notes

REF - Fix loop in APIv4 Entity.get to only load ad-hoc entities once

Merge pull request #21817 from civicrm/5.43

5.43 to master

dev/core#2907 fix for gdpr clobering contact tokens

This is a fix for contact tokens not rendering when gdpr tokens is installed.

Gdpr declares some addtional tokens with the namespace 'contact'. I am inclined to
agree this counts as 'naughty' and so while I have fixed the clobbering by
not processing contact-metadata tokens I have not fixed the enotices.

If we think it is non-naughty I can kill those - but I've already loosened
some strictness that I think had benefits in order to facilitate this

Merge pull request #21803 from colemanw/APIv4EntityEvent

[REF] APIv4 - Dispatch event during Entity.get

Merge pull request #21801 from eileenmcnaughton/case1

dev/core#2895 handle case ids passed via url

Merge pull request #21814 from civicrm/5.43

5.43 to master

Merge pull request #21806 from totten/5.43-as-ctx

EntityTokens - Normalize data-loading via myEntityId (eg activityId)

APIv4 - Dispatch event during Entity.get

This allows extensions to modify the list of entities,
enabling "virtual" entities not based on php files.

dev/core#2895 handle case ids passed via url

Merge pull request #21810 from braders/feature/missing-translations

Make hardcoded strings translatable.

Update to remove use of spacer.gif in ordering component

Merge pull request #21800 from civicrm/5.43

5.43

Make hardcoded strings translatable.

Wraps previously hardcoded strings in {ts} in Smarty .tpl files.

Add alt text to spacer elements

All HTML img elements should contain an alt attribute.
For decorative or spacer elements this alt text should be a blank string,
which indicates to assistive technologies that the image should not be announced.

Longer-term these spacer gifs could be replaced with empty <span> elements.
Inline-block css has been added to allow for this change in future,
however, the HTML element has not been changed for now to avoid breakage with non-default themes
(e.g. civicrm-admin-utilities WordPress plugin which replaces the default css/civicrm.css file)

(NFC) MailingQueryEvent - Update docblock

EntityTokens - getFieldValue() does not need to support actionSearchResult

EntityTokens - Normalize data-loading via `myEntityId` (eg `activityId`)

Before
======

EntityTokens has two protocols for fetching data:

1. For scheduled-reminders, hook into the main query. Add N-ary columns to select
   any fields used by tokens.
2. For everything else, expect a parameter `$row->context['activityId']`. Collect
   the `activityId`s and fetch the needful via APIv4.

This poses a few challenges, but TLDR it's duplicate functionality.  That
means that listeners who consume the `activityId` must check multiple
places.  It means that the data-loading could return slightly different data
(ie based on SQL query vs API data-loader).

After
=====

* EntityTokens has one protocol for fetching data: it must receive `activityId`s and then
  call APIv4 for the batch.
* For scheduled-reminders, EntityTokens uses a small adapter to ensure that `activityId`s
  are available.

Merge pull request #21783 from eileenmcnaughton/money

Use money formatting for currency in templates

[REF] Paramter simplification

Merge pull request #21796 from eileenmcnaughton/543

dev/core#2904 ensure skipStatusCal not passed from the order api

Merge pull request #21784 from eileenmcnaughton/api3

dev/core#2902 revert apiv3 handling of membership

Merge pull request #21799 from eileenmcnaughton/refs

Do not find references for empty values

dev/core#2904 Add tests for automatic date calculation for Memberships in Order API.