Remove unused and largely unreachable code.

The removed lines can only be reached if
a) CMS is Joomla or
b) CMS is Drupal and NULL is not equal to NULL

This seemed a little suspicious - especially since the merge screen still does display the value that should
be but isn't calculated here. After digging I found it is duplicated here
https://github.com/civicrm/civicrm-core/blob/bc77d7c0798081f999e6eca92d75fa874fd04e74/CRM/Contact/Form/Merge.php#L115
and that is the value used in the tpl here
https://github.com/civicrm/civicrm-core/blob/1188c7a8512382b1cb588b6a1ea4b13aa9587bc9/templates/CRM/Contact/Form/Merge.tpl#L217

so these lines can be safely removed as redundant. I also signposted where I think the next cleanups should happen on this
with code comments

Merge pull request #15778 from agh1/reminder-do-not-email-test

dev/core#1378 Scheduled reminders: do not email if do_not_email or on_hold

Merge pull request #16105 from seamuslee001/dev_core_522_204

dev/core#522 CRM-19767 dev/core#204 Add Case Tokens to emails

Merge pull request #15969 from eileenmcnaughton/utfmb8

Add utf8 to utfmb8 conversion api command

Merge pull request #16234 from demeritcowboy/more-t6

dev/core#495 Make admin settings field labels translatable

Merge pull request #16235 from demeritcowboy/more-t7

dev/core#495 Make admin settings field labels translatable

Merge pull request #16236 from demeritcowboy/more-t8

dev/core#495 Make admin settings field labels translatable

Merge pull request #16237 from demeritcowboy/more-t9

dev/core#495 Make admin settings field labels translatable

Merge pull request #16233 from demeritcowboy/more-t5

dev/core#495 Make admin settings field labels translatable

Merge pull request #16232 from demeritcowboy/more-t4

dev/core#495 Make admin settings field labels translatable

make translatable

make translatable

Merge pull request #16228 from demeritcowboy/more-t2

dev/core#495 Make admin settings field labels translatable

Merge pull request #16229 from demeritcowboy/more-t3

dev/core#495 Make admin settings field labels translatable

make translatable

make translatable

make translatable

make translatable

make translatable

make translatable

Merge pull request #16227 from demeritcowboy/more-t

dev/core#495 Make admin settings field labels translatable

make translatable

Merge pull request #16220 from colemanw/checkmark

Change check.gif to an actual gif (was a PNG)

Merge pull request #16222 from colemanw/importable

Refactor CRM_Contact_BAO_Contact::importableFields() to use metadata

Merge pull request #16226 from civicrm/5.22

5.22 to master

Merge pull request #16225 from demeritcowboy/contribution-view-missing-status

dev/core#1508 Regression/E_NOTICE - Missing table on contribution view

Merge pull request #16224 from eileenmcnaughton/set

dev/core#495 Add some ts tags on settings

put back mistakenly removed variable

dev/core#495 Add some ts tags on settings

Refactor CRM_Contact_BAO_Contact::importableFields() to use metadata

This squares up the recently added metadata with a hardcoded list in the BAO,
removing the hardcoded list in favor of metadata.

Merge pull request #15307 from seamuslee001/dev_core_1249

dev/core#1249 Allow IPNs to be validated even if the supplied contact…

Merge pull request #15670 from eileenmcnaughton/dupe_locks

dev/core#1355 Add contention handling for dedupe jobs

Merge pull request #15948 from eileenmcnaughton/export_test

Revised version of test from #15725

Merge pull request #16219 from seamuslee001/nfc_test_followup_16150

[NFC] Test update following PR #16150, assertEquals first param is th…

Change check.gif to an actual gif (was a PNG)

[NFC] Test update following PR #16150, assertEquals first param is the expected value not the current value

Merge pull request #16150 from jitendrapurohit/dev-1499

dev/core#1499 - Case Resource shows contact names that are not access…

Merge pull request #15855 from eileenmcnaughton/test_dep

Deprecate creating partially paid contributions, other than by partially paying a contribution.

Merge pull request #16218 from seamuslee001/nfc_add_more_asserts_change_fee_selection

[NFC] Add in more assertions around the content of line items and tot…

[NFC] Add in more assertions around the content of line items and total amount of the contribution record

Merge pull request #16198 from seamuslee001/parent_fetchall

[REF] Update fetchAll function signature to match parent function

Merge pull request #16213 from seamuslee001/error_handling_fix

Fix Error handling following DB Package upgrade

Merge pull request #16201 from seamuslee001/dev_core_1143_2

dev/core#1143 Support MySQL 8 by wrapping the word groups around in b…

Merge pull request #16202 from seamuslee001/mysql_8_information_schema

dev/core#392 Fix FTS INNDOB indexer query to work in MySQL 8 as well …

Merge pull request #16214 from seamuslee001/mysql_8_upgrade

dev/core#1143 Fix Upgrade tests on MySQL 8 by quoting reserved words

Merge pull request #16211 from seamuslee001/wordpress_2

dev/WordPress#2 Modify the WordPress plugin version using distmaker

Merge pull request #16197 from demeritcowboy/group-recent-items

dev/core#1506 Fix blank group title in recent items list when editing description inline

dev/core#1143 Fix Upgrade tests on MySQL 8 by quoting reserved words

Fix Error handling following DB Package upgrade

