Support tool paths in factory (#73)

---------

Co-authored-by: Martin Stone <[email protected]>

Author: ildyria

lib/PHPExif/Adapter/AbstractAdapter.php

@@ -30,7 +30,7 @@ abstract class AbstractAdapter implements AdapterInterface
      *
      * @param array $options Optional array of data to initialize the object with
      */
-    public function __construct(array $options = array())
+    public function __construct(array $options = [])
     {
         if (count($options) > 0) {
             $this->setOptions($options);
@@ -43,7 +43,7 @@ public function __construct(array $options = array())
      * @param \PHPExif\Contracts\MapperInterface $mapper
      * @return \PHPExif\Contracts\AdapterInterface
      */
-    public function setMapper(MapperInterface $mapper): AdapterInterface
+    final public function setMapper(MapperInterface $mapper): AdapterInterface
     {
         $this->mapper = $mapper;
 

lib/PHPExif/Adapter/Exiftool.php

@@ -32,9 +32,22 @@ class Exiftool extends AbstractAdapter
      */
     protected string $toolPath = '';
     protected bool $numeric = true;
-    protected array $encoding = array();
+    protected array $encoding = [];
     protected string $mapperClass = MapperExiftool::class;
 
+    /**
+     * Set up Exiftool adapter
+     *
+     * @param array $options option to be passed to the parent
+     * @param string $path optional path to the tool
+     * @return self
+     */
+    public function __construct(array $options = [], string $path = '')
+    {
+        parent::__construct($options);
+        $this->toolPath = $path;
+    }
+
     /**
      * Setter for the exiftool binary path
      *

lib/PHPExif/Adapter/FFprobe.php

@@ -42,6 +42,19 @@ class FFprobe extends AbstractAdapter
     protected string $mapperClass = MapperFFprobe::class;
 
 
+    /**
+     * Set up FFprobe adapter
+     *
+     * @param array $options option to be passed to the parent
+     * @param string $path optional path to the tool
+     * @return self
+     */
+    public function __construct(array $options = [], string $path = '')
+    {
+        parent::__construct($options);
+        $this->toolPath = $path;
+    }
+
     /**
      * Setter for the exiftool binary path
      *

lib/PHPExif/Adapter/ImageMagick.php

@@ -52,6 +52,7 @@ class ImageMagick extends AbstractAdapter
     public function getExifFromFile(string $file): Exif
     {
         /* Create the object */
+        /** @disregard P1009 */
         $im = new Imagick($file);
 
         /* Get the EXIF information */

lib/PHPExif/Adapter/Native.php

@@ -41,7 +41,7 @@ class Native extends AbstractAdapter
      *
      * @var array
      */
-    protected array $requiredSections = array();
+    protected array $requiredSections = [];
 
     /**
      * Include the thumbnail in the EXIF data?
@@ -200,7 +200,7 @@ public function getExifFromFile(string $file): Exif
 
         // exif_read_data failed to read exif data (i.e. not a jpg/tiff)
         if (false === $data) {
-            $data = array();
+            $data = [];
             $data['FileSize'] = filesize($file);
             $data['FileName'] = basename($file);
             $data['MimeType'] = $mimeType;
@@ -248,7 +248,7 @@ public function getExifFromFile(string $file): Exif
     public function getIptcData(string $file): array
     {
         getimagesize($file, $info);
-        $arrData = array();
+        $arrData = [];
         if (isset($info['APP13'])) {
             try {
                 $iptc = iptcparse($info['APP13']);

lib/PHPExif/Exif.php

@@ -62,19 +62,19 @@ class Exif
     /**
      * The mapped EXIF data
      */
-    protected array $data = array();
+    protected array $data = [];
 
     /**
      * The raw EXIF data
      */
-    protected array $rawData = array();
+    protected array $rawData = [];
 
     /**
      * Class constructor
      *
      * @param array $data
      */
-    public function __construct(array $data = array())
+    public function __construct(array $data = [])
     {
         $this->setData($data);
     }

lib/PHPExif/Mapper/Exiftool.php

@@ -202,7 +202,7 @@ public function setNumeric(bool $numeric): Exiftool
      */
     public function mapRawData(array $data): array
     {
-        $mappedData = array();
+        $mappedData = [];
 
         foreach ($data as $field => $value) {
             if (!array_key_exists($field, $this->map)) {

lib/PHPExif/Mapper/FFprobe.php

@@ -90,7 +90,7 @@ class FFprobe extends AbstractMapper
      */
     public function mapRawData(array $data): array
     {
-        $mappedData = array();
+        $mappedData = [];
 
         foreach ($data as $field => $value) {
             if ($this->isSection($field) && is_array($value)) {

lib/PHPExif/Mapper/ImageMagick.php

@@ -123,7 +123,7 @@ class ImageMagick extends AbstractMapper
      */
     public function mapRawData(array $data): array
     {
-        $mappedData = array();
+        $mappedData = [];
 
         foreach ($data as $field => $value) {
             if (!array_key_exists($field, $this->map)) {

lib/PHPExif/Mapper/Native.php

@@ -147,7 +147,7 @@ class Native extends AbstractMapper
      */
     public function mapRawData(array $data): array
     {
-        $mappedData = array();
+        $mappedData = [];
 
         foreach ($data as $field => $value) {
             if ($this->isSection($field) && is_array($value)) {