Jetpack_Search_Debug_Bar::render() - Code Metrics - Inspection of "try a new module with woocommerce tracking, ported..." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — try/woocommerce-analytics ( 2b8c13...0a25c9 )

unknown

created 2018-02-21 21:13 UTC

Jetpack_Search_Debug_Bar::render() B

↳ Parent: Jetpack_Search_Debug_Bar

Complexity

Conditions	8
Paths	7

Size

Total Lines	61
Code Lines	48

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	8
eloc	48
nc	7
nop	0
dl	0
loc	61
rs	7.0047
c	0
b	0
f	0

How to fix Long Method

<?php

/**
 * Singleton class instantiated by Jetpack_Searc_Debug_Bar::instance() that handles
 * rendering the Jetpack Search debug bar menu item and panel.
 */
class Jetpack_Search_Debug_Bar extends Debug_Bar_Panel {
	/**
	 * Holds singleton instance
	 *
	 * @var Jetpack_Search_Debug_Bar
	 */
	protected static $instance = null;

	/**
	 * The title to use in the debug bar navigation
	 *
	 * @var string
	 */
	public $title;

	/**
	 * Constructor
	 */
	public function __construct() {
		$this->title( esc_html__( 'Jetpack Search', 'jetpack' ) );
		add_action( 'wp_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
		add_action( 'admin_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
	}

	/**
	 * Returns the singleton instance of Jetpack_Search_Debug_Bar
	 *
	 * @return Jetpack_Search_Debug_Bar
	 */
	public static function instance() {
		if ( is_null( self::$instance ) ) {
			self::$instance = new Jetpack_Search_Debug_Bar();
		}
		return self::$instance;
	}

	/**
	 * Enqueues styles for our panel in the debug bar
	 *
	 * @return void
	 */
	public function enqueue_scripts() {
		wp_enqueue_style(
			'jetpack-search-debug-bar',
			plugins_url( '3rd-party/debug-bar/debug-bar.css', JETPACK__PLUGIN_FILE )
		);
		wp_enqueue_script(
			'jetpack-search-debug-bar',
			plugins_url( '3rd-party/debug-bar/debug-bar.js', JETPACK__PLUGIN_FILE ),
			array( 'jquery' )
		);
	}

	/**
	 * Should the Jetpack Search Debug Bar show?
	 *
	 * Since we've previously done a check for the search module being activated, let's just return true.
	 * Later on, we can update this to only show when `is_search()` is true.
	 *
	 * @return boolean
	 */
	public function is_visible() {
		return true;
	}

	/**
	 * Renders the panel content
	 *
	 * @return void
	 */
	public function render() {
		if ( ! class_exists( 'Jetpack_Search' ) ) {
			return;
		}

		$jetpack_search = Jetpack_Search::instance();
		$last_query_info = $jetpack_search->get_last_query_info();

		// If not empty, let's reshuffle the order of some things.
		if ( ! empty( $last_query_info ) ) {
			$args     = $last_query_info['args'];
			$response = $last_query_info['response'];
			$response_code = $last_query_info['response_code'];

			unset( $last_query_info['args'] );
			unset( $last_query_info['response'] );
			unset( $last_query_info['response_code'] );

			if ( is_null( $last_query_info['es_time'] ) ) {
				$last_query_info['es_time'] = esc_html_x(
					'cache hit',
					'displayed in search results when results are cached',
					'jetpack'
				);
			}

			$temp = array_merge(
				array( 'response_code' => $response_code ),
				array( 'args' => $args ),
				$last_query_info,
				array( 'response' => $response )
			);

			$last_query_info = $temp;
		}
		?>
		<div class="jetpack-search-debug-bar">
			<h2><?php esc_html_e( 'Last query information:', 'jetpack' ); ?></h2>
			<?php if ( empty( $last_query_info ) ) : ?>
					<?php echo esc_html_x( 'None', 'Text displayed when there is no information', 'jetpack' ); ?>
			<?php
				else :
					foreach ( $last_query_info as $key => $info ) :
$collection = json_decode($data, true);
if ( ! is_array($collection)) {
    throw new \RuntimeException('$collection must be an array.');
}

foreach ($collection as $item) { /** ... */ }
					?>
						<h3><?php echo esc_html( $key ); ?></h3>
					<?php
					if ( 'response' !== $key && 'args' !== $key ) :
					?>
						<pre><?php print_r( esc_html( $info ) ); ?></pre>
					<?php
					else :
						$this->render_json_toggle( $info );
					endif;
					?>
					<?php
					endforeach;
			endif;
			?>
		</div><!-- Closes .jetpack-search-debug-bar -->
		<?php
	}

	/**
	 * Responsible for rendering the HTML necessary for the JSON toggle
	 *
	 * @param array $value The resonse from the API as an array.
	 * @return void
	 */
	public function render_json_toggle( $value ) {
	?>
		<div class="json-toggle-wrap">
			<pre class="json"><?php echo wp_json_encode( $value ); ?></pre>
			<span class="pretty toggle"><?php echo esc_html_x( 'Pretty', 'label for formatting JSON', 'jetpack' ); ?></span>
			<span class="ugly toggle"><?php echo esc_html_x( 'Minify', 'label for formatting JSON', 'jetpack' ); ?></span>
		</div>
	<?php
	}
}


1			<?php
2
3			/**
4			* Singleton class instantiated by Jetpack_Searc_Debug_Bar::instance() that handles
5			* rendering the Jetpack Search debug bar menu item and panel.
6			*/
7			class Jetpack_Search_Debug_Bar extends Debug_Bar_Panel {
8			/**
9			* Holds singleton instance
10			*
11			* @var Jetpack_Search_Debug_Bar
12			*/
13			protected static $instance = null;
14
15			/**
16			* The title to use in the debug bar navigation
17			*
18			* @var string
19			*/
20			public $title;
21
22			/**
23			* Constructor
24			*/
25			public function __construct() {
26			$this->title( esc_html__( 'Jetpack Search', 'jetpack' ) );
27			add_action( 'wp_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
28			add_action( 'admin_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
29			}
30
31			/**
32			* Returns the singleton instance of Jetpack_Search_Debug_Bar
33			*
34			* @return Jetpack_Search_Debug_Bar
35			*/
36			public static function instance() {
37			if ( is_null( self::$instance ) ) {
38			self::$instance = new Jetpack_Search_Debug_Bar();
39			}
40			return self::$instance;
41			}
42
43			/**
44			* Enqueues styles for our panel in the debug bar
45			*
46			* @return void
47			*/
48			public function enqueue_scripts() {
49			wp_enqueue_style(
50			'jetpack-search-debug-bar',
51			plugins_url( '3rd-party/debug-bar/debug-bar.css', JETPACK__PLUGIN_FILE )
52			);
53			wp_enqueue_script(
54			'jetpack-search-debug-bar',
55			plugins_url( '3rd-party/debug-bar/debug-bar.js', JETPACK__PLUGIN_FILE ),
56			array( 'jquery' )
57			);
58			}
59
60			/**
61			* Should the Jetpack Search Debug Bar show?
62			*
63			* Since we've previously done a check for the search module being activated, let's just return true.
64			* Later on, we can update this to only show when `is_search()` is true.
65			*
66			* @return boolean
67			*/
68			public function is_visible() {
69			return true;
70			}
71
72			/**
73			* Renders the panel content
74			*
75			* @return void
76			*/
77			public function render() {
78			if ( ! class_exists( 'Jetpack_Search' ) ) {
79			return;
80			}
81
82			$jetpack_search = Jetpack_Search::instance();
83			$last_query_info = $jetpack_search->get_last_query_info();
84
85			// If not empty, let's reshuffle the order of some things.
86			if ( ! empty( $last_query_info ) ) {
87			$args = $last_query_info['args'];
88			$response = $last_query_info['response'];
89			$response_code = $last_query_info['response_code'];
90
91			unset( $last_query_info['args'] );
92			unset( $last_query_info['response'] );
93			unset( $last_query_info['response_code'] );
94
95			if ( is_null( $last_query_info['es_time'] ) ) {
96			$last_query_info['es_time'] = esc_html_x(
97			'cache hit',
98			'displayed in search results when results are cached',
99			'jetpack'
100			);
101			}
102
103			$temp = array_merge(
104			array( 'response_code' => $response_code ),
105			array( 'args' => $args ),
106			$last_query_info,
107			array( 'response' => $response )
108			);
109
110			$last_query_info = $temp;
111			}
112			?>
113			<div class="jetpack-search-debug-bar">
114			<h2><?php esc_html_e( 'Last query information:', 'jetpack' ); ?></h2>
115			<?php if ( empty( $last_query_info ) ) : ?>
116			<?php echo esc_html_x( 'None', 'Text displayed when there is no information', 'jetpack' ); ?>
117			<?php
118			else :
119			foreach ( $last_query_info as $key => $info ) :
			0 ignored issues – show Bug introduced 2018-01-25 15:56 UTC by Report Bug Copy Issue Report The expression `$last_query_info` of type `boolean\|array` is not guaranteed to be traversable. How about adding an additional type check? There are different options of fixing this problem. If you want to be on the safe side, you can add an additional type-check: $collection = json_decode($data, true); if ( ! is_array($collection)) { throw new \RuntimeException('$collection must be an array.'); } foreach ($collection as $item) { /** ... / } If you are sure that the expression is traversable, you might want to add a doc comment cast* to improve IDE auto-completion and static analysis: /** @var array $collection / $collection = json_decode($data, true); foreach ($collection as $item) { /* .. / } Mark the issue as a false-positive*: Just hover the remove button, in the top-right corner of this issue for more options. Loading history...
120			?>
121			<h3><?php echo esc_html( $key ); ?></h3>
122			<?php
123			if ( 'response' !== $key && 'args' !== $key ) :
124			?>
125			<pre><?php print_r( esc_html( $info ) ); ?></pre>
126			<?php
127			else :
128			$this->render_json_toggle( $info );
129			endif;
130			?>
131			<?php
132			endforeach;
133			endif;
134			?>
135			</div><!-- Closes .jetpack-search-debug-bar -->
136			<?php
137			}
138
139			/**
140			* Responsible for rendering the HTML necessary for the JSON toggle
141			*
142			* @param array $value The resonse from the API as an array.
143			* @return void
144			*/
145			public function render_json_toggle( $value ) {
146			?>
147			<div class="json-toggle-wrap">
148			<pre class="json"><?php echo wp_json_encode( $value ); ?></pre>
149			<span class="pretty toggle"><?php echo esc_html_x( 'Pretty', 'label for formatting JSON', 'jetpack' ); ?></span>
150			<span class="ugly toggle"><?php echo esc_html_x( 'Minify', 'label for formatting JSON', 'jetpack' ); ?></span>
151			</div>
152			<?php
153			}
154			}
155

Automattic / jetpack

Push — try/woocommerce-analytics ( 2b8c13...0a25c9 )

Jetpack_Search_Debug_Bar::render() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like