drupol\valuewrapper\Resource\ResourceValue

Complexity

Total Complexity

4

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

40%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	hash()	0	4	1
A	doHash()	0	4	1
A	__construct()	0	11	2

<?php

declare(strict_types = 1);

namespace drupol\valuewrapper\Resource;

use drupol\valuewrapper\AbstractValue;

abstract class ResourceValue extends AbstractValue
{
    /**
     * ResourceValue constructor.
     *
     * @param mixed $value
     */
    public function __construct($value)
    {
        if (false == \is_resource($value)) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            throw new \TypeError(
                'Argument 1 passed to drupol\valuewrapper\Resource\ResourceValue::__construct()' .

The call to TypeError::__construct() has too many arguments starting with 'Argument 1 passed to drupol\\valuewrapper\\Resource\\ResourceValue::__construct()' . 'must be of the type Resource, ' . gettype($value) . ' given.'.

                'must be of the type Resource, ' . gettype($value) . ' given.'
            );
        }

        parent::__construct($value);
    }

    /**
     * {@inheritdoc}
     */
    public function hash(): string
    {
        return $this->doHash(\spl_object_hash($this->get()));
    }

    /**
     * {@inheritdoc}
     */
    protected function doHash(string $string) : string
    {
        return \sha1(gettype($this->get()) . get_class($this->get()) . $string);
    }
}