Wordlift_Geomap_Shortcode::render() - Code Metrics - Inspection of "update changelog" - insideout10/wordlift-plugin - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 370152...d94e50 )

by David

created 2017-06-22 06:17 UTC

Wordlift_Geomap_Shortcode::render() A

↳ Parent: Wordlift_Geomap_Shortcode

Complexity

Conditions	3
Paths	2

Size

Total Lines	54
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	31
nc	2
nop	1
dl	0
loc	54
rs	9.6716
c	0
b	0
f	0

How to fix Long Method

<?php
/**
 * Shortcodes: Geomap Shortcode.
 *
 * `wl_geomap` implementation.
 *
 * @since      3.5.4
 * @package    Wordlift
 * @subpackage Wordlift/includes
 */

/**
 * Define the {@link Wordlift_Geomap_Shortcode} class.
 *
 * @since      3.5.4
 * @package    Wordlift
 * @subpackage Wordlift/includes
 */
class Wordlift_Geomap_Shortcode extends Wordlift_Shortcode {

	const SHORTCODE = 'wl_geomap';

	/**
	 * Create a {@link Wordlift_Geomap_Shortcode} instance.
	 *
	 * @since 3.5.4
	 */
	public function __construct() {
		parent::__construct();

		// Hook to the `amp_post_template_css` to hide ourselves when in AMP
		// rendering.
		add_action( 'amp_post_template_css', array(
			$this,
			'amp_post_template_css',
		) );

	}

	/**
	 * Render the shortcode.
	 *
	 * @since 3.5.4
	 *
	 * @param array $atts An array of shortcode attributes as set by the editor.
	 *
	 * @return string The output html code.
	 */
	public function render( $atts ) {

		// Extract attributes and set default values.
		$geomap_atts = shortcode_atts( array(
			'width'  => '100%',
			'height' => '300px',
			'global' => false,
		), $atts );

		// Get id of the post
		$post_id = get_the_ID();

		if ( $geomap_atts['global'] || is_null( $post_id ) ) {
			// Global geomap
			$geomap_id = 'wl_geomap_global';
			$post_id   = null;
		} else {
			// Post-specific geomap
			$geomap_id = 'wl_geomap_' . $post_id;
		}

		// Add leaflet css and library.
		wp_enqueue_style(
			'leaflet',
			dirname( plugin_dir_url( __FILE__ ) ) . '/bower_components/leaflet/dist/leaflet.css'
		);
		wp_enqueue_script(
			'leaflet',
			dirname( plugin_dir_url( __FILE__ ) ) . '/bower_components/leaflet/dist/leaflet.js'
		);

		// Add wordlift-ui css and library.
		wp_enqueue_style( 'wordlift-ui-css', dirname( plugin_dir_url( __FILE__ ) ) . '/css/wordlift-ui.min.css' );

		$this->enqueue_scripts();

		wp_localize_script( 'wordlift-ui', 'wl_geomap_params', array(
			'ajax_url' => admin_url( 'admin-ajax.php' ),    // Global param
			'action'   => 'wl_geomap'            // Global param
		) );

		// Escaping atts.
		$esc_id      = esc_attr( $geomap_id );
		$esc_width   = esc_attr( $geomap_atts['width'] );
		$esc_height  = esc_attr( $geomap_atts['height'] );
		$esc_post_id = esc_attr( $post_id );

		// Return HTML template.
		return <<<EOF
<div class="wl-geomap"  id="$esc_id" data-post-id="$esc_post_id"
	style="width:$esc_width; height:$esc_height; background-color: gray;">
</div>
EOF;
	}

	/**
	 * Customize the CSS when in AMP.
	 *
	 * See https://github.com/Automattic/amp-wp/blob/master/readme.md#custom-css
	 *
	 * @since 3.13.0
	 *
	 * @param object $amp_template The template.
	 */
	public function amp_post_template_css( $amp_template ) {


		// Hide the `wl-geomap` when in AMP.
		?>
		.wl-geomap { display: none; }
		<?php
	}

}


