Changeset 1860

Timestamp:

Feb 27, 2007, 12:52:22 AM (17 years ago)

Author:

rvelices

Message:

• bug 654: sql error on user comment (since my commit 1849)
• languages: english corrections + keep lang files sorted by key
• admin multi view correction: language was not always properly changed
• refactor function get_computed_categories (with rub's blessing)

Location:

trunk

Files:

• include/filter.inc.php (modified) (2 diffs)
• include/functions_user.inc.php (modified) (7 diffs)
• include/picture_comment.inc.php (modified) (1 diff)
• language/en_UK.iso-8859-1/common.lang.php (modified) (4 diffs)
• language/fr_FR.iso-8859-1/common.lang.php (modified) (6 diffs)
• plugins/admin_multi_view/is_admin.inc.php (modified) (1 diff)
• template/yoga/index.tpl (modified) (1 diff)

trunk/include/filter.inc.php

@@ -4 +4 @@
 // | Copyright (C) 2006-2007 PhpWebGallery Team - http://phpwebgallery.net |
 // +-----------------------------------------------------------------------+
-// | branch        : BSF (Best So Far)
 // | file          : $Id$
 // | last update   : $Date$
@@ -76 +75 @@
     // Need to compute dats
     $filter['check_key'] = get_filter_check_key();
-    $filter['categories'] = get_computed_categories($user['id'], $user['forbidden_categories'], true, $filter['recent_period']);
+    $filter['categories'] = get_computed_categories($user, (int)$filter['recent_period']);
 
     $filter['visible_categories'] = implode(',', array_keys($filter['categories']));

trunk/include/functions_user.inc.php

@@ -273 +273 @@
         calculate_permissions($userdata['id'], $userdata['status']);
 
-      update_user_cache_categories($userdata['id'], $userdata['forbidden_categories']);
+      update_user_cache_categories($userdata);
 
       // Set need update are done
@@ -534 +534 @@
  * get computed array of categories
  *
- * @param int user_id
- * @param list user_forbidden_categories
- * @param bool filter_enabled
- * @param int recent_period
+ * @param array userdata
+ * @param int filter_days number of recent days to filter on or null
  * @return array
  */
-function get_computed_categories($user_id, $user_forbidden_categories, $filter_enabled, $recent_period = 0)
-{
-  $query = '
-SELECT
-  c.id cat_id,
-  date_last max_date_last,
-  nb_images count_images,
-  global_rank';
-
-  if (!$filter_enabled)
-  {
-    $query.= '
-FROM '.CATEGORIES_TABLE.' as c';
+function get_computed_categories($userdata, $filter_days=null)
+{
+  $group_by = '';
+
+  $query = 'SELECT c.id cat_id, global_rank';
+  if ( !isset($filter_days) )
+  {
+    $query .= ',
+    date_last cat_date_last,
+    nb_images cat_nb_images
+  FROM '.CATEGORIES_TABLE.' as c';
   }
   else
   {
     // Count by date_available to avoid count null
-    $query.= ',
-  count(date_available) filtered_count_images,
-  max(date_available) max_date_available
-FROM '.CATEGORIES_TABLE.' as c
+    $query .= ',
+    MAX(date_available) cat_date_last,
+    COUNT(date_available) cat_nb_images
+  FROM '.CATEGORIES_TABLE.' as c
     LEFT JOIN '.IMAGE_CATEGORY_TABLE.' AS ic ON ic.category_id = c.id
     LEFT JOIN '.IMAGES_TABLE.' AS i
-      ON ic.image_id = i.id AND 
-          i.date_available > SUBDATE(CURRENT_DATE,INTERVAL '.$recent_period.' DAY)';
-  }
-
-  if ($user_forbidden_categories != '')
+      ON ic.image_id = i.id AND
+          i.date_available > SUBDATE(CURRENT_DATE,INTERVAL '.$filter_days.' DAY)';
+    $group_by = 'c.id';
+  }
+
+  if ( !empty($userdata['forbidden_categories']) )
   {
     $query.= '
-WHERE
-  c.id NOT IN ('.$user_forbidden_categories.')';
-  }
-
-  if ($filter_enabled)
+  WHERE c.id NOT IN ('.$userdata['forbidden_categories'].')';
+  }
+
+  if ( !empty($group_by) )
   {
     $query.= '
-GROUP BY
-  c.id';
-  }
-  $query.= ';';
+  GROUP BY '.$group_by;
+  }
 
   $result = pwg_query($query);
@@ -587 +581 @@
   while ($row = mysql_fetch_assoc($result))
   {
-    $row['user_id'] = $user_id;
+    $row['user_id'] = $userdata['id'];
     $row['count_categories'] = 0;
-    if ($filter_enabled)
-    {
-      $row['nb_images'] = $row['filtered_count_images'];
-      $row['count_images'] = $row['filtered_count_images'];
-      $row['max_date_last'] = $row['max_date_available'];
-    }
+    $row['count_images'] = $row['cat_nb_images'];
+    $row['max_date_last'] = $row['cat_date_last'];
+
     $cats += array($row['cat_id'] => $row);
   }
@@ -601 +592 @@
   compute_categories_data($cats);
 
-  if ($filter_enabled)
+  if ( isset($filter_days) )
   {
     $cat_tmp = $cats;
     $cats = array();
-  
+
     foreach ($cat_tmp as $category)
     {
@@ -612 +603 @@
         // Re-init counters
         $category['count_categories'] = 0;
-        $category['nb_images'] = $category['filtered_count_images'];
-        $category['count_images'] = $category['filtered_count_images'];
+        $category['count_images'] = $category['cat_nb_images'];
+        // next line for update_cats_with_filtered_data
+        $category['nb_images'] = $category['cat_nb_images'];
         // Keep category
         $cats[$category['cat_id']] = $category;
-        
       }
     }
@@ -622 +613 @@
     compute_categories_data($cats);
   }
