Cycle\Schema\Definition\Relation

Complexity

Total Complexity

14

Size/Duplication

Total Lines	135
Duplicated Lines	0 %

Importance

Changes

0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	isInversed()	0	3	1
A	getOptions()	0	3	1
A	getInverseLoad()	0	3	1
A	getType()	0	7	2
A	getInverseName()	0	3	1
A	setInverse()	0	7	1
A	getInverseType()	0	3	1
A	__construct()	0	3	1
A	setTarget()	0	5	1
A	setType()	0	5	1
A	getTarget()	0	7	2
A	__clone()	0	3	1

<?php
/**
 * Cycle ORM Schema Builder.
 *
 * @license   MIT
 * @author    Anton Titov (Wolfy-J)
 */
declare(strict_types=1);

namespace Cycle\Schema\Definition;

use Cycle\Schema\Definition\Map\OptionMap;
use Cycle\Schema\Exception\RelationException;

final class Relation
{
    /** @var OptionMap */
    private $options;

    /** @var string */
    private $type;

    /** @var string */
    private $target;

    /** @var string|null */
    private $inverse = null;

    /** @var string|null */
    private $inverseType = null;

    /** @var int|null */
    private $inverseLoad = null;

    /**
     * Relation constructor.
     */
    public function __construct()
    {
        $this->options = new OptionMap();
    }

    /**
     * @return OptionMap
     */
    public function getOptions(): OptionMap
    {
        return $this->options;
    }

    /**
     * @param string $type
     * @return Relation
     */
    public function setType(string $type): Relation
    {
        $this->type = $type;

        return $this;
    }

    /**
     * @return string
     */
    public function getType(): string
    {
        if ($this->type === null) {
            throw new RelationException("Relation type must be set");
        }

        return $this->type;
    }

    /**
     * @param string $target
     * @return Relation
     */
    public function setTarget(string $target): Relation
    {
        $this->target = $target;

        return $this;
    }

    /**
     * @return string
     */
    public function getTarget(): string
    {
        if ($this->target === null) {
            throw new RelationException("Relation target must be set");
        }

        return $this->target;
    }

    /**
     * @param string $into
     * @param string $as
     * @param int    $load
     * @return Relation
     */
    public function setInverse(string $into, string $as, ?int $load = null): Relation
    {
        $this->inverse = $into;
        $this->inverseType = $as;
        $this->inverseLoad = $load;

        return $this;
    }

    /**
     * @return bool
     */
    public function isInversed(): bool
    {
        return $this->inverse != null;

It seems like you are loosely comparing $this->inverse of type null|string against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

    }

    /**
     * @return string|null
     */
    public function getInverseName(): ?string
    {
        return $this->inverse;
    }

    /**
     * @return string|null
     */
    public function getInverseType(): ?string
    {
        return $this->inverseType;
    }

    /**
     * @return int|null
     */
    public function getInverseLoad(): ?int
    {
        return $this->inverseLoad;
    }

    /**
     * Cloning.
     */
    public function __clone()
    {
        $this->options = clone $this->options;
    }
}