X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FObject%2FImage.php;h=5cef75180bdb2949f4fac96c903a07bb81d28342;hb=e3692c01053d4714292449d1170ef2d60ad575d7;hp=972b48359f6d51b918b46605ed1f236c58a92b76;hpb=f2c31ef1c0e92208b58d22791fc72d0ad3e3d6ae;p=friendica.git

diff --git a/src/Object/Image.php b/src/Object/Image.php
index 972b48359f..5cef75180b 100644
--- a/src/Object/Image.php
+++ b/src/Object/Image.php
@@ -1,13 +1,28 @@
 <?php
 /**
- * @file src/Object/Image.php
- * @brief This file contains the Image class for image processing
+ * @copyright Copyright (C) 2010-2022, the Friendica project
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
  */
+
 namespace Friendica\Object;
 
 use Exception;
-use Friendica\Core\Config;
-use Friendica\Core\System;
+use Friendica\DI;
 use Friendica\Util\Images;
 use Imagick;
 use ImagickPixel;
@@ -31,7 +46,7 @@ class Image
 	private $types;
 
 	/**
-	 * @brief Constructor
+	 * Constructor
 	 * @param string  $data
 	 * @param boolean $type optional, default null
 	 * @throws \Friendica\Network\HTTPException\InternalServerErrorException
@@ -56,7 +71,8 @@ class Image
 	}
 
 	/**
-	 * @brief Destructor
+	 * Destructor
+	 *
 	 * @return void
 	 */
 	public function __destruct()
@@ -106,17 +122,18 @@ class Image
 			$this->image->setFormat($format);
 
 			// Always coalesce, if it is not a multi-frame image it won't hurt anyway
-			$this->image = $this->image->coalesceImages();
+			try {
+				$this->image = $this->image->coalesceImages();
+			} catch (Exception $e) {
+				return false;
+			}
 
 			/*
 			 * setup the compression here, so we'll do it only once
 			 */
 			switch ($this->getType()) {
 				case "image/png":
-					$quality = Config::get('system', 'png_quality');
-					if ((! $quality) || ($quality > 9)) {
-						$quality = PNG_QUALITY;
-					}
+					$quality = DI::config()->get('system', 'png_quality');
 					/*
 					 * From http://www.imagemagick.org/script/command-line-options.php#quality:
 					 *
@@ -129,10 +146,7 @@ class Image
 					$this->image->setCompressionQuality($quality);
 					break;
 				case "image/jpeg":
-					$quality = Config::get('system', 'jpeg_quality');
-					if ((! $quality) || ($quality > 100)) {
-						$quality = JPEG_QUALITY;
-					}
+					$quality = DI::config()->get('system', 'jpeg_quality');
 					$this->image->setCompressionQuality($quality);
 			}
 
@@ -210,9 +224,13 @@ class Image
 		}
 
 		if ($this->isImagick()) {
-			/* Clean it */
-			$this->image = $this->image->deconstructImages();
-			return $this->image;
+			try {
+				/* Clean it */
+				$this->image = $this->image->deconstructImages();
+				return $this->image;
+			} catch (Exception $e) {
+				return false;
+			}
 		}
 		return $this->image;
 	}
@@ -439,7 +457,6 @@ class Image
 				break;
 		}
 
-		//	Logger::log('exif: ' . print_r($exif,true));
 		return $exif;
 	}
 
