Kirki_Control_Image::content_template() - Code Metrics - Inspection of "possible fix for #1498" - aristath/kirki - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( c69b96...434ccf )

by Aristeides

created 2017-08-09 18:07 UTC

Kirki_Control_Image::content_template() A

↳ Parent: Kirki_Control_Image

Complexity

Conditions	2
Paths	1

Size

Total Lines	55
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	14
nc	1
nop	0
dl	0
loc	55
rs	9.7692
c	0
b	0
f	0

How to fix Long Method

<?php
/**
 * Customizer Control: image.
 *
 * @package     Kirki
 * @subpackage  Controls
 * @copyright   Copyright (c) 2017, Aristeides Stathopoulos
 * @license     http://opensource.org/licenses/https://opensource.org/licenses/MIT
 * @since       3.0.0
 */

/**
 * Adds the image control.
 */
class Kirki_Control_Image extends WP_Customize_Control {

	/**
	 * The control type.
	 *
	 * @access public
	 * @var string
	 */
	public $type = 'kirki-image';

	/**
	 * Used to automatically generate all CSS output.
	 *
	 * @access public
	 * @var array
	 */
	public $output = array();

	/**
	 * Data type
	 *
	 * @access public
	 * @var string
	 */
	public $option_type = 'theme_mod';

	/**
	 * Enqueue control related scripts/styles.
	 *
	 * @access public
	 */
	public function enqueue() {

		wp_enqueue_script( 'kirki-image', trailingslashit( Kirki::$url ) . 'controls/image/image.js', array( 'jquery', 'customize-base' ) );
		wp_enqueue_style( 'kirki-image', trailingslashit( Kirki::$url ) . 'controls/image/image.css', null );
	}

	/**
	 * Refresh the parameters passed to the JavaScript via JSON.
	 *
	 * @access public
	 */
	public function to_json() {
		parent::to_json();

		$this->json['default'] = $this->setting->default;
		if ( isset( $this->default ) ) {
			$this->json['default'] = $this->default;
		}
		$this->json['output']  = $this->output;
		$this->json['value']   = $this->value();
		$this->json['choices'] = $this->choices;
		$this->json['link']    = $this->get_link();
		$this->json['id']      = $this->id;

		$this->json['inputAttrs'] = '';
		foreach ( $this->input_attrs as $attr => $value ) {
			$this->json['inputAttrs'] .= $attr . '="' . esc_attr( $value ) . '" ';
		}
	}

	/**
	 * Render the control's content.
	 *
	 * @see WP_Customize_Control::render_content()
	 */
	protected function render_content() {}

	/**
	 * An Underscore (JS) template for this control's content (but not its container).
	 *
	 * Class variables for this control class are available in the `data` JS object;
	 * export custom variables by overriding {@see WP_Customize_Control::to_json()}.
	 *
	 * @see WP_Customize_Control::print_template()
	 *
	 * @access protected
	 */
	protected function content_template() {
		?>
		<#
		var saveAs = 'url',
		    url    = '';

		if ( ! _.isUndefined( data.choices ) && ! _.isUndefined( data.choices.save_as ) ) {
			saveAs = data.choices.save_as;
		}
		if ( 'url' === saveAs ) {
			url = data.value;
		}
		if ( 'array' === saveAs && data.value['url'] ) {
			url = data.value['url'];
		}
		if ( 'id' === saveAs ) {
			url = <?php echo ( is_int( $this->value() ) ) ? wp_get_attachment_url( $this->value() ) : "''"; ?>

		}
		#>
		<label>
			<span class="customize-control-title">
				{{{ data.label }}}
			</span>
			<# if ( data.description ) { #>
				<span class="description customize-control-description">{{{ data.description }}}</span>
			<# } #>
		</label>
		<div class="image-wrapper attachment-media-view image-upload">
			<# if ( data.value['url'] || '' !== url ) { #>
				<div class="thumbnail thumbnail-image">
					<img src="{{ url }}" alt="" />
				</div>
			<# } else { #>
				<div class="placeholder">
					<?php esc_attr_e( 'No File Selected', 'kirki' ); ?>
				</div>
			<# } #>
			<div class="actions">
				<button class="button image-upload-remove-button<# if ( '' === url ) { #> hidden <# } #>">
					<?php esc_attr_e( 'Remove', 'kirki' ); ?>
				</button>
				<# if ( data.default && '' !== data.default ) { #>
					<button type="button" class="button image-default-button"<# if ( data.default === data.value || ( ! _.isUndefined( data.value.url ) && data.default === data.value.url ) ) { #> style="display:none;"<# } #>>
						<?php esc_attr_e( 'Default', 'kirki' ); ?>
					</button>
				<# } #>
				<button type="button" class="button image-upload-button">
					<?php esc_attr_e( 'Select File', 'kirki' ); ?>
				</button>
			</div>
		</div>
		<# value = ( 'array' === saveAs ) ? JSON.stringify( data.value ) : data.value; #>
		<input class="image-hidden-value" type="hidden" value='{{{ value }}}' {{{ data.link }}}>
		<?php
	}
}


