Shortcode - Code Metrics - Inspection of "First shot at integrating Gamajo_Template_Loader...." - brightnucleus/shortcodes - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 891229...7f6823 )

by Alain

created 2016-02-24 05:38 UTC

Shortcode A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	194
Duplicated Lines	13.92 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

Importance

Changes	7
Bugs	0	Features	1

Metric	Value
wmc	14
c	7
b	0
f	1
lcom	1
cbo	4
dl	27
loc	194
ccs	0
cts	43
cp	0
rs	10

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	15	15	1
A	register()	0	8	2
A	is_needed()	12	12	3
A	get_tag()	0	3	1
A	render()	0	10	2
A	render_view()	0	9	2
A	get_view()	0	10	2
A	do_this()	0	3	1

How to fix Duplicated Code

<?php

/**
 * Shortcode Base Implementation.
 *
 * @package   BrightNucleus\Shortcode
 * @author    Alain Schlesser <[email protected]>
 * @license   GPL-2.0+
 * @link      http://www.brightnucleus.com/
 * @copyright 2015-2016 Alain Schlesser, Bright Nucleus
 */

namespace BrightNucleus\Shortcode;

use Assert;
use BrightNucleus\Config\ConfigInterface;
use BrightNucleus\Config\ConfigTrait;
use BrightNucleus\Dependency\DependencyManagerInterface as DependencyManager;
use BrightNucleus\Exception\DomainException;
use BrightNucleus\Exception\RuntimeException;

/**
 * Base Implementation of the Shortcode Interface.
 *
 * This is a basic implementation of the Shortcode Interface that registers one
 * view and passes all attributes unfiltered to that view.
 *
 * @since   0.1.0
 *
 * @package BrightNucleus\Shortcode
 * @author  Alain Schlesser <[email protected]>
 */
class Shortcode implements ShortcodeInterface {

	use ConfigTrait;

	/**
	 * Name of the shortcode handler.
	 *
	 * @since 0.1.0
	 *
	 * @var string
	 */
	protected $shortcode_tag;

	/**
	 * Parser to parse and validate the shortcode's attributes.
	 *
	 * @since 0.1.0
	 *
	 * @var ShortcodeAttsParserInterface
	 */
	protected $atts_parser;

	/**
	 * Dependencies of the shortcode.
	 *
	 * @since 0.1.0
	 *
	 * @var DependencyManager
	 */
	protected $dependencies;

	/**
	 * Cache context information so we can pass it on to the render() method.
	 *
	 * @var
	 *
	 * @since 0.2.3
	 */
	protected $context;

	/**
	 * Instantiate Basic Shortcode.
	 *
	 * @since 0.1.0
	 *
	 * @param string                 $shortcode_tag Tag that identifies the
	 *                                              shortcode.
	 * @param ConfigInterface        $config        Configuration settings.
	 * @param ShortcodeAttsParser    $atts_parser   Attributes parser and
	 *                                              validator.
	 * @param DependencyManager|null $dependencies  Optional. Dependencies of
	 *                                              the shortcode.
	 * @throws RuntimeException If the config could not be processed.
	 */
	public function __construct(

		$shortcode_tag,
		ConfigInterface $config,
		ShortcodeAttsParser $atts_parser,
		DependencyManager $dependencies = null
	) {

		Assert\that( $shortcode_tag )->string()->notEmpty();

		$this->processConfig( $config );

		$this->shortcode_tag = $shortcode_tag;
		$this->atts_parser   = $atts_parser;
		$this->dependencies  = $dependencies;
	}

	/**
	 * Register the shortcode handler function with WordPress.
	 *
	 * @since 0.1.0
	 *
	 * @param mixed $context Optional. Arguments to pass on to the Registrable.
	 * @return void
	 */
	public function register( $context = null ) {
		if ( ! $this->is_needed( $context ) ) {
			return;
		}
		$this->context = $context;

		\add_shortcode( $this->get_tag(), [ $this, 'render' ] );
	}

	/**
	 * Check whether the shortcode is needed.
	 *
	 * @since 0.2.0
	 *
	 * @param mixed $context Data about the context in which the call is made.
	 * @return boolean Whether the shortcode is needed or not.
	 */
	protected function is_needed( $context = null ) {


		$is_needed = $this->hasConfigKey( 'is_needed' )
			? $this->getConfigKey( 'is_needed' )
			: false;

		if ( is_callable( $is_needed ) ) {
			return $is_needed( $context );
		}

		return (bool) $is_needed;
	}

