Changeset 22879

Timestamp:

May 26, 2013, 9:56:31 PM (11 years ago)

Author:

rvelices

Message:

bug 2097: wrong number of sub-albums
bug 2098: make number of direct sub-albums available for each user

Location:

trunk

Files:

• include/category_cats.inc.php (modified) (4 diffs)
• include/functions_user.inc.php (modified) (6 diffs)
• install/db/136-database.php (added)
• install/piwigo_structure-mysql.sql (modified) (1 diff)

trunk/include/category_cats.inc.php

@@ -38 +38 @@
     max_date_last,
     count_images,
+    nb_categories,
     count_categories
   FROM '.CATEGORIES_TABLE.' c
@@ -120 +121 @@
     if ($conf['representative_cache_on_subcats'] and $row['user_representative_picture_id'] != $image_id)
     {
-      $user_representative_updates_for[ $user['id'].'#'.$row['id'] ] = $image_id;
+      $user_representative_updates_for[ $row['id'] ] = $image_id;
     }
 
@@ -212 +213 @@
           if ($conf['representative_cache_on_level'])
           {
-            $user_representative_updates_for[ $user['id'].'#'.$category['id'] ] = $image_id;
+            $user_representative_updates_for[ $category['id'] ] = $image_id;
           }
 
@@ -247 +248 @@
   $updates = array();
 
-  foreach ($user_representative_updates_for as $user_cat => $image_id)
-  {
-    list($user_id, $cat_id) = explode('#', $user_cat);
-
-    array_push(
-      $updates,
+  foreach ($user_representative_updates_for as $cat_id => $image_id)
+  {
+    $updates[] =
       array(
-        'user_id' => $user_id,
+        'user_id' => $user['id'],
         'cat_id' => $cat_id,
         'user_representative_picture_id' => $image_id,
-        )
-      );
+        );
   }
 

trunk/include/functions_user.inc.php

@@ -411 +411 @@
           {
             $forbidden_ids[] = $cat['cat_id'];
-            unset( $user_cache_cats[$cat['cat_id']] );
+            remove_computed_category($user_cache_cats, $cat);
           }
         }
@@ -442 +442 @@
         (
           'user_id', 'cat_id',
-          'date_last', 'max_date_last', 'nb_images', 'count_images', 'count_categories'
+          'date_last', 'max_date_last', 'nb_images', 'count_images', 'nb_categories', 'count_categories'
         ),
         $user_cache_cats,
@@ -603 +603 @@
 }
 
