From 5b335ad511a9cdef77c1ecb568d0cb92ba2dd2db Mon Sep 17 00:00:00 2001 From: Will Rossiter Date: Mon, 23 Oct 2017 22:40:53 +1300 Subject: [PATCH 1/4] NEW Add support for callable link formatting --- code/Report.php | 52 +++++++++++++++++++++++++++++++++---------------- 1 file changed, 35 insertions(+), 17 deletions(-) diff --git a/code/Report.php b/code/Report.php index 1e727d42..9f08ea8b 100644 --- a/code/Report.php +++ b/code/Report.php @@ -25,7 +25,8 @@ use SilverStripe\Security\Member; use SilverStripe\Security\Permission; use SilverStripe\Security\Security; use SilverStripe\View\ViewableData; - +use SilverStripe\Core\Injector\Injector; +use SilverStripe\Control\HTTPRequest; /** * Base "abstract" class creating reports on your data. * @@ -85,6 +86,8 @@ class Report extends ViewableData */ protected $sort = 0; + protected $params = []; + /** * Reports which should not be collected and returned in get_reports * @@ -97,6 +100,15 @@ class Report extends ViewableData SideReportWrapper::class, ]; + public function __construct() + { + if (Injector::inst()->has(HTTPRequest::class)) { + $this->params = Injector::inst()->get(HTTPRequest::class)->param('filters'); + } + + parent::__construct(); + } + /** * Return the title of this report. * @@ -162,6 +174,11 @@ class Report extends ViewableData } } + public function columns() + { + return []; + } + /** * Return the data class for this report */ @@ -324,9 +341,7 @@ class Report extends ViewableData */ public function getReportField() { - // TODO Remove coupling with global state - $params = isset($_REQUEST['filters']) ? $_REQUEST['filters'] : array(); - $items = $this->sourceRecords($params, null, null); + $items = $this->sourceRecords($this->params, null, null); $gridFieldConfig = GridFieldConfig::create()->addComponents( @@ -334,7 +349,6 @@ class Report extends ViewableData new GridFieldPrintButton('buttons-before-left'), new GridFieldExportButton('buttons-before-left'), new GridFieldToolbarHeader(), - new GridFieldSortableHeader(), new GridFieldDataColumns(), new GridFieldPaginator() ); @@ -361,18 +375,22 @@ class Report extends ViewableData } if (isset($info['link']) && $info['link']) { - $fieldFormatting[$source] = function ($value, $item) { - if ($item instanceof CMSPreviewable) { - /** @var CMSPreviewable $item */ - return sprintf( - '%s', - Convert::raw2att($item->CMSEditLink()), - Convert::raw2att($value), - Convert::raw2xml($value) - ); - } - return $value; - }; + if (is_callable($info['link'])) { + $fieldFormatting[$source] = $info['link']; + } else { + $fieldFormatting[$source] = function ($value, $item) { + if ($item instanceof CMSPreviewable) { + /** @var CMSPreviewable $item */ + return sprintf( + '%s', + Convert::raw2att($item->CMSEditLink()), + Convert::raw2att($value), + Convert::raw2xml($value) + ); + } + return $value; + }; + } } $displayFields[$source] = isset($info['title']) ? $info['title'] : $source; From 86f845972f926d09186eabcd39cdabfa672b9f35 Mon Sep 17 00:00:00 2001 From: Christopher Joe Date: Mon, 27 Nov 2017 14:29:19 +1300 Subject: [PATCH 2/4] BUG Actioned feedback --- code/Report.php | 30 +++++++++++++----------------- 1 file changed, 13 insertions(+), 17 deletions(-) diff --git a/code/Report.php b/code/Report.php index 9f08ea8b..0cdfe155 100644 --- a/code/Report.php +++ b/code/Report.php @@ -86,8 +86,6 @@ class Report extends ViewableData */ protected $sort = 0; - protected $params = []; - /** * Reports which should not be collected and returned in get_reports * @@ -100,15 +98,6 @@ class Report extends ViewableData SideReportWrapper::class, ]; - public function __construct() - { - if (Injector::inst()->has(HTTPRequest::class)) { - $this->params = Injector::inst()->get(HTTPRequest::class)->param('filters'); - } - - parent::__construct(); - } - /** * Return the title of this report. * @@ -341,7 +330,13 @@ class Report extends ViewableData */ public function getReportField() { - $items = $this->sourceRecords($this->params, null, null); + $params = []; + if (Injector::inst()->has(HTTPRequest::class)) { + /** @var HTTPRequest $request */ + $request = Injector::inst()->get(HTTPRequest::class); + $params = $request->param('filters') ?: []; + } + $items = $this->sourceRecords($params, null, null); $gridFieldConfig = GridFieldConfig::create()->addComponents( @@ -349,19 +344,20 @@ class Report extends ViewableData new GridFieldPrintButton('buttons-before-left'), new GridFieldExportButton('buttons-before-left'), new GridFieldToolbarHeader(), + new GridFieldSortableHeader(), new GridFieldDataColumns(), new GridFieldPaginator() ); $gridField = new GridField('Report', null, $items, $gridFieldConfig); - $columns = $gridField->getConfig()->getComponentByType('SilverStripe\\Forms\\GridField\\GridFieldDataColumns'); - $displayFields = array(); - $fieldCasting = array(); - $fieldFormatting = array(); + $columns = $gridField->getConfig()->getComponentByType(GridFieldDataColumns::class); + $displayFields = []; + $fieldCasting = []; + $fieldFormatting = []; // Parse the column information foreach ($this->columns() as $source => $info) { if (is_string($info)) { - $info = array('title' => $info); + $info = ['title' => $info]; } if (isset($info['formatting'])) { From b4a3529339345b88cf9db9503caef1b386709dc6 Mon Sep 17 00:00:00 2001 From: Robbie Averill Date: Thu, 14 Dec 2017 13:32:50 +1300 Subject: [PATCH 3/4] FIX Minor linting violation in Report --- code/Report.php | 1 + 1 file changed, 1 insertion(+) diff --git a/code/Report.php b/code/Report.php index 0cdfe155..d38c4540 100644 --- a/code/Report.php +++ b/code/Report.php @@ -27,6 +27,7 @@ use SilverStripe\Security\Security; use SilverStripe\View\ViewableData; use SilverStripe\Core\Injector\Injector; use SilverStripe\Control\HTTPRequest; + /** * Base "abstract" class creating reports on your data. * From b006a9968ad4d5f073fa1ef784a89ecab8113e50 Mon Sep 17 00:00:00 2001 From: Damian Mooyman Date: Thu, 14 Dec 2017 13:37:25 +1300 Subject: [PATCH 4/4] Linting, phpdoc and code style fixes --- code/Report.php | 25 ++++++++++++++++++++----- 1 file changed, 20 insertions(+), 5 deletions(-) diff --git a/code/Report.php b/code/Report.php index d38c4540..d57d58c3 100644 --- a/code/Report.php +++ b/code/Report.php @@ -3,11 +3,15 @@ namespace SilverStripe\Reports; use ReflectionClass; +use SilverStripe\CMS\Model\SiteTree; use SilverStripe\Control\Controller; +use SilverStripe\Control\HTTPRequest; use SilverStripe\Core\ClassInfo; use SilverStripe\Core\Convert; +use SilverStripe\Core\Injector\Injector; use SilverStripe\Forms\FieldList; use SilverStripe\Forms\FormAction; +use SilverStripe\Forms\FormField; use SilverStripe\Forms\GridField\GridField; use SilverStripe\Forms\GridField\GridFieldButtonRow; use SilverStripe\Forms\GridField\GridFieldConfig; @@ -20,13 +24,13 @@ use SilverStripe\Forms\GridField\GridFieldToolbarHeader; use SilverStripe\Forms\LiteralField; use SilverStripe\ORM\ArrayList; use SilverStripe\ORM\CMSPreviewable; +use SilverStripe\ORM\DataList; +use SilverStripe\ORM\DataQuery; use SilverStripe\ORM\SS_List; use SilverStripe\Security\Member; use SilverStripe\Security\Permission; use SilverStripe\Security\Security; use SilverStripe\View\ViewableData; -use SilverStripe\Core\Injector\Injector; -use SilverStripe\Control\HTTPRequest; /** * Base "abstract" class creating reports on your data. @@ -55,6 +59,8 @@ use SilverStripe\Control\HTTPRequest; * * Right now, all subclasses of SS_Report will be shown in the ReportAdmin. In SS3 there is only * one place where reports can go, so this class is greatly simplifed from its version in SS2. + * + * @method SS_List|DataList sourceRecords($params = [], $sort = null, $limit = null) List of records to show for this report */ class Report extends ViewableData { @@ -79,7 +85,7 @@ class Report extends ViewableData * The class of object being managed by this report. * Set by overriding in your subclass. */ - protected $dataClass = 'SilverStripe\\CMS\\Model\\SiteTree'; + protected $dataClass = SiteTree::class; /** * A field that specifies the sort order of this report @@ -134,7 +140,10 @@ class Report extends ViewableData } /** - * Return the {@link SilverStripe\ORM\Queries\SQLSelect} that provides your report data. + * Return the {@link DataQuery} that provides your report data. + * + * @param array $params + * @return DataQuery */ public function sourceQuery($params) { @@ -147,6 +156,9 @@ class Report extends ViewableData /** * Return a SS_List records for this report. + * + * @param array $params + * @return SS_List */ public function records($params) { @@ -227,7 +239,8 @@ class Report extends ViewableData /** * Return the SS_Report objects making up the given list. - * @return Array of SS_Report objects + * + * @return Report[] Array of Report objects */ public static function get_reports() { @@ -248,6 +261,7 @@ class Report extends ViewableData continue; } + /** @var Report $reportObj */ $reportObj = $report::create(); if ($reportObj->hasMethod('sort')) { // Use the sort method to specify the sort field @@ -291,6 +305,7 @@ class Report extends ViewableData // Add search fields is available if ($this->hasMethod('parameterFields') && $parameterFields = $this->parameterFields()) { + /** @var FormField $field */ foreach ($parameterFields as $field) { // Namespace fields for easier handling in form submissions $field->setName(sprintf('filters[%s]', $field->getName()));