OpenStack\ObjectStore\v1\Models\Object

Complexity

Total Complexity

10

Size/Duplication

Total Lines	144
Duplicated Lines	22.22 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes	7
Bugs	1	Features	6

10 Methods

Rating	Name	Duplication	Size	Complexity
A	populateFromResponse()	10	10	1
A	getPublicUri()	0	4	1
A	create()	0	5	1
A	retrieve()	0	5	1
A	download()	0	5	1
A	delete()	0	4	1
A	copy()	0	5	1
A	mergeMetadata()	11	11	1
A	resetMetadata()	11	11	1
A	getMetadata()	0	5	1

<?php

namespace OpenStack\ObjectStore\v1\Models;

use OpenStack\Common\Transport\Utils;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\StreamInterface;
use OpenStack\Common\Resource\AbstractResource;
use OpenStack\Common\Resource\Creatable;
use OpenStack\Common\Resource\Deletable;
use OpenStack\Common\Resource\HasMetadata;

/**
 * @property \OpenStack\ObjectStore\v1\Api $api
 */
class Object extends AbstractResource implements Creatable, Deletable, HasMetadata
{
    use MetadataTrait;

    const METADATA_PREFIX = 'X-Object-Meta-';

    /** @var string */
    public $containerName;

    /** @var string */
    public $name;

    /** @var string */
    public $hash;

    /** @var string */
    public $contentType;

    /** @var int */
    public $contentLength;

    /** @var string */
    public $lastModified;

    /** @var array */
    public $metadata;

    protected $markerKey = 'name';
    protected $aliases = ['bytes' => 'contentLength'];

    /**
     * {@inheritdoc}
     */
    public function populateFromResponse(ResponseInterface $response)

This method seems to be duplicated in your project.

    {
        parent::populateFromResponse($response);

        $this->hash = $response->getHeaderLine('ETag');
        $this->contentLength = $response->getHeaderLine('Content-Length');

The property $contentLength was declared of type integer, but $response->getHeaderLine('Content-Length') is of type string. Maybe add a type cast?

        $this->lastModified = $response->getHeaderLine('Last-Modified');
        $this->contentType = $response->getHeaderLine('Content-Type');
        $this->metadata = $this->parseMetadata($response);
    }

    /**
     * Retrieves the public URI for this resource.
     *
     * @return \GuzzleHttp\Psr7\Uri
     */
    public function getPublicUri()
    {
        return Utils::addPaths($this->getHttpBaseUrl(), $this->containerName, $this->name);

$this->containerName is of type string, but the function expects a array<integer,?>.

The call to Utils::addPaths() has too many arguments starting with $this->name.

    }

    /**
     * @param array $data {@see \OpenStack\ObjectStore\v1\Api::putObject}
     *
     * @return $this
     */
    public function create(array $data)
    {
        $response = $this->execute($this->api->putObject(), $data + ['containerName' => $this->containerName]);
        return $this->populateFromResponse($response);
    }

    /**
     * {@inheritdoc}
     */
    public function retrieve()
    {
        $response = $this->executeWithState($this->api->headObject());
        $this->populateFromResponse($response);
    }

    /**
     * This call will perform a `GET` HTTP request for the given object and return back its content in the form of a
     * Guzzle Stream object. Downloading an object will transfer all of the content for an object, and is therefore
     * distinct from fetching its metadata (a `HEAD` request). The body of an object is not fetched by default to
     * improve performance when handling large objects.
     *
     * @return StreamInterface
     */
    public function download()
    {
        $response = $this->executeWithState($this->api->getObject());
        return $response->getBody();
    }

    /**
     * {@inheritdoc}
     */
    public function delete()
    {
        $this->executeWithState($this->api->deleteObject());
    }

    /**
     * @param array $options {@see \OpenStack\ObjectStore\v1\Api::copyObject}
     */
    public function copy(array $options)
    {
        $options += ['name' => $this->name, 'containerName' => $this->containerName];
        $this->execute($this->api->copyObject(), $options);
    }

    /**
     * {@inheritdoc}
     */
    public function mergeMetadata(array $metadata)

This method seems to be duplicated in your project.

    {
        $options = [
            'containerName' => $this->containerName,
            'name'          => $this->name,
            'metadata'      => array_merge($metadata, $this->getMetadata()),
        ];

        $response = $this->execute($this->api->postObject(), $options);
        return $this->parseMetadata($response);
    }

    /**
     * {@inheritdoc}
     */
    public function resetMetadata(array $metadata)

This method seems to be duplicated in your project.

    {
        $options = [
            'containerName'  => $this->containerName,
            'name'           => $this->name,
            'metadata'       => $metadata,
        ];

        $response = $this->execute($this->api->postObject(), $options);
        return $this->parseMetadata($response);
    }

    /**
     * {@inheritdoc}
     */
    public function getMetadata()
    {
        $response = $this->executeWithState($this->api->headObject());
        return $this->parseMetadata($response);
    }
}