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

Remove unused property, null constructor

Merge pull request #18419 from colemanw/improveCustomFieldForm

Custom field form reform

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

Reset button height in the WP backend

Simplify button padding and set consistent line height

remove IE6-only CSS

Remove unused button-crm-i CSS

Remove weird styling for "Configure CKEditor" button

Standardize single space (not nbsp) between icon and text

Eliminate support for old jQuery UI based crm-button-icon icon buttons

No need for crm-i-button class

Icons on buttons positioned inline, not absolute

5.30 release-notes: first stab at release notes

Use title instead name in status message

Merge pull request #18365 from eileenmcnaughton/member

[REF] Code cleanup on membership renewal & test

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

Remove error handling from loadObjects

The processors now have their own try->catch so we don't need to catch here.

I wasn't sure about the debug_log_message part so I copied it into the processors
, at risk of duplicating

Comment out implied permissions for administer CiviCRM system per discussion

Fix search extension excluded from tarball

Implement 'super permissions' as described by @totten

This picks up on an idea Tim has pushed several times - ie that instead of giving out 'Administer CiviCRM' willy nilly
we could deprioritise it in favour of 2 more granular permission bundles - ie Administer CiviCRM data & administe CiviCRM system.

This allows us to make some permissions more 'locked away' without endlessly adding new 'administer Payment Processors'
because we've realised not everyone who can create profiles needs to be able to see payment processor credentials.

It also allows us to make system checks less broadly visible where they are not appropriate.

Note that to proceed with this we would need to go through all places that check Administer CiviCRM & put in one
or both of the 2 new permissions. Having Administer CiviCRM implicitly includes anything granted to the existing
permissions so the implementation is smooth-ish there. However, I can imagine we would need a hook allowing people
to categorise themselves or we would find ourselves litigating all sorts

Merge pull request #18401 from seamuslee001/update_eway_upgrade

[REF] Improve the human readable name of the eway upgrade step to be …

[REF] Improve the human readable name of the eway upgrade step to be clearer about what it is doing and also fix issue with a hard fault if the eway single currency payment processor type has been already removed

Merge pull request #18378 from colemanw/noMoreStupidHTMLTypes

Remove redundant custom field types

Merge pull request #18399 from megaphonetech-dennis/contact-form-task_delete-php_spelling-fix

Contact form task delete php spelling fix

Merge pull request #18245 from eileenmcnaughton/loop

Fix for ongoing issues with static upsetting the apple cart

Merge pull request #18396 from swastikpareek/issue-1987-fix-template

dev/core#1987: Enable frontend and backend theme configuration in display preference page for Drupal CMS

dev/core#1854: Add Test to Check Bug Remains Fixed

Added test to check updating a membership's status doesn't cause the wrongful
deletion of related memberships.

fixed spelling of address on lines 122 and 247

fixed capitalization on line 247

fix capitalization fix on line 122

Merge pull request #18309 from demeritcowboy/link-cases-change-status

dev/core#1979 - Incorrect comparison of status_id when changing status of linked cases

Merge pull request #18381 from eileenmcnaughton/depcom

Finish deprecating BaseIPN->completeTransaction

Merge pull request #18372 from eileenmcnaughton/inv

Fix Invoice class to not call validateData

Merge pull request #18373 from eileenmcnaughton/trans

[REF] Simplify loading of related objects in transition components