Cortex\Foundation\Exceptions\GenericException

Complexity

Total Complexity

3

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	getRedirection()	0	4	1
A	getInputs()	0	4	1

<?php

declare(strict_types=1);

namespace Cortex\Foundation\Exceptions;

use Exception;

class GenericException extends Exception
{
    /**
     * The exception inputs.
     *
     * @var array
     */
    protected $inputs;

    /**
     * The exception redirection.
     *
     * @var string
     */
    protected $redirection;

    /**
     * Create a new authorization exception.
     *
     * @param string $message
     * @param array  $redirection

Should the type for parameter $redirection not be array|null? Also, consider making the array more specific, something like array<String>, or String[].

     */
    public function __construct($message = 'This action is unauthorized.', $redirection = null, array $inputs = null)
    {
        parent::__construct($message);

        $this->inputs = $inputs;

It seems like $inputs can be null. However, the property $inputs is declared as array. Maybe change the type of the property to array|null or add a type check?

        $this->redirection = $redirection;

It seems like $redirection can also be of type array. However, the property $redirection is declared as type string. Maybe add an additional type check?

    }

    /**
     * Gets the Exception redirection.
     *
     * @return string
     */
    final public function getRedirection()
    {
        return $this->redirection;
    }

    /**
     * Gets the Exception inputs.
     *
     * @return array
     */
    final public function getInputs()
    {
        return $this->inputs;
    }
}