Skip to content

Fix langswitch for Laravel 11 and 12#153

Merged
askdkc merged 8 commits intomainfrom
fix-langswitch
Mar 9, 2025
Merged

Fix langswitch for Laravel 11 and 12#153
askdkc merged 8 commits intomainfrom
fix-langswitch

Conversation

@askdkc
Copy link
Owner

@askdkc askdkc commented Mar 9, 2025

No description provided.

@what-the-diff
Copy link

what-the-diff bot commented Mar 9, 2025

PR Summary

  • Improved Installation Process for Language Switcher
    The InstallLanguageSwitcher.php command's functionality has been updated. Now, it will ensure the necessary 'middleware' directory exists before copying the Localization middleware. This improvement boosts the reliability of set-up and minimizes issues resulting from missing file directories.

  • Adjustment of Middleware Registration File
    Previously, our system was registering the middleware in the app/Http/Kernel.php file. We've shifted this process to the bootstrap/app.php file. This change aligns with our commitment to streamline our software's structure and enable easier maintenance.

  • Enhanced Success Message Accuracy
    We've amended the success message that appears upon successful registration of middleware. Now, it accurately displays the file where the middleware has been registered, ensuring better communication and clarity for our users.

  • Logic Updates to Facilitate Middleware Addition
    Our team has refined the logic to add the Localization middleware to bootstrap/app.php. Instead of retaining the existing placeholder, this adjusted logic now incorporates the new middleware entry. By doing so, we further strengthen the system's versatility and adaptability.

  • Removal of Redundant 'Kernel.stub' File
    Following our refined logic for middleware addition, the Kernel.stub file, previously used for storing middleware registration placeholders, has become redundant. As part of our commitment to maintaining a clean, uncluttered system, we've removed this file.

@askdkc askdkc merged commit f1890ea into main Mar 9, 2025
9 checks passed
@askdkc askdkc deleted the fix-langswitch branch March 9, 2025 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant