Merge pull request #19063 from christianwach/lab-core-2213

Protect against 404s when wpBasePage is mixed case

Merge pull request #19070 from mattwire/extensionupgrademsg

Extract function that generates upgrade link for extensions

Extract function that generates upgrade link for extensions

Merge pull request #18612 from mattwire/tokenprocessorsimplify

REF Simplify tokenProcessor code

Merge pull request #19025 from sunilpawar/favourite_country

Feature to provide mostly used countries in top section of Country select list

Simplify tokenProcessor code for activities

Merge pull request #19065 from colemanw/campaignTabs

Minor tabs cleanup toward fixing dev/core#2215

Minor tabs cleanup

1. Avoid overwriting CRM.tabSettings.active by cloning the variable rather than copying by reference
2. Initialize tabs only once on the Campaign dashboard

Merge pull request #19059 from eileenmcnaughton/2199_ufgroup_pre_post_hook

Remove use of nullArray in delete hooks

Protect against 404s when wpBasePage is mixed case

Merge pull request #19060 from eileenmcnaughton/ufadd

Deprecate UFGroup::add parameter ids

added test case, ts for help text in setting file, changed static variable logic

Deprecate UFGroup::add parameter ids

This is only passed in from one place. This PR removes it & adds noisy deprecation

Remove use of nullArray in delete hooks

Looking at #18995 I was struck by the delete hook being non-standard. In general we don't load from the DB just to pass
to the hook as the hook can do that itself. However, when I looked at other hooks
I found that they were passing out nullArray - this is a legacy method that precedes
being on a php version that supported default params when passing by reference.

It's further known to introduce intermittent hard to debug issues. This adds the new hook
and also adds standardisation to the other hooks for pre+delete.

I've left the create one for now but GroupContact is a good example of something
close to what we want to standardise on there

Merge pull request #19050 from eileenmcnaughton/pp

[REF] Remove excess handling around contact_id

Merge pull request #19051 from eileenmcnaughton/pps

Remove extra handling around contact id

Merge pull request #18916 from colemanw/crudLink

Remove unused hook_civicrm_crudLink and switch to using metadata for crudLinks

Merge pull request #19052 from colemanw/angModuleCleanup

[REF] Cleanup Ang modules in core to follow conventions

Merge pull request #19058 from mattwire/pcpshortcodeaction

wordpress#63 Add action parameter to PCP shortcode

Merge pull request #19046 from dwoods-encircle/Resource-URL-helptext-update

Update Resource URL Helptext

small change

changed lable and field name from favourite to Pinned

wordpress#63 Add action parameter to PCP shortcode

Merge pull request #19056 from civicrm/5.32

5.32

Merge pull request #19055 from colemanw/fixSearchActionButton

Search ext: Fix search action button

Search ext: Fix search action button

Remove extra handling around contact id

It's retrieved a few rows earlier from the contribution - this is from formerly
shared code that does not apply here

[REF] Remove excess handling around contact_id

The removed code was formerly in a shared function. On looking the
getIDS actually just gets the contact id from the contributionRecur
object & every thing else is extraneous

Merge pull request #19049 from colemanw/crmExampleGone

Delete outdated/unused crmExample Angular module

Merge pull request #19047 from colemanw/crmD3

[REF] Decouple crmD3 angular module from CiviMail

Merge pull request #19045 from seamuslee001/remove_xss_function

[REF] Remove xssString as it is providing a false sense of security

Fix crmStatusPage file name to match module name per convention

Use settingsFactory intead of settings for angular modules.

We are moving toward the use of settingsFactory and away from settings for angular modules.
This would allow the list of modules to be cached as it would only contain static metadata.

Merge pull request #19048 from colemanw/angComment

[NFC] Remove boilerplate comment from .ang.php files

Merge pull request #19054 from eileenmcnaughton/pp1

[REF] Simplify sendNotification determination

Remove boilerplate comment from .ang.php files

Link to wiki was outdated and comment was not very helpful

[REF] Simplify sendNotification determination

Delete outdated/unused crmExample Angular module

Decouple crmD3 angular module from CiviMail

The crmD3 module was being conditionally loaded based on whether CiviMail was enabled.
This changes it to always exist but only loaded as required.

Update Resource URL Helptext

[REF] Remove xssString as it is providing a false sense of security

Merge pull request #19043 from eileenmcnaughton/pp

dev/financial#148 fully deprecate validateData function

Merge pull request #19042 from eileenmcnaughton/pps

Remove legacy check

Merge pull request #19005 from totten/master-afform-dashlet

Afform - Generate dashlets based on `Afform.is_dashlet` property.

dev/financial#148 fully deprecate validateData function

Remove legacy check

This check dates right back to when we started extracting IPN code from code blocks in the processor. I don't believe it's meaningful

Merge pull request #19040 from eileenmcnaughton/pp2

[REF] remove obscure use of objects from A.net

afform - Interpret the 'is_dashlet' property and auto-(de)register a `Dashboard` element

Merge pull request #19006 from colemanw/bsCheckbox

Improve bootstrap3 checkbox theming

