Merge pull request #18628 from seamuslee001/dev_core_2035

dev/core#2035 Add in upgrade step to fix the template copy on current…

dev/core#2035 Add in upgrade step to fix the template copy on current installs

Merge pull request #18602 from eileenmcnaughton/msg

dev/core#2035 Invoice template - show Amount paid even when it is fully paid

Merge pull request #18615 from eileenmcnaughton/530

Fix regression on back-office membership renewal by credit card

Fix regression or back-office membership renewal by credit card

Looking at the code I realised a mistake was made in  https://github.com/civicrm/civicrm-core/pull/17398 whereby
the actual membership id would not be passed through correctly

dev/core#2035 Invoice template - show Amount paid even when it is fully paid

In https://github.com/civicrm/civicrm-core/pull/16680/files?w=1#diff-b211e0dac858fdad5d480c118d645e0bR129 there was a change from
'show amount paid & due depending on pay-later status' to 'show amount paid & due depending on whether payment is due'

This has been experienced as a regression and, being an invoice, there seems no reason to only change it sometimes.

This change just alters the template that new installs will receive. If approved another step is needed to update existing
installs (unless they have customised the template)

Merge pull request #18534 from eileenmcnaughton/5.30

Follow up fix on paypal redirect

Merge pull request #18547 from totten/5.30-ml-test

api_v3_MultilingualTest - Skip Cxn entities

api_v3_MultilingualTest - Skip Cxn entities

Follow up fix on paypal redirect

Turns out we did some function renaming & missed a spot before merge.

As an aside I fixed Paypal to call CRM_Core_Config::singleton()->userSystem->prePostRedirect()
directly rather than via system. I'm tempted to take the System function back out
again - I don't think it offers a better one-liner

Merge pull request #18525 from christianwach/lab-core-1997-530

Fix PayPal Standard session loss in WordPress (5.30)

Merge pull request #18524 from MegaphoneJon/core-2052-rc

fix datepickers to show correct minDate

Merge pull request #18530 from seamuslee001/fix_date_relationship

[NFC] Fix dates in RelationshipTest

[NFC] Fix dates in RelationshipTest

Fix PayPal Standard session loss in WordPress

fix datepickers to show correct minDate

Merge pull request #18520 from colemanw/searchGroupBy

Search ext: Aggregate field fixes

Search ext: Ensure all non-grouped columns are aggregated if using GROUP BY

Search ext: When removing a column from SELECT, also remove from ORDER BY

Search ext: Fix field label for aggregate fields

Merge pull request #18482 from seamuslee001/5.30

release-notes/5.29.1.md

release-notes/5.29.1.md

Merge pull request #18473 from demeritcowboy/install-pear-db

dev/core#2036 - PEAR::DB is no longer in packages, now autoloaded from vendor

DB is no longer in packages, now autoloaded from vendor

Merge pull request #18459 from seamuslee001/dev_financial_147

dev/financial#147 Ensure that jQuery Validation doesn't execute when …

dev/financial#147 Ensure that jQuery Validation doesn't execute when we click the paypal express button and ensure the style of the button doesn't change

Merge pull request #18453 from demeritcowboy/receipt-from-email-5.30

E_NOTICE on contribution confirm page

missing ?? when converted from Array::value

Merge pull request #18437 from colemanw/api4Perms

dev/core#2008 APIv4 - Map specific action names to more generic versions

APIv4 - Map specific action names to more generic versions

Merge pull request #18402 from eileenmcnaughton/5.30

Fix search extension excluded from tarball

Merge pull request #18408 from alifrumin/5.30rn

[NFC] 5.30 release-notes: first stab at annotating and sorting release notes

5.30 release-notes: first stab at release notes

Fix search extension excluded from tarball

Merge pull request #18363 from colemanw/5.30

Forward-merge 5.29 -> 5.30

Merge pull request #18361 from seamuslee001/update_event_checksum

[NFC] Update Event DAO Checksum

Fix customField legacy handling in APIv3

dev/core#1998 Address dedupe clash on dashboard contact

[NFC] Update Event DAO Checksum

Merge pull request #18342 from agh1/5.30.0-releasenotes-initial

5.30.0 release notes initial run

5.30.0 release notes: added boilerplate

5.30.0 release notes: raw from script

Set version to 5.30.beta1

Merge pull request #18335 from eileenmcnaughton/query

Add ability to enable query logging per process

Add ability to enable query logging per process

This allows us to run a command such as

env CIVICRM_DEBUG_LOG_QUERY=1 drush cvapi Contact.get

and the queries for that process (but not others) are logged.

This facilitates a cleaner record of queries as it is just one process and more flexibility
(and less risk of leaving it on by mistake)

Merge pull request #18334 from civicrm/5.29

5.29

Merge pull request #18333 from totten/5.29-rn-catchup

(NFC) 5.29.0.md - Small updates

Merge pull request #18332 from totten/5.29-norm-basepage

dev/wordpress#73 - Be more forgiving about slash in wpBasePage

5.29.0.md - Small copy-edits

Merge pull request #17980 from eileenmcnaughton/dupe

[REF] Move function to delete merged contacts to the Merger class and include all instances of the pair

5.29.0.md - Last minute addition

dev/wordpress#73 - Be more forgiving about slash in wpBasePage

Overview
--------

The correct functioning of `wpBasePage` depends on whether the administrator
omitted or added a trailing slash.  This is apparent on URLs with a trailing slash, eg

```
http://wpmaster.bknix:8001/civicrm/contribute/transact/?reset=1&id=1
```

