Update UploadHandler.php

jbostoen · web-flow · commit 98670fabbf4d · 2017-07-19T19:50:13.000+02:00
You actually have an implementation for a strip-parameter (and its used as stripImage) to remove EXIF tags (eg location). But if you want to retain the original file when it comes to anything else, you need a little change. Took some time to figure out.
diff --git a/server/php/UploadHandler.php b/server/php/UploadHandler.php
@@ -871,7 +871,7 @@ protected function imagick_create_scaled_image($file_name, $version, $options) {
         if (!empty($options['max_height'])) {
             $new_height = $max_height = $options['max_height'];
         }
-        if (!($image_oriented || $max_width < $img_width || $max_height < $img_height)) {
+        if (!($image_oriented || $max_width < $img_width || $max_height < $img_height || $options['strip'] == TRUE )) {
             if ($file_path !== $new_file_path) {
                 return copy($file_path, $new_file_path);
             }

Original file line number	Diff line number	Diff line change
`@@ -871,7 +871,7 @@ protected function imagick_create_scaled_image($file_name, $version, $options) {`
`871`	`871`	`if (!empty($options['max_height'])) {`
`872`	`872`	`$new_height = $max_height = $options['max_height'];`
`873`	`873`	`}`
`874`		`- if (!($image_oriented \|\| $max_width < $img_width \|\| $max_height < $img_height)) {`
	`874`	`+ if (!($image_oriented \|\| $max_width < $img_width \|\| $max_height < $img_height \|\| $options['strip'] == TRUE )) {`
`875`	`875`	`if ($file_path !== $new_file_path) {`
`876`	`876`	`return copy($file_path, $new_file_path);`
`877`	`877`	`}`