crmMailingAB - Only display "Headers and Footers" if there are headers and footers

crmMailing - Only display "Headers and Footers" if there are headers and footers

Merge pull request #15798 from seamuslee001/dev_core_183_imports_2

dev/core#183 Convert database testing to use CRM_Utils_SQL_TempTable …

dev/core#183 Convert database testing to use CRM_Utils_SQL_TempTable fuctionality and remove Incomplete Jobs handling as it is not called

Merge pull request #15790 from civicrm/5.20

5.20

Merge pull request #15787 from eileenmcnaughton/recur_ug

Remove unhelpful alert from contribution search

Merge pull request #15665 from MikeyMJCO/patch-1

Create a GitLab general issue template

Merge pull request #15789 from seamuslee001/5_19_1_version_notes

5.19.1 Version Release Notes

Merge pull request #15783 from eileenmcnaughton/sort_id

[REF] replace copy & paste with shared getSortID function.

Update 5.19.1.md

Add in release notes for 5.19.1 drop

Merge pull request #15786 from eileenmcnaughton/cont_type

Remove obsolete warning

Remove unhelpful alert from contribution search

[REF] replace copy & paste with shared getSortID function.

I've replaced this repetitive code chunk across the forms that inherit from CRM_Core_Form_Search

Merge pull request #15784 from civicrm/5.20

5.20

Remove obsolete warning

The time has come to let this one go.....

Merge pull request #15781 from eileenmcnaughton/cont_search

dev/core#1374 Fix search formValue handling on contribution search

Merge pull request #15782 from seamuslee001/dev_mailing_56_2

Add in unit test for dev/mailing#56 and dev/mailing#57 and also fix i…

dev/core#1374 Fix search formValue handling on contribution search

This fixes 2 issues
1) Search params being lost when editing a related entity per
https://lab.civicrm.org/dev/core/issues/1374
2) force=1&sort_name=p not working in contribution search url

In digging I concluded the problem is we have 3 underlying arrays which we keep jumbling together

1) formValues - the actual submitted values, augmented by any  url passed params
2) the default values - values to load by default on the form
3) our working query params - a copy of formValues that we have prepared for the query

We need to stop mangling them. I added subtle code comments

Merge pull request #15777 from eileenmcnaughton/format

[NFC] Reformat Contact_Form_Search_Advanced

Add in unit test for dev/mailing#56 and dev/mailing#57 and also fix issue where by mailing_name has been namespaced also in 5.20

Merge pull request #15779 from seamuslee001/dev_mailing_56

dev/mailing#56 dev/mailing#57 Ensure that we don't pass in any namesp…

Merge pull request #15773 from civicrm/5.20

5.20

[NFC] Reformat Contact_Form_Search_Advanced

Fix array style

Merge pull request #15775 from seamuslee001/convert_fatal_to_error_system_cache

Converts using deprecated fatal function for exception in CRM/Utils/Sys…

Merge pull request #15776 from eileenmcnaughton/url_params

[NFC] comment fixes - exceptions & don't declare NULL

dev/mailing#56 dev/mailing#57 Ensure that we don't pass in any namespaced fields when building the row array for mailing reports

Merge pull request #15774 from seamuslee001/eileen_ts_15770

Add in translation function around newly converted statusBounce messages

Converts using deprated fatal function for exception in CRM/Utils/System and in CRM/Utils/Cache and changes default for retrieve function in the Request class to throw exceptions

Update doc blocks

[NFC] comment fixes - exceptions & don't declare NULL

Add in translation function around newly converted statusBounce messages

Merge pull request #15770 from eileenmcnaughton/except

Swap some fatals on form & page classes for statusBounces

Merge pull request #15772 from agh1/5.20.0-releasenotes-initial

5.20.0 release notes initial run

5.20.0 release notes: added boilerplate

5.20.0 release notes: raw from script

Merge pull request #15757 from yashodha/participant_edit

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

Swap some fatals on form & page classes for statusBounces

