ByTIC\MediaLibrary\FileAdder\Traits\HasSubjectTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getSubject()	0	3	1
A	hasSubject()	0	13	4
A	setSubject()	0	5	1

<?php

namespace ByTIC\MediaLibrary\FileAdder\Traits;

use ByTIC\MediaLibrary\Exceptions\RuntimeException;
use ByTIC\MediaLibrary\FileAdder\FileAdder;
use ByTIC\MediaLibrary\HasMedia\HasMediaTrait;
use ByTIC\MediaLibrary\HasMedia\Interfaces\HasMedia;
use Nip\Utility\Oop;

/**
 * Trait HasSubjectTrait.
 */
trait HasSubjectTrait
{
    /** @var HasMediaTrait|HasMedia subject */
    protected $subject;

    /**
     * @return HasMediaTrait|HasMedia
     */
    public function getSubject(): HasMedia
    {
        return $this->subject;

The expression return $this->subject could return the type ByTIC\MediaLibrary\HasMedia\HasMediaTrait which is incompatible with the type-hinted return ByTIC\MediaLibrary\HasMedia\Interfaces\HasMedia. Consider adding an additional type-check to rule them out.

    }

    /**
     * @return bool
     */
    public function hasSubject(): bool
    {
        if (!is_object($this->subject)) {
            return false;
        }
        $hasInterface = $this->subject instanceof HasMedia;
        if ($hasInterface) {
            return true;
        }
        if (Oop::classUsesTrait($this->subject, HasMediaTrait::class)) {
            throw new RuntimeException("Class " . get_class($this->subject) . " does not have the HasMedia interface");
        }
        return false;
    }

    /**
     * @param HasMediaTrait $subject
     *
     * @return FileAdder|self
     */
    public function setSubject($subject)
    {
        $this->subject = $subject;

        return $this;
    }
}