[RESOLVED] Forum - Breadcrumb - Parent Forums

[dclamp]

Ok i am making a forum, and it allows unlimited sub-forums. How my db is set up, it is has a parentforumID for the parent forum of that forum. (duh).

If there is no parent forum then it is equal to 0. So i need to do a breadcrumb for this, and i tried using a while loop, but it doesnt seem to work.

What am i doing wrong?

if ($forums->forumExists($_GET['forumid'])) {
    $catID = $forums->getCategoryID($_GET['forumid']);
    $catname = $forums->getCategoryName($catID);
    $forumname = $forums->getForumName($_GET['forumid']);
    $breadcrumb = "&gt; <a href='?action=home#cat_{$catID}'>{$catname}</a> ";
    $forumID = mysql_real_escape_string($_GET['forumid']);
    $sql1 = "SELECT `parentforumID` FROM `".$db_prefix."forums` WHERE forumID='{$forumID}' LIMIT 1";
    $query1 = mysql_query($sql1);
    $res1 = mysql_fetch_array($query1);
    if ($res1['parentforumID'] == 0) {
        $breadcrumb .= "&gt; {$forumname}";
    } else {
        $parentID = $res['parentforumID'];
        while ($parentID!=0) {
            $sql = "SELECT     `parentforumID` FROM `".$db_prefix."forums` WHERE forumID='{$parentID}' LIMIT 1";
            $query = mysql_query($sql) or die(mysql_error());
            $res = mysql_fetch_array($query);
            $name = $forums->getForumName($parentID);
            $breadcrumb .= "&gt; <a href='?action=viewforum&forumid={$parentID}'>{$name}</a> ";
            $parentID = $res['parentforumID'];
        }
    }
} else {
    $breadcrumb = "&gt; Error";
} 

[visualAd]

You need to use a recursive function to produce the breadcrumbs.

[dclamp]

can you give me an example?

[visualAd]

It looks nice to use recursion. But the code above looks ok; so what is the problem?

function breadcrumbs($forumId)
{
    $breadCrumbs = '';

    if($parentForum != 0) {
        return $breadCrumbs;
    } else {
        /* add new link here */
        $breadCrumbs .= breadCrumbs($parentForum);
    }
} 

[dclamp]

my code isnt doing anything. It isnt echoing a breadcrumb or any errors.

[visualAd]

Use a few echos to find out exactly how far it is getting.

[dclamp]

ok i tried to do the breadcrumb, and looking at your code, it confuses me...

[dclamp]

ahhh! i figured it out. My code was all correct. I just misnamed a variable

[visualAd]

ahhh! i figured it out. My code was all correct. I just misnamed a variable

That's why you should always ensure you have error reporting set to E_ALL.

[dclamp]