Changeset 14735 for trunk/include/derivative.inc.php

Timestamp:

05/04/12 06:29:44 (13 months ago)

Author:

rvelices

Message:

multi size - don't fail if image metadata was not previously synced

Files:

• trunk/include/derivative.inc.php (modified) (3 diffs)

trunk/include/derivative.inc.php

@@ -24 +24 @@
   const IS_ORIGINAL = 0x01;
   const IS_MIMETYPE = 0x02;
+  const DIM_NOT_GIVEN = 0x04;
 
   public $id;
@@ -52 +53 @@
       $this->rel_path = trigger_event('get_mimetype_location', get_themeconf('mime_icon_dir').$ext.'.png', $ext );
       $this->flags |= self::IS_MIMETYPE;
-      $this->size = @getimagesize(PHPWG_ROOT_PATH.$this->rel_path);
-    }
-
-    if (!$this->size && isset($infos['width']) && isset($infos['height']))
-    {
-      $width = $infos['width'];
-      $height = $infos['height'];
-
-      $this->rotation = intval($infos['rotation']) % 4;
-      // 1 or 5 =>  90 clockwise
-      // 3 or 7 => 270 clockwise
-      if ($this->rotation % 2)
-      {
-        $width = $infos['height'];
-        $height = $infos['width'];
-      }
-      
-      $this->size = array($width, $height);
+      if ( ($size=@getimagesize(PHPWG_ROOT_PATH.$this->rel_path)) === false)
+      {
+        $this->rel_path = get_themeconf('mime_icon_dir').'unknown.png';
+        $size = getimagesize(PHPWG_ROOT_PATH.$this->rel_path);
+      }
+      $this->size = array($size[0],$size[1]);
+    }
+
+    if (!$this->size)
+    {
+      if (isset($infos['width']) && isset($infos['height']))
+      {
+        $width = $infos['width'];
+        $height = $infos['height'];
+
+        $this->rotation = intval($infos['rotation']) % 4;
+        // 1 or 5 =>  90 clockwise
+        // 3 or 7 => 270 clockwise
+        if ($this->rotation % 2)
+        {
+          $width = $infos['height'];
+          $height = $infos['width'];
+        }
+        
+        $this->size = array($width, $height);
+      }
+      elseif (!array_key_exists('width', $infos))
+      {
+        $this->flags |= self::DIM_NOT_GIVEN;
+      }
     }
   }
@@ -106 +119 @@
   {
     if ($this->size == null)
-      not_implemented(); // get size from file
+    {
+      if ($this->flags & self::DIM_NOT_GIVEN)
+        fatal_error('SrcImage dimensions required but not provided');
+      // probably not metadata synced
+      if ( ($size = getimagesize( $this->get_path() )) !== false)
+      {
+        $this->size = array($size[0],$size[1]);
+        pwg_query('UPDATE '.IMAGES_TABLE.' SET width='.$size[0].', height='.$size[1].' WHERE id='.$this->id);
+      }
+    }
     return $this->size;
   }