In the rules admin ui I have the path "admin/rules/rules/%" as well as others below that like "admin/rules/rules/%/delete" or "admin/rules/rules/%/edit/%". The problem is now that the paths "below" doesn't get the parent correctly assigned. So they loose the overall context, meaning even the administration menu point is closed - what is weird.
The problem is that "admin/rules/rules/%" gets hidden set to -1 and as a consequence the parent is set to 0 and the context is gone. Is there a cause for doing so?
I tried removing this behavior, which fixed the problem for me and i didn't note any other issues with it. Patch attached.
Note: I had the problem in d6, but I think it applies to d7 as well. The patch is for d6, but applies to d7 too.
Comment | File | Size | Author |
---|---|---|---|
#5 | menu_parents_d7.patch | 904 bytes | fago |
menu_parents_fix.patch | 846 bytes | fago | |
Comments
Comment #1
fagofixing title
Comment #2
mitchell CreditAttribution: mitchell commentedComment #3
apadernoDoes the issue apply all times the parent menu contains %?
Comment #5
fagook, it's not so simple.. ;) Menu items appearing in the tree may not have callbacks as parents, as then wouldn't be displayed any more. I've updated the patch, so it allows just callbacks to have other callbacks as parents.
Updated patch for d7 attached, applies to d6 too (and fixes my problem there).
Comment #6
fagoComment #7
chx CreditAttribution: chx commentedNice catch. These are the hardest to find, these half line fixes...
Comment #8
pwolanin CreditAttribution: pwolanin commentedplease re-roll w/ diff -up, can't readily tell what function this is affecting.
Also, I think I understand the problem, but a little further explanation would help.
Also code comment is > 80 chars.
Comment #9
webchickThree things:
1. This sounds like a big edge case we're likely to break again without a test. So let's add one.
2. Unless mine eyes deceive me, that comment is wider than 80 chars.
3. The comment could also use some work. While I'm sure it's technically accurate, it doesn't tell me "why."
Comment #10
pillarsdotnet CreditAttribution: pillarsdotnet commentedSpelling correction in issue title.