Merge pull request #13758 from eileenmcnaughton/no_load

Do not load membership contribution as updateMembershipBasedOnCompletionOfContribution loads it within the function

Merge pull request #13759 from eileenmcnaughton/matt

[REF] towards cleanup of update membership code

Merge pull request #13735 from eileenmcnaughton/query_extract

Extract getSearchSQLParts function

Merge pull request #13746 from eileenmcnaughton/activity_search

Convert activity_date_time field to datepicker and add support for url input

Add enotice handling for weird s....'

Fix related search classes to have the same signature

Add form rule for chronological ordeR

Fix group conversion to add relative key

Old format doesn't seem to have it

Add upgrade routine

Convert activity_date_time field to datepicker and add support for url input

This converts the form to being a metadata defined field & a date-picker field as well
as adding standardised url support for input.

The input format is
activity_date_time_high=20100101

I stripped out the complex and probably broken based on feedback  & not used in core
existing hard-coded url support.

Note that there are some outstanding items
1) the upgrade script item needs to be added - this includes dealing with renaming of the field
since I made it more consistent
2) I stripped out the check to ensure the high date is greater than the low date
- if we still want this we should re-add as a generic thing. My guess is it's a bit optional

Call getSearchSQLParts to get the desired part instead of sql wrangling

Extract getSearchSQLParts function

Merge pull request #13720 from eileenmcnaughton/extract_2

Speed up contribution results by removing join on civicrm_financial_type table when rendering search results.

Merge pull request #13765 from seamuslee001/test_empty_on_hold

Try and add data set example where email_on_hold / on_hold is NULL in…

[REF] towards cleanup of update membership code

This PR limits itself to siphoning off how the memberships are fetched & having that function
- return as an array of arrays rather than an array of objects
- index the array by id rather than membership type

This is a step towards the changes in https://github.com/civicrm/civicrm-core/pull/12542

& it should allow a separation of concerns relating to the LOADING of memberships from the
processing.

I note that the PR I'm looking at changes what gets loaded & that should be it's own PR
but also that there is an extraneous if() here - I didn't alter that because
it is a big white space change & doing on it's own will make that easy to review

Eliminate unnecessary join to civicrm_financial_type table to improve performance

Although this join is indexed it affects index choice, often causing the right index not to
be used and slowing performance. I got subtantial speed improvement by eliminating this join

Add hack for qill

This is a very minor extension of an existing hack but we should de-hack at some point.

Before this the qill is broken on payment instrument & contribution page too

Use generic pseudoconstant declaration methodology to add financial_type to pseudoconstants

Getting away from the hard-hack allows us to have more consistent metadata for generic fields

Merge pull request #13751 from seamuslee001/upgrade_karma

Upgrade Karma version to latest version

Try and add data set eaxmple where email_on_hold / on_hold is NULL in the formValues

Merge pull request #13664 from eileenmcnaughton/activity_perms

Rationalise Activity api ACLs for consistency, to respect the hook & improve performance

Merge pull request #13761 from yashodha/reports

report clean up - remove redundant code

Merge pull request #13726 from avall-llovera/patch-1

dev/core#767 Add 'Cancelled / Refunded Date' and 'Cancellation / Refund Reason'

report clean up - remove redundant code

Merge pull request #13760 from civicrm/5.11

5.11

Merge pull request #13653 from MegaphoneJon/reporting-8

reporting#8 - add thank-you dates to Contribution Summary/Detail reports

Merge pull request #13757 from eileenmcnaughton/payment_extract

[REF] extract getToFinancialAccount from CRM_Contribute_PseudoConstantt::contributionStatus

Merge pull request #13755 from eileenmcnaughton/event_fix

dev/core#766 fix loss of custom fields when creating an event from template

[REF] extract getToFinancialAccount from CRM_Contribute_PseudoConstant::contributionStatus

This makes the function much more readable. I felt that the is_payment logic should go but
resisted & just commented

Do not load membership contribution as updateMembershipBasedOnCompletionOfContribution loads it within the function

Merge pull request #13695 from eileenmcnaughton/extract_activity

[ref] Extract activity payment creation

dev/core#766 fix loss of custom fields when creating an event from template

Merge pull request #13753 from seamuslee001/event_form_class_fix

Fix Custom post outer div class on event registration form

