Added support for loading images/resources to disk instead of holding them in memory

Author: Phil Wilkinson

src/PhpPresentation/Shape/Drawing/AbstractDrawingAdapter.php

@@ -35,7 +35,41 @@ abstract public function getMimeType(): string;
     abstract public function getPath(): string;
 
     /**
+     * @param string $path File path
      * @return self
      */
     abstract public function setPath(string $path);
+
+    /**
+     * Set whether this is a temporary file that should be cleaned up
+     *
+     * @param bool $isTemporary
+     * @return self
+     */
+    abstract public function setIsTemporaryFile(bool $isTemporary);
+
+    /**
+     * Load content into this object using a temporary file
+     *
+     * @param string $content Binary content
+     * @param string $fileName Optional fileName for reference
+     * @param string $prefix Prefix for the temporary file
+     * @return self
+     */
+    public function loadFromContent(string $content, string $fileName = '', string $prefix = 'PhpPresentation'): self
+    {
+        $tmpFile = tempnam(sys_get_temp_dir(), $prefix);
+        file_put_contents($tmpFile, $content);
+
+        // Set path and mark as temporary for automatic cleanup
+        $this->setPath($tmpFile);
+        $this->setIsTemporaryFile(true);
+
+        // Set filename if provided
+        if (!empty($fileName)) {
+            $this->setName($fileName);
+        }
+
+        return $this;
+    }
 }

src/PhpPresentation/Shape/Drawing/File.php

@@ -35,6 +35,11 @@ class File extends AbstractDrawingAdapter
      */
     protected $fileName = '';
 
+    /**
+     * @var bool Flag indicating if this is a temporary file that should be cleaned up
+     */
+    protected $isTemporaryFile = false;
+
     /**
      * Get Path.
      */
@@ -67,6 +72,28 @@ public function setPath(string $pValue = '', bool $pVerifyFile = true): self
         return $this;
     }
 
+    /**
+     * Set whether this is a temporary file that should be cleaned up
+     *
+     * @param bool $isTemporary
+     * @return self
+     */
+    public function setIsTemporaryFile(bool $isTemporary): self
+    {
+        $this->isTemporaryFile = $isTemporary;
+        return $this;
+    }
+
+    /**
+     * Check if this is a temporary file that should be cleaned up
+     *
+     * @return bool
+     */
+    public function isTemporaryFile(): bool
+    {
+        return $this->isTemporaryFile;
+    }
+
     public function getFileName(): string
     {
         return $this->fileName;
@@ -112,4 +139,36 @@ public function getIndexedFilename(): string
 
         return $output;
     }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function loadFromContent(string $content, string $fileName = '', string $prefix = 'PhpPresentation'): AbstractDrawingAdapter
+    {
+        // Create temporary file
+        $tmpFile = tempnam(sys_get_temp_dir(), $prefix);
+        file_put_contents($tmpFile, $content);
+
+        // Set path and mark as temporary
+        $this->setPath($tmpFile);
+        $this->setIsTemporaryFile(true);
+
+        // Set filename if provided
+        if (!empty($fileName)) {
+            $this->setFileName($fileName);
+        }
+
+        return $this;
+    }
+
+    /**
+     * Clean up resources when object is destroyed
+     */
+    public function __destruct()
+    {
+        // Remove temporary file if needed
+        if ($this->isTemporaryFile() && $this->path && file_exists($this->path)) {
+            @unlink($this->path);
+        }
+    }
 }

src/PhpPresentation/Shape/Drawing/Gd.php

@@ -39,7 +39,7 @@ class Gd extends AbstractDrawingAdapter
     /**
      * Image resource.
      *
-     * @var resource
+     * @var GdImage|resource|null
      */
     protected $imageResource;
 
@@ -64,6 +64,11 @@ class Gd extends AbstractDrawingAdapter
      */
     protected $uniqueName;
 
+    /**
+     * @var bool Flag indicating if this is a temporary file that should be cleaned up
+     */
+    protected $isTemporaryFile = false;
+
     /**
      * Gd constructor.
      */
@@ -77,7 +82,7 @@ public function __construct()
      * Get image resource.
      *
      * @param bool $isTransient Avoid the image resource being stored in memory to avoid OOM
