Present is no longer ignoring other rules when field is empty

Author: ThHareau

src/Rules/Present.php

@@ -20,6 +20,20 @@ class Present extends Rule
      */
     public function check($value): bool
     {
+        $this->setAttributeAsRequired();
+
         return $this->validation->hasValue($this->attribute->getKey());
     }
+
+    /**
+     * Set attribute is required if $this->attribute is set
+     *
+     * @return void
+     */
+    protected function setAttributeAsRequired()
+    {
+        if ($this->attribute) {
+            $this->attribute->setRequired(true);
+        }
+    }
 }

src/Rules/Required.php

@@ -38,7 +38,7 @@ public function check($value): bool
     }
 
     /**
-     * Set attribute is required if $this->attribute is true
+     * Set attribute is required if $this->attribute is set
      *
      * @return void
      */

tests/ValidatorTest.php

@@ -681,6 +681,22 @@ public function testIgnoreNextRulesWhenImplicitRulesFails()
         $this->assertNull($errors->first('must_accepted_field:min'));
     }
 
+    public function testNextRulesAppliedWhenEmptyValueWithPresent()
+    {
+        $validation = $this->validator->validate([
+            'must_present_field' => '',
+        ], [
+            'must_present_field' => 'present|array',
+        ]);
+
+        $errors = $validation->errors();
+
+        $this->assertEquals($errors->count(), 1);
+
+        $this->assertNull($errors->first('must_present_field:present'));
+        $this->assertNotNull($errors->first('must_present_field:array'));
+    }
+
     public function testIgnoreOtherRulesWhenAttributeIsNotRequired()
     {
         $validation = $this->validator->validate([