Merge pull request #20752 from samuelsov/ReportGroup

Make report subtotals work for custom fields

Merge pull request #20989 from mattwire/mailtasks

Fix mailing task when there are no tasks

Merge pull request #21036 from eileenmcnaughton/act_mon

Reconcile contribution amount tokens

Reconcile contribution amount tokens

This adds non_deductible_amount, as identified in https://github.com/civicrm/civicrm-core/pull/21034

It also moves the metadata functions to a parent class - this class might wind up
being a temporary class as there are 3 other contenders for where this functionality
should go - the TokenTrait, the abstractProvider or the tokenRow class. The intent
is not to make that decision yet - just to provide visual clarity about
what is generic

Note - I've figured out how to switch over to v4 api for this but will do that in a separate PR

Merge pull request #21037 from eileenmcnaughton/act_source

Fix advertised token for source

Merge pull request #21035 from eileenmcnaughton/act_more

Update contribution_page_id token in sched reminders

Update contribution_page_id token in sched reminders

Fix advertised token for source

Merge pull request #21000 from eileenmcnaughton/act_more

 Reconcile and upgrade tokens for payment instrument, financial type

Reconcile and upgrade tokens for payment instrument, financial type

This switches the last 2 'alias' tokens from Contribute_Tokens to the preferred format,
adds tests & upgrade and switches the advertised tokens. I tested & the token
removed from the sent letter collection still works

Next step is to analyse the remaining differences between the 2 sets of tokens.
My suspicion is that we have enough metadata that we use in search kit that we
don't need a hard-coded white list of tokens in 2 different places & can just
expose 'all the fields' that are not acl or otherwise limited

Merge pull request #20999 from eileenmcnaughton/act_up

Switch contribution action schedule tokens to use advertised tokens for cancel_date & source

Merge pull request #21019 from ejegg/customFieldBulkLog

Fix bulk adding custom fields with logging on

Merge pull request #21033 from kurund/patch-2

Update contributor-key.yml

Update contributor-key.yml

Merge pull request #21032 from seamuslee001/master

5.40 up merge

Merge in 5.40

Merge pull request #21031 from totten/5.40-bump0

5.40.0 - Version bump. More release notes.

Set version to 5.40.0

Tweak release-notes/5.40.0.md

Merge pull request #21028 from eileenmcnaughton/540

dev/event#62 - Expose all price fields to backend form (5.40)

Add release-notes/5.39.1.md

dev/event#62 expose all price fields to backend form

Merge pull request #21020 from colemanw/fixWPUrlHash

[REGRESSION] Fix CRM.url to not encode hash on WP

Merge pull request #21014 from eileenmcnaughton/pf

dev/core#2739 - Fix contribution tasks using wrong IDs

Merge pull request #21012 from eileenmcnaughton/540

dev/core#2725 Fix regression permitting circular group resolution

Merge pull request #21022 from eileenmcnaughton/sched

dev/core#2735 fix for effective start & end date not updating

Fix CRM.url to not encode hash on WP

dev/core#2735 fix for effective start & end date not updating

dev/core#2725 Fix regression permitting circular group resolution

Merge pull request #21005 from colemanw/afformFieldDefaults

Afform - support default values for fields

Merge pull request #21017 from totten/5.40-captcha-link

dev/core#2728 - ReCAPTCHA - Consistently present link to settings

Merge pull request #21016 from eileenmcnaughton/jamie

Temporary revert of changes in 20002 which caused a regression

Fix adding many custom fields with logging on

This code is only triggered in the 'bulk' writeRecords codepath,
not in the createField codepath which is used for creating custom
fields via the UI.

To replicate: on a wmff build, turn loggin on, check out
https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/709873
and run drush update-custom-fields

Result prior to this patch: SQL syntax error message and new field
not added to logging table

Result with this patch: no error message, new field created on log
table.

Merge pull request #21015 from civicrm/5.40

5.40

dev/core#2728 - ReCAPTCHA - Consistently present link to settings

See also: https://lab.civicrm.org/dev/core/-/issues/2728

Before
------

"ReCAPTCHA Settings" appears in "Customize Data and Screens" in the nav-menu, and it is missing in the `civicrm/admin` dashboard.

After
-----

"ReCAPTCHA Settings" appears "System Settings" for both the nav-menu and the `civicrm/admin` dashboard.

Afform - support default values for fields

This adds an "afform_default" property to the field definition.
It does not use the "default_value" property from getFields because that's more
to do with Civi's schema and often not appropriate for a form.
Fixes dev/core#2734

Avoid test failures on revert

Revert "Issue 53 (2)"

This reverts commit 4f4fb809a77cdfb7876d662f331d524fb2eeb1b3.

Add related contact display test

Merge pull request #21013 from seamuslee001/apiv4_utf8mb4

[NFC] Fix APIv4 Utf8mb4 test to ensure no mixed collation errors

Merge pull request #20929 from eileenmcnaughton/5.40

dev/wordpress#109 fix bug when is_monetary is not checked

dev/core#2739 - Fix contribution tasks using wrong IDs

This fixes an issue where contribution tasks may perform actions
on unrelated contribution IDs when they're called via contact
search forms.

Merge pull request #21008 from jmcclelland/grant-pagination

enable paging if contact has more then 50 grants.

[NFC] Fix APIv4 Utf8mb4 test to ensure no mixed collation errors

Merge pull request #21011 from seamuslee001/master

5.40

Merge branch '5.40'

Merge pull request #21009 from agh1/5.40.0-releasenotes-final

5.40.0 release notes - final edits

5.40.0 release notes: reorganized and final edits

5.40.0 release notes: added late changes

enable paging if contact has more then 50 grants.

Release notes: Contributor key no longer needs Jira handles

