Remove some calls to OptionValue deleteRecord
authorEileen McNaughton <emcnaughton@wikimedia.org>
Wed, 1 Mar 2023 19:51:53 +0000 (08:51 +1300)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Wed, 1 Mar 2023 19:51:53 +0000 (08:51 +1300)
CRM/Admin/Form/OptionGroup.php
CRM/Admin/Form/Options.php
CRM/Campaign/Form/Survey/Main.php
CRM/Campaign/Form/SurveyType.php
CRM/Core/BAO/CustomField.php
api/v3/OptionValue.php

index 6de152bb9e83a7ad7bf513f3bcf790d8e2f0dfdf..fbf87dfb07cc9b95f472f1e2def8b03221c39ddd 100644 (file)
@@ -120,7 +120,7 @@ class CRM_Admin_Form_OptionGroup extends CRM_Admin_Form {
     CRM_Utils_System::flushCache();
 
     if ($this->_action & CRM_Core_Action::DELETE) {
-      CRM_Core_BAO_OptionGroup::del($this->_id);
+      CRM_Core_BAO_OptionGroup::deleteRecord(['id' => $this->_id]);
       CRM_Core_Session::setStatus(ts('Selected option group has been deleted.'), ts('Record Deleted'), 'success');
     }
     else {
index 5a36fc49ba5190fe926a2dcef153518604d2b6ed..d63a963897b8b77b3ffd83813651cd792fdd62a3 100644 (file)
@@ -459,7 +459,7 @@ class CRM_Admin_Form_Options extends CRM_Admin_Form {
       $fieldValues = ['option_group_id' => $this->_gid];
       CRM_Utils_Weight::delWeight('CRM_Core_DAO_OptionValue', $this->_id, $fieldValues);
 
-      if (CRM_Core_BAO_OptionValue::del($this->_id)) {
+      if (CRM_Core_BAO_OptionValue::deleteRecord(['id' => $this->_id])) {
         if ($this->_gName == 'phone_type') {
           CRM_Core_BAO_Phone::setOptionToNull(CRM_Utils_Array::value('value', $this->_defaultValues));
         }
index 44b760763e68aaaf691d87fb3618ff8f0468b8cb..72fc38c7c1616ba9dfcf71a4077660a842097e2c 100644 (file)
@@ -187,7 +187,7 @@ class CRM_Campaign_Form_Survey_Main extends CRM_Campaign_Form_Survey {
       );
       // delete option group if no any survey is using it.
       if (!$countSurvey) {
-        CRM_Core_BAO_OptionGroup::del($this->_values['result_id']);
+        CRM_Core_BAO_OptionGroup::deleteRecord(['id' => $this->_values['result_id']]);
       }
     }
 
index 713830405922e321700bf7f7847dc93215080162..02e8836ef5aa795c5b7c8438b5e6a3e66f788bc0 100644 (file)
@@ -122,7 +122,7 @@ class CRM_Campaign_Form_SurveyType extends CRM_Admin_Form {
       $fieldValues = ['option_group_id' => $this->_gid];
       $wt = CRM_Utils_Weight::delWeight('CRM_Core_DAO_OptionValue', $this->_id, $fieldValues);
 
-      if (CRM_Core_BAO_OptionValue::del($this->_id)) {
+      if (CRM_Core_BAO_OptionValue::deleteRecord(['id' => $this->_id])) {
         CRM_Core_Session::setStatus(ts('Selected Survey type has been deleted.'), ts('Record Deleted'), 'success');
       }
     }
index 05900d6caf0a532d6c5831443d64f7a8a72405ff..67cc1142d5a81c1f9f2cab3a8bda9ae6c717a8b5 100644 (file)
@@ -2485,7 +2485,7 @@ WHERE  option_group_id = {$optionGroupId}";
 
     if ($count < 2) {
       //delete the option group
-      CRM_Core_BAO_OptionGroup::del($optionGroupId);
+      CRM_Core_BAO_OptionGroup::deleteRecord(['id' => $optionGroupId]);
     }
   }
 
index 34a2b00507fb6e45f9c2feea6def655acb9e9ae3..3c09f146248db39264c4c7db426ba2a34a5fb8ae 100644 (file)
@@ -91,7 +91,7 @@ function _civicrm_api3_option_value_create_spec(&$params) {
 function civicrm_api3_option_value_delete($params) {
   // We will get the option group id before deleting so we can flush pseudoconstants.
   $optionGroupID = civicrm_api('option_value', 'getvalue', ['version' => 3, 'id' => $params['id'], 'return' => 'option_group_id']);
-  $result = CRM_Core_BAO_OptionValue::del($params['id']);
+  $result = CRM_Core_BAO_OptionValue::deleteRecord($params);
   if ($result) {
     civicrm_api('option_value', 'getfields', ['version' => 3, 'cache_clear' => 1, 'option_group_id' => $optionGroupID]);
     return civicrm_api3_create_success();