Igni\Network\Http\Stream

Complexity

Total Complexity

11

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

50%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	fromString()	0	8	1
A	fromFile()	0	8	2
B	create()	0	18	8

<?php declare(strict_types=1);

namespace Igni\Network\Http;

use Igni\Exception\InvalidArgumentException;
use Psr\Http\Message\StreamInterface;
use Zend\Diactoros\Stream as BaseStream;

use function strpos;
use function is_string;
use function is_resource;

/**
 * @package Igni\Http
 */
class Stream extends BaseStream
{
    public static function fromString(string $content): self
    {
        $stream = new self('php://temp', 'wb+');
        $stream->write($content);
        $stream->rewind();

        return $stream;
    }

    public static function fromFile(string $path, string $mode = 'r'): self
    {
        if ($path === 'php://input') {
            return new self($path, 'r');
        }

        return new self($path, $mode);
    }

    public static function create($stream, string $mode = 'r'): self
    {
        if (!is_string($stream) && !is_resource($stream) && !$stream instanceof StreamInterface) {
            throw new InvalidArgumentException(
                'Stream must be a string stream resource identifier, '
                . 'an actual stream resource, '
                . 'or a Psr\Http\Message\StreamInterface implementation'
            );
        }

        if (is_string($stream) && (empty($stream) || 0 !== strpos('php://', $stream))) {
            return self::fromString($stream);
        }

        return $stream instanceof StreamInterface
            ? $stream
            : self::fromFile($stream, $mode);

It seems like $stream can also be of type resource; however, Igni\Network\Http\Stream::fromFile() does only seem to accept string, maybe add an additional type check?

    }
}