Fix conditional

Merge pull request #28296 from eileenmcnaughton/tax

Fix notice on unassigned tax

Merge pull request #28292 from eileenmcnaughton/main_am

Clean up `getMainContributionAmount()` function

Fix notice on unassigned tax

Merge pull request #28281 from eileenmcnaughton/part

Use local variable rather than undefined property to build array

Merge pull request #28284 from eileenmcnaughton/renew

Minor consolidation on `isTest()`

Merge pull request #28290 from braders/loaded-mapping-id

[REF][PHP8.2] Remove unused property $_loadedMappingId

Clean up getMainContributionAmount() function

Merge pull request #28287 from eileenmcnaughton/current_membership

Extract code that defines the renewal membership

[REF][PHP8.2] Remove unused property $_loadedMappingId

Merge pull request #28288 from eileenmcnaughton/address_opps

Fix recently-merged typo

Merge pull request #28166 from eileenmcnaughton/part_send

Update send_receipt once actually sent, rather than predicting

Fix recently-merged typo

Extract code that defines the renewal membership

Merge pull request #28021 from colemanw/afformErrorMsg

Afform - improve display of error messages

Merge pull request #28115 from colemanw/chainSelectFix

dev/core#4702 - Fix country/state chainSelect validation when country is deselected

Merge pull request #28226 from eileenmcnaughton/no_billing

Stop passing billingID to `CRM_Utils_Address::getFormattedBillingAddressFieldsFromParameters`

Merge pull request #28280 from eileenmcnaughton/billing

Use function for getBillingID() rather than relying on form property

Merge pull request #28279 from mlutfy/recurringRequired

Recurring Activity: avoid validation if the fieldset is visible but not used

Merge pull request #28235 from eileenmcnaughton/test_clean

Minor test clean up

Minor consolidation on isTest()

I think this makes sense as a publicly supported function we should consolidate in as many forms
as possible

Merge pull request #28261 from eileenmcnaughton/prod

Add Premium Product v4 api

Merge pull request #28270 from colemanw/civiCaseCleanup

CiviCase - Clear value before reusing editCaseRoleDialog

Merge pull request #28221 from eileenmcnaughton/price_set

Fix always present parameter

Add pseudoconstant look up for entity tables

Per the code comment only contribution page

CiviCase - Simplify overcomplicated code that hides inactive roles

Add premium api

Add Premium Product v4 api

Use local variable rather than undefined property to build array

This variable is specific to the function it lives in

Use function for getBillingID() rather than relying on form property

Merge pull request #28229 from sunilpawar/dev_contactID_reminder

Pass contact id to sendmail function

Recurring Activity: avoid validation if the fieldset is visible but not used

Correct contact ID name

Merge pull request #28254 from eileenmcnaughton/premium_deduct_more

Unshare toxic function `getContributionParams()`

Merge pull request #28275 from colemanw/isDeleted

AdminUI - Remove deleted contacts from group count

Merge pull request #28274 from colemanw/searchBuilderNotice

SearchBuilder - Add deprecation notice to users

Merge pull request #28256 from eileenmcnaughton/get_tax_details

Fix getTaxDetails notices on Confirm page

Merge pull request #28271 from eileenmcnaughton/payment_function

dev/core#4798 Fixes to functions to determine if isMembership, isSeparatePayment

Fix test to have correct input params

The test was exploiting a code gremlin which did not
require the right submitted values to be set to trigger
a secondary contribution - this fixed.

other_amount is not a real field as the presented field is always price_x

Merge pull request #28257 from wmortada/core#4791

dev/core#4791 - Set cache folder for dompdf

AdminUI - Remove deleted contacts from group count

Fixes https://lab.civicrm.org/dev/core/-/issues/4792

SearchBuilder - Add deprecation notice to users

Fixes to functions to determine if isMembership, isSeparatePayment

In the case of isSeparatePayment we have a function to determine if the form
supports separate payments - but are using this interchangeably with whether the
user has selected more than one option (if they only selected 1 there is only 1 payment)

Merge pull request #28264 from eileenmcnaughton/lines

[LineItem cleanup] Standardise a couple of calls to get values from the line items

CiviCase - Clear value before reusing editCaseRoleDialog

Merge pull request #28269 from eileenmcnaughton/thank

Superficial cleanup in Contribution Thank You form

Merge pull request #28266 from eileenmcnaughton/line_member

[LineItem cleanup] Clean up & clarify code splitting line items in Contribution page membership flow

Merge pull request #28258 from christianwach/lab-core-4735

Use currentPath method to build QuickForm post URL

Set cache folder for dompdf

Merge pull request #28240 from eileenmcnaughton/standardise

Online membership receipt simplification

Merge pull request #28267 from eileenmcnaughton/property

Replace ref to undeclared property with tax-inclusive lookup

