GuzzleHttp\Command\Guzzle\RequestLocation\MultiPartLocation

Complexity

Total Complexity

4

Size/Duplication

Total Lines	64
Duplicated Lines	18.75 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	visit()	12	12	1
A	after()	0	19	2

<?php
namespace GuzzleHttp\Command\Guzzle\RequestLocation;

use GuzzleHttp\Command\CommandInterface;
use GuzzleHttp\Command\Guzzle\Operation;
use GuzzleHttp\Command\Guzzle\Parameter;
use GuzzleHttp\Psr7;
use Psr\Http\Message\RequestInterface;

/**
 * Adds POST files to a request
 */
class MultiPartLocation extends AbstractLocation
{
    /** @var string $contentType */
    protected $contentType = 'multipart/form-data; boundary=';

    /** @var array $formParamsData */
    protected $multipartData = [];

    /**
     * Set the name of the location
     *
     * @param string $locationName
     */
    public function __construct($locationName = 'multipart')
    {
        parent::__construct($locationName);
    }

    /**
     * @param CommandInterface $command
     * @param RequestInterface $request
     * @param Parameter $param
     * @return RequestInterface
     */
    public function visit(

This method seems to be duplicated in your project.

        CommandInterface $command,
        RequestInterface $request,
        Parameter $param
    ) {
        $this->multipartData[] = [
            'name' => $param->getWireName(),
            'contents' => $this->prepareValue($command[$param->getName()], $param)

Consider using $param->name. There is an issue with getName() and APC-enabled PHP versions.

        ];

        return $request;
    }


    /**
     * @param CommandInterface $command
     * @param RequestInterface $request
     * @param Operation $operation
     * @return RequestInterface
     */
    public function after(
        CommandInterface $command,
        RequestInterface $request,
        Operation $operation
    ) {
        $data = $this->multipartData;
        $this->multipartData = [];
        $modify = [];

        $body = new Psr7\MultipartStream($data);
        $modify['body'] = Psr7\stream_for($body);
        $request = Psr7\modify_request($request, $modify);
        if ($request->getBody() instanceof Psr7\MultipartStream) {

The class GuzzleHttp\Psr7\MultipartStream does not exist. Did you forget a USE statement, or did you not list all dependencies?

            // Use a multipart/form-data POST if a Content-Type is not set.
            $request->withHeader('Content-Type', $this->contentType . $request->getBody()->getBoundary());
        }

        return $request;
    }
}