Nip\Controllers\Traits\RedirectTrait

Complexity

Total Complexity

4

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	flashRedirect()	0	5	2
A	redirect()	0	9	2

<?php

namespace Nip\Controllers\Traits;

/**
 * Trait NameWorksTrait
 * @package Nip\Controllers\Traits
 */
trait RedirectTrait
{

    /**
     * @param $message
     * @param $url
     * @param string $type
     * @param bool $name
     */
    protected function flashRedirect($message, $url, $type = 'success', $name = false)
    {
        $name = $name ? $name : $this->getName();

It seems like getName() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        app('flash.messages')->add($name, $type, $message);
        $this->redirect($url);
    }

    /**
     * @param $url
     * @param null $code

Are you sure the doc-type for parameter $code is correct as it would always require null to be passed?

     */
    protected function redirect($url, $code = null)
    {
        switch ($code) {
            case '301':
                header("HTTP/1.1 301 Moved Permanently");
                break;
        }
        header("Location: ".$url);
        exit();
    }
}