Add submitWithAdditionalValues() method in Symfony\Component\BrowserKit\Client

alexislefebvre · alexislefebvre · commit 582a494d4104 · 2016-03-27T16:42:12.000+02:00
Extract code that converts fields to arrays in a new method convertFieldsToArray()
diff --git a/src/Symfony/Component/BrowserKit/Client.php b/src/Symfony/Component/BrowserKit/Client.php
@@ -262,6 +262,31 @@ public function submit(Form $form, array $values = array())
         return $this->request($form->getMethod(), $form->getUri(), $form->getPhpValues(), $form->getPhpFiles());
     }
 
+    /**
+     * Submits a form.
+     *
+     * @param Form  $form             A Form instance
+     * @param array $values           An array of form field values
+     * @param array $additionalValues An array of additional field values
+     *
+     * @return Crawler
+     */
+    public function submitWithAdditionalValues(Form $form, array $values = array(), $additionalValues = array())
+    {
+        $form->setValues($values);
+
+        $values = $form->getPhpValues();
+
+        if (!empty($additionalValues)) {
+            $values = array_merge_recursive(
+                $values,
+                $form->convertFieldsToArray($additionalValues)
+            );
+        }
+
+        return $this->request($form->getMethod(), $form->getUri(), $values, $form->getPhpFiles());
+    }
+
     /**
      * Calls a URI.
      *
diff --git a/src/Symfony/Component/BrowserKit/Tests/ClientTest.php b/src/Symfony/Component/BrowserKit/Tests/ClientTest.php
@@ -286,6 +286,43 @@ public function testSubmit()
         $this->assertEquals('http://www.example.com/foo', $client->getRequest()->getUri(), '->submit() submit forms');
     }
 
+    /**
+     * @expectedException \InvalidArgumentException
+     * @expectedExceptionMessage Unreachable field "foo"
+     */
+    public function testSubmitMissingField()
+    {
+        $client = new TestClient();
+        $client->setNextResponse(new Response('<html><form action="/foo"><input type="submit" /></form></html>'));
+        $crawler = $client->request('GET', 'http://www.example.com/foo/foobar');
+
+        $client->submit($crawler->filter('input')->form(), array('foo[0]' => 'bar'));
+    }
+
+    /**
+     * Submit the same data that in testSubmitMissingField() but
+     * without triggering an Exception.
+     */
+    public function testSubmitWithAdditionalValues()
+    {
+        $client = new TestClient();
+        $client->setNextResponse(new Response('<html><form action="/foo"><input type="submit" /></form></html>'));
+        $crawler = $client->request('GET', 'http://www.example.com/foo/foobar');
+
+        $client->submitWithAdditionalValues($crawler->filter('input')->form(), array(), array('foo[0]' => 'bar'));
+
+        $this->assertEquals('http://www.example.com/foo', $client->getRequest()->getUri(), '->submit() submit forms');
+
+        // The field "foo[0]" have been converted to an array "foo => 0".
+        $this->assertEquals(
+            array('foo' => array(
+                '0' => 'bar',
+            )),
+            $client->getRequest()->getParameters(),
+            'parameters have not been added'
+        );
+    }
+
     public function testSubmitPreserveAuth()
     {
         $client = new TestClient(array('PHP_AUTH_USER' => 'foo', 'PHP_AUTH_PW' => 'bar'));
diff --git a/src/Symfony/Component/DomCrawler/Form.php b/src/Symfony/Component/DomCrawler/Form.php
@@ -130,17 +130,16 @@ public function getFiles()
     }
 
     /**
-     * Gets the field values as PHP.
-     *
      * This method converts fields with the array notation
      * (like foo[bar] to arrays) like PHP does.
      *
      * @return array An array of field values.
      */
-    public function getPhpValues()
+    public function convertFieldsToArray($fields)
     {
         $values = array();
-        foreach ($this->getValues() as $name => $value) {
+
+        foreach ($fields as $name => $value) {
             $qs = http_build_query(array($name => $value), '', '&');
             if (!empty($qs)) {
                 parse_str($qs, $expandedValue);
@@ -152,6 +151,16 @@ public function getPhpValues()
         return $values;
     }
 
+    /**
+     * Gets the field values as PHP.
+     *
+     * @return array An array of field values.
+     */
+    public function getPhpValues()
+    {
+        return $this->convertFieldsToArray($this->getValues());
+    }
+
     /**
      * Gets the file field values as PHP.
      *
