Fix bug where tax_amount is miscalculated on membership renewals

In order to replicate
1) enable tax & invoicing
2) configure a financial type with sales tax
3) renew a membership, entering an amount other than the default membership amount.
4) view the membership - tax amount will be the same as it would be if the default amount had been used.

Tangental bugs I noticed
1) regression in master on adding financial accounts - I will do a separate PR
2) tax term not assigned to the template - out of scope for now but I believe a pre-hook type
structure should exist, assigning this to the template when enabled (as part of moving some code to
a core-extension).
3) the form shows  the tax amount & does not update it - this is pretty big already so that is also out of scope for now

This bug affects both membership & membership renewal pages. This PR only fixes for membership renewal
but the goal is to fix for membership too once this is merged. It is a fairly 'bug' fix in that it
introduces a new class with new methods to get the information otherwise provided by
CRM_Price_BAO_PriceSet::processAmount. processAmount is one of those functions that is hard to read
and understand & has been hacked to do various things. Code that calls it is hard
to read because of this. Having a cleaner way to operate has been a goal for a while. We
have plans to build up the Order api to be much more usable and used. However, we really
need to build up sensible helpers before we can address that. This PR adds a class
that has a whole lot of functions that are more readable to do what the processAmount otherwise
does. The goal is to deprecate processAmount.

I started on MembershipRenewal as
1) it has a bug currently and
2) it is pretty simple in that it doesn't actually support pricesets - which is especially
helpful when testing :-)

Merge pull request #16695 from mattwire/pcpshortcodes

Add shortcodes for Personal Campaign Pages

Merge pull request #16767 from civicrm/5.24

5.24

Merge pull request #16766 from seamuslee001/5_23_2_release_notes

release-notes/5.23.2.md

release-notes/5.23.2.md

Merge pull request #16761 from eileenmcnaughton/joomla

dev/financial#120 - Fix CiviCRM base url on Joomla frontend

Merge pull request #16759 from eileenmcnaughton/fatal2

Remove instances of fatal

Merge pull request #16764 from eileenmcnaughton/mem_tax

[NFC] cleanup in test class

[NFC] cleanup in test class

Merge pull request #16710 from colemanw/arrayValue2

[REF] Use ?? operator instead of CRM_Utils_Array::value()

Alternate Joomla! fix - move to CMS override

Merge pull request #16758 from pradpnayak/priceSetSchema

add pseudoconstant for location type

Remove instances of fatal

add pseudoconstant for location type

Merge pull request #16730 from demeritcowboy/unused-activitytypename

[REF] dev/core#1116 - Remove unused misnamed activityTypeName variable

Merge pull request #16754 from pradpnayak/priceSetSchema

Added pseudoconstant for priceset

iAdded pseudoconstant for priceset

Added pseudoconstant for priceset

Merge pull request #16694 from eileenmcnaughton/dedupebug

Fix bug where a % in a serialized array can lead to the data being broken

Fix bug where a % in a serialized array can lead to the data being broken

It turns out that is a field in a serialized array has a %2 (for example) this gets swapped in executeQuery for the
%2 value (in this case srcID - rendering the serialized array invalid. This proposes that we
explicitly handle arrays as  a data type in compose query.

Some thoughts
1) we could make serialized arrays valid types in validate (not done here)
2) we could iterate through the  array keys  & values escaping them -
at this stage it's left in the calling function
3) there are whole bikeshed factories to keep in business on discussion of whether
'Array-1', 'Array-2' etc are the right format

Use ?? operator instead of CRM_Utils_Array::value() in array assignments with null-ish defaults

Use ?? operator instead of CRM_Utils_Array::value() in array assignments

Use ?? operator instead of CRM_Utils_Array::value() when fetching ids

Merge pull request #16691 from mattwire/eventcart_cleanup

Event Cart cleanup

Merge pull request #16749 from civicrm/5.24

5.24

Merge pull request #16750 from eileenmcnaughton/sgroups

Fix smart group custom field  check to cope  with api error

Merge pull request #16741 from mattwire/addrecurid2

Add recurring contribution ID to doCancelRecurring

Merge pull request #16678 from seamuslee001/php_7_1_bump

Bump Minimum Install version to be 7.1 (Do not merge until 5.24 RC is cut)

Merge pull request #16651 from kartik1000/core-1613

dev/core#1613/Updated Misleading labels

Rebased

