problem between advanced_forum and themes
| Project: | Advanced Forum |
| Version: | 6.x-1.0-alpha2 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Jump to:
Hello Michelle,
and thanks for this module I like a lot :)
I am not sure what is exactly the problem with advanced forum, except that it is the same symptom as this old one : http://drupal.org/node/64409
I checked with advanced_forum disabled and there is no problem with assigning blocks on differents themes, whereas when the module is enabled, the bug above happens. Another bug (more easy to detect) is as follows :
When advf is enabled, the offline mode page comes without any theming, only the message appears (no css is included in the html code) ; when advf is disabled, offline mode shows the usual minnelli theme with the message. I tried also to define a customized maintenance page with no effect.
Probably this is due to some wrong theme() call in advanced_forum.module, but since I am not a coder, I was unable to understand where the problem is.
Thank you again,
emmanuel.
#1
I don't have a themed maint page to test, but I'm pretty sure I fixed it. There was some code in there to make the Zen theme work right in D5 that had been just ported as is and merlinofchaos said on IRC that was likely it.
Thanks for the report.
Michelle
#2
Michelle,
can you tell me which version to use so that I test it for you ?
I tried the latest dev release and also the cvs latest forum.module where I have seen your changes (using also d6-compat.inc which is needed) but with no success. By the way, using the latest cvs forum.module together with d6-compat.inc gives me the 'blank page' syndrom ...
If I can help, don't hesitate to ask me for some testing ;)
emmanuel
#3
"using the latest cvs forum.module together with d6-compat.inc"
Erm... Why on Earth would you be doing that? Don't touch your forum.module and don't try and add d6_compat.inc to the D6 branch of advforum. Just download the 6.x-1.x-dev tarball from this morning and try that. If it works, stay on that until then next alpha. I don't normally recommend running the dev but there have only been minor changes since alpha 2 at this point.
Michelle
#4
" Erm... Why on Earth would you be doing that? "
You're right, but I was unsure ... so I just tested. OK, sorry but as I told you, I tested also the dev tarball and the problem still remains the same (although this is not a crucial problem ...).
Emmanuel
#5
Hmm... I thought for sure that would fix it. I'll have to investigate more but I have a bunch of other things I need to do first so it may be a while.
Michelle
#6
Looks like it's working fine in the latest dev.
Michelle
#7
Automatically closed -- issue fixed for two weeks with no activity.
#8
Hi Michelle,
Sorry for the late reply. I confirm that the problem does not appear anymore with the latest dev ;-)
Thanks,
Emmanuel.