[PR #4903] [MERGED] Framework: Upgrade from Laravel 9 to 10 #6422

New Issue

OVERLORD · 2026-02-05T10:31:53+03:00

OVERLORD commented

2026-02-05 10:31:53 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/4903
Author: @ssddanbrown
Created: 3/16/2024
Status: ✅ Merged
Merged: 3/17/2024
Merged by: @ssddanbrown

Base: development ← Head: laravel10

📝 Commits (7)

794671e Framework: Upgrade from Laravel 9 to 10
d6b7717 Framework: Fixed issues breaking tests
45d52f2 Migrations: Updated with type hints instead of php doc
3250fc7 Testing: Updated PHPUnit from 9 to 10
2345fd4 Deps: Updated intervention library from 2 to 3
b4b84f8 Deps: Updated custom symfony/mailer package
28d6292 Framework: Addressed deprecations

📊 Changes

110 files changed (+932 additions, -1232 deletions)

View changed files

📝 app/Access/Controllers/MfaTotpController.php (+11 -6)
📝 app/Access/Mfa/TotpValidationRule.php (+12 -22)
📝 app/App/Providers/AppServiceProvider.php (+13 -17)
📝 app/App/Providers/AuthServiceProvider.php (+2 -6)
📝 app/App/Providers/EventServiceProvider.php (+2 -6)
📝 app/App/Providers/RouteServiceProvider.php (+4 -12)
📝 app/App/Providers/ThemeServiceProvider.php (+2 -6)
📝 app/App/Providers/TranslationServiceProvider.php (+2 -6)
📝 app/App/Providers/ViewTweaksServiceProvider.php (+1 -3)
📝 app/Config/app.php (+11 -34)
📝 app/Config/cache.php (+2 -1)
📝 app/Config/hashing.php (+2 -1)
📝 app/Config/logging.php (+8 -0)
📝 app/Config/queue.php (+6 -0)
📝 app/Config/session.php (+7 -0)
📝 app/Http/Kernel.php (+2 -2)
📝 app/Http/Middleware/RedirectIfAuthenticated.php (+3 -6)
📝 app/Http/Middleware/ThrottleApiRequests.php (+1 -1)
📝 app/Http/Middleware/TrustHosts.php (+2 -2)
📝 app/Translation/FileLoader.php (+9 -3)

...and 80 more files

📄 Description

Upgrade of Laravel 9 to Laravel 10.

Todo

Get existing tests passing
Add type hints to database migrations
Upgrade PHPUnit to 10
Check remaining composer deps
Check for deprecations (like in the add_search_indexes migration)

Docs Updates

Requirements Update: Composer v2.2.0 minimum required (2021-12-22 release date)
Update Advisory: Warn of composer requirement change, link to part on update page.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/4903 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 3/16/2024 **Status:** ✅ Merged **Merged:** 3/17/2024 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `development` ← **Head:** `laravel10` --- ### 📝 Commits (7) - [`794671e`](https://github.com/BookStackApp/BookStack/commit/794671ef32f8f67ebe30b9575bea0fa0e74d644b) Framework: Upgrade from Laravel 9 to 10 - [`d6b7717`](https://github.com/BookStackApp/BookStack/commit/d6b77179856420b21a4d428d070b7d784c90b468) Framework: Fixed issues breaking tests - [`45d52f2`](https://github.com/BookStackApp/BookStack/commit/45d52f27ae578e00b8d45d6848ffcefede6015d6) Migrations: Updated with type hints instead of php doc - [`3250fc7`](https://github.com/BookStackApp/BookStack/commit/3250fc732c74ae09566a48da2739c3f5e450665f) Testing: Updated PHPUnit from 9 to 10 - [`2345fd4`](https://github.com/BookStackApp/BookStack/commit/2345fd46777828cd81919f4e230a1d74ab835add) Deps: Updated intervention library from 2 to 3 - [`b4b84f8`](https://github.com/BookStackApp/BookStack/commit/b4b84f81a0f5a85cf0a37a870d46d02c9ca84c98) Deps: Updated custom symfony/mailer package - [`28d6292`](https://github.com/BookStackApp/BookStack/commit/28d6292278b9ce2ab728e62a2b622a0beee62c1c) Framework: Addressed deprecations ### 📊 Changes **110 files changed** (+932 additions, -1232 deletions) <details> <summary>View changed files</summary> 📝 `app/Access/Controllers/MfaTotpController.php` (+11 -6) 📝 `app/Access/Mfa/TotpValidationRule.php` (+12 -22) 📝 `app/App/Providers/AppServiceProvider.php` (+13 -17) 📝 `app/App/Providers/AuthServiceProvider.php` (+2 -6) 📝 `app/App/Providers/EventServiceProvider.php` (+2 -6) 📝 `app/App/Providers/RouteServiceProvider.php` (+4 -12) 📝 `app/App/Providers/ThemeServiceProvider.php` (+2 -6) 📝 `app/App/Providers/TranslationServiceProvider.php` (+2 -6) 📝 `app/App/Providers/ViewTweaksServiceProvider.php` (+1 -3) 📝 `app/Config/app.php` (+11 -34) 📝 `app/Config/cache.php` (+2 -1) 📝 `app/Config/hashing.php` (+2 -1) 📝 `app/Config/logging.php` (+8 -0) 📝 `app/Config/queue.php` (+6 -0) 📝 `app/Config/session.php` (+7 -0) 📝 `app/Http/Kernel.php` (+2 -2) 📝 `app/Http/Middleware/RedirectIfAuthenticated.php` (+3 -6) 📝 `app/Http/Middleware/ThrottleApiRequests.php` (+1 -1) 📝 `app/Http/Middleware/TrustHosts.php` (+2 -2) 📝 `app/Translation/FileLoader.php` (+9 -3) _...and 80 more files_ </details> ### 📄 Description Upgrade of Laravel 9 to Laravel 10. ## Todo - [x] Get existing tests passing - [x] Add type hints to database migrations - [x] Upgrade PHPUnit to 10 - [x] Check remaining composer deps - [x] Check for deprecations (like in the add_search_indexes migration) ## Docs Updates - Requirements Update: Composer v2.2.0 minimum required (2021-12-22 release date) - Update Advisory: Warn of composer requirement change, link to part on update page. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-05 10:31:53 +03:00

OVERLORD closed this issue

2026-02-05 10:31:55 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#6422