finished annotating

annotating

finishing changes with git lab issues

linking and annotating

annotating

annotating

contributors

organizing by git lab issue and condensing as well as pulling out obvious misc

5.15.0 release notes: added boilerplate

5.15.0 release notes: raw from script

Merge pull request #14690 from eileenmcnaughton/profile_copy

dev/core#1088 Fix fatal error when copying profile with civirules enabled

Merge pull request #14692 from agh1/pseudoconstant-caps-5.15

dev/joomla#13 lower-case C on CRM_Core_PseudoConstant breaks 5.14 upgrade on Joomla w/ CiviCase

Use correct caps for CRM_Core_PseudoConstant everywhere

dev/joomla#13 CiviCase: use correct caps for CRM_Core_PseudoConstant

dev/core#1088 Fix fatal error when copying profile with civirules enabled

Fixes a bug where the DAO name was not passed to the hook when copying a profile.

Merge pull request #14656 from eileenmcnaughton/ad_hoc_mailing

dev/core#1030 fix mis-saving of hidden smart group when sending mail from search builder

Merge pull request #14657 from eileenmcnaughton/smart_15

Remove cached smart group entries when removing from a group

Store header to statics in unit tests

dev/core#1074 fix failure of actions from manage groups screen

dev/core#1030 fix mis-saving of hidden smart group when sending mail from search builder

Fixes a bug where sending a civimail from search builder was sending out to the wrong list

Merge pull request #14659 from seamuslee001/5.15

dev/core#1082 properly regenerate civicrm_generated file

Remove cached smart group entries when removing from a group

In 5.15 we have added fixes for smart groups regarding removed entries. Possibly related to this but also possibly pre-existing
I'm seeing contacts who have been removed from smart groups showing up after removal in search results for the duration of the smart group cache.

Regardless the fix feels safe, sensible and correct so I'm comfortable targetting the rc without further research as to whether it
is a regression. The fix being removal of the cache entry when the group entry is removed.

dev/core#1082 properly regenerate civicrm_generated file

Merge pull request #14571 from eileenmcnaughton/mem

dev/core#1038 partial - fix checksum url for cancelling recurring with mid in the url

Merge pull request #14618 from eileenmcnaughton/5.15

dev/core#926 & dev/core#1048 Fix group search builder bugs

Cast = to IN as tests show this PR otherwise breaks on resolving parent-child

Merge pull request #14609 from seamuslee001/5.15

[NFC] Ensure that test is properly cleaned up

Merge pull request #14604 from colemanw/dev/core#1066

dev/core#1066 Fix mailing permissions

[NFC] Ensure that test is properly cleanuped

Fix regression when searching on empty in group search

dev/core#926 Fixes bug on searching for removed members of smartgroups

dev/core#1048 test replicating bug

dev/core#1066 Fix mailing permissions

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

dev/core#1038 partial - fix checksum url for cancelling recurring with mid in the url

This is a partial fix targetting when the mid is the identifier in this url - eg.
ttp://dmaster.local/civicrm/contribute/unsubscribe?reset=1&mid=35&cs=2eefe06c8d2f07bd644233c525dde0d6_1560813106_168
addressing bugs in checksum handling

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".