Widget_Container::_is_valid_attach() - Code Metrics - Inspection of "Merge branch 'feature/2.0/rest-api-support' into m..." - htmlburger/carbon-fields - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — milestone/2.0 ( 8a1186...26a446 )

unknown

created 2017-02-10 15:10 UTC

Widget_Container::_is_valid_attach() A

↳ Parent: Widget_Container

Complexity

Conditions	4
Paths	6

Size

Total Lines	7
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	5
c	1
b	0
f	0
nc	6
nop	0
dl	0
loc	7
rs	9.2

<?php

namespace Carbon_Fields\Container;

/**
 * Widget container class
 */
class Widget_Container extends Container {

	/**
	 * Create a new widget
	 *
	 * @param string $unique_id Unique ID of the widget
	 * @param string $title Ignored
	 * @param string $type Container type
	 **/
	public function __construct( $unique_id, $title, $type ) {
		$this->id = $unique_id;
		$this->title = '';
		$this->type = $type;
	}

	/**
	 * Perform instance initialization
	 **/
	public function init() {
		$this->_attach();
		$this->render();

		return $this;
	}

	/**
	 * Perform checks whether the container should be attached during the current request
	 *
	 * @return bool True if the container is allowed to be attached
	 **/
	public function is_valid_attach_for_request() {
		$screen = get_current_screen();
		$request_action = isset( $_REQUEST['action'] ) ? $_REQUEST['action'] : '';

		$is_widget_save = ( $request_action === 'save-widget' );

		return $screen && $screen->id === 'widgets' || $is_widget_save;
	}

	/**
	 * Check container attachment rules against object id
	 *
	 * @return bool
	 **/
	public function is_valid_attach_for_object( $object_id = null ) {
		return true;
	}

	/**
	 * Output the container markup
	 **/
	public function render() {
		include \Carbon_Fields\DIR . '/templates/Container/widget.php';
	}

	/**
	 * Returns an array that holds the container data, suitable for JSON representation.
	 * This data will be available in the Underscore template and the Backbone Model.
	 *
	 * @param bool $load  Should the value be loaded from the database or use the value from the current instance.
	 * @return array
	 */
	public function to_json( $load ) {
		return parent::to_json( false );
	}
}



1			<?php
2
3			namespace Carbon_Fields\Container;
4
5			/**
6			* Widget container class
7			*/
8			class Widget_Container extends Container {
9
10			/**
11			* Create a new widget
12			*
13			* @param string $unique_id Unique ID of the widget
14			* @param string $title Ignored
15			* @param string $type Container type
16			**/
17			public function __construct( $unique_id, $title, $type ) {
18			$this->id = $unique_id;
19			$this->title = '';
20			$this->type = $type;
21			}
22
23			/**
24			* Perform instance initialization
25			**/
26			public function init() {
27			$this->_attach();
28			$this->render();
29
30			return $this;
31			}
32
33			/**
34			* Perform checks whether the container should be attached during the current request
35			*
36			* @return bool True if the container is allowed to be attached
37			**/
38			public function is_valid_attach_for_request() {
39			$screen = get_current_screen();
40			$request_action = isset( $_REQUEST['action'] ) ? $_REQUEST['action'] : '';
			1 ignored issue – show introduced 2016-09-06 07:48 UTC by Report Bug Copy Issue Report Detected access of super global var $_REQUEST, probably need manual inspection. Loading history... introduced 2016-09-06 07:48 UTC by Report Bug Copy Issue Report Detected usage of a non-sanitized input variable: $_REQUEST Loading history...
41			$is_widget_save = ( $request_action === 'save-widget' );
42
43			return $screen && $screen->id === 'widgets' \|\| $is_widget_save;
44			}
45
46			/**
47			* Check container attachment rules against object id
48			*
49			* @return bool
50			**/
51			public function is_valid_attach_for_object( $object_id = null ) {
52			return true;
53			}
54
55			/**
56			* Output the container markup
57			**/
58			public function render() {
59			include \Carbon_Fields\DIR . '/templates/Container/widget.php';
60			}
61
62			/**
63			* Returns an array that holds the container data, suitable for JSON representation.
64			* This data will be available in the Underscore template and the Backbone Model.
65			*
66			* @param bool $load Should the value be loaded from the database or use the value from the current instance.
67			* @return array
68			*/
69			public function to_json( $load ) {
70			return parent::to_json( false );
71			}
72			}
73
74

htmlburger / carbon-fields

Push — milestone/2.0 ( 8a1186...26a446 )

Widget_Container::_is_valid_attach() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like