Skip to content

Commit 4d1199b

Browse files
gpenvernesanpii
authored andcommitted
Do not transform not string files before send to client
1 parent 5a0d505 commit 4d1199b

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

src/HttpCall/Request/BrowserKit.php

+5-5
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,9 @@ public function getContent()
5858
public function send($method, $url, $parameters = [], $files = [], $content = null, $headers = [])
5959
{
6060
foreach ($files as $originalName => &$file) {
61-
$file = new UploadedFile($file, $originalName);
61+
if (is_string($file)) {
62+
$file = new UploadedFile($file, $originalName);
63+
}
6264
}
6365

6466
$client = $this->mink->getSession()->getDriver()->getClient();
@@ -109,12 +111,10 @@ public function getHttpHeader($name)
109111
if (isset($headers[$name])) {
110112
if (is_array($headers[$name])) {
111113
$value = implode(', ', $headers[$name]);
112-
}
113-
else {
114+
} else {
114115
$value = $headers[$name];
115116
}
116-
}
117-
else {
117+
} else {
118118
throw new \OutOfBoundsException(
119119
"The header '$name' doesn't exist"
120120
);

src/HttpCall/Request/Goutte.php

+3-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,9 @@ class Goutte extends BrowserKit
1717
public function send($method, $url, $parameters = [], $files = [], $content = null, $headers = [])
1818
{
1919
foreach ($files as $originalName => &$file) {
20-
$file = new UploadedFile($file, $originalName);
20+
if (is_string($file)) {
21+
$file = new UploadedFile($file, $originalName);
22+
}
2123
}
2224
$page = parent::send($method, $url, $parameters, $files, $content, $this->requestHeaders);
2325
$this->resetHttpHeaders();

0 commit comments

Comments
 (0)