Changeset 21817

Timestamp:

Mar 24, 2013, 7:46:35 AM (11 years ago)

Author:

rvelices

Message:

feature 2836: display the number of comments/tags in the menubar

Location:

trunk

Files:

• admin/include/functions.php (modified) (4 diffs)
• include/functions.inc.php (modified) (1 diff)
• include/functions_comment.inc.php (modified) (5 diffs)
• include/functions_tag.inc.php (modified) (1 diff)
• include/menubar.inc.php (modified) (4 diffs)
• themes/default/template/menubar_menu.tpl (modified) (1 diff)

trunk/admin/include/functions.php

@@ -1368 +1368 @@
     $inserts
     );
+  invalidate_user_cache_nb_tags();
 }
 
@@ -1398 +1399 @@
 ;';
   pwg_query($query);
+
+  invalidate_user_cache_nb_tags();
 }
 
@@ -1487 +1490 @@
         );
     }
+
+    invalidate_user_cache_nb_tags();
   }
 }
@@ -1693 +1698 @@
   }
   trigger_action('invalidate_user_cache', $full);
+}
+
+
+function invalidate_user_cache_nb_tags()
+{
+  global $user;
+  unset($user['nb_available_tags']);
+  $query = '
+UPDATE '.USER_CACHE_TABLE.'
+  SET nb_available_tags = NULL';
+  pwg_query($query);
 }
 

trunk/include/functions.inc.php

@@ -1712 +1712 @@
   }
 }
+
+/** returns the number of available comments for the connected user */
+function get_nb_available_comments()
+{
+  global $user;
+  if (!isset($user['nb_available_comments']))
+  {
+    $where = array();
+    if ( !is_admin() )
+      $where[] = 'validated=\'true\'';
+    $where[] = get_sql_condition_FandF
+      (
+        array
+          (
+            'forbidden_categories' => 'category_id',
+            'visible_categories' => 'category_id',
+            'visible_images' => 'ic.image_id'
+          ),
+        '', true
+      );
+
+    $query = '
+SELECT COUNT(DISTINCT(com.id))
+  FROM '.IMAGE_CATEGORY_TABLE.' AS ic
+    INNER JOIN '.COMMENTS_TABLE.' AS com
+    ON ic.image_id = com.image_id
+  WHERE '.implode('
+    AND ', $where);
+    list($user['nb_available_comments']) = pwg_db_fetch_row(pwg_query($query));
+
+    single_update(USER_CACHE_TABLE, 
+      array('nb_available_comments'=>$user['nb_available_comments']),
+      array('user_id'=>$user['id'])
+      );
+  }
+  return $user['nb_available_comments'];
+}
+
 ?>

trunk/include/functions_comment.inc.php

@@ -221 +221 @@
   )
 ';
-
     pwg_query($query);
-
     $comm['id'] = pwg_db_insert_id(COMMENTS_TABLE);
+
+    invalidate_user_cache_nb_comments();
 
     if ( ($conf['email_admin_on_comment'] && 'validate' == $comment_action)
@@ -284 +284 @@
 $user_where_clause.'
 ;';
-  $result = pwg_query($query);
-  
-  if ($result) 
-  {
+  
+  if ( pwg_db_changes(pwg_query($query)) )
+  {
+    invalidate_user_cache_nb_comments();
+
     email_admin('delete', 
                 array('author' => $GLOBALS['user']['username'],
                       'comment_id' => $comment_id
                   ));
-  }
-  
-  trigger_action('user_comment_deletion', $comment_id);
+    trigger_action('user_comment_deletion', $comment_id);
+  }
 }
 
@@ -345 +345 @@
     if (!url_check_format($comment['website_url']))
     {
-      array_push($page['errors'], l10n('Your website URL is invalid'));
+      $page['errors'][] = l10n('Your website URL is invalid');
       $comment_action='reject';
     }
@@ -394 +394 @@
     }
     // just mail admin
-    else if ($result)
+    elseif ($result)
     {
       email_admin('edit', array('author' => $GLOBALS['user']['username'],
@@ -482 +482 @@
   pwg_query($query);
   
+  invalidate_user_cache_nb_comments();
   trigger_action('user_comment_validation', $comment_id);
 }
+
+
+function invalidate_user_cache_nb_comments()
+{
+  global $user;
+  unset($user['nb_available_comments']);
+  $query = '
+UPDATE '.USER_CACHE_TABLE.'
+  SET nb_available_comments = NULL';
+  pwg_query($query);
+}
+
 ?>

trunk/include/functions_tag.inc.php

@@ -23 +23 @@
 
 
+/** returns the number of available tags for the connected user */
+function get_nb_available_tags()
+{
+  global $user;
+  if (!isset($user['nb_available_tags']))
+  {
+    $user['nb_available_tags'] = count(get_available_tags());
+    single_update(USER_CACHE_TABLE, 
+      array('nb_available_tags'=>$user['nb_available_tags']),
+      array('user_id'=>$user['id'])
+      );
+  }
+  return $user['nb_available_tags'];
+}
+
 /**
  * Tags available. Each return tag is represented as an array with its id,

trunk/include/menubar.inc.php

@@ -192 +192 @@
     }
 
+    $block->data['recent_pics'] =
+      array(
+        'URL' => make_index_url(array('section' => 'recent_pics')),
+        'TITLE' => l10n('display most recent photos'),
+        'NAME' => l10n('Recent photos'),
+      );
+
+    $block->data['recent_cats'] =
+      array(
+        'URL' => make_index_url(array('section' => 'recent_cats')),
+        'TITLE' => l10n('display recently updated albums'),
+        'NAME' => l10n('Recent albums'),
+      );
+
     $block->data['random'] =
       array(
@@ -199 +213 @@
         'REL'=> 'rel="nofollow"'
       );
-
-    $block->data['recent_pics'] =
-      array(
-        'URL' => make_index_url(array('section' => 'recent_pics')),
-        'TITLE' => l10n('display most recent photos'),
-        'NAME' => l10n('Recent photos'),
-      );
-
-    $block->data['recent_cats'] =
-      array(
-        'URL' => make_index_url(array('section' => 'recent_cats')),
-        'TITLE' => l10n('display recently updated albums'),
-        'NAME' => l10n('Recent albums'),
-      );
-
 
     $block->data['calendar'] =
@@ -247 +246 @@
         'NAME' => l10n('Tags'),
         'URL'=> get_root_url().'tags.php',
+        'COUNTER' => get_nb_available_tags(),
       );
 
@@ -266 +266 @@
           'NAME'=>l10n('Comments'),
           'URL'=> get_root_url().'comments.php',
+          'COUNTER' => get_nb_available_comments(),
         );
     }

trunk/themes/default/template/menubar_menu.tpl

@@ -12 +12 @@
         {foreach from=$block->data item=link}
                 {if is_array($link)}
-                        <li><a href="{$link.URL}" title="{$link.TITLE}"{if isset($link.REL)} {$link.REL}{/if}>{$link.NAME}</a></li>
+                        <li><a href="{$link.URL}" title="{$link.TITLE}"{if isset($link.REL)} {$link.REL}{/if}>{$link.NAME}</a>{if isset($link.COUNTER)} ({$link.COUNTER}){/if}</li>
                 {/if}
         {/foreach}