Remove unnecessary query when updating org, if no name udpate.
authoreileen <emcnaughton@wikimedia.org>
Wed, 11 Dec 2019 09:05:08 +0000 (22:05 +1300)
committereileen <emcnaughton@wikimedia.org>
Wed, 11 Dec 2019 23:02:00 +0000 (12:02 +1300)
commit91c00d11d405e665cf9e0fd4260b59b1a142b220
treecd04753ee4363ec1b9f4f7267b2deeceae385133
parent005b96f9e11a8e50aaa6ade064aadcba16c3c175
Remove unnecessary query when updating org, if no name udpate.

This removes an unnecessary query that can be locking at high volume.

The updateCurrentEmployer function sets the organization_name for any related contacts. We don't need to
do this if we are creating the organization for the first time, or updating the organization but not setting (passing in)
organization_name. We can check & avoid in php.

Also we don't need to check for an existing employer_id unless we are dealing with an Individual record. We can
skip that query for Orgs & Households.

These are cheap queries but at high volume can contribute to locks
CRM/Contact/BAO/Contact.php
CRM/Contact/BAO/Contact/Utils.php