JodelAccount::__construct() - Code Metrics - Inspection of "Code refactoring" - mmainstreet/jodel-web - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( fc7130...f4e497 )

by mains

created 2017-02-12 22:07 UTC

JodelAccount::__construct() A

↳ Parent: JodelAccount

Complexity

Conditions	3
Paths	4

Size

Total Lines	23
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	10
nc	4
nop	2
dl	0
loc	23
rs	9.0856
c	0
b	0
f	0

<?php

class JodelAccount
namespace YourVendor;

class YourClass { }
{
    public $accessToken;
    public $expirationDate;
    public $refreshToken;
    public $distinctId;
    public $deviceUid;

    //is the Account a Bot or Spider?
    public $isBot;
    
    // array of voted Jodels
    public $votes;

    //Location of the Account
    public $location;

    function __construct($deviceUid = NULL, $isBot = FALSE)

    {
        if($deviceUid == NULL)
        {
            //Create New Account
            $this->deviceUid = $deviceUid;
        }
        else
        {
            $this->deviceUid = $deviceUid;
        }

        $this->isBot        = $isBot;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->location     = $this::getLocation();
$a = "a";
$ab = "ab";
$abc = "abc";

        if(!$this::isTokenFresh())
        {
            $this::refreshToken();
        }
        $this->accessToken  = $this::getAccessToken();
$a = "a";
$ab = "ab";
$abc = "abc";


    }

    function isAccountVerified()

    {
        $accountCreator = new GetUserConfig();
        $accountCreator->setAccessToken($this->accessToken);
        $data = $accountCreator->execute();

        return $data['verified'];
    }

    function setLocation()
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        //Is Channel or City
        if(substr($_GET['city'], 0, 1) === '#')
        {
            return htmlspecialchars($_GET['city']) . " " . $this->location->cityName;
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        }                
        else
        {
            $url = 'https://maps.googleapis.com/maps/api/geocode/json?address=' . htmlspecialchars($_GET['city']) . '&key=AIzaSyCwhnja-or07012HqrhPW7prHEDuSvFT4w';
$a = "a";
$ab = "ab";
$abc = "abc";
            $result = Requests::post($url);
            if(json_decode($result->body, true)['status'] == 'ZERO_RESULTS' || json_decode($result->body, true)['status'] == 'INVALID_REQUEST')

            {
                return "0 results";
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
            }
            else
            {
                $name = json_decode($result->body, true)['results']['0']['address_components']['0']['long_name'];
                $lat = json_decode($result->body, true)['results']['0']['geometry']['location']['lat'];
$a = "a";
$ab = "ab";
$abc = "abc";
                $lng = json_decode($result->body, true)['results']['0']['geometry']['location']['lng'];
$a = "a";
$ab = "ab";
$abc = "abc";

                $location = new Location();
                $location->setLat($lat);
                $location->setLng($lng);
                $location->setCityName($name);
                $accountCreator = new UpdateLocation();
                $accountCreator->setLocation($location);
                $accountCreator->setAccessToken($this->accessToken);
                $data = $accountCreator->execute();

                //safe location to db
                $db = new DatabaseConnect();


                if($data == 'Success')
                {
                    $result = $db->query("UPDATE accounts 
                            SET name='" . $name . "',
                                lat='" . $lat . "',
                                lng='" . $lng . "'
                            WHERE access_token='" . $this->accessToken . "'");

                    if($result === false)
                    {
                            echo "Updating location failed: (" . $db->errno . ") " . $db->error;
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
                    }
                    else
                    {
                        error_log('User with JodelDeviceId:' . $this->deviceUid .  ' [' . $_SERVER['REMOTE_ADDR'] . '][' . $_SERVER ['HTTP_USER_AGENT'] . '] changed to Location: ' . $name);

                    }
                }

                return $name;
            }
        }
    }

    function getLocation()

