Merge pull request #12540 from colemanw/Link

Link to option group page from option group menu item

Merge pull request #12539 from colemanw/recursiveUtils

(NFC) Remove deprecated recursive array utils

Merge pull request #12473 from mattwire/optiongroup_ui_improvements

dev/core#259 OptionGroup Admin UI workflow improvements

Link to option group page from option group menu item

Merge pull request #12529 from colemanw/Deprecate

(NFC) Deprecate redundant array util function

Remove deprecated recursive array utils

Improve optiongroups admin UI workflow

Merge pull request #12534 from civicrm/5.4

5.4 to master

Merge pull request #12533 from JMAConsulting/dev-core-272-54

dev/core#272 : Fatal Error (Regression) on PCP pages associated with Events

dev/core#272 : Fatal Error (Regression) on PCP pages associated with Events

Merge pull request #12526 from agh1/relpermicons

Icons in colored boxes for relationship permissions

Relationship permissions: put icons first when viewing a single relationship

Deprecate redundant array util function

Merge pull request #12528 from eileenmcnaughton/tim

(dev/core#217) Add skeletal PrevNextCache service

Merge pull request #12505 from eileenmcnaughton/export_fix_return_order

Extract code for getting additional return properties, test

Merge pull request #12523 from eileenmcnaughton/product

Fix signature on BAO_Product::add to make ids optional

(dev/core#217) Add skeletal PrevNextCache service

Merge pull request #12524 from mukeshcompucorp/SHOR-9-regression-issues-advanced-search

Shoreditch support: Fix following change in core that resulted in visual regression in shoreditch in advanced search

Export Move relationship type definition to processor class

Export: refactor default return properties retrieval to the Processor class

Move paymentTableID to processor class

paymentid

Refactor out paymentFields

Code tidy up on payent fields var

Remove caching of static arrays & reference marker as we are getting inconsisten results from this function during testing

Extract code for getting additional return properties, test

Fix for test fail when full suite run, make sure Case is enabled, fix params

Remove ids from BAO_Product::create (renamed from add)  signature altogether

This is actually a new function so we don't need to respect the signature of the deprecated one

Relationship permissions: standardize all permission icons into template

Relationship permissions: color the squares

Merge pull request #12518 from eileenmcnaughton/export_notice_tests

Export : Add a lot of unit tests. Fix an enotice

Relationship permissions: move icons to template, change to stacked on squares

Merge pull request #12207 from JMAConsulting/access-3-contri

dev/accessiblity#3 Add aria-label to form elements of contribution page which don't have accessible labels

SHOR-9: Fix regression issues advanced search

Fix signature on BAO_Product::add to make ids optional

Merge pull request #12520 from civicrm/5.4

5.4

Merge pull request #12517 from eileenmcnaughton/case_export_fail

Fix regression on case export from recent export fix

Merge pull request #12519 from totten/5.4-purify

(Forward port) Add Smarty modifier to purify HTML

Add Smarty modifier to purify HTML

With this modifier, we can write Smarty code like:

    <div>{$untrustedHTML|purify}</div>

By using the purify modifier, we've protected against XSS, even if the
output variable contains HTML.

Fix regression on case export from recent export fix

https://github.com/civicrm/civicrm-core/commit/3b4adc92793dccfa446cf66f0740edb543f110d7 changed the way componentmode is defined

It now seems inconsistent

Remove e-notice causing clause.

I'm pretty convinced that this would never be true - although it WOULD be true [] would have the case fields. We are calling queryFields both with case enabled & disabled which is the only thing changing it's output

Test sql Column definitions for all default fields

Merge pull request #12515 from civicrm/5.4

5.4 to master

Merge pull request #12514 from seamuslee001/5-3-1-update-notes

Update 5.3.1 Release Notes to include final SA

Update 5.3.1 Release Notes to include final SA

Merge pull request #12503 from eileenmcnaughton/strtolower_1

Remove LOWER from street_address search, rely on mysql to handle.

Merge pull request #12483 from colemanw/DAODefault

Respect '0' as a default when generating DAOs

Merge pull request #12511 from eileenmcnaughton/master

Merge 5.4 to master

Merge branch '5.4' of https://github.com/civicrm/civicrm-core

Merge pull request #12504 from eileenmcnaughton/yashi

Fix misformatted start_date default on new contribution page

Add in 5.3.1 Release notes

Add escape to various lines in merge.tpl

Escape js strings in smarty templates

security/core#3 Improve output escaping for errors

- In the template, use either `escape` or `purify` for all outputs.
- Remove `htmlspecialchars()` call in PHP since it's now happening in
  Smarty via `escape`.

security/core#2 Harden some of the "mode" inputs

security/core#1 Escape outputs in report stats

Resolve #19 Fix regression following upgrade to latest CKEditor by getting uploaded file response back in json

Update to latest CKEdtior

Add HTML purification to status messages

This is a security protection measure that protects us just a little bit
more against XSS.

security/core#14 Add output encoding within js

security/core#14 Validate "context" inputs

When "context" is passed as a GET parameter, ensure that its values is
a valid "Alphanumeric" type. This helps prevent XSS when the "context"
value finds its way into templates that lack HTML output encoding.

Replace...

    CRM_Utils_Request::retrieve\((['"])context\1,(\s*)(['"])String\3

...with...

    CRM_Utils_Request::retrieve\($1context$1,$3Alphanumeric$3

Also search for the following and manually fix:

    \$_GET\[(['"])context\1\]
    \$_POST\[(['"])context\1\]
    \$_REQUEST\[(['"])context\1\]

Merge pull request #12492 from eileenmcnaughton/msg_fix

dev/core/#/258 Fix message update routine.

Merge pull request #11197 from agileware/CRM-21104

CRM-21104: Forcing reCaptcha on Contribution pages(with online payments) which have no Profile associated with them.

Merge pull request #12499 from wannesderoy/patch-4

Allow Drupal 8 vendor folder outside webroot

Merge pull request #12508 from civicrm/5.4

5.4 to master

Merge pull request #12502 from eileenmcnaughton/no_pay

Prevent financial transactions from being saved with no payment instr…

Merge pull request #12469 from eileenmcnaughton/export_extract

Export cleanup Extract bulk of the transformation for each field to its own function

Merge pull request #12489 from eileenmcnaughton/mergees

dev/core/#/233 Expose information about where a contact has been merged to

Merge pull request #12507 from eileenmcnaughton/report_test_fix

Fix enotice in test

Fix enotice in test

Not sure how the last fix got merged - did I misread the test result - this was a clear error

Fix misformatted start_date default on new contribution page

Unreleased regression from https://github.com/civicrm/civicrm-core/pull/11881
see https://lab.civicrm.org/dev/core/issues/263

Merge pull request #12500 from pradpnayak/28

dev/financial/issues/28, Financial Account cannot be change to non de…

Remove LOWER from street_address search, rely on mysql to handle.

Per https://github.com/civicrm/civicrm-core/pull/12494 the use of LOWER
- hurts performance
- fails to return results on some char sets
- messes with REGEX

This is part of a continued (we removed from contribution search fields last year)
staggered approach to removing this old mechanism

Merge pull request #12487 from agh1/rel-permission-icon-titles-aydun

dev/core#34 Add permission details in `title` attribute of icons

Fix failing test

Prevent financial transactions from being saved with no payment instrument

Merge pull request #12490 from eileenmcnaughton/report

Test fix, use separate emails so distinct doesn't meld them

dev/financial/issues/28, Financial Account cannot be change to non deductible

Allow Drupal 8 vendor folder outside webroot

Test fix, use separate emails so distinct doesn't meld them

A change was merged today that would have required this change to the test to pass. However,
tests were not re-run & it was missed.

a DISTINCT is used so 2 separate emails make sense to test this

Merge pull request #12485 from eileenmcnaughton/queryFields

Export cleanup - pass processor object rather than the query object

Fix message update routine.

Turns out this was seeming to pass but not because the test wasn't running.

Merge pull request #12468 from eileenmcnaughton/export_return

Minor refactor preparatory to function extraction

dev/core/#/233 Use merged data api to display navigation help on contact deleted by merge

dev/core/#/233 Add apis to retrieve information about the ultimate destination contact and source contacts for merged contacts.

This 'data structure' is 'owned' by core & somewhat subject to future change so establishing a tested methodology for retrieving contact history in
core can support extensions such as
- extended reports - which exposes an address_history tab
- privacy extensions - which need to recover and potentially delete contacts that were merged
into the current contact

Merge pull request #12482 from mattwire/nfc_cc_premium_financial

NFC cleanup to financial classes

Merge pull request #12230 from JMAConsulting/dev_core_150

dev/core#150 : Chain select for country/state in Search Builder does not stay within OR groupings

Merge pull request #12486 from eileenmcnaughton/savedSearch

dev/core#230 : set mapping id null on delete action

Set, don't pass queryOperator

Start passing around processor instead of query object

Extract bulk of the transformation for each field to its own function

Minor refactor preparatory to function extraction

Respect '0' as a default when generating DAOs

Merge pull request #12415 from aydun/core-34-v2

core-34 Add view-only option on permissioned relationships

dev/core#34 relationships: add display names and permission explanation as title for permission icons

NFC: make phpcs happy with spacing and no caps for `and` and `or`

dev/core#230 : set mapping id null on delete action

Merge pull request #12452 from colemanw/arrayPath

Cleanup redundant array functions

Merge pull request #12460 from freephile/patch-1

Fix false negatives in checkResourceUrl()

Merge pull request #12484 from eileenmcnaughton/queryFields

Export code tidy up Use queryFields rather than query. Look to stop passing around query.

Merge pull request #11660 from JMAConsulting/CRM-21754

CRM-21754: Duplicate rows in Activity Details report when address fields are displayed

Fix false positives in checkResourceUrl()

Without this proposed fix, I'm receiving false errors about the CiviCRM Resource URL not being set correctly.

The stripos() function will return '0' if the needle string is the first thing found in the haystack. Thus, you must use equivalence rather than a true/false test where "position zero" evaluates to false.

Tested on Civi 5.3.0 on WordPress 4.9.7

I can access http://coastaltrails.org/wp-content/plugins/civicrm/civicrm/packages/jquery/css/images/arrow.png But WITHOUT the fix, I get an error
```
The Resource URL is not set correctly. Please set the CiviCRM Resource URL.
```
I have CiviCRM Resource URL set to `[civicrm.root]/`

My URL Variables are calculated as:
```
[cms.root] http://coastaltrails.org/
[civicrm.root] http://coastaltrails.org/wp-content/plugins/civicrm/civicrm/
[civicrm.files] http://coastaltrails.org/wp-content/uploads/civicrm/
These variables are computed automatically using civicrm.settings.php and its options, such as CIVICRM_TEMPLATE_COMPILEDIR.
```

Use queryFields rather than query. Look to stop passing around query.