symfony
diff --git a/‎src/Symfony/Component/BrowserKit/HttpBrowser.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/BrowserKit/HttpBrowser.php
+1-1Lines changed: 1 addition & 1 deletion b/‎src/Symfony/Component/BrowserKit/HttpBrowser.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/BrowserKit/HttpBrowser.php
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Symfony/Component/BrowserKit/Tests/HttpBrowserTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/BrowserKit/Tests/HttpBrowserTest.php
+40Lines changed: 40 additions & 0 deletions b/‎src/Symfony/Component/BrowserKit/Tests/HttpBrowserTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/BrowserKit/Tests/HttpBrowserTest.php
+40Lines changed: 40 additions & 0 deletions
@@ -75,7 +75,7 @@ private function getBodyAndExtraHeaders(Request $request): array
         $fields = $request->getParameters();
 
         if ($uploadedFiles = $this->getUploadedFiles($request->getFiles())) {
-            $part = new FormDataPart($uploadedFiles);
+            $part = new FormDataPart(array_merge($fields, $uploadedFiles));
 
             return [$part->bodyToIterable(), $part->getPreparedHeaders()->toArray()];
         }
 
@@ -134,6 +134,28 @@ public function testMultiPartRequestWithInvalidItem()
         ]);
     }
 
+    public function testMultiPartRequestWithAdditionalParameters()
+    {
+        $client = $this->createMock(HttpClientInterface::class);
+        $this->expectClientToSendRequestWithFiles($client, ['file1_content', 'baz']);
+
+        $browser = new HttpBrowser($client);
+        $browser->request('POST', 'http://example.com/', ['bar' => 'baz'], [
+            'file1' => $this->getUploadedFile('file1'),
+        ]);
+    }
+
+    public function testMultiPartRequestWithAdditionalParametersOfTheSameName()
+    {
+        $client = $this->createMock(HttpClientInterface::class);
+        $this->expectClientToNotSendRequestWithFiles($client, ['baz']);
+
+        $browser = new HttpBrowser($client);
+        $browser->request('POST', 'http://example.com/', ['file1' => 'baz'], [
+            'file1' => $this->getUploadedFile('file1'),
+        ]);
+    }
+
     private function uploadFile(string $data): string
     {
         $path = tempnam(sys_get_temp_dir(), 'http');
@@ -167,4 +189,22 @@ protected function expectClientToSendRequestWithFiles(HttpClientInterface $clien
             }))
             ->willReturn($this->createMock(ResponseInterface::class));
     }
+
+    protected function expectClientToNotSendRequestWithFiles(HttpClientInterface $client, $fileContents)
+    {
+        $client
+            ->expects($this->once())
+            ->method('request')
+            ->with('POST', 'http://example.com/', $this->callback(function ($options) use ($fileContents) {
+                $this->assertStringContainsString('Content-Type: multipart/form-data', implode('', $options['headers']));
+                $this->assertInstanceOf('\Generator', $options['body']);
+                $body = implode('', iterator_to_array($options['body'], false));
+                foreach ($fileContents as $content) {
+                    $this->assertStringNotContainsString($content, $body);
+                }
+
+                return true;
+            }))
+            ->willReturn($this->createMock(ResponseInterface::class));
+    }
 }
Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ private function getBodyAndExtraHeaders(Request $request): array`
`75`	`75`	`$fields = $request->getParameters();`
`76`	`76`
`77`	`77`	`if ($uploadedFiles = $this->getUploadedFiles($request->getFiles())) {`
`78`		`- $part = new FormDataPart($uploadedFiles);`
	`78`	`+ $part = new FormDataPart(array_merge($fields, $uploadedFiles));`
`79`	`79`
`80`	`80`	`return [$part->bodyToIterable(), $part->getPreparedHeaders()->toArray()];`
`81`	`81`	`}`