AdminPageFrameworkLoader_Demo::_shouldLoadDemo() - Code Metrics - Inspection of "Daily Inspection: - Advanced the version to help i..." - michaeluno/admin-page-framework - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — dev (aae49a)

unknown

created 2015-11-22 18:40 UTC

AdminPageFrameworkLoader_Demo::_shouldLoadDemo() A

↳ Parent: AdminPageFrameworkLoader_Demo

Complexity

Conditions	3
Paths	3

Size

Total Lines	15
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Metric	Value
dl	0
loc	15
rs	9.4286
ccs	0
cts	7
cp	0
cc	3
eloc	7
nc	3
nop	0
crap	12

<?php
/**
 * Loads the demo components.
 * 
 * @package      Admin Page Framework Loader
 * @copyright    Copyright (c) 2014, Michael Uno
 * @author       Michael Uno
 * @authorurl    http://michaeluno.jp
 * @since        3.5.0
 */

 /**
  * 
  * 
  * @action     do      admin_page_framework_loader_action_before_loading_demo
  * @action     do      admin_page_framework_loader_action_after_loading_demo
  */
class AdminPageFrameworkLoader_Demo {
namespace YourVendor;

class YourClass { }
    
    public function __construct() {
        
        if ( ! $this->_shouldLoadDemo() ) {
            return;
        }
        
        // Backward compatibility.
        define( 'APFDEMO_FILE', AdminPageFrameworkLoader_Registry::$sFilePath );
        define( 'APFDEMO_DIRNAME', AdminPageFrameworkLoader_Registry::$sDirPath );
        
        do_action( 'admin_page_framework_loader_action_before_loading_demo' );
        
        // Include example components.
        include( AdminPageFrameworkLoader_Registry::$sDirPath . '/example/admin-page-framework-demo-bootstrap.php' );
        
        do_action( 'admin_page_framework_loader_action_after_loading_demo' );
        
    }
        /**
         * @since       3.7.4
         * @return      boolean
         */
        private function _shouldLoadDemo() {


            if ( AdminPageFrameworkLoader_Utility::isSilentMode() ) {
                return false;
            }
        
            // Check if the demo is enabled.
            $_oOption = AdminPageFrameworkLoader_Option::getInstance();
            if ( ! $_oOption->get( 'enable_demo' ) ) {
                return false;
            }           
            
            return true;
            
        }
 
}
// Incorrect placement according to PSR2
class MyClass
{
    public function foo()
    {

    }
    // This blank line is not allowed.

}

// Correct
class MyClass
{
    public function foo()
    {

    } // No blank lines after this line.
}

1			<?php
2			/**
3			* Loads the demo components.
4			*
5			* @package Admin Page Framework Loader
6			* @copyright Copyright (c) 2014, Michael Uno
7			* @author Michael Uno
8			* @authorurl http://michaeluno.jp
9			* @since 3.5.0
10			*/
11
12			/**
13			*
14			*
15			* @action do admin_page_framework_loader_action_before_loading_demo
16			* @action do admin_page_framework_loader_action_after_loading_demo
17			*/
18			class AdminPageFrameworkLoader_Demo {
			2 ignored issues – show Coding Style Compatibility introduced 2015-11-21 18:44 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... Coding Style introduced 2015-11-21 18:44 UTC by Report Bug Copy Issue Report As per PSR2, the opening brace for this class should be on a new line. Loading history...
19
20			public function __construct() {
21
22			if ( ! $this->_shouldLoadDemo() ) {
23			return;
24			}
25
26			// Backward compatibility.
27			define( 'APFDEMO_FILE', AdminPageFrameworkLoader_Registry::$sFilePath );
28			define( 'APFDEMO_DIRNAME', AdminPageFrameworkLoader_Registry::$sDirPath );
29
30			do_action( 'admin_page_framework_loader_action_before_loading_demo' );
31
32			// Include example components.
33			include( AdminPageFrameworkLoader_Registry::$sDirPath . '/example/admin-page-framework-demo-bootstrap.php' );
34
35			do_action( 'admin_page_framework_loader_action_after_loading_demo' );
36
37			}
38			/**
39			* @since 3.7.4
40			* @return boolean
41			*/
42			private function _shouldLoadDemo() {
			1 ignored issue – show Coding Style introduced 2015-11-21 18:44 UTC by Report Bug Copy Issue Report Method name "_shouldLoadDemo" should not be prefixed with an underscore to indicate visibility Loading history...
43
44			if ( AdminPageFrameworkLoader_Utility::isSilentMode() ) {
45			return false;
46			}
47
48			// Check if the demo is enabled.
49			$_oOption = AdminPageFrameworkLoader_Option::getInstance();
50			if ( ! $_oOption->get( 'enable_demo' ) ) {
51			return false;
52			}
53
54			return true;
55
56			}
57
58			}
			1 ignored issue – show Coding Style introduced 2015-11-21 18:44 UTC by Report Bug Copy Issue Report According to PSR2, the closing brace of classes should be placed on the next line directly after the body. Below you find some examples: // Incorrect placement according to PSR2 class MyClass { public function foo() { } // This blank line is not allowed. } // Correct class MyClass { public function foo() { } // No blank lines after this line. } Loading history...

michaeluno / admin-page-framework

Branch — dev (aae49a)

AdminPageFrameworkLoader_Demo::_shouldLoadDemo() A

Complexity

Size

Duplication

Code Coverage

Duplication Side-by-Side

Filter issues like