This fixes the regression from D6 on #296693: Hide empty admin categories, doesn't cover menu links, that's increasingly looking like a D8 task, just the items in the blocks.

Attached screenshot demonstrates the bug - disable shortcuts module to see it. Didn't attach an after screenshot, you just won't see the block if there's nothing in it.

Critical because #296693: Hide empty admin categories and #591682: Config pages are effectively hardcoded to some subcategory of 'admin/config' are still marked as critical.

Files: 
CommentFileSizeAuthor
#1 hide_empty_blocks.patch1.11 KBcatch
PASSED: [[SimpleTest]]: [MySQL] 20,426 pass(es). View
hide_empty_blocks.patch1.17 KBcatch
PASSED: [[SimpleTest]]: [MySQL] 20,429 pass(es). View
screenshot_020.png2.07 KBcatch

Comments

catch’s picture

FileSize
1.11 KB
PASSED: [[SimpleTest]]: [MySQL] 20,426 pass(es). View

Moving the condition above the in-line comment.

mikey_p’s picture

Status: Needs review » Reviewed & tested by the community

Works as advertised.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix

Automatically closed -- issue fixed for 2 weeks with no activity.