Merge pull request #14579 from civicrm/5.15

5.15

Merge pull request #14568 from greenpeace-cee/fix-schemahandler-cache

Fix use of cached schema information in SchemaHandler

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…

Merge pull request #14562 from agileware/CIVICRM-1244

dev/core#1049: Use FrontEndPaymentFormTrait to assign line items…

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

Merge pull request #14569 from pradpnayak/HardCodes

Removed hardcoded value for registered participant status

Removed hardcoded value for registered participant statu

Fix use of cached schema information in SchemaHandler

This fixes an issue where CRM_Core_BAO_SchemaHandler uses outdated
schema/column information from cache when rebuilding triggers. In
certain scenarios, for example when multiple custom fields are deleted
during a single script execution, this could cause generated SQL to
reference deleted columns, throwing an error.

Merge pull request #14543 from eileenmcnaughton/wtf_null

[TEST] add assert to ensure nullArray & nullObject are not contaminated

Merge pull request #14566 from civicrm/5.15

5.15 to master

Merge pull request #14565 from vingle/patch-6

Resolve a conflict with Wordpress CSS

Merge pull request #14564 from seamuslee001/less_null_array

Remove more uses of CRM_Core_DAO::$_nullArray that are unncessary

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.

Remove a few more uses

Remove more uses of CRM_Core_DAO:: that are unncessary

Merge pull request #14560 from seamuslee001/null_array_profile

Remove unncessary $_nullArray usage when calling createProfileContact function

Merge pull request #14561 from seamuslee001/null_array_execute_query

Remove unneeded use of CRM_Core_DAO::$_nullArray in executeQuery or s…

Remove unneeded use of CRM_Core_DAO::$_nullArray in executeQuery or similar calls

Remove unncessary usage of CRM_Core_DAO::$_nullArray when calling createProfileContact function

And annother instance

CIVICRM-1244: Use FrontEndPaymentFormTrait to assign line items on Event Registration Confirm and ThankYou "forms"

Merge pull request #14558 from seamuslee001/price_bao_null

[REF] Remove more instances of _nullArray

Merge pull request #14559 from seamuslee001/entity_setting_legacy_cache

Alter PSR16 code to take into account of new entitysetting release wi…

Alter PSR16 code to take into account of new entitysetting release with upgraded caching

Remove more instances of _nullArray

Merge pull request #14136 from mattwire/buttonpopups

Support button elements in ajax popups

Merge pull request #14509 from mattwire/buttons_settings_cleanupcache_standard

Add buttons to 'Cleanup caches and update paths' in standard way

Add buttons to 'Cleanup caches and update paths' in standard way

Support button elements in ajax popups

Merge pull request #14557 from civicrm/5.15

5.15 to master

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

Temporarily remove activity submit once pending PR 14519

Standardise exception use in test suite

Using less different types of exceptions is cleaner- our own exception is pretty much our go-to
so using here

[TEST] add assert to ensure nullArray & nullObject are not contaminated

Merge pull request #14556 from eileenmcnaughton/null_another

dev/core#1047 Fix instance of null contamination

Merge pull request #14555 from eileenmcnaughton/null

dev/core#1047 Fix instance of null contamination

dev/core#1047 Fix instance of null contamination

Merge pull request #14552 from eileenmcnaughton/tear

[test] Call parent tearDown more consistently

dev/core#1047 Fix instance of null contamination

[test] Call parent tearDown more consistently

Primarliy consistentcy - also supports checking nullArray is cleared per

https://github.com/civicrm/civicrm-core/pull/14550

Merge pull request #14550 from eileenmcnaughton/null_fix

dev/core#1047 Fix instance of NULL contamination

Merge pull request #14551 from totten/master-theme-cleanup

CRM-18792, dev/core#378 - Catch-up on settings metadata

CRM-18792, dev/core#378 - Catch-up on settings metadata

This is a follow-up to #12929.  In the interim between the writing and
merging of the PR, the version numbers changed.

Fix instance of NULL contamination

Merge pull request #14443 from eileenmcnaughton/export

Fix duplicate households on 'Merge same household' exports

Merge pull request #14549 from seamuslee001/master

5.15

Merge in 5.15

Merge pull request #14525 from seamuslee001/feature_timeout_setting

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

Merge pull request #12929 from totten/master-theming-rb

CRM-18792, dev/core#378 - Create CSS theming subsystem

Merge pull request #14542 from eileenmcnaughton/tests

[TEST] Fix intermittent test fail on NULL array getting contaminated

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 #14548 from seamuslee001/nfc_declaration_fix

[NFC] Fix Test function delcaration to match change in CiviUnitTestCa…

CRM-18792 - CRM_Core_Theme - Add helper for loading CSS files from themes

CRM-18792 - CRM_Core_Resources - Load civicrm.css through theme system

CRM-18792 - Rename `CRM_Core_Theme` to `\Civi\Core\Theme`

CRM-18792 - Civi\Core\Theme - Remove statics

WIP

CRM_Core_Resources::addCoreStyles - Revert change

CRM-18792 - addStyleFile - Always pass through to theme. Support fallback.

Rename `Civi\Core\Theme` to `Civi\Core\Themes`

The class manages a list of themes -- not just a single theme.

CRM-18792 - Add org.civicrm.demotheme

CRM-18792 - Add uncommitted test files (`Civi\Core\Themes`)

CRM-18792 - Fix regression in CRM_Core_ResourceTest

CRM-18792 - Theme naming - Use prefix '_' for hidden themes

