Merge pull request #2 from stefliekens/identify-multipages

Make sure to select the first image when parsing multi-paged images

composer.json

@@ -19,7 +19,7 @@
     "zendframework/zend-stdLib": "~2.3"
   },
   "require-dev": {
-    "phpspec/phpspec": "dev-master",
+    "phpspec/phpspec": "~2.0.1",
     "fabpot/PHP-CS-Fixer": "*"
   },
   "autoload": {

spec/Phpro/Filesystem/Metadata/Image/IdentifySpec.php

@@ -22,9 +22,10 @@ public function it_is_initializable()
     public function it_should_be_able_to_load_metadata(Filesystem $filesystem, \Imagick $imagick)
     {
         $file = new LocalFile($this->file);
-        $imagick->readimage($this->file)->shouldBeCalled();
-        $imagick->identifyimage()->willReturn(['key' => 'value']);
-        $imagick->getimageproperties('*SpotColor*')->willReturn([1]);
+        $imagick->readImage($this->file)->shouldBeCalled();
+        $imagick->getNumberImages()->willReturn(0);
+        $imagick->identifyImage()->willReturn(['key' => 'value']);
+        $imagick->getImageProperties('*SpotColor*')->willReturn([1]);
         $imagick->clear()->shouldBeCalled();
 
         $this->getMetadataForFile($file, ['extended' => true])->shouldReturn([

src/Phpro/Filesystem/Metadata/Image/Identify.php

@@ -53,6 +53,8 @@ protected function parseIdentify(FileInterface $file, array $options = [])
             $source = $this->alterSource($file->getPath(), $format);
 
             $image->readImage($source);
+            $this->handleMultiPages($image);
+
             $identifyData = $image->identifyImage();
 
             if (isset($options['extended']) && $options['extended']) {
@@ -98,4 +100,16 @@ protected function alterSource($source, $format)
 
         return $source;
     }
+
+    /**
+     * Make sure to select the first image when parsing multi-paged images.
+     *
+     * @param Imagick $image
+     */
+    protected function handleMultiPages($image)
+    {
+        if ($image->getNumberImages() > 1) {
+            $image->setIteratorIndex(0);
+        }
+    }
 }