[NFC][Test] Preliminary cleanup

Margin cleanup - declare exceptions, use right quotes, remove unused params

Merge pull request #17167 from colemanw/moreOptions

APIv4 - Allow field options to be returned in multiple formats

Merge pull request #17196 from eileenmcnaughton/session

[REF] Cleanup a few instances where we use old call to get loggedInUserID

APIv4 - Improve pseudoconstants in HAVING and Explorer

APIv4 - Allow field options to be returned in multiple formats

Merge pull request #17209 from civicrm/5.25

5.25

Merge pull request #17208 from seamuslee001/5.25

Update 5.24.6.md

Update 5.24.6.md

Merge pull request #17207 from civicrm/5.25

5.25

Merge pull request #17205 from colemanw/noMultiJoins

APIv4 - Don't advertise implicit multi-joins in Explorer

Merge pull request #17206 from seamuslee001/5.25

5.24.6 Release Notes

Update release-notes/5.24.6

Add release-notes/5.24.6.md

Merge pull request #17201 from civicrm/5.25

5.25

Merge pull request #17176 from eileenmcnaughton/batch1

Move batch-form support code back to the form

APIv4 - Don't advertise implicit multi-joins in Explorer

These joins are problematic and may be removed. The Explorer should not display them.

Merge pull request #17195 from eileenmcnaughton/act_report2

[REF] Simplify determination of enabled components

Merge pull request #17200 from eileenmcnaughton/rep

dev/core#1729 Fix filters skipping on activity report

dev/core#1729 Fix filters skipping onn activity report

Merge pull request #17190 from mattwire/contributionpageselectedchildfromactions

dev/core#1738 Fix opening specific contribution page tab from Manage Contribution Pages

Merge pull request #17199 from seamuslee001/new_master

5.25

merge in 5.25

Merge pull request #17188 from totten/5.25-contrib-virt-itemcanon

 Fix loading, saving, and upgrade of CiviContribute settings

Merge pull request #17194 from eileenmcnaughton/act_report

[REF] stop overriding postProcess function in activity report detail

[REF] Cleanup a few instances where we use old call to get loggedInUserID

[REF] stop overriding postProcess function in activity report detail

The function is the same as master except one extra thing that can be done in the function intended for
report-specific preProcess overrides

[REF] Simplify determination of enabled components

testHandlingOfContributionInvoiceSetting() - More realistic data+assertion

In practice, for the old setting `contribution_invoice_settings`, the
`invoicing` property could take one of two values:

1. If enabled, the value is an array with trueish subkey: `['invoicing'=>['invoicing'=>1]]`
2. If disabled, the value is a numeric `0`: `['invoicing'=>0]`

Following this means better compatibility.

The format is counter-intuitive, and the unit-test sets the value to enabled
using a more intuitive format (`['invoicing'=>1]`).  It's currently valid to
`set()` this way, but `get()`ting the value produces the
more-compatible/historical format.

dev/core#1724 - Backward-compatible handling of 'contribution_invoice_settings.invoicing'

In the old setting `contribution_invoice_settings`, the `invoicing`
property (aka "CiviContribute Component Settings" => "Enable Tax and
Invoicing") could take one of two values:

1. If enabled, it's an array with trueish subkey: `['invoicing'=>1]`
2. If disabled, it's a numeric zero.

This encoding is weird, and `contribution_invoice_settings` is generally deprecated
and has been replaced by a virtual rendition which needs to be consistent with the
old representation -- so that a couple-dozen things continue working.

For testing, it helped me to try these commands at different points:

```
cv ev 'return CRM_Invoicing_Utils::isInvoicingEnabled();'
cv api setting.get return=invoicing
cv api setting.get return=contribution_invoice_settings

select * from civicrm_setting where name like 'contrib%' or name like 'invoic%';
```

FiveTwentyFive - Convert settings during upgrade

Consider this use-case:

* Install 5.22
* Configure `CiviContribute Component Settings`
* Upgrade to 5.25.beta

The settings are lost during the upgrade.

Set version to 5.25.beta3

Merge pull request #17068 from colemanw/currentPath

[REF] Reconcile CRM_Utils_System::getUrlPath and CRM_Utils_System::currentPath

Fix opening specific contribution page tab from Manage Contribution Pages

Merge pull request #17125 from eileenmcnaughton/dupe1

Fix Dedupe entity_tag mangling bug

Merge pull request #17187 from alexymik/recur_contribution_source

Add contribution source to recurring contribution report

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

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

dev/core#1724 - Fix defaults for invoice-related settings

The default values were previously stored in 'contribution_invoice_settings', but
the updates are stored in individual settings, which makes it difficult to reconcile
the default values and explicit values.

With this revision, 'contribution_invoice_settings' is strictly virtual, and
all the other settings (like 'invoice_due_date') are canonical.

(REF) dev/core#1724 - SettingsBag - Convert to computeVirtual/updateVirtual

The `SettingsBag` uses the `$combined` property to (locally) cache a full
view of the active settings (based on combining different layers of data --
default-values, explicit-values, and mandatory-values). This design is
good for read-mostly data.

This patch changes the `__ContributionSettings` to be another layer in the
construction of `$combined` -- the virtual-values layer.  It fixes issues
wherein:

1. The virtual value is recomputed during every call to `get($key)`.

2. The virtual value is only based on explicit-values -- it disregards
   default-values and mandatory-values.

dev/core#1724 - SettingsBagTest - Add coverage for contribution_invoice_settings

Add contribution source to recurring contribution report

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

dev/core#1724 Fix Changes to CiviContribute Component Settings not saved

It seems that, depending on the order in which settings are loaded, this is not always populated. Moving it here
addresses, although medium term we should address all references to it

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

Move batch-form support code back to the form

I've dug around onn this and there is no reason for this code to be in the BAO - it is only
relevant to support this form & should live there. Longer term the form should call
api v4....

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

Fix Dedupe entity_tag mangling bug