Changeset 13021

Timestamp:

Feb 2, 2012, 9:59:41 PM (12 years ago)

Author:

rvelices

Message:

feature 2548 multisize - custom sizes restricted to those requested by theme/plugin
code refacto

Location:

trunk

Files:

• admin/derivatives.php (modified) (1 diff)
• admin/include/functions.php (modified) (1 diff)
• admin/themes/default/template/derivatives_build.tpl (modified) (1 diff)
• i.php (modified) (5 diffs)
• include/derivative_params.inc.php (modified) (6 diffs)
• include/derivative_std_params.inc.php (modified) (5 diffs)
• include/picture_comment.inc.php (modified) (1 diff)
• include/template.class.php (modified) (1 diff)
• themes/default/template/picture.tpl (modified) (1 diff)

trunk/admin/derivatives.php

@@ -215 +215 @@
         if (isset($enabled[$type]))
         {// now disabled, before was enabled
+          $changed_types[] = $type;
           $disabled[$type] = $enabled[$type];
           unset($enabled[$type]);

trunk/admin/include/functions.php

@@ -203 +203 @@
 
     $ok = true;
-    foreach ($files as $path)
-    {
-      if (is_file($path) and !unlink($path))
-      {
-        $ok = false;
-        trigger_error('"'.$path.'" cannot be removed', E_USER_WARNING);
-        break;
-      }
-    }
+                if (!isset($conf['never_delete_originals']))
+                {
+                        foreach ($files as $path)
+                        {
+                                if (is_file($path) and !unlink($path))
+                                {
+                                        $ok = false;
+                                        trigger_error('"'.$path.'" cannot be removed', E_USER_WARNING);
+                                        break;
+                                }
+                        }
+                }
 
     if ($ok)

trunk/admin/themes/default/template/derivatives_build.tpl

@@ -45 +45 @@
 {footer_script require='jquery.effects.slide'}{literal}
 
-var loader = new ImageLoader( {onChanged: loaderChanged} )
+var loader = new ImageLoader( {onChanged: loaderChanged, maxRequests:1 } )
         , pending_next_page = null
         , last_image_show_time = 0

trunk/i.php

@@ -66 +66 @@
 function ilog()
 {
-  global $conf, $ilogfh;
+  global $conf;
   if (!$conf['enable_i_log']) return;
-  if(!$ilogfh)
-  {
-    $dir=PHPWG_ROOT_PATH.$conf['data_location'].'tmp/';
-    if (!mkgetdir($dir) or ! ($ilogfh=fopen($dir.'i.log', 'a')) )
-      return;
-  }
+
   $line = date("c");
   foreach( func_get_args() as $arg)
@@ -87 +82 @@
     }
   }
-  fwrite($ilogfh, $line."\n");
+        $file=PHPWG_ROOT_PATH.$conf['data_location'].'tmp/i.log';
+  if (false == file_put_contents($file, $line."\n", FILE_APPEND))
+        {
+                mkgetdir(dirname($file));
+        }
 }
 
@@ -125 +124 @@
 }
 