Signed-off-by: Kartik Kathuria <kathuriakartik0@gmail.com>

Add recur ID to doCancelRecurring

Fix smart group custom field  check to cope  with api error

As we have added new fields to the saved_search table this will lead to a fatal if it runs before
the database update. This catches that error and handles it

Merge pull request #16748 from seamuslee001/5_23_1_release_notes

5 23 1 release notes

Update 5.23.1.md

Add release-notes/5.23.1.md

Merge pull request #16744 from civicrm/5.24

5.24

Merge pull request #16740 from mattwire/addrecurid

REF Switch to getter on cancelSubscription form

Merge pull request #16743 from mlutfy/dev547c

dev/core#547 Fix Event financial_type_id getOptions

Merge pull request #16735 from totten/5.24-paths

dev/core#1637 - Multiple fixes for Civi/Core/Paths.php

dev/core#547 Fix Event financial_type_id getOptions

Merge pull request #16734 from eileenmcnaughton/array2

[REF] Minor code simplification

Switch to getter function for subscriptionDetails

E2E_Extern_CliRunnerTest - Update to match change in `./`

Per 16735, the interpretation of `getPath('[foo]/.`)` changed
slightly - from:

* 5.22: Inconsistent/undefined tail end (may or may not have trailing `/`)
* 5.23.0: Defined to always end with `/`
* #16735: Defined to never end with `/`

Merge pull request #16501 from eileenmcnaughton/cancel_recur

Update cancelSubscription  form to use updated methodology

Merge pull request #16732 from demeritcowboy/conformance

[REF] Hopefully fix regularly failing conformance test

Merge pull request #16718 from colemanw/respond

Improve conditional in api3 ChainSubscriber

[REF] Minor code simplification

Move the return earlier, simplify the IF

Merge pull request #16737 from artfulrobot/artfulrobot-array-values-followup

CRM_Utils_Array::values follow up from #16699

Fix array value accessing non existent properties on an ArrayAccess object

add failing test

Merge pull request #16699 from eileenmcnaughton/array

Support PropertyBag in CRM_Utils_Array

CRM_Core_I18n - Fix usage of `getPath('[foo]/.')`

The interpretation of `/.` is evolving per #16735:

* When this code was first written, it was unspecified/variable whether the value `[foo]/.` would end in `/`
* During most of the testing of 5.23.beta1, this was defined to always return `/`
* During a regression fix in 5.23.1, we're flipping it back the other way so that `[foo]/.` never ends in `/`.

Support PropertyBag in CRM_Utils_Array

We now use the PropertyBag in payment processors - but as @mattwire discovered the CRM_Utils_Array::value
function is commonly used to access values now potentially 'in the bag' and the 'is_array'
filtering here means it is not being returned

Merge pull request #16729 from colemanw/arrayValue4

[REF] Use empty() instead of CRM_Utils_Array::value() in inline conditionals

Merge pull request #16728 from civicrm/5.24

5.24

dev/core#1637 - Multiple fixes for Civi/Core/Paths.php

For a full write-up, see PR

dev/core#1637 - PathsTest - Redefine "[foo]/", "[foo]/0", "[foo]/."

The rationale will be discussed more via PR description.

(NFC) PathsTest - Make it easier to identify/debug test cases

When browsing the list of outputs from this test class, each of the test
cases was identified by its numerical position in the list of `$exs`.  This
makes it hard to keep track of the failures.

1. Add a symbolic name to each case (that's easier to search on)
2. Add more verbose output for failed assertions

(NFC) dev/core#1637 - Add E2E test for generation of Civi-WP backend URLs without trailing /

Revert "Revert "Make $civicrm_paths less sensitive to trailing slashes. Add tests.""

This reverts commit cbcbfd69d34c9895342e3d813a42f97d93dbb76d.

hopefully fix failing test

Merge pull request #16727 from pradpnayak/priceSetSchema

pseudoconstant for bounce type

Merge pull request #16719 from colemanw/arrayValue3

[REF] Use ?? operator instead of CRM_Utils_Array::value() in return statements

remove unused misnamed variable

Use empty() instead of CRM_Utils_Array::value() in inline conditionals

Merge pull request #16720 from agh1/5.24.0-release-notes

5.24.0 release notes: initial run

pseudoconstant for bounce type

Merge pull request #16711 from eileenmcnaughton/deprecated

Fix another instance of NULL being passed to CRM_Utils_Array::value

Merge pull request #16725 from civicrm/5.24

5.24

Merge pull request #16721 from seamuslee001/524_1637

dev/core#1637 Revert "Make $civicrm_paths less sensitive to trailing slashes. Add t…

Merge pull request #16692 from MegaphoneJon/financial-119

Fix PayPal Express on events

Revert "Make $civicrm_paths less sensitive to trailing slashes. Add tests."

This is currently causing breakage on wordpress sites where clean urls are not enabled.

Compare the 2 urls below - the top one has an extra (breaking) slash added by this PR.

I propose a quick revert  & patch release followed by 'the right' fix at a slower pace
/wp-admin/admin.php/?page=CiviCRM&q=civicrm%2Fupgrade%2Fqueue%2Fajax%2FrunNext
/wp-admin/admin.php?page=CiviCRM&q=civicrm%2Fupgrade%2Fqueue%2Fajax%2FrunNext

This reverts commit 232fdd3dfac4ea09749d7f178818139514e48e38.

5.24.0 release notes: added boilerplate

5.24.0 release notes: raw from script

Use ?? operator instead of CRM_Utils_Array::value() in return statements

Improve conditional in api3 ChainSubscriber

Merge pull request #16713 from eileenmcnaughton/regress

dev/core#1637 regression fix on WP urls

Merge pull request #16712 from eileenmcnaughton/dep2

Fix another instance of silly use of CRM_Utils_Array::value

Merge pull request #16693 from eileenmcnaughton/prior

[NFC] Fix comments, use single quotes

Revert "Make $civicrm_paths less sensitive to trailing slashes. Add tests."

This is currently causing breakage on wordpress sites where clean urls are not enabled.

Compare the 2 urls below - the top one has an extra (breaking) slash added by this PR.

I propose a quick revert  & patch release followed by 'the right' fix at a slower pace
/wp-admin/admin.php/?page=CiviCRM&q=civicrm%2Fupgrade%2Fqueue%2Fajax%2FrunNext
/wp-admin/admin.php?page=CiviCRM&q=civicrm%2Fupgrade%2Fqueue%2Fajax%2FrunNext

This reverts commit 232fdd3dfac4ea09749d7f178818139514e48e38.

Fix another instance of silly use of CRM_Utils_Array::value

Merge pull request #16703 from pradpnayak/priceSetSchema

Added pseudoconstant for civicrm_tag.parent_id

Fix another instance of NULL being passed to CRM_Utils_Array::value

Note that the test has only cosmetic changes picked up while running the test to identify the bug

Merge pull request #16709 from civicrm/5.24

5.24

Merge pull request #16707 from demeritcowboy/smartgroup-warning

dev/core#1636 - In system status check avoid E_NOTICEs for smart groups that don't have form_values[0]

Merge pull request #16706 from eileenmcnaughton/array2

Remove an instance of passing non-arraay to CRM_Utils_Array::value(

Merge pull request #16704 from colemanw/arrayValue1

[REF] CRM_Utils_Array::value() -> empty()

skip smart groups that don't have a form_values[0]

Remove an instance of passing non-arraay to CRM_Utils_Array::value(

Merge pull request #16696 from eileenmcnaughton/remove

Remove calls & deprecate CRM_Core_BAO_PrevNextCache::setItem

Merge pull request #16705 from mlutfy/api4-ajax-perm

Api4 AJAX endpoint: change required permission to access AJAX API.

CRM_Utils_Array::value() -> empty()

Replaces CRM_Utils_Array::value() with empty() when the value is immediately cast to boolean.
The two statements have identical meaning but the latter is faster & easier to read.

Api4 AJAX endpoint: change required permission to access AJAX API.

Flush pseudoconstant cache when saving a tag

Merge pull request #16701 from eileenmcnaughton/array2

Fix passing a non-array to CRM_Utils_Array::value

Bump Minimum Install version to be 7.1

Fix passing a non-array to CRM_Utils_Array::value

As surfaced in https://test.civicrm.org/job/CiviCRM-Core-PR/32543/console when we tested deprecating
passing in a non-array

Added pseudoconstant for civicrm_tag.parent_id

Merge pull request #16690 from kartik1000/Shift_Update_Membership

Shifted UpdateMemberhsip to BAO Layer

Merge pull request #16689 from pradpnayak/priceSetSchema

Use dbAlias to generate where clause for date field in activity report