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

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

Merge pull request #18317 from eileenmcnaughton/bug_test

Add postAssert to check payments and contributions are valid on all tests.

Merge pull request #18412 from civicrm/5.30

5.30

Merge pull request #18409 from eileenmcnaughton/j

Fix formatLocaleNumericRoundedByCurrency

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

dev/core#1603 fix tangental bug on form handling of long options

This fixes the bug that derailed the fix for https://lab.civicrm.org/dev/core/-/issues/1603

On further testing I agree with Jitendra that the price field form was mis-saving the longer values (I only
tested with Euro style decimal separators but maybe on both) and it was bad data caused by this
that made the merged fix look like a regression.

I also observed that the Money functions intended to round only appeared to since it was rounding
just fine to 2 places, as tested, but it was still rounding to 2 places when we wanted more.

This fixes both the New Price field screen (when options are entered) and the edit options screen

Remove obsolete OFC flash-to-image code

Fix formatLocaleNumericRoundedByCurrency

The function was rounding by a max of 2 regardless of the number of places passed - it
was only tested for 2 so seemed to work

More radical cleanup of button CSS

Fix CSS for new buttons

Fix JS for input buttons that are now button buttons

Remove deprecated submitOnce js function

No need for crm-button wrappers for real buttons

Give buttons a value when the submitted value is used to identify them

Put icons inside of button elements

Swap out button/submit inputs for button elements