+function url_to_size($s)
+{
+  $pos = strpos($s, 'x');
+  if ($pos===false)
+  {
+    return array((int)$s, (int)$s);
+  }
+  return array((int)substr($s,0,$pos), (int)substr($s,$pos+1));
+}
+
+function parse_custom_params($tokens)
+{
+  if (count($tokens)<1)
+    ierror('Empty array while parsing Sizing', 400);
+
+  $crop = 0;
+  $min_size = null;
+
+  $token = array_shift($tokens);
+  if ($token[0]=='s')
+  {
+    $size = url_to_size( substr($token,1) );
+  }
+  elseif ($token[0]=='e')
+  {
+    $crop = 1;
+    $size = $min_size = url_to_size( substr($token,1) );
+  }
+  else
+  {
+    $size = url_to_size( $token );
+    if (count($tokens)<2)
+      ierror('Sizing arr', 400);
+
+    $token = array_shift($tokens);
+    $crop = char_to_fraction($token);
+
+    $token = array_shift($tokens);
+    $min_size = url_to_size( $token );
+  }
+  return new DerivativeParams( new SizingParams($size, $crop, $min_size) );
+}
+
 function parse_request()
 {
@@ -200 +242 @@
   if ($page['derivative_type'] == IMG_CUSTOM)
   {
-    try
-    {
-      $params = $page['derivative_params'] = DerivativeParams::from_url_tokens($deriv);
-    }
-    catch (Exception $e)
-    {
-      ierror($e->getMessage(), 400);
-    }
+    $params = $page['derivative_params'] = parse_custom_params($deriv);
+
     if ($params->sizing->ideal_size[0] < 20 or $params->sizing->ideal_size[1] < 20)
     {
@@ -215 +251 @@
     {
       ierror('Invalid crop', 400);
+    }
+    $greatest = ImageStdParams::get_by_type(IMG_XXLARGE);
+    if ($params->max_width() > $greatest->max_width() || $params->max_height() > $greatest->max_height())
+    {
+      ierror('Too big', 403);
+    }
+    
+    $key = array();
+    $params->add_url_tokens($key);
+    $key = implode('_', $key);
+    if (!isset(ImageStdParams::$custom[$key]))
+    {
+      ierror('Size not allowed', 403);
     }
   }

trunk/include/derivative_params.inc.php

@@ -34 +34 @@
 }
 
-function url_to_size($s)
-{
-  $pos = strpos($s, 'x');
-  if ($pos===false)
-  {
-    return array((int)$s, (int)$s);
-  }
-  return array((int)substr($s,0,$pos), (int)substr($s,$pos+1));
-}
-
 function size_equals($s1, $s2)
 {
@@ -49 +39 @@
 }
 
+function char_to_fraction($c)
+{
+        return (ord($c) - ord('a'))/25;
+}
+
+function fraction_to_char($f)
+{
+        return ord('a') + round($f*25);
+}
 
 /** small utility to manipulate a 'rectangle'*/
@@ -80 +79 @@
     if (!empty($coi))
     {
-      $coil = floor($this->r * (ord($coi[0]) - ord('a'))/25);
-      $coir = ceil($this->r * (ord($coi[2]) - ord('a'))/25);
+      $coil = floor($this->r * char_to_fraction($coi[0]));
+      $coir = ceil($this->r * char_to_fraction($coi[2]));
       $availableL = $coil > $this->l ? $coil - $this->l : 0;
       $availableR = $coir < $this->r ? $this->r - $coir : 0;
@@ -116 +115 @@
     if (!empty($coi))
     {
-      $coit = floor($this->b * (ord($coi[1]) - ord('a'))/25);
-      $coib = ceil($this->b * (ord($coi[3]) - ord('a'))/25);
+      $coit = floor($this->b * char_to_fraction($coi[1]));
+      $coib = ceil($this->b * char_to_fraction($coi[3]));
       $availableT = $coit > $this->t ? $coit - $this->t : 0;
       $availableB = $coib < $this->b ? $this->b - $coib : 0;
@@ -180 +179 @@
       {
         $tokens[] = size_to_url($this->ideal_size);
-        $tokens[] = sprintf('%02x', round(100*$this->max_crop) );
+        $tokens[] = fraction_to_char($this->max_crop);
         $tokens[] = size_to_url($this->min_size);
       }
   }
-
-  static function from_url_tokens($tokens)
-  {
-    if (count($tokens)<1)
-      throw new Exception('Empty array while parsing Sizing');
-    $token = array_shift($tokens);
-    if ($token[0]=='s')
-    {
-      return new SizingParams( url_to_size( substr($token,1) ) );
-    }
-    if ($token[0]=='e')
-    {
-      $s = url_to_size( substr($token,1) );
-      return new SizingParams($s, 1, $s);
-    }
-
-    $ideal_size = url_to_size( $token );
-    if (count($tokens)<2)
-      throw new Exception('Sizing arr');
-
-    $token = array_shift($tokens);
-    $crop = hexdec($token) / 100;
-
-    $token = array_shift($tokens);
-    $min_size = url_to_size( $token );
-    return new SizingParams($ideal_size, $crop, $min_size);
-  }
-
 
   function compute($in_size, $coi, &$crop_rect, &$scale_size)
@@ -303 +274 @@
   }
 
-  static function from_url_tokens($tokens)
-  {
-    $sizing = SizingParams::from_url_tokens($tokens);
-    $ret = new DerivativeParams($sizing);
-    return $ret;
-  }
-
   function compute_final_size($in_size, $coi)
   {

trunk/include/derivative_std_params.inc.php

@@ -47 +47 @@
   private static $undefined_type_map = array();
   private static $watermark;
+  public static $custom = array();
 
   static function get_all_types()
@@ -71 +72 @@
   {
     return self::$all_type_map[$type];
+  }
+  
+  static function get_custom($w, $h, $crop=0, $minw=null, $minh=null)
+  {
+    $params = new DerivativeParams( new SizingParams( array($w,$h), $crop, array($minw,$minh)) );
+    self::apply_global($params);
+
+    $key = array();
+    $params->add_url_tokens($key);
+    $key = implode('_',$key);
+    if ( @self::$custom[$key] < time() - 24*3600)
+    {
+      self::$custom[$key] = time();
+      self::save();
+    }
+    return $params;
   }
 
@@ -104 +121 @@
       self::$watermark = @$arr['w'];
       if (!self::$watermark) self::$watermark = new WatermarkParams();
+      self::$custom = @$arr['c'];
+      if (!self::$custom) self::$custom = array();
     }
     else
@@ -119 +138 @@
   static function set_and_save($map)
   {
+    self::$type_map = $map;
+    self::save();
+    self::build_maps();
+  }
+
+  static function save()
+  {
     global $conf;
-    self::$type_map = $map;
 
     $ser = serialize( array(
       'd' => self::$type_map,
       'w' => self::$watermark,
+      'c' => self::$custom,
       ) );
     conf_update_param('derivatives', addslashes($ser) );
     file_put_contents(PHPWG_ROOT_PATH.$conf['data_location'].'derivatives.dat', $ser);
-    self::build_maps();
   }
 
@@ -143 +168 @@
   }
 