	/**
	 * Get the shortcode tag.
	 *
	 * @since 0.1.0
	 *
	 * @return string Shortcode tag.
	 */
	public function get_tag() {
		return (string) $this->shortcode_tag;
	}

	/**
	 * Render the shortcode.
	 *
	 * @since 0.1.0
	 *
	 * @throws DomainException
	 *
	 * @param  array       $atts    Attributes to modify the standard behavior
	 *                              of the shortcode.
	 * @param  string|null $content Optional. Content between enclosing
	 *                              shortcodes.
	 * @param string|null  $tag     Optional. The tag of the shortcode to
	 *                              render.
	 * @return string               The shortcode's HTML output.
	 */
	public function render( $atts, $content = null, $tag = null ) {
		$context = $this->context;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
		$atts    = $this->atts_parser->parse_atts( $atts, $this->get_tag() );

		if ( $this->dependencies ) {
			$this->dependencies->enqueue( $atts );
		}

		return $this->render_view( $this->get_view() );
	}

	/**
	 * Get the rendered HTML for a given view.
	 *
	 * @since 0.2.6
	 *
	 * @param string $view The view to render.
	 * @return string HTML rendering of the view.
	 */
	protected function render_view( $view ) {
		if ( empty( $view ) ) {
			return '';
		}

		ob_start();
		include( $view );
		return ob_get_clean();
	}

	/**
	 * Get the name of the view to render.
	 *
	 * @since 0.2.6
	 *
	 * @return string Name of the view to render.
	 */
	protected function get_view() {
		if ( ! $this->hasConfigKey( 'view' ) ) {
			return '';
		}
		$view = $this->getConfigKey( 'view' );

		Assert\that( $view )->string()->notEmpty()->file();

		return $view;
	}

	/**
	 * Execute this shortcode directly from code.
	 *
	 * @since 0.2.4
	 *
	 * @param array       $atts    Array of attributes to pass to the shortcode.
	 * @param string|null $content Inner content to pass to the shortcode.
	 * @return string|false Rendered HTML.

	 */
	public function do_this( array $atts = [ ], $content = null ) {
		\BrightNucleus\Shortcode\do_tag( $this->get_tag(), $atts, $content );
	}
}


1		<?php
		0 ignored issues – show Coding Style introduced 2016-02-22 10:58 UTC by Report Bug Copy Issue Report End of line character is invalid; expected "\n" but found "\r\n" Loading history...
