Wordlift_Geomap_Shortcode

Complexity

Total Complexity

5

Size/Duplication

Total Lines	106
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
B	render()	0	57	3
A	amp_post_template_css()	0	7	1

<?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'
		);

		// Use the registered style which define an optional dependency to font-awesome.
		//
		// @see https://github.com/insideout10/wordlift-plugin/issues/699
		//		wp_enqueue_style( 'wordlift-ui', dirname( plugin_dir_url( __FILE__ ) ) . '/css/wordlift-ui.min.css' );

44% of this comment could be valid code. Did you maybe forget this after debugging?

		wp_enqueue_style( 'wordlift-ui' );

		$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 ) {

The parameter $amp_template is not used and could be removed.

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

}