Merge pull request #14193 from seamuslee001/5.14

On PHP 7.2 we cannot start session until it has been configured by Dr…

On PHP 7.2 we cannot start session until it has been configured by Drupal.

Merge pull request #14191 from seamuslee001/5_13_release_notes

Add in release notes for 5.13.1 drop

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

Set version to 5.14.beta1

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

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

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

Merge pull request #14139 from agh1/upgrade-rebuild-caseact-views

dev/core#832 CiviCase: rebuild case activity views during upgrade

dev/core#832 CiviCase: rebuild case activity views during upgrade

Use CRM_Core_I18n::isMultiLingual() to detect if is multilingual

Fix extra space

Better variable name

Alter menubar breakpoint to match WP admin theme

Add hook to influince menubar css variables

Add css variable for menubar breakpoint

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

Removes language from advanced search results mailings column
headers for non-multilingual sites.

Merge pull request #14131 from colemanw/resourceList

[REF] Use events for CMS resource loading

Add unit tests on contribution recur trxn_id, contribution recur processor id

This is preliminary cleanup towards exposing cancel_reason as a search field, as committed to in proposal to
add the field. This change

- adds unit tests for the trxn_id & processor_id fields
- makes the handling of them more generic
- adds contribution recur field data to the contribution query object so it can be accessed.

Some points about the last of these. Basically we have a situation where the query object needs metadata to
avoid field by field handling. We have done this is some wierd & wonderful ways with dependencies on 'import' &
'export' metadata info to the point where they were given to so many fields as to become meaningless.

I think this method is better - ie. just adding the additional metadata to the query class for all the entities it
deals with. A recently merged PR means 'where' data is still available using this method.

Also I have added uniquenames to the contribution recur fields. Uniquenames no longer really matter outside
- the query object
- profiles
- possibly import & export

We only really expose recurring contribution info through the query object of these so it should be safe. In a future happy
space apiv4 & namespacing will render uniquenames a distant memory.

I couldn't rip out the special handling entirely without dealing with the labels that go into the qill. I figure
we should merge this & then address that - but briefly the issue is that Matt has just changed all the labels
to not quite work in this context so we need to discuss / agree how to manage 'Recurring Contribution Trxn ID' vs
'Trxn ID'.

I also didn't want to rip out the special handling fully in advance of https://github.com/civicrm/civicrm-core/pull/14118
being merged as there could be a performance regression if we let more code hit that unnecessary join line

Merge pull request #14133 from civicrm/5.13

5.13

Merge pull request #14132 from seamuslee001/5.13

Add 5.12.4 release notes

Update 5.12.4.md

Update 5.12.4.md

Update release notes