Changeset 12749 for trunk/admin/include

Timestamp:

Dec 16, 2011, 6:00:06 PM (13 years ago)

Author:

patdenice

Message:

bug:2535
Image is not cropped and ressized if requested size is bigger than image size.

File:

• trunk/admin/include/image.class.php (modified) (6 diffs)

trunk/admin/include/image.class.php

@@ -98 +98 @@
     $source_height = $this->image->get_height();
 
-    // Crop image
-    if ($crop)
-    {
-      $x = 0;
-      $y = 0;
-
-      if ($source_width < $source_height and $follow_orientation)
-      {
-        list($max_width, $max_height) = array($max_height, $max_width);
-      }
-
-      $img_ratio = $source_width / $source_height;
-      $dest_ratio = $max_width / $max_height;
-
-      if($dest_ratio > $img_ratio)
-      {
-        $destHeight = round($source_width * $max_height / $max_width);
-        $y = round(($source_height - $destHeight) / 2 );
-        $source_height = $destHeight;
-      }
-      elseif ($dest_ratio < $img_ratio)
-      {
-        $destWidth = round($source_height * $max_width / $max_height);
-        $x = round(($source_width - $destWidth) / 2 );
-        $source_width = $destWidth;
-      }
-
-      $this->image->crop($source_width, $source_height, $x, $y);
-    }
-
     $rotation = null;
     if ($automatic_rotation)
@@ -133 +103 @@
       $rotation = self::get_rotation_angle($this->source_filepath);
     }
-    $resize_dimensions = self::get_resize_dimensions($source_width, $source_height, $max_width, $max_height, $rotation);
+    $resize_dimensions = self::get_resize_dimensions($source_width, $source_height, $max_width, $max_height, $rotation, $crop, $follow_orientation);
 
     // testing on height is useless in theory: if width is unchanged, there
@@ -151 +121 @@
       $this->image->strip();
     }
+
+    if (isset($resize_dimensions['crop']))
+    {
+      $this->image->crop($resize_dimensions['crop']['width'], $resize_dimensions['crop']['height'], $resize_dimensions['crop']['x'], $resize_dimensions['crop']['y']);
+    }
     
     $this->image->resize($resize_dimensions['width'], $resize_dimensions['height']);
@@ -165 +140 @@
   }
 
-  static function get_resize_dimensions($width, $height, $max_width, $max_height, $rotation=null)
+  static function get_resize_dimensions($width, $height, $max_width, $max_height, $rotation=null, $crop=false, $follow_orientation=true)
   {
     $rotate_for_dimensions = false;
@@ -176 +151 @@
     {
       list($width, $height) = array($height, $width);
+    }
+
+    if ($crop)
+    {
+      $x = 0;
+      $y = 0;
+
+      if ($width < $height and $follow_orientation)
+      {
+        list($max_width, $max_height) = array($max_height, $max_width);
+      }
+
+      $img_ratio = $width / $height;
+      $dest_ratio = $max_width / $max_height;
+
+      if($dest_ratio > $img_ratio)
+      {
+        $destHeight = round($width * $max_height / $max_width);
+        $y = round(($height - $destHeight) / 2 );
+        $height = $destHeight;
+      }
+      elseif ($dest_ratio < $img_ratio)
+      {
+        $destWidth = round($height * $max_width / $max_height);
+        $x = round(($width - $destWidth) / 2 );
+        $width = $destWidth;
+      }
     }
     
@@ -203 +205 @@
     }
 
-    return array(
+    $result = array(
       'width' => $destination_width,
       'height'=> $destination_height,
       );
+
+    if ($crop and ($x or $y))
+    {
+      $result['crop'] = array(
+        'width' => $width,
+        'height' => $height,
+        'x' => $x,
+        'y' => $y,
+        );
+    }
+    return $result;
   }