@@ -500,7 +517,7 @@ class Image
 	}
 
 	/**
-	 * @brief Scale image to target dimensions
+	 * Scale image to target dimensions
 	 *
 	 * @param int $dest_width
 	 * @param int $dest_height
@@ -522,7 +539,12 @@ class Image
 			do {
 				// FIXME - implement horizontal bias for scaling as in following GD functions
 				// to allow very tall images to be constrained only horizontally.
-				$this->image->scaleImage($dest_width, $dest_height);
+				try {
+					$this->image->scaleImage($dest_width, $dest_height);
+				} catch (Exception $e) {
+					// Imagick couldn't use the data
+					return false;
+				}
 			} while ($this->image->nextImage());
 
 			// These may not be necessary anymore
@@ -551,6 +573,21 @@ class Image
 		return true;
 	}
 
+	/**
+	 * Convert a GIF to a PNG to make it static
+	 */
+	public function toStatic()
+	{
+		if ($this->type != 'image/gif') {
+			return;
+		}
+
+		if ($this->isImagick()) {
+			$this->type == 'image/png';
+			$this->image->setFormat('png');
+		}
+	}
+
 	/**
 	 * @param integer $max maximum
 	 * @param integer $x   x coordinate
@@ -595,27 +632,7 @@ class Image
 	}
 
 	/**
-	 * @param string $path file path
-	 * @return mixed
-	 * @throws \Friendica\Network\HTTPException\InternalServerErrorException
-	 */
-	public function saveToFilePath($path)
-	{
-		if (!$this->isValid()) {
-			return false;
-		}
-
-		$string = $this->asString();
-
-		$a = \get_app();
-
-		$stamp1 = microtime(true);
-		file_put_contents($path, $string);
-		$a->getProfiler()->saveTimestamp($stamp1, "file", System::callstack());
-	}
-
-	/**
-	 * @brief Magic method allowing string casting of an Image object
+	 * Magic method allowing string casting of an Image object
 	 *
 	 * Ex: $data = $Image->asString();
 	 * can be replaced by
@@ -639,10 +656,14 @@ class Image
 		}
 
 		if ($this->isImagick()) {
-			/* Clean it */
-			$this->image = $this->image->deconstructImages();
-			$string = $this->image->getImagesBlob();
-			return $string;
+			try {
+				/* Clean it */
+				$this->image = $this->image->deconstructImages();
+				$string = $this->image->getImagesBlob();
+				return $string;
+			} catch (Exception $e) {
+				return false;
+			}
 		}
 
 		ob_start();
@@ -652,17 +673,11 @@ class Image
 
 		switch ($this->getType()) {
 			case "image/png":
-				$quality = Config::get('system', 'png_quality');
-				if ((!$quality) || ($quality > 9)) {
-					$quality = PNG_QUALITY;
-				}
+				$quality = DI::config()->get('system', 'png_quality');
 				imagepng($this->image, null, $quality);
 				break;
 			case "image/jpeg":
-				$quality = Config::get('system', 'jpeg_quality');
-				if ((!$quality) || ($quality > 100)) {
-					$quality = JPEG_QUALITY;
-				}
+				$quality = DI::config()->get('system', 'jpeg_quality');
 				imagejpeg($this->image, null, $quality);
 		}
 		$string = ob_get_contents();
@@ -670,63 +685,4 @@ class Image
 
 		return $string;
 	}
-
-	/**
-	 * @brief supported mimetypes and corresponding file extensions
-	 * @return array
-	 * @deprecated in version 2019.12 please use Util\Images::supportedTypes() instead.
-	 */
-	public static function supportedTypes()
-	{
-		return Images::supportedTypes();
-	}
-
-	/**
-	 * @brief Maps Mime types to Imagick formats
-	 * @return array With with image formats (mime type as key)
-	 * @deprecated in version 2019.12 please use Util\Images::getFormatsMap() instead.
-	 */
-	public static function getFormatsMap()
-	{
-		return Images::getFormatsMap();
-	}
-
-	/**
-	 * Guess image mimetype from filename or from Content-Type header
-	 *
-	 * @param string  $filename Image filename
-	 * @param boolean $fromcurl Check Content-Type header from curl request
-	 * @param string  $header   passed headers to take into account
-	 *
-	 * @return string|null
-	 * @throws Exception
-	 * @deprecated in version 2019.12 please use Util\Images::guessType() instead.
-	 */
-	public static function guessType($filename, $fromcurl = false, $header = '')
-	{
-		return Images::guessType($filename, $fromcurl, $header);
-	}
-
-	/**
-	 * @param string $url url
-	 * @return array
-	 * @throws \Friendica\Network\HTTPException\InternalServerErrorException
-	 * @deprecated in version 2019.12 please use Util\Images::getInfoFromURLCached() instead.
-	 */
-	public static function getInfoFromURL($url)
-	{
-		return Images::getInfoFromURLCached($url);
-	}
-
-	/**
-	 * @param integer $width  width
-	 * @param integer $height height
-	 * @param integer $max    max
-	 * @return array
-	 * @deprecated in version 2019.12 please use Util\Images::getScalingDimensions() instead.
-	 */
-	public static function getScalingDimensions($width, $height, $max)
-	{
-		return Images::getScalingDimensions($width, $height, $max);
-	}
 }