Clean up & clarify code splitting line items in Contribution page membership flow

It was really confusing - but now it is separated into a function which explains it

Note there is specic test cover in
testSubmitMembershipComplexPriceSetPaymentPaymentProcessorRecurInstantPayment
and the changes to that test reflects the fact that the order in which
the line items are created changes slightly (in a fundamentally neutral way

Superficial cleanup in Contribution THank You form

Replace ref to undeclared property with tax-inclusive lookup

Standardise a couple of calls to get values from the line items

Merge pull request #28265 from civicrm/5.68

5.68

Merge pull request #28220 from eileenmcnaughton/membership_log

Add Membership log entity

Merge pull request #28238 from eileenmcnaughton/premium

[Premiums] Fix access to undeclared property, remove unreachable code

Merge pull request #28250 from eileenmcnaughton/premium_2

Fix notices on manage premiums page

Merge pull request #28246 from eileenmcnaughton/profile

dev/core#4766 Fix event merging profiles together

Merge pull request #28224 from eileenmcnaughton/remove_unused

Remove unused property from previously shared code `_membershipTypeValues`

A couple more minor fixes

Online membership receipt simplification

Rather than having one section in an IF for the scenario where we have a separate payment situation
but there is no contribution payment, consolidate this scenario with the general 'we
have a membership but no contribution in the receipt

Fix event merging profiles together

https://lab.civicrm.org/dev/core/-/issues/4766

Use currentPath method to build QuickForm post URL

Merge pull request #28253 from eileenmcnaughton/premium_deduct

Unshared toxic function `getNonDeductibleAmount()`

Merge pull request #28215 from eileenmcnaughton/amount_level

Stop setting unused key

Merge pull request #28234 from eileenmcnaughton/line_pass

Reduce passing of `lineItems`

Merge pull request #28211 from eileenmcnaughton/smarty3

Use token to avoid breaky-smarty3-maths

Merge pull request #28214 from eileenmcnaughton/message_improve

Improve test output message

Merge pull request #28222 from eileenmcnaughton/current_member

Replace undefined property `_currentMemberships` with a define

Merge pull request #28227 from eileenmcnaughton/express

Declare property from paypal express as public

Merge pull request #28233 from eileenmcnaughton/second

Clarify which line items are passed in` processSecondaryMembership`

Merge pull request #28232 from eileenmcnaughton/assign

Remove assignments for template that are not needed for updated template

Merge pull request #28252 from eileenmcnaughton/thank_notice

Extend line item notice fix from Confirm to Thank you page

Fix getTaxDetails notices on Confirm page

Merge pull request #28247 from civicrm/5.68

5.68

Unshare another toxic function

Unshared toxic function getNonDeductibleAmount

Extend line item notice fix from Confirm to Thank you page

Merge pull request #28251 from eileenmcnaughton/fix_regress

Fix regression on line item display on thank you

Fix regression on line item display on thank you

Fix notices on manage premiums page

This addresses notices at
https://dmaster.localhost:32353/civicrm/admin/contribute/managePremiums?reset=1

These are probably also addressed by the admin ui / sk conversion but note that this also
covers the tab on the contribution page

Note that combined with https://github.com/civicrm/civicrm-core/pull/28238
this fixes most of the issues I encountered trying to configure a premimum
but there are still a bunch of notices when I attempt to use one.

Also, having switched to using apiv4 I have added some tpl escaping

Merge pull request #28244 from eileenmcnaughton/568

Fix reference to wrong value

Merge pull request #28243 from demeritcowboy/lost-time

dev/financial#221 - Bring back missing time field

Fix reference to wrong value

Merge pull request #28241 from civicrm/5.68

5.68

bring back time field

Merge pull request #28239 from eileenmcnaughton/568_prod

Fix regression in 5.68 on selecting a premium (when at least one is unavailable)

Merge pull request #28237 from eileenmcnaughton/568

Add weight to premium config (tab on contribution page edit)

Fix regression in 5.68 on premium

Single appears to be never true on this form too

Fix access to undeclared property, remove unreachable code

Add weight to premium config (tab on contribution page edit)

Merge pull request #28236 from civicrm/5.68

5.68 to master

Merge pull request #28231 from agh1/5.68.0-releasenotes-initial

5.68.0 release notes initial run

Minor test clean up

The test is calling preProcess so doesn't need these hacks

Reduce passing of lineItems

The get() function is called when setting the order line items so calling getLineItems()
will return the same thing. The value passed as formLineItems is simply the value
from getLineItems() so rather than pass it get it at the point where it is needed

Clarify which line items are passed in processSecondaryMembership

It has traditionally been very confusing what the line items are when they get to
this point - but now we have a function that gets them so call it at the point when needed rather than
pass the array around

Remove more assignments & create some noise