Display malformed sql in test failures
authoreileenmcnaugton <eileen@fuzion.co.nz>
Mon, 29 Feb 2016 06:49:26 +0000 (19:49 +1300)
committereileenmcnaugton <eileen@fuzion.co.nz>
Mon, 29 Feb 2016 06:49:26 +0000 (19:49 +1300)
(yes, it might be the permission should be reviewed too as more nuanced options exist now but leaving that out of scope)

api/v3/utils.php

index 1d67d9120489a4a82753e6630724401db1e2f627..3cd75f7a66c8f870d7d9f000e4c29bcf52d1afd6 100644 (file)
@@ -131,9 +131,10 @@ function civicrm_api3_verify_mandatory($params, $daoName = NULL, $keys = array()
 function civicrm_api3_create_error($msg, $data = array()) {
   $data['is_error'] = 1;
   $data['error_message'] = $msg;
+
   // 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')) {
+  if (isset($data['sql']) && (CRM_Core_Permission::check('Administer CiviCRM') || CIVICRM_UF == 'UnitTests')) {
     // Isn't this redundant?
     $data['debug_information'] = $data['sql'];
   }