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

StatusFileSize
new1.11 KB
PASSED: [[SimpleTest]]: [MySQL] 20,426 pass(es).
[ View ]

Moving the condition above the in-line comment.

Status:Needs review» Reviewed & tested by the community

Works as advertised.

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.