Prevent intermittant fail on pcp api calls.
authoreileen <emcnaughton@wikimedia.org>
Mon, 29 Jan 2018 21:53:13 +0000 (10:53 +1300)
committereileen <emcnaughton@wikimedia.org>
Mon, 29 Jan 2018 22:41:28 +0000 (11:41 +1300)
When I run the SyntaxConformance tests locally I get a fatal error because it tries
to include api_v3_pcp.php after having included api_v3_PCP.php & thinks they are 2 files.
This is a Mac so obviously the handling is different to the jenkins method. Not overwriting the
 param if passed in works locally

api/v3/utils.php

index f6f73bc5aab604a62be7770d37f9561ac1d7d0e0..7640e9ae127743894555e35da2fd9b04f2bda977 100644 (file)
@@ -1235,7 +1235,7 @@ function formatCheckBoxField(&$checkboxFieldValue, $customFieldLabel, $entity) {
  * @return array
  */
 function _civicrm_api3_basic_get($bao_name, $params, $returnAsSuccess = TRUE, $entity = "", $sql = NULL, $uniqueFields = FALSE) {
-  $entity = CRM_Core_DAO_AllCoreTables::getBriefName(str_replace('_BAO_', '_DAO_', $bao_name));
+  $entity = $entity ?: CRM_Core_DAO_AllCoreTables::getBriefName(str_replace('_BAO_', '_DAO_', $bao_name));
   $options = _civicrm_api3_get_options_from_params($params);
 
   $query = new \Civi\API\Api3SelectQuery($entity, CRM_Utils_Array::value('check_permissions', $params, FALSE));