Merge pull request #18474 from civicrm/5.30

5.30

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

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

Merge pull request #18461 from demeritcowboy/entity-drupal-9

dev/drupal#138 - Drupal 9 deprecations

DB is no longer in packages, now autoloaded from vendor

Merge pull request #18472 from totten/master-symlink

dev/core#1393 - distmaker - Fix URL-based installation on D7/BD

dev/core#1393, dev/core#1990 - Exclude README.rst symlink

dev/core#1393, dev/core#1990 - Assert that Drupal-style tarballs have no symlinks

Merge pull request #18469 from eileenmcnaughton/iso2

dev/core#1921 Further removal of iso date handling

Merge pull request #18431 from colemanw/searchUpdate

Search extension: edit smart groups

dev/core#1921 Further removal of iso date handling

Overview
----------------------------------------
Same deal as https://github.com/civicrm/civicrm-core/pull/18359

Before
----------------------------------------
Casting to ISO date to avoid 2014 bug

After
----------------------------------------
We are stuck in 2020

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

Comments
----------------------------------------
relevant tests
api_v3_CaseTest::testCaseMerge

Merge pull request #18467 from mattwire/membershiptestfix

Fix parameters for MembershipTest

APIv4 Explorer: Only show result length if result is array

Update links to APIv4 smart groups to point to the new search ext

Search ext: load saved searches

Search ext: improve debug output

Fix parameters for MembershipTest

Merge pull request #18460 from eileenmcnaughton/comment

Update code comments

Merge pull request #18462 from civicrm/5.30

5.30 to master

Merge pull request #18459 from seamuslee001/dev_financial_147

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

Update code comments

Making these a little less paypal specific

drupal 9 deprecations

Merge pull request #18456 from eileenmcnaughton/iso1

dev/core#1921 remove isoToMysql

Merge pull request #18432 from eileenmcnaughton/mem

[REF] Simplify array construction

Merge pull request #18457 from eileenmcnaughton/mailer

dev/core#1921 remove isoToMysql

Merge pull request #18435 from eileenmcnaughton/ms

[REF] minor tidy up on membershipStatus::create & add

Merge pull request #18450 from eileenmcnaughton/activity2

Pass in activity type rather than calculate it

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 #18452 from eileenmcnaughton/user

Move definition of userName to where it is used and remove an unused parameter

Merge pull request #18454 from eileenmcnaughton/pay

dev/financial#149 Add permission = false to apiv4 call to address unreleased regression

dev/core#1921 remove isoToMysql

Overview
----------------------------------------
Remove unnecessary isoToDate function

Before
----------------------------------------
Code compatible with packages code of 6 years ago

After
----------------------------------------
will break if you revert 6 year old fix....

Technical Details
----------------------------------------
The test testMailerStats passes through this line of code - which
I think is probably about the most we can confirm testing wise as we expect there
to be no reason to cast to mysql anymore

Comments
----------------------------------------
https://lab.civicrm.org/dev/core/-/issues/1921

dev/core#1921 remove isoToMysql

The test testPaymentDontChangeReceiveDate passes through these 2 lines of code - which
I think is probably about the most we can confirm testing wise as we expect there
to be no reason to cast to mysql anymore

Merge pull request #18455 from civicrm/5.30

5.30 to master

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

E_NOTICE on contribution confirm page

Move definition of userName to where it is used and remove an unused paramter

dev/financial#149 Add permission = false to apiv4 call to address regression

This retrieves payment processors using FALSE for get permissions. I was able to replicate the bug in the test.

I also updated one line that I passed through when stepping through the test that related to the addActivity.
It just passes in activityType rather than leaving addActivity to figure that out

missing ?? when converted from Array::value

Pass in activity type rather than calculate it

This line is hit in CRM_Event_BAO_ParticipantTest.testCreate
api_v3_ContributionTest.testCompleteTransactionWithParticipantRecord

I think we should be moving the logic back to the calling functions in general - to the point of them
just callling the v4 api directly, but this just clarifies on tiny bit

Merge pull request #18434 from agh1/titlesingular

Offer singular entity titles

Search ext: update to plural entity titles

DAOs with singular/plural options for entity titles

Merge pull request #18446 from colemanw/noDebugInApi3Test

Remove unnecessary debug from tests which messes up array output

Remove unnecessary debug from tests which messes up array output

GetValue and debug can't really be used together. If the output is scalar,
then there's no place for the debug info to go. If the output is an array of values,
the debug info will be injected into the values, messing up the test.

Merge pull request #18443 from demeritcowboy/symfony-public-more

Make symfony aliased services public

make service public like in symfony 2.x

Merge pull request #18440 from civicrm/5.30

5.30 to master

Merge pull request #18437 from colemanw/api4Perms

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

Merge pull request #18438 from eileenmcnaughton/line

[REF] Folllow up cleanup - remove now unused param

APIv4 - Map specific action names to more generic versions

[REF] Folllow up cleanup - remove now unused param

In a recent regression fix we stopped returning  from getLine. This fully removes the, now unused,
parameter. The functions that call getLine calculate the total now

Merge pull request #18433 from eileenmcnaughton/memstatus

dev/core#2017 Remove unused property,  override function that just calls parent

[REF] minor tidy up on membershipStatus::create & add

We have been deprecating & removing the  array from BAO create & add actions - this continues that
(long slow) process'

Remove unused property, null constructor

Merge pull request #18419 from colemanw/improveCustomFieldForm

Custom field form reform

[REF] Simplify array construction

In trying to interpret https://github.com/civicrm/civicrm-core/pull/17992

