Keys - Code Metrics - infinitypaul/laravel-multistep-forms - Measure and Improve Code Quality continuously with Scrutinizer

Keys A
last analyzed 2020-07-27 14:22 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

Metric	Value
wmc	12
lcom	0
cbo	0
dl	0
loc	81
rs	10
c	0
b	0
f	0

3 Methods

Rating	Name	Size	Complexity
B	getPool()	28	7
A	secureCrypt()	19	3
A	getHashedToken()	10	2

<?php

namespace Infinitypaul\MultiStep\Controller;

class Keys
{
    /**
     * Get the pool to use based on the type of prefix hash.
     * @param  string $type
     * @return string
     */
    private static function getPool($type = 'alnum')
    {
        switch ($type) {
            case 'alnum':
                $pool = '0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
                break;
            case 'alpha':
                $pool = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
                break;
            case 'hexdec':
                $pool = '0123456789abcdef';
                break;
            case 'numeric':
                $pool = '0123456789';
                break;
            case 'nozero':
                $pool = '123456789';
                break;
            case 'distinct':
                $pool = '2345679ACDEFHJKLMNPRSTUVWXYZ';
                break;
            default:
                $pool = (string) $type;
                break;
        }

        return $pool;
    }

    /**
     * Generate a random secure crypt figure.
     * @param  int $min
     * @param  int $max
     * @return int
     */
    private static function secureCrypt($min, $max)
    {
        $range = $max - $min;

        if ($range < 0) {
            return $min; // not so random...
        }

        $log = log($range, 2);
        $bytes = (int) ($log / 8) + 1; // length in bytes
        $bits = (int) $log + 1; // length in bits
        $filter = (int) (1 << $bits) - 1; // set all lower bits to 1
        do {
            $rnd = hexdec(bin2hex(openssl_random_pseudo_bytes($bytes)));
            $rnd = $rnd & $filter; // discard irrelevant bits
        } while ($rnd >= $range);

        return $min + $rnd;
    }

    /**
     * Finally, generate a hashed token.
     *
     * @param string $key
     * @param int $length
     *
     * @return string
     */
    public static function getHashedToken($key = '', $length = 25)
    {
        $token = '';
        $max = strlen(static::getPool());
class YourClass
{
    private static function getTemperature() {
        return "3422 °C";
}

public static function getSomeVariable()
{
    return static::getTemperature();
}
        for ($i = 0; $i < $length; $i++) {
            $token .= static::getPool()[static::secureCrypt(0, $max)];
class YourClass
{
    private static function getTemperature() {
        return "3422 °C";
}

public static function getSomeVariable()
{
    return static::getTemperature();
}
        }

        return $key.$token;
    }
}


1			<?php
2
3			namespace Infinitypaul\MultiStep\Controller;
4
5			class Keys
6			{
7			/**
8			* Get the pool to use based on the type of prefix hash.
9			* @param string $type
10			* @return string
11			*/
12			private static function getPool($type = 'alnum')
13			{
14			switch ($type) {
15			case 'alnum':
16			$pool = '0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
17			break;
18			case 'alpha':
19			$pool = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
20			break;
21			case 'hexdec':
22			$pool = '0123456789abcdef';
23			break;
24			case 'numeric':
25			$pool = '0123456789';
26			break;
27			case 'nozero':
28			$pool = '123456789';
29			break;
30			case 'distinct':
31			$pool = '2345679ACDEFHJKLMNPRSTUVWXYZ';
32			break;
33			default:
34			$pool = (string) $type;
35			break;
36			}
37
38			return $pool;
39			}
40
41			/**
42			* Generate a random secure crypt figure.
43			* @param int $min
44			* @param int $max
45			* @return int
46			*/
47			private static function secureCrypt($min, $max)
48			{
49			$range = $max - $min;
50
51			if ($range < 0) {
52			return $min; // not so random...
53			}
54
55			$log = log($range, 2);
56			$bytes = (int) ($log / 8) + 1; // length in bytes
57			$bits = (int) $log + 1; // length in bits
58			$filter = (int) (1 << $bits) - 1; // set all lower bits to 1
59			do {
60			$rnd = hexdec(bin2hex(openssl_random_pseudo_bytes($bytes)));
61			$rnd = $rnd & $filter; // discard irrelevant bits
62			} while ($rnd >= $range);
63
64			return $min + $rnd;
65			}
66
67			/**
68			* Finally, generate a hashed token.
69			*
70			* @param string $key
71			* @param int $length
72			*
73			* @return string
74			*/
75			public static function getHashedToken($key = '', $length = 25)
76			{
77			$token = '';
78			$max = strlen(static::getPool());
			0 ignored issues – show Bug introduced 2020-06-20 19:51 UTC by Report Bug Copy Issue Report Since `getPool()` is declared private, calling it with `static` will lead to errors in possible sub-classes. You can either use `self`, or increase the visibility of `getPool()` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return static::getTemperature(); } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { private static function getTemperature() { return "-182 °C"; } } print YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return self::getTemperature(); } } Loading history...
79			for ($i = 0; $i < $length; $i++) {
80			$token .= static::getPool()[static::secureCrypt(0, $max)];
			0 ignored issues – show Bug introduced 2020-06-20 19:51 UTC by Report Bug Copy Issue Report Since `getPool()` is declared private, calling it with `static` will lead to errors in possible sub-classes. You can either use `self`, or increase the visibility of `getPool()` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return static::getTemperature(); } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { private static function getTemperature() { return "-182 °C"; } } print YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return self::getTemperature(); } } Loading history... Bug introduced 2020-06-20 19:51 UTC by Report Bug Copy Issue Report Since `secureCrypt()` is declared private, calling it with `static` will lead to errors in possible sub-classes. You can either use `self`, or increase the visibility of `secureCrypt()` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return static::getTemperature(); } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { private static function getTemperature() { return "-182 °C"; } } print YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return self::getTemperature(); } } Loading history...
81			}
82
83			return $key.$token;
84			}
85			}
86

infinitypaul / laravel-multistep-forms

Keys A last analyzed 2020-07-27 14:22 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

3 Methods

Duplication Side-by-Side

Filter issues like

Keys A
last analyzed 2020-07-27 14:22 UTC