mirror of
https://github.com/silverstripe/silverstripe-reports
synced 2024-10-22 11:05:53 +02:00
Merge pull request #14 from christopherdarling/patch-2
FIX: SS_Report canView should check permissions
This commit is contained in:
commit
fc81b1f582
@ -308,8 +308,15 @@ class SS_Report extends ViewableData {
|
|||||||
if(!$member && $member !== FALSE) {
|
if(!$member && $member !== FALSE) {
|
||||||
$member = Member::currentUser();
|
$member = Member::currentUser();
|
||||||
}
|
}
|
||||||
|
|
||||||
return true;
|
$extended = $this->extendedCan('canView', $member);
|
||||||
|
if($extended !== null) return $extended;
|
||||||
|
|
||||||
|
if($member && Permission::checkMember($member, array('CMS_ACCESS_LeftAndMain', 'CMS_ACCESS_ReportAdmin'))) {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user