The links will be gone, but if someone can guess, they would be able to add/delete/edit blocks/sections and so on.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | 3109476-10.patch | 15.63 KB | swentel |
The links will be gone, but if someone can guess, they would be able to add/delete/edit blocks/sections and so on.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | 3109476-10.patch | 15.63 KB | swentel |
Comments
Comment #2
swentel commentedThis patch uses a technique which is also used by Layout Builder Perms, so they could end up clashing with each other when used together.
It swaps the LayoutBuilderAccessCheck class.
I've been wondering though whether it wouldn't make more sense to override OverridesSectionStorage (which then might clash with layout builder at, but since I maintain both, I can easily write a workaround for that).
Comment #3
swentel commentedAnd now with patch
Comment #4
swentel commentedWe now have tests. Uploaded failing and pass test patches.
Still needs check for all other routes.
Comment #5
swentel commentedremoved the debug drupal_set_message call
Comment #7
swentel commentedNew patch with a more elegant solution: adding a new access check so both are chained! Credits to Tim for this one! :)
Comment #8
swentel commentedMore route checking, LOCKED_SECTION_BLOCK_MOVE is the last one
Comment #9
swentel commentedWith LOCKED_SECTION_BLOCK_MOVE now too ... manually tested to be sure as I don't really call a route right now in the test.
Comment #10
swentel commentedAnother check for removal of section
Comment #12
swentel commentedDone
Comment #13
swentel commented