Merge pull request #15769 from seamuslee001/mathieu_shared_address_improvement

Improve link title in shared address as per Mathieu

Improve link title in shared address as per Mathieu

Merge pull request #15666 from revati90/shared_address

modified shared address note under contact dashboard

Merge pull request #15768 from civicrm/5.20

5.20 to master

Merge pull request #15765 from seamuslee001/5.20

dev/core#1376 Catch Invalid Argument Exception when looking for APIv4…

Merge pull request #15764 from civicrm/5.20

5.20

Merge pull request #15761 from eileenmcnaughton/cache_stuffed

Fix cachekey string

dev/core#1376 Catch Invalid Argument Exception when looking for APIv4 Services

Merge pull request #15248 from MegaphoneJon/reporting-19

reporting#19 - Performance improvements on report display/export

Merge pull request #15762 from seamuslee001/dev_core_1371

dev/core#1371 Fix e-notice caused by trying to pass in known  to the …

dev/core#1371 Fix e-notice caused by trying to pass in known  to the extension downloader when checking requirements

Merge pull request #15758 from yashodha/contributors

add contributor name

Merge pull request #15595 from eileenmcnaughton/dedupe3

dev/core#1339 Dedupe screen check to carry across any data where the other contact has none by default.

Fix cachekey string

I hit a bug (in a unit test) where a user without permissions to access custom data would do a
check_permissions api call followed by a check_permissions = 0 call. In the second call
custom fields were not returned
as they were already excluded from the cached key.

Merge pull request #1 from totten/MikeyMJCO-patch-1

Proposed updates to Gitlab issue template

Issue template - Add "improvement.md"

This is basically the same as the bug-report template, except:

* "Reproduction steps" become "Example use-case"
* "Environment" is removed - because it's not usually a factor. (And if it is,
  it's generally explicit part of the requirements.)

Issue template - More consistent punctuation

* End comment-placeholder with period, like in other comment-placeholders
* Remove colons from titles - don't think we do that anywhere else

Issue template - Small tweak

Issue template - Rename "general-bug-or-issue" to "problem"

Because... fewer words

Issue template - Move error-msg block from "Reproduction steps" to "Current behavior"

Issue template - No screenshot for expected behavior

The "Expected Behavior" is specifically what the system does *not* do, so...
it might be hard to give a screenshot of that...

