Handle case where viewportHeigh is equal to heighOffset

1FF · Sep 20, 2024 · 1daed76 · 1daed76
1 parent 61b8eee
commit 1daed76
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/src/Codeception/Module/VisualCeption.php b/src/Codeception/Module/VisualCeption.php
@@ -521,12 +521,14 @@ private function createScreenshot($identifier, array $coords, $fullScreenshot, a
                 }
             }
 
-            $screenshotBinary = $this->webDriver->takeScreenshot();
-            $screenShotImage->readimageblob($screenshotBinary);
-            $heightOffset = $viewportHeight - ($height - (intval($itr) * $viewportHeight));
+            if (!is_int($itr) || $height <= $viewportHeight) {
+                $screenshotBinary = $this->webDriver->takeScreenshot();
+                $screenShotImage->readimageblob($screenshotBinary);
+                $heightOffset = $viewportHeight - ($height - (intval($itr) * $viewportHeight));
 
-            if ($isViewPortHeightBiggerThanPageHeight) {
-                $screenShotImage->cropImage(0, 0, 0, $heightOffset * $devicePixelRatio);
+                if ($isViewPortHeightBiggerThanPageHeight) {
+                    $screenShotImage->cropImage(0, 0, 0, $heightOffset * $devicePixelRatio);
+                }
             }
 
             $screenShotImage->resetIterator();