Merge pull request #10487 from mfb/CRM-20713

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

diff --git a/api/v3/UFField.php b/api/v3/UFField.php
index fbe76ff50b5554d37ff46e4d45467a3cc88354aa..5dba94ecee614144f91f3507e452c6ca709ed013 100644 (file)
--- a/api/v3/UFField.php
+++ b/api/v3/UFField.php
@@ -3,7 +3,7 @@
  +--------------------------------------------------------------------+
  | CiviCRM version 4.7                                                |
  +--------------------------------------------------------------------+
- | Copyright CiviCRM LLC (c) 2004-2016                                |
+ | Copyright CiviCRM LLC (c) 2004-2017                                |
  +--------------------------------------------------------------------+
  | This file is a part of CiviCRM.                                    |
  |                                                                    |
@@ -43,66 +43,7 @@
  *   Newly created $ufFieldArray
  */
 function civicrm_api3_uf_field_create($params) {
-  // CRM-14756: kind of a hack-ish fix. If the user gives the id, uf_group_id is retrieved and then set.
-  if (isset($params['id'])) {
-    $groupId = civicrm_api3('UFField', 'getvalue', array(
-      'return' => 'uf_group_id',
-      'id' => $params['id'],
-    ));
-  }
-  else {
-    $groupId = CRM_Utils_Array::value('uf_group_id', $params);
-  }
-
-  $field_type       = CRM_Utils_Array::value('field_type', $params);
-  $field_name       = CRM_Utils_Array::value('field_name', $params);
-  $location_type_id = CRM_Utils_Array::value('location_type_id', $params, CRM_Utils_Array::value('website_type_id', $params));
-  $phone_type       = CRM_Utils_Array::value('phone_type_id', $params, CRM_Utils_Array::value('phone_type', $params));
-
-  if (strpos($field_name, 'formatting') !== 0 && !CRM_Core_BAO_UFField::isValidFieldName($field_name)) {
-    throw new API_Exception('The field_name is not valid');
-  }
-  $params['field_name'] = array($field_type, $field_name, $location_type_id, $phone_type);
-
-  if (!(CRM_Utils_Array::value('group_id', $params))) {
-    $params['group_id'] = $groupId;
-  }
-
-  $ids = $ufFieldArray = array();
-  $ids['uf_group'] = $groupId;
-
-  $fieldId = CRM_Utils_Array::value('id', $params);
-  if (!empty($fieldId)) {
-    $UFField = new CRM_Core_BAO_UFField();
-    $UFField->id = $fieldId;
-    if ($UFField->find(TRUE)) {
-      $ids['uf_group'] = $UFField->uf_group_id;
-      if (!(CRM_Utils_Array::value('group_id', $params))) {
-        // this copied here from previous api function - not sure if required
-        $params['group_id'] = $UFField->uf_group_id;
-      }
-    }
-    else {
-      throw new API_Exception("there is no field for this fieldId");
-    }
-    $ids['uf_field'] = $fieldId;
-  }
-
-  if (CRM_Core_BAO_UFField::duplicateField($params, $ids)) {
-    throw new API_Exception("The field was not added. It already exists in this profile.");
-  }
-  //@todo why is this even optional? Surely weight should just be 'managed' ??
-  if (CRM_Utils_Array::value('option.autoweight', $params, TRUE)) {
-    $params['weight'] = CRM_Core_BAO_UFField::autoWeight($params);
-  }
-  $ufField = CRM_Core_BAO_UFField::add($params);
-
-  $fieldsType = CRM_Core_BAO_UFGroup::calculateGroupType($groupId, TRUE);
-  CRM_Core_BAO_UFGroup::updateGroupTypes($groupId, $fieldsType);
-
-  _civicrm_api3_object_to_array($ufField, $ufFieldArray[$ufField->id]);
-  civicrm_api3('profile', 'getfields', array('cache_clear' => TRUE));
-  return civicrm_api3_create_success($ufFieldArray, $params);
+  return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);
 }
 
 /**