1			<?php
2			/**
3			* Customizer Control: image.
4			*
5			* @package Kirki
6			* @subpackage Controls
7			* @copyright Copyright (c) 2017, Aristeides Stathopoulos
8			* @license http://opensource.org/licenses/https://opensource.org/licenses/MIT
9			* @since 3.0.0
10			*/
11
12			/**
13			* Adds the image control.
14			*/
15			class Kirki_Control_Image extends WP_Customize_Control {
16
17			/**
18			* The control type.
19			*
20			* @access public
21			* @var string
22			*/
23			public $type = 'kirki-image';
24
25			/**
26			* Used to automatically generate all CSS output.
27			*
28			* @access public
29			* @var array
30			*/
31			public $output = array();
32
33			/**
34			* Data type
35			*
36			* @access public
37			* @var string
38			*/
39			public $option_type = 'theme_mod';
40
41			/**
42			* Enqueue control related scripts/styles.
43			*
44			* @access public
45			*/
46			public function enqueue() {
47
48			wp_enqueue_script( 'kirki-image', trailingslashit( Kirki::$url ) . 'controls/image/image.js', array( 'jquery', 'customize-base' ) );
49			wp_enqueue_style( 'kirki-image', trailingslashit( Kirki::$url ) . 'controls/image/image.css', null );
50			}
51
52			/**
53			* Refresh the parameters passed to the JavaScript via JSON.
54			*
55			* @access public
56			*/
57			public function to_json() {
58			parent::to_json();
59
60			$this->json['default'] = $this->setting->default;
61			if ( isset( $this->default ) ) {
62			$this->json['default'] = $this->default;
63			}
64			$this->json['output'] = $this->output;
65			$this->json['value'] = $this->value();
66			$this->json['choices'] = $this->choices;
67			$this->json['link'] = $this->get_link();
68			$this->json['id'] = $this->id;
69
70			$this->json['inputAttrs'] = '';
71			foreach ( $this->input_attrs as $attr => $value ) {
72			$this->json['inputAttrs'] .= $attr . '="' . esc_attr( $value ) . '" ';
73			}
74			}
75
76			/**
77			* Render the control's content.
78			*
79			* @see WP_Customize_Control::render_content()
80			*/
81			protected function render_content() {}
82
83			/**
84			* An Underscore (JS) template for this control's content (but not its container).
85			*
86			* Class variables for this control class are available in the `data` JS object;
87			* export custom variables by overriding {@see WP_Customize_Control::to_json()}.
88			*
89			* @see WP_Customize_Control::print_template()
90			*
91			* @access protected
92			*/
93			protected function content_template() {
94			?>
95			<#
96			var saveAs = 'url',
97			url = '';
98
99			if ( ! _.isUndefined( data.choices ) && ! _.isUndefined( data.choices.save_as ) ) {
100			saveAs = data.choices.save_as;
101			}
102			if ( 'url' === saveAs ) {
103			url = data.value;
104			}
105			if ( 'array' === saveAs && data.value['url'] ) {
106			url = data.value['url'];
107			}
108			if ( 'id' === saveAs ) {
109			url = <?php echo ( is_int( $this->value() ) ) ? wp_get_attachment_url( $this->value() ) : "''"; ?>
			0 ignored issues – show introduced 2017-08-09 18:09 UTC by Report Bug Copy Issue Report Expected next thing to be a escaping function, not '(' Loading history...
110			}
111			#>
112			<label>
113			<span class="customize-control-title">
114			{{{ data.label }}}
115			</span>
116			<# if ( data.description ) { #>
117			<span class="description customize-control-description">{{{ data.description }}}</span>
118			<# } #>
119			</label>
120			<div class="image-wrapper attachment-media-view image-upload">
121			<# if ( data.value['url'] \|\| '' !== url ) { #>
122			<div class="thumbnail thumbnail-image">
123			<img src="{{ url }}" alt="" />
124			</div>
125			<# } else { #>
126			<div class="placeholder">
127			<?php esc_attr_e( 'No File Selected', 'kirki' ); ?>
128			</div>
129			<# } #>
130			<div class="actions">
131			<button class="button image-upload-remove-button<# if ( '' === url ) { #> hidden <# } #>">
132			<?php esc_attr_e( 'Remove', 'kirki' ); ?>
133			</button>
134			<# if ( data.default && '' !== data.default ) { #>
135			<button type="button" class="button image-default-button"<# if ( data.default === data.value \|\| ( ! _.isUndefined( data.value.url ) && data.default === data.value.url ) ) { #> style="display:none;"<# } #>>
136			<?php esc_attr_e( 'Default', 'kirki' ); ?>
137			</button>
138			<# } #>
139			<button type="button" class="button image-upload-button">
140			<?php esc_attr_e( 'Select File', 'kirki' ); ?>
141			</button>
142			</div>
143			</div>
144			<# value = ( 'array' === saveAs ) ? JSON.stringify( data.value ) : data.value; #>
145			<input class="image-hidden-value" type="hidden" value='{{{ value }}}' {{{ data.link }}}>
146			<?php
147			}
148			}
149

aristath / kirki

Push — develop ( c69b96...434ccf )

Kirki_Control_Image::content_template() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like