Suppress notice

Remove unnecessary pass-by-ref

Merge pull request #28339 from colemanw/inlineArray

[NFC] Array formatting cleanup

Merge pull request #28316 from eileenmcnaughton/prem_2

Check financial acls for product in extension

Array formatting cleanup

Multiline arrays aren't always the most readable.
These were all done by an autoformatting tool which got a little too zealous
about breaking every array into multiple lines. They are especially silly within if() statements.

Merge pull request #28336 from eileenmcnaughton/part_form

Remove handling from previously shared code

Merge pull request #28365 from eileenmcnaughton/confirm_notices

Ensure onBehalfEmail always assigned to prevent notices

Merge pull request #28360 from eileenmcnaughton/code

Remove unreachable code

Remove handling from previously shared code

Ensure onBehalfEmail always assigned to prevent notices

Merge pull request #28362 from eileenmcnaughton/notices

Add notice fix for Confirm & thank you pages

Merge pull request #28358 from eileenmcnaughton/thank_notice

Thank you page notice - ensure friendText & linkText is assigned

Merge pull request #28359 from eileenmcnaughton/social

Notice fix - specify emailMode is false when including SocialNetwork.tpl

Add notice fix for Confirm & thank you pages

Remove unreachable code

Notice fix - specify emailMode is false when including SocialNetwork.tpl

Move the pre-assign of pcpBlock further up

This is assigned false, & then assigned again if there is a value to prevent notices (already)
but the first assign isn't always done so we still have notices

Thank you page notice - ensure friendText & linkText is assigned

Merge pull request #28357 from eileenmcnaughton/price_get

Fix fatal adding price set on contribution page

Fix fatal adding price set on contribution page

Merge pull request #28353 from civicrm/5.68

Merge 5.68 to master

Merge pull request #28343 from artfulrobot/issue-core-4697

Update path for user profile page to avoid conflict #4697

Merge pull request #28352 from bsilvern/core-4761-5.68

dev/core#4761 - CRM\Core\DAO - Generate unique name, case-insensitive

Merge pull request #28350 from eileenmcnaughton/currency

Get currency from the event for backoffice participant form

Merge pull request #28346 from eileenmcnaughton/remove_deprecated

Remove unused functions noisily deprecated in 2021

Merge pull request #28347 from eileenmcnaughton/main_form

PHP 8.x add getFromEmails() function, remove use of undeclared property

Merge pull request #28345 from eileenmcnaughton/undefined

Make undefined property local variable

dev/core#4761 - CRM\Core\DAO - Generate unique name, case-insensitve

This duplicates on the 5.68 branch changes made in PR-28168 to master.

Get currency from the event

Merge pull request #28276 from eileenmcnaughton/isset

dev/core#4798 Stop setting unused, undefined property `_lineItem` in contribution forms

Consolidate fee preProcess into feeBuildForm

Merge pull request #28289 from eileenmcnaughton/part_send

Use getCurrency() function for currency retrieval

PHP 8.x add getFromEmails() function, remove use of undeclared property

Remove unused functions noisily deprecated in 2021

Merge pull request #28342 from braders/remove-tempclause

Remove unused _tempClause

Add deprecation notices

Make undefined property local variable

Merge pull request #28341 from braders/contact-selector-unused-param

[REF][PHP8.2] Remove unused property from CRM_Contact_Selector

Consolidate getCurrency()

Use getCurrency() function for currency retrieval

Merge pull request #28308 from eileenmcnaughton/discounts

Simplify handling of discount_id in participant form postProcess

Merge pull request #28310 from eileenmcnaughton/billing

Deprecate passing `billingID`to `mapParams`

Update path for user profile page to avoid conflict #4697

Remove unused _tempClause

[REF][PHP8.2] Remove unused property from CRM_Contact_Selector

Merge pull request #28340 from colemanw/suppertime

Go to bed without supper 🥪🚫

Go to bed without supper

Merge pull request #28314 from demeritcowboy/php8-page

[php8] undeclared property action and id in Page

Merge pull request #28262 from eileenmcnaughton/strings

Replace a few sub_str with str_contains

Merge pull request #28337 from eileenmcnaughton/ref

Remove unnecessary pass-by-reference

