Changeset 14143

Timestamp:

Apr 14, 2012, 6:52:20 AM (12 years ago)

Author:

rvelices

Message:

bug 2615 php notice in calendar amd web service
multisize improve handling of cases where the original is smaller than a requested derivative, but rotation/watermarking is required

Location:

trunk

Files:

• admin/batch_manager_global.php (modified) (1 diff)
• include/calendar_monthly.class.php (modified) (2 diffs)
• include/derivative.inc.php (modified) (3 diffs)
• include/ws_functions.inc.php (modified) (1 diff)

trunk/admin/batch_manager_global.php

@@ -735 +735 @@
 
   $query = '
-SELECT id,path,representative_ext,file,filesize,level,name
+SELECT id,path,representative_ext,file,filesize,level,name,width,height,rotation
   FROM '.IMAGES_TABLE;
   

trunk/include/calendar_monthly.class.php

@@ -351 +351 @@
     $page['chronology_date'][CDAY]=$day;
     $query = '
-SELECT id, file,representative_ext,path,width, height, '.pwg_db_get_dayofweek($this->date_field).'-1 as dow';
+SELECT id, file,representative_ext,path,width,height,rotation, '.pwg_db_get_dayofweek($this->date_field).'-1 as dow';
     $query.= $this->inner_sql;
     $query.= $this->get_date_where();
@@ -393 +393 @@
 
     list($cell_width, $cell_height) = ImageStdParams::get_by_type(IMG_SQUARE)->sizing->ideal_size;
+    if ($cell_width>120)
+    {
+      $cell_width = $cell_height = 120;
+    }
 
     $tpl_weeks    = array();

trunk/include/derivative.inc.php

@@ -27 +27 @@
   public $id;
   public $rel_path;
+  public $rotation = 0;
 
   private $size=null;
@@ -59 +60 @@
       $height = $infos['height'];
 
+      $this->rotation = intval($infos['rotation']) % 4;
       // 1 or 5 =>  90 clockwise
       // 3 or 7 => 270 clockwise
-      if ($infos['rotation'] % 2 != 0)
+      if ($this->rotation % 2)
       {
         $width = $infos['height'];
@@ -174 +176 @@
     if ( $src->has_size() && $params->is_identity( $src->get_size() ) )
     {
-      // todo - what if we have a watermark maybe return a smaller size?
-      $params = null;
-      $rel_path = $rel_url = $src->rel_path;
-      return;
+      if (!$params->use_watermark && !$src->rotation)
+      {
+        $params = null;
+        $rel_path = $rel_url = $src->rel_path;
+        return;
+      }
+      $defined_types = array_keys(ImageStdParams::get_defined_type_map());
+      for ($i=0; $i<count($defined_types); $i++)
+      {
+        if ($defined_types[$i] == $params->type)
+        {
+          for ($i--; $i>=0; $i--)
+          {
+            $smaller = ImageStdParams::get_by_type($defined_types[$i]);
+            if ($smaller->sizing->max_crop==$params->sizing->max_crop && $smaller->is_identity( $src->get_size() ))
+            {
+              $params = $smaller;
+              self::build($src, $params, $rel_path, $rel_url, $is_cached);
+              return;
+            }
+          }
+          break;
+        }
+      }
     }
 

trunk/include/ws_functions.inc.php

@@ -225 +225 @@
   }
 
-  $query_model = 'SELECT id, path, representative_ext, width, height
+  $query_model = 'SELECT id, path, representative_ext, width,height,rotation
   FROM '.IMAGES_TABLE.'
   WHERE '.implode(' AND ', $where_clauses).'