Merge pull request #15632 from eileenmcnaughton/pay_form

Partial conversion of Payment form to use the EntityFormTrait & add title

Merge pull request #15641 from eileenmcnaughton/cont_clean

[REF] Move calls to CRM_Core_BAO_FinancialTrxn::createDeferredTrxn back to the calling functions.

Merge pull request #15651 from seamuslee001/dev_core_1344

dev/core#1344 Fix logic for displaying BillingName and Credit Card De…

Merge pull request #15588 from seamuslee001/ref_group_contact_cache_rebuild

[REF] Refactor Smart Group Cache population code to be less intensive

Merge pull request #15592 from seamuslee001/ref_acl_contact_cache_generation

[REF] Refactor ACL Contact Cache generation to be more efficient

Merge pull request #15679 from eileenmcnaughton/script_fee

dev/event#19 remove obsolete scriptFee & scriptArray

Merge pull request #15688 from eileenmcnaughton/billing_name

Fix BillingName to be assigned in email receipts IF available

Fix BillingName to be assigned in email receipts IF available

Previously it was assigned at the form layer, now we assign it in the email sending routine, depending on
whether the contribution has address_id & there is a value assigned

Merge pull request #15691 from civicrm/5.19

5.19

Merge pull request #15690 from seamuslee001/nfc_authorize_test_fix

[NFC] Fix AuthorizeNet Test failure due to credit card being expired

[NFC] Fix AuthorizeNet Test failure due to credit card being expired

dev/core#1344 Fix logic for displaying BillingName and Credit Card Details in email Receipts

Merge pull request #15682 from seamuslee001/dev_core_1344_aditional_if

dev/core#1344 Re-work if statements and remove excess ifs

dev/core#1344 Re-work if statements and remove excess ifs

Merge pull request #15638 from eileenmcnaughton/vangelis

dev/core#1282 unit test for bug

Merge pull request #15683 from eileenmcnaughton/test

[NFC] test cleanup

Merge pull request #15681 from eileenmcnaughton/nfc

[NFC] define variable type

[NFC] test cleanup

I failed to replicate what I was trying to replicate but   adding the test cleanup for now

Merge pull request #15677 from seamuslee001/event_smart_group_upgrade

Add in upgrade step for upgrading Event date and Relationship Active Date smart groups

Merge pull request #15680 from seamuslee001/dev_core_1344_straightforward

dev/core#1344 Fix complicated logic on credit card and billing name s…

Add in upgrade step for upgrading Event date smart groups

Include relation_active_period_date field for upgrade

Merge pull request #15671 from eileenmcnaughton/rel_action

Jcalendar conversion on relation_active_period_date

[NFC] define variable type

dev/core#1344 Fix complicated logic on credit card and billing name sections as agreed in #15651

dev/event#19 remove obsolete scriptFee & scriptArray

https://lab.civicrm.org/dev/event/issues/19#note_25984

Both @mattwire & myself determined these are obsolete, unused parameters, removing

Merge pull request #15674 from eileenmcnaughton/tpls

Simplify billingName logic on xml/templates/message_templates/members…

Remove test on legacy relative dates handling

Jcalendar converstion on relation_active_period_date

Same as https://github.com/civicrm/civicrm-core/pull/15661

Conversion excluded for now

Merge pull request #15362 from yashodha/edit

