Make route getName optional

bezhanSalleh / filament-language-switch

A versatile and user-friendly plugin designed for Filament Panels.

MIT License

210 stars 45 forks source link

Open jyrkidn opened 3 months ago

jyrkidn commented 3 months ago

Since 404 routes return null this breaks calling getName(). Adding ? fixes this

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

Improved handling of route name retrivals The code in the LanguageSwitch.php file, specifically in the isVisibleOutsidePanels method, has been updated to retrieve the name of a route in a safer way. It now uses an updated syntax request()->route()?->getName(), ensuring the application avoids potential errors when trying to access the name of a route that might not exist. The updated code provides enhanced stability and reduces the chance of unexpected crashes.