Merge pull request #14091 from civicrm/5.13

5.13

Merge pull request #14073 from seamuslee001/dev_core_881_unit_test

dev/core#881 Add in unit test demonstrating issue Stoob found with is…

Merge pull request #14092 from eileenmcnaughton/search_513

dev/core#887 Fix fatal error on pledge search

Merge pull request #14083 from grischard/patch-1

Fix upgrade message db setting location

Merge pull request #14052 from yashodha/dev-876

(dev/core#876) Allow url to set IS NULL/ NOT NULL in report for opera…

Merge pull request #14059 from Stoob/patch-2

change is_override logic

Merge pull request #14077 from pradpnayak/AutoComplete

Fixed auto populate of contact refrence field on membership signup form

Merge pull request #14050 from eileenmcnaughton/import

[REF] cleanup input parameters on extracted function

dev/core#887 Fix fatal error on pledge search

Merge pull request #14090 from seamuslee001/5.13

Add release-notes/5.12.2.md

Add release-notes/5.12.2.md

Merge pull request #14085 from totten/master-cron-doc

Cron Status Check - Better hyperlinks

Merge pull request #14079 from colemanw/setValues

(REF) Fix accepted params in CustomValueTable::setValues

Merge pull request #14075 from colemanw/contactBAO

(REF) Fix unnecessary required param in contact BAO update

Merge pull request #14067 from colemanw/menuZindex

Menubar z-index fixes

Cron Status Check - Better hyperlinks

* The old link pointed to a frozen wiki page with bad styling
* The old link was buried

CRM_Utils_System - Allow docURL functions to support "sysadmin" and "dev" links

Merge pull request #14014 from MegaphoneJon/reporting-14

fix getAddressColumns() to respect tableAlias

Fix upgrade message db setting location

Better variable name

Simplify hiding CMS elements behind modal dialogs

Using css instead of js makes this a lot simpler

Merge pull request #14062 from civicrm/5.13

5.13

Merge pull request #13956 from seamuslee001/dev_mail_41

dev/mail#41 Do not generate tracking urls if no mailing id has been p…

Merge pull request #14076 from seamuslee001/e2e_rest_test_get_admin_contact_id

(NFC) Add in unit test checking that E2E test returns sensible contac…

Fix unnecessary required param in contact BAO create

Normally a BAO should require an id and nothing else for an update op
But the contact BAO required contact_type for no good reason.

Drupal#56 - fix quicksearch text color on D8

dev/mail#41 Do not generate tracking urls if no mailing id has been passed in

Merge pull request #14078 from mattwire/mastercard_branding

Update mastercard icon to match branding guidelines

Merge pull request #14080 from MegaphoneJon/drupal-56

Drupal#56 - fix quicksearch text color on D8

Drupal#56 - fix quicksearch text color on D8

/usr/local/sbin/gitcom

(REF) Fix accepted params in CustomValueTable::setValues

Update mastercard icon to match branding guidelines

Fixed auto populate of contact refrence field on membership signup form

(NFC) Add in unit test checking that E2E test returns sensible contact_id when passed in a special token proving that session is working

Fix test on first run

Merge pull request #14070 from eileenmcnaughton/bee_free

dev/core#562 Remove remaining free() instances from tests

Merge pull request #14072 from eileenmcnaughton/dao

Make DAO fields more consistent

Merge pull request #14071 from eileenmcnaughton/bee_freeer

dev/core#562 Remove more DAO->free() calls

dev/core#881 Add in unit test demonstrating issue Stoob found with is_override

Merge pull request #14068 from eileenmcnaughton/search_test

[REF] Clean up CRM_Contribute_Form_SearchTest

Regenerate DAO files to have 'where' fields defined

Update dao.tpl to not make presence of import or export required to add 'where' param

Import & export have been hopelessly overloaded & misued in the code. One reason is that without them
'where' is not present. Logically import & export should have a narrow usage and should not
be a pre-requisite for being available as a search field, in reality not so much.

This simply makes the 'where' field available regardless of the presence of these fields

By doing this we can make the metadata in the Query object able to handle more fields
without special handling (e.g Contribution Recur fields)

dev/core#562 Remove more DAO->free() calls

We have been removing a few of these each release. They no longer provide any benefit
as the resources are released in the deconstructor. OTOH they can in rare circumstances cause bugs.

Previous removals have not caused any issues so the slow approach is starting to
look overly cautious now

Remove remaining free() instances from tests

Merge pull request #14065 from colemanw/summaryBlock

Ensure edit tooltip is always initially hidden in contact summary block

Clean up CRM_Contribute_Form_SearchTest

This is mostly about switching to a data provider to include flexibility & re-usability
of the test

Remove unnecessary d7 script

This is now obsolete due to the new SmartMenus, the drupal tool-drawer
is no longer in the way of dialogs.

Lower z-index of menubar in body

Avoid menubar appearing on top of dialogs which have a z-index of 101

Correctly size UI dialog accounting for height of menubar

Add body class when dialog is open

This should make it easier for reacting to the presence of a dialog with css.

Ensure edit tooltip is always initially hidden in contact summary block

5.12.1.md - Fix attribution

Merge pull request #14061 from totten/5.13-port-notes

Add release-notes/5.12.1.md

Add release-notes/5.12.1.md

Merge pull request #14057 from civicrm/5.13

5.13

whhops this is more like it

change is_override logic

if (!$dao->is_override was prior code.  IS NOT NULL not functional equivalent

Merge pull request #14051 from eileenmcnaughton/inv

dev/core#534 Re-instate pay-now button

(dev/core#876) Allow url to set IS NULL/ NOT NULL in report for operations

dev/core#534 Re-instate pay-now button

This is regression - likely for a few months? Fix adds a unit test & does some additional tidy up

We should consider for 5.12

Merge pull request #14011 from colemanw/typeValidateCleanup

(REF) Reduce redundant code in CRM_Utils_Type::validate

[REF] remove fields from input parameters of formatLocationBlock

As part of a tidy up of this function I am removing  as an input parameter & getting it from
a function. Unit testing added in testAddressWithCustomData to ensure custom data is still imported.

Additional related tests are in that function. This is preliminary to fixing
primary handling & related test for phone & email import - and to tidying up general mess in this function

Add unit test to ensure address custom data is imported

Merge pull request #14010 from colemanw/assetCheck

Skip adding core resources when building assets

Merge pull request #14033 from eileenmcnaughton/recur_cancel_api

Switch to using ContributionRecur.cancel api from CancelSubscription form

Merge pull request #14030 from aydun/casecontact_hooks

Make hook_civicrm_pre() and hook_civicrm_post() fire on CaseContacts

Merge pull request #13993 from eileenmcnaughton/recur_fixes

dev/core#846 Rationalise loading of payment processor across the 3 update forms

Merge pull request #14040 from eileenmcnaughton/import

[REF] Extract formatLocationBlock from import parsing

Merge pull request #13837 from eileenmcnaughton/event_conf

Create contribution before taking payment, per contribution page workflow

Merge pull request #14047 from totten/master-api-wrap

(dev/core#873) civi.api.prepare - Allow dynamic wrappers

Merge pull request #14048 from civicrm/5.13

5.13

Merge pull request #14046 from totten/5.13-schema-fmt

(NFC) SchemaStructure.php - Fix up mismatch between stored+generated code

(dev/core#873) civi.api.prepare - Allow dynamic wrappers

Overview
--------

Allow extensions to dynamically wrap an API. There are some existing mechanisms which sort-of allow wrapping, but
this enables additional use-cases.

Before
------

There are a few techniques for wrapping or overriding an API, but none is suitable to my current use-case. Limitations:

* `hook_civicrm_apiWrappers` - This allows registering things before and after the API call, but it does not allow
   changing the underlying API call.
* `civi.api.{authorize,prepare,respond}` events - Same as above. These are a bit more nuanced/fine-grained, but still does not allow changing
* `civi.api.resolve` event with `AdhocProvider` - This allows you to swap an API completely, but it doesn't allow you
  to delegate back to the original API call (if you've got nothing to add).

After
------

One may subscribe to `civi.api.prepare` and then call the `wrapApi()` helper:

```php
function onPrepare($event) {
  if ($event->getApiRequestSig() === '3.widget.frobnicate') {
    $event->wrapApi(function($apiRequest, $continue){
      echo "Hello\n";
      $r = $continue($apiRequest);
      echo "Goodbye\n";
      return $r;
    });
  }
}
```

Key characteristics:

* The wrapper only applies if you register it specifically for the given API call.
* The wrapper allows you to defer to the original implementation (`$continue`).
* The wrapper allows you to perform logic before and after.
* The wrapper allows you to *conditionally* replace -- you might call `$continue` or something entirely different.

The style here is more event-oriented, but you can see the same concept in OOP systems, such as PHP's function-override notation.
This would be analogous:

```php
class MyChild extends MyParent {
  function frobnicate($arg1) {
    echo "Hello\n";
    $r = parent::frobnicate($arg1);
    echo "Goodbye\n";
    return $r;
  }
}
```

Merge pull request #14044 from totten/master-test-trait-2

(REF; dev/core#873) CiviUnitTestCase - Extract traits to facilitate extension testing

(NFC) SchemaStructure.php - Fix up mismatch between stored+generated code

Overview
--------

The class `CRM_Core_I18n_SchemaStructure` is autogenerated via GenCode, and it is also commited to git.
The two forms don't match because of the recent code-style cleanup.

Before
------

After running GenCode, there appears to be uncommitted changes in `CRM_Core_I18n_SchemaStructure`.
The changes indicate a reversion in code-style (e.g. `null` vs `NULL`; some whitespace).

After
-----

GenCode produces output which matches the recent cleanup.

Comments
--------

Fixing `null` / `NULL` was easy. However, the whitespace mismatch was more subtle -- because the
`PHP_Beautifier` was messing it up. To resolve, I disabled `PHP_Beautifier` for this file, and fixed
the underlying templates to generate well-formed code.

The output of the process matches the existing code; therefore, the change
have no functional impact (NFC).  You can see this by running `setup.sh` and
checking the `git status`.

ContactTestTrait::createLoggedInUser() - Fix domain_id error when running from extension

Move helpers from CiviUnitTestCase to MailingTestTrait

* `createMailing(...)`
* `deleteMailing(...)`

Move helpers from CiviUnitTestCase to DbTestTrait

* `assertDBState(...)`
* `assertDBNotNull(...)`
* `assertDBNull(...)`
* `assertDBRowNotExist(...)`
* `assertDBRowExist(...)`
* `assertDBCompareValue(...)`
* `assertDBCompareValues(...)`
* `assertDBQuery(...)`

Move helpers from CiviUnitTestCase to GenericAssertionsTrait

* `assertType(...)`
* `assertTreeEquals(...)`
* `assertApproxEquals(...)`
* `assertAttributesEquals(...)`
* `assertArrayKeyExists(...)`
* `assertArrayValueNotNull(...)`
* `assertArrayValuesEqual(...)`

Move helpers from CiviUnitTestCase to ContactTestTrait

* `createLoggedInUser(...)`
* `organizationCreate(...)`
* `individualCreate(...)`
* `householdCreate(...)`
* `sampleContact(...)`
* `_contactCreate(...)`
* `contactDelete(...)`
* `groupContactCreate(...)`
* `groupCreate(...)`
* `groupDelete(...)`

Merge pull request #14041 from colemanw/fieldTitles

Add pseudoconstant to UFField dao

Merge pull request #14028 from yashodha/dev-865

(dev/core#865) Result filter criteria doesn't show IS NULL/IS NOT NUL…

Merge pull request #14039 from eileenmcnaughton/cont_recur

Remove deprecated function

Add pseudoconstant to UFField dao

add missing assignment

Extract formatLocationBlock from import parsing

Next steps here are to remove the markIncomplete in testImportPrimaryAddress
and get Primary importing working on Phone & then email and to
call this extracted function instead of ->formatContactParameters(, );
when we are formatting a location (for code simplification as formatContactParameters
effectively just calls this but from much lower down

Merge pull request #14037 from colemanw/dev/core#562

dev/core#562 - Remove dao->free from api

Remove deprecated function

This was deprecated in May 2018 as not called from within core  - this removes

dev/core#562 - Remove dao->free from api

Merge pull request #14036 from eileenmcnaughton/5.13

Arg I put these fixes in 5.12 & master while trying for 5.13

Merge pull request #14035 from civicrm/5.13

5.13 to master

Merge pull request #14034 from eileenmcnaughton/5.13

dev/core#859 - Dedupe fails on address field in 5.12

Merge pull request #14031 from seamuslee001/phpcs_disable_mcrypt

Disable phpcs checking in the Crypt files where mcrypt functions are

Merge pull request #14032 from seamuslee001/ingore_eval

(NFC) Ensure phpcs ignores eval notice in these files as it is required

add test

fix array and notice

Switch to using ContributionRecur.cancel api from CancelSubscription form

The function cancelRecurContribution is only called from the form ant from the api.

This makes it so the form uses the api, which we have started to add unit testing to. Follow
ing this we can use the api for transaction handling and move more of the form logic into
the api, making it available to others & allowing us to better move to a new form layer later
without as much logic in the forms

(NFC) Ensure phpcs ignores eval notice in these files as it is required

Disable phpcs checking in the Crypt files where mcrypt functions are

Make hook_civicrm_pre() and hook_civicrm_post() fire on CaseContacts