Merge pull request #5044 from colemanw/Comment

[civicrm-core.git] / api / v3 / MembershipType.php

diff --git a/api/v3/MembershipType.php b/api/v3/MembershipType.php
index 0287161634dd87f4d5b54ef4090ae18740e3a71a..1fe6772f5690bf29a4c905a84588653d741ce4c0 100644 (file)
--- a/api/v3/MembershipType.php
+++ b/api/v3/MembershipType.php
@@ -1,5 +1,4 @@
 <?php
-
 /*
  +--------------------------------------------------------------------+
  | CiviCRM version 4.6                                                |
@@ -24,7 +23,7 @@
  | GNU Affero General Public License or the licensing of CiviCRM,     |
  | see the CiviCRM license FAQ at http://civicrm.org/licensing        |
  +--------------------------------------------------------------------+
-*/
+ */
 
 /**
  * File for the CiviCRM APIv3 membership type functions
@@ -43,22 +42,11 @@
  * @param array $params
  *   An associative array of name/value property values of civicrm_membership_type.
  *
- * @return array $result newly created or updated membership type property values.
- * @access public
- * {getfields MembershipType_get}
+ * @return array
+ *   newly created or updated membership type property values.
  */
 function civicrm_api3_membership_type_create($params) {
-  $ids['membershipType'] = CRM_Utils_Array::value('id', $params);
-  $ids['memberOfContact'] = CRM_Utils_Array::value('member_of_contact_id', $params);
-  $ids['contributionType'] = CRM_Utils_Array::value('financial_type_id', $params);
-
-  $membershipTypeBAO = CRM_Member_BAO_MembershipType::add($params, $ids);
-  $membershipType = array();
-  _civicrm_api3_object_to_array($membershipTypeBAO, $membershipType[$membershipTypeBAO->id]);
-  CRM_Member_PseudoConstant::membershipType(NULL, TRUE);
-  civicrm_api3('membership', 'getfields', array('cache_clear' => 1, 'fieldname' => 'membership_type_id'));
-  civicrm_api3('profile', 'getfields', array('action' => 'submit', 'cache_clear' => 1));
-  return civicrm_api3_create_success($membershipType, $params, 'membership_type', 'create', $membershipTypeBAO);
+  return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params, 'Membership_type');
 }
 
 /**
@@ -87,8 +75,8 @@ function _civicrm_api3_membership_type_create_spec(&$params) {
  *   An associative array of name/value property values of civicrm_membership_type.
  * {getfields MembershipType_get}
  *
- * @return Array of all found membership type property values.
- * @access public
+ * @return array
+ *   Array of all found membership type property values.
  */
 function civicrm_api3_membership_type_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -102,9 +90,8 @@ function civicrm_api3_membership_type_get($params) {
  *
  * @param array $params
  *
- * @return boolean        true if success, else false
- * @access public
- * {getfields MembershipType_delete}
+ * @return bool
+ *   true if success, else false
  */
 function civicrm_api3_membership_type_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);