Merge pull request #19041 from eileenmcnaughton/ppp

[REF] Stop passing objects to recur in paypal pro - pass specific objects

[REF] Remove obscure objects from A.net

Merge pull request #19039 from eileenmcnaughton/pp

[REF] remove obscure use of objects from Anet.ipn

Stop passing objects to recur - pass specific objects

[REF] remove obscure use of objects from Anet.ipn

Merge pull request #19038 from eileenmcnaughton/coleman_cheer

[REF] Remove code Coleman hates

afGuiEditor - Expose the `is_dashlet' property

(REF) afGuiEditor - Add form-* elements to 'config-form.html'

This should bring the markup closer to Bootstrap3 convention.

afform/core - Add property 'is_dashlet'

afform/mock - Update phpunix.xml.dist for phpunit7

Merge pull request #19037 from eileenmcnaughton/dep

Remove some deprecated code

Merge pull request #19032 from eileenmcnaughton/obj

[REF] Simplify single function to receive contribution not objects

Merge pull request #19033 from eileenmcnaughton/obj2

dev/financial#148 fold call to loadObjects

Merge pull request #19035 from eileenmcnaughton/obj4

dev/financial#148 fold call to loadObjects - a.net

Merge pull request #19034 from eileenmcnaughton/obj3

dev/financial#148 duplicate out call to validateObjects

[REF] Remove code Coleman hates

Remove some deprecated code

This was deprecated over a year ago just in case anyone was passing in the legacy param - fully removing now

Merge pull request #19036 from civicrm/5.32

5.32 to master

Merge pull request #19031 from colemanw/bullets

Improve options in list search display

Improve bootstrap3 checkbox theming

Standardizes our styling of checkboxes in boostrap markup to:
`<label class="form-control"><input type="checkbox"> <span>Title</span></label>`

Merge pull request #19030 from civicrm/5.32

5.32

Merge pull request #19027 from mlutfy/fixEvent30

Event Full: fix translation regression

dev/financial#148 fold call to loadObjects

dev/financial#148 duplicate out call to validateObjects

dev/financial#148 fold call to loadObjects

Simplify single function to receive contribution not objects

Contribution is now the only object used in this function - so it should be the only one passed in

Merge pull request #19015 from eileenmcnaughton/fail

dev/core#927 Update ContributionCancelActions to also handle 'failed'

Improve options in list search display

Merge pull request #19029 from seamuslee001/5_31_1_rn

Add release-notes/5.31.1.md

Add release-notes/5.31.1.md

Merge pull request #19020 from eileenmcnaughton/grant

Add Grant v4 api

Event Full: fix translation regression

Add Grant v4 api

I created a spec provider for a default currency as I didn't think it made sense to require it

Update APIv4 Grant docblock

Remove test as not valid

It is no longer possible to enter a contribution with a cancelled or failed status via the membership form so
this test is not valid. Only pending or completed are permitted

Update ContributionCancelActions to also handle 'failed'

The failed action has no discernable difference from the no-longer-used cancel action. The cancel action had some minor refactoring
done to work through the conclusion that the call to addRecurLineItems was never reached in a meaningful way
but I think we can skip all that & see that fail is not functionally different to cancel & just deprecate the function.

At this point we are very close to simply removing the last core handling for failed & cance & making the extension visible.

The only thing I can see that still needs checking in the 2 functions still to be removed is the handling of
is_override & override_date is tested - which I will do as a follow up

Feature to provide mostly used countries in top section of Country select list

Merge pull request #19022 from civicrm/5.32

5.32

Merge pull request #18978 from AsylumSeekersCentre/fixReportFilterMultiSelect

Fix issue #2162: allow reports to filter multi-select fields and find entities with multiple selections

Merge pull request #19007 from alexymik/patch-1

Check for membership type fee before applying tax

Merge pull request #19012 from eileenmcnaughton/trans3

dev/core#927 [REF] Further removal on unreachable code in transitionComponents

Merge pull request #19018 from demeritcowboy/case-tasks-deprecated

dev/core#2205 - Deprecated function warning when choosing task from Find Cases results

Merge pull request #19016 from eileenmcnaughton/obj

dev/financial#152 [REF] Pass specific BAO into the recur function rather than the vague 'objects'

Merge pull request #19010 from colemanw/bridges

Use trait instead of class for Entity Bridges; add OptionList trait

Check for membership type fee before applying tax

deprecated base function

dev/financial#152 Pass specific BAO into the recur function rather than the vague 'objects'

[REF] Further removal on unreachable code in transitionComponents

Once #18997 & #18998 is merged it becomes clear that processContribution can never be true & hence this code is unreachable

Merge pull request #18998 from eileenmcnaughton/trans2

Remove unused params, return params from processFail

Merge pull request #18997 from eileenmcnaughton/trans

Remove unused parameters from cancel

Merge pull request #18880 from vingle/master

changes social media iframes/scripts to links, simplifies markup, adds email & bootstrap classes

Merge pull request #18959 from seamuslee001/dev_core_2153_2

dev/core#2153 #REF Remove outdated updateCustomValues function