    {
        $db = new DatabaseConnect();
$a = "a";
$ab = "ab";
$abc = "abc";
        $result = $db->query("SELECT * FROM accounts WHERE device_uid='" . $this->deviceUid  . "'");
        
        $location = new Location();
        
        if ($result->num_rows > 0)
        {
            // output data of each row
            while($row = $result->fetch_assoc())
            {
                $location->setLat($row['lat']);
                $location->setLng($row['lng']);
                $location->setCityName($row['name']);
            }
        }
        else
        {
            echo "Error: 0 results";
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
            error_log("Error no Location found - getLocation");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        }

        return $location;
    }

    function verifyCaptcha()
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        $solution = $_GET['solution'];
        $solution = array_map('intval', explode('-', $solution));

        $accountCreator = new PostCaptcha();
        $accountCreator->setAccessToken($this->accessToken);
        $accountCreator->captchaKey = $_GET['key'];
$a = "a";
$ab = "ab";
$abc = "abc";
        $accountCreator->captchaSolution = $solution;
        $verified = $accountCreator->execute();
$a = "a";
$ab = "ab";
$abc = "abc";

        return $verified['verified'];
    }

    //ToDo Spider Check

    function votePostId()
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        if(!$this->isAccountVerified())
        {
            $view = new View();
            $view->showCaptcha($this->accessToken);
        }

        if(!$this->deviceUidHasVotedThisPostId($_GET['postID']))
        {
            if($_GET['vote'] == "up")
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
            {
                $accountCreator = new Upvote();
            }
            else if($_GET['vote'] == "down")
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
            {
                $accountCreator = new Downvote();
            }
            $accountCreator->setAccessToken($this->accessToken);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
            $accountCreator->postId = htmlspecialchars($_GET['postID']);
            $data = $accountCreator->execute();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}


            $this->addVoteWithPostIdAndTypeToDeviceUid($_GET['postID'], $_GET['vote']);
        }

        
        if(isset($_GET['getPostDetails']) && isset($_GET['getPostDetails']))
        {
            header('Location: index.php?getPostDetails=true&postID=' . htmlspecialchars($_GET['postID_parent']) . '#postId-' . htmlspecialchars($_GET['postID']));

        }
        else
        {
            header("Location: index.php#postId-" . htmlspecialchars($_GET['postID']));
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        }   
        die();

    }

    //ToDo Spider Check

    function sendJodel()
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        if(!$this->isAccountVerified())
        {
            showCaptcha($this->accessToken);
        }

        $accountCreator = new SendJodel();

        if(isset($_POST['ancestor']))
        {
            $ancestor = $_POST['ancestor'];
$a = "a";
$ab = "ab";
$abc = "abc";
            $accountCreator->ancestor = $ancestor;
        }
        if(isset($_POST['color']))
        {
            $color = $_POST['color'];
            switch ($color) {
                case '8ABDB0':
                    $color = '8ABDB0';
                    break;
                case '9EC41C':
                    $color = '9EC41C';
                    break;
                case '06A3CB':
                    $color = '06A3CB';
                    break;
                case 'FFBA00':
                    $color = 'FFBA00';
                    break;
                case 'DD5F5F':
                    $color = 'DD5F5F';
                    break;
                case 'FF9908':
                    $color = 'FF9908';
                    break;
                default:
                    $color = '8ABDB0';
                    break;
            }
            $accountCreator->color = $color;
        }

        $accountCreatorLocation = new UpdateLocation();

        $accountCreatorLocation->setLocation($this->location);
        $accountCreatorLocation->setAccessToken($this->accessToken);
        $data = $accountCreatorLocation->execute();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        
        $accountCreator->location = $this->location;
        
        $accountCreator->setAccessToken($this->accessToken);
        $data = $accountCreator->execute();

        if(isset($_POST['ancestor']))
        {
            $actual_link = 'http://' . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];
            header('Location: ' . $actual_link . '#postId-' . htmlspecialchars($data['post_id']));
            exit;

        }
        else
        {
            header('Location: ./#');
            exit;

        }
    }

    function isTokenFresh()

    {
        $db = new DatabaseConnect();  
$a = "a";
$ab = "ab";
$abc = "abc";
        $result = $db->query("SELECT * FROM accounts WHERE device_uid='" . $this->deviceUid . "'");

        $access_token;
function someFunction() {
    $x = 5;
    echo $x;
}

        if ($result->num_rows > 0)
        {
                // output data of each row
                while($row = $result->fetch_assoc()) {
                        //$access_token = $row["access_token"];

                        $expiration_date = $row["expiration_date"];
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
                        $deviceUid = $row["device_uid"];
$a = "a";
$ab = "ab";
$abc = "abc";
                        $access_token = $row["access_token"];
$a = "a";
$ab = "ab";
$abc = "abc";
                }
        }
        else
        {
                echo '0 results';
        }

        if($expiration_date <= time()) {
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
            $accountCreator = new CreateUser();
            $accountCreator->setAccessToken($access_token);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
            $accountCreator->setDeviceUid($deviceUid);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
            $accountCreator->setLocation($this->location);
            $data = $accountCreator->execute();

            $access_token = (string)$data[0]['access_token'];
$a = "a";
$ab = "ab";
$abc = "abc";
            $expiration_date = $data[0]['expiration_date'];
            $device_uid = (string)$data[1];
$a = "a";
$ab = "ab";
$abc = "abc";
            
            $db = new DatabaseConnect();  
$a = "a";
$ab = "ab";
$abc = "abc";
            $result = $db->query("UPDATE accounts 
                                    SET access_token='" . $access_token . "',
                                        expiration_date='" . $expiration_date . "'
                                    WHERE device_uid='" . $device_uid . "'");

            if($result === false){
                    echo "Adding account failed: (" . $db->errno . ") " . $db->error;
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
            }   
        }
        
        return $access_token;
    }

    function refreshToken()

    {

    }



    function getAccessToken()

    {
        $db = new DatabaseConnect();
$a = "a";
$ab = "ab";
$abc = "abc";
        $result = $db->query("SELECT * FROM accounts WHERE device_uid='" . $this->deviceUid  . "'");
        
        $accessToken;
function someFunction() {
    $x = 5;
    echo $x;
}
        
        if ($result->num_rows > 0)
        {
            // output data of each row
            while($row = $result->fetch_assoc())
            {
                $accessToken = $row['access_token'];
            }
        }
        else
        {
            echo "Error: 0 results";
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        }

        return $accessToken;
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
    }


    function getKarma()

    {
        $accountCreator = new GetKarma();
        $accountCreator->setAccessToken($this->accessToken);
        $data = $accountCreator->execute();
        
        return $data["karma"];
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
    }

    function deviceUidHasVotedThisPostId($postId)

    {
        $db = new DatabaseConnect();


        $postId = $db->real_escape_string($postId);


        $result = $db->query("SELECT id
                              FROM votes
                              WHERE (postId = '" . $postId . "' AND device_uid = '" . $this->deviceUid . "')");
        
        if($result === false)

        {
            $error = db_error();
            echo $error;
            echo "Adding Vote failed: (" . $result->errno . ") " . $result->error;
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        }

        if($result->num_rows == 0)

        {
            return FALSE;
        }
        else
        {
            return TRUE;
        }
    }

    function addVoteWithPostIdAndTypeToDeviceUid($postId, $voteType)

    {
        $db = new DatabaseConnect();  


        $postId = $db->real_escape_string($postId);
$a = "a";
$ab = "ab";
$abc = "abc";
        $voteType = $db->real_escape_string($voteType);

        
        if($this->deviceUidHasVotedThisPostId($postId))
        {
            return "Already voted";
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        }

        $result = $db->query("INSERT INTO votes (device_uid, postId, type)
                        VALUES ('" . $this->deviceUid . "','" . $postId . "','" . $voteType . "')");
        
        if($result === false){

                $error = db_error();
                echo $error;
                echo "Adding Vote failed: (" . $result->errno . ") " . $result->error;
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        }       
    }

    function registerAccount($location) {

        $accountCreator = new CreateUser();
        $accountCreator->setLocation($location);
        $data = $accountCreator->execute();
        
        $access_token = (string)$data[0]['access_token'];
$a = "a";
$ab = "ab";
$abc = "abc";
        $refresh_token = (string)$data[0]['refresh_token'];
$a = "a";
$ab = "ab";
$abc = "abc";
        $token_type = (string)$data[0]['token_type'];
$a = "a";
$ab = "ab";
$abc = "abc";
        $expires_in = $data[0]['expires_in'];
$a = "a";
$ab = "ab";
$abc = "abc";
        $expiration_date = $data[0]['expiration_date'];
        $distinct_id = (string)$data[0]['distinct_id'];
$a = "a";
$ab = "ab";
$abc = "abc";
        $device_uid = (string)$data[1];
$a = "a";
$ab = "ab";
$abc = "abc";

        $name = $location->cityName;
        $lat = $location->lat;
$a = "a";
$ab = "ab";
$abc = "abc";
        $lng = $location->lng;
$a = "a";
$ab = "ab";
$abc = "abc";
        
        $db = new DatabaseConnect();  
$a = "a";
$ab = "ab";
$abc = "abc";
        $result = $db->query("INSERT INTO accounts (access_token, refresh_token, token_type,
                        expires_in, expiration_date, distinct_id, device_uid, name, lat, lng)
                        VALUES ('" . $access_token . "','" . $refresh_token . "','" . $token_type .
                        "','" .  $expires_in . "','" . $expiration_date . "','" . $distinct_id .
                        "','" . $device_uid . "','" . $name . "','" . $lat . "','" . $lng . "') ");

        $success = TRUE;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        if($result === false){

                $error = db_error();
                echo $error;
                echo "Adding account failed: (" . $result->errno . ") " . $result->error;
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
                $success = FALSE;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        }   
        
        return $device_uid;
    }

    function createAccount()

    {
        $config = parse_ini_file('config/config.ini.php');
$a = "a";
$ab = "ab";
$abc = "abc";
        $location = new Location();
        $location->setLat($config['default_lat']);
        $location->setLng($config['default_lng']);
        $location->setCityName($config['default_location']);

        $deviceUid = registerAccount($location);

        return $deviceUid;
    }
}

Push — master ( fc7130...f4e497 )

JodelAccount::__construct() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1			<?php
2
3			class JodelAccount
			0 ignored issues – show Coding Style Compatibility introduced 2017-02-12 22:10 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
4			{
5			public $accessToken;
6			public $expirationDate;
7			public $refreshToken;
8			public $distinctId;
9			public $deviceUid;
10
11			//is the Account a Bot or Spider?
12			public $isBot;
13
14			// array of voted Jodels
15			public $votes;
16
17			//Location of the Account
18			public $location;
19
20			function __construct($deviceUid = NULL, $isBot = FALSE)
			0 ignored issues – show Best Practice introduced 2017-02-12 22: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...
21			{
22			if($deviceUid == NULL)
23			{
24			//Create New Account
25			$this->deviceUid = $deviceUid;
26			}
27			else
28			{
29			$this->deviceUid = $deviceUid;
30			}
31
32			$this->isBot = $isBot;
			0 ignored issues – show Coding Style introduced 2017-02-12 22:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 4 spaces but found 8 spaces This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
33			$this->location = $this::getLocation();
			0 ignored issues – show Coding Style introduced 2017-02-12 22:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 1 space but found 5 spaces This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
34
35			if(!$this::isTokenFresh())
36			{

mmainstreet / jodel-web

Push — master ( fc7130...f4e497 )

JodelAccount::__construct() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like