Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pwolanin’s picture

Status: Active » Needs review
FileSize
1.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

and simpler code for #8 in the other issue.

pwolanin’s picture

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

redndahead’s picture

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

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

Status: Fixed » Closed (fixed)

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