Foreach Loop

[Tefra]

I suggest you one really helpful change to calculate the total entries of each category during the add,delete,move of the download entries. This way you will save the count query.

PHP Code:

// Part One This could be cached in datastore to save even more queries//
// The cache should be re-written each time you manage the categories and after the add/delete/move of a download entry //
$sql = $db->query_read("SELECT * FROM `" . TABLE_PREFIX . "xsdl_cats` WHERE 1");
while ($row = $db->fetch_array($sql))
{
    $catcache[$row['cid']] =  $row;
    $icatcache[$row['par_cat']][$row['cid']] = $row['cid'];
}

$vbulletin->input->clean_array_gpc('g', 'id', TYPE_INT);
$id = (empty($vbulletin->GPC['id'])) 0 : $vbulletin->GPC['id'];

// Part Two Build teh cat list //
foreach ($icatcache[$id] AS $cid)
{
    if (empty($icatcache[$id])) continue;
    
    $cname             = $catcache[$cid]['cat_name'];
    $description     = $catcache[$cid]['cat_description']; 
    $file_loc         = $catcache[$cid]['file_loc']; 
    $screen_loc     = $catcache[$cid]['screen_loc']; 
    $count            = xs_calc_total($cid);
    foreach($icatcache[$cid] AS $subcid) // Grab one level of subcats //
    {
    
        $sub_cname             = $catcache[$subcid]['cat_name'];
        $sub_description     = $catcache[$subcid]['cat_description']; 
        $sub_file_loc         = $catcache[$subcid]['file_loc']; 
        $sub_screen_loc     = $catcache[$subcid]['screen_loc'];
        eval('$downloads_subs_bit .= "' . fetch_template('xs_downloads_subs_bit') . '";'); 
    }    
    eval('$downloads_bit .= "' . fetch_template('xs_downloads_bit') . '";'); 
}

function xs_calc_total($id, $total = 0)
{
    global $icatcache, $catcache;
    
    $total += $catcache[$id]['total']; // Grab the total from the current category //
    
    foreach($icatcache[$id] AS $cid) // Recursive grab the total from all the subs //
    {
        $total += xs_calc_total($cid, $total);
    }
    return $total;
}