Arcanedev\LaravelSeo\Redirectors\AbstractRedirector

Complexity

Total Complexity

10

Size/Duplication

Total Lines	119
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getOption()	0	4	1
A	determineRedirectStatusCode()	0	4	2
A	getRedirectFor()	0	20	2
A	determineRedirectUrl()	0	6	2
A	resolveRouterParameters()	0	8	2

<?php namespace Arcanedev\LaravelSeo\Redirectors;

use Arcanedev\LaravelSeo\Contracts\Redirector;
use Illuminate\Http\Request;
use Illuminate\Support\Arr;
use Symfony\Component\HttpFoundation\Response;
use Illuminate\Contracts\Routing\Registrar as Router;

/**
 * Class     AbstractRedirector
 *
 * @package  Arcanedev\LaravelSeo\Redirectors
 * @author   ARCANEDEV <arcanedev.maroc@gmail.com>
 */
abstract class AbstractRedirector implements Redirector
{
    /* -----------------------------------------------------------------
     |  Properties
     | -----------------------------------------------------------------
     */

    /** @var \Illuminate\Contracts\Routing\Registrar */
    protected $router;

    /** @var  \Illuminate\Http\Request */
    protected $request;

    /** @var  array */
    protected $options = [];

    /* -----------------------------------------------------------------
     |  Constructor
     | -----------------------------------------------------------------
     */

    /**
     * AbstractRedirector constructor.
     *
     * @param  \Illuminate\Contracts\Routing\Registrar  $router
     * @param  array                                    $options
     */
    public function __construct(Router $router, array $options)
    {
        $this->router  = $router;
        $this->options = $options;
    }

    /* -----------------------------------------------------------------
     |  Getters & Setters
     | -----------------------------------------------------------------
     */
    /**
     * Get an option.
     *
     * @param  string      $key
     * @param  mixed|null  $default
     *
     * @return mixed
     */
    public function getOption($key, $default = null)
    {
        return Arr::get($this->options, $key, $default);
    }

    /* -----------------------------------------------------------------
     |  Main Methods
     | -----------------------------------------------------------------
     */

    /**
     * Get the redirect url.
     *
     * @param  \Illuminate\Http\Request  $request
     *
     * @return \Illuminate\Http\Response|null
     */
    public function getRedirectFor(Request $request)

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

    {
        $this->request = $request;

        collect($this->getRedirectedUrls())->each(function ($redirects, $missingUrl) {
            $this->router->get($missingUrl, function () use ($redirects) {
                return redirect()->to(
                    $this->determineRedirectUrl($redirects),
                    $this->determineRedirectStatusCode($redirects)
                );
            });
        });

        try {
            return $this->router->dispatch($request);
        }
        catch (\Exception $e) {
            return null;
        }
    }

    /**
     * @param  array|string  $redirects
     *
     * @return string
     */
    protected function determineRedirectUrl($redirects)
    {
        return $this->resolveRouterParameters(
            is_array($redirects) ? reset($redirects) : $redirects
        );
    }

    /**
     * @param  mixed  $redirects
     *
     * @return int
     */
    protected function determineRedirectStatusCode($redirects)
    {
        return is_array($redirects) ? end($redirects) : Response::HTTP_MOVED_PERMANENTLY;
    }

    /**
     * @param  string  $redirectUrl
     *
     * @return string
     */
    protected function resolveRouterParameters($redirectUrl)
    {
        foreach ($this->router->getCurrentRoute()->parameters() as $key => $value) {
            $redirectUrl = str_replace("{{$key}}", $value, $redirectUrl);
        }

        return $redirectUrl;
    }
}