Changeset 252 for branches/release-1_3

Timestamp:

Jan 3, 2004, 10:20:53 PM (20 years ago)

Author:

z0rglub

Message:

• use function get_user_plain_structure, create_user_structure

• using categories.uppercats instead of the calculated structure

• improving queries to retrieve pictures to display

File:

• branches/release-1_3/category.php (modified) (9 diffs)

branches/release-1_3/category.php

@@ -35 +35 @@
 }
 //-------------------------------------------------- access authorization check
-// creating the plain structure : array of all the available categories and
-// their relative informations, see the definition of the function
-// get_plain_structure for further details.
-$page['plain_structure'] = get_plain_structure();
-
 check_cat_id( $_GET['cat'] );
 check_login_authorization();
@@ -47 +42 @@
 }
 //-------------------------------------------------------------- initialization
+// detection of the start picture to display
+if ( !isset( $_GET['start'] )
+     or !is_numeric( $_GET['start'] )
+     or ( is_numeric( $_GET['start'] ) and $_GET['start'] < 0 ) )
+  $page['start'] = 0;
+else
+  $page['start'] = $_GET['start'];
+
+initialize_category();
+
 // creation of the array containing the cat ids to expand in the menu
 // $page['tab_expand'] contains an array with the category ids
@@ -57 +62 @@
     if ( is_numeric( $id ) ) array_push( $page['tab_expand'], $id );
   }
-  if ( is_numeric( $page['cat'] ) )
-  {
-    // the category displayed (in the URL cat=23) must be seen in the menu ->
-    // parent categories must be expanded
-    $parent = $page['plain_structure'][$page['cat']]['id_uppercat'];
-    while ( $parent != '' )
-    {
-      array_push( $page['tab_expand'], $parent );
-      $parent = $page['plain_structure'][$parent]['id_uppercat'];
-    }
-  }
-  $page['expand'] = implode( ',', $page['tab_expand'] );
-}
+}
+if ( is_numeric( $page['cat'] ) )
+{
+  // the category displayed (in the URL cat=23) must be seen in the menu ->
+  // parent categories must be expanded
+  $uppercats = explode( ',', $page['uppercats'] );
+  foreach ( $uppercats as $uppercat ) {
+    array_push( $page['tab_expand'], $uppercat );
+  }
+}
+$page['tab_expand'] = array_unique( $page['tab_expand'] );
+$page['expand'] = implode( ',', $page['tab_expand'] );
 // in case of expanding all authorized cats
 // The $page['expand'] equals 'all' and
@@ -77 +81 @@
   $page['tab_expand'] = array();
   $page['expand'] = 'all';
-}
-// detection of the start picture to display
-if ( !isset( $_GET['start'] )
-     or !is_numeric( $_GET['start'] )
-     or ( is_numeric( $_GET['start'] ) and $_GET['start'] < 0 ) )
-{
-  $page['start'] = 0;
-}
-else
-{
-  $page['start'] = $_GET['start'];
 }
 // Sometimes, a "num" is provided in the URL. It is the number
@@ -99 +92 @@
 }
 // creating the structure of the categories (useful for displaying the menu)
-$page['structure'] = create_structure( '', $user['restrictions'] );
+// creating the plain structure : array of all the available categories and
+// their relative informations, see the definition of the function
+// get_user_plain_structure for further details.
+$page['plain_structure'] = get_user_plain_structure();
+$page['structure'] = create_user_structure( '' );
 $page['structure'] = update_structure( $page['structure'] );
-initialize_category();
 //----------------------------------------------------- template initialization
 $vtp = new VTemplate;
@@ -128 +124 @@
 $icon_long = get_icon( time() - ( $user['short_period'] * 24 * 60 * 60 + 1 ) );
 $vtp->setGlobalVar( $handle, 'icon_long', $icon_long );
-$nb_total_pictures = count_images( $page['structure'] );
+$nb_total_pictures = count_user_total_images();
 $vtp->setGlobalVar( $handle, 'nb_total_pictures',$nb_total_pictures );
+
 //------------------------------------------------------------- categories menu
 // normal categories
@@ -255 +252 @@
   $query.= ',storage_category_id';
   $query.= ' FROM '.PREFIX_TABLE.'images AS i';
-  $query.= ' LEFT JOIN '.PREFIX_TABLE.'image_category AS ic ON id=ic.image_id';
+  $query.=' INNER JOIN '.PREFIX_TABLE.'image_category AS ic ON id=ic.image_id';
   $query.= $page['where'];
   $query.= $conf['order_by'];
@@ -351 +348 @@
 elseif ( ( isset( $page['cat'] )
            and is_numeric( $page['cat'] )
-           and $page['cat_nb_images'] == 0 )
+           and $page['cat_nb_images'] == 0
+           and $page['plain_structure'][$page['cat']]['nb_sub_categories'] > 0)
          or $_GET['cat'] == '' )
 {
@@ -371 +369 @@
     $query.= ' FROM '.PREFIX_TABLE.'categories';
     $query.= ' WHERE id = '.$non_empty_id;
+    $query.= ';';
     $row = mysql_fetch_array( mysql_query( $query ) );
     
     $query = 'SELECT file,tn_ext,storage_category_id';
-    $query.= ' FROM '.PREFIX_TABLE.'images';
-    $query.= ' LEFT JOIN '.PREFIX_TABLE.'image_category ON id = image_id';
+    $query.= ' FROM '.PREFIX_TABLE.'images, '.PREFIX_TABLE.'image_category';
     $query.= ' WHERE category_id = '.$non_empty_id;
+    $query.= ' AND id = image_id';
     // if the category has a representative picture, this is its thumbnail
     // tha will be displayed !