Allow 0 for and parameters in

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 All Notable changes to `laravel-medialibrary` will be documented in this file
 
+##3.2.5
+- Allow 0 for `x` and `y` parameters in `setRectangle`
+
 ##3.2.4
 - Removed dependency on spatie/eloquent-sortable
 

src/Conversion/Conversion.php

@@ -301,8 +301,14 @@ public function setFit($fit)
     public function setRectangle($width, $height, $x, $y)
     {
         foreach (compact('width', 'height', 'x', 'y') as $name => $value) {
-            if (!is_numeric($value) || $value < 1) {
-                throw new InvalidConversionParameter($name.' should be numeric and greater than 1');
+            if (! is_numeric($value)) {
+                throw new InvalidConversionParameter($name.' should be numeric');
+            }
+        }
+
+        foreach (compact('width', 'height') as $name => $value) {
+            if ($value < 1) {
+                throw new InvalidConversionParameter($name.' should be greater than 1');
             }
         }
 

tests/Conversion/ConversionTest.php

@@ -231,7 +231,19 @@ public function it_can_add_rectangle_to_a_manipulation()
     /**
      * @test
      */
-    public function it_throw_an_exception_for_an_invalid_rectangle()
+    public function it_allows_zero_x_y_coordinates_in_rectangle_manipulations()
+    {
+        $conversion = $this->conversion->setRectangle(100, 200, 0, 0);
+
+        $this->arrayHasKey('rect', $this->conversion->getManipulations()[0]);
+        $this->assertEquals('100,200,0,0', $this->conversion->getManipulations()[0]['rect']);
+        $this->assertInstanceOf(\Spatie\MediaLibrary\Conversion\Conversion::class, $conversion);
+    }
+
+    /**
+     * @test
+     */
+    public function it_throws_an_exception_for_an_invalid_rectangle()
     {
         $this->setExpectedException(\Spatie\MediaLibrary\Exceptions\InvalidConversionParameter::class);
         $this->conversion->setRectangle('blabla', 200, 300, 400);