1			<?php
2			/**
3			* Shortcodes: Geomap Shortcode.
4			*
5			* `wl_geomap` implementation.
6			*
7			* @since 3.5.4
8			* @package Wordlift
9			* @subpackage Wordlift/includes
10			*/
11
12			/**
13			* Define the {@link Wordlift_Geomap_Shortcode} class.
14			*
15			* @since 3.5.4
16			* @package Wordlift
17			* @subpackage Wordlift/includes
18			*/
19			class Wordlift_Geomap_Shortcode extends Wordlift_Shortcode {
20
21			const SHORTCODE = 'wl_geomap';
22
23			/**
24			* Create a {@link Wordlift_Geomap_Shortcode} instance.
25			*
26			* @since 3.5.4
27			*/
28			public function __construct() {
29			parent::__construct();
30
31			// Hook to the `amp_post_template_css` to hide ourselves when in AMP
32			// rendering.
33			add_action( 'amp_post_template_css', array(
34			$this,
35			'amp_post_template_css',
36			) );
37
38			}
39
40			/**
41			* Render the shortcode.
42			*
43			* @since 3.5.4
44			*
45			* @param array $atts An array of shortcode attributes as set by the editor.
46			*
47			* @return string The output html code.
48			*/
49			public function render( $atts ) {
50
51			// Extract attributes and set default values.
52			$geomap_atts = shortcode_atts( array(
53			'width' => '100%',
54			'height' => '300px',
55			'global' => false,
56			), $atts );
57
58			// Get id of the post
59			$post_id = get_the_ID();
60
61			if ( $geomap_atts['global'] \|\| is_null( $post_id ) ) {
62			// Global geomap
63			$geomap_id = 'wl_geomap_global';
64			$post_id = null;
65			} else {
66			// Post-specific geomap
67			$geomap_id = 'wl_geomap_' . $post_id;
68			}
69
70			// Add leaflet css and library.
71			wp_enqueue_style(
72			'leaflet',
73			dirname( plugin_dir_url( __FILE__ ) ) . '/bower_components/leaflet/dist/leaflet.css'
74			);
75			wp_enqueue_script(
76			'leaflet',
77			dirname( plugin_dir_url( __FILE__ ) ) . '/bower_components/leaflet/dist/leaflet.js'
78			);
79
80			// Add wordlift-ui css and library.
81			wp_enqueue_style( 'wordlift-ui-css', dirname( plugin_dir_url( __FILE__ ) ) . '/css/wordlift-ui.min.css' );
82
83			$this->enqueue_scripts();
84
85			wp_localize_script( 'wordlift-ui', 'wl_geomap_params', array(
86			'ajax_url' => admin_url( 'admin-ajax.php' ), // Global param
87			'action' => 'wl_geomap' // Global param
88			) );
89
90			// Escaping atts.
91			$esc_id = esc_attr( $geomap_id );
92			$esc_width = esc_attr( $geomap_atts['width'] );
93			$esc_height = esc_attr( $geomap_atts['height'] );
94			$esc_post_id = esc_attr( $post_id );
95
96			// Return HTML template.
97			return <<<EOF
98			<div class="wl-geomap" id="$esc_id" data-post-id="$esc_post_id"
99			style="width:$esc_width; height:$esc_height; background-color: gray;">
100			</div>
101			EOF;
102			}
103
104			/**
105			* Customize the CSS when in AMP.
106			*
107			* See https://github.com/Automattic/amp-wp/blob/master/readme.md#custom-css
108			*
109			* @since 3.13.0
110			*
111			* @param object $amp_template The template.
112			*/
113			public function amp_post_template_css( $amp_template ) {
			0 ignored issues – show Unused Code introduced 2017-06-22 05:31 UTC by Report Bug Copy Issue Report The parameter `$amp_template` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
114
115			// Hide the `wl-geomap` when in AMP.
116			?>
117			.wl-geomap { display: none; }
118			<?php
119			}
120
121			}
122

insideout10 / wordlift-plugin

Push — develop ( 370152...d94e50 )

Wordlift_Geomap_Shortcode::render() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like