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

Comments

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

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

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented
Status: Active » Closed (won't fix)

This problem will go away once #484052: Make a view for the topic list page is in.

Michelle

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented
Title: Remove hacky table header code » Merge topic list changes in AF into overridden forum bits
Status: Closed (won't fix) » Active

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

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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