Binary: Rewind stream after reading contents

Icinga · Oct 20, 2023 · 44bd711 · 44bd711
1 parent fab2c91
commit 44bd711
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/src/Behavior/Binary.php b/src/Behavior/Binary.php
@@ -29,7 +29,10 @@ public function fromDb($value, $key, $_)
 
         if ($value !== null) {
             if (is_resource($value)) {
-                return stream_get_contents($value);
+                $content = stream_get_contents($value);
+                rewind($value);
+
+                return $content;
             }
 
             return $value;

diff --git a/tests/Behavior/BinaryTest.php b/tests/Behavior/BinaryTest.php
@@ -37,6 +37,22 @@ public function testRetrievePropertyReturnsStreamContentsIfAdapterIsPostgreSQL()
         );
     }
 
+    public function testRetrievePropertyRewindsAStreamIfAdapterIsPostgreSQL(): void
+    {
+        $stream = fopen('php://temp', 'r+');
+        fputs($stream, static::TEST_BINARY_VALUE);
+        rewind($stream);
+
+        $this->assertSame(
+            static::TEST_BINARY_VALUE,
+            $this->behavior(true)->retrieveProperty($stream, static::TEST_COLUMN)
+        );
+        $this->assertSame(
+            static::TEST_BINARY_VALUE,
+            $this->behavior(true)->retrieveProperty($stream, static::TEST_COLUMN)
+        );
+    }
+
     public function testPersistPropertyReturnsVanillaValueIfAdapterIsNotPostgreSQL(): void
     {
         $this->assertSame(