insideout10 / wordlift-plugin

src/public/class-wordlift-faceted-search-shortcode.php

Indentation +193 added lines, -193 removed lines

@@ -17,146 +17,146 @@ 
  */
 class Wordlift_Faceted_Search_Shortcode extends Wordlift_Shortcode {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	const SHORTCODE = 'wl_faceted_search';
-
-	public function __construct() {
-		parent::__construct();
-		$this->register_block_type();
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function render( $atts ) {
-
-		return Wordlift_AMP_Service::is_amp_endpoint() ? $this->amp_shortcode( $atts )
-			: $this->web_shortcode( $atts );
-	}
-
-	private function register_block_type() {
-
-		$scope = $this;
-
-		add_action( 'init', function () use ( $scope ) {
-			if ( ! function_exists( 'register_block_type' ) ) {
-				// Gutenberg is not active.
-				return;
-			}
-
-			register_block_type( 'wordlift/faceted-search', array(
-				'editor_script'   => 'wl-block-editor',
-				'render_callback' => function ( $attributes ) use ( $scope ) {
-					$attr_code = '';
-					foreach ( $attributes as $key => $value ) {
-						$attr_code .= $key . '="' . htmlentities( $value ) . '" ';
-					}
-
-					return '[' . $scope::SHORTCODE . ' ' . $attr_code . ']';
-				},
-				'attributes'      => array(
-					'title'       => array(
-						'type'    => 'string',
-						'default' => __( 'Related articles', 'wordlift' ),
-					),
-					'template_id' => array(
-						'type' => 'string',
-						'default' => '',
-					),
-					'post_id'     => array(
-						'type'    => 'number',
-						'default' => '',
-					),
-					'uniqid'      => array(
-						'type'    => 'string',
-						'default' => '',
-					),
-					'limit'       => array(
-						'type'    => 'number',
-						'default' => apply_filters( 'wl_faceted_search_default_limit', 10 ),
-					),
-					'preview'     => array(
-						'type'    => 'boolean',
-						'default' => false,
-					),
-					'preview_src' => array(
-						'type'    => 'string',
-						'default' => WP_CONTENT_URL . '/plugins/wordlift/images/block-previews/faceted-search.png',
-					),
-				),
-			) );
-		} );
-	}
-
-	/**
-	 * Shared function used by web_shortcode and amp_shortcode
-	 * Bootstrap logic for attributes extraction and boolean filtering
-	 *
-	 * @since      3.20.0
-	 *
-	 * @param array $atts Shortcode attributes.
-	 *
-	 * @return array $shortcode_atts
-	 */
-	private function make_shortcode_atts( $atts ) {
-
-		// Extract attributes and set default values.
-		$shortcode_atts = shortcode_atts( array(
-			'title'       => __( 'Related articles', 'wordlift' ),
-			'limit'       => apply_filters( 'wl_faceted_search_default_limit', 10 ),
-			'post_id'     => '',
-			'template_id' => '',
-			'uniqid'      => uniqid( 'wl-faceted-widget-' ),
-		), $atts );
-
-		return $shortcode_atts;
-	}
-
-	/**
-	 * Function in charge of diplaying the [wl-faceted-search] in web mode.
-	 *
-	 * @since 3.20.0
-	 *
-	 * @param array $atts Shortcode attributes.
-	 *
-	 * @return string Shortcode HTML for web
-	 */
-	private function web_shortcode( $atts ) {
-
-		// attributes extraction and boolean filtering
-		$shortcode_atts = $this->make_shortcode_atts( $atts );
-
-		// avoid building the widget when no post_id is specified and there is a list of posts.
-		if ( empty( $shortcode_atts['post_id'] ) && ! is_singular() ) {
-			return;
-		}
-
-		$post        = ! empty( $shortcode_atts['post_id'] ) ? get_post( intval( sanitize_text_field( $shortcode_atts['post_id'] ) ) ) : get_post();
-		$title       = esc_attr( sanitize_text_field( $shortcode_atts['title'] ) );
-		$template_id = esc_attr( sanitize_text_field( $shortcode_atts['template_id'] ) );
-		$limit       = esc_attr( sanitize_text_field( $shortcode_atts['limit'] ) );
-		$faceted_id  = ! empty( $shortcode_atts['uniqid'] ) ? esc_attr( sanitize_text_field( $shortcode_atts['uniqid'] ) ) : uniqid( 'wl-faceted-widget-' );
-
-		$permalink_structure = get_option( 'permalink_structure' );
-		$delimiter           = empty( $permalink_structure ) ? '&' : '?';
-		$rest_url            = $post ? rest_url( WL_REST_ROUTE_DEFAULT_NAMESPACE . '/faceted-search' . $delimiter . build_query( array(
-				'post_id' => $post->ID,
-				'limit'   => $limit
-			) ) ) : false;
-		$rest_url = esc_attr( $rest_url );
-
-		// avoid building the widget when no valid $rest_url
-		if ( ! $rest_url ) {
-			return;
-		}
-
-		wp_enqueue_script( 'wordlift-cloud' );
-		$json_faceted_id = wp_json_encode( $faceted_id );
-		$template_url = get_rest_url( null, '/wordlift/v1/faceted-search/template' );
-
-		return <<<HTML
+    /**
+     * {@inheritdoc}
+     */
+    const SHORTCODE = 'wl_faceted_search';
+
+    public function __construct() {
+        parent::__construct();
+        $this->register_block_type();
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function render( $atts ) {
+
+        return Wordlift_AMP_Service::is_amp_endpoint() ? $this->amp_shortcode( $atts )
+            : $this->web_shortcode( $atts );
+    }
+
+    private function register_block_type() {
+
+        $scope = $this;
+
+        add_action( 'init', function () use ( $scope ) {
+            if ( ! function_exists( 'register_block_type' ) ) {
+                // Gutenberg is not active.
+                return;
+            }
+
+            register_block_type( 'wordlift/faceted-search', array(
+                'editor_script'   => 'wl-block-editor',
+                'render_callback' => function ( $attributes ) use ( $scope ) {
+                    $attr_code = '';
+                    foreach ( $attributes as $key => $value ) {
+                        $attr_code .= $key . '="' . htmlentities( $value ) . '" ';
+                    }
+
+                    return '[' . $scope::SHORTCODE . ' ' . $attr_code . ']';
+                },
+                'attributes'      => array(
+                    'title'       => array(
+                        'type'    => 'string',
+                        'default' => __( 'Related articles', 'wordlift' ),
+                    ),
+                    'template_id' => array(
+                        'type' => 'string',
+                        'default' => '',
+                    ),
+                    'post_id'     => array(
+                        'type'    => 'number',
+                        'default' => '',
+                    ),
+                    'uniqid'      => array(
+                        'type'    => 'string',
+                        'default' => '',
+                    ),
+                    'limit'       => array(
+                        'type'    => 'number',
+                        'default' => apply_filters( 'wl_faceted_search_default_limit', 10 ),
+                    ),
+                    'preview'     => array(
+                        'type'    => 'boolean',
+                        'default' => false,
+                    ),
+                    'preview_src' => array(
+                        'type'    => 'string',
+                        'default' => WP_CONTENT_URL . '/plugins/wordlift/images/block-previews/faceted-search.png',
+                    ),
+                ),
+            ) );
+        } );
+    }
+
+    /**
+     * Shared function used by web_shortcode and amp_shortcode
+     * Bootstrap logic for attributes extraction and boolean filtering
+     *
+     * @since      3.20.0
+     *
+     * @param array $atts Shortcode attributes.
+     *
+     * @return array $shortcode_atts
+     */
+    private function make_shortcode_atts( $atts ) {
+
+        // Extract attributes and set default values.
+        $shortcode_atts = shortcode_atts( array(
+            'title'       => __( 'Related articles', 'wordlift' ),
+            'limit'       => apply_filters( 'wl_faceted_search_default_limit', 10 ),
+            'post_id'     => '',
+            'template_id' => '',
+            'uniqid'      => uniqid( 'wl-faceted-widget-' ),
+        ), $atts );
+
+        return $shortcode_atts;
+    }
+
+    /**
+     * Function in charge of diplaying the [wl-faceted-search] in web mode.
+     *
+     * @since 3.20.0
+     *
+     * @param array $atts Shortcode attributes.
+     *
+     * @return string Shortcode HTML for web
+     */
+    private function web_shortcode( $atts ) {
+
+        // attributes extraction and boolean filtering
+        $shortcode_atts = $this->make_shortcode_atts( $atts );
+
+        // avoid building the widget when no post_id is specified and there is a list of posts.
+        if ( empty( $shortcode_atts['post_id'] ) && ! is_singular() ) {
+            return;
+        }
+
+        $post        = ! empty( $shortcode_atts['post_id'] ) ? get_post( intval( sanitize_text_field( $shortcode_atts['post_id'] ) ) ) : get_post();
+        $title       = esc_attr( sanitize_text_field( $shortcode_atts['title'] ) );
+        $template_id = esc_attr( sanitize_text_field( $shortcode_atts['template_id'] ) );
+        $limit       = esc_attr( sanitize_text_field( $shortcode_atts['limit'] ) );
+        $faceted_id  = ! empty( $shortcode_atts['uniqid'] ) ? esc_attr( sanitize_text_field( $shortcode_atts['uniqid'] ) ) : uniqid( 'wl-faceted-widget-' );
+
+        $permalink_structure = get_option( 'permalink_structure' );
+        $delimiter           = empty( $permalink_structure ) ? '&' : '?';
+        $rest_url            = $post ? rest_url( WL_REST_ROUTE_DEFAULT_NAMESPACE . '/faceted-search' . $delimiter . build_query( array(
+                'post_id' => $post->ID,
+                'limit'   => $limit
+            ) ) ) : false;
+        $rest_url = esc_attr( $rest_url );
+
+        // avoid building the widget when no valid $rest_url
+        if ( ! $rest_url ) {
+            return;
+        }
+
+        wp_enqueue_script( 'wordlift-cloud' );
+        $json_faceted_id = wp_json_encode( $faceted_id );
+        $template_url = get_rest_url( null, '/wordlift/v1/faceted-search/template' );
+
+        return <<<HTML
 			<!-- Faceted {$faceted_id} -->
 			<script type="application/javascript">
 				window.wlFaceteds = window.wlFaceteds || []; wlFaceteds.push({$json_faceted_id});
@@ -169,58 +169,58 @@ 
 				 data-template-url="{$template_url}"></div>
 			<!-- /Faceted {$faceted_id} -->
 HTML;
-	}
-
-	/**
-	 * Function in charge of diplaying the [wl-faceted-search] in amp mode.
-	 *
-	 * @since 3.20.0
-	 *
-	 * @param array $atts Shortcode attributes.
-	 *
-	 * @return string Shortcode HTML for amp
-	 */
-	private function amp_shortcode( $atts ) {
-
-		// attributes extraction and boolean filtering
-		$shortcode_atts = $this->make_shortcode_atts( $atts );
-
-		// avoid building the widget when no post_id is specified and there is a list of posts.
-		if ( empty( $shortcode_atts['post_id'] ) && ! is_singular() ) {
-			return;
-		}
-
-		$post        = ! empty( $shortcode_atts['post_id'] ) ? get_post( intval( sanitize_text_field( $shortcode_atts['post_id'] ) ) ) : get_post();
-		$title       = esc_attr( sanitize_text_field( $shortcode_atts['title'] ) );
-		$template_id = esc_attr( sanitize_text_field( $shortcode_atts['template_id'] ) );
-		$limit       = esc_attr( sanitize_text_field( $shortcode_atts['limit'] ) );
-		$faceted_id  = ! empty( $shortcode_atts['uniqid'] ) ? esc_attr( sanitize_text_field( $shortcode_atts['uniqid'] ) ) : uniqid( 'wl-faceted-widget-' );
-
-		$permalink_structure = get_option( 'permalink_structure' );
-		$delimiter           = empty( $permalink_structure ) ? '&' : '?';
-		$rest_url            = $post ? rest_url( WL_REST_ROUTE_DEFAULT_NAMESPACE . '/faceted-search' . $delimiter . build_query( array(
-				'amp',
-				'post_id' => $post->ID,
-				'limit'   => $limit
-			) ) ) : false;
-
-		$rest_url = $post ? rest_url( sprintf( "wordlift/v1/faceted-search?amp&post_id=%s&limit=%s", $post->ID, $limit ) ) : false;
-
-		// avoid building the widget when no valid $rest_url
-		if ( ! $rest_url ) {
-			return;
-		}
-
-		// Use a protocol-relative URL as amp-list spec says that URL's protocol must be HTTPS.
-		// This is a hackish way, but this works for http and https URLs
-		$rest_url = str_replace( array( 'http:', 'https:' ), '', $rest_url );
-
-		if ( empty( $template_id ) ) {
-			$template_id = "template-" . $faceted_id;
-			wp_enqueue_style( 'wordlift-amp-custom', plugin_dir_url( dirname( __FILE__ ) ) . '/css/wordlift-amp-custom.min.css' );
-		}
-
-		return <<<HTML
+    }
+
+    /**
+     * Function in charge of diplaying the [wl-faceted-search] in amp mode.
+     *
+     * @since 3.20.0
+     *
+     * @param array $atts Shortcode attributes.
+     *
+     * @return string Shortcode HTML for amp
+     */
+    private function amp_shortcode( $atts ) {
+
+        // attributes extraction and boolean filtering
+        $shortcode_atts = $this->make_shortcode_atts( $atts );
+
+        // avoid building the widget when no post_id is specified and there is a list of posts.
+        if ( empty( $shortcode_atts['post_id'] ) && ! is_singular() ) {
+            return;
+        }
+
+        $post        = ! empty( $shortcode_atts['post_id'] ) ? get_post( intval( sanitize_text_field( $shortcode_atts['post_id'] ) ) ) : get_post();
+        $title       = esc_attr( sanitize_text_field( $shortcode_atts['title'] ) );
+        $template_id = esc_attr( sanitize_text_field( $shortcode_atts['template_id'] ) );
+        $limit       = esc_attr( sanitize_text_field( $shortcode_atts['limit'] ) );
+        $faceted_id  = ! empty( $shortcode_atts['uniqid'] ) ? esc_attr( sanitize_text_field( $shortcode_atts['uniqid'] ) ) : uniqid( 'wl-faceted-widget-' );
+
+        $permalink_structure = get_option( 'permalink_structure' );
+        $delimiter           = empty( $permalink_structure ) ? '&' : '?';
+        $rest_url            = $post ? rest_url( WL_REST_ROUTE_DEFAULT_NAMESPACE . '/faceted-search' . $delimiter . build_query( array(
+                'amp',
+                'post_id' => $post->ID,
+                'limit'   => $limit
+            ) ) ) : false;
+
+        $rest_url = $post ? rest_url( sprintf( "wordlift/v1/faceted-search?amp&post_id=%s&limit=%s", $post->ID, $limit ) ) : false;
+
+        // avoid building the widget when no valid $rest_url
+        if ( ! $rest_url ) {
+            return;
+        }
+
+        // Use a protocol-relative URL as amp-list spec says that URL's protocol must be HTTPS.
+        // This is a hackish way, but this works for http and https URLs
+        $rest_url = str_replace( array( 'http:', 'https:' ), '', $rest_url );
+
+        if ( empty( $template_id ) ) {
+            $template_id = "template-" . $faceted_id;
+            wp_enqueue_style( 'wordlift-amp-custom', plugin_dir_url( dirname( __FILE__ ) ) . '/css/wordlift-amp-custom.min.css' );
+        }
+
+        return <<<HTML
 		<div id="{$faceted_id}" class="wl-amp-faceted">
 			<h2 class="wl-headline">{$title}</h2>
 			<amp-state id="referencedPosts">
@@ -303,6 +303,6 @@ 
 			</section>
 		</div>
 HTML;
-	}
+    }
 
 }

Spacing +55 added lines, -55 removed lines

@@ -30,36 +30,36 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function render( $atts ) {
+	public function render($atts) {
 
-		return Wordlift_AMP_Service::is_amp_endpoint() ? $this->amp_shortcode( $atts )
-			: $this->web_shortcode( $atts );
+		return Wordlift_AMP_Service::is_amp_endpoint() ? $this->amp_shortcode($atts)
+			: $this->web_shortcode($atts);
 	}
 
 	private function register_block_type() {
 
 		$scope = $this;
 
-		add_action( 'init', function () use ( $scope ) {
-			if ( ! function_exists( 'register_block_type' ) ) {
+		add_action('init', function() use ($scope) {
+			if ( ! function_exists('register_block_type')) {
 				// Gutenberg is not active.
 				return;
 			}
 
-			register_block_type( 'wordlift/faceted-search', array(
+			register_block_type('wordlift/faceted-search', array(
 				'editor_script'   => 'wl-block-editor',
-				'render_callback' => function ( $attributes ) use ( $scope ) {
+				'render_callback' => function($attributes) use ($scope) {
 					$attr_code = '';
-					foreach ( $attributes as $key => $value ) {
-						$attr_code .= $key . '="' . htmlentities( $value ) . '" ';
+					foreach ($attributes as $key => $value) {
+						$attr_code .= $key.'="'.htmlentities($value).'" ';
 					}
 
-					return '[' . $scope::SHORTCODE . ' ' . $attr_code . ']';
+					return '['.$scope::SHORTCODE.' '.$attr_code.']';
 				},
 				'attributes'      => array(
 					'title'       => array(
 						'type'    => 'string',
-						'default' => __( 'Related articles', 'wordlift' ),
+						'default' => __('Related articles', 'wordlift'),
 					),
 					'template_id' => array(
 						'type' => 'string',
@@ -75,7 +75,7 @@ 
 					),
 					'limit'       => array(
 						'type'    => 'number',
-						'default' => apply_filters( 'wl_faceted_search_default_limit', 10 ),
+						'default' => apply_filters('wl_faceted_search_default_limit', 10),
 					),
 					'preview'     => array(
 						'type'    => 'boolean',
@@ -83,10 +83,10 @@ 
 					),
 					'preview_src' => array(
 						'type'    => 'string',
-						'default' => WP_CONTENT_URL . '/plugins/wordlift/images/block-previews/faceted-search.png',
+						'default' => WP_CONTENT_URL.'/plugins/wordlift/images/block-previews/faceted-search.png',
 					),
 				),
-			) );
+			));
 		} );
 	}
 
@@ -100,16 +100,16 @@ 
 	 *
 	 * @return array $shortcode_atts
 	 */
-	private function make_shortcode_atts( $atts ) {
+	private function make_shortcode_atts($atts) {
 
 		// Extract attributes and set default values.
-		$shortcode_atts = shortcode_atts( array(
-			'title'       => __( 'Related articles', 'wordlift' ),
-			'limit'       => apply_filters( 'wl_faceted_search_default_limit', 10 ),
+		$shortcode_atts = shortcode_atts(array(
+			'title'       => __('Related articles', 'wordlift'),
+			'limit'       => apply_filters('wl_faceted_search_default_limit', 10),
 			'post_id'     => '',
 			'template_id' => '',
-			'uniqid'      => uniqid( 'wl-faceted-widget-' ),
-		), $atts );
+			'uniqid'      => uniqid('wl-faceted-widget-'),
+		), $atts);
 
 		return $shortcode_atts;
 	}
@@ -123,38 +123,38 @@ 
 	 *
 	 * @return string Shortcode HTML for web
 	 */
-	private function web_shortcode( $atts ) {
+	private function web_shortcode($atts) {
 
 		// attributes extraction and boolean filtering
-		$shortcode_atts = $this->make_shortcode_atts( $atts );
+		$shortcode_atts = $this->make_shortcode_atts($atts);
 
 		// avoid building the widget when no post_id is specified and there is a list of posts.
-		if ( empty( $shortcode_atts['post_id'] ) && ! is_singular() ) {
+		if (empty($shortcode_atts['post_id']) && ! is_singular()) {
 			return;
 		}
 
-		$post        = ! empty( $shortcode_atts['post_id'] ) ? get_post( intval( sanitize_text_field( $shortcode_atts['post_id'] ) ) ) : get_post();
-		$title       = esc_attr( sanitize_text_field( $shortcode_atts['title'] ) );
-		$template_id = esc_attr( sanitize_text_field( $shortcode_atts['template_id'] ) );
-		$limit       = esc_attr( sanitize_text_field( $shortcode_atts['limit'] ) );
-		$faceted_id  = ! empty( $shortcode_atts['uniqid'] ) ? esc_attr( sanitize_text_field( $shortcode_atts['uniqid'] ) ) : uniqid( 'wl-faceted-widget-' );
+		$post        = ! empty($shortcode_atts['post_id']) ? get_post(intval(sanitize_text_field($shortcode_atts['post_id']))) : get_post();
+		$title       = esc_attr(sanitize_text_field($shortcode_atts['title']));
+		$template_id = esc_attr(sanitize_text_field($shortcode_atts['template_id']));
+		$limit       = esc_attr(sanitize_text_field($shortcode_atts['limit']));
+		$faceted_id  = ! empty($shortcode_atts['uniqid']) ? esc_attr(sanitize_text_field($shortcode_atts['uniqid'])) : uniqid('wl-faceted-widget-');
 
-		$permalink_structure = get_option( 'permalink_structure' );
-		$delimiter           = empty( $permalink_structure ) ? '&' : '?';
-		$rest_url            = $post ? rest_url( WL_REST_ROUTE_DEFAULT_NAMESPACE . '/faceted-search' . $delimiter . build_query( array(
+		$permalink_structure = get_option('permalink_structure');
+		$delimiter           = empty($permalink_structure) ? '&' : '?';
+		$rest_url            = $post ? rest_url(WL_REST_ROUTE_DEFAULT_NAMESPACE.'/faceted-search'.$delimiter.build_query(array(
 				'post_id' => $post->ID,
 				'limit'   => $limit
-			) ) ) : false;
-		$rest_url = esc_attr( $rest_url );
+			))) : false;
+		$rest_url = esc_attr($rest_url);
 
 		// avoid building the widget when no valid $rest_url
-		if ( ! $rest_url ) {
+		if ( ! $rest_url) {
 			return;
 		}
 
-		wp_enqueue_script( 'wordlift-cloud' );
-		$json_faceted_id = wp_json_encode( $faceted_id );
-		$template_url = get_rest_url( null, '/wordlift/v1/faceted-search/template' );
+		wp_enqueue_script('wordlift-cloud');
+		$json_faceted_id = wp_json_encode($faceted_id);
+		$template_url = get_rest_url(null, '/wordlift/v1/faceted-search/template');
 
 		return <<<HTML
 			<!-- Faceted {$faceted_id} -->
@@ -180,44 +180,44 @@ 
 	 *
 	 * @return string Shortcode HTML for amp
 	 */
-	private function amp_shortcode( $atts ) {
+	private function amp_shortcode($atts) {
 
 		// attributes extraction and boolean filtering
-		$shortcode_atts = $this->make_shortcode_atts( $atts );
+		$shortcode_atts = $this->make_shortcode_atts($atts);
 
 		// avoid building the widget when no post_id is specified and there is a list of posts.
-		if ( empty( $shortcode_atts['post_id'] ) && ! is_singular() ) {
+		if (empty($shortcode_atts['post_id']) && ! is_singular()) {
 			return;
 		}
 
-		$post        = ! empty( $shortcode_atts['post_id'] ) ? get_post( intval( sanitize_text_field( $shortcode_atts['post_id'] ) ) ) : get_post();
-		$title       = esc_attr( sanitize_text_field( $shortcode_atts['title'] ) );
-		$template_id = esc_attr( sanitize_text_field( $shortcode_atts['template_id'] ) );
-		$limit       = esc_attr( sanitize_text_field( $shortcode_atts['limit'] ) );
-		$faceted_id  = ! empty( $shortcode_atts['uniqid'] ) ? esc_attr( sanitize_text_field( $shortcode_atts['uniqid'] ) ) : uniqid( 'wl-faceted-widget-' );
+		$post        = ! empty($shortcode_atts['post_id']) ? get_post(intval(sanitize_text_field($shortcode_atts['post_id']))) : get_post();
+		$title       = esc_attr(sanitize_text_field($shortcode_atts['title']));
+		$template_id = esc_attr(sanitize_text_field($shortcode_atts['template_id']));
+		$limit       = esc_attr(sanitize_text_field($shortcode_atts['limit']));
+		$faceted_id  = ! empty($shortcode_atts['uniqid']) ? esc_attr(sanitize_text_field($shortcode_atts['uniqid'])) : uniqid('wl-faceted-widget-');
 
-		$permalink_structure = get_option( 'permalink_structure' );
-		$delimiter           = empty( $permalink_structure ) ? '&' : '?';
-		$rest_url            = $post ? rest_url( WL_REST_ROUTE_DEFAULT_NAMESPACE . '/faceted-search' . $delimiter . build_query( array(
+		$permalink_structure = get_option('permalink_structure');
+		$delimiter           = empty($permalink_structure) ? '&' : '?';
+		$rest_url            = $post ? rest_url(WL_REST_ROUTE_DEFAULT_NAMESPACE.'/faceted-search'.$delimiter.build_query(array(
 				'amp',
 				'post_id' => $post->ID,
 				'limit'   => $limit
-			) ) ) : false;
+			))) : false;
 
-		$rest_url = $post ? rest_url( sprintf( "wordlift/v1/faceted-search?amp&post_id=%s&limit=%s", $post->ID, $limit ) ) : false;
+		$rest_url = $post ? rest_url(sprintf("wordlift/v1/faceted-search?amp&post_id=%s&limit=%s", $post->ID, $limit)) : false;
 
 		// avoid building the widget when no valid $rest_url
-		if ( ! $rest_url ) {
+		if ( ! $rest_url) {
 			return;
 		}
 
 		// Use a protocol-relative URL as amp-list spec says that URL's protocol must be HTTPS.
 		// This is a hackish way, but this works for http and https URLs
-		$rest_url = str_replace( array( 'http:', 'https:' ), '', $rest_url );
+		$rest_url = str_replace(array('http:', 'https:'), '', $rest_url);
 
-		if ( empty( $template_id ) ) {
-			$template_id = "template-" . $faceted_id;
-			wp_enqueue_style( 'wordlift-amp-custom', plugin_dir_url( dirname( __FILE__ ) ) . '/css/wordlift-amp-custom.min.css' );
+		if (empty($template_id)) {
+			$template_id = "template-".$faceted_id;
+			wp_enqueue_style('wordlift-amp-custom', plugin_dir_url(dirname(__FILE__)).'/css/wordlift-amp-custom.min.css');
 		}
 
 		return <<<HTML