ProtocolFileContent::hasEnabled() - Code Metrics - pedro151/orm-generator - Measure and Improve Code Quality continuously with Scrutinizer

ProtocolFileContent::hasEnabled() B
last analyzed 2017-03-30 19:56 UTC

↳ Parent: ProtocolFileContent

Complexity

Conditions	7
Paths	7

Size

Total Lines	17
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	17
rs	8.2222
c	0
b	0
f	0
cc	7
eloc	12
nc	7
nop	1

<?php
/**
 * Created by PhpStorm.
 * User: pedro
 * Date: 24/11/16
 * Time: 17:19
 */

namespace Classes\Update;


class ProtocolFileContent
{
    /**
     * @type string
     */
    protected $protocol;
    /**
     * @type Content
     */
    private static $objProtocol;

    private function __construct ()
    {
        $this->parseContentProtocol ();
    }

    /**
     * @return \Classes\Update\ProtocolFileContent
     */
    public static function getInstance ()
    {
        if ( empty( self::$objProtocol ) )
        {
            self::$objProtocol = new ProtocolFileContent();

        }

        return self::$objProtocol;

    }

    /**
     * @return bool
     */
    public function hasProtocol ()
    {
        return ! empty( $this->protocol );
    }

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

    /**
     *
     */
    private function parseContentProtocol ()
    {
        if ( $this->hasEnabled ( 'file_content' ) )
        {
            $this->protocol = 'file_content';
        } elseif ( $this->hasEnabled ( 'steam_content' ) )
        {
            $this->protocol = 'steam_content';
        } elseif ( $this->hasEnabled ( 'curl' ) )
        {
            $this->protocol = 'curl';
        }
    }

    /**
     * @return bool
     */
    private function hasEnabled ( $type )
    {
        switch ( $type )
        {
            case 'curl':
                return function_exists ( 'fopen' ) && function_exists ( 'curl_exec' );
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 'file_content':
                return function_exists ( 'file_get_contents' ) && function_exists ( 'stream_get_contents' );
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 'steam_content':
                return function_exists ( 'fopen' ) && function_exists ( 'stream_get_contents' );
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
        }

        throw new \Exception("\033[0;31mError: enable in php.ini 'file_get_contents' and  'stream_get_contents' or  'fopen' and 'curl'. \033[0m\n");
    }

}

1			<?php
2			/**
3			* Created by PhpStorm.
4			* User: pedro
5			* Date: 24/11/16
6			* Time: 17:19
7			*/
8
9			namespace Classes\Update;
10
11
12			class ProtocolFileContent
13			{
14			/**
15			* @type string
16			*/
17			protected $protocol;
18			/**
19			* @type Content
20			*/
21			private static $objProtocol;
22
23			private function __construct ()
24			{
25			$this->parseContentProtocol ();
26			}
27
28			/**
29			* @return \Classes\Update\ProtocolFileContent
30			*/
31			public static function getInstance ()
32			{
33			if ( empty( self::$objProtocol ) )
34			{
35			self::$objProtocol = new ProtocolFileContent();
			0 ignored issues – show Documentation Bug introduced 2016-12-02 17:15 UTC by Report Bug Copy Issue Report It seems like `new \Classes\Update\ProtocolFileContent()` of type `object<Classes\Update\ProtocolFileContent>` is incompatible with the declared type `object<Classes\Update\Content>` of property `$objProtocol`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
36			}
37
38			return self::$objProtocol;
			0 ignored issues – show Bug Compatibility introduced 2016-12-02 17:15 UTC by Report Bug Copy Issue Report The expression `self::$objProtocol;` of type `Classes\Update\ProtocolF...\|Classes\Update\Content` adds the type `Classes\Update\Content` to the return on line 38 which is incompatible with the return type documented by `Classes\Update\ProtocolFileContent::getInstance` of type `Classes\Update\ProtocolFileContent`. Loading history...
39			}
40
41			/**
42			* @return bool
43			*/
44			public function hasProtocol ()
45			{
46			return ! empty( $this->protocol );
47			}
48
49			/**
50			* @return string
51			*/
52			public function getProtocol ()
53			{
54			return $this->protocol;
55			}
56
57			/**
58			*
59			*/
60			private function parseContentProtocol ()
61			{
62			if ( $this->hasEnabled ( 'file_content' ) )
63			{
64			$this->protocol = 'file_content';
65			} elseif ( $this->hasEnabled ( 'steam_content' ) )
66			{
67			$this->protocol = 'steam_content';
68			} elseif ( $this->hasEnabled ( 'curl' ) )
69			{
70			$this->protocol = 'curl';
71			}
72			}
73
74			/**
75			* @return bool
76			*/
77			private function hasEnabled ( $type )
78			{
79			switch ( $type )
80			{
81			case 'curl':
82			return function_exists ( 'fopen' ) && function_exists ( 'curl_exec' );
83			break;
			0 ignored issues – show Unused Code introduced 2016-12-02 17:15 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
84			case 'file_content':
85			return function_exists ( 'file_get_contents' ) && function_exists ( 'stream_get_contents' );
86			break;
			0 ignored issues – show Unused Code introduced 2016-12-02 17:15 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
87			case 'steam_content':
88			return function_exists ( 'fopen' ) && function_exists ( 'stream_get_contents' );
89			break;
			0 ignored issues – show Unused Code introduced 2016-12-02 17:15 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
90			}
91
92			throw new \Exception("\033[0;31mError: enable in php.ini 'file_get_contents' and 'stream_get_contents' or 'fopen' and 'curl'. \033[0m\n");
93			}
94
95			}

pedro151 / orm-generator

GitHub Access Token became invalid

ProtocolFileContent::hasEnabled() B last analyzed 2017-03-30 19:56 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ProtocolFileContent::hasEnabled() B
last analyzed 2017-03-30 19:56 UTC