optimizations (php only)

- removed unnecessary call to array_unique in get_image_ids_for_tags
- put back in smarty exec time counting (debug mode) for each included template
- prefix some smarty modifiers with @ in some templates
- do not call format_date (still very slow due to mktime and date even after prev commit) from category_default.inc.php - if used in templates format_date can be used as a smarty modifier

git-svn-id: http://piwigo.org/svn/trunk@3120 68402e56-0260-453c-a942-63ccdbb3a9ee

3 files changed, 29 insertions, 27 deletions

diff --git a/include/category_default.inc.php b/include/category_default.inc.php
index afb195a3a..da2386445 100644
--- a/include/category_default.inc.php
+++ b/include/category_default.inc.php
@@ -106,28 +106,25 @@ foreach ($pictures as $row)
 
   $tpl_var =
     array(
-      'ID'            => $row['id'],
-      'TN_SRC'         => $thumbnail_url,
-      'TN_ALT'     => $row['file'],
-      'TN_TITLE'   => get_thumbnail_title($row),
-      'ICON_TS'      => get_icon($row['date_available']),
-      'URL'    => $url,
+      'ID'        => $row['id'],
+      'TN_SRC'    => $thumbnail_url,
+      'TN_ALT'    => $row['file'],
+      'TN_TITLE'  => get_thumbnail_title($row),
+      'ICON_TS'   => get_icon($row['date_available']),
+      'URL'       => $url,
 
    /* Fields for template-extension usage */
       'FILE_PATH' => $row['path'],
-      'FILE_POSTED' => format_date($row['date_available'], 'mysql_datetime'),
-      'FILE_CREATED' => (empty($row['date_creation'])) ? 
-        '-': format_date($row['date_creation'], 'mysql_date'),
-      'FILE_DESC' => (empty($row['comment'])) ? '-' : $row['comment'],
-      'FILE_AUTHOR' => (empty($row['author'])) ? '-' : $row['author'],
+      'FILE_POSTED' => $row['date_available'],
+      'FILE_CREATED' => $row['date_creation'],
+      'FILE_DESC' => $row['comment'],
+      'FILE_AUTHOR' => $row['author'],
       'FILE_HIT' => $row['hit'],
-      'FILE_SIZE' => (empty($row['filesize'])) ? '-' : $row['filesize'],
-      'FILE_WIDTH' => (empty($row['width'])) ? '-' : $row['width'],
-      'FILE_HEIGHT' => (empty($row['height'])) ? '-' : $row['height'],
-      'FILE_METADATE' => (empty($row['date_metadata_update'])) ? 
-        '-': format_date($row['date_metadata_update'], 'mysql_date'),
-      'FILE_RATE' => (empty($row['rate'])) ? '-' : $row['rate'], 
-      'FILE_HAS_HD' => ($row['has_high'] and $user['enabled_high']=='true') ? 
+      'FILE_SIZE' => $row['filesize'],
+      'FILE_WIDTH' => $row['width'],
+      'FILE_HEIGHT' => $row['height'],
+      'FILE_METADATE' => $row['date_metadata_update'],
+      'FILE_HAS_HD' => ($row['has_high'] and $user['enabled_high']=='true') ?
                 true:false, /* lack of include/functions_picture.inc.php */
     );
 
@@ -184,4 +181,4 @@ trigger_action('loc_end_index_thumbnails', $pictures);
 $template->assign_var_from_handle('THUMBNAILS', 'index_thumbnails');
 
 pwg_debug('end include/category_default.inc.php');
-?>
+?>
\ No newline at end of file
diff --git a/include/functions_tag.inc.php b/include/functions_tag.inc.php
index fdb1674d7..033967ca4 100644
--- a/include/functions_tag.inc.php
+++ b/include/functions_tag.inc.php
@@ -189,8 +189,7 @@ SELECT image_id
       {
         $items = array_intersect($items, $images);
       }
-
-      return array_unique($items);
+      return $items;
       break;
     }
     case 'OR':
diff --git a/include/smarty/libs/Smarty.class.php b/include/smarty/libs/Smarty.class.php
index 95ec67bab..8e56346f9 100644
--- a/include/smarty/libs/Smarty.class.php
+++ b/include/smarty/libs/Smarty.class.php
@@ -20,7 +20,7 @@
  *
  * For questions, help, comments, discussion, etc., please join the
  * Smarty mailing list. Send a blank e-mail to
- * smarty-discussion-subscribe@googlegroups.com 
+ * smarty-discussion-subscribe@googlegroups.com
  *
  * @link http://www.smarty.net/
  * @copyright 2001-2005 New Digital Group, Inc.
@@ -1057,7 +1057,7 @@ class Smarty
         } else {
             // var non-existant, return valid reference
             $_tmp = null;
-            return $_tmp;   
+            return $_tmp;
         }
     }
 
@@ -1116,7 +1116,7 @@ class Smarty
     function fetch($resource_name, $cache_id = null, $compile_id = null, $display = false)
     {
         static $_cache_info = array();
-        
+
         $_smarty_old_error_level = $this->debugging ? error_reporting() : error_reporting(isset($this->error_reporting)
                ? $this->error_reporting : error_reporting() & ~E_NOTICE);
 
@@ -1303,6 +1303,12 @@ class Smarty
             error_reporting($_smarty_old_error_level);
             return;
         } else {
+            if ($this->debugging) {
+                // capture time for debugging info
+                $_params = array();
+                require_once(SMARTY_CORE_DIR . 'core.get_microtime.php');
+                $this->_smarty_debug_info[$_included_tpls_idx]['exec_time'] = (smarty_core_get_microtime($_params, $this) - $_debug_start_time);
+            }
             error_reporting($_smarty_old_error_level);
             if (isset($_smarty_results)) { return $_smarty_results; }
         }
@@ -1932,10 +1938,10 @@ class Smarty
     {
         return eval($code);
     }
-    
+
     /**
      * Extracts the filter name from the given callback
-     * 
+     *
      * @param callback $function
      * @return string
      */
@@ -1950,7 +1956,7 @@ class Smarty
 			return $function;
 		}
 	}
-    
+
     /**#@-*/
 
 }