CRM-17789 - api/v3/ReportTemplate - Indicate type correctly
authorTim Otten <totten@civicrm.org>
Wed, 14 Sep 2016 20:52:46 +0000 (13:52 -0700)
committerTim Otten <totten@civicrm.org>
Wed, 14 Sep 2016 22:28:41 +0000 (15:28 -0700)
The return value of `civicrm_api*()` is generally `array`, but when used
with `getvalue` it can be scalar (specifically, `string` in this case).
This produces a type-warning in phan because it cannot tell that the true
return type is `string`.

api/v3/ReportTemplate.php

index e8e98580e7c545d6acfdcaac536491fe35bdb050..e699044778233fa21234e4fc36788b0acc0fa6bb 100644 (file)
@@ -130,7 +130,7 @@ function _civicrm_api3_report_template_getrows($params) {
     $params['report_id'] = civicrm_api3('report_instance', 'getvalue', array('id' => $params['instance_id'], 'return' => 'report_id'));
   }
 
-  $class = civicrm_api3('option_value', 'getvalue', array(
+  $class = (string) civicrm_api3('option_value', 'getvalue', array(
     'option_group_name' => 'report_template',
     'return' => 'name',
     'value' => $params['report_id'],