Merge topic list changes in AF into overridden forum bits
Michelle - June 1, 2009 - 22:04
| Project: | Advanced Forum |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Michelle |
| Status: | closed |
Jump to:
Description
To make AF compatable with nodecomment, I needed to override forum_get_topics. That gives AF more control over the topic listing and so I can take out some of the hackiness I had to do for the table headers.
Michelle
#1
Another note... Can move the views counting into the original query. Probably other things can be consolodated as well. Consider taking over the whole preprocess.
Michelle
#2
This problem will go away once #484052: Make a view for the topic list page is in.
Michelle
#3
Changed my mind. I'd like AF to degrade nicely if views/nodecomment isn't installed so it's still worth taking the time to clean this up.
Michelle
#4
#5
Automatically closed -- issue fixed for 2 weeks with no activity.