tonyair
/
silverstripe-reports
mirror of https://github.com/silverstripe/silverstripe-reports
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

ENH Replace E_USER_ERROR errors with exceptions

This commit is contained in:
Robbie Averill 2020-09-24 17:11:47 -07:00
parent 275a1bc7f4
commit 9b9e673e4b
2 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
code/Report.php
View File

@ -147,11 +147,13 @@ class Report extends ViewableData
*/ */
public function sourceQuery($params) public function sourceQuery($params)
{ {
if ($this->hasMethod('sourceRecords')) { if (!$this->hasMethod('sourceRecords')) {
return $this->sourceRecords($params, null, null)->dataQuery(); throw new \RuntimeException(
} else { 'Please override sourceQuery()/sourceRecords() and columns() or, if necessary, override getReportField()'
user_error("Please override sourceQuery()/sourceRecords() and columns() or, if necessary, override getReportField()", E_USER_ERROR); );
} }
return $this->sourceRecords($params, null, null)->dataQuery();
} }
/** /**

4
code/ReportWrapper.php
View File

@ -75,7 +75,9 @@ abstract class ReportWrapper extends Report
$this->afterQuery(); $this->afterQuery();
return $query; return $query;
} else { } else {
user_error("Please override sourceQuery()/sourceRecords() and columns() in your base report", E_USER_ERROR); throw new \RuntimeException(
"Please override sourceQuery()/sourceRecords() and columns() in your base report"
);
} }
} }