[PR #262] [MERGED] Entity repo refactor #5618

New Issue

OVERLORD · 2026-02-05T10:12:29+03:00

OVERLORD commented

2026-02-05 10:12:29 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/262
Author: @ssddanbrown
Created: 1/2/2017
Status: ✅ Merged
Merged: 1/2/2017
Merged by: @ssddanbrown

Base: master ← Head: entity_repo_refactor

📝 Commits (5)

7f9de2c Started refactor to merge entity repos
2412782 Refactored search and slug repo components
bab2746 Fixed issue where default user was over-fetched
65796cf Rewrote book children query
8453191 Finished refactor of entity repos

📊 Changes

26 files changed (+1200 additions, -1548 deletions)

View changed files

📝 app/Chapter.php (+5 -2)
📝 app/Entity.php (+6 -4)
📝 app/Http/Controllers/AttachmentController.php (+12 -12)
📝 app/Http/Controllers/BookController.php (+31 -41)
📝 app/Http/Controllers/ChapterController.php (+31 -43)
📝 app/Http/Controllers/HomeController.php (+4 -3)
📝 app/Http/Controllers/ImageController.php (+4 -3)
📝 app/Http/Controllers/PageController.php (+77 -115)
📝 app/Http/Controllers/SearchController.php (+18 -24)
📝 app/Http/Middleware/Authenticate.php (+0 -2)
📝 app/Page.php (+4 -0)
➖ app/Repos/BookRepo.php (+0 -295)
➖ app/Repos/ChapterRepo.php (+0 -226)
📝 app/Repos/EntityRepo.php (+903 -44)
➖ app/Repos/PageRepo.php (+0 -664)
📝 app/Repos/TagRepo.php (+1 -1)
📝 app/Repos/UserRepo.php (+3 -3)
📝 app/Services/ActivityService.php (+1 -1)
📝 app/Services/PermissionService.php (+70 -44)
📝 app/Services/ViewService.php (+7 -8)

...and 6 more files

📄 Description

No description provided

_{🔄 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/262 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 1/2/2017 **Status:** ✅ Merged **Merged:** 1/2/2017 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `entity_repo_refactor` --- ### 📝 Commits (5) - [`7f9de2c`](https://github.com/BookStackApp/BookStack/commit/7f9de2c8ab1b137b1cc9e9a3c1dc969b010e4da4) Started refactor to merge entity repos - [`2412782`](https://github.com/BookStackApp/BookStack/commit/241278226f8a3c7296e900f4d1f3f47d3148ed80) Refactored search and slug repo components - [`bab2746`](https://github.com/BookStackApp/BookStack/commit/bab27462aba7ec4370ce906913b663a13a289110) Fixed issue where default user was over-fetched - [`65796cf`](https://github.com/BookStackApp/BookStack/commit/65796cfc7b7a04f0e61b2ce682c6cfce9d421b23) Rewrote book children query - [`8453191`](https://github.com/BookStackApp/BookStack/commit/8453191dfb029ffa7a43e292eeeca1dc6f6c9077) Finished refactor of entity repos ### 📊 Changes **26 files changed** (+1200 additions, -1548 deletions) <details> <summary>View changed files</summary> 📝 `app/Chapter.php` (+5 -2) 📝 `app/Entity.php` (+6 -4) 📝 `app/Http/Controllers/AttachmentController.php` (+12 -12) 📝 `app/Http/Controllers/BookController.php` (+31 -41) 📝 `app/Http/Controllers/ChapterController.php` (+31 -43) 📝 `app/Http/Controllers/HomeController.php` (+4 -3) 📝 `app/Http/Controllers/ImageController.php` (+4 -3) 📝 `app/Http/Controllers/PageController.php` (+77 -115) 📝 `app/Http/Controllers/SearchController.php` (+18 -24) 📝 `app/Http/Middleware/Authenticate.php` (+0 -2) 📝 `app/Page.php` (+4 -0) ➖ `app/Repos/BookRepo.php` (+0 -295) ➖ `app/Repos/ChapterRepo.php` (+0 -226) 📝 `app/Repos/EntityRepo.php` (+903 -44) ➖ `app/Repos/PageRepo.php` (+0 -664) 📝 `app/Repos/TagRepo.php` (+1 -1) 📝 `app/Repos/UserRepo.php` (+3 -3) 📝 `app/Services/ActivityService.php` (+1 -1) 📝 `app/Services/PermissionService.php` (+70 -44) 📝 `app/Services/ViewService.php` (+7 -8) _...and 6 more files_ </details> ### 📄 Description _No description provided_ --- <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:12:29 +03:00

OVERLORD closed this issue

2026-02-05 10:12:32 +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#5618