Merge pull request #17186 from demeritcowboy/gitlab-issue-template

Gitlab issue template - Ask for link to stackexchange or chat conversations

ask for link to external conversations

Merge pull request #17182 from eileenmcnaughton/dep

Remove deprecated calls to optionValueQuery and no-longer-need param

Merge pull request #17177 from mattwire/showsendreceiptonrecurdetail

Show email receipt status on view recurring contribution

Remove deprecated calls to optionValueQuery and no-longer-need param

All calls to optionValueQuery that don't pass  have been deprecated for well over a year. We can remove them now,
and hence the handling of

Merge pull request #17127 from totten/master-dispatch-policy

dev/core#1460 - CiviEventDispatcher - Add policy options

Merge pull request #17129 from eileenmcnaughton/email2

Switch cc field on emailTrait to use entity reference

Display email receipt status on recur details

Convert view recurring contribution to use EntityPageTrait

Merge pull request #17174 from pradpnayak/ruleTitle

show title on status message instead machine name

Merge pull request #16882 from wmortada/wp#46

dev/wp#46 - Remove styling that attempts to make inputs match select2

Merge pull request #17071 from eileenmcnaughton/payment

Refactor api3 Payment.Get API to support options + most fields in civicrm_financial_trxn

show title on status message instead machine name

Refactor api3 Payment.Get API to support options + most fields in civicrm_financial_trxn

Unit test for PR#16603

Alternative for fix ups on Payment.get

Merge pull request #17173 from civicrm/5.25

5.25 to master

Merge pull request #17128 from seamuslee001/dev_core_1710

dev/core#1710 Ensure that civicrm_case_activity is properly populated…

dev/core#1710 Ensure that civicrm_case_activity is properly populated for use by report query when run in force mode

Move table definition into constructor to set alias using standard ways and only rely on the code in the buildQuery to add in the case_id column as necessary

Fix hiding of case_id column

Update to include Case ID column

Merge pull request #17131 from eileenmcnaughton/batch

Remove unused parameter, immediately overwritten parameter

Merge pull request #16733 from eileenmcnaughton/smarty

Do not create smarty cached templates for processed greetings

Merge pull request #17171 from civicrm/5.25

5.25

Merge pull request #17170 from colemanw/optionValueDefault

OptionValue - Use DB defaults instead of setting them in BAO::add

Merge pull request #17164 from pradpnayak/invoicingHidden

Invoice action not shown when invoicing enabled

Invoice action not shown when invoicing enabled

OptionValue - Use DB defaults instead of setting them in BAO::add

Of the 4 defaults being set in the DAO, 3 already had db defaults; I added DEFAULT NULL for the last.

Merge pull request #17168 from colemanw/contactFieldPerms

[REF] APIv4 - Enforce contact field permissions via metadata

Merge pull request #17165 from pradpnayak/ActivitySource

Add contribution id as activity source record when downloading or emailing invoice

Merge pull request #17166 from colemanw/optionGroupTitle

Correct option_group pseudoconstant and regenerate dao

Merge pull request #17161 from eileenmcnaughton/out

[REF] Code readability changes on activity tokens.

APIv4 - Enforce contact field permissions via metadata instead of class override

Now that 'permission' metadata has been added to the schema, there's no need for specialized code for that field.
This adds generic code to handle any field with permissions declared.

Correct option_group pseudoconstant and regenerate dao

The option_group_id pseudoconstant was returning the `name` field as both "name" and "label"
whereas the `title` field works better as "label".

activity api accepts name in activity type

Add contribution id as activity source record when downloading or emailing invoice

Merge pull request #17159 from eileenmcnaughton/ids

Less svn

[REF] Code readability changes on activity tokens.

I've been trying to get to grips with this code and this addresses 2 things I found very confusing - notably

- using activityId when we were referring to a fieldname - we use a combination of  &  for
variables but normally when it's closely related to a query we use activity_id
- the function name getEntityContextSchema() was unclear to me - I concluded that in fact we were retrieving
the nname of the field for the entity ID

Merge pull request #17160 from eileenmcnaughton/out

[REF] Minor extraction

[REF] Minor extraction

Extract the code that renders the generated html it pdfs or word docs

Less svn

Merge pull request #17158 from colemanw/optionNames

OptionValues - Munge custom value names and return names in validate context

REF - Code cleaup: replace clunky coalesce conditionals with simple coalesce expressions using ??

OptionValues - Munge custom value names and return names in validate context

Fixes a couple core bugs, where option value names were not being munged by default,
and custom field options were not being returned correctly in the 'validate' context.

This is the core change as part of APIV4 work
See https://lab.civicrm.org/dev/core/-/issues/1705

Merge pull request #17155 from eileenmcnaughton/pdf

[NFC] Superficial code clean up

Merge pull request #17157 from eileenmcnaughton/ids

[NFC] A bit less svn in our code

[NFC] A bit less svn in our code

[NFC] Superficial code clean up

Merge pull request #17154 from eileenmcnaughton/disp

[REF] Consistently lookup dispatcher via `Civi::dispatcher()`

Merge pull request #17153 from eileenmcnaughton/acti

Convert test to use APIV4 in setup for cleaner code

[REF] Make instantiation of dispatcher consistent.

We have 3 ways of iniatiging a dispatcher

