From: Domovoy <domovoy@errlock.org>
Date: Sun, 22 Jul 2012 14:19:35 +0000 (+0200)
Subject: getExt should not use Imagick format, mapping ext and mime type is more accurate.
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=ab484ebc76722624bb8823e9de459c9ce026465a;p=friendica.git

getExt should not use Imagick format, mapping ext and mime type is more accurate.
---

diff --git a/include/Photo.php b/include/Photo.php
index 830f5213e8..4ac84d9015 100644
--- a/include/Photo.php
+++ b/include/Photo.php
@@ -4,7 +4,6 @@ if(! class_exists("Photo")) {
 class Photo {
 
     private $image;
-    private $ext;
 
     /**
      * Put back gd stuff, not everybody have Imagick
@@ -41,6 +40,7 @@ class Photo {
 
     public function __construct($data, $type=null) {
         $this->imagick = class_exists('Imagick');
+        $this->types = $this->supportedTypes();
 
         if($this->is_imagick()) {
             $this->image = new Imagick();
@@ -54,10 +54,7 @@ class Photo {
 
             // If it is a gif, it may be animated, get it ready for any future operations
             if($this->image->getFormat() !== "GIF") $this->image = $this->image->coalesceImages();
-
-            $this->ext = strtolower($this->image->getImageFormat());
         } else {
-            $this->types = $this->supportedTypes();
             if (!array_key_exists($type,$this->types)){
                 $type='image/jpeg';
             }
@@ -139,9 +136,7 @@ class Photo {
         if(!$this->is_valid())
             return FALSE;
 
-        if($this->is_imagick())
-            return $this->ext;
-        return $this->types[$this->type];
+        return $this->types[$this->getType()];
     }
 
     public function scaleImage($max) {