Make it less sensitive.

Before
------

URL is OK with default setting `wpBasePage=civicrm`

URL fails with setting `wpBasePage=civicrm/`

After
-----

URL is OK with either setting `wpBasePage=civicrm` or `wpBasePage=civicrm/`

Technical Details
-----------------

I believe the old symptom arose because of the formula which produces the WP
rewrite rules (`'^' .  $config->wpBasePage .  '/([^?]*)?'`) expects that
there is no trailing slash. You could theoretically patch there, but this
seems like it'll provide more thorough normalization.

When testing, the results affect the WP rewrite rules, so you may need to be particularly
aggressive about clearing caches whenever you make a change in code or settings, eg

```
wp cache flush ; cv flush;  wp rewrite flush
```

Merge pull request #18323 from demeritcowboy/reminder-groups-dropdown

dev/core#1956 - Typo in call to nestedGroup on scheduled reminders admin form

Merge pull request #18327 from civicrm/5.29

5.29

Merge pull request #18325 from aydun/spec_fix

dev/mail#24, item 1 - Fix spec labels MailingEventSubscribe.create

Merge pull request #18326 from agh1/5.29.0-releasenotes

5.29.0 release notes final

Merge pull request #18320 from civicrm/5.29

5.29

5.29.0 release notes: get sections in right order

5.29.0 release notes: finished writing

sorting and annotating

commenting and sorting

5.29 Release Notes first pass

Fix spec labels - see dev/mail#24, item 1

typo in call to nestedGroup

Merge pull request #18319 from seamuslee001/5.29

5.28.4 release notes

Merge pull request #18315 from eileenmcnaughton/ordergo

[REF] Remove most interaction with $objects in completeOrder

Merge pull request #18295 from eileenmcnaughton/linetests

Test & remove handling for max_related in renewal form

Fix anchors in release-notes/5.28.3.md

Add release-notes/5.28.4.md

Merge pull request #18316 from civicrm/5.29

5.29 to master

Merge pull request #18272 from eileenmcnaughton/custchange

dev/core#1974 Fix incorrect handling of serialize key when changing custom field type

Merge pull request #18307 from agh1/5.29.0-releasenotes-initial

5.29.0 release notes initial run

Merge pull request #18310 from colemanw/revert-18005

Revert #18005 Fix buggy placement of icons

Merge pull request #18311 from mattwire/repeattrxn_unused

REF Remove unused passbyreference and var from ipn_process_transaction

[REF] Remove most interaction with

With this change we are only looking up
1) objects['paymentProcessor'] - we should probably just pass in the id
2) objects['contribution'] - we could make this a param in it's own right & remove objects
3) objects['event'] - just used to get event title - we could do a query off participant

dev/core#1974 - ChangeFieldType - "Select (w/serialize)" field should be treated like older "Multi-Select" fields

Remove unused passbyreference and var from ipn_process_transaction

Revert #18005 Fix buggy placement of icons

Merge pull request #18304 from highfalutin/noah/dev-core-1974

dev/core#1974: correct UI regression on custom field edit

5.29.0 release notes: yank changes already in 5.8.x

5.29.0 release notes: added boilerplate

5.29.0 release notes: raw from script

Merge pull request #18303 from eileenmcnaughton/event_batch

[REF] Clean up return variables on updateContributionStatus, updatePendingOnlineContribution

Merge pull request #18279 from eileenmcnaughton/ordertest

Fix test set up to call Order.create to create the correct line items

Merge pull request #18294 from eileenmcnaughton/linecreate

[REF] Separate and move line-item specific portion of checkTaxAmount to LineItem api

Merge pull request #18305 from civicrm/5.29

5.29

Merge pull request #18293 from eileenmcnaughton/fin

Move LineItem acl handling from v3 api to financialacls core extension

Merge pull request #18290 from eileenmcnaughton/tax

dev/core#1983 Fix to tax calculation on multi-line-item

Custom field create/edit form: replace dysfunctional checkbox with static label in edit mode

Merge pull request #17810 from seamuslee001/negative_payment_create

[REF] Fix financial item allocation of negative payments against completed payments

[REF] Clean up return variables on updateContributionStatus, updatePendingOnlineContribution

The return variable from updatePendingOnlineContribution is never used - ergo don't return it. This means
the return value from updateContributionStatus is also never used - ergo - you get it. In addition don't call the function
with empty variables - I've moved the check as to whether they are empty into the calling function - although likely
in a second pass the checks will be removed or change again

Merge pull request #18237 from mattwire/mailingsearchcolumns

Remove 'hack' that overwrites result of searchColumns hook in mailings list

Merge pull request #18302 from civicrm/5.29

5.29 to master

Merge pull request #18301 from demeritcowboy/dev-core-1982-5.29

dev/core#1982 - 5.29 version of PR 18282

[REF] permit negative payments to be made against completed payments

Add in unit test for adding additional refund payment to contribution already refunded and also adding a negative payment to a contribution that is negative already

faulty check for simplexml node value - see also PR 18282

Merge pull request #18067 from MegaphoneJon/event-34-reroll

event#34: allow negative self-service/transfer time

Merge pull request #18292 from eileenmcnaughton/receipt

Add test to check, remove unnecessary lines

Merge pull request #18300 from MegaphoneJon/clean-non-deductible

Clean money for non-deductible amount

Merge pull request #18291 from eileenmcnaughton/depit

[REF] [test] Slight increase in test cover & make functions more re-usable