[PR #4021] [MERGED] Upgrade framework to Laravel 9 #6296

New Issue

OVERLORD · 2026-02-05T10:28:45+03:00

OVERLORD commented

2026-02-05 10:28:45 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/4021
Author: @ssddanbrown
Created: 2/6/2023
Status: ✅ Merged
Merged: 2/7/2023
Merged by: @ssddanbrown

Base: development ← Head: laravel9

📝 Commits (4)

008e7a4 Followed Laravel 9 update steps and file changes
9ca088a Fixed static analysis issues
5e8ec56 Fixed issues found from tests
bc18dc7 Removed parallel testing, updated predis

📊 Changes

619 files changed (+1759 additions, -2608 deletions)

View changed files

📝 .github/workflows/test-migrations.yml (+1 -1)
📝 .github/workflows/test-php.yml (+1 -1)
📝 app/Config/app.php (+13 -54)
📝 app/Config/broadcasting.php (+2 -16)
📝 app/Config/cache.php (+1 -1)
📝 app/Config/filesystems.php (+4 -0)
📝 app/Config/logging.php (+9 -0)
📝 app/Config/mail.php (+34 -22)
📝 app/Entities/Tools/Markdown/CustomListItemRenderer.php (+12 -12)
📝 app/Entities/Tools/Markdown/CustomStrikeThroughExtension.php (+3 -3)
📝 app/Entities/Tools/Markdown/CustomStrikethroughRenderer.php (+8 -10)
📝 app/Entities/Tools/Markdown/MarkdownToHtml.php (+7 -5)
📝 app/Exceptions/BookStackExceptionHandlerPage.php (+7 -14)
📝 app/Exceptions/Handler.php (+4 -3)
📝 app/Http/Request.php (+2 -6)
📝 app/Providers/AppServiceProvider.php (+3 -3)
📝 app/Providers/EventServiceProvider.php (+12 -1)
📝 app/Providers/RouteServiceProvider.php (+1 -1)
📝 app/Theming/ThemeEvents.php (+2 -2)
📝 app/Uploads/AttachmentService.php (+2 -2)

...and 80 more files

📄 Description

Todo

Fix phpstan issues.
Test commonmark (https://packagist.org/packages/league/commonmark)
Test s3 usage (https://packagist.org/packages/league/flysystem-aws-s3-v3)
Parallel testing (Removed paratest package from composer)
Test Redis usage (Updated predis)
Test custom error handler display.
Moved lang folder, check crowdin integration after (Re-do branch).

Doc Updates

PHP minimum version changed to 8.0.2.
Arguments for the COMMONMARK_ENVIRONMENT_CONFIGURE theme event have changed.

_{🔄 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/4021 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 2/6/2023 **Status:** ✅ Merged **Merged:** 2/7/2023 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `development` ← **Head:** `laravel9` --- ### 📝 Commits (4) - [`008e7a4`](https://github.com/BookStackApp/BookStack/commit/008e7a4d25b96c72469120ef4c7f760fd8662a08) Followed Laravel 9 update steps and file changes - [`9ca088a`](https://github.com/BookStackApp/BookStack/commit/9ca088a4e2a952250583d920d2a1cc0aaca51189) Fixed static analysis issues - [`5e8ec56`](https://github.com/BookStackApp/BookStack/commit/5e8ec56196f94a86b9600a2dec031ec92afc0170) Fixed issues found from tests - [`bc18dc7`](https://github.com/BookStackApp/BookStack/commit/bc18dc7da6fe61f311b865e31f36883fa37af0da) Removed parallel testing, updated predis ### 📊 Changes **619 files changed** (+1759 additions, -2608 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/test-migrations.yml` (+1 -1) 📝 `.github/workflows/test-php.yml` (+1 -1) 📝 `app/Config/app.php` (+13 -54) 📝 `app/Config/broadcasting.php` (+2 -16) 📝 `app/Config/cache.php` (+1 -1) 📝 `app/Config/filesystems.php` (+4 -0) 📝 `app/Config/logging.php` (+9 -0) 📝 `app/Config/mail.php` (+34 -22) 📝 `app/Entities/Tools/Markdown/CustomListItemRenderer.php` (+12 -12) 📝 `app/Entities/Tools/Markdown/CustomStrikeThroughExtension.php` (+3 -3) 📝 `app/Entities/Tools/Markdown/CustomStrikethroughRenderer.php` (+8 -10) 📝 `app/Entities/Tools/Markdown/MarkdownToHtml.php` (+7 -5) 📝 `app/Exceptions/BookStackExceptionHandlerPage.php` (+7 -14) 📝 `app/Exceptions/Handler.php` (+4 -3) 📝 `app/Http/Request.php` (+2 -6) 📝 `app/Providers/AppServiceProvider.php` (+3 -3) 📝 `app/Providers/EventServiceProvider.php` (+12 -1) 📝 `app/Providers/RouteServiceProvider.php` (+1 -1) 📝 `app/Theming/ThemeEvents.php` (+2 -2) 📝 `app/Uploads/AttachmentService.php` (+2 -2) _...and 80 more files_ </details> ### 📄 Description ### Todo - [x] Fix phpstan issues. - [x] Test commonmark (https://packagist.org/packages/league/commonmark) - [x] Test s3 usage (https://packagist.org/packages/league/flysystem-aws-s3-v3) - [x] Parallel testing (Removed paratest package from composer) - [x] Test Redis usage (Updated predis) - [x] Test custom error handler display. - [x] Moved lang folder, check crowdin integration after (Re-do branch). ### Doc Updates - PHP minimum version changed to 8.0.2. - Arguments for the `COMMONMARK_ENVIRONMENT_CONFIGURE` theme event [have changed](https://github.com/BookStackApp/BookStack/commit/9ca088a4e2a952250583d920d2a1cc0aaca51189#diff-123541cd03cdf3e9d796574924e91fd64c3b2016f13a454409b1e71a9e9115deR68-R69). --- <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:28:45 +03:00

OVERLORD closed this issue

2026-02-05 10:28:46 +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#6296