-/**
- * compute data of categories branches (one branch only)
- */
-function compute_branch_cat_data(&$cats, &$list_cat_id, &$level, &$ref_level)
-{
-  $date = '';
-  $count_images = 0;
-  $count_categories = 0;
-  do
-  {
-    $cat_id = array_pop($list_cat_id);
-    if (!is_null($cat_id))
-    {
-      // Count images and categories
-      $cats[$cat_id]['count_images'] += $count_images;
-      $cats[$cat_id]['count_categories'] += $count_categories;
-      $count_images = $cats[$cat_id]['count_images'];
-      $count_categories = $cats[$cat_id]['count_categories'] + 1;
-
-      if ((empty($cats[$cat_id]['max_date_last'])) or ($cats[$cat_id]['max_date_last'] < $date))
-      {
-        $cats[$cat_id]['max_date_last'] = $date;
-      }
-      else
-      {
-        $date = $cats[$cat_id]['max_date_last'];
-      }
-      $ref_level = substr_count($cats[$cat_id]['global_rank'], '.') + 1;
-    }
-    else
-    {
-      $ref_level = 0;
-    }
-  } while ($level <= $ref_level);
-
-  // Last cat updating must be added to list for next branch
-  if ($ref_level <> 0)
-  {
-    $list_cat_id[] = $cat_id;
-  }
-}
-
-/**
- * compute data of categories branches
- */
-function compute_categories_data(&$cats)
-{
-  $ref_level = 0;
-  $level = 0;
-  $list_cat_id = array();
-
-  foreach ($cats as $id => $category)
-  {
-    // Compute
-    $level = substr_count($category['global_rank'], '.') + 1;
-    if ($level > $ref_level)
-    {
-      $list_cat_id[] = $id;
-    }
-    else
-    {
-      compute_branch_cat_data($cats, $list_cat_id, $level, $ref_level);
-      $list_cat_id[] = $id;
-    }
-    $ref_level = $level;
-  }
-
-  $level = 1;
-  compute_branch_cat_data($cats, $list_cat_id, $level, $ref_level);
+
+/*update counters with a category removal*/
+function remove_computed_category(&$cats, $cat)
+{
+  if ( isset( $cats[$cat['id_uppercat']] ) )
+  {
+    $parent = & $cats[ $cat['id_uppercat'] ];
+    $parent['nb_categories']--;
+
+    do
+    {
+
+      $parent['count_images'] -= $cat['nb_images'];
+      $parent['count_categories'] -= 1+$cat['count_categories'];
+
+      if ( !isset( $cats[$parent['id_uppercat']] ) )
+        break;
+      $parent = & $cats[$parent['id_uppercat']];
+    }
+    while (true);
+  }
+
+  unset($cats[$cat['cat_id']]);
 }
 
@@ -683 +637 @@
 function get_computed_categories(&$userdata, $filter_days=null)
 {
-  $query = 'SELECT c.id AS cat_id, global_rank';
+  $query = 'SELECT c.id AS cat_id, id_uppercat';
   // Count by date_available to avoid count null
   $query .= ',
@@ -714 +668 @@
   {
     $row['user_id'] = $userdata['id'];
+    $row['nb_categories'] = 0;
     $row['count_categories'] = 0;
     $row['count_images'] = (int)$row['nb_images'];
@@ -722 +677 @@
     }
 
-    $cats += array($row['cat_id'] => $row);
-  }
-  uasort($cats, 'global_rank_compare');
-
-  compute_categories_data($cats);
+    $cats[$row['cat_id']] = $row;
+  }
+
+  foreach ($cats as $cat)
+  {
+    if ( !isset( $cat['id_uppercat'] ) )
+      continue;
+
+    $parent = & $cats[ $cat['id_uppercat'] ];
+    $parent['nb_categories']++;
+
+    do
+    {
+      $parent['count_images'] += $cat['nb_images'];
+      $parent['count_categories']++;
+
+      if ((empty($parent['max_date_last'])) or ($parent['max_date_last'] < $cat['date_last']))
+      {
+        $parent['max_date_last'] = $cat['date_last'];
+      }
+
+      if ( !isset( $parent['id_uppercat'] ) )
+        break;
+      $parent = & $cats[$parent['id_uppercat']];
+    }
+    while (true);
+    unset($parent);
+  }
 
   if ( isset($filter_days) )
   {
-    $cat_tmp = $cats;
-    $cats = array();
-
-    foreach ($cat_tmp as $category)
-    {
-      if (!empty($category['max_date_last']))
-      {
-        // Re-init counters
-        $category['count_categories'] = 0;
-        $category['count_images'] = (int)$category['nb_images'];
-        // Keep category
-        $cats[$category['cat_id']] = $category;
-      }
-    }
-    // Compute a second time
-    compute_categories_data($cats);
+    foreach ($cats as $category)
+    {
+      if (empty($category['max_date_last']))
+      {
+        remove_computed_category($cats, $category);
+      }
+    }
   }
   return $cats;

trunk/install/piwigo_structure-mysql.sql

@@ -373 +373 @@
   `nb_images` mediumint(8) unsigned NOT NULL default '0',
   `count_images` mediumint(8) unsigned default '0',
+  `nb_categories` mediumint(8) unsigned default '0',
   `count_categories` mediumint(8) unsigned default '0',
   `user_representative_picture_id` mediumint(8) unsigned default NULL,