dev/core#1077 Fixes making report listing actions links hookable

Merge pull request #15071 from totten/master-gat

(NFC) GenericAssertionsTrait - Improve docblocks

(NFC) GenericAssertionsTrait - Improve docblocks

This code was previously extracted from CiviUnitTestCase. It had weak docblocks. Make them better.

Changes:

* Add more types and examples.
* Describe some of the subtleties in the different forms of array-comparison.
* Remove dependency on `ts()`.  Unit-test assertions are not actually translated.

Merge pull request #15050 from eileenmcnaughton/loadmappingtest

[Import] [Test] Initial test on loadSavedMapping function.

Merge pull request #15066 from seamuslee001/attachment_api_stablize

Stabalize Attachment API Test

Merge pull request #15067 from seamuslee001/fix_test_syntax

[NFC] Fix test syntax

Initial test on loadSavedMapping function.

To facilitate this I
1) made it public - in the end it will go anyway
2) removed a 'pass-through' variable from the return array - it is passed in & returned unchanged
3) made it cope without a notice when a class property is not defined - that will go too soon

Merge pull request #15049 from eileenmcnaughton/loadMapping

[NFC] swap instances where double = is used but it should be triple =

[NFC] Fix test syntax

Stabalize Attachment API Test

Merge pull request #15065 from civicrm/5.17

5.17

Merge pull request #15018 from eileenmcnaughton/metadata

[Ref] [Import] move metadata calculations to a trait

Merge pull request #15048 from eileenmcnaughton/5.17

Fix unreleased regression on import

Merge pull request #15043 from prondubuisi/fix-civi-api3-exception

dev/core#1188 Fix API v3  error Class or interface CiviCRM_API3_Exception does not exist

Merge pull request #15059 from civicrm/5.17

5.17

Merge pull request #15060 from bsilvern/event-10.1

dev/event#10 - Prevent old notes from being exposed in event confirmation email

dev/event#10 - Prevent old notes exposed in event confirmation email

Merge pull request #15058 from seamuslee001/dev_core_1186

dev/core#1186 add in unit test to lock in fix from dmeritcowboy in #1…

Merge pull request #15057 from seamuslee001/nfc_upgrade_template_reformat

[NFC] Update Upgrade Template to use short array syntax

Merge pull request #15055 from demeritcowboy/file-no-description

dev/core#1186 - ignore exception if no description field

dev/core#1186 add in unit test to lock in fix from dmeritcowboy in #15055

[NFC] Update Upgrade Template to use short array syntax

Merge pull request #15046 from 19ATF77/Fix_Activitytpl

Do not show Activity Separation on when viewing an Activity

Merge pull request #15056 from civicrm/5.17

5.17

Merge pull request #15054 from colemanw/select2

Update select2 to latest 3.x + patches

Merge pull request #15047 from eileenmcnaughton/pptx

dev/core#1190 Add pptx to safe file types

catch exception when no description

Update select2 to latest 3.x + patches

Merge pull request #15029 from reecebenson/bugfix/reports-remove-sorting-filters

Allow for the deselection of sorting filters

Do not show Activity Separation on when viewing an Activity

Fix undefined index when setting value to empty when removing order by filters

Extract metadata definition into a trait.

This metadata needs to be shared between multiple import forms. They do not have a common parent.

Curently this is done via calculating once & passing via quickform. However, the formats
are convoluted & hard to read.

Information that is user generated should be calculated per form - not so much metadata

[NFC] swap instances where double = is used but it should be triple =

In all the changed cases this is a specific comparison with a string so the change is best practice but not functional.

One unused variable removed

[NFC] code tidy ups

Merge pull request #15015 from eileenmcnaughton/import_message

[REF] [Import] Minor cleanup on status setting.

Merge pull request #15017 from eileenmcnaughton/import_cont

[REF] [Import] extract saved mapping code to parent class

Merge pull request #14981 from eileenmcnaughton/load_extract

[Import] [Cleanup] remove unused parameters

Merge pull request #15028 from eileenmcnaughton/import_processor

[Test] [Ref] [Import]Add wrapper class for importProcessor

dev/core#1190 Add pptx to safe file types

I feel like it would be better to store these as an array somewhere - maybe as the first step to
a metadata construct for Option groups...

Fix unreleased regression on import

https://github.com/civicrm/civicrm-core/commit/2adc819d0e59d71ffcf14c6799a379f108b07d60
made the code a little more readable but changed the indexing from 0-based to column based.

See https://pear.php.net/manual/en/package.database.db.db-common.setfetchmode.php

THis can be replicated by trying to update an existing mapping

Merge pull request #15032 from demeritcowboy/human-label-function

dev/core#1116 - towards fixing activity type name vs label

Merge pull request #15045 from mattwire/detaillogging_viewdelete

Enable view/revert delete action for detail logging

Merge pull request #15035 from scardinius/core-1185

Display only own assignees contact(s) on batch activity form

Merge pull request #14992 from sushantpaste/membership-fee-token

Membership fee token fix dev/core/#377

Enable view/revert delete action for detail logging

fix API v3 exception class not found erro

Merge pull request #15042 from civicrm/5.17

5.17

Merge pull request #15041 from seamuslee001/5.17

Add release-notes/5.16.2.md