Issue template - For triple `, use multiline

Issue template - Update "Environment"; combine type+version fields

We want "versions" for everything. Making separate lines for the version just makes it bit
harder to write (you have to remove more things) and to read (you have to skim more lines).

Changes:

* Combine "CMS/CMS Version"
* Combine "Database/Database Version"
* For "Web Server", also include version.
* Add "PHP"

add contributor name

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

modified shared address note

Merge pull request #15755 from seamuslee001/copywrite_date_update

[NFC] Update copyright date for 2020

Merge pull request #15739 from eileenmcnaughton/dompdf

Update dompdf (0.8.0=>0.8.3) & dependencies

[NFC] Style Fix Prepare Event

(NFC) Style fixups

Update copyright date for 2020

Set version to 5.21.alpha1

Set version to 5.20.beta1

Merge pull request #15754 from seamuslee001/master

5.19

Merge in 5.19

Set version to 5.19.0

Merge pull request #15752 from agh1/5.19.0-releasenotes-final

5.19.0 release notes: added late changes

Update 5.19.0.md

Merge pull request #15751 from magnolia61/workflow_template_contribution_terms

changed wording to Total Fee & Total Paid

5.19.0 release notes: added late changes

Merge pull request #15748 from eileenmcnaughton/line_item_belt_braces

Provide precautionary handling for theoretical error scenario.

changed wording to Total Fee & Total Paid

Provide precautionary handling for theoretical error scenario.

While testing payments I hit a bug where I tried to add a payment to a contribution with no financial items.

I never managed to replicate it again or determine how the payment came to be in that state but
it's been playing in my mind that people could get fatal errors if the financial_items don't exist
and dealing with those as regression reports will very tough. So my plan is
- for 5.20 add this extra routine to create it if it does not exist
- use this mechanism + more digging to figure out how legit an issue it is https://github.com/civicrm/civicrm-core/pull/15706
- in future releases 'get noisy' about having to create them if they don't exist
- eventually remove this routine

Merge pull request #15740 from eileenmcnaughton/partial

dev/financial#40 Fix for non-allocation of payment to fully reversed checkboxes option

Merge pull request #15745 from seamuslee001/master

Update template upgrade list as per PR #15742

Merge pull request #15750 from eileenmcnaughton/except

[NFC] Cleanup on exceptions

Merge pull request #15744 from yashodha/participant_edit

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

Merge pull request #15749 from seamuslee001/Additional_Fixes_to_workflow_templates

Additional fixes to workflow templates

Update template upgrade list as per PR #15742

Cleanup on exceptions

Fixes a low level function to throw a CRM_Core_Exception & tidies up functions
above to specify that. Fixes casing on a function call, removes a fatal
for status bounce

Fix Smarty Syntax in payment_or_refund_notification_text template and update unit tests to match the change in langauge in the templates

Merge pull request #15746 from civicrm/5.19

5.19

Merge pull request #15741 from totten/5.19-volinst

(dev/core#1093) Fix regression during extension activation

Merge pull request #15743 from eileenmcnaughton/validate_test

Fix test to be more valid

Additional changes to workflow message templates

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

Fix test to be more valid

I've added a check to run after tests & before tearDown to ensure that all payments have the
right entity_financial_trxn records.

This required fixing one ttest to be valid. It's not possible to create a contribution using the
back office for with status set to Partially Paid (it's not an option in the drop down, so
the test should create as Pending & then add a payment to get to partially paid

(dev/core#1093) Fix permission error when activating extension

Overview
--------

This fixes a recent/unreleased regression from #15726 in which the D7 demo build fails.

Before
------

When running `civibuild reinstall ...` on a `drupal-demo` site, it fails at this step:

```
+++ drush -y cvapi extension.install key=org.civicrm.volunteer debug=1
Authorization failed                                                                                                                                                   [error]
Array
(
    [error_code] => unauthorized
    [entity] => Extension
    [action] => install
    [trace] => #0 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/API/Kernel.php(166): Civi\API\Kernel->authorize(Object(Civi\Api4\Provider\ActionObjectProvider), Object(Civi\Api4\Generic\DAOSaveAction))
1 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/Api4/Generic/AbstractAction.php(235): Civi\API\Kernel->runRequest(Object(Civi\Api4\Generic\DAOSaveAction))
2 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/CRM/Utils/Migrate/Import.php(374): Civi\Api4\Generic\AbstractAction->execute()
3 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/CRM/Utils/Migrate/Import.php(82): CRM_Utils_Migrate_Import->customFields(Object(SimpleXMLElement), Array)
4 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/tools/extensions/civivolunteer/CRM/Volunteer/Upgrader.php(836): CRM_Utils_Migrate_Import->runXmlElement(Object(SimpleXMLElement))
5 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/tools/extensions/civivolunteer/CRM/Volunteer/Upgrader.php(48): CRM_Volunteer_Upgrader->executeCustomDataTemplateFile('volunteer-custo...')
6 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/tools/extensions/civivolunteer/CRM/Volunteer/Upgrader/Base.php(306): CRM_Volunteer_Upgrader->install()
7 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/tools/extensions/civivolunteer/volunteer.civix.php(131): CRM_Volunteer_Upgrader_Base->onInstall()
8 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/tools/extensions/civivolunteer/volunteer.php(204): _volunteer_civix_civicrm_install()
9 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/CRM/Extension/Manager/Module.php(76): volunteer_civicrm_install()
10 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/CRM/Extension/Manager/Module.php(48): CRM_Extension_Manager_Module->callHook(Object(CRM_Extension_Info), 'install')
11 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/CRM/Extension/Manager.php(264): CRM_Extension_Manager_Module->onPreInstall(Object(CRM_Extension_Info))
12 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/api/v3/Extension.php(58): CRM_Extension_Manager->install(Array)
13 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/API/Provider/MagicFunctionProvider.php(101): civicrm_api3_extension_install(Array)
14 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/API/Kernel.php(168): Civi\API\Provider\MagicFunctionProvider->invoke(Array)
15 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/API/Kernel.php(99): Civi\API\Kernel->runRequest(Array)
16 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/api/api.php(23): Civi\API\Kernel->runSafe('extension', 'install', Array, NULL)
17 /Users/myuser/bknix/build/dmaster/web/sites/all/modules/civicrm/drupal/drush/civicrm.drush.inc(1557): civicrm_api('extension', 'install', Array)
18 phar:///Users/myuser/bknix/civicrm-buildkit/extern/drush8.phar/includes/command.inc(422): drush_civicrm_api('extension.insta...', 'key=org.civicrm...', 'debug=1')
19 phar:///Users/myuser/bknix/civicrm-buildkit/extern/drush8.phar/includes/command.inc(231): _drush_invoke_hooks(Array, Array)
20 phar:///Users/myuser/bknix/civicrm-buildkit/extern/drush8.phar/includes/command.inc(199): drush_command('extension.insta...', 'key=org.civicrm...', 'debug=1')
21 phar:///Users/myuser/bknix/civicrm-buildkit/extern/drush8.phar/lib/Drush/Boot/BaseBoot.php(67): drush_dispatch(Array)
22 phar:///Users/myuser/bknix/civicrm-buildkit/extern/drush8.phar/includes/preflight.inc(66): Drush\Boot\BaseBoot->bootstrap_and_dispatch()
23 phar:///Users/myuser/bknix/civicrm-buildkit/extern/drush8.phar/includes/startup.inc(465): drush_main()
24 phar:///Users/myuser/bknix/civicrm-buildkit/extern/drush8.phar/includes/startup.inc(369): drush_run_main(false, '/', 'Phar detected. ...')
25 phar:///Users/myuser/bknix/civicrm-buildkit/extern/drush8.phar/drush(114): drush_startup(Array)
26 /Users/myuser/bknix/civicrm-buildkit/extern/drush8.phar(10): require('phar:///Users/t...')
27 {main}
    [is_error] => 1
    [error_message] => Authorization failed
)
```

After
-----

It works again.

Merge pull request #15738 from agileware/CIVICRM-1354

Fix the relationship direction in testSingleMembershipForTwoRelationships

Fix for non-allocation of payment to fully reversed checkboxes option

This addresses an issue partially identified in https://lab.civicrm.org/dev/financial/issues/98 whereby
civicrm_entity_financial_trxn records for the civicrm_financial_item table are not created when
refunding against line items with a quantity of zero  - ie where there were selected checkboxes & they
were reversed.

The reason this check was there relates to former logic - now we have a calculated allocation
we don't need this check.

I want to do a test but need to do a bit of work to get it set up & I think this should hit 5.20
as it relates to an issue identified by Kevin in reviewing patches merged to that version

Merge pull request #15731 from eileenmcnaughton/rel_clean

dev/core#1361 Fix bug where 2 memberships created for same relationship

Update dompdf & dependencies

DomPDF issues warnings on php 7.3
The latest version boasts 'Improved compatibility with PHP 7.3'
https://github.com/dompdf/dompdf/releases/tag/v0.8.3

To get there I needed to do
composer   update dompdf/dompdf phenx/php-svg-lib sabberworm/php-css-parser

& also got https://github.com/sabberworm/PHP-CSS-Parser/releases
onto the version that supports php 7+ - no obvious issues in the release notes  and

https://github.com/PhenX/php-svg-lib/releases
- moving us from Bruised Bumblebee to Fanatical Furet

Once againt nothing alarming & improved php 7+ support

Merge pull request #15556 from demeritcowboy/convert-label-to-name-2

Upgrade script for dev/core#1046