(dev/core#1065) Member in Edit mode needs to be shown consistently

Merge pull request #15661 from eileenmcnaughton/event

Jcalendar replacement for event date

Merge pull request #15673 from eileenmcnaughton/set

Ensure contributionRecurID is set on processor

Merge pull request #15672 from eileenmcnaughton/unreachable

[NFC] Remove unreachable lines

Ensure contributionRecurID is available to payment processors that need it in cancelSubscription

Discussion starts here https://chat.civicrm.org/civicrm/pl/9m1goccmtpdy58cb7jyx791t7e

Simplify billingName logic on xml/templates/message_templates/membership_online_receipt_html.tpl

Remove unreachable lines

Merge pull request #15669 from eileenmcnaughton/yesno

Use Yes-No radio instead of checkbox on search form.

Jcalendar replacement for event date

Merge pull request #15662 from eileenmcnaughton/lines

[NFC] Test cleanup -  switch to OrderApi in test setup, add throws, use sin…

Use Yes-No radio instead of checkbox on search form.

When a field has an html type of 'Checkbox' it means that when editing it we want a checkbox for
'yes' or 'no'. However in the search forms there is a 3rd option 'don't care'.

This change takes the context into account for those forms

Merge pull request #15650 from demeritcowboy/mode-value-missing

dev/core#1347 - Fix php warning in advanced search when opening some accordions

Merge pull request #15667 from JMAConsulting/generalise-qill-1

Add getQillValue fn to generalise qill string construct

Merge pull request #15664 from eileenmcnaughton/odd_payment2

dev/financial#94 fix eroneous payment creation when editing line items

Add CRM_Contact_BAO_Query::getQillValue fn

dev/financial#94 fix eroneous payment creation when editing line items

Per https://lab.civicrm.org/dev/financial/issues/94 an eroneous payment transaction is being created when an extra line is
added to an order. The code comments indicate that during the last refactoring the chunk of code
doing this was identified as highly suspect and on re-review of it I'm sure it should go
as we are altering the items purchases, not recording payment in this routine.

I think it dates back to an earlier concept of the flow where altering what was purchased implictly
altered the payments towards that purchase

Test cleanup -  switch to OrderApi in test setup, add throws, use single quotes

[NFC] code cleanup on test class

Merge pull request #15660 from totten/master-no-persistent

Remove unused, hidden subsystem for "persistent DB tpl strings"

Upgrader - Drop table "civicrm_persistent" (unless it has data)

sql/civicrm_generated.mysql - Update

DB Tpl Strings - Remove table "civicrm_persistent"

DB Tpl Strings - Remove unused form 'civicrm/admin/tplstrings'

If you grep for this (in v3.2, v5.20, and universe), you will find that the
only usages are self-referential or pro-forma:

- There is / never been a navigation item pointing to the page.
- There is no codepath which *consumes* data produced here.

DB Tpl Strings - Remove unused Smarty function crmDBTpl

I've searched `universe` and haven't found any references to this.  This
appears to be the only code-path which consumes information from
`civicrm_persistent`.

I also went back to spot-check a few SVN branches of CiviCRM (v3.2, v3.3).
It appears that this was added in v3.2 *but never used or documented*.

CRM_Upgrade_Incremental_Base - Add helper dropTableIfEmpty()

CRM_Core_DAO - Add helper checkTableHasData()

Merge pull request #15653 from eileenmcnaughton/billin_name

dev/core#1344 Simplify email template logic on displaying card inform…

Merge pull request #15659 from civicrm/5.19

5.19

Merge pull request #15657 from demeritcowboy/pledge-search-backport

dev/core#1348 - pledge search not working backport of 15656

Merge pull request #15658 from MikeyMJCO/patch-1

Delete mkdocs.yml

Merge pull request #15655 from demeritcowboy/activity-search-form-values

dev/core#1348 - backport of PR 15654

Merge pull request #15652 from eileenmcnaughton/schema

Add uniquenames for mailing_name, mailing_job_status

Delete mkdocs.yml

We may as well remove the old mkdocs.yml file as it's not used.

pledge search not working

activity search not working at all

dev/core#1344 Simplify email template logic on displaying card information

Same deal as https://github.com/civicrm/civicrm-core/pull/15646

Add uniquenames for mailing_name, mailing_job_status

This is in support of https://github.com/civicrm/civicrm-core/pull/15369

Where the names in the search do not match the 'real names' we have 2 techniques

1) where adding a unique name to the schema gives a match we do that - here that is
the case for mailing_name
2) otherwise we ensure the schema has a unique name if appropriate &
rename the field in the upgrade scriptt

In this case when we look at CRM_Mailing_BAO_Query we don't see mailing_name
as present (only mailing_id which should be an entity reference field). However it
is used in the confusingly-not-related Mailing search so lets rename this
one at the same time.

Merge pull request #15646 from eileenmcnaughton/add_pay

dev/core#1344 Simplify email template logic on displaying billing detail

Merge pull request #15626 from demeritcowboy/case-pdf-merge-order

dev/core#893 - fix misfiled case print/merge pdfs (alternate to PR 15609/15364)

Merge pull request #14763 from eileenmcnaughton/payment_3

dev/financial#34 Fix line allocation in Payment.create

fix warning when no searchPane mapping for mode

Merge pull request #15648 from seamuslee001/relationship_date_cleanup_fixes

Cleanup following smart group conversions and fix the old name of the…

Cleanup following smart group conversions and fix the old name of the relationship date relative fields for conversion and add a unit test

Merge pull request #15647 from demeritcowboy/regen-upper-lower

