[Bb-trac] Re: [bbPress] #620: Front-end changes to enable forum hierarchy

[bbPress]

#620: Front-end changes to enable forum hierarchy
-------------------------+--------------------------------------------------
 Reporter:  sambauers    |        Owner:  sambauers
     Type:  enhancement  |       Status:  new      
 Priority:  normal       |    Milestone:  1.0      
Component:  Front-end    |      Version:  0.8.1    
 Severity:  normal       |   Resolution:           
 Keywords:  has-patch    |  
-------------------------+--------------------------------------------------
Comment (by sambauers):

 I'm not sure how sound this is in the end. There are some problems with
 the WordPress Walker class that meant I had to hack around it about to
 build my extended Walker classes.

 See attachment: ''subforums-display-with-walker-build789.patch''

 This puts the hierarchical forum retrieval into classes as suggested. Some
 files are re-organised to allow for this. I suggest anyone wanting to
 review this should do so on a clean install.

 BB_Cache::get_forums() has been modified to allow for some additional args
 to be passed. Because the Walker class doesn't allow you to specify a root
 id to start at, I had to work out a decent way to limit the returned
 forums.

 This patch also deals with the administration forum list as well, although
 a nasty hack was required to get the forum id into the opening ul tags.

 It's all kind of messy, but I guess now we have two options on the table
 for solving this.

-- 
Ticket URL: <http://trac.bbpress.org/ticket/620#comment:6>
bbPress <http://bbpress.org/>
Innovative forum development