Cycle\Annotated\Annotation\Inheritance\JoinedTable

Complexity

Total Complexity

4

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	1
A	getFkAction()	0	3	1
A	getOuterKey()	0	3	1
A	isCreateFk()	0	3	1

<?php

declare(strict_types=1);

namespace Cycle\Annotated\Annotation\Inheritance;

use Cycle\Annotated\Annotation\Inheritance;
use Doctrine\Common\Annotations\Annotation\NamedArgumentConstructor;
use JetBrains\PhpStorm\ExpectedValues;

/**
 * @Annotation
 * @NamedArgumentConstructor
 * @Target("CLASS")
 */
#[\Attribute(\Attribute::TARGET_CLASS), NamedArgumentConstructor]
class JoinedTable extends Inheritance
{
    /**
     * @param non-empty-string|null $outerKey Outer (parent) key name.

The doc comment non-empty-string|null at position 0 could not be parsed: Unknown type name 'non-empty-string' at position 0 in non-empty-string|null.

     * @param bool $fkCreate Set to true to automatically create FK on outerKey.
     * @param non-empty-string|null $fkAction FK onDelete and onUpdate action.
     */
    public function __construct(
        private ?string $outerKey = null,
        private bool $fkCreate = true,
        /**
         * @Enum({"NO ACTION", "CASCADE", "SET NULL"})
         */
        #[ExpectedValues(values: ['NO ACTION', 'CASCADE', 'SET NULL'])]
        private ?string $fkAction = 'CASCADE',
    ) {
        parent::__construct('joined');
    }

    public function getOuterKey(): ?string
    {
        return $this->outerKey;
    }

    public function isCreateFk(): bool
    {
        return $this->fkCreate;
    }

    public function getFkAction(): string
    {
        return $this->fkAction;

The expression return $this->fkAction could return the type null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }
}