RedirectionIO::findRedirect() - Code Metrics - Inspection of "Make last ruleId BC" - redirectionio/proxy-wordpress - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 4770e0...cf1e90 )

by Joel

created 2018-08-01 12:56 UTC

RedirectionIO::findRedirect() B

↳ Parent: RedirectionIO

Complexity

Conditions	11
Paths	37

Size

Total Lines	52
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	28
CRAP Score	11.968

Importance

Changes

Metric	Value
cc	11
eloc	31
nc	37
nop	0
dl	0
loc	52
ccs	28
cts	35
cp	0.8
crap	11.968
rs	7.3166
c	0
b	0
f	0

How to fix Long Method Complexity

<?php

namespace RedirectionIO\Client\Wordpress;

use RedirectionIO\Client\Sdk\Client;
use RedirectionIO\Client\Sdk\HttpMessage\Request;
use RedirectionIO\Client\Sdk\HttpMessage\Response;

/**
 * Main plugin file.
 *
 * This class is the core logic of the plugin.
 */
class RedirectionIO
{
    private $client;

    private $lastRuleId = null;

    public function __construct()
    {
        add_action('plugins_loaded', [$this, 'findRedirect']);
        add_action('plugins_loaded', [$this, 'findRedirect']);
        add_action('template_redirect', [$this, 'log']);
    }

    public function setUp()
    {
        add_option('redirectionio', [
            'connections' => [
                [
                    'name' => '',
                    'remote_socket' => '',
                ],
            ],
            'doNotRedirectAdmin' => true,
        ]);
    }

    public function findRedirect()
    {
        $options = get_option('redirectionio');
        $options = /** @scrutinizer ignore-call */ get_option('redirectionio');
        $connections = [];

        if (false === $options || !isset($options['connections'])) {
            return false;
        }

        foreach ($options['connections'] as $connection) {
            $connections[$connection['name']] = $connection['remote_socket'];
        }

        $this->client = new Client($connections);
        $scheme = 'http';

        if (isset($_SERVER['HTTP_X_FORWARDED_PROTO'])) {
            $scheme = $_SERVER['HTTP_X_FORWARDED_PROTO'];
        } elseif (!empty($_SERVER['HTTPS'])) {
            $scheme = 'https';
        }

        $request = new Request(
            $_SERVER['HTTP_HOST'],
            $_SERVER['REQUEST_URI'],
            $_SERVER['HTTP_USER_AGENT'],
            $_SERVER['HTTP_REFERER'],
            $scheme
        );

        if ($this->isAdminPage($request) && $options['doNotRedirectAdmin']) {
            return false;
        }

        $response = $this->client->findRedirect($request);

        if (null === $response) {
            return false;
        }

        if (method_exists($response, 'getRuleId')) {
            $this->lastRuleId = $response->getRuleId();
        }

        if ($response->getStatusCode() === 410) {
            define('DONOTCACHEPAGE', true); // WP Super Cache and W3 Total Cache recognise this
            status_header(410);
        } else {
            wp_redirect($response->getLocation(), $response->getStatusCode());
            wp_redirect($response->/** @scrutinizer ignore-call */ getLocation(), $response->getStatusCode());
        }

        $this->exitCode();
    }

    public function log()
    {
        $scheme = 'http';

        if (isset($_SERVER['HTTP_X_FORWARDED_PROTO'])) {
            $scheme = $_SERVER['HTTP_X_FORWARDED_PROTO'];
        } elseif (!empty($_SERVER['HTTPS'])) {
            $scheme = 'https';
        }

        $location = null;
        $headers = headers_list();

        foreach ($headers as $header) {
            $locationPos = stripos($header, 'Location: ');

            if ($locationPos !== false) {
                $location = substr($header, $locationPos);
            }
        }

        $request = new Request(
            $_SERVER['HTTP_HOST'],
            $_SERVER['REQUEST_URI'],
            $_SERVER['HTTP_USER_AGENT'],
            $_SERVER['HTTP_REFERER'],
            $scheme
        );

        $response = new Response(http_response_code(), $this->lastRuleId, $location);
        $response = /** @scrutinizer ignore-call */ new Response(http_response_code(), $this->lastRuleId, $location);

        $this->client->log($request, $response);
    }

    public function exitCode()
    {
        exit;

    }

    /**
     * Check if the requested page belongs to admin area.
     *
     * @param Request $request
     */
    private function isAdminPage(Request $request)
    {
        $adminRoot = str_replace(get_site_url(), '', get_admin_url());
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        $requestPath = substr($request->getPath(), 0, strlen($adminRoot));

        if ($adminRoot === $requestPath) {
            return true;
        }

        return false;
    }
}


redirectionio / proxy-wordpress

Push — master ( 4770e0...cf1e90 )

RedirectionIO::findRedirect() B

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like