Changeset 2094

Timestamp:

Sep 19, 2007, 5:06:16 AM (17 years ago)

Author:

rvelices

Message:

• fix: quick search error if the query term contained " or '
• optimized sql query in sync_users (called every time on admin page) - a lot faster when #user_cache_categories is big
• added function Template->delete_block_vars (opposite of assign_block_vars)

Location:

branches/branch-1_7

Files:

• admin/include/functions.php (modified) (1 diff)
• include/category_cats.inc.php (modified) (2 diffs)
• include/template.php (modified) (1 diff)
• qsearch.php (modified) (1 diff)

branches/branch-1_7/admin/include/functions.php

@@ -1160 +1160 @@
   {
     $query = '
-SELECT user_id
+SELECT DISTINCT user_id
   FROM '.$table.'
 ;';

branches/branch-1_7/include/category_cats.inc.php

@@ -224 +224 @@
                                   ),
           'DESCRIPTION' =>
-            trigger_event('render_category_literal_description', 
-              trigger_event('render_category_description', 
+            trigger_event('render_category_literal_description',
+              trigger_event('render_category_description',
                 @$category['comment'])),
           'NAME'  => $name,
           )
         );
-        
+
       //plugins need to add/modify sth in this loop ?
       trigger_action('loc_index_category_thumbnail',
@@ -315 +315 @@
 
     $template->assign_var_from_handle('CATEGORIES', 'thumbnails');
-    unset( $template->_tpldata['thumbnails.'] );//maybe write a func for that
+    $template->delete_block_vars('thumbnails', true); // category_default reuse them
   }
   trigger_action('loc_end_index_category_thumbnails', $categories);

branches/branch-1_7/include/template.php

@@ -297 +297 @@
 
   /**
+   * Block-level variable deletion. Deletes the last block iteration.
+   * if all is true - all blocks are removed
+   * return true if a deletion occured
+   */
+  function delete_block_vars($blockname, $all=false)
+    {
+      $blocks = explode('.', $blockname);
+      $blockcount = count($blocks);
+      $str = '$this->_tpldata';
+      for ($i = 0; $i < $blockcount; $i++)
+      {
+        $str .= '[\'' . $blocks[$i] . '.\']';
+        eval('$lastiteration = isset('.$str.') ? sizeof('.$str.')-1:-1;');
+        if ($lastiteration==-1)
+        {
+          return false;
+        }
+        if ($i==$blockcount-1)
+        {
+          break;
+        }
+        $str .= '[' . $lastiteration . ']';
+      }
+
+      if ($all==true or $lastiteration==0)
+      {
+        $str ='unset('.$str.');';
+      }
+      else
+      {
+        $str ='unset('.$str.'['.$lastiteration.']);';
+      }
+      eval($str);
+      return true;
+    }
+
+  /**
    * Root-level variable assignment. Adds to current assignments, overriding
    * any existing variable assignment with the same name.

branches/branch-1_7/qsearch.php

@@ -44 +44 @@
   (rules, last_seen)
   VALUES
-  (\''.serialize($search).'\', NOW() )
+  (\''.addslashes(serialize($search)).'\', NOW() )
 ;';
 pwg_query($query);