Merge pull request #28307 from eileenmcnaughton/part_amount

Use form property rather than passing mode

Merge pull request #28335 from eileenmcnaughton/event_car

Fix eventcart to not interact with _tpl_vars directly

Merge pull request #28334 from eileenmcnaughton/merge_screen

Fix Smarty3 notices on merge screen

Remove unnecessary pass-by-referenc

Fix eventcart to not interact with _tpl_vars directly

Fix Smarty3 notices on merge screen

Merge pull request #28330 from eileenmcnaughton/location_check

Add system check for if people disabled default location type

Merge pull request #28331 from colemanw/getViewOnlyActivityTypeIDs

Activity - Fix backwards logic in getViewOnlyActivityTypeIDs

Update CRM/Contribute/Form/Contribution/Confirm.php

Co-authored-by: colemanw <coleman@civicrm.org>

Merge pull request #28327 from eileenmcnaughton/data_test

Fix dataSource test to test permissions

Activity - Fix backwards logic in getViewOnlyActivityTypeIDs

Add system check for if people disabled default location type

Merge pull request #28329 from colemanw/addSelectWhereClauseUser

BAO - Handle hypothetical userId in addSelectWhereClause

BAO - Handle hypothetical userId in addSelectWhereClause

This is aspirational - CiviCRM doesn't fully support checking permissions for anyone other than the current user,
but this nudges us a bit closer to what that support would look like.

Merge pull request #28328 from civicrm/5.68

5.68 to master

Merge pull request #28325 from colemanw/userJobClause

UserJob - Fix deprecated syntax in addSelectWhereClause

Fix dataSource test to test permissions

Replace a few sub_str with str_contains

use our more recent functions

Merge pull request #28323 from colemanw/strstr

[REF] Switch strstr to str_contains

Merge pull request #28324 from eileenmcnaughton/comment

Update comment todos to referred to preferred method

UserJob - Fix deprecated syntax in addSelectWhereClause

Update comment todos to referred to preferred method

Merge pull request #28322 from MegaphoneJon/price-field-shutup

don't warn users about invalid financial types on unused price sets

Stop setting unused, undefined property _lineItem

This removes the last reference to _lineItem in the Contribution flow.

However, magic methods are added to support most extension access to them

Switch strstr to str_contains

When cast to boolean by a conditional, these 2 functions behave identically, but str_contains is more readable

don't warn users about invalid financial types on unused price sets

Merge pull request #28298 from eileenmcnaughton/lines

Remove code cruft which is now ignored

Remove now-unused handling for amount, line items

Merge pull request #28285 from eileenmcnaughton/renew_notice

Fix notice on membership online main page

Merge pull request #28312 from eileenmcnaughton/amount_level

Move getAmountLevel() to Order class

Merge pull request #28315 from eileenmcnaughton/prem

Copy buildPremiumsBlock back to contribution page base

Merge pull request #28317 from eileenmcnaughton/tax

Fix tax when selection a contribution with the membership, test

Merge pull request #28318 from eileenmcnaughton/requires

Make financialacls require civi-contribute

Merge pull request #28319 from omarabuhussein/fix-priceset-single-line

Set correct financial type when creating membership using price set with single line item

Components - Disable reverse dependencies when disabling

Set correct financial type when creating membership using price set with single line item

Make financialacls require civi-contribute

Check financial acls for product in extension

Fix tax when selection a contribution with the membership, test

Merge pull request #28309 from eileenmcnaughton/notice_stand

Notice fix

Update CRM/Contribute/Form/AbstractEditPayment.php

Co-authored-by: demeritcowboy <demeritcowboy@hotmail.com>

Merge pull request #28306 from eileenmcnaughton/currency_test

dev/core#4802 Fix amount truncation with non-English formatting

Copy buildPremiumsBlock back to contribution page base

Merge pull request #28176 from mattwire/downloader

Log extension download errors

Merge pull request #28313 from demeritcowboy/regen

Regen after 28268

php notices

regen

Merge pull request #28268 from eileenmcnaughton/no_online_cont

Blank out text version of online_contribution_receipt

Merge pull request #28295 from eileenmcnaughton/confirm_notice

Add some assigns to prevent notices on Confirm

Move getAmountLevel() to Order class