Cortex\Foundation\Overrides\Illuminate\Routing\UrlGenerator::addQueryString()

Complexity

Conditions	3
Paths	4

Size

Total Lines	14
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	UrlGenerator::routeUrl()	0	8	2

<?php

/*
 * NOTICE OF LICENSE
 *
 * Part of the Cortex Foundation Module.
 *
 * This source file is subject to The MIT License (MIT)
 * that is bundled with this package in the LICENSE file.
 *
 * Package: Cortex Foundation Module
 * License: The MIT License (MIT)
 * Link:    https://rinvex.com
 */

declare(strict_types=1);

namespace Cortex\Foundation\Overrides\Illuminate\Routing;

use Illuminate\Routing\UrlGenerator as BaseUrlGenerator;
use Mcamara\LaravelLocalization\Facades\LaravelLocalization;

class UrlGenerator extends BaseUrlGenerator
{
    /**
     * The backend URI.
     *
     * @var string
     */
    protected $backendUri = 'backend';

    /**
     * Set the backend URI.
     *
     * @param string $backendUri
     *
     * @return void
     */
    public function setBackendUri($backendUri)
    {
        $this->backendUri = $backendUri;
    }

    /**
     * Generate an absolute URL to the given admin path.
     *
     * @param string $path
     * @param array  $parameters
     * @param bool   $secure

Should the type for parameter $secure not be boolean|null?

     *
     * @return string
     */
    public function toBackend($path, array $parameters = [], $secure = null)
    {
        return $this->to("{$this->backendUri}/{$path}", $parameters, $secure);
    }

    /**
     * Generate a absolute URL to the given path.
     *
     * @param string    $path
     * @param mixed     $extra
     * @param bool|null $secure
     *
     * @return string
     */
    public function to($path, $extra = [], $secure = null)
    {
        return config('rinvex.cortex.route.locale_prefix')
            ? LaravelLocalization::localizeURL(parent::to($path, $extra, $secure))
            : parent::to($path, $extra, $secure);
    }

    /**
     * {@inheritdoc}
     */
    protected function routeUrl()
    {
        if (! $this->routeGenerator) {
            $this->routeGenerator = new RouteUrlGenerator($this, $this->request);
        }

        return $this->routeGenerator;
    }

    /**
     * {@inheritdoc}
     */
    public function previous($fallback = false)
    {
        return ($previousUrl = $this->request->input('previous_url')) ? $this->to($previousUrl) : parent::previous($fallback);

It seems like $previousUrl defined by $this->request->input('previous_url') on line 91 can also be of type array; however, Cortex\Foundation\Overrides\Illuminate\Routing\UrlGenerator::to() does only seem to accept string, maybe add an additional type check?

This line exceeds maximum limit of 120 characters; contains 126 characters

    }

    /**
     * {@inheritdoc}
     */
    protected function toRoute($route, $parameters, $absolute)
    {
        // Bind {locale} route parameter
        if (config('rinvex.cortex.route.locale_prefix') && ! isset($parameters['locale'])) {
            $parameters['locale'] = LaravelLocalization::getCurrentLocale();
        }

        return $this->routeUrl()->to(
            $route, $this->formatParameters($parameters), $absolute
        );
    }
}