core - Code Metrics - Inspection of "extraced core methods to core class" - DavidCramer/uix - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 704bfc...0cb6ff )

by David

created 2018-06-21 09:37 UTC

core A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	149
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

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

7 Methods

Rating	Name	Size	Complexity
A	actions()	12	2
A	set_assets()	8	3
A	enqueue_core()	10	1
A	enqueue_actions()	15	1
A	core_assets()	11	1
A	set_active_styles()	2	1
A	base_color()	18	3

<?php

/**
 * Interface for WordPress core
 *
 * @package   uix
 * @author    David Cramer
 * @license   GPL-2.0+
 * @link
 * @copyright 2016 David Cramer
 */

namespace uix\ui\core;

class core {

	/**
	 * List of core object scripts ( common scripts )
	 *
	 * @since  1.0.0
	 * @access protected
	 * @var      array
	 */
	protected $scripts = [];

	/**
	 * List of core object styles ( common styles )
	 *
	 * @since  1.0.0
	 * @access protected
	 * @var      array
	 */
	protected $styles = [];


	/**
	 * setup actions and hooks - override to add specific hooks. use
	 * parent::actions() to keep admin head
	 *
	 * @since  1.0.0
	 * @access protected
	 */
	protected function actions() {

		// init uix after loaded.
		add_action( 'init', [ $this, 'init' ] );
		// set location.
		$location = 'wp_print_styles';
		if ( is_admin() ) {
			$location = 'admin_enqueue_scripts';
		}
		// init UIX headers.
		add_action( $location, [ $this, 'enqueue_core' ] );
	}


	/**
	 * Define core UIX styles - override to register core ( common styles for
	 * uix type )
	 *
	 * @since  1.0.0
	 * @access public
	 */
	public function set_assets() {
		if ( ! empty( $this->struct['style'] ) ) {
			$this->assets['style'] = array_merge( $this->assets['style'], $this->struct['style'] );
class MyClass { }

$x = new MyClass();
$x->foo = true;
		}
		if ( ! empty( $this->struct['script'] ) ) {
			$this->assets['script'] = array_merge( $this->assets['script'], $this->struct['script'] );
		}
	}

	/**
	 * enqueue core assets
	 *
	 * @since  1.0.0
	 * @access public
	 */
	public function enqueue_core() {

		// Register uix core asset.
		$this->core_assets();
		$this->enqueue_actions();
		// push assets to ui manager.
		uix()->set_assets( $this->assets );
		// done enqueuing - dpo inline or manual enqueue.
		$this->set_active_styles();
	}

	/**
	 * Do core enqueue actions.
	 *
	 * @since  3.0.0
	 * @access public
	 */
	private function enqueue_actions() {
		/**
		 * Do object initilisation.
		 *
		 * @param object current uix instance
		 */
		do_action( 'uix_admin_enqueue_scripts_' . $this->type, $this );
class MyClass { }

$x = new MyClass();
$x->foo = true;

		/**
		 * Do object initilisation for specific slug.
		 *
		 * @param object current uix instance
		 */
		do_action( 'uix_admin_enqueue_scripts_' . $this->type . '_' . $this->slug, $this );
class MyClass { }

$x = new MyClass();
$x->foo = true;
	}

	/**
	 * Register UIX depend js and css and call set assets
	 *
	 * @since  1.0.0
	 * @access protected
	 */
	protected function core_assets() {
		wp_register_script( 'uix', $this->url . 'assets/js/core' . UIX_ASSET_DEBUG . '.js' );
class MyClass { }

$x = new MyClass();
$x->foo = true;
		wp_register_style( 'uix', $this->url . 'assets/css/core' . UIX_ASSET_DEBUG . '.css', [ 'dashicons' ] );
		wp_localize_script( 'uix', 'uixApi', [
			'root'  => esc_url_raw( rest_url() ),
			'nonce' => wp_create_nonce( 'wp_rest' ),
		] );

		// set assets . methods at before this point can set assets, after this not so much.
		$this->set_assets();
	}

	/**
	 * runs after assets have been enqueued
	 *
	 * @since  1.0.0
	 * @access protected
	 */
	protected function set_active_styles() {
	}

	/**
	 * Base color helper
	 *
	 * @since  1.0.0
	 * @access public
	 */
	protected function base_color() {
		$color = '#D84315';
		if ( empty( $this->struct['base_color'] ) ) {
			if ( ! empty( $this->parent ) ) {
				$color = $this->parent->base_color();
class MyClass { }

$x = new MyClass();
$x->foo = true;
			}
		} else {
			$color = $this->struct['base_color'];
		}

		/**
		 * do object initilisation for specific slug
		 *
		 * @param object current uix instance
		 */
		return apply_filters( 'uix_base_color_' . $this->type . '_' . $this->slug, $color );

	}
}



