MINOR using better sorting api (from r96483) (from r98197)

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/cms/trunk@105841 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Ingo Schommer 2010-05-28 02:30:40 +00:00
parent 2e0afb879e
commit 65a35d92fc

View File

@ -11,9 +11,8 @@ class BrokenLinksReport extends SS_Report {
return _t('BrokenLinksReport.BROKENLINKS',"Broken links report"); return _t('BrokenLinksReport.BROKENLINKS',"Broken links report");
} }
function sourceRecords($params, $sort, $limit) { function sourceRecords($params, $sort, $limit) {
if(!$sort || !in_array(preg_replace('/(\s+?)(A|DE)SC$/', '', $sort), array('Title','LastEdited'))) $sort = "LastEdited ASC"; if (!isset($_REQUEST['CheckSite']) || $params['CheckSite'] == 'Published') $ret = Versioned::get_by_stage('SiteTree', 'Live', "(HasBrokenLink = 1 OR HasBrokenFile = 1)");
if (!isset($_REQUEST['CheckSite']) || $params['CheckSite'] == 'Published') $ret = Versioned::get_by_stage('SiteTree', 'Live', "(HasBrokenLink = 1 OR HasBrokenFile = 1)", $sort); else $ret = DataObject::get('SiteTree', "(HasBrokenFile = 1 OR HasBrokenLink = 1)");
else $ret = DataObject::get('SiteTree', "(HasBrokenFile = 1 OR HasBrokenLink = 1)", $sort);
$returnSet = new DataObjectSet(); $returnSet = new DataObjectSet();
if ($ret) foreach($ret as $record) { if ($ret) foreach($ret as $record) {
@ -51,6 +50,8 @@ class BrokenLinksReport extends SS_Report {
} }
} }
if ($sort) $returnSet->sort($sort);
return $returnSet; return $returnSet;
} }
function columns() { function columns() {