Merge pull request #2 from sunnysideup/master

adding counts to report links
This commit is contained in:
Daniel Hensby 2015-07-22 13:23:52 +01:00
commit 0759392980
2 changed files with 20 additions and 4 deletions

View File

@ -136,6 +136,7 @@ class SS_Report extends ViewableData {
return $this->dataClass;
}
public function getLink($action = null) {
return Controller::join_links(
'admin/reports/',
@ -145,6 +146,21 @@ class SS_Report extends ViewableData {
);
}
/**
* counts the number of objects returned
* @param Array $params - any parameters for the sourceRecords
* @return Int
*/
public function getCount($params = array()){
$sourceRecords = $this->sourceRecords($params, null, null);
if(!$sourceRecords instanceOf SS_List){
user_error($this->class."::sourceRecords does not return an SS_List", E_USER_NOTICE);
return "-1";
}
return $sourceRecords->count();
}
/**
* Exclude certain reports classes from the list of Reports in the CMS
* @param $reportClass Can be either a string with the report classname or an array of reports classnames

View File

@ -170,9 +170,9 @@ class ReportAdmin extends LeftAndMain implements PermissionProvider {
'title' => _t('ReportAdmin.ReportTitle', 'Title'),
));
$columns->setFieldFormatting(array(
'title' => '<a href=\"$Link\" class=\"cms-panel-link\">$value</a>'
));
$columns->setFieldFormatting(array(
'title' => '<a href=\"$Link\" class=\"cms-panel-link\">$value ($Count)</a>'
));
$gridField->addExtraClass('all-reports-gridfield');
$fields->push($gridField);
}
@ -186,4 +186,4 @@ class ReportAdmin extends LeftAndMain implements PermissionProvider {
return $form;
}
}
}