diff --git a/src/Extensions/MenuSetParentExtension.php b/src/Extensions/MenuSetParentExtension.php index 59b5672..0af1c1e 100644 --- a/src/Extensions/MenuSetParentExtension.php +++ b/src/Extensions/MenuSetParentExtension.php @@ -31,6 +31,10 @@ class MenuSetParentExtension extends DataExtension 'MenuSets' ]; + private static $cascade_duplicates = [ + 'MenuSets' + ]; + public function updateCMSFields(FieldList $fields) { $fields->removeByName('MenuSets'); diff --git a/src/Model/MenuItem.php b/src/Model/MenuItem.php index f6cdb65..b4b0e25 100644 --- a/src/Model/MenuItem.php +++ b/src/Model/MenuItem.php @@ -18,12 +18,14 @@ use SilverStripe\Forms\GridField\GridFieldPageCount; use SilverStripe\Forms\GridField\GridFieldPaginator; use SilverStripe\Forms\GridField\GridFieldToolbarHeader; +use SilverStripe\Security\Permission; +use SilverStripe\Security\PermissionProvider; use SilverStripe\Versioned\Versioned; use Symbiote\GridFieldExtensions\GridFieldAddNewMultiClass; use Symbiote\GridFieldExtensions\GridFieldOrderableRows; use UncleCheese\DisplayLogic\Forms\Wrapper; -class MenuItem extends SuperLink +class MenuItem extends SuperLink implements PermissionProvider { const SUBMENU_SITETREE = 'sitetree'; const SUBMENU_MANUAL = 'manual'; @@ -69,6 +71,10 @@ class MenuItem extends SuperLink 'Children' ]; + private static $cascade_duplicates = [ + 'Children' + ]; + private static $field_labels = [ 'SubmenuMode' => 'Submenu', 'SubmenuSiteTree' => 'Select Page' @@ -309,33 +315,62 @@ public function CMSEditLink() public function canView($member = null) { - if ($this->ParentID) { - return $this->Parent()->canView($member); + $can = Permission::checkMember($member, 'MANAGE_MENUITEMS'); + if ($can) { + if ($this->ParentID) { + $can = $this->Parent()->canView($member, $context); + } } - return $this->MenuSet()->canView($member); + return $can; } public function canEdit($member = null) { - if ($this->ParentID) { - return $this->Parent()->canEdit($member); + $can = Permission::checkMember($member, 'MANAGE_MENUITEMS'); + if ($can) { + if ($this->ParentID) { + $can = $this->Parent()->canEdit($member, $context); + } } - return $this->MenuSet()->canEdit($member); + return $can; } public function canDelete($member = null) { - if ($this->ParentID) { - return $this->Parent()->canDelete($member); + $can = Permission::checkMember($member, 'DELETE_MENUITEMS'); + if ($can) { + if ($this->ParentID) { + $can = $this->Parent()->canDelete($member, $context); + } } - return $this->MenuSet()->canDelete($member); + return $can; } public function canCreate($member = null, $context = []) { - if ($this->ParentID) { - return $this->Parent()->canCreate($member, $context); + $can = Permission::checkMember($member, 'CREATE_MENUITEMS'); + if ($can) { + if ($this->ParentID) { + $can = $this->Parent()->canCreate($member, $context); + } } - return $this->MenuSet()->canEdit($member); + return $can; + } + + public function providePermissions() { + return [ + 'MANAGE_MENUITEMS' => array( + 'name' => 'Manage menu items', + 'category' => 'Menus', + ), + 'CREATE_MENUITEMS' => array( + 'name' => 'Create menu items', + 'category' => 'Menus', + ), + 'DELETE_MENUITEMS' => array( + 'name' => 'Delete menu items', + 'category' => 'Menus', + ) + ]; } } diff --git a/src/Model/MenuItemSiteTree.php b/src/Model/MenuItemSiteTree.php index dc6b3b0..7406a45 100644 --- a/src/Model/MenuItemSiteTree.php +++ b/src/Model/MenuItemSiteTree.php @@ -3,9 +3,6 @@ namespace Fromholdio\SuperLinkerMenus\Model; use Fromholdio\SuperLinker\Extensions\SiteTreeLink; -use SilverStripe\Forms\CheckboxField; -use SilverStripe\Forms\FieldGroup; -use UncleCheese\DisplayLogic\Forms\Wrapper; class MenuItemSiteTree extends MenuItem { diff --git a/src/Model/MenuSet.php b/src/Model/MenuSet.php index 73ac7df..e855ec2 100644 --- a/src/Model/MenuSet.php +++ b/src/Model/MenuSet.php @@ -26,12 +26,14 @@ use SilverStripe\ORM\DataObject; use SilverStripe\SiteConfig\SiteConfig; use SilverStripe\SiteConfig\SiteConfigLeftAndMain; +use SilverStripe\Security\Permission; +use SilverStripe\Security\PermissionProvider; use SilverStripe\Versioned\Versioned; use SilverStripe\View\SSViewer; use Symbiote\GridFieldExtensions\GridFieldAddNewMultiClass; use Symbiote\GridFieldExtensions\GridFieldOrderableRows; -class MenuSet extends DataObject +class MenuSet extends DataObject implements PermissionProvider { private static $table_name = 'MenuSet'; private static $singular_name = 'Menu'; @@ -64,6 +66,10 @@ class MenuSet extends DataObject 'Items' ]; + private static $cascade_duplicates = [ + 'Items' + ]; + private static $defaults = [ 'IsTitleEnabled' => false, 'IsHighlightEnabled' => false, @@ -473,29 +479,46 @@ public function CMSEditLink() public function canCreate($member = null, $context = null) { - return false; + return Permission::checkMember($member, 'CREATE_MENUSETS'); } public function canDelete($member = null) { - return false; + return Permission::checkMember($member, 'DELETE_MENUSETS'); } public function canEdit($member = null) { - $can = $this->Parent()->canEdit($member); - if ($can === false) { - return false; + $can = Permission::checkMember($member, 'MANAGE_MENUSETS'); + if ($can) { + $can = $this->Parent()->canEdit($member); } - return parent::canEdit($member); + return $can; } public function canView($member = null) { - $can = $this->Parent()->canView($member); - if ($can === false) { - return false; + $can = Permission::checkMember($member, 'MANAGE_MENUSETS'); + if ($can) { + $can = $this->Parent()->canView($member); } - return parent::canView($member); + return $can; + } + + public function providePermissions() { + return [ + 'MANAGE_MENUSETS' => array( + 'name' => 'Manage menu sets', + 'category' => 'Menus', + ), + 'CREATE_MENUSETS' => array( + 'name' => 'Create menu sets', + 'category' => 'Menus', + ), + 'DELETE_MENUSETS' => array( + 'name' => 'Delete menu sets', + 'category' => 'Menus', + ) + ]; } }