CRM-20858: Remove Try/Catch Blocks that Ignored Caught Exceptions

The try catch blocks were apparently added because not all fields passed were
custom fields, generating exceptions when the API call was made to obtain its
option group. This happened because actually all fields selected for merge
were being inserted into the $submittedCustomFields array, instead of just
custom fields. Plus, obtaining the field's ID on a field that was not custom
rendered strange strings to be used on search of the field, as this ID is
obtained by getting a substring from the field's key, that should have the
format 'move_custom_<fieldid>' (fields that are not custom don't have that
format).

CRM-20858: Call moveContactBelongings Only Once

moveContactBelongings() was bing called twice, first to move selected related
tables, and the second to move all other tables that reference contacts table.

Refactored moveContactBelongings() method so that it can be called only once,
by calculating in one go the tables that have to be taken into account when
merging, according to what has been selected to move.  Basically, it searches
for all tables related to the contact, removes related tables, and then
includes only those related tables selected to be merged.

CRM-20858: Fix Failing Test on Custom Field Merge

Test for merging custom fields with is_view flag on was failing. This was
happening because the same field was being used on prior tests, setting the
is_view flag at the start of the failing test. Fixed by creating a new custom
field with the is_view flag set as 1, and using that field on the test.

CRM-20858: Fix Style Warnings for Upper Case Constants

CRM-20858: Add Documentation Block for customRecordExists

CRM-20858: Fix Merging of Unselected Custom Fields and Refactor Code

When a Custom Group had more than one custom field and only some of those
fields were selected to be merged, all the fields in the custom group were
merged. This happened exclusively on contacts that had no record on the
custom group's table.

Fixed this by inserting a record on the custom group table before updating
references from source contact to target contact, this way, all values in the
custom group will start empty and only submitted values will get assigned.

Also refactored the solution to make it easier to read and used more descriptive
variable and function names.

CRM-20858: Add Failing Tests to Verify Merging of Custom Fields

If a contact that has values stored in a custom group table is merged into a
contact that doesn't have a record in that table, all values in the source
target will be merged into the target contact, even if those values were
not explicity selected to the merge operation.

The merging operation updates all database references from the source contact
to the target contact on all tables. When the target contact has a record on
a custom group table, this update will fail, since the table's configuration
enforces the entity_id (the reference to the contacts table) to be unique.
However, when the target contact has no record on that custom group table, the
source contact does have a record and the merge is performed, the original
record will be updated to reference the target contact, and hence inherit ALL
values of the source contact, even if none of the values were selected to be
merged.

Added two tests, the first one to verify that merging a contact with a record
on a custom group table does not merge its values into a target contact with
no record in the custom group table; the second one to check that if only some
fields of a custom group table are selected to be merged, only those values
are copied, leaving all others unset.

All [] arrays replaced by array()

Indentation fixed

PhpStorm's tabs are now fixed to spaces.

Update Merger.php

1. I think we'll need some more error-handling. The code should probably throw an exception if any of those API calls fail?
-Done

2. There are two calls to 'moveContactBelongings' - won't the first call be affected by the additional code you've added that 'ignores' changes in they're not in the array?
- Handled

Fix for jira issue CRM-20858

This PR is for jira issue https://issues.civicrm.org/jira/browse/CRM-20858

Merge pull request #10586 from colemanw/CRM-20794

CRM-20794 - Colors for case status

Merge pull request #9784 from monishdeb/CRM-19702

CRM-19702: Fatal error when merging contact records with custom file fields

Merge pull request #10637 from mickadoo/CRM-20849-allow-duplicate-psr4-prefix

CRM-20849: Allow duplicate PSR-4 prefixes

