3

src/MenuItem.php

@@ -22,17 +22,10 @@ class MenuItem extends DataObject implements PermissionProvider
      */
     private static string $table_name = 'MenuItem';
 
-    /**
-     * @var array
-     */
     private static array $db = [
-        // If you want to customise the MenuTitle use this field - leaving blank will use MenuTitle of associated Page
         'MenuTitle' => 'Varchar(255)',
-        // This field is used for external links (picking a page from the dropdown will overwrite this link)
         'Link' => 'Text',
-        // Sort order
         'Sort' => 'Int',
-        // Can be used as a check for adding target="_blank"
         'IsNewWindow' => 'Boolean'
     ];
 
@@ -84,7 +77,13 @@ public function providePermissions(): array
      */
     public function canCreate($member = null, $context = []): bool
     {
-        return Permission::check('MANAGE_MENU_ITEMS');
+        $extended = $this->extendedCan(__FUNCTION__, $member);
+
+        if ($extended !== null) {
+            return $extended;
+        }
+
+        return Permission::checkMember($member, 'MANAGE_MENU_ITEMS');
     }
 
     /**
@@ -93,6 +92,12 @@ public function canCreate($member = null, $context = []): bool
      */
     public function canDelete($member = null): bool
     {
+        $extended = $this->extendedCan(__FUNCTION__, $member);
+
+        if ($extended !== null) {
+            return $extended;
+        }
+
         return Permission::check('MANAGE_MENU_ITEMS');
     }
 
@@ -102,6 +107,12 @@ public function canDelete($member = null): bool
      */
     public function canEdit($member = null): bool
     {
+        $extended = $this->extendedCan(__FUNCTION__, $member);
+
+        if ($extended !== null) {
+            return $extended;
+        }
+
         return Permission::check('MANAGE_MENU_ITEMS');
     }
 
@@ -111,6 +122,12 @@ public function canEdit($member = null): bool
      */
     public function canView($member = null): bool
     {
+        $extended = $this->extendedCan(__FUNCTION__, $member);
+
+        if ($extended !== null) {
+            return $extended;
+        }
+
         return Permission::check('MANAGE_MENU_ITEMS');
     }
 

src/MenuSet.php

@@ -91,11 +91,12 @@ public function validate()
      */
     public function canCreate($member = null, $context = []): bool
     {
-        if (Permission::check('MANAGE_MENU_SETS')) {
-            return true;
+        $extended = $this->extendedCan(__FUNCTION__, $member);
+        if ($extended !== null) {
+            return $extended;
         }
 
-        return parent::canCreate($member, $context);
+        return Permission::check('MANAGE_MENU_SETS');
     }
 
     /**
@@ -104,19 +105,17 @@ public function canCreate($member = null, $context = []): bool
      */
     public function canDelete($member = null): bool
     {
-        $canDelete = parent::canDelete($member);
-
         // Backwards compatibility for duplicate default sets
         $existing = MenuManagerTemplateProvider::MenuSet($this->Name);
         $isDuplicate = $existing && $existing->ID !== $this->ID;
 
         if ($this->isDefaultSet() && !$isDuplicate) {
-            // Default menu's cannot be deleted
-            $canDelete = false;
+            return false;
         }
 
-        if ($canDelete !== null) {
-            return $canDelete;
+        $extended = $this->extendedCan(__FUNCTION__, $member);
+        if ($extended !== null) {
+            return $extended;
         }
 
         return Permission::check('MANAGE_MENU_SETS');
@@ -128,11 +127,12 @@ public function canDelete($member = null): bool
      */
     public function canEdit($member = null): bool
     {
-        if (Permission::check('MANAGE_MENU_SETS') || Permission::check('MANAGE_MENU_ITEMS')) {
-            return true;
+        $extended = $this->extendedCan(__FUNCTION__, $member);
+        if ($extended !== null) {
+            return $extended;
         }
 
-        return parent::canEdit($member);
+        return (Permission::check('MANAGE_MENU_SETS') || Permission::check('MANAGE_MENU_ITEMS'));
     }
 
     /**
@@ -141,11 +141,13 @@ public function canEdit($member = null): bool
      */
     public function canView($member = null): bool
     {
-        if (Permission::check('MANAGE_MENU_SETS') || Permission::check('MANAGE_MENU_ITEMS')) {
-            return true;
+        $extended = $this->extendedCan(__FUNCTION__, $member);
+        if ($extended !== null) {
+            return $extended;
         }
 
-        return parent::canView($member);
+
+        return (Permission::check('MANAGE_MENU_SETS') || Permission::check('MANAGE_MENU_ITEMS'));
     }