Merge pull request #15628 from eileenmcnaughton/mailing

[NFC] Add date information to MailingJob Schema

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 …

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

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

Merge pull request #15548 from eileenmcnaughton/order

Support chaining Payment.create from Order api

street address sorting for contact detail report

Merge pull request #15577 from mattwire/translate_additionalpayment

Translate two string in AdditionalPayment form

Translate two string in AdditionalPayment form

Issue 1205: allow custom fields of type file for entities

Fix parameters for statusBounce in AdditionalPayment

Remove a redundant call to formatParamsForPaymentProcessor in AdditionalPayment form

Merge pull request #15520 from compucorp/fix-date-format-values

Make sure labels match the actual date input format

Merge pull request #15572 from civicrm/5.19

5.19 to master

Merge pull request #15569 from eileenmcnaughton/pay_notice

Enotice fix (unreleased enotice)

Merge pull request #15537 from eileenmcnaughton/pay_check

Fix a regression whereby payment details are not saved from the AdditionalPayment form

Enotice fix

per https://github.com/civicrm/civicrm-core/pull/15545#issuecomment-544531438

Support payment related fields on Payment.Create

On some digging I found that various payment related fields like pan_truncation & card_type_id were
being quietly ignored rather than saved by Payment.create

The additional payment form now uses this api so it is a regression on that form.

This fixes the metadata, tests & support for payment-related trxn fields

Merge pull request #15567 from yashodha/report_cleanup

CRM-21677 - report clean up for case demographic report

Merge pull request #15568 from eileenmcnaughton/test_pp

Combine IF clause for readability.

Merge pull request #15565 from eileenmcnaughton/payment_processor_label

Fix  bug whereby cidZero does not prepopulate billing details for selected contact for pay later.

Update 'Thank You' & other corrections in message templates

Merge pull request #14894 from eileenmcnaughton/date_fisc

Add freeform relative date for 'This Fiscal Year'