From 99df8789dca6bb6c5cc30b19259526f3d76741f4 Mon Sep 17 00:00:00 2001 From: eileen Date: Fri, 10 Apr 2020 13:38:07 +1200 Subject: [PATCH] [NFC] Fix use of pattern-hated-by-Coleman Since I tried to use this as an example of a 'tidy' hook call I'd better fix it --- CRM/Core/BAO/Phone.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CRM/Core/BAO/Phone.php b/CRM/Core/BAO/Phone.php index 0a61703b0c..bf4dd2c043 100644 --- a/CRM/Core/BAO/Phone.php +++ b/CRM/Core/BAO/Phone.php @@ -58,7 +58,7 @@ class CRM_Core_BAO_Phone extends CRM_Core_DAO_Phone { CRM_Core_DAO::checkSqlFunctionsExist(); $hook = empty($params['id']) ? 'create' : 'edit'; - CRM_Utils_Hook::pre($hook, 'Phone', CRM_Utils_Array::value('id', $params), $params); + CRM_Utils_Hook::pre($hook, 'Phone', $params['id'] ?? NULL, $params); $phone = new CRM_Core_DAO_Phone(); $phone->copyValues($params); -- 2.25.1