PHP 7.4 support + modernize codestyle

Author: janbarasek

README.md

@@ -9,7 +9,7 @@ Install
 By Composer:
 
 ```shell
-composer require baraja-core/php-pdf-to-image
+$ composer require baraja-core/php-pdf-to-image
 ```
 
 Usage
@@ -21,4 +21,4 @@ $imagePath = __DIR__ . '/example.jpg';
 
 // Render PDF to image and save to disk.
 \Baraja\PdfToImage\Convertor::convert($pdfPath, $imagePath, 'jpg');
-```
+```

composer.json

@@ -9,12 +9,13 @@
 		}
 	],
 	"require": {
-		"php": ">=7.1.0"
+		"php": ">=7.4.0"
 	},
 	"require-dev": {
 		"phpstan/phpstan": "^0.12.18",
 		"tracy/tracy": "^2.7",
-		"phpstan/phpstan-nette": "^0.12.6"
+		"phpstan/phpstan-nette": "^0.12.6",
+		"symplify/easy-coding-standard": "^7.2"
 	},
 	"autoload": {
 		"classmap": [

src/Convertor.php

@@ -16,9 +16,7 @@ final class Convertor
 	public const SUPPORTED_FORMATS = [self::FORMAT_JPG, self::FORMAT_PNG, self::FORMAT_GIF];
 
 
-	/**
-	 * @throws \Error
-	 */
+	/** @throws \Error */
 	public function __construct()
 	{
 		throw new \Error('Class ' . get_class($this) . ' is static and cannot be instantiated.');
@@ -28,25 +26,18 @@ public function __construct()
 	/**
 	 * Convert first page of PDF to image and save to disk.
 	 *
-	 * @param string $pdfPath
-	 * @param string $savePath
-	 * @param string $format
-	 * @param bool $trim
 	 * @throws ConvertorException
 	 */
 	public static function convert(string $pdfPath, string $savePath, string $format = 'jpg', bool $trim = false): void
 	{
 		if (\in_array($format = strtolower($format), self::SUPPORTED_FORMATS, true) === false) {
 			ConvertorException::unsupportedFormat($format);
 		}
-
 		if (\is_file($pdfPath) === false) {
 			ConvertorException::fileDoesNotExist($pdfPath);
 		}
-
 		try {
 			$im = self::process($pdfPath, $savePath);
-
 			if ($trim === true) {
 				$im->setImageBorderColor('rgb(255,255,255)');
 				$im->trimImage(1);
@@ -59,8 +50,6 @@ public static function convert(string $pdfPath, string $savePath, string $format
 
 
 	/**
-	 * @param string $pdfPath
-	 * @param string $savePath
 	 * @return \Imagick
 	 * @throws ConvertorException|\ImagickException
 	 */
@@ -79,12 +68,8 @@ private static function process(string $pdfPath, string $savePath): \Imagick
 
 
 	/**
-	 * Writes a string to a file.
-	 * Moved from nette/utils
+	 * Writes a string to a file. Moved from nette/utils
 	 *
-	 * @param string $file
-	 * @param string $content
-	 * @param int|null $mode
 	 * @throws ConvertorException
 	 */
 	private static function write(string $file, string $content, ?int $mode = 0666): void
@@ -100,11 +85,8 @@ private static function write(string $file, string $content, ?int $mode = 0666):
 
 
 	/**
-	 * Creates a directory.
-	 * Moved from nette/utils
+	 * Creates a directory. Moved from nette/utils
 	 *
-	 * @param string $dir
-	 * @param int $mode
 	 * @throws ConvertorException
 	 */
 	private static function createDir(string $dir, int $mode = 0777): void
@@ -115,13 +97,8 @@ private static function createDir(string $dir, int $mode = 0777): void
 	}
 
 
-	/**
-	 * Moved from nette/utils
-	 *
-	 * @return string
-	 */
 	private static function getLastError(): string
 	{
 		return (string) preg_replace('#^\w+\(.*?\): #', '', error_get_last()['message']);
 	}
-}
+}

src/ConvertorException.php

@@ -9,7 +9,6 @@ final class ConvertorException extends \Exception
 {
 
 	/**
-	 * @param string $path
 	 * @throws ConvertorException
 	 */
 	public static function fileDoesNotExist(string $path): void
@@ -19,7 +18,6 @@ public static function fileDoesNotExist(string $path): void
 
 
 	/**
-	 * @param string $format
 	 * @throws ConvertorException
 	 */
 	public static function unsupportedFormat(string $format): void
@@ -38,4 +36,4 @@ public static function imagicKIsNotInstalled(): void
 	{
 		throw new self('Imagick is not installed.');
 	}
-}
+}