Patch attached.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | panels_page_disabled_4.patch | 695 bytes | pancho |
| panels_page_disabled.patch | 3.94 KB | yched |
Patch attached.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | panels_page_disabled_4.patch | 695 bytes | pancho |
| panels_page_disabled.patch | 3.94 KB | yched |
Comments
Comment #1
panchoOh, this does seem to be critical. The patch bloats the menu generation code even more though. I'm gonna see how I can fix this and #261989 in a code rewrite.
Comment #2
merlinofchaos commentedIt might be a little clearn to do things like: if (!empty($panel_page->disabled)) { continue; }
Comment #3
panchoYeah, I'd also do it this way, if we didn't need a complete rewrite for:
Comment #4
panchoOh, I figured out we need to do this one by one. And this is definitely a bugfix that needs to go into beta5. So sorry for holding this back until now, and thank you very much for catching and fixing this!
Tested and committed merlinofchaos' early-out version: http://drupal.org/cvs?commit=119289.
Comment #5
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.