2		/**
3		* Shortcode Base Implementation.
4		*
5		* @package BrightNucleus\Shortcode
6		* @author Alain Schlesser <[email protected]>
7		* @license GPL-2.0+
8		* @link http://www.brightnucleus.com/
9		* @copyright 2015-2016 Alain Schlesser, Bright Nucleus
10		*/
11
12		namespace BrightNucleus\Shortcode;
13
14		use Assert;
15		use BrightNucleus\Config\ConfigInterface;
16		use BrightNucleus\Config\ConfigTrait;
17		use BrightNucleus\Dependency\DependencyManagerInterface as DependencyManager;
18		use BrightNucleus\Exception\DomainException;
19		use BrightNucleus\Exception\RuntimeException;
20
21		/**
22		* Base Implementation of the Shortcode Interface.
23		*
24		* This is a basic implementation of the Shortcode Interface that registers one
25		* view and passes all attributes unfiltered to that view.
26		*
27		* @since 0.1.0
28		*
29		* @package BrightNucleus\Shortcode
30		* @author Alain Schlesser <[email protected]>
31		*/
32		class Shortcode implements ShortcodeInterface {
33
34		use ConfigTrait;
35
36		/**
37		* Name of the shortcode handler.
38		*
39		* @since 0.1.0
40		*
41		* @var string
42		*/
43		protected $shortcode_tag;
44
45		/**
46		* Parser to parse and validate the shortcode's attributes.
47		*
48		* @since 0.1.0
49		*
50		* @var ShortcodeAttsParserInterface
51		*/
52		protected $atts_parser;
53
54		/**
55		* Dependencies of the shortcode.
56		*
57		* @since 0.1.0
58		*
59		* @var DependencyManager
60		*/
61		protected $dependencies;
62
63		/**
64		* Cache context information so we can pass it on to the render() method.
65		*
66		* @var
67		*
68		* @since 0.2.3
69		*/
70		protected $context;
71
72		/**
73		* Instantiate Basic Shortcode.
74		*
75		* @since 0.1.0
76		*
77		* @param string $shortcode_tag Tag that identifies the
78		* shortcode.
79		* @param ConfigInterface $config Configuration settings.
80		* @param ShortcodeAttsParser $atts_parser Attributes parser and
81		* validator.
82		* @param DependencyManager\|null $dependencies Optional. Dependencies of
83		* the shortcode.
84		* @throws RuntimeException If the config could not be processed.
85		*/
86	View Code Duplication	public function __construct(
		0 ignored issues – show Duplication introduced 2016-02-17 20:01 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
87		$shortcode_tag,
88		ConfigInterface $config,
89		ShortcodeAttsParser $atts_parser,
90		DependencyManager $dependencies = null
91		) {
92
93		Assert\that( $shortcode_tag )->string()->notEmpty();
94
95		$this->processConfig( $config );
96
97		$this->shortcode_tag = $shortcode_tag;
98		$this->atts_parser = $atts_parser;
99		$this->dependencies = $dependencies;
100		}
101
102		/**
103		* Register the shortcode handler function with WordPress.
104		*
105		* @since 0.1.0
106		*
107		* @param mixed $context Optional. Arguments to pass on to the Registrable.
108		* @return void
109		*/
110		public function register( $context = null ) {
111		if ( ! $this->is_needed( $context ) ) {
112		return;
113		}
114		$this->context = $context;
115
116		\add_shortcode( $this->get_tag(), [ $this, 'render' ] );
117		}
118
119		/**
120		* Check whether the shortcode is needed.
121		*
122		* @since 0.2.0
123		*
124		* @param mixed $context Data about the context in which the call is made.
125		* @return boolean Whether the shortcode is needed or not.
126		*/
127	View Code Duplication	protected function is_needed( $context = null ) {
		0 ignored issues – show Duplication introduced 2016-02-18 10:36 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
128
129		$is_needed = $this->hasConfigKey( 'is_needed' )
130		? $this->getConfigKey( 'is_needed' )
131		: false;
132
133		if ( is_callable( $is_needed ) ) {
134		return $is_needed( $context );
135		}
136
137		return (bool) $is_needed;
138		}
139
140		/**
141		* Get the shortcode tag.
142		*
143		* @since 0.1.0
144		*
145		* @return string Shortcode tag.
146		*/
147		public function get_tag() {
148		return (string) $this->shortcode_tag;
149		}
150
151		/**
152		* Render the shortcode.
153		*
154		* @since 0.1.0
155		*
156		* @throws DomainException
157		*
158		* @param array $atts Attributes to modify the standard behavior
159		* of the shortcode.
160		* @param string\|null $content Optional. Content between enclosing
161		* shortcodes.
162		* @param string\|null $tag Optional. The tag of the shortcode to
163		* render.
164		* @return string The shortcode's HTML output.
165		*/
166		public function render( $atts, $content = null, $tag = null ) {
167		$context = $this->context;
		0 ignored issues – show Unused Code introduced 2016-02-24 05:41 UTC by Report Bug Copy Issue Report `$context` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
168		$atts = $this->atts_parser->parse_atts( $atts, $this->get_tag() );
169
170		if ( $this->dependencies ) {
171		$this->dependencies->enqueue( $atts );
172		}
173
174		return $this->render_view( $this->get_view() );
175		}
176
177		/**
178		* Get the rendered HTML for a given view.
179		*
180		* @since 0.2.6
181		*
182		* @param string $view The view to render.
183		* @return string HTML rendering of the view.
184		*/
185		protected function render_view( $view ) {
186		if ( empty( $view ) ) {
187		return '';
188		}
189
190		ob_start();
191		include( $view );
192		return ob_get_clean();
193		}
194
195		/**
196		* Get the name of the view to render.
197		*
198		* @since 0.2.6
199		*
200		* @return string Name of the view to render.
201		*/
202		protected function get_view() {
203		if ( ! $this->hasConfigKey( 'view' ) ) {
204		return '';
205		}
206		$view = $this->getConfigKey( 'view' );
207
208		Assert\that( $view )->string()->notEmpty()->file();
209
210		return $view;
211		}
212
213		/**
214		* Execute this shortcode directly from code.
215		*
216		* @since 0.2.4
217		*
218		* @param array $atts Array of attributes to pass to the shortcode.
219		* @param string\|null $content Inner content to pass to the shortcode.
220		* @return string\|false Rendered HTML.
		0 ignored issues – show Documentation introduced 2016-02-22 10:34 UTC by Report Bug Copy Issue Report Should the return type not be `string\|false\|null`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
221		*/
222		public function do_this( array $atts = [ ], $content = null ) {
223		\BrightNucleus\Shortcode\do_tag( $this->get_tag(), $atts, $content );
224		}
225		}
226

brightnucleus / shortcodes

Push — master ( 891229...7f6823 )

Shortcode A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

8 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like