There are a few related bugs with the Main Content Block being able to be assigned to the disabled list (as in http://img.skitch.com/20110116-puffkjbfb89tfafhy4y5eskmaq.jpg ) and also the normal block permissions such as restricting by role, or page, appear in the block configuration for this block but should not. Presently if one sets these permissions for visibility they are ignored.

CommentFileSizeAuthor
#1 1029140-v1.patch1.83 KBdfletcher

Comments

dfletcher’s picture

dfletcher commented
StatusFileSize
new 1029140-v1.patch1.83 KB

Possible solution patch attached.

The configuration link is removed entirely for the block that has module == 'system' and delta == 'main'.

Patch also contains a new form validation function that gives an error message if an attempt is made to disable the block. Since validation doesn't pass, the new weights are not saved, so the block snaps back to whatever region it was in before the attempt to disable it.

dfletcher’s picture

dfletcher commented
Status: Active » Needs review

Forgot to flip status to "needs review". Fixing.

catch’s picture

catch
he/him
Primary language English
commented
Priority: Major » Normal

Patch looks fine but I disagree with 'major', these are options that don't do anything, that are hidden behind a link.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Closed (works as designed)

This has been and still is a deliberate design decision.

#516150: Add fallback for main content block rendering
#687686: Prevent hiding the Content Block on particular pages