Merge pull request #14572 from seamuslee001/dev_core_1052

Fix dev/core#1052 Activities causing a fatal error if target_contact is not accessable to logged in user

dev/core#1052 fix fatal error when first target contact is not visible to the logged in user and update test appropriately

Add in unit test demonstrating the fatal error generated

Merge pull request #14575 from seamuslee001/5_14_1_release_notes

Add release-notes for 5.14.1

Add release-notes/5.14.1.md

Merge pull request #14573 from seamuslee001/5.15

Update http_timeout setting version to match the fact it is being bac…

Update http_timeout setting version to match the fact it is being backported to 5.14

Merge pull request #14565 from vingle/patch-6

Resolve a conflict with Wordpress CSS

Resolve a conflict with Wordpress CSS

Issue was affecting menu item wrap/overflow, described here: https://github.com/veda-consulting/uk.co.vedaconsulting.gdpr/issues/192.

Merge pull request #14545 from demeritcowboy/temp-remove-submit-once

Temporarily remove activity submit once pending PR 14519

Merge pull request #14525 from seamuslee001/feature_timeout_setting

dev/core#1035 Add in new setting http_timeout to set how long in seco…

dev/core#1035 Add in new setting http_timeout to set how long in seconds should HTTP requests last for to fix dev/core#1035

Handle situations where a 0 timeout is passed in

Remove from form

Update variable name as per EIleen

Merge pull request #14544 from totten/5.15-JPG

