CRM-18555: API utils: avoid unset() on empty array.

diff --git a/api/v3/utils.php b/api/v3/utils.php
index 916eae4c046079b0cef7dd93f00490181693e755..cb1709391fd97340a61a44fdf0c54b3cc875caf8 100644 (file)
--- a/api/v3/utils.php
+++ b/api/v3/utils.php
@@ -134,12 +134,14 @@ function civicrm_api3_create_error($msg, $data = array()) {
 
   // we will show sql to privileged user only (not sure of a specific
   // security hole here but seems sensible - perhaps should apply to the trace as well?)
-  if (isset($data['sql']) && (CRM_Core_Permission::check('Administer CiviCRM') || CIVICRM_UF == 'UnitTests')) {
-    // Isn't this redundant?
-    $data['debug_information'] = $data['sql'];
-  }
-  else {
-    unset($data['sql']);
+  if (isset($data['sql'])) {
+    if (CRM_Core_Permission::check('Administer CiviCRM') || CIVICRM_UF == 'UnitTests') {
+      // Isn't this redundant?
+      $data['debug_information'] = $data['sql'];
+    }
+    else {
+      unset($data['sql']);
+    }
   }
   return $data;
 }