Merge pull request #14200 from demeritcowboy/to-be-or-not-to-be

contributor-keys file - jira/lab.c.o username fix

Merge pull request #14177 from eileenmcnaughton/free

IDE re-format of tools directory

Merge pull request #14180 from eileenmcnaughton/recur_search

Enotice fix on creating a smart group

jira

Merge pull request #14192 from civicrm/5.14

5.14

Merge pull request #14191 from seamuslee001/5_13_release_notes

Add in release notes for 5.13.1 drop

Merge pull request #14074 from mfb/session-start

Don't call session_start() before CMS bootstrap (PHP 7.2 compat)

Merge pull request #14135 from colemanw/menubarVars

Hook to alter menubar css variables & fix breakpoint in WP

Update 5.13.1.md

Merge pull request #14189 from MegaphoneJon/wordpress-26-5.14

wordpress#26 - fix REST

Add in release notes for 5.13.1 drop

wordpress#26 - fix REST

Merge pull request #14188 from seamuslee001/dev_core_931_5_14

Resolve dev/core#931 by not doing translation on the query if field e…

Resolve dev/core#931 by not doing translation on the query if field exists during the upgrade process

Merge pull request #14171 from eileenmcnaughton/copy_dao

[ref] Move copyCustomFields function from Event to Core_DAO for re-usablibilty

Merge pull request #14152 from seamuslee001/dev_translation_26

dev/translation#26 Fix Notice errors caused by l10n rewriting of quer…

IDE re-format of tools directory

We didn't fix array syntax on this one yet

Enotice fix

Dates back to https://github.com/civicrm/civicrm-core/commit/eda34f9b15c494715e8fc54f09b5ea4308d16b17#diff-8041069091a94b730ff8ad60bdd24042R70
(Mar 2018).

To recreate do a first name search & create a smart group from it

Merge pull request #14178 from eileenmcnaughton/config

[cleanup] Remove deprecated pattern &CRM_Core_Config::singleton()

Merge pull request #14179 from eileenmcnaughton/free_dao

dev/core#562 [cleanup] remove some more instances of ->free()

Set version to 5.15.alpha1

Set version to 5.14.beta1

dev/core#562 remove some more instances of ->free()

[cleanup] Remove deprecated pattern &CRM_Core_Config::singleton()

Merge pull request #14176 from civicrm/5.13

5.13

Merge pull request #14056 from agh1/5.13.0-releasenotes

5.13.0 release notes

Update 5.13.0.md

Merge pull request #14173 from mlutfy/cart-payment-cid

Event Cart: pass the contactID to fix payment on Stripe

Merge pull request #14170 from mlutfy/cart-emails

Event Cart: fix sending of email receipts

Merge pull request #14175 from mlutfy/cart-cc-icons

Event Cart: add support for the Credit Card type icons

Merge pull request #14174 from mlutfy/cart-allow-same-participant

Event Cart: honor the allow_same_participant_emails setting

Event Cart: pass the contactID to fix payment on Stripe

5.13.0 release notes: late changes

uppercase if work is continued or begun

moving gitlab issues to the end

5.13.0 release notes: bulk of edits

5.13.0 release notes: added boilerplate

5.13.0 release notes: raw from script

Event Cart: add support for the Credit Card type icons

Event Cart: honor the allow_same_participant_emails setting

Follow up by making more generic

[ref] Move copyCustomFields function from Event to Core_DAO for re-usablibilty

Merge pull request #14169 from mlutfy/cart-i18n

Event Cart: fix start_date formatting in line items during checkout

Merge pull request #14168 from eileenmcnaughton/enotice

dev/core#918 enotice fix part 2 #14165

Event Cart: fix sending of email receipts

Event Cart: fix start_date formatting in line items during checkout

dev/core#918 enotice fix part 2 #14165

Merge pull request #14166 from civicrm/5.13

5.13

Merge pull request #14163 from seamuslee001/flexmailer_26