-
   return $cats;
 }
@@ -629 +619 @@
  * update data of user_cache_categories
  *
- * @param int user_id
- * @param list user_forbidden_categories
- * @param bool filter_enabled
+ * @param array userdata
  * @return null
  */
-function update_user_cache_categories($user_id, $user_forbidden_categories)
+function update_user_cache_categories($userdata)
 {
   // delete user cache
   $query = '
 DELETE FROM '.USER_CACHE_CATEGORIES_TABLE.'
-  WHERE user_id = '.$user_id.'
+  WHERE user_id = '.$userdata['id'].'
 ;';
   pwg_query($query);
 
-  $cats = get_computed_categories($user_id, $user_forbidden_categories, false);
+  $cats = get_computed_categories($userdata, null);
 
   include_once(PHPWG_ROOT_PATH.'admin/include/functions.php');

trunk/include/picture_comment.inc.php

@@ -57 +57 @@
   include_once(PHPWG_ROOT_PATH.'include/functions_comment.inc.php');
   
-  $comment_action = insert_user_comment( 
-      $comm, @$_POST['key'], $page['image_id'], $infos
-    );
+  $comment_action = insert_user_comment($comm, @$_POST['key'], $infos );
 
   switch ($comment_action)

trunk/language/en_UK.iso-8859-1/common.lang.php

@@ -490 +490 @@
 $lang['maxwidth'] = 'Maximum width of the pictures';
 $lang['maxwidth_error'] = 'Maximum width must be a number superior to 50';
-$lang['flat_hint'] = 'flat display elements of categories and sub-categories';
-$lang['start_filter_hint'] = 'displays only recent elements';
-$lang['stop_filter_hint'] = 'return to display all elements';
 $lang['mode_created_hint'] = 'displays a calendar by creation date';
+$lang['mode_flat_hint'] = 'displays all elements in all sub-categories';
 $lang['mode_normal_hint'] = 'return to normal view mode';
-$lang['mode_posted_hint'] = 'displays a calendar by date posted';
+$lang['mode_posted_hint'] = 'displays a calendar by posted date';
 $lang['month'][10] = 'October';
 $lang['month'][11] = 'November';
@@ -521 +519 @@
 $lang['no_category'] = 'Home';
 $lang['no_rate'] = 'no rate';
+$lang['note_filter_day'] = 'Only displays elements posted within the last %s day.';
+$lang['note_filter_days'] = 'Only displays elements posted within the last %s days.';
 $lang['password updated'] = 'password updated';
 $lang['periods_error'] = 'Recent period must be a positive integer value';
@@ -580 +580 @@
 $lang['special_categories'] = 'Specials';
 $lang['sql_queries_in'] = 'SQL queries in';
+$lang['start_filter_hint'] = 'displays only recently posted elements';
+$lang['stop_filter_hint'] = 'return to the display of all elements';
 $lang['submit'] = 'Submit';
 $lang['the beginning'] = 'the beginning';
@@ -612 +614 @@
 $lang['w_month'] = 'Month';
 $lang['yes'] = 'Yes';
-$lang['note_filter_day'] = 'The whole of the elements are filtered in order to diplay the recent elements of less %s day.';
-$lang['note_filter_days'] = 'The whole of the elements are filtered in order to diplay the recent elements of less %s days.';
 $lang['page_end'] = 'Page bottom';
-
 ?>

trunk/language/fr_FR.iso-8859-1/common.lang.php

@@ -490 +490 @@
 $lang['maxwidth'] = 'Largeur maximum des images';
 $lang['maxwidth_error'] = 'La largeur des images doit être supérieure à 50';
-$lang['flat_hint'] = 'affiche à plat les éléments des catégories et des sous-catégories';
-$lang['start_filter_hint'] = 'afficher que les éléments récents';
-$lang['stop_filter_hint'] = 'retourner à l\'affichage de tous les éléments';
 $lang['mode_created_hint'] = 'afficher un calendrier par date de création';
+$lang['mode_flat_hint'] = 'afficher à plat les éléments des catégories et des sous-catégories';
 $lang['mode_normal_hint'] = 'retourner à la vue normale';
 $lang['mode_posted_hint'] = 'afficher un calendrier par date d\'ajout';
@@ -521 +519 @@
 $lang['no_category'] = 'Accueil';
 $lang['no_rate'] = 'pas de note';
+$lang['note_filter_day'] = 'L\'ensemble des éléments est filtré pour n\'afficher que les éléments récents de moins de %d jour.';
+$lang['note_filter_days'] = 'L\'ensemble des éléments est filtré pour n\'afficher que les éléments récents de moins de %d jours.';
 $lang['password updated'] = 'mot de passe mis à jour';
 $lang['periods_error'] = 'La période de nouveauté doit être un entier positif';
@@ -558 +558 @@
 $lang['search_categories'] = 'Rechercher dans les catégories';
 $lang['search_date'] = 'Recherche par date';
-$lang['search_date_creation'] = 'Création';
 $lang['search_date_from'] = 'Date';
 $lang['search_date_to'] = 'Date de fin';
@@ -564 +563 @@
 $lang['search_descending'] = 'Décroissant';
 $lang['search_keywords'] = 'Recherche de mot';
-$lang['sear ch_mode_and'] = 'Rechercher tous les mots';
+$lang['search_mode_and'] = 'Rechercher tous les mots';
 $lang['search_mode_or'] = 'Rechercher un des mots';
 $lang['search_one_clause_at_least'] = 'Requête vide. Aucun critère fourni.';
@@ -581 +580 @@
 $lang['special_categories'] = 'Spéciales';
 $lang['sql_queries_in'] = 'requêtes SQL en';
+$lang['start_filter_hint'] = 'afficher que les éléments récents';
+$lang['stop_filter_hint'] = 'retourner à l\'affichage de tous les éléments';
 $lang['submit'] = 'Valider';
 $lang['the beginning'] = 'le début';
@@ -613 +614 @@
 $lang['w_month'] = 'Mois';
 $lang['yes'] = 'Oui';
-$lang['note_filter_day'] = 'L\'ensemble des éléments est filtré pour n\'afficher que les éléments récents de moins de %d jour.';
-$lang['note_filter_days'] = 'L\'ensemble des éléments est filtré pour n\'afficher que les éléments récents de moins de %d jours.';
 $lang['page_end'] = 'Bas de page';
-
 ?>

trunk/plugins/admin_multi_view/is_admin.inc.php

@@ -14 +14 @@
   }
   $lang = pwg_get_session_var( 'multiview_lang', '' );
-  if ( !empty($theme) )
+  if ( !empty($lang) )
   {
     $user['language'] = $lang;

trunk/template/yoga/index.tpl

@@ -33 +33 @@
 
       <!-- BEGIN flat -->
-      <li><a href="{flat.URL}" title="{lang:flat_hint}" rel="nofollow"><img src="{pwg_root}{themeconf:icon_dir}/flat.png" class="button" alt="{lang:flat_hint}"></a></li>
+      <li><a href="{flat.URL}" title="{lang:mode_flat_hint}" rel="nofollow"><img src="{pwg_root}{themeconf:icon_dir}/flat.png" class="button" alt="{lang:flat_hint}"></a></li>
       <!-- END flat -->