Skip to content

Remove our stream implementation #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jul 14, 2016
Merged

Remove our stream implementation #1

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
ba28c21
Removed stream implementation
Nyholm Jul 14, 2016
7b3e245
update readme
Nyholm Jul 14, 2016
997c6a3
minor
Nyholm Jul 14, 2016
963307f
Added tests
Nyholm Jul 14, 2016
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
minor
  • Loading branch information
@Nyholm
Nyholm committed Jul 14, 2016
commit 997c6a30317c48f0ce24b2af1a1cce339e3b33ec
15 changes: 7 additions & 8 deletions src/MultipartStreamBuilder.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ class MultipartStreamBuilder
private $boundary;

/**
* @var array array Element where each Element is an array with keys ['contents', 'headers', 'filename']
* @var array Element where each Element is an array with keys ['contents', 'headers', 'filename']
*/
private $data;

Expand Down Expand Up @@ -83,26 +83,25 @@ public function addResource($name, $resource, array $options)
* Build the stream.
*
* @return StreamInterface
* @throws \Exception
*/
public function build()
{
$streams = [];
foreach ($this->data as $name => $data) {
$streams = '';
foreach ($this->data as $data) {

// Add start and headers
$streams[] = "--{$this->getBoundary()}\r\n".
$streams .= "--{$this->getBoundary()}\r\n".
$this->getHeaders($data['headers'])."\r\n\r\n";

// Convert the stream to string
$streams[] = (string) $data['contents'];
$streams[] .= "\r\n";
$streams .= (string) $data['contents'];
$streams .= "\r\n";
}

// Append end
$streams[] = "--{$this->getBoundary()}--\r\n";

return $this->streamFactory->createStream(implode('', $streams));
return $this->streamFactory->createStream($streams);
}

/**
Expand Down