Merge pull request #5473 from aydun/CRM-16160
[civicrm-core.git] / api / v3 / Profile.php
index 7553d0765fc75a6d515dfc8a1e05bb4be440584e..23682854ed30d0f6eed2892390413fc1d33f0768 100644 (file)
@@ -1,10 +1,9 @@
 <?php
-
 /*
   +--------------------------------------------------------------------+
-  | CiviCRM version 4.5                                                |
+  | CiviCRM version 4.6                                                |
   +--------------------------------------------------------------------+
-  | Copyright CiviCRM LLC (c) 2004-2014                                |
+  | Copyright CiviCRM LLC (c) 2004-2015                                |
   +--------------------------------------------------------------------+
   | This file is a part of CiviCRM.                                    |
   |                                                                    |
   | 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 activity profile functions
+ * This api exposes CiviCRM profiles.
  *
- * @package CiviCRM_APIv3
- * @subpackage API_ActivityProfile
- * @copyright CiviCRM LLC (c) 2004-2014
- * @version $Id: ActivityProfile.php 30486 2011-05-20 16:12:09Z rajan $
+ * Profiles are collections of fields used as forms, listings, search columns, etc.
  *
+ * @package CiviCRM_APIv3
  */
 
 /**
  * Retrieve Profile field values.
  *
- * @param array $params       Associative array of property name/value
- *                             pairs to get profile field values
- *
- * @throws API_Exception
- * @return Profile field values|CRM_Error
- *
  * NOTE this api is not standard & since it is tested we need to honour that
  * but the correct behaviour is for it to return an id indexed array as this supports
  * multiple instances - if a  single profile is passed in we will not return a normal api result array
  * in order to avoid breaking code. (This could still be confusing :-( but we have to keep the tested behaviour working
  *
  * Note that if contact_id is empty an array of defaults is returned
+ *
+ * @param array $params
+ *   Associative array of property name/value.
+ *   pairs to get profile field values
+ *
+ * @throws API_Exception
+ * @return array
  */
 function civicrm_api3_profile_get($params) {
-  $nonStandardLegacyBehaviour = is_numeric($params['profile_id']) ?  TRUE : FALSE;
-  if(!empty($params['check_permissions']) && !empty($params['contact_id']) && !1 === civicrm_api3('contact', 'getcount', array('contact_id' => $params['contact_id'], 'check_permissions' => 1))) {
+  $nonStandardLegacyBehaviour = is_numeric($params['profile_id']) ? TRUE : FALSE;
+  if (!empty($params['check_permissions']) && !empty($params['contact_id']) && !1 === civicrm_api3('contact', 'getcount', array('contact_id' => $params['contact_id'], 'check_permissions' => 1))) {
     throw new API_Exception('permission denied');
   }
   $profiles = (array) $params['profile_id'];
@@ -67,7 +65,7 @@ function civicrm_api3_profile_get($params) {
       $values[$profileID] = _civicrm_api3_profile_getbillingpseudoprofile($params);
       continue;
     }
-    if(!CRM_Core_DAO::getFieldValue('CRM_Core_DAO_UFGroup', $profileID, 'is_active')) {
+    if (!CRM_Core_DAO::getFieldValue('CRM_Core_DAO_UFGroup', $profileID, 'is_active')) {
       throw new API_Exception('Invalid value for profile_id : ' . $profileID);
     }
 
@@ -85,50 +83,49 @@ function civicrm_api3_profile_get($params) {
       CRM_Core_Permission::EDIT
     );
 
+    if ($isContactActivityProfile) {
+      civicrm_api3_verify_mandatory($params, NULL, array('activity_id'));
 
-  if ($isContactActivityProfile) {
-    civicrm_api3_verify_mandatory($params, NULL, array('activity_id'));
-
-    $errors = CRM_Profile_Form::validateContactActivityProfile($params['activity_id'],
+      $errors = CRM_Profile_Form::validateContactActivityProfile($params['activity_id'],
       $params['contact_id'],
       $params['profile_id']
-    );
-    if (!empty($errors)) {
-      throw new API_Exception(array_pop($errors));
-    }
-
-    $contactFields = $activityFields = array();
-    foreach ($profileFields as $fieldName => $field) {
-      if (CRM_Utils_Array::value('field_type', $field) == 'Activity') {
-        $activityFields[$fieldName] = $field;
+      );
+      if (!empty($errors)) {
+        throw new API_Exception(array_pop($errors));
       }
-      else {
-        $contactFields[$fieldName] = $field;
+
+      $contactFields = $activityFields = array();
+      foreach ($profileFields as $fieldName => $field) {
+        if (CRM_Utils_Array::value('field_type', $field) == 'Activity') {
+          $activityFields[$fieldName] = $field;
+        }
+        else {
+          $contactFields[$fieldName] = $field;
           // we should return 'Primary' with & without capitalisation. it is more consistent with api to not
           // capitalise, but for form support we need it for now. Hopefully we can move away from it
           $contactFields[strtolower($fieldName)] = $field;
         }
       }
 
-    $ufGroupBAO->setProfileDefaults($params['contact_id'], $contactFields, $values[$profileID], TRUE);
+      $ufGroupBAO->setProfileDefaults($params['contact_id'], $contactFields, $values[$profileID], TRUE);
 
-    if ($params['activity_id']) {
-      $ufGroupBAO->setComponentDefaults($activityFields, $params['activity_id'], 'Activity', $values[$profileID], TRUE);
+      if ($params['activity_id']) {
+        $ufGroupBAO->setComponentDefaults($activityFields, $params['activity_id'], 'Activity', $values[$profileID], TRUE);
+      }
     }
-  }
-  elseif(!empty($params['contact_id'])) {
-    $ufGroupBAO->setProfileDefaults($params['contact_id'], $profileFields, $values[$profileID], TRUE);
-      foreach ($values[$profileID] as $fieldName => $field){
+    elseif (!empty($params['contact_id'])) {
+      $ufGroupBAO->setProfileDefaults($params['contact_id'], $profileFields, $values[$profileID], TRUE);
+      foreach ($values[$profileID] as $fieldName => $field) {
         // we should return 'Primary' with & without capitalisation. it is more consistent with api to not
         // capitalise, but for form support we need it for now. Hopefully we can move away from it
         $values[$profileID][strtolower($fieldName)] = $field;
       }
     }
-    else{
+    else {
       $values[$profileID] = array_fill_keys(array_keys($profileFields), '');
     }
   }
-  if($nonStandardLegacyBehaviour) {
+  if ($nonStandardLegacyBehaviour) {
     $result = civicrm_api3_create_success();
     $result['values'] = $values[$profileID];
     return $result;
@@ -139,7 +136,9 @@ function civicrm_api3_profile_get($params) {
 }
 
 /**
- * @param $params
+ * Adjust profile get function metadata.
+ *
+ * @param array $params
  */
 function _civicrm_api3_profile_get_spec(&$params) {
   $params['profile_id']['api.required'] = TRUE;
@@ -150,13 +149,15 @@ function _civicrm_api3_profile_get_spec(&$params) {
 
 /**
  * Submit a set of fields against a profile.
+ *
  * Note choice of submit versus create is discussed CRM-13234 & related to the fact
  * 'profile' is being treated as a data-entry entity
  *
  * @param array $params
  *
  * @throws API_Exception
- * @return array API result array
+ * @return array
+ *   API result array
  */
 function civicrm_api3_profile_submit($params) {
   $profileID = _civicrm_api3_profile_getProfileID($params['profile_id']);
@@ -173,7 +174,7 @@ function civicrm_api3_profile_submit($params) {
 
   $contactParams = $activityParams = $missingParams = array();
 
-  $profileFields = civicrm_api3('profile', 'getfields', array('action' => 'submit', 'profile_id' => $profileID));
+  $profileFields = civicrm_api3('Profile', 'getfields', array('action' => 'submit', 'profile_id' => $profileID));
   $profileFields = $profileFields['values'];
   if ($isContactActivityProfile) {
     civicrm_api3_verify_mandatory($params, NULL, array('activity_id'));
@@ -200,13 +201,13 @@ function civicrm_api3_profile_submit($params) {
     $locationEntities = array('email', 'address', 'phone', 'website', 'im');
 
     $entity = strtolower(CRM_Utils_Array::value('entity', $field));
-    if($entity && !in_array($entity, array_merge($contactEntities, $locationEntities))) {
+    if ($entity && !in_array($entity, array_merge($contactEntities, $locationEntities))) {
       $contactParams['api.' . $entity . '.create'][$fieldName] = $value;
       //@todo we are not currently declaring this option
-      if(isset($params['batch_id']) && strtolower($entity) == 'contribution') {
+      if (isset($params['batch_id']) && strtolower($entity) == 'contribution') {
         $contactParams['api.' . $entity . '.create']['batch_id'] = $params['batch_id'];
       }
-      if(isset($params[$entity . '_id'])) {
+      if (isset($params[$entity . '_id'])) {
         //todo possibly declare $entity_id in getfields ?
         $contactParams['api.' . $entity . '.create']['id'] = $params[$entity . '_id'];
       }
@@ -215,17 +216,17 @@ function civicrm_api3_profile_submit($params) {
       $contactParams[_civicrm_api3_profile_translate_fieldnames_for_bao($fieldName)] = $value;
     }
   }
-  if(isset($contactParams['api.contribution.create']) && isset($contactParams['api.membership.create'])) {
+  if (isset($contactParams['api.contribution.create']) && isset($contactParams['api.membership.create'])) {
     $contactParams['api.membership_payment.create'] = array(
       'contribution_id' => '$value.api.contribution.create.id',
-      'membership_id' => '$value.api.membership.create.id'
+      'membership_id' => '$value.api.membership.create.id',
     );
   }
 
-  if(isset($contactParams['api.contribution.create']) && isset($contactParams['api.participant.create'])) {
+  if (isset($contactParams['api.contribution.create']) && isset($contactParams['api.participant.create'])) {
     $contactParams['api.participant_payment.create'] = array(
       'contribution_id' => '$value.api.contribution.create.id',
-      'participant_id' => '$value.api.participant.create.id'
+      'participant_id' => '$value.api.participant.create.id',
     );
   }
 
@@ -244,69 +245,36 @@ function civicrm_api3_profile_submit($params) {
     $profileParams['api.activity.create'] = $activityParams;
   }
 
-  $groups = $tags = array();
-  if (isset($profileParams['group'])) {
-    $groups = $profileParams['group'];
-    unset($profileParams['group']);
-  }
-
-  if (isset($profileParams['tag'])) {
-    $tags = $profileParams['tag'];
-    unset($profileParams['tag']);
-  }
-
   return civicrm_api3('contact', 'create', $profileParams);
-
-  $ufGroupDetails = array();
-  $ufGroupParams = array('id' => $profileID);
-  CRM_Core_BAO_UFGroup::retrieve($ufGroupParams, $ufGroupDetails);
-
-  if (isset($profileFields['group'])) {
-    CRM_Contact_BAO_GroupContact::create($groups,
-      $params['contact_id'],
-      FALSE,
-      'Admin'
-    );
-  }
-
-  if (isset($profileFields['tag'])) {
-    CRM_Core_BAO_EntityTag::create($tags,
-      'civicrm_contact',
-      $params['contact_id']
-    );
-  }
-
-  if (!empty($ufGroupDetails['add_to_group_id'])) {
-    $contactIds = array($params['contact_id']);
-    CRM_Contact_BAO_GroupContact::addContactsToGroup($contactIds,
-      $ufGroupDetails['add_to_group_id']
-    );
-  }
-
-  return $result;
-
 }
 
 /**
+ * Translate field names for BAO.
+ *
  * The api standards expect field names to be lower case but the BAO uses mixed case
  * so we accept 'email-primary' but pass 'email-Primary' to the BAO
  * we could make the BAO handle email-primary but this would alter the fieldname seen by hooks
  * & we would need to consider that change
- * @param string $fieldName API field name
  *
- * @return string BAO Field Name
+ * @param string $fieldName
+ *   API field name.
+ *
+ * @return string
+ *   BAO Field Name
  */
-function _civicrm_api3_profile_translate_fieldnames_for_bao($fieldName){
+function _civicrm_api3_profile_translate_fieldnames_for_bao($fieldName) {
   $fieldName = str_replace('url', 'URL', $fieldName);
   return str_replace('primary', 'Primary', $fieldName);
 }
+
 /**
- * metadata for submit action
+ * Metadata for submit action.
+ *
  * @param array $params
  * @param array $apirequest
  */
 function _civicrm_api3_profile_submit_spec(&$params, $apirequest) {
-  if(isset($apirequest['params']['profile_id'])) {
+  if (isset($apirequest['params']['profile_id'])) {
     // we will return what is required for this profile
     // note the problem with simply over-riding getfields & then calling generic if needbe is we don't have the
     // api request array to pass to it.
@@ -314,48 +282,51 @@ function _civicrm_api3_profile_submit_spec(&$params, $apirequest) {
     //@todo get_options should take an array - @ the moment it is only takes 'all' - which is supported
     // by other getfields fn
     // we don't resolve state, country & county for performance reasons
-    $resolveOptions = CRM_Utils_Array::value('get_options',$apirequest['params']) == 'all' ? True : False;
+    $resolveOptions = CRM_Utils_Array::value('get_options', $apirequest['params']) == 'all' ? TRUE : FALSE;
     $profileID = _civicrm_api3_profile_getProfileID($apirequest['params']['profile_id']);
     $params = _civicrm_api3_buildprofile_submitfields($profileID, $resolveOptions, CRM_Utils_Array::value('cache_clear', $params));
   }
   elseif (isset($apirequest['params']['cache_clear'])) {
-    _civicrm_api3_buildprofile_submitfields(FALSE, FALSE, True);
+    _civicrm_api3_buildprofile_submitfields(FALSE, FALSE, TRUE);
   }
   $params['profile_id']['api.required'] = TRUE;
   $params['profile_id']['title'] = 'Profile ID';
 }
 
 /**
- * @deprecated - calling this function directly is deprecated as 'set' is not a clear action
- * use submit
  * Update Profile field values.
  *
- * @param array  $params       Associative array of property name/value
- *                             pairs to update profile field values
- *
- * @return array Updated Contact/ Activity object|CRM_Error
+ * @deprecated - calling this function directly is deprecated as 'set' is not a clear action
+ * use submit
  *
+ * @param array $params
+ *   Array of property name/value.
+ *   pairs to update profile field values
  *
+ * @return array
+ *   Updated Contact/ Activity object|CRM_Error
  */
 function civicrm_api3_profile_set($params) {
   return civicrm_api3('profile', 'submit', $params);
 }
 
 /**
+ * Apply profile.
+ *
  * @deprecated - appears to be an internal function - should not be accessible via api
  * Provide formatted values for profile fields.
  *
- * @param array $params       Associative array of property name/value
- *                             pairs to profile field values
+ * @param array $params
+ *   Array of property name/value.
+ *   pairs to profile field values
  *
  * @throws API_Exception
- * @return formatted profile field values|CRM_Error
+ * @return array
  *
  * @todo add example
  * @todo add test cases
  */
 function civicrm_api3_profile_apply($params) {
-
   $profileFields = CRM_Core_BAO_UFGroup::getFields($params['profile_id'],
     FALSE,
     NULL,
@@ -385,6 +356,8 @@ function civicrm_api3_profile_apply($params) {
 
 
 /**
+ * Get pseudo profile 'billing'.
+ *
  * This is a function to help us 'pretend' billing is a profile & treat it like it is one.
  * It gets standard credit card address fields etc
  * Note this is 'better' that the inbuilt version as it will pull in fallback values
@@ -393,18 +366,18 @@ function civicrm_api3_profile_apply($params) {
  *  Note that that since the existing code for deriving a blank profile is not easily accessible our
  *  interim solution is just to return an empty array
  *
- * @param $params
+ * @param array $params
  *
  * @return array
  */
 function _civicrm_api3_profile_getbillingpseudoprofile(&$params) {
 
-  $locations = civicrm_api3('address', 'getoptions', array('field' => 'location_type_id'));
+  $locations = civicrm_api3('address', 'getoptions', array('field' => 'location_type_id', 'context' => 'validate'));
   $locationTypeID = array_search('Billing', $locations['values']);
 
-  if(empty($params['contact_id'])) {
+  if (empty($params['contact_id'])) {
     $config = CRM_Core_Config::singleton();
-    $blanks =  array(
+    $blanks = array(
       'billing_first_name' => '',
       'billing_middle_name' => '',
       'billing_last_name' => '',
@@ -422,11 +395,11 @@ function _civicrm_api3_profile_getbillingpseudoprofile(&$params) {
   $addressFields = array('street_address', 'city', 'state_province_id', 'country_id', 'postal_code');
   $result = civicrm_api3('contact', 'getsingle', array(
     'id' => $params['contact_id'],
-    'api.address.get.1' => array('location_type_id' => 'Billing',  'return' => $addressFields),
+    'api.address.get.1' => array('location_type_id' => 'Billing', 'return' => $addressFields),
     // getting the is_billing required or not is an extra db call but probably cheap enough as this isn't an import api
-    'api.address.get.2' => array('is_billing' => True, 'return' => $addressFields),
-    'api.email.get.1' => array('location_type_id' => 'Billing',),
-    'api.email.get.2' => array('is_billing' => True,),
+    'api.address.get.2' => array('is_billing' => TRUE, 'return' => $addressFields),
+    'api.email.get.1' => array('location_type_id' => 'Billing'),
+    'api.email.get.2' => array('is_billing' => TRUE),
     'return' => 'api.email.get, api.address.get, api.address.getoptions, country, state_province, email, first_name, last_name, middle_name, ' . implode($addressFields, ','),
    )
   );
@@ -437,39 +410,40 @@ function _civicrm_api3_profile_getbillingpseudoprofile(&$params) {
     'billing_last_name' => $result['last_name'],
   );
 
-  if(!empty($result['api.address.get.1']['count'])) {
+  if (!empty($result['api.address.get.1']['count'])) {
     foreach ($addressFields as $fieldname) {
-      $values['billing_' . $fieldname . '-' . $locationTypeID] = isset($result['api.address.get.1']['values'][0][$fieldname])  ? $result['api.address.get.1']['values'][0][$fieldname] : '';
+      $values['billing_' . $fieldname . '-' . $locationTypeID] = isset($result['api.address.get.1']['values'][0][$fieldname]) ? $result['api.address.get.1']['values'][0][$fieldname] : '';
     }
   }
-  elseif(!empty($result['api.address.get.2']['count'])) {
+  elseif (!empty($result['api.address.get.2']['count'])) {
     foreach ($addressFields as $fieldname) {
-      $values['billing_' . $fieldname . '-' . $locationTypeID] = isset($result['api.address.get.2']['values'][0][$fieldname])  ? $result['api.address.get.2']['values'][0][$fieldname] : '';
+      $values['billing_' . $fieldname . '-' . $locationTypeID] = isset($result['api.address.get.2']['values'][0][$fieldname]) ? $result['api.address.get.2']['values'][0][$fieldname] : '';
     }
   }
-  else{
+  else {
     foreach ($addressFields as $fieldname) {
       $values['billing_' . $fieldname . '-' . $locationTypeID] = isset($result[$fieldname]) ? $result[$fieldname] : '';
     }
   }
 
-  if(!empty($result['api.email.get.1']['count'])) {
-    $values['billing-email'. '-' . $locationTypeID] = $result['api.email.get.1']['values'][0]['email'];
+  if (!empty($result['api.email.get.1']['count'])) {
+    $values['billing-email' . '-' . $locationTypeID] = $result['api.email.get.1']['values'][0]['email'];
   }
-  elseif(!empty($result['api.email.get.2']['count'])) {
-    $values['billing-email'. '-' . $locationTypeID] = $result['api.email.get.2']['values'][0]['email'];
+  elseif (!empty($result['api.email.get.2']['count'])) {
+    $values['billing-email' . '-' . $locationTypeID] = $result['api.email.get.2']['values'][0]['email'];
   }
-  else{
-    $values['billing-email'. '-' . $locationTypeID] = $result['email'];
+  else {
+    $values['billing-email' . '-' . $locationTypeID] = $result['email'];
   }
   // return both variants of email to reflect inconsistencies in form layer
-  $values['email'. '-' . $locationTypeID] = $values['billing-email'. '-' . $locationTypeID];
+  $values['email' . '-' . $locationTypeID] = $values['billing-email' . '-' . $locationTypeID];
   return $values;
 }
 
 /**
- * Here we will build  up getfields type data for all the fields in the profile. Because the integration with the
- * form layer in core is so hard-coded we are not going to attempt to re-use it
+ * Here we will build  up getfields type data for all the fields in the profile.
+ *
+ * Because the integration with the form layer in core is so hard-coded we are not going to attempt to re-use it
  * However, as this function is unit-tested & hence 'locked in' we can aspire to extract sharable
  * code out of the form-layer over time.
  *
@@ -478,35 +452,33 @@ function _civicrm_api3_profile_getbillingpseudoprofile(&$params) {
  * & participant has 'participant_status' so we have to standardise from the outside in here -
  * find the oddities, 'mask them' at this layer, add tests & work to standardise over time so we can remove this handling
  *
- * @param integer $profileID
- * @param integer $optionsBehaviour 0 = don't resolve, 1 = resolve non-aggressively, 2 = resolve aggressively - ie include country & state
+ * @param int $profileID
+ * @param int $optionsBehaviour
+ *   0 = don't resolve, 1 = resolve non-aggressively, 2 = resolve aggressively - ie include country & state.
  * @param $is_flush
  *
- * @internal param $params
- *
- * @return
+ * @return array|void
  */
-
 function _civicrm_api3_buildprofile_submitfields($profileID, $optionsBehaviour = 1, $is_flush) {
   static $profileFields = array();
-  if($is_flush) {
+  if ($is_flush) {
     $profileFields = array();
-    if(empty($profileID)) {
-      return;
+    if (empty($profileID)) {
+      return NULL;
     }
   }
-  if(isset($profileFields[$profileID])) {
+  if (isset($profileFields[$profileID])) {
     return $profileFields[$profileID];
   }
   $fields = civicrm_api3('uf_field', 'get', array('uf_group_id' => $profileID));
   $entities = array();
   foreach ($fields['values'] as $field) {
-    if(!$field['is_active']) {
+    if (!$field['is_active']) {
       continue;
     }
     list($entity, $fieldName) = _civicrm_api3_map_profile_fields_to_entity($field);
     $aliasArray = array();
-    if(strtolower($fieldName) != $fieldName) {
+    if (strtolower($fieldName) != $fieldName) {
       $aliasArray['api.aliases'] = array($fieldName);
       $fieldName = strtolower($fieldName);
     }
@@ -520,7 +492,7 @@ function _civicrm_api3_buildprofile_submitfields($profileID, $optionsBehaviour =
     ), $aliasArray);
 
     $ufFieldTaleFieldName = $field['field_name'];
-    if(isset($entity[$ufFieldTaleFieldName]['name'])) {
+    if (isset($entity[$ufFieldTaleFieldName]['name'])) {
       // in the case where we are dealing with an alias we map back to a name
       // this will be tested by 'membership_type_id' field
       $ufFieldTaleFieldName = $entity[$ufFieldTaleFieldName]['name'];
@@ -540,7 +512,7 @@ function _civicrm_api3_buildprofile_submitfields($profileID, $optionsBehaviour =
       'soft_credit_type' => 'soft_credit_type_id',
     );
 
-    if(array_key_exists($ufFieldTaleFieldName, $hardCodedEntityFields)) {
+    if (array_key_exists($ufFieldTaleFieldName, $hardCodedEntityFields)) {
       $ufFieldTaleFieldName = $hardCodedEntityFields[$ufFieldTaleFieldName];
     }
 
@@ -552,49 +524,49 @@ function _civicrm_api3_buildprofile_submitfields($profileID, $optionsBehaviour =
     $entityGetFieldsResult = _civicrm_api3_profile_appendaliases($result['values'], $entity);
     foreach ($entityFields as $entityfield => $realName) {
       $fieldName = strtolower($entityfield);
-      if(!strstr($fieldName, '-')) {
-         if(strtolower($realName) != $fieldName) {
-        // we want to keep the '-' pattern for locations but otherwise
-        // we are going to make the api-standard field the main / preferred name but support the db name
-        // in future naming the fields in the DB to reflect the way the rest of the api / BAO / metadata works would
-        // reduce code
-        $fieldName = strtolower($realName);
+      if (!strstr($fieldName, '-')) {
+        if (strtolower($realName) != $fieldName) {
+          // we want to keep the '-' pattern for locations but otherwise
+          // we are going to make the api-standard field the main / preferred name but support the db name
+          // in future naming the fields in the DB to reflect the way the rest of the api / BAO / metadata works would
+          // reduce code
+          $fieldName = strtolower($realName);
         }
-        if(isset($entityGetFieldsResult[$realName]['uniqueName'])) {
+        if (isset($entityGetFieldsResult[$realName]['uniqueName'])) {
           // we won't alias the field name on here are we are using uniqueNames for the possibility of needing to differentiate
           // which entity 'status_id' belongs to
           $fieldName = $entityGetFieldsResult[$realName]['uniqueName'];
         }
-        else{
-          if(isset($entityGetFieldsResult[$realName]['name'])) {
+        else {
+          if (isset($entityGetFieldsResult[$realName]['name'])) {
             // this will sort out membership_type_id vs membership_type
             $fieldName = $entityGetFieldsResult[$realName]['name'];
           }
         }
       }
       $profileFields[$profileID][$fieldName] = array_merge($entityGetFieldsResult[$realName], $profileFields[$profileID][$entityfield]);
-      if(!isset($profileFields[$profileID][$fieldName]['api.aliases'])) {
-       $profileFields[$profileID][$fieldName]['api.aliases'] = array();
+      if (!isset($profileFields[$profileID][$fieldName]['api.aliases'])) {
+        $profileFields[$profileID][$fieldName]['api.aliases'] = array();
       }
-      if($optionsBehaviour && !empty($entityGetFieldsResult[$realName]['pseudoconstant'])) {
-        if($optionsBehaviour > 1  || !in_array($realName, array('state_province_id', 'county_id', 'country_id'))) {
+      if ($optionsBehaviour && !empty($entityGetFieldsResult[$realName]['pseudoconstant'])) {
+        if ($optionsBehaviour > 1  || !in_array($realName, array('state_province_id', 'county_id', 'country_id'))) {
           $options = civicrm_api3($entity, 'getoptions', array('field' => $realName));
           $profileFields[$profileID][$fieldName]['options'] = $options['values'];
         }
       }
 
-      if($entityfield != $fieldName) {
-        if(isset($profileFields[$profileID][$entityfield])) {
+      if ($entityfield != $fieldName) {
+        if (isset($profileFields[$profileID][$entityfield])) {
           unset($profileFields[$profileID][$entityfield]);
         }
-        if(!in_array($entityfield, $profileFields[$profileID][$fieldName]['api.aliases'])) {
+        if (!in_array($entityfield, $profileFields[$profileID][$fieldName]['api.aliases'])) {
           // we will make the mixed case version (e.g. of 'Primary') an alias
           $profileFields[$profileID][$fieldName]['api.aliases'][] = $entityfield;
         }
       }
       /**
        * putting this on hold -this would cause the api to set the default - but could have unexpected behaviour
-      if(isset($result['values'][$realName]['default_value'])) {
+      if (isset($result['values'][$realName]['default_value'])) {
       //this would be the case for a custom field with a configured default
       $profileFields[$profileID][$entityfield]['api.default'] = $result['values'][$realName]['default_value'];
       }
@@ -626,26 +598,26 @@ function _civicrm_api3_order_by_weight($a, $b) {
 function _civicrm_api3_map_profile_fields_to_entity(&$field) {
   $entity = $field['field_type'];
   $contactTypes = civicrm_api3('contact', 'getoptions', array('field' => 'contact_type'));
-  if(in_array($entity, $contactTypes['values'])) {
+  if (in_array($entity, $contactTypes['values'])) {
     $entity = 'contact';
   }
   $entity = _civicrm_api_get_entity_name_from_camel($entity);
   $locationFields = array('email' => 'email');
   $fieldName = $field['field_name'];
-  if(!empty($field['location_type_id'])) {
-    if($fieldName == 'email') {
+  if (!empty($field['location_type_id'])) {
+    if ($fieldName == 'email') {
       $entity = 'email';
     }
-    else{
+    else {
       $entity = 'address';
     }
     $fieldName .= '-' . $field['location_type_id'];
   }
-  elseif(array_key_exists($fieldName, $locationFields)) {
+  elseif (array_key_exists($fieldName, $locationFields)) {
     $fieldName .= '-Primary';
     $entity = 'email';
   }
-  if(!empty($field['phone_type_id'])) {
+  if (!empty($field['phone_type_id'])) {
     $fieldName .= '-' . $field['location_type_id'];
     $entity = 'phone';
   }
@@ -680,8 +652,8 @@ function _civicrm_api3_map_profile_fields_to_entity(&$field) {
     'soft_credit_type' => 'contribution_soft',
     'group' => 'group_contact',
     'tag' => 'entity_tag',
-   );
-  if(array_key_exists($fieldName, $hardCodedEntityMappings)) {
+  );
+  if (array_key_exists($fieldName, $hardCodedEntityMappings)) {
     $entity = $hardCodedEntityMappings[$fieldName];
   }
   return array($entity, $fieldName);
@@ -691,13 +663,13 @@ function _civicrm_api3_map_profile_fields_to_entity(&$field) {
  * @todo this should be handled by the api wrapper using getfields info - need to check
  * how we add a a pseudoconstant to this pseudo api to make that work
  *
- * @param $profileID
+ * @param int $profileID
  *
- * @return integer|string
+ * @return int|string
  * @throws CiviCRM_API3_Exception
  */
 function _civicrm_api3_profile_getProfileID($profileID) {
-  if(!empty($profileID) && strtolower($profileID) != 'billing' && !is_numeric($profileID)) {
+  if (!empty($profileID) && strtolower($profileID) != 'billing' && !is_numeric($profileID)) {
     $profileID = civicrm_api3('uf_group', 'getvalue', array('return' => 'id', 'name' => $profileID));
   }
   return $profileID;
@@ -717,17 +689,17 @@ function _civicrm_api3_profile_getProfileID($profileID) {
  */
 function _civicrm_api3_profile_appendaliases($values, $entity) {
   foreach ($values as $field => $spec) {
-    if(!empty($spec['api.aliases'])) {
+    if (!empty($spec['api.aliases'])) {
       foreach ($spec['api.aliases'] as $alias) {
         $values[$alias] = $spec;
       }
     }
-    if(!empty($spec['uniqueName'])) {
+    if (!empty($spec['uniqueName'])) {
       $values[$spec['uniqueName']] = $spec;
     }
   }
   //special case on membership & contribution - can't see how to handle in a generic way
-  if(in_array($entity, array('membership', 'contribution'))) {
+  if (in_array($entity, array('membership', 'contribution'))) {
     $values['send_receipt'] = array('title' => 'Send Receipt', 'type' => (int) 16);
   }
   return $values;
@@ -735,7 +707,8 @@ function _civicrm_api3_profile_appendaliases($values, $entity) {
 
 /**
  * @deprecated api notice
- * @return array of deprecated actions
+ * @return array
+ *   Array of deprecated actions
  */
 function _civicrm_api3_profile_deprecation() {
   return array(