Add release-notes/5.16.2.md

Merge pull request #15038 from civicrm/5.17

5.17 to master

Merge pull request #15037 from demeritcowboy/fin-account-limbo

dev/financial#65 - Revert recent changes that cause financial account edit form to be blank

dev/financial#65

Merge pull request #15036 from reecebenson/contributor-add

Add myself to the contributor list

make temporary function towards bigger changes

Add myself to the contributor list

Merge pull request #15033 from eileenmcnaughton/dumb

Remove silly dumb unreachable lines

dev/core#1185 Display only own assignees contact(s) on batch activity form

Remove silly dumb unreachable lines

Add wrapper class for importProcessor

With the export cleanup creating a new sensible class & migrating code to it turned out to be a good approach.

This adds a class that wraps around the Importer Object, taking the mapping_field db format of fields
as a format - this is also where we wound up with the Export - using that format as it is what is saed.

There is a gap as I noted in https://lab.civicrm.org/dev/core/issues/1172 around the schema's adequacy.

That is not such an issue for this - in that we are not dealing with Contribution import and
this is really just exposing the wrapper for the unit tests at this stage - the only change to
'live' code is a little less php v4 support - we have removed a bunch of these from the constructor of the
other objects already with no observed issues

Merge pull request #15000 from demeritcowboy/activityName-1116

dev/core#1116 - rename local var activityName

Merge pull request #15031 from mfb/upgrade-fix

Fix upgrade version for #13487

Fix upgrade version for https://github.com/civicrm/civicrm-core/pull/13487

Merge pull request #15027 from demeritcowboy/is-nullish

[Test] Demonstrate the truthiness of CRM_Utils_System::isNull

Allow for the deselection of sorting filters

Merge pull request #15012 from seamuslee001/dev_core_752

dev/core#752 Add cid parameter when loading custom data via backend f…

Merge pull request #14988 from eileenmcnaughton/sc

Unit test for custom date parsing

Merge pull request #14999 from demeritcowboy/activityTName-1116

dev/core#1116 - just rename local var activityTName

Merge pull request #15026 from civicrm/5.17

5.17 to master

Merge pull request #15025 from seamuslee001/5_16_1_release_notes

5.16.1 release notes

Merge pull request #15016 from eileenmcnaughton/act_speed

[NFC] [Test] dev/core#1098 Add unit test & code comments relating to the slow activity search

Update 5.16.1.md

Add in release notes for 5.16.1

Merge pull request #15024 from civicrm/5.17

5.17

Merge pull request #15021 from seamuslee001/5.17

core#1182 - revert crm.menubar.js changes

demonstrate isNull

core#1182 - revert crm.menubar.js changes

Merge pull request #15005 from colemanw/empty

Swap CRM_Utils_Array::value for empty() in conditionals

Merge pull request #15020 from MegaphoneJon/core-1182-517

core#1182 - revert crm.menubar.js changes

Merge pull request #15019 from MegaphoneJon/core-1182

core#1182 - revert crm.menubar.js changes

core#1182 - revert crm.menubar.js changes

core#1182 - revert crm.menubar.js changes

Swap CRM_Utils_Array::value for empty() or isset() in conditionals

[REF] [Import] extract saved mapping code to parent class

I will follow up with narrower adds

Merge pull request #15013 from eileenmcnaughton/member_import

[Import] Minor cleanup on membership import

dev/core#1098 Add unit test & code comments relating to the slow activity search

The issue is the joining in of component filtering - we should fix this but do it correctly....
We probably need to extend the test to reflect contact ACLs first

[Import] Minor cleanup on membership import

We have a nasty construct (perhaps due to php4 limitations) where the code iterates the _mapperKeys
array & sets a class property specifying the key it occupies. We later use this to check it occupies a key.

This could be much more readable by just checking the array at the right time.

Also I removed an unused private property & a require_once to api.php which is always included

Minor cleanup on status setting.

It appears that this complicated handling is because in the past the setStatus did not have the
ability to ensure a given message was set only once - hence it's being set multiple times

dev/core#752 Add cid paramater when loading custom data via backend forms so that hooks can modifiy based on contact_id being viewed

Add in handling for cid not being set

Fix assign of cid and switch to using javascript if rather than smarty

Merge pull request #15010 from eileenmcnaughton/event_inv

Event confirm - (very) minor cleanup + test

Update test to reflect test setup change

Event confirm - minor cleanup + test

I've just started delving into this & have
1) added a test
2) removed an assign that is  ALSO done in the sendMail function - ie participantID - this is tested
3) removed a pass-by-reference that I checked was not required

Merge pull request #15008 from civicrm/5.17

5.17

Merge pull request #15007 from MegaphoneJon/core-1175-517

core#1175 - fix custom searches

core#1175 - fix custom searches

just rename local var activityTName

Merge pull request #14995 from mfb/context-fix

Error log improvements: Provide priority level

Merge pull request #14715 from eileenmcnaughton/cache_sub

Rewrite subTypeInfo to use caching mechanism

Merge pull request #14998 from seamuslee001/manual_geo_code_export

Allow Manual Geo Code flag to be exported

Allow Manual Geo Code flag to be exported

FIx Export tests by altering to include the manual_geo_code field

Update to title case