XFrameOptions::getValue() - Code Metrics - Inspection of "Add support XFrame options headers" - Logi-CE/fmup - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#13)

unknown

created 2017-10-20 08:05 UTC

XFrameOptions::getValue() A

↳ Parent: XFrameOptions

Complexity

Conditions	3
Paths	2

Size

Total Lines	14
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	10
c	1
b	0
f	0
nc	2
nop	0
dl	0
loc	14
ccs	9
cts	9
cp	1
crap	3
rs	9.4285

<?php

namespace FMUP\Response\Header;

use FMUP\Response\Header;

class XFrameOptions extends Header
{

    const TYPE = 'X-Frame-Options';
    const OPTIONS_DENY = 'Deny';
    const OPTIONS_SAMEORIGIN = 'Sameorigin';
    const OPTIONS_ALLOW_FROM = 'ALLOW_FROM';
    const OPTIONS_ALLOW_FROM_URI_DEFAULT = '*';

    private $options = self::OPTIONS_DENY;
    private $uri = self::OPTIONS_DENY;


    /**
     * XFrameOptions constructor.
     * @param $options
     * @param array $uri
     */
    public function __construct($options, $uri = array())
    {
        $this->setOptions($options);
        $this->setUri($uri);
    }

    /**
     * Value returned in the header
     * @return string
     */
    public function getValue()
    {
        $return = '';
        $options = $this->getOptions();
        if ($options == self::OPTIONS_ALLOW_FROM) {
            $urls = $this->getUri();
            foreach ($urls as $url) {
                $return .= $options . ' ' . $url . ';';
            }
        } else {
            $return = $options;
        }
        return $return;
    }

    /**
     * @return string
     */
    public function getOptions()
    {
        return $this->options;
    }

    /**
     * @param string $options
     * @return $this
     */
    public function setOptions($options)
    {
        $this->options = $options;
        return $this;
    }

    /**
     * @return array
     */
    public function getUri()
    {
        if (!isset($this->uri) or empty($this->uri)) {
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
            $this->uri = array(self::OPTIONS_ALLOW_FROM_URI_DEFAULT);
        }
        return $this->uri;
    }

    /**
     * @param array $uri
     * @return $this
     */
    public function setUri($uri)
    {
        $this->uri = $uri;
        return $this;
    }

    /**
     * Type for the header. Can be used to determine header to send
     * @return string
     */
    public function getType()
    {
        return self::TYPE;
    }
}


Logi-CE / fmup

Pull Request — master (#13)

XFrameOptions::getValue() A

Complexity

Size

Duplication

Code Coverage

Importance

Logical Operators are used for Control-Flow

Duplication Side-by-Side

Filter issues like