drupol\valuewrapper\Resource\ResourceValue

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

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

<?php

declare(strict_types = 1);

namespace drupol\valuewrapper\Resource;

use drupol\valuewrapper\AbstractValue;
use drupol\valuewrapper\ValueInterface;

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}
     */
    abstract public function hash(): string;

    /**
     * {@inheritdoc}
     */
    public function equals(ValueInterface $item, $strict = true) : bool
    {
        return $this->hash() == $item->hash();
    }
}