ChinLeung\LaravelMultilingualRoutes\MultilingualRoutePendingRegistration

Complexity

Total Complexity

9

Size/Duplication

Total Lines	161
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	register()	0	11	1
A	except()	0	9	1
A	name()	0	6	1
A	method()	0	6	1
A	names()	0	6	1
A	only()	0	9	1
A	__destruct()	0	6	2

<?php

namespace ChinLeung\LaravelMultilingualRoutes;

use Illuminate\Routing\RouteCollection;
use Illuminate\Support\Arr;

class MultilingualRoutePendingRegistration
{
    /**
     * The handle of the routes.
     *
     * @var mixed
     */
    protected $handle;

    /**
     * The translation key of the routes.
     *
     * @var string
     */
    protected $key;

    /**
     * The list of locales for the route.
     *
     * @var array
     */
    protected $locales = [];

    /**
     * The options of the routes.
     *
     * @var array
     */
    protected $options = [];

    /**
     * The resource's registration status.
     *
     * @var bool
     */
    protected $registered = false;

    /**
     * The resource registrar.
     *
     * @var \Illuminate\Routing\ResourceRegistrar
     */
    protected $registrar;

    /**
     * Constructor of the class.
     *
     * @param  \ChinLeung\LaravelMultilingualRoutes\MultilingualRegistrar  $registrar
     * @param  string  $key
     * @param  mixed  $handle
     * @param  array  $locales
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(MultilingualRegistrar $registrar, string $key, $handle, array $locales = [])
    {
        $this->key = $key;
        $this->registrar = $registrar;

It seems like $registrar of type object<ChinLeung\LaravelMultilingualRoutes\MultilingualRegistrar> is incompatible with the declared type object<Illuminate\Routing\ResourceRegistrar> of property $registrar.

        $this->handle = $handle;
        $this->locales = $locales;
    }

    /**
     * Register the resource route.
     *
     * @return \Illuminate\Routing\RouteCollection
     */
    public function register() : RouteCollection
    {
        $this->registered = true;

        return $this->registrar->register(
            $this->key,
            $this->handle,
            $this->options['locales'] ?? $this->locales,
            $this->options

The call to ResourceRegistrar::register() has too many arguments starting with $this->options.

        );
    }

    /**
     * Add one or many locale to the exception.
     *
     * @param  string|array  $locales
     * @return self
     */
    public function except($locales) : self
    {
        $this->options['locales'] = array_diff(
            $this->locales,
            Arr::wrap($locales)
        );

        return $this;
    }

    /**
     * Set the name of the routes.
     *
     * @param  string  $name
     * @return self
     */
    public function name(string $name) : self
    {
        $this->options['name'] = $name;

        return $this;
    }

    /**
     * Set the method of the routes.
     *
     * @param  string  $method
     * @return self
     */
    public function method(string $method) : self
    {
        $this->options['method'] = $method;

        return $this;
    }

    /**
     * Set the name of each locale for the routes.
     *
     * @param  array  $names
     * @return self
     */
    public function names(array $names) : self
    {
        $this->options['names'] = $names;

        return $this;
    }

    /**
     * Set the route for a list of locales only.
     *
     * @param  string|array  $locales
     * @return self
     */
    public function only($locales) : self
    {
        $this->options['locales'] = array_intersect(
            $this->locales,
            Arr::wrap($locales)
        );

        return $this;
    }

    /**
     * Handle the object's destruction.
     *
     * @return void
     */
    public function __destruct()
    {
        if (! $this->registered) {
            $this->register();
        }
    }
}