Fix loop validation input context is mutable.

Fix #16

Author: Maks3w

src/BaseInputFilter.php

@@ -231,6 +231,8 @@ public function isValid($context = null)
      */
     protected function validateInputs(array $inputs, $data = [], $context = null)
     {
+        $inputContext = $context ?: (array_merge($this->getRawValues(), (array) $data));
+
         $this->validInputs   = [];
         $this->invalidInputs = [];
         $valid               = true;
@@ -261,14 +263,7 @@ protected function validateInputs(array $inputs, $data = [], $context = null)
                 continue;
             }
 
-            // Make sure we have a value (empty) for validation of context
-            if (!array_key_exists($name, $data)) {
-                $data[$name] = null;
-            }
-
             // Validate an input
-            $inputContext = $context ?: $data;
-
             if (!$input->isValid($inputContext)) {
                 // Validation failure
                 $this->invalidInputs[$name] = $input;

test/BaseInputFilterTest.php

@@ -445,10 +445,84 @@ public function testCanConditionallyInvokeValidators()
     }
      */
 
-    /**
+    /*
      * Idea for this one is that validation may need to rely on context -- e.g., a "password confirmation"
      * field may need to know what the original password entered was in order to compare.
      */
+
+    public function contextProvider()
+    {
+        $data = ['fooInput' => 'fooValue'];
+        $arrayAccessData = new ArrayObject(['fooInput' => 'fooValue']);
+        $expectedFromData = ['fooInput' => 'fooValue'];
+
+        return [
+            // Description => [$data, $customContext, $expectedContext]
+            'by default get context from data (array)' => [$data, null, $expectedFromData],
+            'by default get context from data (ArrayAccess)' => [$arrayAccessData, null, $expectedFromData],
+            'use custom context' => [[], 'fooContext', 'fooContext'],
+        ];
+    }
+
+    /**
+     * @dataProvider contextProvider
+     */
+    public function testValidationContext($data, $customContext, $expectedContext)
+    {
+        $filter = new InputFilter();
+
+        $input = $this->createInputInterfaceMock(true, true, $expectedContext);
+        $filter->add($input, 'fooInput');
+
+        $filter->setData($data);
+
+        $this->assertTrue(
+            $filter->isValid($customContext),
+            'isValid() value not match. Detail: ' . json_encode($filter->getMessages())
+        );
+    }
+
+    public function testBuildValidationContextUsingInputGetRawValue()
+    {
+        $data = [];
+        $expectedContext = ['fooInput' => 'fooRawValue'];
+        $filter = new InputFilter();
+
+        $input = $this->createInputInterfaceMock(true, true, $expectedContext, 'fooRawValue');
+        $filter->add($input, 'fooInput');
+
+        $filter->setData($data);
+
+        $this->assertTrue(
+            $filter->isValid(),
+            'isValid() value not match. Detail: ' . json_encode($filter->getMessages())
+        );
+    }
+
+    public function testContextIsTheSameWhenARequiredInputIsGivenAndOptionalInputIsMissing()
+    {
+        $data = [
+            'inputRequired' => 'inputRequiredValue',
+        ];
+        $expectedContext = [
+            'inputRequired' => 'inputRequiredValue',
+            'inputOptional' => null,
+        ];
+        $inputRequired = $this->createInputInterfaceMock(true, true, $expectedContext);
+        $inputOptional = $this->createInputInterfaceMock(false);
+
+        $filter = new InputFilter();
+        $filter->add($inputRequired, 'inputRequired');
+        $filter->add($inputOptional, 'inputOptional');
+
+        $filter->setData($data);
+
+        $this->assertTrue(
+            $filter->isValid(),
+            'isValid() value not match. Detail: ' . json_encode($filter->getMessages())
+        );
+    }
+
     public function testValidationCanUseContext()
     {
         $filter = new InputFilter();
@@ -1051,4 +1125,36 @@ public function testAllowsValidatingArrayAccessData()
         $filter->setData($data);
         $this->assertTrue($filter->isValid());
     }
+
+    /**
+     * @param null|bool $isValid
+     * @param mixed $expectedContext
+     * @param mixed $getRawValue
+     *
+     * @return MockObject|InputInterface
+     */
+    protected function createInputInterfaceMock($isRequired, $isValid = null, $expectedContext = 'not-set', $getRawValue = 'not-set')
+    {
+        /** @var InputInterface|MockObject $input */
+        $input = $this->getMock(InputInterface::class);
+        $input->method('isRequired')
+            ->willReturn($isRequired)
+        ;
+        if ($getRawValue !== 'not-set') {
+            $input->method('getRawValue')
+                ->willReturn($getRawValue)
+            ;
+        }
+        if ($isValid !== null) {
+            $mockMethod = $input->expects($this->once())
+                ->method('isValid')
+                ->willReturn($isValid)
+            ;
+            if ($expectedContext !== 'not-set') {
+                $mockMethod->with($expectedContext);
+            }
+        }
+
+        return $input;
+    }
 }