Merge pull request #19658 from eileenmcnaughton/member

[REF] Move code that relates to receipts when paying by credit card out of the shared function

Merge pull request #19677 from totten/master-joomla-syncuser

Joomla::synchronizeUsers - Fix notice due to old style reference

Merge pull request #19679 from civicrm/5.35

5.35 to master

Merge pull request #19678 from totten/master-guzzle-url

GuzzleMiddleware::url() - Distinguish frontend and backend routes

GuzzleMiddleware::url() - Distinguish frontend and backend routes

Before: The `route://` scheme assumed backend access.

After: The `frontend://` and `backend://` schemes are unambiguous.
       The `route://` scheme will make a best-guess by determining
       if the menu item has `is_public`.

Joomla::synchronizeUsers - Fix notice due to old style reference

This fixes a PHP notice when synchronizing all users/contacts:

PHP Notice:  Only variables should be assigned by reference in ...CRM/Utils/System/Joomla.php:855

Merge pull request #19672 from colemanw/contentEditableUI

Afform - move contentEditable directive into crmUI library for reusability

Merge pull request #19662 from colemanw/searchDisplayRewrite

SearchKit - cleanup display configuration and add "rewrite" feature

Merge pull request #19671 from jaapjansma/dev_2424

Fixes dev/core#2424 navigation items order

Afform - move contentEditable directive into crmUI library for reusability

Fixes dev/core#2424 navigation items order

Merge pull request #19670 from eileenmcnaughton/aff

Switch afform back to '=' to unbreak deduper

Switch afform back to '=' to unbreak deduper

https://github.com/eileenmcnaughton/deduper/pull/9

Merge pull request #19669 from eileenmcnaughton/cont_status

Metadata: Add html:label for Contribution Status ID

Metadata: Add html:label for Contribution Status ID

Merge pull request #19667 from eileenmcnaughton/recurse

Fix master-only regression

Fix master-only regression

It seems the tests ran before the test that covered this
was merged giving a false positive

See https://github.com/civicrm/civicrm-core/pull/19478#issuecomment-785388559 for rationale

Merge pull request #19638 from colemanw/entityRefId

 EntityRef - Search by id when an integer is entered

Merge pull request #19664 from demeritcowboy/case-rel-minor-cleanup2

[NFC] minor cleanup in testCaseCreateWithResolvedStatus

Merge pull request #19663 from demeritcowboy/case-rel-minor-cleanup

Minor cleanup - ChangeCaseStatus

Merge pull request #19665 from seamuslee001/date_search_select

Afform: Only permit Date fields to be made into a select list if it i…

EntityRef - Fix merging api params when searching by id

Afform: Only permit Date fields to be made into a select list if it is a search form

Merge pull request #19632 from colemanw/afformDateRange

Afform - Support date range filters for search displays

minor cleanup

minor cleanup

SearchKit - add option to rewrite field output in displays

Merge pull request #19478 from eileenmcnaughton/ref

[REF] do not needlessly pass as reference, enforce valid param

Merge pull request #19610 from ahed-compucorp/dev/core#2372

dev/core#2372 Enable submitOnce for some forms

Merge pull request #19661 from civicrm/5.35

5.35 to master

Merge pull request #19659 from demeritcowboy/revert-18782

revert 18782

Merge pull request #19655 from mlutfy/tr32fix

dev/translation#32 Contribution ThankYou: partial fix for recurring units

SearchKit - More efficient use of space when configuring display fields

SearchKit - Move common displayAdmin fields into shared templates

Merge pull request #19643 from colemanw/searchKitButtons

SearchKit - Don't save on enter key

dev/translation#32 Contribution ThankYou: partial fix for recurring units

revert 18782

Merge pull request #19650 from tapashdatta/Enable-"Do-no-trade"-filter-1

Enable-"Do-no-trade"-filter

Merge pull request #19645 from colemanw/afformLink

Afform - Add 'View Page' link from afform edit screen

Merge pull request #19657 from totten/master-aff-perm-blank

dev/core#2404 - Afform - Resolve ambiguity of omitted permissions

[REF] Move code that relates to receipts when paying by credit card out of the shared function

The emailReceipt function is shared between the Batch Entry form and the backoffice
credit card form, while the emailMembershipReceipt is only used in this function.

I aim to copy the function onto the Batch Entry form & split them up but
am first moving this out of the shared code as it only
relates to credit card payments, which are not reachable from the Batch Entry form

Merge pull request #19654 from eileenmcnaughton/member

[REF] Simplfy parameter

Merge pull request #19648 from eileenmcnaughton/mm

dev/core#778 Do not increment tax on validation error

(dev/core#2404) afform_core - If "permission" is empty-string, then use default

(dev/core#2404) afform_admin - Treat "permission" as a required field

[REF] Simplfiy parameter

Rather than pass in contributionParams & params & combine in the function,
combine when passing in. Note receive_date is always 'now' in this
workflow as it is specifically the card payment workflow (for recurring)
being altered here & that field is not displayed on
credit card payments

Merge pull request #19656 from civicrm/5.35

5.35 to master

Merge pull request #19653 from eileenmcnaughton/535

dev/core#2360 - Escape the word `rows` in sql query

Merge pull request #19652 from JMAConsulting/money_defaults

Use specific function when formatting money for a default

Merge pull request #19651 from lunar-debian/fix-i18n-data-templates-changelog

Fix i18n issue with changeLog in data templates

dev/core#2360 - Escape the word `rows` in sql query

Use specific function when formatting money for a default

Merge pull request #19549 from eileenmcnaughton/member

[REF] Use form property rather than formValues to pass generated invoice ID on membership forms

Fix i18n issue with changeLog in data templates

There was a small mistake with the translation delimiters of `Change Log` in data templates.

Merge pull request #19649 from eileenmcnaughton/bfhe

Mark Old Cividiscount hook support deprecated

Update Form.php

dev/core#778 Do not increment tax on validation error

After digging into & cleaning up the code that handles tax on the membership form
I concluded that we can fix https://lab.civicrm.org/dev/core/-/issues/778 without
regressing https://issues.civicrm.org/jira/browse/CRM-18827 by simply doing less.
If the form has been already submitted, just load up what was submitted, do not
call the script to mess with stuff.

The script does 3 things
1) checks or unchecks the record_contribution checkbox. This is fine
to leave alone as it will have the right values on new & reload
without intervention
2) sets financial type id - ditto - only needs to be changed
when something else changes
3) sets total_amount - ditto
4) sets the text that says how much tax is included