Merge pull request #16210 from mattwire/caseformactivity_simplifyif

REF Simplify if statement on case activity form

Modify the plugin version using distmaker

dev/core#1143 Support MySQL 8 by wrapping the word groups around in backticks as it is now a reserved word

Simplify if statement

Merge pull request #16203 from seamuslee001/dev_core_1143_translate

dev/core#1143 Ensure that columns that are reserved words e.g. groupi…

[REF] Update fetchAll function signature to match parent function

Merge pull request #16204 from seamuslee001/dev_core_392_logging

dev/core#392 Ensure that the log_date column on logging tables is NOT…

Merge pull request #16207 from seamuslee001/membership_392

dev/core#392 Fix handling of dates in getMembershipStarts function to…

Merge pull request #16206 from seamuslee001/dev_core_1143_export

dev/core#1143 Ensure that any field name when performaning an export …

dev/core#392 Fix handling of dates in getMembershipStarts function to ensure date is passed to query as yyyy-mm-dd to fix test failure on MySQL 8

Merge pull request #16199 from mattwire/copyright_headers

Update copyright headers following merge of #14662

Merge pull request #16196 from eileenmcnaughton/mem_add

[REF] Eliminate ['userID'] as an input for BAO_Membership::create

Merge pull request #16193 from seamuslee001/enable_backticks_dao

dev/core#1143 enable auto adding of backticks when doing an insert() or update() function

Merge pull request #16146 from eileenmcnaughton/index_api2

Update function to repair indexes to take table as a parameter

dev/core#392 Ensure that the log_date column on logging tables is NOT NULL in MySQL 8

dev/core#1143 Ensure that any field name when performaning an export is quoted so that if it contained a reserved word the export will still work in MySQL 8

dev/core#1143 Ensure that columns that are reserved words e.g. grouping are quoted for use in MySQL 8 when creating the view generation queries

dev/core#392 Fix FTS INNDOB indexer query to work in MySQL 8 as well as previous MySQL versions

Merge pull request #16185 from seamuslee001/dev_core_711

dev/core#711 Add in upgrade step for removing Google+ from website ty…

Merge pull request #16191 from seamuslee001/mem_custom_add

[REF] Move the storing of custom data into the add function rather th…

Update copyright headers following merge of #14662

Merge pull request #14662 from eileenmcnaughton/activity_pdf_71

Add PDF letter functionality for Activities using new token processor

fix blank group title in recent items list when editing description

Eliminate ['userID'] as an input for BAO_Membership::create

Merge pull request #16194 from civicrm/5.22

5.22

Merge pull request #16192 from eileenmcnaughton/mem_add

Remove  $ids as parameter for Membership::add

Merge pull request #16190 from alifrumin/rn5.22

5.22 Release Notes Annotating and Sorting

dev/core#711 Add in upgrade step for removing Google+ from website type option value

WIP enable auto adding of backticks

Remove  as paramter for Membership::add

The only place it's called with  is from create so we can handle userId there

[REF] Move the storing of custom data into the add function rather than in the create function so it is processed before the post hook

5.22 Release Notes Annotating and Sorting

Merge pull request #16166 from eileenmcnaughton/mod_member

Permit modified_id as a parameter for membership create api

Merge pull request #16183 from eileenmcnaughton/acl

Remove unindexed join from getACLRoles query

Merge pull request #16188 from civicrm/5.22

5.22

Merge pull request #16177 from eileenmcnaughton/sess

dev/drupal#98 Fix masquerade issue caused by drupal update change

Merge pull request #16187 from agh1/5.22.0-releasenotes

5.22.0 release notes - initial run

Merge pull request #16186 from mlutfy/fixSubscribeSpaces

Mailing Subscribe Form: remove nbsp from descriptions

5.22.0 release notes: added boilerplate

5.22.0 release notes: raw from script

Mailing Subscribe Form: remove nbsp from descriptions.

Merge pull request #16184 from seamuslee001/sl_update

Update my contributor details

Permit modified_id as a parameter for membership create api

Adding a membership create api parameter makes this accessible via the api and also allows us to deprecate & remove
the  array. Note this PR does nothing at the api level to make it available - it just takes advantage of apiv3 passing params
through to the BAO. The support is added in the BAO. However, I think we should add formal apiv4 support for modified_id
as a parameter for Membership.create - only issue is that of course it's not a 'real' param so we need to discuss /
understand how that needs to look. In general I think there are a few BAO that store modified_id

Merge pull request #16099 from agileware/CIVICRM-1243

dev/core#711 remove Google+

Update my contributor details

Set version to 5.23.alpha1

Set version to 5.22.beta1

Merge pull request #16182 from seamuslee001/dev_core_1144

dev/core#1144 (NFC) Handle fact that MySQL 8 always returns the colum…

Remove unindexed join from getACLRoles query

Replaces the indexed join with a  WHERE  on appropriate entity_id values.

I'm not a fan of CRM_Core_OptionGroup::values('acl_role') but it's what is used on the form that saves this.

The form adds in two pseudovalues but they should stay out to respect the INNER join

dev/core#1144 (NFC) Handle fact that MySQL 8 always returns the column name as it is in the DB not as it is in the query

Merge pull request #16180 from eileenmcnaughton/act2

[REF] remove unused  from getACLRoles