Better attempt to strip image meta
authorLaury GvR <laury@gaslightmedia.com>
Thu, 22 Jun 2017 19:17:30 +0000 (15:17 -0400)
committerLaury GvR <laury@gaslightmedia.com>
Thu, 22 Jun 2017 19:17:30 +0000 (15:17 -0400)
models/admin/ajax/imageUpload.php

index 769e0fd..ab8fb2b 100644 (file)
@@ -193,9 +193,6 @@ class GlmMembersAdmin_ajax_imageUpload extends GlmDataImages
             $newFilename = strtolower($fInfo['filename'].'_'.time().'.'.$fInfo['extension']);
         }
 
-        // Strip meta data off image - this prevents EXIF orientation shenanigans
-        $file['tmp_name'] = $this->stripImageMeta($file['tmp_name']);
-
         // Get new image using temporary file name
         $newImage = wp_get_image_editor($file['tmp_name']);
 
@@ -304,7 +301,17 @@ class GlmMembersAdmin_ajax_imageUpload extends GlmDataImages
     public function stripImageMeta($imageName) {
 
         if (extension_loaded('imagick')) {
-            $imageName->stripImage();
+            $imagick = new Imagick($imageName);
+            $imagick->setCompression(imagick::COMPRESSION_JPEG);
+            $imagick->setCompressionQuality(100);
+//            $profiles = $imagick->getImageProfiles("icc", true);
+            $imagick->stripImage();
+//            if(!empty($profiles)) {
+//                $imagick->profileImage("icc", $profiles['icc']);
+//            }
+            $imagick->writeImage($imageName);
+
+
         } else if (extension_loaded('gd') && function_exists('gd_info')) {
             $image_type = exif_imagetype($imageName);
             switch ($image_type) {