CRM-20851 php notice (#10639)

 Fix php notice when is_primary is not set

CRM-19702: Fatal error when merging contact records with custom file fields

Merge pull request #10638 from mattwire/CRM-20772_statusbounce

CRM-20850 Replace fatal with statusBounce

Merge pull request #10604 from jitendrapurohit/CRM-20809

CRM-20809 : Modify extension cache file to get created in uploads directory

Merge pull request #10642 from civicrm/revert-10370-CRM-20321

Revert "CRM-20321, CRM-20652 : Changing membership type should change related contribution and generalise changeFeeSelection function to be used for any entity"

Revert "CRM-20321, CRM-20652 : Changing membership type should change related contribution and generalise changeFeeSelection function to be used for any entity"

modify fix to place cache dir in upload dir

Merge pull request #10431 from JMAConsulting/NoticeFixes-1

[ready for core team to review] CRM-20649, Fixed notice error on form rule

Merge pull request #10450 from JMAConsulting/CRM-20667

[ready for core team to review] CRM-20667, fixed addtional line creation when multiple membership is …

Replace fatal with statusBounce

CRM-20849: Use "addPsr4" instead of "setPsr4", which replaces existing PSR-4 prefixes with the same name in the autoloader.

Merge pull request #10370 from JMAConsulting/CRM-20321

CRM-20321, CRM-20652 : Changing membership type should change related contribution and generalise changeFeeSelection function to be used for any entity

Merge pull request #10629 from jitendrapurohit/CRM-20834

CRM-20834 - Drupal user not created via Profile due to missing email …

Merge pull request #10635 from civicrm/4.7.22-rc

4.7.22 rc

Merge pull request #10636 from jitendrapurohit/CRM-20834-rc

CRM-20834 - Drupal user not created via Profile due to missing email

CRM-20834 - Drupal user not created via Profile due to missing email

Merge pull request #10615 from mepps/CRM-20828

CRM-20828 Remove limit 1 from saved mappings selection

Merge pull request #10597 from jitendrapurohit/CRM-20805

CRM-20805 : Fatal Error when adding new membership type on a contribu…

Merge pull request #10633 from colemanw/CRM-20842

CRM-20842 - Update api explorer page title

Merge pull request #10634 from JMAConsulting/CRM-20673

CRM-20673: maintain same tag order on both edit form and manage tag page

CRM-20673: maintain same tag order on both edit form and manage tag page

Merge pull request #10626 from totten/4.7.22-port

CRM-20829 Set port to null on default to stop failing requirements

Merge pull request #10631 from twomice/CRM-20509_contibution_report_note_is_null

CRM-20509: CiviReport: ACL limits in WHERE were too restrictive.

CRM-20842 - Update api explorer page title

Merge pull request #10632 from mickadoo/CRM-20833-update-api4-entity-namespace

CRM-20833: Change APIv4 Entity Namespace Again

CRM-20833: Change api4 entity namespace again

CRM-20509: CiviReport: ACL limits in WHERE were too restrictive.

Merge pull request #10588 from JMAConsulting/CRM-20793

[ready for core team review] CRM-20793, CRM-20799: Add filter - activity date and status on search criteria of activity listing AND make activity filter user preference an optional setting

Merge pull request #10068 from agh1/crm-20351

CRM-20351 Multiple pages repeat stuff in the page run() method, invoking hooks multiple times

CRM-20834 - Drupal user not created via Profile due to missing email field

Merge pull request #10622 from seamuslee001/smtp_socket_fix

Put back _ in socket and socket_options to refer correctly to staic v…

Put back _ in socket and socket_options to refer correctly to staic variables

Merge pull request #10625 from mickadoo/CRM-20833-update-api4-entity-namespace

CRM-20833: Change APIv4 Entity Namespace

Merge remote-tracking branch 'origin/4.7.22-rc'

CRM-20829 Uppercase NULL for style

CRM-20829 Set port to null on default to avoid failing requirements

Merge pull request #10624 from totten/master-4723

Rename v4.7.22 to v4.7.23

CRM-20833: Change api4 entity namespace

Rename v4.7.22 to v4.7.23 (continued)

Rename v4.7.22 to v4.7.23

There seems to be a consensus for issuing a quick patch to `4.7.21` as `4.7.22`. This means that `master` needs
to be renumbered as `4.7.23`.

Set version to 4.7.22

fix test failures

CRM-20351 ACL: Don't repeat stuff in the page run() method

additional fixes on QA

CRM-20652: Generalise changeFeeSelection function to be used for any entity other

CRM-20321: Changing membership type should change related contribution

Merge pull request #10621 from agh1/fix-bgm

Release notes: Mathieu primary affiliation still Coop SymbioTIC

Release notes: Mathieu primary affiliation still Coop SymbioTIC

Merge pull request #10619 from francescbassas/patch-11

Misspelling

Merge pull request #10613 from jitendrapurohit/CRM-20823

CRM-20823 : Price Set field with an Expiry Date still being required …

Merge pull request #10535 from jitendrapurohit/CRM-20752

CRM-20752: Editing a Cancelled Pledge updates the status of it to Pen…

Merge pull request #10565 from eileenmcnaughton/extract

CRM-20759, add ability to import to 'Primary' address add unit testing, move deprecated function into calling class

Misspelling

Merge pull request #10609 from totten/master-case-settings

CRM-20816 - Expose CiviCase settings through "Settings" framework

CRM-20816 - Rename civicaseAllowMultipleCaseClients to civicaseAllowMultipleClients

The second instance of the word "case" seems redundant.  Nip it in the bud.

CRM-20816 - XMLProcessor - Consolidate implementation of setting-lookup

CRM-20816 - Rename `auto` to `default`

Merge pull request #10616 from civicrm/4.7.21-rc

4.7.21 rc

Merge pull request #10551 from jitendrapurohit/CRM-20633

CRM-20633 : custom field set displayed twice on register event Partic…

Merge pull request #10578 from eileenmcnaughton/deprecated

CRM-20786 Move deprecated utils functions to the import classes

Merge pull request #10580 from eileenmcnaughton/deprecated3

CRM-20786 stop calling deprecated function, that does not do anything…

CRM-20828 Remove limit 1 to saved mappings selection

Merge pull request #10607 from JMAConsulting/CRM-20773

CRM-20773: Contribution tab shows Receive Date twice instead of ThankYou date

CRM-20823 : Price Set field with an Expiry Date still being required after being expired

Merge pull request #10587 from JMAConsulting/isPayment

CRM-20797, fixed sql to use is_payment = 1 to get payments

Merge pull request #10611 from totten/4.7.21-rc-notes

Release notes - Multiple updates for v4.7.21

release-notes/4.7.21 - Fix references to subordinate PRs

release-notes/4.7.21 - Add section for security advisories

Merge pull request #10610 from agh1/4.7.21-releasenotes

4.7.21 release notes

4.7.21 release notes: updated contributors

4.7.21 release notes: bulk of notes

raw from script

CRM-20816 - Add more help text

CRM-20816 - CiviCase `Settings.xml` - Add deprecation notices

CRM-20816 - CiviCase - Settings framework takes precedence over "Settings.xml"

CiviCase was originally developed before the creation of the "Settings"
framework (aka `$civicrm_settings`, `Settings API`, `civicrm_settings`,
`*.setting.php`).  At the time, it stored some settings in a special file
named "Settings.xml".

Today, this file is an anomaly which supports fewer dataflows.

With this revision:
 * For backward compatibility, the default setting is `auto`, which reads
   the setting from XML.
 * For maximum compliance with the Settings framework, the
   value from `Civi::settings()->get()` takes precedence (if defined).

CRM-20816 - Add `civicrm/admin/setting/case` to navbar

CRM-20816 - Implement new screen `civicrm/admin/setting/case`

CRM-20816 - Case.setting.php - Define settings for CiviCase

CRM-20759 enable Primary as an import location type.

This goes to the default location type if the contact has no primary, or overwrites primary,
in keeping with profile behaviour

Merge pull request #10606 from jitendrapurohit/CRM-20812

CRM-20812 : Notice on live contribution when only pay later is selected

Merge pull request #10603 from JMAConsulting/CRM-20640

CRM-20640: Added unit test for the fix

CRM-20773: Contribution tab shows Receive Date twice instead of Thank You date

Merge pull request #10594 from eileenmcnaughton/extract2

[NFC] Add unit test towards CRM-20759

CRM-20812 : Notice on live contribution when only pay later is selected

CRM-20809 : Create .gitignore file to ignore extensions cache files

CRM-20640: Added unit test for the fix