| @@ -76,9 +76,13 @@ | ||
| 76 | 76 | { |
| 77 | 77 | $menu = [ |
| 78 | 78 | 'label' => __('Products', 'fluent-cart'), |
| 79 | 79 | 'link' => $baseUrl . 'products', |
| 80 | - 'permission' => ['products/view'] | |
| 80 | + // Any-of: a reviews-only role (reviews/manage without | |
| 81 | + // products/view) must still see the parent, or its authorized | |
| 82 | + // Reviews child could never render. Children gate individually — | |
| 83 | + // including on the reviews module being switched on at all. | |
| 84 | + 'permission' => ['products/view', 'reviews/manage'] | |
| 81 | 85 | ]; |
| 82 | 86 | |
| 83 | 87 | $children = []; |
| 84 | 88 | |
| @@ -103,8 +107,19 @@ | ||
| 103 | 107 | $children['product_inventory'] = [ |
| 104 | 108 | 'label' => __('Inventory', 'fluent-cart'), |
| 105 | 109 | 'link' => $baseUrl . 'products/inventory', |
| 106 | 110 | 'permission' => ['products/view'] |
| 111 | + ]; | |
| 112 | + } | |
| 113 | + | |
| 114 | + // Only while the store has reviews switched on. The screen behind it | |
| 115 | + // already redirects to the dashboard when the module is off, so an | |
| 116 | + // entry here would be a menu item that bounces whoever clicks it. | |
| 117 | + if (ModuleSettings::isActive('reviews')) { | |
| 118 | + $children['reviews'] = [ | |
| 119 | + 'label' => __('Reviews', 'fluent-cart'), | |
| 120 | + 'link' => $baseUrl . 'reviews', | |
| 121 | + 'permission' => ['reviews/manage'] | |
| 107 | 122 | ]; |
| 108 | 123 | } |
| 109 | 124 | |
| 110 | 125 | $menu['children'] = $children; |
