ParticipantStatusType - Switch to writeRecord

diff --git a/CRM/Admin/Form/ParticipantStatusType.php b/CRM/Admin/Form/ParticipantStatusType.php
index a3097c7de7e986c2f167ae68344941196bac5ef8..7eb952356895f5b04702a0da49567102b2387bfb 100644 (file)
--- a/CRM/Admin/Form/ParticipantStatusType.php
+++ b/CRM/Admin/Form/ParticipantStatusType.php
@@ -119,7 +119,7 @@ class CRM_Admin_Form_ParticipantStatusType extends CRM_Admin_Form {
     }
     $params['weight'] = CRM_Utils_Weight::updateOtherWeights('CRM_Event_DAO_ParticipantStatusType', $oldWeight, $params['weight']);
 
-    $participantStatus = CRM_Event_BAO_ParticipantStatusType::create($params);
+    $participantStatus = CRM_Event_BAO_ParticipantStatusType::writeRecord($params);
 
     if ($participantStatus->id) {
       if ($this->_action & CRM_Core_Action::UPDATE) {

diff --git a/CRM/Event/BAO/ParticipantStatusType.php b/CRM/Event/BAO/ParticipantStatusType.php
index 7176dcc48b9e5d38f2aca7dbcd4a9203cb3f36cb..7632f634e7071110b3204532b05c27f4df2a9b82 100644 (file)
--- a/CRM/Event/BAO/ParticipantStatusType.php
+++ b/CRM/Event/BAO/ParticipantStatusType.php
@@ -17,33 +17,23 @@
 class CRM_Event_BAO_ParticipantStatusType extends CRM_Event_DAO_ParticipantStatusType {
 
   /**
+   * @deprecated
    * @param array $params
    *
    * @return self|null
    */
   public static function add(&$params) {
-    if (empty($params)) {
-      return NULL;
-    }
-    $dao = new CRM_Event_DAO_ParticipantStatusType();
-    $dao->copyValues($params);
-    return $dao->save();
+    return self::writeRecord($params);
   }
 
   /**
+   * @deprecated
    * @param array $params
    *
    * @return self|null
    */
-  public static function &create(&$params) {
-    $transaction = new CRM_Core_Transaction();
-    $statusType = self::add($params);
-    if (is_a($statusType, 'CRM_Core_Error')) {
-      $transaction->rollback();
-      return $statusType;
-    }
-    $transaction->commit();
-    return $statusType;
+  public static function create(&$params) {
+    return self::writeRecord($params);
   }
 
   /**

diff --git a/tests/phpunit/CRM/Event/BAO/ParticipantStatusTest.php b/tests/phpunit/CRM/Event/BAO/ParticipantStatusTest.php
index 190232af432835fcee5e5dd20dda3e984a5d6eb3..01dedbfc576dfbd522a0ec1f0b94dfa7ca03c003 100644 (file)
--- a/tests/phpunit/CRM/Event/BAO/ParticipantStatusTest.php
+++ b/tests/phpunit/CRM/Event/BAO/ParticipantStatusTest.php
@@ -31,7 +31,7 @@ class CRM_Event_BAO_ParticipantStatusTest extends CiviUnitTestCase {
       'visibility_id' => 1,
     ];
 
-    $statusType = CRM_Event_BAO_ParticipantStatusType::create($params);
+    $statusType = CRM_Event_BAO_ParticipantStatusType::writeRecord($params);
     // Checking for participant status type id in db.
     $statusTypeId = $this->assertDBNotNull('CRM_Event_DAO_ParticipantStatusType', $statusType->id, 'id',
       'id', 'Check DB for status type id'
@@ -60,7 +60,7 @@ class CRM_Event_BAO_ParticipantStatusTest extends CiviUnitTestCase {
     ];
 
     // check for add participant status type
-    $statusType = CRM_Event_BAO_ParticipantStatusType::add($params);
+    $statusType = CRM_Event_BAO_ParticipantStatusType::writeRecord($params);
     foreach ($params as $param => $value) {
       $this->assertEquals($value, $statusType->$param);
     }
@@ -77,7 +77,7 @@ class CRM_Event_BAO_ParticipantStatusTest extends CiviUnitTestCase {
     ];
 
     // check for add participant status type
-    $statusType = CRM_Event_BAO_ParticipantStatusType::add($params);
+    $statusType = CRM_Event_BAO_ParticipantStatusType::writeRecord($params);
     foreach ($params as $param => $value) {
       $this->assertEquals($value, $statusType->$param);
     }
@@ -98,7 +98,7 @@ class CRM_Event_BAO_ParticipantStatusTest extends CiviUnitTestCase {
       'visibility_id' => 1,
     ];
 
-    $statusType = CRM_Event_BAO_ParticipantStatusType::create($params);
+    $statusType = CRM_Event_BAO_ParticipantStatusType::writeRecord($params);
 
     // retrieve status type
     $retrieveParams = ['id' => $statusType->id];