Civi\Core\Container::singleton()->get('dispatcher')
Civi::service('dispatcher')
Civi::dispatcher()

Tim confirms there is no functional difference, so this consolidates on the shortest one :-)

Convert test to use APIV4 in setup for cleaner code

Merge pull request #17152 from eileenmcnaughton/act2

[NFC] [Test] minor code cleanup

Merge pull request #17138 from colemanw/api4pseudoconstants

dev/core#1705 APIv4 - Support pseudoconstant lookups

[NFC] [Test] minor code cleanup

Merge pull request #17150 from civicrm/5.25

5.25

Merge pull request #17149 from seamuslee001/5.25

Add release-notes/5.24.5.md

Add release-notes/5.24.5.md

APIv4 - Fix contactFieldsToRemove to work with joins and pseudoconstants

APIv4 - Support pseudoconstant lookups

Appending a suffix to a field name e.g. location_type_id:name or gender_id:label
will automatically lookup values from field option lists.

Merge pull request #17141 from mlutfy/reportOutput

[NFC] Cleanup CRM_Report_Form

Merge pull request #17136 from demeritcowboy/portugal-provinces

Update more Portugal provinces

Merge pull request #17139 from civicrm/5.25

5.25

Merge pull request #17134 from jitendrapurohit/core-1723-rc

dev/core#1723 - Adv Search - Reciprocal relationship search with custom fields leads to error

RC fix for #17132

unit test fix

Switch cc field on emailTrait to use entity reference

Merge pull request #17133 from eileenmcnaughton/dep

(NFC) Comment clarification in test class

Cleanup CRM_Report_Form

Merge pull request #17137 from seamuslee001/dev_core_1717

dev/core#1717 - Fix SMTP failure involving `disconnect()` method

dev/core#1717 Fix SMTP failure on fail to disconnect due to new wrapper smtp mailer

Add in wrapper around to check if we can call it

Merge pull request #17105 from colemanw/psr-4

dev/core#1684 - Use PSR-4 autoloader instead of PSR-0 for "Civi" namespace

Merge pull request #16716 from mattwire/cancelsubscriptionaddemail

Add contributor email address to cancelSubscription form so it is cle…

update Portugal provinces

Merge pull request #17135 from demeritcowboy/leftover-description

dev/user-interface#19 - Remove leftover description text on contribution form mistakenly left in

leftover description text from net amount field removal years ago

Add contributor email address to cancelSubscription form so it is clear where it will be sent

Comment clarification in test class

I just updated the comments on this helper to clarify the limitations of the function & the
fact that it should not be our only way to test thousand separators.

I was noticing perfect was becoming the enemy of the good here. The function was marked as deprecated
because it doesn't cover all scenarios - but the upshot was that we stopped increasing out
thousand separator testing. In fact we need lots of form tests to do some testing of
the separators and a very small number to test more variants - this latter has been added
& the comments point to the need for more without going as far as deprecating

Remove unused parameter, immediately overwritten parameter

dev/core#1460 - CiviEventDispatcher - Add policy options

Merge pull request #17123 from civicrm/5.25

5.25

Merge pull request #17099 from lcdservices/dev-core-1718

dev/core#1718 membership batch entry join date fix

Merge pull request #17119 from agh1/5.25.0-releasenotes

5.25.0 release notes first run

Merge pull request #17064 from eileenmcnaughton/email3

Convert bcc field to use an entity reference.

Merge pull request #17121 from eileenmcnaughton/update

[REF] Minor var simplification

[REF] Minor var simplification

Merge pull request #17106 from pradpnayak/statepro1

Update Colmbra state/province to Coimbra

Merge pull request #17109 from colemanw/noAlias

APIv4 - Prevent field alias conflicts.

5.25.0 release notes: added boilerplate

5.25.0 release notes: raw from script

Merge pull request #17113 from colemanw/restoreApiSql

Restore #16947 - APIv4 support for sql functions and grouping

Restore #16947 - APIv4 support for sql functions and grouping

Merge pull request #17111 from seamuslee001/master

5.25

Merge in 5.25

Merge pull request #17093 from eileenmcnaughton/cont

[NFC] Remove calculation of unused parameter

Merge pull request #17108 from colemanw/revertSqlFn

Revert #16947 from 5.25RC

APIv4 - Prevent field alias conflicts.

Do not allow regular fields to be aliased - only expressions.
Prevent an alias from using the same name as an existing field.

Merge pull request #17069 from colemanw/removeUselessChecks

[REF] Remove duplicate checks for an array key existing

Revert "APIv4 - Add rudimentary support for groupBy"

This reverts commit fba513f62ec8815e08fa838e0d0501279bf34501.

Revert "Api4SelectQuery - add more metadata to apiFieldSpec"

This reverts commit 9b06167d3c8dc54bb51e22e3583b18799a46c930.

Revert "APIv4 - Add SQL expression handling and aggregate functions"

This reverts commit 3176b04cb62b0e8f94454e367736f50454f89de8.

updated civicrm_generated.mysql file

Update Colmbra state/province to Coimbra

dev/core#1684 Use psr-4 autoloader instead of psr-0 for Civi directory

This change is to allow underscores in class names, which were being misinterpreted as directory separators.

Merge pull request #17080 from colemanw/importExtract

[REF] Import - extract duplicate code to function