dev/core#1282, 1324 unit test for bug

Merge pull request #15541 from ixiam/dev_issue#1324

Adds missing join on the customtable of address customfields while exposed as profile

Merge pull request #15634 from eileenmcnaughton/mailing_date

Improve metadata support for table civicrm_mailing_job in search

Merge pull request #15633 from seamuslee001/dev_core_561_mailing

dev/core#561 Convert mailing date search field to using datepicker

Merge pull request #15631 from eileenmcnaughton/fix_it

[REF] Further deconstruction of updateFinancialAccounts

Pass dbFieldName to dbFieldName parameter

Improve metadata support for table civicrm_mailing_job in search

This makes the civicrm_mailing_job_start_date field available as
metadata and cleans up the code specifying how
civicrm_mailing_job gets joined in.

It turns out the code design has a pretty sensible methodology for ensuring that required
tables are added but unaware of this the tables
got configured all over the place in the where clause
functions.

dev/core#561 Convert mailing date search field to using datepicker

Some more cleanup and also open tab if specified in the URL

Merge pull request #15629 from eileenmcnaughton/mailing_form

Change parent class on mailing_form

Merge pull request #15627 from eileenmcnaughton/nfc

[NFC] reformat BAO_Case file

Further deconstruction of updateFinancialAccounts

updateFinancialAccounts is only called from one place with the context of 'changedStatus'. When called this
way the function updateFinancialAccountsOnContributionStatusChange is called & that 'tells'
the calling function whether to call the rest of this function.

Since this handling is specific to one place from which the function is called we can move that logic
back to the place that calls it & further simplify the updateFinancialAccounts function
which has been a bit of a haven of chaos

Merge pull request #15601 from eileenmcnaughton/items

[REF] simplify definition of isARefund

Merge pull request #15628 from eileenmcnaughton/mailing

[NFC] Add date information to MailingJob Schema

Add call to getSearchFieldMetadata

Change parent class on mailing_form

[REF] simplify definition of isARefund

Add date information to MailingJob Schema

Merge pull request #15625 from seamuslee001/e2e_cache_debugging

[NFC] Add in debugging to try and resolve E2E Cache intermitant test …

[NFC] reformat BAO_Case file

Exceptions

Update ArrayDecoratorTest.php

Merge pull request #15613 from eileenmcnaughton/lines_extract

[REF] extract chunk of code that creates the financial items for a given line.

[NFC] Add in debugging to try and resolve E2E Cache intermitant test failures

Merge pull request #15620 from eileenmcnaughton/order

Fix test setup Function to use order api

Merge pull request #15623 from seamuslee001/type_check_exceptions

dev/core#560 [REF] Throw exceptions for invalid type errors when aborting

Merge pull request #15621 from eileenmcnaughton/float

Cast result of getContributionBalance to float to match comment block.

Fix test setup Function to use order api

After ongoing issues resolving https://github.com/civicrm/civicrm-core/pull/14763 I have concluded the underlying
issue on the failing tests is the test setup. Specifically the use of 'partial_amount_to_pay' does not
create the correct underlying entities - it 'sort of' creates a payment without linking it to
the financial items.

These parameters are part of our first attempt at partial payments. I am removing them from the test here but
will deprecate later from other places in the code

[REF] Throw execptions for invalid type erorrs when aborting

Merge pull request #15622 from eileenmcnaughton/deprecate_bad

Change test to use preferred methods, fix revealed money bug

Change test to use preferred methods, fix revealed money bug

In switching this test over I found that Payment.create does not handle cleaning money - which it should to bring it
into line with Contribution.create api

Cast result of getContributionBalance to float to match comment block.

Without this it is possible for it to return 'null' which is not useful / in keeping. I did a bit of an audit &
did not find places where this would cause an === comparison to fail

Merge pull request #15619 from seamuslee001/fix_force_case_form_date

Ensure that using case_start_date_high and case_start_date_low etc in…

Merge pull request #15513 from magnolia61/Display_name_in_some_workflow_templates

dev/core#1320 Include displayname in subjects to prevent grouping in email clients

Ensure that using case_start_date_high and case_start_date_low etc in url variables works in force=1 mode

Merge pull request #15618 from eileenmcnaughton/case_from_to

Case datepicker field conversion - Follow up cleanup

Include Displayname in some workflow templates

Merge pull request #15614 from seamuslee001/convert_case_date_jcalendar

dev/core#561 Convert Case date fields on search forms from jcalendar …

Merge pull request #15616 from eileenmcnaughton/qill

Minimal fix for new enotice on case.subject

dev/core#561 Convert Case date fields on search forms from jcalendar to datepicker

Swap to using buildDateRangeQuery function

Fix notice error and generically handle cases where we have specified to use the LIKE operator for strings

Merge pull request #15580 from jaapjansma/dev_1205

Issue 1205: allow custom fields of type file for entities

Merge pull request #15615 from colemanw/explorerFix

Api4 explorer: Fix variable leaking to global scope

Minimal fix for new enotice on case.subject

The new enotice is caused by treating it as a LIKE string rather than a strict match (which is generally
a good thing)

Follow up cleanup

Api4 explorer: Fix variable leaking to global scope

Merge pull request #15370 from JMAConsulting/core-692-2

Core/dev#692 : Support search arguments to Case Search form

Merge pull request #15611 from seamuslee001/ref_deleteentry_ussage

[REF] Remove usage of CRM_ACL_BAO_Cache::deleteEntry in favour of usi…

Merge pull request #15608 from seamuslee001/more_fatal_exception

dev/core#560 Convert some more CRM_Core_Error::fatal to exception

