Hi,

I tested out this module, and disabled & uninstalled it. I noticed that the shortcut link 'blocks' (linking to path admin/content/blocks) didn't get removed.
This is actually kinda annoying.

CommentFileSizeAuthor
#4 bean_uninstall_issue.png9.08 KB30equals
#1 bean-remove_shortcuts-1982468-1.patch674 bytes30equals

Comments

30equals’s picture

30equals commented
Status: Active » Needs review
StatusFileSize
new bean-remove_shortcuts-1982468-1.patch674 bytes

Patch!

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
commented
Version: 7.x-1.1 » 7.x-1.x-dev
Category: bug » feature
Status: Needs review » Fixed

Hmm... I am unable to recreate this problem using dev. Can you confirm if its still an issue?

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
commented
Category: feature » bug

whoops... is a bug report.

30equals’s picture

30equals commented
StatusFileSize
new bean_uninstall_issue.png9.08 KB

Yes, i just tested this on a vanilla D7 install with both 7.x-1.1 and 7.x-1.x-dev. Just install bean, disable & uninstall bean and notice that the shortcut 'blocks' still exists. And when you install the module again, another shortcut gets added etc etc... (see attached image).

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
commented
Status: Fixed » Needs review

Sorry meant to put this as postponed not fixed earlier. I'll try to replicate. Patch reviews welcome here...

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
commented
Status: Needs review » Fixed

Ok I was able to replicate and the patch does indeed fix the bug. I'm also noticing that these 'Blocks' shortcuts do not show up using navbar (from Spark) which is why I wasn't seeing the error earlier. Tested on a clean D7 using the normal toolbar I can see the errors and the patch fixed it. Even deleted the multiple instances (like in the image from #4).

So... http://drupalcode.org/project/bean.git/commit/d57a519

Status: Fixed » Closed (fixed)

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