[REF] Minor code cleanup

Do not pass params as reference as unchanged

Merge pull request #17222 from eileenmcnaughton/obh

[REF] Minor code cleanup.

Merge pull request #17224 from eileenmcnaughton/testc

[NFC] Test cleanup

Merge pull request #17223 from demeritcowboy/conference-slots

Fix admin page url for "conference slots" option values

[NFC] Test cleanup

fix admin page url

Merge pull request #17221 from civicrm/5.25

5.25

[REF] Minor code cleanup.

Once we compress the 2 ifs we find the value in the 'else' is never used

Merge pull request #17218 from seamuslee001/5.25

[REF] Fix retrieving invoice related settings when doing complete tra…

Merge pull request #17216 from totten/master-disp-cleanup

dev/core#1460 - Small cleanups in CiviEventDispatcher{,Test}

Merge pull request #17217 from civicrm/5.25

5.25

[REF] Fix retreiving invoice related settings when doing complete transaction

Merge pull request #17215 from eileenmcnaughton/act

dev/core#1714 Fix mis-filtering on activity type

Merge pull request #17172 from colemanw/writeRecord

[REF] Reduce boilerplate code in BAO add/create functions

Merge pull request #17151 from eileenmcnaughton/actsched

[REF] Minor cleanup around action schedule code.

Merge pull request #17213 from eileenmcnaughton/test

[NFC][Test] Preliminary cleanup

(NFC) CiviEventDispatcher - Improve docblocks

CiviEventDispatcherTest - Fix warning on newer versions of PHP

Merge pull request #17184 from mattwire/opencasenofatal

Cleanup and throw exceptions in OpenCase form

dev/core#1714 Fix mis-filtering on activity type

This fixes a bug where the activity_type_id was being compared against the label of the option value

Merge pull request #17211 from eileenmcnaughton/id

(NFC) Remove $Id$ artifacts from old SCM

Merge pull request #17122 from colemanw/buildOptions

Cleanup core pseudoconstant buildOptions

REF - Reduce boilerplate code in BAO add/create functions

The new CRM_Core_DAO::writeRecord function can be used instead of boilerplate

[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

A bit less svn

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

Cleanup and throw exceptions in OpenCase form

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.

[REF] Minor cleanup around actionn schedule code.

CRM_Core_BAO_ActionSchedule::processQueue is only called from one place. It either succeeds or throws
an exception so there is no value in returning anything & the parsing of what it does return assumes that it
could return false

Some slight tightenig around typing as buildRecipientContact is also only called from one place

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