The links will be gone, but if someone can guess, they would be able to add/delete/edit blocks/sections and so on.

Comments

swentel created an issue. See original summary.

swentel’s picture

Status: Active » Needs review

This 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).

swentel’s picture

StatusFileSize
new10.5 KB

And now with patch

swentel’s picture

StatusFileSize
new4.34 KB
new13.21 KB

We now have tests. Uploaded failing and pass test patches.
Still needs check for all other routes.

swentel’s picture

StatusFileSize
new13.15 KB

removed the debug drupal_set_message call

swentel’s picture

StatusFileSize
new13.27 KB

New patch with a more elegant solution: adding a new access check so both are chained! Credits to Tim for this one! :)

swentel’s picture

StatusFileSize
new15.24 KB

More route checking, LOCKED_SECTION_BLOCK_MOVE is the last one

swentel’s picture

StatusFileSize
new15.55 KB

With LOCKED_SECTION_BLOCK_MOVE now too ... manually tested to be sure as I don't really call a route right now in the test.

swentel’s picture

StatusFileSize
new15.63 KB

Another check for removal of section

  • swentel committed 2fa1a7a on 8.x-1.x
    Issue #3109476: protect route access
    
swentel’s picture

Status: Needs review » Fixed

Done

swentel’s picture

Status: Fixed » Closed (fixed)