Merge pull request #15610 from seamuslee001/remove_crm_exception

[REF] Remove CRM_Exception in favour of CRM_Core_Exception

[REF] extract chunk of code that creates tthe financial items for a given line.

Private function since more refactoring steps can follow

[REF] Remove usage of CRM_ACL_BAO_Cache::deleteEntry in favour of using functions that just delete from the ACL caches

Merge pull request #15578 from mattwire/redundant_format

Remove a redundant call to formatParamsForPaymentProcessor in AdditionalPayment form

dev/core#560 Convert some more CRM_Core_Error::fatal to exception

Switch from statusBounce to Exception in CRM_Mailing_BAO_Mailing::checkPermission as not on form layer

Merge pull request #15599 from MiyaNoctem/dev-core-1093-fix-issue-when-updating-custom-fields-with.logging

dev/core#1093: Fix Issue When Updating Custom Fields With Logging Enabled

[REF] Remove CRM_Exception in favor of CRM_Core_Exception

Merge pull request #15607 from eileenmcnaughton/matt_tidy

[NFC] comments tidy up …

[NFC] comments tidy up per https://github.com/civicrm/civicrm-core/pull/15479

Merge pull request #15604 from mattwire/multiplelocks

Remove CIVICRM_SUPPORT_MULTIPLE_LOCKS and make it always enabled if available

dev/core#1093: Add Custom Fields to Logging Tables

BulkSave method was adding fields to custom group table, but not to custom
group logging table. If these tables were updated on installation, before all
the schema is synced bewtween main and logging tables, fatal errors would be
thrown, as triggers tried to insert info into log tables missing required
fields.

Fixed by building log table after all fields are added to main table, and
before rebuilding triggers.

Merge pull request #15594 from eileenmcnaughton/dedupe2

[REF] minor refactor towards removing complexity.

Merge pull request #15590 from alifrumin/subjecthelp

Add help text to payment processor subject field

Remove CIVICRM_SUPPORT_MULTIPLE_LOCKS and make it always enabled if available

Merge pull request #15603 from seamuslee001/nfc_acl_doc_block_update

[NFC] dev/core#1336 Update doc blocks for various ACL related functions

Merge pull request #15602 from eileenmcnaughton/item_2

[REF] minor extraction - Extract code to update line items to paid

[NFC] dev/core#1336 Update doc blocks for various ACL related functions

Merge pull request #15509 from eileenmcnaughton/setter

dev/financial#82 Add getters & setters for contributionID on CRM_Core_Payment.

Extract code to update line items to paid

Add a bunch of additional getters & setters

Merge pull request #15576 from mattwire/settingtraitfilter

Generic Settings Pages: Make getSettingPageFilter() public so we can use it in hooks

Merge pull request #15503 from magnolia61/Generalize_Thank_You_in_Message_Templates

dev/core#1316 Update 'Thank You' & other corrections

Merge pull request #15600 from civicrm/5.19

5.19

Add getters & setters for contributionID on CRM_Core_Payment.

This will enable PaymentProcessor.pay to do ->setContributionID(['contribution_id'])
and set a pattern for definining other standardised parameters.

Merge pull request #15597 from colemanw/apiPath

Update api3 explorer url path for consistency

Make getSettingPageFilter() public so we can conditionally load resources in hooks

Merge pull request #15598 from alifrumin/rn5.19

5.19 Release Notes - first pass of sorting, annotating and editing

5.19 Release Notes - first pass

Merge pull request #1 from civicrm/master

Updating Fork with Commits on Base

Update api3 explorer url path for consistency

Remove for-each as silly

Remove IF as always true

[REF] minor refactor towards removing complexity.

This change actually makes the foreach & if following it redundant. I left them out for legibility
but basically all the wrangling is on the 'other' contact. We do just one thing with the 'main'
contact so the whole foreach & if is silly. Will follow up with removal of those

Merge pull request #15546 from eileenmcnaughton/group_type

dev/core#1321 [test] add unit test to demonstrate behaviour

Merge pull request #15564 from eileenmcnaughton/deprectate

dev/financial#79 minimal deprecation of Contribution.transact.

Merge pull request #15591 from eileenmcnaughton/noisy_deprecate_transact

Add noisy deprecation warning to Contribution.transact

dev/financial#79 minimal deprecation of Contribution.transact.

It seems urgent that we so some deprecation of this api - this is the absolute minimum.

More would involve creating some noise when people use it but there
is a separate issue for that. This is just to geet it out of being 'promoted'
by the api explorer

Merge pull request #15589 from mfb/obsolete-method

Remove obsolete supportStorageOfAccents() method.

Add noisy deprecation warning to Contribution.transact

Note that the Order api has been an adequate replacement for over a year - per efforts by wordpress integration

add help text to payment processor subject field

Remove obsolete method.

Merge pull request #15504 from mattwire/transact

Extract deprecated Contribution.transact API into it's own file

Adds missing join on the custom table

Merge pull request #15587 from eileenmcnaughton/dedupe

[REF] minor extraction in dedupe code

updated test

Merge pull request #15581 from mountev/street-address-sorting-contact-detail-report

Support street address sorting for contact detail report

[REF] extract part of the function to build the location blocks

Who hates the variable 'moniker'?

Merge pull request #15584 from civicrm/5.19

5.19

Merge pull request #15583 from seamuslee001/5.19

5.18.4 Release notes

Merge pull request #15579 from mattwire/additionalpayment_fixstatusbounce

Fix parameters for statusBounce in AdditionalPayment

Update 5.18.4.md

5.18.4.md

There was some ambiguity between original problem-report and PR-description.
Update per MM.

release-notes/5.18.4.md