The module .info file is missing core's block module dependency.
Patch attached in comments.

CommentFileSizeAuthor
#1 menu_block-1698992-add-core-block-module-dependency.patch634 bytesamontero
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amontero’s picture

amontero
Location Barcelona
CreditAttribution: amontero commented
Status: Active » Needs review
FileSize
menu_block-1698992-add-core-block-module-dependency.patch634 bytes
JohnAlbin’s picture

JohnAlbin
he/they/他
Primary language English
Location Taipei, Taiwan
CreditAttribution: JohnAlbin commented
Status: Needs review » Closed (works as designed)

But menu block module works with Panels. So it actually doesn't require the block module to be enabled.

amontero’s picture

amontero
Location Barcelona
CreditAttribution: amontero commented
Status: Closed (works as designed) » Needs review

Oh. However, it caused uninstallation problems if it was being used together with block module. Maybe some sanity checks at uninstall time would be enough. Can't remember the message, though. I have it in the work computer.

JohnAlbin’s picture

JohnAlbin
he/they/他
Primary language English
Location Taipei, Taiwan
CreditAttribution: JohnAlbin commented
Title: Add core's block module dependency to .info file » hook_uninstall assumes block module dependency

Ah! I see what you mean.

JohnAlbin’s picture

JohnAlbin
he/they/他
Primary language English
Location Taipei, Taiwan
CreditAttribution: JohnAlbin commented
Status: Needs review » Fixed

Fixed! http://drupalcode.org/project/menu_block.git/commitdiff/bd85c63

amontero’s picture

amontero
Location Barcelona
CreditAttribution: amontero commented
Assigned: amontero » Unassigned

Status: Fixed » Closed (fixed)

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