This cleans up a few things:

 * Previously, there was a special case for using FALLBACK in `search_order`.
 * If you're creating a multitheme extension, you may want to define a base theme
   (which is extended by the others). Previously, you were required to show this
   base theme as a user-selectable option. Now, it can be hidden.
 * There was a bug where `resolveUrl()` would sometimes call the wrong callback.
   (It used resolver for `$active` instead of `$themeKey`.)

CRM-18792 - Themes - File overrides and excludes should use same naming

Previously, when using `addStyleFile($cssExt,$css$file)`, the file overrides
and exlcudes would combine them differently e.g.

 * For `addStyleFile('civicrm','css/bootstrap.css')`
   * Override `css/bootstrap.css`
   * Exclude `civicrm:css/bootstrap.css`
 * For `('org.foo.bar','css/bang.css')`
   * Override `org.foo.bar-css/bang.css`
   * Exclude `org.foo.bar:css/bang.css`

Now, they use the same notation:

 * For `addStyleFile('civicrm','css/bootstrap.css')`
   * Override `css/bootstrap.css`
   * Exclude `css/bootstrap.css`
 * For `('org.foo.bar','css/bang.css')`
   * Override `org.foo.bar-css/bang.css`
   * Exclude `org.foo.bar-css/bang.css`

"Display Preferences" - Add the `theme_backend` and `theme_frontend` settings

hook_civicrm_activeTheme - Allow extensions and CMS modules to choose active theme

CRM_Utils_Hook::themes() - Tweak docblock

Civi\Core\Themes - Move cache from `short` to `long`

Remove tools/extensions/org.civicrm.demotheme

Fix merge ahem errors

Merge pull request #14547 from civicrm/5.15

5.15 to master

Fix intermittent test fail on NULL array getting contaminated & causing later tests to pass the wrong thing

[NFC] Fix Test function delcaration to match change in CiviUnitTestCase class

Merge pull request #14544 from totten/5.15-JPG

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

Merge pull request #14540 from eileenmcnaughton/anettest

Fix A.net to resolve time when using default

Merge pull request #14533 from eileenmcnaughton/deprecate

Deprecate contribution_date as a parameter

Merge pull request #14537 from colemanw/jsDecode

Add CRM_Utils_JS::decode function for decoding js objects

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 #14524 from eileenmcnaughton/csv

Add csv reader package

Merge pull request #14538 from civicrm/eileenmcnaughton-patch-2

[test] truncate pledge block when cleaning up financial entities

Merge pull request #14535 from colemanw/addressCustom

Address BAO - Handle standard 'custom' param as well as individual fields

Merge pull request #14541 from civicrm/5.15

5.15 to master

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.

Fix A.net to resolve time when using default

This is presumably passed through to the contribution api & hence it is not so grumpy but when
passed to the activity create fn it is failing when the test is run in isolation (& maybe other scenarios?)
due to 'now' not being parsed

[test] truncate pledge block when cleaning up financial entities

Keep hunting down the test cleanup issue

Add CRM_Utils_JS::decode function for decoding js objects

Merge pull request #14536 from civicrm/5.15

5.15 to master

Merge pull request #14511 from eileenmcnaughton/jungle

Add Unit test for testing smart group flow

Merge pull request #14522 from colemanw/ActivityForm

ActivityForm - Redirect to contact page or activity view in standalone mode

Add Unit test for testing smart group flow

Deprecate contribution_date as a parameter

Merge pull request #14512 from agh1/refactor-before-14349

[REF] CRM_Case_BAO_Case::addcaseActivityLinks to CRM_Case_Selector_Search

Merge pull request #14486 from seamuslee001/contribution_date_search_561

[REF] dev/core#561 Convert Contribution Date field to use date picker…

Merge pull request #14531 from eileenmcnaughton/72

Php 7.2 notices fix on import

Merge pull request #14530 from yashodha/mem_reports

Expose Primary member only/Non primary member only filter in membersh…

Merge pull request #14532 from brianPHM/Updating_contributor_yml

Adding myself to the contributors file

Adding myself to the contributors file

Php 7.2 fix on import

Fixes a known issue where php 7.2 is 'noisy' if you try to count NULL - solution is to use an empty array

Address BAO - Handle standard 'custom' param as well as individual fields

The Address BAO was an oddball in that it took raw unformatted custom field input and processed it.
Other BAOs expect the custom field input to alreay be processed.
This allows it to handle either format.

Expose Primary member only/Non primary member only filter in membership reports

Merge pull request #14523 from kcristiano/cividist

CiviDist fails on BSD flavor of 'cp' with '-r -p' switch to '-R -p'

Add csv reader package

Adds https://csv.thephpleague.com/9.0/ because we can now (since we are php 7.x+ now).

Apart from being a pretty good package it will help with writing tests for csv output

It will also help us cleanup some of our csv code down the way

Merge pull request #14528 from civicrm/5.15

5.15

Merge pull request #14526 from seamuslee001/nfc_fix_template_indent_misc

[NFC] Fix indenting in Misc Setting Template

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

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

Merge pull request #14505 from seamuslee001/legacy_psr16_multisite

Update PSR16 handling for multisite extension legacy caching group

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

[NFC] Fix indenting in Misc Setting Template

CiviDist fails on BSD flavor of 'cp' with '-r -p' switch to '-R -p'

Add in deprecated notice about code path being removed at end of 2019

ActivityForm - Redirect to contact page or activity view in standalone mode

Merge pull request #14501 from mattwire/attachment_description

Display description next to 'paperclip' file icon - usually the filename

Merge pull request #14513 from eileenmcnaughton/export_sc_just_change

dev/core#1015 fix regression on exporting soft credits - more robust …

Merge pull request #14518 from colemanw/paymentEditFix

dev/core#965 Fix destination in payment edit