(dev/core#1044) Extension/MIME matching should be case insensitive

temporarily remove activity submit once pending PR 14519

(dev/core#1044) Extension/MIME matching should be case insensitive

Overview
--------

For CIVI-SA-2019-15, the delivery of file attachments was tightened to
ensure that the file-extension and mime-type were in agreement.  However,
the check yields a false-negative in the common case where the filename has
been capitalized.  It should treat `foo.jpg`, `foo.JPG`, and `FOO.JPG` as
equally valid.

Before
------

* When viewing a contact profile image ending with `.JPG`, there is an error
  message, `Supplied mime-type does not match file extension`.

After
-----

* When viewing a contact profile image ending with `.JPG`, the image is
  delivered.

Comments
--------

See also:

* https://civicrm.org/advisory/civi-sa-2019-15-xss-via-forged-mime-type
* https://lab.civicrm.org/dev/core/issues/1044

Merge pull request #14539 from totten/5.15-tap-yaml

Civi/Test/TAP - Replace Symfony YAML dependency

Civi/Test/TAP - Replace Symfony YAML dependency

This allows two paths:

1. If you have the YAML PECL extension, then that will be used.
   To get the best output, you can install it.
2. If you don't, then it'll output JSON. JSON is considered valid YAML.
   It's not clear that everyone reading the document will handle
   the full range YAML (incl JSON), but... we don't actually have any
   listeners that care to parse the TAP output...

tools/scripts/phpunit - Flip back to preferring 'php' for subprocess

Rationale:

* On the test servers, `php` is wrapper script.
* Outside the test servers, you should be able to use `phpunit` directly
  (rather than calling `tools/scripts/phpunit`), and then it's
  not our problem.

Merge pull request #14511 from eileenmcnaughton/jungle

Add Unit test for testing smart group flow

Add Unit test for testing smart group flow

Merge pull request #14527 from totten/5.15-yaml-dg

(REF) Convert YAML fixtures to JSON. Reset symfony/yaml dependency.

(REF) Convert YAML fixtures to JSON fixtures. Reset symfony/yaml dependency.

Overview
--------

The test suite includes some tests that read fixture data files (i.e. YAML files). This patch
provides a safer way to address a dependency-management issue with parsing YAML files.

Before
------

* In the last public release (`5.14`), the `symfony/yaml` package was not included with `civicrm-core`.
  However, it was loaded during testing by virtue of running `phpunit6`.
* In the `5.15` RC code, `symfony/yaml` was added -- so that the test suite worked in `phpunit6`.
  However, this also added `symfony/yaml` to the general runtime distribution - which is liable to create conflicts
  with Civi extensions and CMS modules that also bundle `symfony/yaml`. (It's a popular package.)

After
-----

* The test suite no longer uses YAML, and the build no longer requires `symfony/yaml`. Instead, it uses JSON.

Merge pull request #14516 from eileenmcnaughton/except

[test] Add PrematureExitException to support unit testing where the code would otherwise exit

Add PrematureExitException to support unit testing where the code would otherwise exit

Merge pull request #14468 from seamuslee001/e2e_asset_builder_test_wordpress

Add CRM_Utils_System::sendResponse(). Fix AssetBuilder's status-code on WP.

Incorporate Tim's Cleanup

Merge pull request #14482 from seamuslee001/update_exception_dao

[NFC] Update Exception DAO class

Update Exception DAO class

Slimplify functions down as per Tim's review comments

Update to use PSR HTTP Message Response Interface as per @totten

Move content of returning response into CRM_Utils_System and have user_system based method to set HTTP Status code

Set version to 5.15.beta1

Merge pull request #14472 from eileenmcnaughton/from_to_test

[nfc] [test] Skip testGetFromTo on date transition

Merge pull request #14470 from eileenmcnaughton/alike

[NFC][test-fix] centralise use of assertLike for comparing sql

Merge pull request #14448 from eileenmcnaughton/exception2

Fix Exception api to save lower id number as contact 1

Merge pull request #14473 from eileenmcnaughton/custom_filter

dev/core#1025 fix unreleased regression on contact.get with custom fields

Fix Exception api to save lower id number as contact 1

This is already actioned (& tested) via ajax fn markNonDuplicates - this moves the functionality from
the ajax layer to the BAO layer & stdises it under unit tests

Merge pull request #14447 from eileenmcnaughton/exception

Fix Exception to remove result from duplicates

Merge pull request #14475 from eileenmcnaughton/contact_def

Permit sort_name as a url parameter on advanced search

Convert sort_name to use search form methodology to pass by url

Merge pull request #14476 from eileenmcnaughton/basic_search

[REF] Move retrieval of basicSearchFields to 'get' fn

Move retrieval of basicSearchFields to 'get' fn

This is a partial of https://github.com/civicrm/civicrm-core/pull/14418

dev/core#1025 fix unreleased regression on contact.get with custom fields

Fixes a bug where contact.get does not retrieve results if a custom field is requested and an activity custom field exists

This bug came about because a recent fix made metadata more consistent but it turns out this class was filtering the
metadata by the passed in  - and then iterating through the metadata. Now it iterates the ids instead.

Merge pull request #14469 from eileenmcnaughton/pledge_msg

Fix version for pledge message update, also remove 'please print'

Skip testGetFromTo on date transition

This avoids fails when the expectations were calculated right at the end of a day but then are compared with the new day's values.

Results in this case look like

CRM_Report_FormTest::testGetFromTo with data set #1 ('20190606000000', '20190606235959', 'this.day', '', '')
fail on data set [ this.day ,  ,  ]. Local php time is 2019-06-07 00:00:13 and mysql time is 2019-06-07 00:00:13
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    0 => '20190606000000'
-    1 => '20190606235959'
+    0 => '20190607000000'
+    1 => '20190607235959'

[NFC][test-fix] centralise use of assertLike for comparing sql

This function makes the tests cope with minor changes to whitespace in sql strings being compared

Merge pull request #14461 from colemanw/displayPrefsStyle

Use crm-checkbox-list styling in display prefs tpl

Merge pull request #14450 from eileenmcnaughton/pay_rs

Wrap titles in metadata in ts on payment.create api

Merge pull request #14433 from eileenmcnaughton/mailin_settings

[Outbound mail admin form] fix some more notices resulting from recent change

Merge pull request #14427 from eileenmcnaughton/search_upgrade

Fix upgrade on participant_register date

Fix version for pledge message update, also remove 'please print'

Merge pull request #14467 from mlutfy/ts-fixes-5-16

Misc translation 'ts' fixes

Merge pull request #14465 from philmb/dev/core#1010

dev/core#1010 pledge_acknowledge template - capitalization and greeting

Merge pull request #14442 from colemanw/drupal8jquery

Conditionally use Drupal's jQuery only if available

Merge pull request #14464 from mattwire/smartgroupeditcriteria

Don't run search when editing smartgroup criteria

Misc translation 'ts' fixes

dev/core#1010 pledge_acknowledge template - capitalization and greeting

Pledge Acknowledgements have incorrect capitalization in greetings, and greetings do not use email_greeting.

Conditionally use Drupal's jQuery only if available

Don't search before showing smartgroup criteria

Whitespace formatting in CRM/Group/Form/Edit.tpl

Merge pull request #14463 from seamuslee001/5_14_master

5.14

Merge in 5.14

Use crm-checkbox-list styling in display prefs tpl

Merge pull request #14394 from eileenmcnaughton/merge_get_conflicts

Add Contact.get_merge_conflicts api

Update 5.14.0.md

install/index.php - Fix misleading install text

During GUI install, there's a health-check based on the PHP version.  Now
that PHP 7 is required, this message *always* talks about PHP 7.x. But
it's presented under a bucket labeled "PHP5".

Update 5.14.0.md

Set version to 5.14.0

Merge pull request #14451 from eileenmcnaughton/format

[NFC] array formatting in api_v3_paymentTest class

Merge pull request #14458 from eileenmcnaughton/cachekey

Test tweaks that *might* help with intermittant test weirdness

Update 5.14.0.md

Merge pull request #14201 from seamuslee001/5.14

Fix AssetBuilderTest when running on WordPress. Use Guzzle.

Merge pull request #14449 from demeritcowboy/bulksms-enotice

dev/core#1023 - E_NOTICE when creating bulk SMS

Merge pull request #14457 from civicrm/5.14

5.14

AssetBuilderTest - Add more assertions for Guzzle variant of test

Test tweaks that *might* help with intermittant test weirdness

It's technically possible for the 2 cache keys to be the same - which would break stuff - although it seems
a bit too frequent to be the reason...

Also we clear one cache in tear down & not the other.

I'm not optimistic either of these will solve the intermittant fails but they seem more correct changed

Also I added verbosity

Merge pull request #14445 from demeritcowboy/deletedActivitiesSearchFilter

dev/core#1022 - Manage Case deleted activities search filter no longer working

Fix Invalid asset builder test by using guzzle

Revert "Fix Asset builder test by altering error_reporting level to match Drupal and changing regex to not care about HTTP version"

This reverts commit be441aa368ef7f1e5a8dd115c065caa172f95a6a.

Update 5.14.0.md

Update 5.14.0.md

Merge pull request #14212 from seamuslee001/wordpress_e2e_assetbuilder_fix

Fix AssetBuilderTest by altering error_reporting level to match Dru…

Merge pull request #14439 from seamuslee001/5_14_0_release_notes_update

Update 5.14.0 release notes with latest changes

Merge pull request #14455 from civicrm/5.14

5.14 to master

Merge pull request #14454 from eileenmcnaughton/php72

(NFC) Remove without param tests

Merge pull request #14453 from eileenmcnaughton/export_sc_514

dev/core#1015 fix regression on exporting soft credits

Remove without param tests

These really date back to when we were creating the api wrapper. I don't think
they add much value now - esp since php is not quiet if we break the wrapper
contract

dev/core#1015 fix regression on exporting soft credits

Cut down from https://github.com/civicrm/civicrm-core/pull/14440/commits due to test fails

Update Release 5.14.0 release notes with latest changes

Update release notes for PR #14440

Withdraw soft credits item

fix E_NOTICE

[NFC] array formatting in api_v3_paymentTest class

Wrap titles in metadata in ts on payment.create api

Fix Exception to remove result from duplicates

This updates, and tests, the Exception.create function (via the api) to remove duplicates from the
prevnext_cache. This feels like a better fix than the cartwheels the prevnext cache code does to avoid
retrieving cached duplicates.

Merge pull request #14441 from colemanw/sortable

dev/core#628 - Make Quicksearch options sortable

Merge pull request #14411 from eileenmcnaughton/merge_get_duplicates

Add api for Dedupe.getduplicates

make search filter work again

Update get_merge_conflicts api

Increase result nuance in response to review

dev/core#628 - Make Quicksearch options sortable

Merge pull request #14438 from eileenmcnaughton/display

(dev/core#1000) Fixes contact's displayname not appearing in membership edit, if no registered email is found

Merge pull request #14363 from mfb/error-msg

Avoid logging error message if status check query fails.