[TheMilkCarton]

Remove all the $output lines.

In place of the inner-most (the one indented the most) $output, you need an eval to call a template with the correct code. It will most likely contain similar code, unless you're not using an unordered list.

You should probably also remove the function line, and the last }... and include the PHP file you took that code from, since there are most likely a lot of variables it needs to be defined already.

In place of the return $output line, you'd need to use an eval to call another template that'd look something like so.

menu_forumlist

<ul>
$menu_forumlistbits
</ul>

"$menu_forumlistbits" being the variable that the code EVALs in place of that inner-most $output.

A typical eval line looks like so:

PHP Code:

eval('$menu_forumlistbits = "' . fetch_template('menu_forumlistbits') . '";'); 


"menu_forumlistbits" being the template containing each individual list code.

The second eval line (that replaces return $output) would look like so:

PHP Code:

eval('$menu_forumlist = "' . fetch_template('menu_forumlist') . '";'); 


And then you'd need to put $menu_forumlist into whatever template this menu is in.

Also.. you can probably get rid of all that $indent crap too. I think that's just to make the HTML look cleaner.

menu_forumlistbits could be as simple as this:

<li>$forum_link$forum['title_clean']</a></li>

----

OH... haha. You'd also need to change all the $forum_link lines to reflect the correct code, since that code is for the archive and not the normal forums.

.... and btw, in your code example, I don't think "showforum.php" is exactly right... Does that even exist? =p

----

Sorry if that seems like alot... I think that explaining how to do it instead of just giving you code to try will help you become better at these kinds of things, right?

I apologize if I've missed anything... hopefully someone smarter can come in and correct anything I forgot.. I'm quite new with this stuff as well.