Merge pull request #1 from civicrm/master

merge new

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

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

Add 5.12.4 release notes

Use events for CMS resource loading

Merge pull request #14086 from agh1/caseactqueryfix

Case activity query performs poorly on large databases

Merge pull request #14122 from colemanw/menubarHeight

[REF] Use variable for menubar height

Use variable for menubar height

Merge pull request #13689 from eileenmcnaughton/no_record_payment

Call Payment.create from payment.cancel

Merge pull request #14127 from civicrm/5.13

5.13

Merge pull request #14126 from seamuslee001/dev_core_880

Fix typo in DROP table query.

Merge pull request #14045 from totten/master-ab-winning

(REF; dev/core#873) MailingAB - Migrate "copy winner" logic from JS to PHP

Fix typo in DROP table query.

Partial converstion of Group form to be an EntityForm

This is mostly for demo purposes and it should be noted that these changes are generally done in a
'step in the right direction' kinda way, and in accordance with this I have made a very minimal change.

 In this case the change gets the form to be using the entity form
and converts two field. The change didn't initially work for the title field because the schema didn't define the
html type for the field - fixing these things is a feature not a bug of switching to an entity form

We can remove the apply filter line  & the addButtons fn as the parent does these (actually it adds the custom
data too but I've left that out of scope).

If we converted a few more fields we would see that
1) the saved_search_id doesn't have a pseudoconstant defined in the schema, as it should
2) the various field description & help info is not in the schema
3) when we convert the tpl file we add custom data support in, although this form already has it

When we finally switch to a non-quickform layer the new forms will be built of the schema so by
transferring information into the schema and out of thousands of differently coded forms
we take steps in the direction of facilitating that.

That perhaps seems a little unachievable but the api was originally as incoherent as the form
layer and it was only by incremental cleanup it became more consisten

Merge pull request #14110 from mattwire/caseactivitylinks_partial

REF Don't pass activity object to addCaseActivityLinks

Merge pull request #14112 from mattwire/action_filterlinks

Add function filterLinks to return an array of links for an entity that can be used by the API / form layer to generate a link

Merge pull request #14109 from mattwire/refactor_inboundemailshared

REF Use shared function to check inbound email permissions

Merge pull request #14121 from civicrm/revert-14063-core-553

Revert "dev/core#553: Creating new event takes value from default value not from saved template for custom fields"

Merge pull request #14117 from eileenmcnaughton/import_3

[REF] Follow up tidy up on import fixes

Follow up tidy up on import - use same fn for address

This is covered by testImportPrimaryAddressUpdate

Merge pull request #14123 from mfb/drop-bug

dev/core#880: Fix typo in DROP table query.

Merge pull request #14043 from eileenmcnaughton/import_2

Fix import primary phone, primary email, add tests

Fix typo in DROP table query.

Revert "dev/core#553: Creating new event takes value from default value not from saved template for custom fields"

Merge pull request #13994 from eileenmcnaughton/recur_cancel

dev/core#846 Convert Recurring forms to entity forms

Merge pull request #14063 from JMAConsulting/core-553

dev/core#553: Creating new event takes value from default value not from saved template for custom fields

Fix handling of importing 'primary
' phone & email and enable test

Prior to this selecting 'Primary' as location_type_id for phone or email will result in duplicates.

This also simplifies the indexing to be by location type, as it is with addresses, as the code is kinda
confusing

Merge pull request #14108 from mattwire/form_activity_hardcoded_ids

REF Replace hardcoded IDs with pseudoconstants in activity form

Merge pull request #14115 from civicrm/5.13

5.13

Merge pull request #14114 from lcdservices/5.13

dev/core#891 condition on id existenced when retrieving mailing hash

dev/core#891 condition on id existenced when retrieving mailing hash

Merge pull request #14084 from colemanw/UFFieldBAO

(REF) Standardize UFField create function

Add UFField to syntaxConformanceTest

Merge pull request #14105 from eileenmcnaughton/import

 [REF] calll formatLocationBlock from parent function.