1			<?php
2
3			/**
4			* Interface for WordPress core
5			*
6			* @package uix
7			* @author David Cramer
8			* @license GPL-2.0+
9			* @link
10			* @copyright 2016 David Cramer
11			*/
12
13			namespace uix\ui\core;
14
15			class core {
16
17			/**
18			* List of core object scripts ( common scripts )
19			*
20			* @since 1.0.0
21			* @access protected
22			* @var array
23			*/
24			protected $scripts = [];
25
26			/**
27			* List of core object styles ( common styles )
28			*
29			* @since 1.0.0
30			* @access protected
31			* @var array
32			*/
33			protected $styles = [];
34
35
36			/**
37			* setup actions and hooks - override to add specific hooks. use
38			* parent::actions() to keep admin head
39			*
40			* @since 1.0.0
41			* @access protected
42			*/
43			protected function actions() {
44
45			// init uix after loaded.
46			add_action( 'init', [ $this, 'init' ] );
47			// set location.
48			$location = 'wp_print_styles';
49			if ( is_admin() ) {
50			$location = 'admin_enqueue_scripts';
51			}
52			// init UIX headers.
53			add_action( $location, [ $this, 'enqueue_core' ] );
54			}
55
56
57			/**
58			* Define core UIX styles - override to register core ( common styles for
59			* uix type )
60			*
61			* @since 1.0.0
62			* @access public
63			*/
64			public function set_assets() {
65			if ( ! empty( $this->struct['style'] ) ) {
66			$this->assets['style'] = array_merge( $this->assets['style'], $this->struct['style'] );
			0 ignored issues – show Bug introduced 2018-06-21 10:04 UTC by Report Bug Copy Issue Report The property `assets` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history... Bug introduced 2018-06-21 10:04 UTC by Report Bug Copy Issue Report The property `struct` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
67			}
68			if ( ! empty( $this->struct['script'] ) ) {
69			$this->assets['script'] = array_merge( $this->assets['script'], $this->struct['script'] );
70			}
71			}
72
73			/**
74			* enqueue core assets
75			*
76			* @since 1.0.0
77			* @access public
78			*/
79			public function enqueue_core() {
80
81			// Register uix core asset.
82			$this->core_assets();
83			$this->enqueue_actions();
84			// push assets to ui manager.
85			uix()->set_assets( $this->assets );
86			// done enqueuing - dpo inline or manual enqueue.
87			$this->set_active_styles();
88			}
89
90			/**
91			* Do core enqueue actions.
92			*
93			* @since 3.0.0
94			* @access public
95			*/
96			private function enqueue_actions() {
97			/**
98			* Do object initilisation.
99			*
100			* @param object current uix instance
101			*/
102			do_action( 'uix_admin_enqueue_scripts_' . $this->type, $this );
			0 ignored issues – show Bug introduced 2018-06-21 10:04 UTC by Report Bug Copy Issue Report The property `type` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
103
104			/**
105			* Do object initilisation for specific slug.
106			*
107			* @param object current uix instance
108			*/
109			do_action( 'uix_admin_enqueue_scripts_' . $this->type . '_' . $this->slug, $this );
			0 ignored issues – show Bug introduced 2018-06-21 10:04 UTC by Report Bug Copy Issue Report The property `slug` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
110			}
111
112			/**
113			* Register UIX depend js and css and call set assets
114			*
115			* @since 1.0.0
116			* @access protected
117			*/
118			protected function core_assets() {
119			wp_register_script( 'uix', $this->url . 'assets/js/core' . UIX_ASSET_DEBUG . '.js' );
			0 ignored issues – show Bug introduced 2018-06-21 10:04 UTC by Report Bug Copy Issue Report The property `url` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
120			wp_register_style( 'uix', $this->url . 'assets/css/core' . UIX_ASSET_DEBUG . '.css', [ 'dashicons' ] );
121			wp_localize_script( 'uix', 'uixApi', [
122			'root' => esc_url_raw( rest_url() ),
123			'nonce' => wp_create_nonce( 'wp_rest' ),
124			] );
125
126			// set assets . methods at before this point can set assets, after this not so much.
127			$this->set_assets();
128			}
129
130			/**
131			* runs after assets have been enqueued
132			*
133			* @since 1.0.0
134			* @access protected
135			*/
136			protected function set_active_styles() {
137			}
138
139			/**
140			* Base color helper
141			*
142			* @since 1.0.0
143			* @access public
144			*/
145			protected function base_color() {
146			$color = '#D84315';
147			if ( empty( $this->struct['base_color'] ) ) {
148			if ( ! empty( $this->parent ) ) {
149			$color = $this->parent->base_color();
			0 ignored issues – show Bug introduced 2018-06-21 10:04 UTC by Report Bug Copy Issue Report The property `parent` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
150			}
151			} else {
152			$color = $this->struct['base_color'];
153			}
154
155			/**
156			* do object initilisation for specific slug
157			*
158			* @param object current uix instance
159			*/
160			return apply_filters( 'uix_base_color_' . $this->type . '_' . $this->slug, $color );
161
162			}
163			}
164
165

DavidCramer / uix

Push — develop ( 704bfc...0cb6ff )

core A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

7 Methods

Duplication Side-by-Side

Filter issues like