-  public static function apply_global($params)
+  static function apply_global($params)
   {
     if (!empty(self::$watermark->file) &&

trunk/include/picture_comment.inc.php

@@ -127 +127 @@
   {
     // comments order (get, session, conf)
-    if (!empty($_GET['comments_order']))
+    if (!empty($_GET['comments_order']) && in_array(strtoupper($_GET['comments_order']), array('ASC', 'DESC')))
     {
-      if (in_array(strtoupper($_GET['comments_order']), array('ASC', 'DESC')))
-      {
-        $comments_order = $_GET['comments_order'];
-        pwg_set_session_var('comments_order', $comments_order);
-      }
-      else
-      {
-        $comments_order = $conf['comments_order'];
-      }
+      pwg_set_session_var('comments_order', $_GET['comments_order']);
     }
-    else if (pwg_get_session_var('comments_order') !== null)
-    {
-      $comments_order = pwg_get_session_var('comments_order');
-    }
-    else
-    {
-      $comments_order = $conf['comments_order'];
-    }
+    $comments_order = pwg_get_session_var('comments_order', $conf['comments_order']);
+
     $template->assign(array(
       'COMMENTS_ORDER_URL' => duplicate_picture_url().'&comments_order='.($comments_order == 'ASC' ? 'DESC' : 'ASC'),

trunk/include/template.class.php

@@ -560 +560 @@
     !empty($params['height']) or fatal_error('define_derviative missing height');
 
-    $derivative = new DerivativeParams( SizingParams::classic( intval($params['width']), intval($params['height'])) );
+    $w = intval($params['width']);
+    $h = intval($params['height']);
+    $crop = 0;
+    $minw=null;
+    $minh=null;
+    
     if (isset($params['crop']))
     {
       if (is_bool($params['crop']))
       {
-        $derivative->sizing->max_crop = $params['crop'] ? 1:0;
+        $crop = $params['crop'] ? 1:0;
       }
       else
       {
-        $derivative->sizing->max_crop = round($params['crop']/100, 2);
-      }
-
-      if ($derivative->sizing->max_crop)
-      {
-        $minw = empty($params['min_width']) ? $derivative->max_width() : intval($params['min_width']);
-        $minw <= $derivative->max_width() or fatal_error('define_derviative invalid min_width');
-        $minh = empty($params['min_height']) ? $derivative->max_height() : intval($params['min_height']);
-        $minh <= $derivative->max_height() or fatal_error('define_derviative invalid min_height');
-
-        $derivative->sizing->min_size = array($minw, $minh);
-      }
-    }
-
-    ImageStdParams::apply_global($derivative);
-    $smarty->assign( $params['name'], $derivative);
+        $crop = round($params['crop']/100, 2);
+      }
+
+      if ($crop)
+      {
+        $minw = empty($params['min_width']) ? $w : intval($params['min_width']);
+        $minw <= $w or fatal_error('define_derviative invalid min_width');
+        $minh = empty($params['min_height']) ? $h : intval($params['min_height']);
+        $minh <= $h or fatal_error('define_derviative invalid min_height');
+      }
+    }
+
+    $smarty->assign( $params['name'], ImageStdParams::get_custom($w, $h, $crop, $minw, $minh) );
   }
 

trunk/themes/default/template/picture.tpl

@@ -279 +279 @@
         {if $COMMENT_COUNT > 0}
                 <h3>{$pwg->l10n_dec('%d comment', '%d comments',$COMMENT_COUNT)}</h3>
-        {/if}
-  {if $COMMENT_COUNT > 2}
-    {'Sort order'|@translate} : <a href="{$COMMENTS_ORDER_URL}#comments">{$COMMENTS_ORDER_TITLE}</a>
-  {/if}
+        {if $COMMENT_COUNT > 2}
+                {'Sort order'|@translate}: <a href="{$COMMENTS_ORDER_URL}#comments" rel="nofollow">{$COMMENTS_ORDER_TITLE}</a>
+        {/if}
+        {/if}
         {if !empty($navbar)}{include file='navigation_bar.tpl'|@get_extent:'navbar'}{/if}