Fix shouldRegisterNavigation

bezhanSalleh / filament-shield

The easiest and most intuitive way to add access management to your Filament Admin Resources, Pages & Widgets through `spatie/laravel-permission`

MIT License

1.35k stars 166 forks source link

Open rominronin opened 3 weeks ago

rominronin commented 3 weeks ago

This patch is to fix an issue which prevents composer from installing.

There was an existing patch #249 - however this could not apply to the latest version (currently 3.2.5).

So I created this.

what-the-diff[bot] commented 3 weeks ago

Update to shouldRegisterNavigation() Method in HasPageShield.php File

This Section of code was revised to now accept additional inputs. The benefit of this is that it enhances the function's ability to maneuver more complex scenarios. Aside from this, the conditions for achieving a successful operation have been altered. It now checks if users can view, rather than just access the page, increasing the usability and security levels of our application.