Allow regen.sh to work with either upper/lower case for CMS name to match elsewhere

Merge pull request #15637 from eileenmcnaughton/relationdates

Convert relationship start & end dates to datepicker

work with either upper/lower to match elsewhere

Convert relationship_start_date & relationship_end_date to datepicker

This seems to work OK - look forwards to seeing you kick the tyres on it & see if you find any issues

Merge pull request #15635 from eileenmcnaughton/contact_date

dev/core#561 Convert birth_date & deceased_date to datepicker

Convert birth_date & deceased_date to datepicker

Merge pull request #15636 from eileenmcnaughton/relationship_date

Add unique name for relationship start & end dates

dev/core#1344 Simplify email template logic on displaying billing detail

Per https://lab.civicrm.org/dev/core/issues/1344 we currently use complex & fragile logic whereas we can simply display
what we have gathered. Since email templates are already being upgraded this release it feels like a good
opportunity to clean these up too.

This just includes one template - but I'll rip through the rest if this gets merged & people agree

Merge pull request #15643 from omarabuhussein/fixing-optionvalues-xml-processing

Fixing a fatal error when installing extensions with Option Values defined in XML

Merge pull request #15630 from eileenmcnaughton/pay_fail

Fix cancel payment action to reverse financial items related to cancelled payment

Merge pull request #15644 from civicrm/5.19

5.19 to master

Merge pull request #15642 from MiyaNoctem/dev-core-1093-backport-fix-to-v5.19-rc

dev/core#1093: Bckport Fix to v.5.19 to Add Custom Fields to Logging Tables

Fixing a fatal error when installing extensions with Option Values
defined in XML

If an Extension has an option value defined in an XML file and the
option value belongs to an option group that is already defined in Civi
But in the XML (e.g activity_type option group) then installing the
extension will result in a fatal error with the following message :

"A fatal error was triggered: One of parameters (value: ) is not of the type Integer"

The error happen because CiviCRM fails to find the option group in the
XML file so it look for it in the database, but the code that look for
it pass the option group name as a SimpleXML object (since SimpleXML
does not return scalar values and rather objects of Type SimpleXML) and
because of that the code won't be able to find the value which will
result in executing a query that expect an integer but rather a NULL
pass to it which result in the fatal error.

I fixed it by casting the option group name inside the SimpleXML object
to string.

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.

Move calls to CRM_Core_BAO_FinancialTrxn::createDeferredTrxn back to the calling functions.

Although this seems like we are increasing duplication - by having more calls to one line of code I feel like
we are better off moving it back to the calling functions as there is 'preparatory wrangling' that also belongs
in the calling functions - notably in changeFinancialType.

I believe we should be able to get to the point where the updateFinancialAccounts funcgtion does not
need to receive context as a parameter because all the context specific stuff will be done in the calling functions
and it will have a clear objective - which I have finally realised is creating the financial transaction
and the financial items - however in some cases it is called twice to do a reversal followed by a line item
and I believe the calling functions could be cleaned up a lot more by 'returning' code to them as
passing  by reference leaves us very unclear as to what is happening to them at each point in the process.

Merge pull request #15639 from eileenmcnaughton/pay

dev/financial#77 ++ Make contribution_id mandatory for PaymentProcessor.pay, pass invoiceID

dev/financial#77 ++ Make contribution_id mandatory for PaymentProcessor.pay, pass incoieID

Replaces https://github.com/civicrm/civicrm-core/pull/15477 & also resolves https://lab.civicrm.org/dev/financial/issues/77

by requiring contribution_id for PaymentProcessor.pay as a way to ensure that it is only called after the order is created
per our preferred flow (people could still get past this but it feels like they would at least know they werre
hacking our process & take responsibility for any issues if it breaks or we suddenly start enforcing that
it is a valid contribution.

This also sets some of the recommended variables. Note that I had to use a few more lines to ensure we were always setting
contactID, contributionRecurID to an integer. I do think this stricter approach is better but it wound up
more verbose

Merge pull request #15375 from ixiam/dev_issue#1282

dev/core#1282 Takes care of customfields of type multiselect that were not being rendered

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

Add unique name for relationship start & end dates

Preliminary to fixing for datepicker

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

Partial conversion of Payment form to use the EntityFormTrait & add title

This partially converts the form to the new EntityForm format

Merge pull request #15629 from eileenmcnaughton/mailing_form

Change parent class on mailing_form