To participate you must create an account on apostrophenow.org. If you have already done so, click Login.

Changeset 2329

Show
Ignore:
Timestamp:
10/04/10 18:51:51 (4 years ago)
Author:
tboutell
Message:

Fixed #626 (don't play with EXIF if the file is not a JPEG)

Location:
plugins/apostrophePlugin/trunk/lib/toolkit
Files:
2 modified

Legend:

Unmodified
Added
Removed
  • plugins/apostrophePlugin/trunk/lib/toolkit/BaseaTools.class.php

    r2301 r2329  
    644644  static public function getRealUrl() 
    645645  { 
    646     if(isset(aTools::$realUrl)) 
     646    if (isset(aTools::$realUrl)) 
    647647    { 
    648648      return aTools::$realUrl; 
  • plugins/apostrophePlugin/trunk/lib/toolkit/aImageConverter.class.php

    r2315 r2329  
    122122  static private function scaleBody($fileIn, $fileOut, $scaleParameters = array(), $cropParameters = array(), $quality = 75)  
    123123  {     
    124     error_log("ENTERING scaleBody"); 
    125124    if (sfConfig::get('app_aimageconverter_netpbm', true)) 
    126125    { 
     
    132131      if (sfConfig::get('app_aimageconverter_netpbm', true) === 'always') 
    133132      { 
    134         error_log("Calling scaleNetpbm"); 
    135133        return self::scaleNetpbm($fileIn, $fileOut, $scaleParameters, $cropParameters, $quality); 
    136134      } 
     
    139137      if (($info !== false) && (($info[0] <= 1024) && ($info[1] <= 768)) && function_exists('imagetypes') && isset($mapTypes[$info[2]]) && (imagetypes() & $mapTypes[$info[2]])) 
    140138      { 
    141         error_log("FALLBACK"); 
    142139        return self::scaleGd($fileIn, $fileOut, $scaleParameters, $cropParameters, $quality); 
    143140      } 
     
    150147    else 
    151148    { 
    152       error_log("GD"); 
    153149      return self::scaleGd($fileIn, $fileOut, $scaleParameters, $cropParameters, $quality); 
    154150    } 
     
    165161  // case 8:    // 90 rotate left 
    166162 
    167   static public function getRotation($file) 
    168   { 
     163  static public function getRotation($file, $getimagesize = null) 
     164  { 
     165    if (is_null($getimagesize)) 
     166    { 
     167      $getimagesize = getimagesize($file); 
     168    } 
     169    if ($getimagesize[2] !== IMAGETYPE_JPEG) 
     170    { 
     171      return 1; 
     172    } 
    169173    if (!extension_loaded("exif")) 
    170174    { 
     
    263267    $rotate = ''; 
    264268     
    265     $rotation = aImageConverter::getRotation($fileIn); 
     269    $rotation = aImageConverter::getRotation($fileIn, $info); 
    266270    switch ($rotation) 
    267271    { 
     
    361365      $width = $imageInfo[0]; 
    362366      $height = $imageInfo[1]; 
    363       $orientation = aImageConverter::getRotation($fileIn); 
     367      $orientation = aImageConverter::getRotation($fileIn, $imageInfo); 
    364368      if ($imageInfo[2] === IMAGETYPE_JPEG) 
    365369      { 
     
    728732      { 
    729733        // Some EXIF orientations swap width and height 
    730         switch (aImageConverter::getRotation($file)) 
     734        switch (aImageConverter::getRotation($file, $data)) 
    731735        { 
    732736          case 5: // vertical flip + 90 rotate right