see: http://drupal.org/node/249537

CommentFileSizeAuthor
#2 suppress-admin-menu-285020-2.patch1.59 KBpwolanin
#1 suppress-admin-menu-285020-1.patch1.57 KBpwolanin

Comments

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new suppress-admin-menu-285020-1.patch1.57 KB

This patch obviously depends on the final form of the one in the other issue, but works with the patch there in #7.

pwolanin’s picture

pwolanin commented
StatusFileSize
new suppress-admin-menu-285020-2.patch1.59 KB

and simpler code for #8 in the other issue.

pwolanin’s picture

pwolanin commented

I've testing this patch and it works - don't know if you want someone independent to set as rtbc.

redndahead’s picture

redndahead commented
Status: Needs review » Reviewed & tested by the community

Quickly did a manual patch and this seems to be working fine. Can't say if it needs a reroll, but looks like it will be ok.

Thanks pwolanin

merlinofchaos’s picture

merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

This is committed. I think we need a better way to do this in the long term, but Drupal 6 may not ever have it. I'm not really happy about advanced help or admin menu having to know about the other for this to work; that said, there's no way around it, and I agree that having that menu on these pages is bad. So we'll go with this for now, and leave an asterisk on it.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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