Fix Custom post outer div class on event registration form

Merge pull request #13741 from jitendrapurohit/core-770

dev/core#770 - View Case Activity page displays disabled custom fields

Merge pull request #13690 from eileenmcnaughton/cleanup_payment_create

Clean up Payment.create function

Merge pull request #13400 from mfb/format-resource-url

fix broken logic in CRM_Utils_System_DrupalBase::formatResourceUrl()

Upgrade Karma version to latest version

Merge pull request #13750 from seamuslee001/master

5.11

Merge in 5.11

Merge pull request #13747 from eileenmcnaughton/merge_master

merge 5.11 to master

CRM/Contact - Fix fatal error on tag search

This fixes a fatal error caused by the CommaSeparatedIntegers not
accepting empty strings. This can be triggered by performing a
search with "Tag(s) IS NULL" using the search builder.

Add upgrade routine to convert on_hold to an array for sites with
civimail_multiple_bulk_emails set.

with that set we get a select box which needs an array rather than a checkbox.

Note from my testing there is problem loading the defaults due to the field name being
wrong - this aligns groups created before & after the 5.9 upgrade but does not resolve that.

    // preferred communication method
    if (Civi::settings()->get('civimail_multiple_bulk_emails')) {
      ->addSelect('email_on_hold',
        array('entity' => 'email', 'multiple' => 'multiple', 'label' => ts('Email On Hold'), 'options' => CRM_Core_PseudoConstant::emailOnHoldOptions()));
    }
    else {
      ->add('advcheckbox', 'email_on_hold', ts('Email On Hold')

dev/membership#10 fix incorrect validation error saying Start date must be the same or later than Member since

This problem is because both dates exclude time but the default setting includes it - leading to a
comparison of a datetime with a date+midnight time

dev/core#770 - View Case Activity page displays disabled custom fields

Merge pull request #13731 from eileenmcnaughton/5.11

dev/core#731 Add upgrade routine to convert on_hold to an array for sites with

Merge pull request #13745 from mattwire/viewcancelledrecur

Allow viewing of cancelled recurring contributions

Merge pull request #13743 from greenpeace-cee/fix-tag-search

CRM/Contact - Fix fatal error on tag search

Allow viewing of cancelled recurring contributions

Merge pull request #13744 from mattwire/upgrader_fix

Upgrader: Don't abort if state_province already exists

Don't abort if state_province already exists when upgrading

CRM/Contact - Fix fatal error on tag search

This fixes a fatal error caused by the CommaSeparatedIntegers not
accepting empty strings. This can be triggered by performing a
search with "Tag(s) IS NULL" using the search builder.

Merge pull request #13566 from pradpnayak/ModifiedBy

dev/core#708, Fix Qill for Added by and Modified By

Minor Payment.create tidy up.

Rationalise methodology to decide if payment creates the contribution

Merge pull request #13739 from seamuslee001/participant_payment_hook_correct

Fix the invocation of post hook for ParticipantPayment ensuring that …

Merge pull request #13734 from eileenmcnaughton/mem

dev/membership#10 fix incorrect validation error saying Start date must be the same or later than Member since

Merge pull request #13718 from eileenmcnaughton/payment_ref

Decommision getPartialPaymentTrxn function

Merge pull request #13740 from civicrm/5.11

5.11

Merge pull request #13738 from jitendrapurohit/core-768-rc

dev/core#768 - Fix fatal error on group search

Merge pull request #13737 from seamuslee001/pdf_receipts_fix_enotice

Fix up e-notice error when using a user from email address same issue…

Fix the invocation of post hook for ParticipantPayment ensuring that an id is passed in post hook

dev/core#768 - Fix fatal error on group search

Merge pull request #13724 from agileware/CIVIEWAY-42

Status of test contribution is not fetched on ThankYou page.

Fix up e-notice error when using a user from email address same issue as dev/core#644

Add upgrade routine to convert on_hold to an array for sites with
civimail_multiple_bulk_emails set.

with that set we get a select box which needs an array rather than a checkbox.

Note from my testing there is problem loading the defaults due to the field name being
wrong - this aligns groups created before & after the 5.9 upgrade but does not resolve that.

    // preferred communication method
    if (Civi::settings()->get('civimail_multiple_bulk_emails')) {
      ->addSelect('email_on_hold',
        array('entity' => 'email', 'multiple' => 'multiple', 'label' => ts('Email On Hold'), 'options' => CRM_Core_PseudoConstant::emailOnHoldOptions()));
    }
    else {
      ->add('advcheckbox', 'email_on_hold', ts('Email On Hold')

Merge pull request #13736 from agileware/CIVICRM-1149

CiviCRM Membership Detail report, add column to display if membership is Primary or Inherited

Merge pull request #13728 from JMAConsulting/dev/core#769

dev/core#769 - Fix for ZipArchive->open() PHP bug

Merge pull request #13732 from mattwire/refactor_smartgroupquery_upper1

dev/core#748 Move UPPER() from sql to php domain

dev/membership#10 fix incorrect validation error saying Start date must be the same or later than Member since

This problem is because both dates exclude time but the default setting includes it - leading to a
comparison of a datetime with a date+midnight time

Merge pull request #13730 from civicrm/5.11

5.11

Move UPPER() from sql to php domain

Merge pull request #13717 from eileenmcnaughton/extract

[REF] Extract lines to add the pseudoconstant to the select

Merge pull request #13723 from eileenmcnaughton/5.11

financial#46 - clean money format at form layer - Fix mistreatment of event fees > 1000 when saving

Merge pull request #13729 from colemanw/recentitems

Mark de.systopia.recentitems obsolete

Merge pull request #13725 from JMAConsulting/CRM-21754-rc

Don't break loop if address fields not found while formatting rows in Reports

Mark de.systopia.recentitems obsolete

dev/core#769 - Fix for ZipArchive->open() PHP bug

dev/core#767 Add 'Cancelled / Refunded Date' and 'Cancellation / Refund Reason'

Add the fields and filters 'Cancelled / Refunded Date' and 'Cancellation / Refund Reason' in the Detail Contributions Report

Don't break loop if address fields not found while formatting rows in Reports

Fix mistreatment of event fees > 1000 when saving

Merge pull request #13721 from MegaphoneJon/5.11

financial#46 - clean money format at form layer

Merge pull request #13722 from civicrm/5.11

5.11 to master

Merge pull request #13711 from greenpeace-cee/fix-multi-currency-soft-credit

dev/core#756 - CRM/Contribute - Fix multi-currency soft credit summary

Merge pull request #13704 from colemanw/age_asof_date

dev/core#561 Upgrade age_asof_date to datepicker in search

financial#46 - clean money format at form layer

Decommision getPartialPaymentTrxn function

This function is only called from one place in the code & also from a test.

It is misleading as it implies a 'get' when it actually does an update
and it groups functionality in a way that doesn't make sense from the calling code pov

In this commit the function is removed and the lines are directly copied to the calling
function, as preparation for cleaning up the calling function

Spelling fix

Extract lines to add the pseudoconstant to the select

Merge pull request #13686 from seamuslee001/phpword_upgrade

Upgrade PHPWord

dev/core#756 - CRM/Contribute - Fix multi-currency soft credit summary

This fixes an exception caused by multiple currencies being passed to
CRM_Utils_Money::format. It also resolves some inconsistencies in how
the summary table is rendered for soft credits.

Upgrade age_asof_date to datepicker in search

CIVIEWAY-42: Unit test to verify the correct outcome message on ThankYou Page.

Merge pull request #13715 from eileenmcnaughton/order_fix

[minor cleanup] reduce params passed to searchQuery

Merge pull request #13701 from colemanw/changeCaseTypeDate

Migrate date field to datepicker on ChangeCaseType form

[REF] extract activity payment.

THis is likely not it's last resting place but it does make the functions involved a lot simpler now

Merge pull request #13692 from eileenmcnaughton/extract_record

[REF] Extract recordPayment portion

[minor cleanup] reduce params passed to searchQuery

Where we are retrieving the sql we can call the inner function rather than passing 'true'
in other cases we are passing default values through

Merge pull request #13710 from civicrm/5.11

Merge 5.11 to master

Merge pull request #13696 from mattwire/entityform_13578_fix

Move assign of currency for entityForm outside of foreach so order of fields don't matter

Merge pull request #13705 from eileenmcnaughton/atest

[TEST FIX] Increase uniqueness in testSingleNowDates