With this patch 4 will not happen on a validation error. I think
this is OK as it is a bit of an edge case and no message is
better than the 2 previous issues of the amount being wrong.

I did think about fixing but it adds a lot of complexity that
will be easy to do once other changes are done to make
accessing the tax_amount easier, but which are relatively hard
now & my guess is that no-one will notice it not being there

Assign tax_rate directly rather than looking up via financial_type

Further simplification

Merge pull request #19647 from eileenmcnaughton/more

Remove unreachable code & variable cleanup in membership back office form template

Merge pull request #19633 from eileenmcnaughton/campaign

[Regression] campaign name no longer accepted

Afform Admin - Ensure buttons have 'type' declared

Variable simplification - assign membershipType to variable

[REF] Minor clarification - rename membershipType to membershipTypeID

Remove unreachable code

Term is always set at this point unless we are dealing with a priceset - which
this handling appears to be for, ergo term is always set.

We also know that this code is unreachable as in the 6 years it has been in the
codebase no-one has reported the bug in it. It uses total_amount not total_amount_numerci
in the first part of the calculation - this would fail when the decimal separator
is not a decimal

[REF] Remove unused variables

existingAmount is never used except in this if so cannot be true. feeTotal
is used above this point but not below it - so this code is extraneous

Afform - Add 'View Page' link from afform edit screen

Mark Old Cividiscount hook support deprecated

This seems to have existed to support a really old approach
(pre regions). It's gone from cividiscount with
https://github.com/civicrm/org.civicrm.module.cividiscount/pull/251
merged so this just adds comments

APIv3 - Legacy support for campaign_id fields which used to have a pseudoconstant

Merge pull request #19605 from eileenmcnaughton/money

dev/translation#65 Further remove moneyValueFormat

SearchKit - Don't save on enter key

Merge pull request #19626 from colemanw/multiRecordCustomViewLink

SearchKit - Add links to view multi-record custom data

Merge pull request #19618 from colemanw/searchAliasFix

dev/core#2388 SearchKit - Fix display of pseudoconstant fields

Merge pull request #19640 from mattwire/tokendeprecated

Remove single use of extraParams in CRM_Utils_Token::getTokenDetails()

Remove single use of extraParams in CRM_Utils_Token::getTokenDetails()

Merge pull request #19609 from demeritcowboy/case-views

[NFC/Unit Test] dev/core#2385 - Add test for getCaseActivityQuery

EntityRef - Search by id when an integer is entered

Merge pull request #19348 from sunilpawar/dev_age_at_deceased_date

Show Contact age on deceased date

Merge pull request #19637 from demeritcowboy/static-event

dev/event#50 - Non static function called statically for event ical downloads

non static function called statically

Merge pull request #19631 from colemanw/searchKitLinks

SearchKit - Enable links for implicit joins

Merge pull request #19636 from demeritcowboy/saventest

dev/core#2398 - Fix save'n'test button on mail account form

Merge pull request #19620 from colemanw/searchKitSelectCurrentUser

Search kit select current user

Merge pull request #19597 from eileenmcnaughton/nfc

[REF] simplify parameter, use preferred strict

Merge pull request #19608 from totten/master-wp-status

WordPress - If we hit a "permission denied" error, return HTTP 403

Merge pull request #19625 from colemanw/fixTs

Fix incorrect use of ts when setting custom data form title

fix save'n'test button

Merge pull request #19634 from eileenmcnaughton/iif

dev/financial#166 Clean up money formatting in iif

Merge pull request #19627 from colemanw/readonly

Metadata - Add 'readonly' to id fields, custom fields & contact fields

dev/financial#166 Clean up money formatting in iif

This does not turn out to be necessary but I tested & it works & is more correct

Afform - Support date range filters for search displays

Merge pull request #19579 from totten/master-aff-test

Afform - Allow public submission (PUB-CONTACT). Expand E2E test coverage.

mockPublicForm.test.php - Relax assertion

Note that the actual type reported varies slightly by CMS/configuration.
assertContentType() should factor-out those differences. We just need
loose claim here.

SearchKit - Enable links for implicit joins

Automatically create links for any implicitly joined entity linked through an fk field

Fix APIv4 unit test for 'readonly' attribute

Regenerate DAOs with readonly attribute for id fields

Add 'readonly' attribute to schema fields

For core fields, applies to primaryKey fields and fields tagged <readonly>.
For custom fields, this comes through via the 'is_view' property.

This will help autogenerated forms to know whether a field is appropriate to show to the user.

SearchKit - Add links to view multi-record custom data

Ideally custom entities would have a full set of CRUD links, but the form code is a bizarre mess
and requires a bunch of extra url params that have to be calcultated when constructing the links.

Just getting the VIEW link was difficult because I had to remove the requirement of passing CID
& teach the form to look it up.

Fix incorrect use of ts when setting custom data form title

Merge pull request #19598 from eileenmcnaughton/msg_tpl_less

Convert Smarty & domain token processing to use token processor