App\Utility\Redirect

Complexity

Total Complexity

3

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	to()	0	22	3

<?php
/**
 * This file is part of product_management
 * User: Sinan TURGUT <mail@sinanturgut.com.tr>
 * Date: 24.06.2019
 * php version 7.2
 *
 * @category Assessment
 * @package  ProductManagement
 * @author   Sinan TURGUT <mail@sinanturgut.com.tr>
 * @license  See LICENSE file
 * @link     https://dev.sinanturgut.com.tr
 */

namespace App\Utility;


/**
 * Class Redirect
 * @package App\Utility
 */
class Redirect
{


    /**
     * @param string $location
     * @param null $errCode

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

     * @param null $replaceMessage

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

     */
    public static function to($location = "", $errCode=null, $replaceMessage=null)
    {
        if ($location) {
            if ($location === 404) {

The condition $location === 404 is always false.

                http_response_code(404);
                $errorStrings = json_decode(file_get_contents(VDIR.'/errorStrings.json'), true);
                $errorString =  $errorStrings[$errCode];
                $errorString = str_replace('@errMessage', $replaceMessage, $errorString);

                extract($errorString);

                ob_start();

                require VDIR . '/template/404.php';

                echo ob_get_clean();
            } else {
                header("Location: " . $location);
//                header("Location: /index.php?url=login");
            }
            exit;

Using exit here is not recommended.

        }
    }
}