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

Merge pull request #13745 from mattwire/viewcancelledrecur

Allow viewing of cancelled recurring contributions

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

Merge pull request #13566 from pradpnayak/ModifiedBy

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

Merge pull request #13739 from seamuslee001/participant_payment_hook_correct

Fix the invocation of post hook for ParticipantPayment ensuring that …

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

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

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

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

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

Merge pull request #13638 from eileenmcnaughton/squash_products

dev/core#735 Do not include product in search results if site has none

Merge pull request #13645 from eileenmcnaughton/ft_acl_test

Code cleanup - remove extraneous permissions clause

Merge pull request #13709 from JMAConsulting/dev-core#757-rc

Viewing Contact Note comments is broken in 5.10.4.

Merge pull request #13680 from eileenmcnaughton/order_test

Fix & test searchQuery order by to be less dependent on what is selected for search

Fix bug in test set up

Extract recordPayment portion

Viewing Contact Note comments is broken in 5.10.4.
Update Javascript to use Note-NNNNN rather than cnote_NNNN as the ID of Note rows.
Completes change started in commit 57abcbbf5cf85b1266377e0ad9af4fb5865d5bc0.

Increase uniqueness in testSingleNowDates

Reduce instances of
AuthorizeNet failed for unknown reason.E00012: You have submitted a duplicate of Subscription 5642898. A duplicate subscription will not be created

Merge pull request #13702 from mattwire/contributionrecur_payprocid_pseudoconstant_schema

Add pseudoconstant for payment_processor_id to contributionrecur

Add pseudoconstant for payment_processor_id to contributionrecur

Merge pull request #13666 from eileenmcnaughton/case_fix

dev/core#739 Fix case detail report breaking when sorted by case type.

Merge pull request #13658 from mfb/temp-force-utf8

Phase out CIVICRM_TEMP_FORCE_UTF8

Merge pull request #13643 from eileenmcnaughton/recur

[REF] minor refactor around retrieving processor id for recur

Merge pull request #13694 from eileenmcnaughton/extract_refund

Extract record refund function

Migrate date field to datepicker on ChangeCaseType form

Extract record refund function

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

Merge pull request #13582 from colemanw/kam

Migrate KAM smartmenus to core

Upgrade PHPWord

Patch phpword install to fix global handling of disable libxml entity loader

Fix phpoffice/common as well

Fix & test searchQuery order by to be less dependent on what is selected for search

The pseudoconstant array is built when doing a select - if a field is in
order by but not select it will fail - as in the test.

This is a bit hard to hit outside the test for contribution but makes it more robust`
allowing us to address other bugs & performance issues.

Merge pull request #13693 from civicrm/revert-13691-record_extract

Revert "[REF] Extract record refund function"

Revert "[REF] Extract record refund function"

Merge pull request #13691 from eileenmcnaughton/record_extract

[REF] Extract record refund function

Extract record refund function

Merge pull request #13289 from mfb/pear-mail

Move pear/mail from packages to composer.json

Merge pull request #13340 from mfb/street-number-max

Do not attempt to store out-of-range street number

dev/core#735 Do not include product in search results if site has none

Merge pull request #13668 from eileenmcnaughton/extract_order

[REF] Extract getSearchSQL from getSearchQuery.

Add test case for parsing out-of-range street number.

Merge pull request #13688 from mfb/export-temp-table

Use CRM_Utils_SQL_TempTable to drop and create table.

Do not attempt to store out-of-range street number.

Merge pull request #13276 from mfb/trash-change-log

Record change log entry when contact is moved to or restored from trash

Merge pull request #13346 from mfb/geocode-job-db-error

geocode job: Do not return more messages than can fit in the log data column

Deprecate CIVICRM_TEMP_FORCE_UTF8.

Merge pull request #13578 from mattwire/entityform_view_towards

Towards supporting EntityForm for 'View Action'

Merge pull request #13588 from jitendrapurohit/dev-631

dev/core#631 - Enable 'add new' by default on merge screen

Merge pull request #13657 from MegaphoneJon/deprecateBasicContactFields

reporting#9: parity between getContactFields and getBasicContactFields

Merge pull request #13654 from seamuslee001/lab_core_742

dev/core#742 Fix XML parasing by swapping & for ,

Merge pull request #13042 from jackrabbithanna/dev-core-475

only set custom field to null if it is really null, not string 'null'

Use CRM_Utils_SQL_TempTable to drop and create table.

Merge pull request #12641 from mfb/reply-forwarding

CiviMail: Fix reply forwarding for mailers with From: and Return-path: limitations

Merge pull request #12337 from lcdservices/dev-core-190

dev/core#190 / CRM-21643 ensure custom data multi record profile returns correct …

Merge pull request #13441 from greenpeace-cee/fix-schema-parsing

CRM/Logging - Fix various bugs in schema parsing

Merge pull request #13682 from mfb/utf8mb4-check-exception

Force utf8mb4 query to throw exception as the check expects

Merge pull request #13687 from eileenmcnaughton/activity_clean

Minor code cleanup

Merge pull request #13634 from eileenmcnaughton/fin_type_speed

[NFC, test class] formatting, remove unused variables

Minor code cleanup

This is a reviewer's commit of https://github.com/civicrm/civicrm-core/pull/13672

I pulled out some lines I've checked & agree with to simplify that commit

Merge pull request #13644 from mfb/temporary-tables

Refactor CRM_Utils_SQL_TempTable::build()->createWithQuery($sql) interface to support MEMORY tabls

CRM/Logging - Improve enum handling in schema diff

Instead of storing permitted enum values in the LENGTH array key
when extracting column information, this adds a separate ENUM_VALUES
key. When schema differences are calculated for enum columns, this
value triggers a change when new permitted values are added.

CRM/Logging - Fix various bugs in schema parsing

This fixes a couple of bugs in the schema parsing methods used by
Civi's extended logging feature:

- CRM_Logging_Schema::getIndexesForTable only queried for constraints,
  not returning any indexes.
- CRM_Logging_Schema::getIndexesForTable returned an array in the form
  [0 => ['constraint_name' => 'foo']] rather than the expected array
  of index names (i.e. ['foo']).
- CRM_Logging_Schema::columnSpecsOf contained an off-by-one error and
  a wrongly used substr parameter causing column lengths to include
  surrounding parenthesis. This would result in a "varchar(42)"
  column returning a length of "(42)" instead of "42".

Merge pull request #13685 from seamuslee001/tests_746

dev/core#746 Add in unit tests to ensure that where clause is as is w…