Only use coalesceImages() when needed

Maybe we could add a check like one that already exists at:
web/modules/contrib/imagick/src/Plugin/ImageToolkit/Operation/imagick/ImagickOperationTrait.php line 36:

    if (isset($image_format) && in_array($image_format, ['GIF', 'WEBP'])) {
      // Get each frame in the GIF
      $resource = $resource->coalesceImages();
      do {
        if (!$this->process($resource, $arguments)) {
          $success = FALSE;
          break;
        }
      } while ($resource->nextImage());

      $resource->deconstructImages();
    }

And then we could change inside ImagickToolkit.php this:

  public function getWidth() {
    if (!$this->getResource()) {
      return NULL;
    }

    return $this->getResource()->coalesceImages()->getImageGeometry()['width'];
  }

into something like:

  public function getWidth() {
    $resource = $this->getResource();
    if (!$resource) {
      return NULL;
    }

    // If preferred format is set, use it as prefix for writeImage
    // If not this will throw a ImagickException exception
    try {
      $image_format = $resource->getImageFormat();
    } catch (ImagickException $e) {}

    // Use coalesceImages() for images with multiple frames.
    if (isset($image_format) && in_array($image_format, ['GIF', 'WEBP'])) {
      return $this->getResource()->coalesceImages()->getImageGeometry()['width'];
    }

    // Otherwise, get with without using coalesceImages().
    return $this->getResource()->getImageGeometry()['width'];
  }

and the same goes for the getHeight() function.

Attempt at a patch.

bceyssens → credited j_bekaert → .

thank you for the patch @flyke

Automatically closed - issue fixed for 2 weeks with no activity.