It's Andie ;)

Merge pull request #21003 from eileenmcnaughton/swapsies

Use template variable to add or remove swap button.

Merge pull request #20990 from eileenmcnaughton/cs

[Ref] [Test] Move custom search tests to extension

Merge pull request #20996 from colemanw/fixAfformContactType

Afform - Use correct contact type when loading field options

Merge pull request #21004 from seamuslee001/teardownftsquery

[NFC] Do a proper tear down of fts in QueryFormatterTest

[NFC] Do a proper tear down of fts in QueryFormatterTest

[NFC] Do a proper tear down of fts in QueryFormatterTest

Use template variable to add or remove swap button.

In combination with email notifications this button can be dangerous - ie an accidental click and
a donor rather than a staff member gets the email.

My thinking is to disable this via the activity assignee extension,
https://lab.civicrm.org/extensions/assignee/-/issues/2 - but it's cleaner for
an extension to intervene by setting a smarty variable than using javascript
to remove the button and I feel like it makes more sense for the form to
assign 'instruction variables' than for the decision to freeze to
also have 'hidden' effects further down

Switch contribution action schedule tokens to use advertised tokens for cancel_date & source

Scheduled reminders was working with the incorrect and not advertised variants - this
switches over and upgrades ( a bit precautionary since the removed ones were not
in the widget)

Merge pull request #21002 from demeritcowboy/smarty-managecase

Smarty warnings on Manage Case

smarty warnings

Merge pull request #20998 from eileenmcnaughton/id

Rename legacy token for contribution id to match sched reminder tokens

Rename legacy token for contribution id to match sched reminder tokens

In this case it turns out

1) the token being advertised by scheduled reminders is more correct ('id')
2) the legacy method resolves either 'id' or 'contribution_id' and
3) changing the advertised token from 'contribution.contribution_id' to contribution.id
does not cause contribution_id to stop working

So in summary - the advertised token was working in 'send letter' but not
in scheduled reminders. And changing which is advertised doesn't stop
the other from working in send letter

Merge pull request #20994 from demeritcowboy/upgrade-message-2

Fix upgrade message when nothing was changed

Afform - Use correct contact type when loading field options

Fixes dev/core#2702

fix empty message

Merge pull request #20993 from kainuk/vcard_notice

Do not generate a notice when a vcard is downloaded with a non-primary email or non-primary phone number

Merge pull request #20958 from mlutfy/myCaseLabels

Case Dashboard: fix labels on my/all radio buttons, and simplify label text

Do not generate a notice when a vcard is downloaded

Merge pull request #20921 from eileenmcnaughton/cleanup3

Fix for failure (fatal error and silent failure) to clean up full text indices

Merge pull request #20978 from eileenmcnaughton/act_up

Add upgrade routine to remove old action schedule token

Fix for failure to cleanup full text indices

[Ref] [Test] Move custom search tests to extension

This is a bit of a big lift because I had to re-write the tests a bit to run in the extension context -
but all part of the process of core extensionising. Still not gonna tackle another one of the classes today

Add upgrade routine to remove old action schedule token

This adds a routing to swap out the action schedule tokens on upgrade.

Given the token is completely invisible in the UI it's unlikely it is in use anywhere but
rather than just remove it from Contribute_Tokens this adds the upgrade
routine to remove this token, and others

Merge pull request #20947 from colemanw/sqlFunctions

dev/core#2704 SearchKit - Add support for SQL functions

Merge pull request #20986 from demeritcowboy/upgrade-message-formatting

Upgrade message is scrunched up next to the previous sentence

Fix mailing task when there are no tasks

Merge pull request #20988 from colemanw/fixCaseExport

Move/rename case export files for consistency

Move/rename case export files for consistency

Merge pull request #20944 from eileenmcnaughton/tiles

Test, enotice fixes, handling for permissions key for Member_Tasks

Merge pull request #20909 from eileenmcnaughton/index

Improve cleanup on indexer test

Merge pull request #20969 from eileenmcnaughton/mem

[REF] minor fix - move last of processing before receipting

Merge pull request #20971 from eileenmcnaughton/inv2

dev/core#2719 [REF] Remove legacy reference to contribution_invoice_settings

Merge pull request #20977 from eileenmcnaughton/status_name

Update label on payment instrument

Merge pull request #20979 from mlutfy/memberTaxes

New membership: fix tax amount translation

Merge pull request #20951 from eileenmcnaughton/act

Activity tasks - don't add invalid tasks

message is scrunched up next to the previous sentence

Hide REPLACE sql function from SearchKit

Either way this describes the 3 params, but the new way is better understood by SearchKit,
which to-date cannot handle more than one param.

Merge pull request #20872 from eileenmcnaughton/fprocessor

dev/core#2715 [REF] Start the process of moving financial processing to own class

Merge pull request #20905 from eileenmcnaughton/utf8

Fix databaseSupportsUTF8MB4

Merge pull request #20980 from demeritcowboy/upgrade-enotice

PHP E_NOTICE during upgrade

php notice

Merge pull request #20096 from larssandergreen/copy-template_type-when-cloning-mailing

Make final mailing from A/B test same template type and options as A/B so it can be copied

New membership: fix tax amount translation

Merge pull request #20952 from eileenmcnaughton/cs

dev/core#2721 Move custom search framework to a core extension

Merge pull request #20974 from erawat/core#2762

dev/core#2726 - Set ssl option when using DSN with SSL

Update label on payment instrument

dev/core#2721 Move custom search framework to a core extension

Put back the handling for custom in StateMachine_Search

Update civicrm_generated.mysql

Move the rest of the menu over

Merge pull request #20961 from eileenmcnaughton/status_name

dev/core#2650 Add support for names & labels for token pseudoconstants