integration::__construct() - Code Metrics - devtoolboxuk/soteria - Measure and Improve Code Quality continuously with Scrutinizer

integration::__construct() A
last analyzed 2019-10-08 11:29 UTC

↳ Parent: integration

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	5
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	3

<?php

namespace devtoolboxuk\soteria;

use PHPUnit\Framework\TestCase;

class integration extends TestCase
{
    private $security;

    function __construct($name = null, array $data = [], $dataName = '')

    {
        parent::__construct($name, $data, $dataName);
        $this->security = new SoteriaService();
    }

    function testEmailForUrl()

    {
        $sanitise = $this->security->sanitise();
        $email = '[email protected]';
        echo "\nUnclean String: " . $email;
        echo "\n";
        echo "\nSanitised Url: " . $sanitise->removeUrl($email);
        if ($sanitise->isSanitised()) {
            echo "\n1";
        }
    }

    function inte()

    {
        $xss = $this->security->xss();
        $sanitise = $this->security->sanitise();


        echo "\nXSS";
        $unCleanString = 'Visit my website http://www.doajob.org?redirect=https://www.google.com';


        echo "\nUnclean String: " . $unCleanString;
        $cleanString = $xss->clean($unCleanString);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
        echo "\nXSS Cleaned String: " . $cleanString;
        $cleanString = $xss->cleanUrl($unCleanString);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
        echo "\nXSS Cleaned Url: " . $cleanString;
        echo "\n";

        echo "\nSanitised Url: " . $sanitise->removeUrl($unCleanString);
        if ($sanitise->isSanitised()) {
            echo "\n1";
        }

        echo "\nString without a Url: " . $sanitise->removeUrl("Rob WIlson");
        if ($sanitise->isSanitised()) {
            echo "\n1";
        }


    }

}


1			<?php
2
3			namespace devtoolboxuk\soteria;
4
5			use PHPUnit\Framework\TestCase;
6
7			class integration extends TestCase
8			{
9			private $security;
10
11			function __construct($name = null, array $data = [], $dataName = '')
			0 ignored issues – show Best Practice introduced 2019-06-02 11:22 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
12			{
13			parent::__construct($name, $data, $dataName);
14			$this->security = new SoteriaService();
15			}
16
17			function testEmailForUrl()
			0 ignored issues – show Best Practice introduced 2019-06-23 12:10 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
18			{
19			$sanitise = $this->security->sanitise();
20			$email = '[email protected]';
21			echo "\nUnclean String: " . $email;
22			echo "\n";
23			echo "\nSanitised Url: " . $sanitise->removeUrl($email);
24			if ($sanitise->isSanitised()) {
25			echo "\n1";
26			}
27			}
28
29			function inte()
			0 ignored issues – show Best Practice introduced 2019-06-23 12:10 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
30			{
31			$xss = $this->security->xss();
32			$sanitise = $this->security->sanitise();
33
34
35			echo "\nXSS";
36			$unCleanString = 'Visit my website http://www.doajob.org?redirect=https://www.google.com';
37
38
39			echo "\nUnclean String: " . $unCleanString;
40			$cleanString = $xss->clean($unCleanString);
			0 ignored issues – show Bug introduced 2019-06-02 11:35 UTC by Report Bug Copy Issue Report The method `clean` does only exist in `devtoolboxuk\soteria\handlers\Xss`, but not in `devtoolboxuk\soteria\handlers\Sanitise`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
41			echo "\nXSS Cleaned String: " . $cleanString;
42			$cleanString = $xss->cleanUrl($unCleanString);
			0 ignored issues – show Bug introduced 2019-06-02 11:35 UTC by Report Bug Copy Issue Report The method `cleanUrl` does only exist in `devtoolboxuk\soteria\handlers\Xss`, but not in `devtoolboxuk\soteria\handlers\Sanitise`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
43			echo "\nXSS Cleaned Url: " . $cleanString;
44			echo "\n";
45
46			echo "\nSanitised Url: " . $sanitise->removeUrl($unCleanString);
47			if ($sanitise->isSanitised()) {
48			echo "\n1";
49			}
50
51			echo "\nString without a Url: " . $sanitise->removeUrl("Rob WIlson");
52			if ($sanitise->isSanitised()) {
53			echo "\n1";
54			}
55
56
57			}
58
59			}
60

devtoolboxuk / soteria

integration::__construct() A last analyzed 2019-10-08 11:29 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

integration::__construct() A
last analyzed 2019-10-08 11:29 UTC