-     * @return ?resource
+     * @return ?GdImage|?resource
      */
     public function getImageResource(bool $isTransient = false)
     {
@@ -240,9 +245,86 @@ public function getPath(): string
         return $this->path;
     }
 
+    /**
+     * Set Path.
+     *
+     * @param string $path File path
+     * @return self
+     */
     public function setPath(string $path): self
     {
         $this->path = $path;
+        return $this;
+    }
+
+    /**
+     * Set whether this is a temporary file that should be cleaned up
+     *
+     * @param bool $isTemporary
+     * @return self
+     */
+    public function setIsTemporaryFile(bool $isTemporary): self
+    {
+        $this->isTemporaryFile = $isTemporary;
+        return $this;
+    }
+
+    /**
+     * Check if this is a temporary file that should be cleaned up
+     *
+     * @return bool
+     */
+    public function isTemporaryFile(): bool
+    {
+        return $this->isTemporaryFile;
+    }
+
+    /**
+     * Clean up resources when object is destroyed
+     */
+    public function __destruct()
+    {
+        // Free GD image resource if it exists
+        if ($this->imageResource) {
+            imagedestroy($this->imageResource);
+            $this->imageResource = null;
+        }
+
+        // Remove temporary file if needed
+        if ($this->isTemporaryFile && !empty($this->path) && file_exists($this->path)) {
+            @unlink($this->path);
+        }
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function loadFromContent(string $content, string $fileName = '', string $prefix = 'PhpPresentationGd'): AbstractDrawingAdapter
+    {
+        $image = @imagecreatefromstring($content);
+        if ($image === false) {
+            return $this;
+        }
+
+        $tmpFile = tempnam(sys_get_temp_dir(), $prefix);
+        file_put_contents($tmpFile, $content);
+
+        // Set image resource
+        $this->setImageResource($image);
+
+        // Set path and mark as temporary for automatic cleanup
+        $this->setPath($tmpFile);
+        $this->setIsTemporaryFile(true);
+
+        if (!empty($fileName)) {
+            $this->setName($fileName);
+        }
+
+        $info = getimagesizefromstring($content);
+        if (isset($info['mime'])) {
+            $this->setMimeType($info['mime']);
+            $this->setRenderingFunction(str_replace('/', '', $info['mime']));
+        }
 
         return $this;
     }

src/PhpPresentation/Slide/Background/Image.php

@@ -21,6 +21,7 @@
 namespace PhpOffice\PhpPresentation\Slide\Background;
 
 use PhpOffice\PhpPresentation\Exception\FileNotFoundException;
+use PhpOffice\PhpPresentation\Shape\Drawing\AbstractDrawingAdapter;
 use PhpOffice\PhpPresentation\Slide\AbstractBackground;
 
 class Image extends AbstractBackground
@@ -47,6 +48,11 @@ class Image extends AbstractBackground
      */
     protected $width;
 
+    /**
+     * @var AbstractDrawingAdapter|null
+     */
+    protected $image;
+
     /**
      * Get Path.
      */
@@ -63,7 +69,7 @@ public function getPath(): ?string
      *
      * @return self
      */
-    public function setPath(string $pValue = '', bool $pVerifyFile = true)
+    public function setPath(string $pValue = '', bool $pVerifyFile = true): Image
     {
         if ($pVerifyFile) {
             if (!file_exists($pValue)) {
@@ -80,6 +86,20 @@ public function setPath(string $pValue = '', bool $pVerifyFile = true)
         return $this;
     }
 
+    /**
+     * Set the image using a drawing adapter (keeps a reference to the object to manage file lifecycle)
+     *
+     * @param AbstractDrawingAdapter $image Drawing adapter containing image data
+     * @return self
+     * @throws FileNotFoundException
+     */
+    public function setImage(AbstractDrawingAdapter $image): self
+    {
+        $this->image = $image;
+        $this->setPath($image->getPath());
+        return $this;
+    }
+
     /**
      * Get Filename.
      */
@@ -105,7 +125,7 @@ public function getExtension(): string
      *
      * @return string
      */
-    public function getIndexedFilename($numSlide)
+    public function getIndexedFilename($numSlide): string
     {
         return 'background_' . $numSlide . '.' . $this->getExtension();
     }