[PR #103] [MERGED] Started work towards adding role view permissions #5593

New Issue

OVERLORD · 2026-02-05T10:11:50+03:00

OVERLORD commented

2026-02-05 10:11:50 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/103
Author: @ssddanbrown
Created: 4/20/2016
Status: ✅ Merged
Merged: 5/2/2016
Merged by: @ssddanbrown

Base: master ← Head: add_role_view_permissions

📝 Commits (9)

6e03078 Started work towards adding role view permissions
043cdea Merge branch 'master' into add_role_view_permissions
ea287eb Started creation of intermediate permission table
ada7c83 Continued with database work for permissions overhaul
a81a567 Rolled out new permissions system throughout application
9a31b83 Worked around create permission quirks
59367b3 Improved permission regen performance by factor of 4
05666ef Added hidden public role to fit with new permissions system
75a4fc9 Major permission naming refactor and database migration cleanup

📊 Changes

53 files changed (+1314 additions, -616 deletions)

View changed files

📝 app/Activity.php (+1 -3)
📝 app/Book.php (+26 -6)
📝 app/Chapter.php (+21 -3)
➕ app/Console/Commands/RegeneratePermissions.php (+51 -0)
📝 app/Console/Kernel.php (+1 -0)
📝 app/EmailConfirmation.php (+7 -6)
📝 app/Entity.php (+36 -6)
📝 app/EntityPermission.php (+3 -6)
📝 app/Http/Controllers/BookController.php (+6 -11)
📝 app/Http/Controllers/ChapterController.php (+5 -7)
📝 app/Http/Controllers/PageController.php (+6 -4)
📝 app/Http/Controllers/PermissionController.php (+3 -0)
📝 app/Http/Controllers/UserController.php (+4 -2)
➕ app/JointPermission.php (+24 -0)
➕ app/Model.php (+19 -0)
📝 app/Ownable.php (+2 -3)
📝 app/Page.php (+33 -7)
📝 app/PageRevision.php (+2 -3)
📝 app/Providers/AuthServiceProvider.php (+2 -1)
📝 app/Providers/CustomFacadeProvider.php (+16 -9)

...and 33 more files

📄 Description

In reference to #101

_{🔄 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/103 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 4/20/2016 **Status:** ✅ Merged **Merged:** 5/2/2016 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `add_role_view_permissions` --- ### 📝 Commits (9) - [`6e03078`](https://github.com/BookStackApp/BookStack/commit/6e03078de3860cc82e30b14ebbcd192a797fd75f) Started work towards adding role view permissions - [`043cdea`](https://github.com/BookStackApp/BookStack/commit/043cdeafb3773288fd7d07ac3dbdbf989ef94bb8) Merge branch 'master' into add_role_view_permissions - [`ea287eb`](https://github.com/BookStackApp/BookStack/commit/ea287ebf86e2cafcab167eb048a04b6f80146d16) Started creation of intermediate permission table - [`ada7c83`](https://github.com/BookStackApp/BookStack/commit/ada7c83e96f35a6b484869d6d27939555e1942f7) Continued with database work for permissions overhaul - [`a81a567`](https://github.com/BookStackApp/BookStack/commit/a81a56706e8be77586631f3619ad84df36c8d84e) Rolled out new permissions system throughout application - [`9a31b83`](https://github.com/BookStackApp/BookStack/commit/9a31b83b2aaa2e5c9e4416cf378bcca74a397bef) Worked around create permission quirks - [`59367b3`](https://github.com/BookStackApp/BookStack/commit/59367b34178739c3aedb4d21e863f3cc58401a2d) Improved permission regen performance by factor of 4 - [`05666ef`](https://github.com/BookStackApp/BookStack/commit/05666efda9a51f245d23fc33e65cd598180e97da) Added hidden public role to fit with new permissions system - [`75a4fc9`](https://github.com/BookStackApp/BookStack/commit/75a4fc905ba3bb6a8b011caf7354b208256841f5) Major permission naming refactor and database migration cleanup ### 📊 Changes **53 files changed** (+1314 additions, -616 deletions) <details> <summary>View changed files</summary> 📝 `app/Activity.php` (+1 -3) 📝 `app/Book.php` (+26 -6) 📝 `app/Chapter.php` (+21 -3) ➕ `app/Console/Commands/RegeneratePermissions.php` (+51 -0) 📝 `app/Console/Kernel.php` (+1 -0) 📝 `app/EmailConfirmation.php` (+7 -6) 📝 `app/Entity.php` (+36 -6) 📝 `app/EntityPermission.php` (+3 -6) 📝 `app/Http/Controllers/BookController.php` (+6 -11) 📝 `app/Http/Controllers/ChapterController.php` (+5 -7) 📝 `app/Http/Controllers/PageController.php` (+6 -4) 📝 `app/Http/Controllers/PermissionController.php` (+3 -0) 📝 `app/Http/Controllers/UserController.php` (+4 -2) ➕ `app/JointPermission.php` (+24 -0) ➕ `app/Model.php` (+19 -0) 📝 `app/Ownable.php` (+2 -3) 📝 `app/Page.php` (+33 -7) 📝 `app/PageRevision.php` (+2 -3) 📝 `app/Providers/AuthServiceProvider.php` (+2 -1) 📝 `app/Providers/CustomFacadeProvider.php` (+16 -9) _...and 33 more files_ </details> ### 📄 Description In reference to #101 --- <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:11:50 +03:00

OVERLORD closed this issue

2026-02-05 10:11:52 +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#5593