issue implode() instead of looping around an array, arun

git-svn-id: file:///var/lib/svn/phpfspot/trunk@309 fa6a889d-dae6-447d-9e79-4ba9a3039384

diff --git a/phpfspot.class.php b/phpfspot.class.php
index 61b01c0aa6aa6066c97675991dbb336233ada1a3..d67c7e39dc4f1b1543296ce209ed7e0071a4beaa 100644 (file)
--- a/phpfspot.class.php
+++ b/phpfspot.class.php
@@ -253,13 +253,7 @@ class PHPFSPOT {
             INNER JOIN tags t
                ON pt.tag_id=t.id
             WHERE p.id='". $idx ."'
             INNER JOIN tags t
                ON pt.tag_id=t.id
             WHERE p.id='". $idx ."'

-            AND t.name IN (
-         ";
-         foreach($this->cfg->show_tags as $tag) {
-            $query_str.= "'". $tag ."',";
-         }
-         $query_str = substr($query_str, 0, strlen($query_str)-1);
-         $query_str.= ")";
+            AND t.name IN ('".implode("','",$this->cfg->show_tags)."')";

       }
       else {
          $query_str.= "
       }
       else {
          $query_str.= "


@@ -666,11 +660,7 @@ class PHPFSPOT {
             $query_str.= "AND ". $additional_where_cond ." ";
 
          if(isset($this->cfg->show_tags) && !empty($this->cfg->show_tags)) {
             $query_str.= "AND ". $additional_where_cond ." ";
 
          if(isset($this->cfg->show_tags) && !empty($this->cfg->show_tags)) {

-            $query_str.= "AND t.name IN (";
-            foreach($this->cfg->show_tags as $tag) {
-               $query_str.= "'". $tag ."',";
-            }
-            $query_str = substr($query_str, 0, strlen($query_str)-1) . ")";
+            $query_str.= "AND t.name IN ('".implode("','",$this->cfg->show_tags)."')";

          }
          
          if(isset($order_str))
          }
          
          if(isset($order_str))


@@ -704,11 +694,7 @@ class PHPFSPOT {
                $query_str.= "AND ". $additional_where_cond ." ";
 
             if(isset($this->cfg->show_tags) && !empty($this->cfg->show_tags)) {
                $query_str.= "AND ". $additional_where_cond ." ";
 
             if(isset($this->cfg->show_tags) && !empty($this->cfg->show_tags)) {

-               $query_str.= "AND t.name IN (";
-               foreach($this->cfg->show_tags as $tag) {
-                  $query_str.= "'". $tag ."',";
-               }
-               $query_str = substr($query_str, 0, strlen($query_str)-1) . ")";
+               $query_str.= "AND t.name IN ('".implode("','",$this->cfg->show_tags)."')";

             }
 
             if(isset($order_str))
             }
 
             if(isset($order_str))


@@ -763,11 +749,7 @@ class PHPFSPOT {
                $query_str.= "AND ". $additional_where_cond;
 
             if(isset($this->cfg->show_tags) && !empty($this->cfg->show_tags)) {
                $query_str.= "AND ". $additional_where_cond;
 
             if(isset($this->cfg->show_tags) && !empty($this->cfg->show_tags)) {

-               $query_str.= "AND t.name IN (";
-               foreach($this->cfg->show_tags as $tag) {
-                  $query_str.= "'". $tag ."',";
-               }
-               $query_str = substr($query_str, 0, strlen($query_str)-1) . ")";
+               $query_str.= "AND t.name IN ('".implode("','",$this->cfg->show_tags). "')";

             }
 
             if(isset($order_str))
             }
 
             if(isset($order_str))


@@ -795,11 +777,7 @@ class PHPFSPOT {
          $query_str.= "WHERE ". $additional_where_cond ." ";
 
       if(isset($this->cfg->show_tags) && !empty($this->cfg->show_tags)) {
          $query_str.= "WHERE ". $additional_where_cond ." ";
 
       if(isset($this->cfg->show_tags) && !empty($this->cfg->show_tags)) {

-         $query_str.= "AND t.name IN (";
-         foreach($this->cfg->show_tags as $tag) {
-            $query_str.= "'". $tag ."',";
-         }
-         $query_str = substr($query_str, 0, strlen($query_str)-1) . ")";
+         $query_str.= "AND t.name IN ('".implode("','",$this->cfg->show_tags). "')";

       }
  
       if(isset($order_str))
       }
  
       if(isset($order_str))


@@ -1997,14 +1975,15 @@ class PHPFSPOT {
     */
    public function get_thumb_path($width, $photo)
    {
     */
    public function get_thumb_path($width, $photo)
    {

-      $sub_path = substr($this->getMD5($photo), 0, 2);
+      $md5 = $this->getMD5($photo);
+      $sub_path = substr($md5, 0, 2);

       return $this->cfg->thumb_path
          . "/"
          . $sub_path
          . "/"
          . $width
          . "_"
       return $this->cfg->thumb_path
          . "/"
          . $sub_path
          . "/"
          . $width
          . "_"

-         . $this->getMD5($photo);
+         . $md5;

 
    } // get_thumb_path()
 
 
    } // get_thumb_path()