tonyair/silverstripe-cms
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-cms synced 2024-10-22 08:05:56 +02:00
Code Issues Projects Releases Wiki Activity

NEW Reinstated filters in ReportAdmin

Browse Source 
- Moved report table generation back to its original location in Report->getCMSFields(),
in order to keep it customisable rather than duplicating it in ReportAdmin.
- Using History.js to set URL state and reload panel
- Namespacing filter GET parameters in order to only include them in URL state
This commit is contained in:
Ingo Schommer 2012-07-12 16:14:17 +02:00
parent d77c43d6c7
commit 57c615d6b2
3 changed files with 79 additions and 158 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
code/controllers/ReportAdmin.php
View File

@ -51,6 +51,7 @@ class ReportAdmin extends LeftAndMain implements PermissionProvider {
// Always block the HtmlEditorField.js otherwise it will be sent with an ajax request // Always block the HtmlEditorField.js otherwise it will be sent with an ajax request
Requirements::block(FRAMEWORK_DIR . '/javascript/HtmlEditorField.js'); Requirements::block(FRAMEWORK_DIR . '/javascript/HtmlEditorField.js');
Requirements::javascript(CMS_DIR . '/javascript/ReportAdmin.js');
} }
/** /**
@ -147,48 +148,12 @@ class ReportAdmin extends LeftAndMain implements PermissionProvider {
} }
public function getEditForm($id = null, $fields = null) { public function getEditForm($id = null, $fields = null) {
$fields = new FieldList();
$report = $this->reportObject; $report = $this->reportObject;
if($report) { if($report) {
// List all reports $fields = $report->getCMSFields();
$gridFieldConfig = GridFieldConfig::create()->addComponents(
new GridFieldToolbarHeader(),
new GridFieldSortableHeader(),
new GridFieldDataColumns(),
new GridFieldPaginator(),
new GridFieldPrintButton(),
new GridFieldExportButton()
);
$gridField = new GridField('Report',$report->title(), $report->sourceRecords(array(), null, null), $gridFieldConfig);
$columns = $gridField->getConfig()->getComponentByType('GridFieldDataColumns');
$displayFields = array();
$fieldCasting = array();
$fieldFormatting = array();
// Parse the column information
foreach($report->columns() as $source => $info) {
if(is_string($info)) $info = array('title' => $info);
if(isset($info['formatting'])) $fieldFormatting[$source] = $info['formatting'];
if(isset($info['csvFormatting'])) $csvFieldFormatting[$source] = $info['csvFormatting'];
if(isset($info['casting'])) $fieldCasting[$source] = $info['casting'];
if(isset($info['link']) && $info['link']) {
$link = singleton('CMSPageEditController')->Link('show');
$fieldFormatting[$source] = '<a href=\"' . $link . '/$ID\">$value</a>';
}
$displayFields[$source] = isset($info['title']) ? $info['title'] : $source;
}
$columns->setDisplayFields($displayFields);
$columns->setFieldCasting($fieldCasting);
$columns->setFieldFormatting($fieldFormatting);
$fields->push($gridField);
} else { } else {
// List all reports // List all reports
$fields = new FieldList();
$gridFieldConfig = GridFieldConfig::create()->addComponents( $gridFieldConfig = GridFieldConfig::create()->addComponents(
new GridFieldToolbarHeader(), new GridFieldToolbarHeader(),
new GridFieldSortableHeader(), new GridFieldSortableHeader(),
@ -210,6 +175,7 @@ class ReportAdmin extends LeftAndMain implements PermissionProvider {
$actions = new FieldList(); $actions = new FieldList();
$form = new Form($this, "EditForm", $fields, $actions); $form = new Form($this, "EditForm", $fields, $actions);
$form->addExtraClass('cms-edit-form cms-panel-padded center ' . $this->BaseCSSClasses()); $form->addExtraClass('cms-edit-form cms-panel-padded center ' . $this->BaseCSSClasses());
$form->loadDataFrom($this->request->getVars());
$this->extend('updateEditForm', $form); $this->extend('updateEditForm', $form);

167
code/reports/Report.php
View File

@ -97,8 +97,7 @@ class SS_Report extends ViewableData {
*/ */
function sourceQuery($params) { function sourceQuery($params) {
if($this->hasMethod('sourceRecords')) { if($this->hasMethod('sourceRecords')) {
$query = new SS_Report_FakeQuery($this, 'sourceRecords', $params); return $this->sourceRecords()->dataQuery();
return $query;
} else { } else {
user_error("Please override sourceQuery()/sourceRecords() and columns() or, if necessary, override getReportField()", E_USER_ERROR); user_error("Please override sourceQuery()/sourceRecords() and columns() or, if necessary, override getReportField()", E_USER_ERROR);
} }
@ -108,14 +107,20 @@ class SS_Report extends ViewableData {
* Return a SS_List records for this report. * Return a SS_List records for this report.
*/ */
function records($params) { function records($params) {
if($this->hasMethod('sourceRecords')) return $this->sourceRecords($params, null, null); if($this->hasMethod('sourceRecords')) {
else { return $this->sourceRecords($params, null, null);
} else {
$query = $this->sourceQuery(); $query = $this->sourceQuery();
return singleton($this->dataClass())->buildDataObjectSet($query->execute(), "DataObjectSet", $query); $results = new ArrayList();
foreach($query->execute() as $data) {
$class = $this->dataClass();
$result = new $class($data);
$results->push($result);
}
return $results;
} }
} }
/** /**
* Return the data class for this report * Return the data class for this report
*/ */
@ -132,8 +137,6 @@ class SS_Report extends ViewableData {
); );
} }
/** /**
* @deprecated 3.0 * @deprecated 3.0
* All subclasses of SS_Report now appear in the report admin, no need to register or unregister. * All subclasses of SS_Report now appear in the report admin, no need to register or unregister.
@ -171,7 +174,6 @@ class SS_Report extends ViewableData {
} }
} }
/** /**
* Return an array of excluded reports. That is, reports that will not be included in * Return an array of excluded reports. That is, reports that will not be included in
* the list of reports in report admin in the CMS. * the list of reports in report admin in the CMS.
@ -225,27 +227,27 @@ class SS_Report extends ViewableData {
* @return FieldList * @return FieldList
*/ */
function getCMSFields() { function getCMSFields() {
$fields = new FieldList( $fields = new FieldList();
new LiteralField(
'ReportTitle',
"<h3>{$this->title()}</h3>"
)
);
if($this->description()) $fields->push( if($title = $this->title()) {
new LiteralField('ReportDescription', "<p>" . $this->description() . "</p>")); $fields->push(new LiteralField('ReportTitle', "<h3>{$title}</h3>"));
}
if($description = $this->description()) {
$fields->push(new LiteralField('ReportDescription', "<p>" . $description . "</p>"));
}
// Add search fields is available // Add search fields is available
if($params = $this->parameterFields()) { if($fields = $this->parameterFields()) {
$filters = new FieldGroup('Filters'); foreach($fields as $field) {
foreach($params as $param) { // Namespace fields for easier handling in form submissions
if ($param instanceof HiddenField) $fields->push($param); $field->setName(sprintf('filters[%s]', $field->getName()));
else $filters->push($param); $field->addExtraClass('no-change-track'); // ignore in changetracker
$fields->push($field);
} }
$fields->push($filters);
// Add a search button // Add a search button
$fields->push(new FormAction('updatereport', 'Filter')); $fields->push(new FormAction('updatereport', _t('GridField.Filter')));
} }
$fields->push($this->getReportField()); $fields->push($this->getReportField());
@ -263,7 +265,7 @@ class SS_Report extends ViewableData {
} }
/** /**
* Return a field, such as a {@link ComplexTableField} that is * Return a field, such as a {@link GridField} that is
* used to show and manipulate data relating to this report. * used to show and manipulate data relating to this report.
* *
* Generally, you should override {@link columns()} and {@link records()} to make your report, * Generally, you should override {@link columns()} and {@link records()} to make your report,
@ -272,10 +274,23 @@ class SS_Report extends ViewableData {
* @return FormField subclass * @return FormField subclass
*/ */
function getReportField() { function getReportField() {
$columnTitles = array(); // TODO Remove coupling with global state
$fieldFormatting = array(); $params = isset($_REQUEST['filters']) ? $_REQUEST['filters'] : array();
$csvFieldFormatting = array(); $items = $this->sourceRecords($params, null, null);
$gridFieldConfig = GridFieldConfig::create()->addComponents(
new GridFieldToolbarHeader(),
new GridFieldSortableHeader(),
new GridFieldDataColumns(),
new GridFieldPaginator(),
new GridFieldPrintButton(),
new GridFieldExportButton()
);
$gridField = new GridField('Report',$this->title(), $items, $gridFieldConfig);
$columns = $gridField->getConfig()->getComponentByType('GridFieldDataColumns');
$displayFields = array();
$fieldCasting = array(); $fieldCasting = array();
$fieldFormatting = array();
// Parse the column information // Parse the column information
foreach($this->columns() as $source => $info) { foreach($this->columns() as $source => $info) {
@ -284,28 +299,19 @@ class SS_Report extends ViewableData {
if(isset($info['formatting'])) $fieldFormatting[$source] = $info['formatting']; if(isset($info['formatting'])) $fieldFormatting[$source] = $info['formatting'];
if(isset($info['csvFormatting'])) $csvFieldFormatting[$source] = $info['csvFormatting']; if(isset($info['csvFormatting'])) $csvFieldFormatting[$source] = $info['csvFormatting'];
if(isset($info['casting'])) $fieldCasting[$source] = $info['casting']; if(isset($info['casting'])) $fieldCasting[$source] = $info['casting'];
$columnTitles[$source] = isset($info['title']) ? $info['title'] : $source;
if(isset($info['link']) && $info['link']) {
$link = singleton('CMSPageEditController')->Link('show');
$fieldFormatting[$source] = '<a href=\"' . $link . '/$ID\">$value</a>';
} }
// To do: implement pagination $displayFields[$source] = isset($info['title']) ? $info['title'] : $source;
$query = $this->sourceQuery($_REQUEST);
$tlf = new TableListField('ReportContent', $this->dataClass(), $columnTitles);
$tlf->setCustomQuery($query);
$tlf->setShowPagination(true);
$tlf->setPageSize(50);
$tlf->setPermissions(array('export', 'print'));
// Hack to figure out if we are printing
if (isset($_REQUEST['url']) && array_pop(explode('/', $_REQUEST['url'])) == 'printall') {
$tlf->setTemplate('SSReportTableField');
} }
$columns->setDisplayFields($displayFields);
$columns->setFieldCasting($fieldCasting);
$columns->setFieldFormatting($fieldFormatting);
if($fieldFormatting) $tlf->setFieldFormatting($fieldFormatting); return $gridField;
if($csvFieldFormatting) $tlf->setCSVFieldFormatting($csvFieldFormatting);
if($fieldCasting) $tlf->setFieldCasting($fieldCasting);
return $tlf;
} }
/** /**
@ -333,75 +339,6 @@ class SS_Report extends ViewableData {
return $this->title(); return $this->title();
} }
}
/**
* This is an object that can be used to dress up a more complex querying mechanism in the clothing
* of a SQLQuery object. This means that you can inject it into a TableListField.
*
* Use it like this:
*
* function sourceQuery($params) {
* return new SS_Report_FakeQuery($this, 'sourceRecords', $params)
* }
* function sourceRecords($params, $sort, $limit) {
* // Do some stuff
* // Return a SS_List of actual objects.
* }
*
* This object is used by the default implementation of sourceQuery() on SS_Report, to make use of
* a sourceReords() method if one exists.
*/
class SS_Report_FakeQuery extends SQLQuery {
public $orderby;
public $limit;
protected $obj, $method, $params;
function __construct($obj, $method, $params) {
$this->obj = $obj;
$this->method = $method;
$this->params = $params;
}
/**
* Provide a method that will return a list of columns that can be used to sort.
*/
function setSortColumnMethod($sortColMethod) {
$this->sortColMethod = $sortColMethod;
}
function limit($limit, $offset = 0) {
$this->limit = $limit;
}
function unlimitedRowCount($column = null) {
$source = $this->obj->{$this->method}($this->params, null, null);
return $source ? $source->Count() : 0;
}
function execute() {
$output = array();
$source = $this->obj->{$this->method}($this->params, $this->orderby, $this->limit);
if($source) foreach($source as $item) {
$mapItem = $item->toMap();
$mapItem['RecordClassName'] = get_class($item);
$output[] = $mapItem;
}
return $output;
}
function canSortBy($fieldName) {
$fieldName = preg_replace('/(\s+?)(A|DE)SC$/', '', $fieldName);
if($this->sortColMethod) {
$columns = $this->obj->{$this->sortColMethod}();
return in_array($fieldName, $columns);
} else {
return false;
}
}
} }
/** /**

18
javascript/ReportAdmin.js Normal file
View File

@ -0,0 +1,18 @@
/**
* File: ReportAdmin.js
*/
(function($) {
$.entwine('ss', function($){
$('.ReportAdmin .cms-edit-form').entwine({
onsubmit: function(e) {
var url = document.location.href, params = this.find(':input[name^=filters]').serializeArray();
params = $.grep(params, function(param) {return (param.value);}); // filter out empty
if(params) url = $.path.addSearchParams(url, params);
$('.cms-container').loadPanel(url);
return false;
}
});
});
})(jQuery);