(flexmailer#29) civicrm/mailing/view - Use Mailing.preview API. Fix compatibility with Flexmailer.

Merge pull request #13887 from eileenmcnaughton/group_entity

dev/core#818 Partial conversion of Group form to be an EntityForm

Merge pull request #14164 from eileenmcnaughton/in_fix

Fix deprecation handling [intra-rc-follow up] on #13999]

Return MailingID rather than ['id'] as id may not always be supplied

(flexmailer#29) civicrm/mailing/view - Generate content via Mailing.preview API

A root cause of flexmailer#29 is that the flexmailer has to override
multiple parts of CiviMail.  Case in point: it overrides the
`civicrm/mailing/view` and forces it to generate content via
`Mailing.preview` API.  This is unfortunate because flexmailer's variant is
missing other features (regarding permissioning and contact IDs).

This revision makes it unnecessary for flexmailer to override
`civicrm/mailing/view`.

Merge pull request #14165 from eileenmcnaughton/enotice

dev/core#918 Enotice fix

Merge pull request #14161 from eileenmcnaughton/really

Convert dedupe select to select 2 and remove not-used var

(flexmailer#29) Mailing.preview API - Fix array access warning. Make unit-test pass.

(flexmailer#29) Mailing.preview API - Add unit-test for call without contact ID

Enotice fix

The check is looking for status at the wrong level in the array

flexmailer#26 Fix Flexmailer handling of annonmous user access to view Mailings by porting handling of annon situation to api

Fix deprecation handling

Turns out we were one of the sites naughtily using the BAO directly who needed this handling
to work - but because we passed a number in quote it didn't - this fixes

Merge pull request #14157 from eileenmcnaughton/ref_dedupe

[REF] function extraction in dedupe code

Merge pull request #14162 from civicrm/5.13

5.13

Merge pull request #14064 from JMAConsulting/core-338

dev/core#338: Batch update membership type

Merge pull request #14156 from totten/5.13-mail-tokens

CiviMail - Restore support for preview of "mailing"/"action" tokens via TokenProcessor/Flexmailer

Convert dedupe select to select 2 and remove not-used var

CRM_Mailing_TokensTest - Update spec to match expected behavior

The preceding commits revised the behavior of `{mailing.*}` and `{action.*}`
when previewed via `TokenProcessor` (so that they match the preview
logic in other cases).

This simply changes the spec to match.

Merge pull request #14159 from civicrm/5.13

5.13 to master

Merge pull request #14155 from seamuslee001/php_7_2_bin_cli_class

dev/core#907 Fix Deprecation notice for PHP7.2 in bin/cli.class.php

[REF] function extraction in dedupe code

CiviMail - Restore support for previewing mailing-tokens via TokenProcessor/Flexmailer

See preceding commit for general description - this simply applies the same
concept for another set of tokens.

CiviMail - Restore support for previewing action-tokens via TokenProcessor/Flexmailer

Overview
--------

When using `TokenProcessor` to generate a mailing (e.g.  as with Flexmailer/Mosaico), the action-tokens (e.g.
`{action.optOutUrl}`) are generated via `CRM_Mailing_ActionTokens`.  To properly generate them,
`CRM_Mailing_ActionTokens` relies on certain information (e.g.  mailing/job ID).  However, that information is no
longer available when performing a "Preview" -- leading to misbehavior in previews.  This patch allows Flexmailer to
restore parity for previewing those tokens.

Before (Pre-5.6)
----------------

* When a user begins composing a mailing, CiviMail creates a draft mailing with a concrete ID (e.g.  `mailing #123`).
* To preview the mailing, the UI calls `Mailing.preview` API with the ID of the mailing.
* Flexmailer/Mosaico generates the preview by calling `TokenProcessor` and therefore `CRM_Mailing_ActionTokens`.
* `CRM_Mailing_ActionTokens` has strictness checks. These pass because the ID is available.

Before (5.6-5.12)
----------------

As a performance enhancement, CiviCRM 5.6 (PR #12509; [dev/mail#20](https://lab.civicrm.org/dev/mail/issues/20)) revised
the signature for `Mailing.preview` API to allow previews *without* having a specific mailing record/job/ID. Consequently:

* When a user begins composing a mailing, CiviMail creates a draft mailing with a concrete ID (e.g.  `mailing #123`).
* To preview the mailing, the UI calls `Mailing.preview` API ~~with~~ **without** the ID of the mailing.
* Flexmailer/Mosaico generates the preview by calling `TokenProcessor` and therefore `CRM_Mailing_ActionTokens`.
* `CRM_Mailing_ActionTokens` has strictness checks. These ~~pass~~ **fail** because the ID is ~~available~~ **unavailable**.

After
----------------

* When a user begins composing a mailing, CiviMail creates a draft mailing with a concrete ID (e.g.  `mailing #123`).
* To preview the mailing, the UI calls `Mailing.preview` API ~~with~~ **without** the ID of the mailing.
* Flexmailer/Mosaico generates the preview by calling `TokenProcessor` and therefore `CRM_Mailing_ActionTokens`.
* `CRM_Mailing_ActionTokens` has ~~strictness~~ **less strict** checks. These **pass** because the `context[schema]` hints that
  a mailing ID *will be available* when needed.

Merge pull request #13844 from mattwire/corepaymentmore_essay

Update docblock for doPayment function

dev/core#907 Fix Deprecation notice for PHP7.2 in bin/cli.class.php

Update docblock for doPayment function

Merge pull request #14145 from demeritcowboy/submitOften

dev/core#904 - undo part of 13333

Merge pull request #14144 from eileenmcnaughton/dedupe4

dev/core#723 [REF] extract chunk of code that definitely does something.

Merge pull request #14148 from eileenmcnaughton/dedupe8

[REF] extract getConflicts function

[REF] extract chunk of code that definitely does something.

This is a really blunt extraction of a chunk of code whose Raison D'Etre I don't quite know as yet.

I've done the absolute minimum variable cleanup to keep it really clean in the first instance

[REF] extract getConflicts function

This is for readability but I also want to add an api that retrieves conflicts without
trying a merge for more control

Merge pull request #14151 from eileenmcnaughton/dedupe7

[REF] Extract get cfields function

dev/translation#26 Fix Notice errors caused by l10n rewriting of query so query outputed create view rather than the create table

Merge pull request #14150 from eileenmcnaughton/last_free_jcal

dev/core#561 Remove last free jcalendar

Extract get cfields function

Merge pull request #14119 from eileenmcnaughton/recur_meta

dev/core#830 [REF] Add unit tests on contribution recur trxn_id, contribution recur processor id

Remove last free jcalendar

This is the last jcalendar outside of buildDateRange instances.

It turns out to be 'just a form' and pretty straight forward

Merge pull request #14146 from eileenmcnaughton/dedupe6

dev/core#723 [REF] Declare contactType var rather than using long array syntax.

Merge pull request #14149 from eileenmcnaughton/exreport

[cleanup] remove CRM_Report_Form_Extended

fix save and new

Merge pull request #14134 from stesi561/dev/core/248

dev/core#248 - Fixes advanced search mailings results headers

Merge pull request #14141 from seamuslee001/view_url_double_protocol_test

(NFC) Expand unit test to show off double protocol error

(NFC) Expand unit test to show off double protocol error

Expand test to include situation if the Other protocol is set rather than a specific protocol

Make test portable

[cleanup] remove CRM_Report_Form_Extended

This file was added to core around 4.2 but for various reasons the ExtendedReport intiative
moved to an extension & it was never used. This just removes it

Declare contactType var rather than using long array syntax.

Very small tidy up for legibility

Merge pull request #14142 from eileenmcnaughton/dedupe2

 [REF] move gathering of location info to relevant function (towards dev/core#723)

Merge pull request #14143 from eileenmcnaughton/dedupe3

dev/core#560 Remove a couple of instances of fatal errors in favour of throwing exceptions

Merge pull request #14140 from eileenmcnaughton/dedupe

[REF] Move the qfbug handling to it's own function (towards dev/core#723)

Merge pull request #14128 from agileware/CIVICRM-1143

Fixed double protocol being added in link by CKEditor.

Remove a couple of instances of fatal errors in favour of throwing exceptions

https://lab.civicrm.org/dev/core/issues/560

[REF] move gathering of location info to relevant function

We have a loop that parses the migrationInfo to get just location fields which we loop through, but
migrationInfo is already available in the moveLocationInfo fn so we can just handle that param in there
& improve code readability (we do a little less array looping but it's pretty cheap & the code
readabiliity is a real issue here

Move the qfbug handling to it's own function

This is a really toxi function. In order to save the pain or iterating multiple times through a small array
we are dealing with an awful lot of complexity. I'm changing this to iterate through it multiple
times in order to rationalise the code - I think performance is minimal & being able to read the code....