Remove unnecessary code
authorEileen McNaughton <emcnaughton@wikimedia.org>
Sat, 23 Oct 2021 02:47:17 +0000 (15:47 +1300)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Fri, 29 Oct 2021 06:14:47 +0000 (19:14 +1300)
commit3e3079caa25dad7ebd467786a6a4e829c73cf1d6
tree78b4ebf8aa3c979838ac4a28c81d1fdf55706a4d
parent88d97ccfb5c3eedf7ad5a320efbac9d7f88b249d
Remove unnecessary code

This function is called from 2 places

Contact::create
Contact::updateGreetingsOnTokenFieldChange

In both cases the contact has already been saved (to the extent that is relevant
for the operation). The latter operation will not change
the selected options - it will only potentially change tokens within it. In the
former case the nulling will already have taken place if needed ie if there
is no id entity for the relevant greeing
ensureGreetingParamsAreSet will already have set it to 'null'
before the save - leading to it already being updated to NULL
CRM/Contact/BAO/Contact.php