I realised that the code copies values from the dao to an array, copies the array
to another array, unsets most of the variables and then uses that second array.

This simplifies.

The easiest way to work through this code is in a debugger running

testProcessMembershipUpdateStatus

Merge pull request #18405 from pradpnayak/lineItemFix

Replace & to and in button label

Custom field admin form reform

This overhauls the custom field administration form:
- Gets rid of the difficult-to-use hierarchcal select
- Removes changeFieldType as a separate form
- Allows changing field type on the main form, with improved validation
- Fixes up some metadata
- Improves choosing default values

Custom Field Form - Cleanup country/state leftover cruft

CustomField - Reformat data when modifying field serialize property.

Adds or removes CRM_Core_DAO::VALUE_SEPARATOR from custom values when switching a field from single to muti-valued.

Merge pull request #18429 from artfulrobot/fixpropertybagsetamount

property bag's setAmount should ensure dot decimal point

Replace & to and in button label

Add tests for PropertyBag::setAmount()

Merge pull request #18428 from lcdservices/dev-core-2009

dev/core#2009 filter grant dashboard to exclude trashed contacts

dev/core#2009 filter grant dashboard to exclude trashed contacts

Merge pull request #18425 from seamuslee001/revert_credit_card_getter

Revert "PropertyBag - add cardNumber to getters"

Merge pull request #18426 from mattwire/clarifygetmembershipcount

NFC Clarify what CRM_Price_BAO_Priceset::getMembershipCount does

Merge pull request #18388 from eileenmcnaughton/line4

dev/core#1980 Add v4 api LineItem entity

Revert "PropertyBag - add cardNumber to getters"

This reverts commit 194ae0f4206d0de00dea44553ac8a679cd771ea4.

Clarify what getMembershipCount does

Merge pull request #18404 from mattwire/membershipfrontenddep

Switch frontend contribution form to cached/non-deprecated functions for membershipTypes

Merge pull request #17595 from eileenmcnaughton/settings

PropertyBag - add cardNumber to getters

Add v4 api LineItem entity

Merge pull request #18395 from eileenmcnaughton/memtype

Make period_type mandatory for MembershipType

Merge pull request #17742 from MiyaNoctem/dev-core-1854-fix-resetting-overridden-status-of-related-memberships

dev/core#1854: Fix Calculation of Overridden Membership Status

Merge pull request #18406 from pradpnayak/MessageChange

Use title instead name in status message

Merge pull request #18393 from eileenmcnaughton/just_load

Remove error handling from loadObjects

Merge pull request #18421 from eileenmcnaughton/calc

Minor code simplification on date handling in getMembershipStatusByDate

Minor code simplification on date handling in getMembershipStatusByDate

This deprecates 'today' as a synonym for 'now' as the latter is more standard and slightly simplifies the date handling

Merge pull request #18424 from eileenmcnaughton/dep

Remove CRM_Contact_BAO_Contact::getPrimaryOpenId

Merge pull request #18422 from eileenmcnaughton/iso

dev/core#1921 Remove more instances of civi 4.2 date handling

Merge pull request #18420 from MegaphoneJon/membership-detail-custom-fields

report#49: add contact type custom fields to contribution details

Remove CRM_Contact_BAO_Contact::getPrimaryOpenId

It seems it's only ever called from aa test now

dev/core#1921 Remove more instances of civi 4.2 date handling

Per https://lab.civicrm.org/dev/core/-/issues/1921

Merge pull request #18146 from eileenmcnaughton/export_custom

Export fix on long custom fields

Make period_type mandatory for MembershipType

We have a test ensuring period_type is mandatory at the api level. However, it isn't for api
v4 - the error is a different issue (handling of pseudoconstants when mapping v3 to v4). This
fixes that separate issue & declares required to re-fix for v4

Merge pull request #18418 from eileenmcnaughton/ref

[ref] Extract failContribution code

add custom fields to contribution details

Merge pull request #18410 from agh1/buttonrama

Buttonrama

Merge pull request #16482 from eileenmcnaughton/super-perm

Implement more nuanced 'Administer CiviCRM permisions

Extract failContribution code

Basic extraction

Merge pull request #17641 from MegaphoneJon/core-1590

dev/core#1590: Don't send reminders on deleted events

Merge pull request #18232 from sunilpawar/dev_1942_new

dev/core#1942 handle multiple membership of same membership type to u…

Merge pull request #18413 from eileenmcnaughton/memtest

Membership form test cleanup, date cleanup on form

Merge pull request #18414 from seamuslee001/jaap_test

[REF] Refactor price field form to allow for unit testing of the form

[REF] Refactor price field form to allow for unit testing of the form

Set the assert equals to be an actual float number

Merge pull request #18415 from eileenmcnaughton/acl

[REF] Minor readability fix

Merge pull request #18076 from MegaphoneJon/better-on-behalf-of-2

[REF] change deprecated function to API4 call

Merge pull request #18297 from eileenmcnaughton/jaap

dev/core#2003 Civicrm_price_field_value.amount truncation when localisation in play

Membership form test cleanup, date cleanup on form

This converts the test set up to use the api and also does some mild date clean up on the form.

Supports https://github.com/civicrm/civicrm-core/pull/18395

[REF] Minor readability fix

None of the table names in this query are localisable so we don't need to use variables
to refer to them.

Note the legacy dao->query() would require it if they were localisable whereas CRM_Core_DAO::executeQuery()
does not

core#1590: Don't send reminders to add'l recipients on deleted/inactive/template events

support reminders with event types/templates and multiple event/type/template IDs

fix