insideout10 / wordlift-plugin

src/includes/class-wordlift-content-filter-service.php

Indentation +294 added lines, -294 removed lines

@@ -18,302 +18,302 @@
  */
 class Wordlift_Content_Filter_Service {
 
-	/**
-	 * The pattern to find entities in text.
-	 *
-	 * @since 3.8.0
-	 */
-	const PATTERN = '/<(\\w+)[^<]*class="([^"]*)"\\sitemid=\"([^"]+)\"[^>]*>([^<]*)<\\/\\1>/i';
-
-	/**
-	 * A {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.8.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
-
-	/**
-	 * The {@link Wordlift_Configuration_Service} instance.
-	 *
-	 * @since  3.13.0
-	 * @access private
-	 * @var \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
-	 */
-	private $configuration_service;
-
-	/**
-	 * The `link by default` setting.
-	 *
-	 * @since  3.13.0
-	 * @access private
-	 * @var bool True if link by default is enabled otherwise false.
-	 */
-	private $is_link_by_default;
-
-	private $entity_post_ids_linked_from_post_content = array();
-
-	/**
-	 * The {@link Wordlift_Entity_Uri_Service} instance.
-	 *
-	 * @since  3.16.3
-	 * @access private
-	 * @var \Wordlift_Entity_Uri_Service $entity_uri_service The {@link Wordlift_Entity_Uri_Service} instance.
-	 */
-	private $entity_uri_service;
-
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since 3.16.0
-	 *
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * The {@link Wordlift_Content_Filter_Service} singleton instance.
-	 *
-	 * @since  3.14.2
-	 * @access private
-	 * @var \Wordlift_Content_Filter_Service $instance The {@link Wordlift_Content_Filter_Service} singleton instance.
-	 */
-	private static $instance;
-
-	/**
-	 * Create a {@link Wordlift_Content_Filter_Service} instance.
-	 *
-	 * @param \Wordlift_Entity_Service        $entity_service The {@link Wordlift_Entity_Service} instance.
-	 * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
-	 * @param \Wordlift_Entity_Uri_Service    $entity_uri_service The {@link Wordlift_Entity_Uri_Service} instance.
-	 *
-	 * @since 3.8.0
-	 *
-	 */
-	public function __construct( $entity_service, $configuration_service, $entity_uri_service ) {
-
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
-
-		$this->entity_service        = $entity_service;
-		$this->configuration_service = $configuration_service;
-		$this->entity_uri_service    = $entity_uri_service;
-
-		self::$instance = $this;
-
-	}
-
-	/**
-	 * Get the {@link Wordlift_Content_Filter_Service} singleton instance.
-	 *
-	 * @return \Wordlift_Content_Filter_Service The {@link Wordlift_Content_Filter_Service} singleton instance.
-	 * @since 3.14.2
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Mangle the content by adding links to the entity pages. This function is
-	 * hooked to the 'the_content' WP's filter.
-	 *
-	 * @param string $content The content being filtered.
-	 *
-	 * @return string The filtered content.
-	 * @since 3.8.0
-	 *
-	 */
-	public function the_content( $content ) {
-
-		$this->log->trace( "Filtering content [ " . ( is_singular() ? 'yes' : 'no' ) . " ]..." );
-
-		// Links should be added only on the front end and not for RSS.
-		if ( is_feed() ) {
-			return $content;
-		}
-
-		// Preload the `link by default` setting.
-		$this->is_link_by_default = $this->configuration_service->is_link_by_default();
-
-		// Reset the array of of entity post ids linked from the post content.
-		// This is used to avoid linking more the once the same post.
-		$this->entity_post_ids_linked_from_post_content = array();
-
-		// Preload URIs.
-		$matches = array();
-		preg_match_all( self::PATTERN, $content, $matches );
-
-		// Bail out if there are no URIs.
-		if ( empty( $matches[3] ) ) {
-			return $content;
-		}
-
-		// Preload the URIs.
-		$this->entity_uri_service->preload_uris( $matches[3] );
-
-		// Replace each match of the entity tag with the entity link. If an error
-		// occurs fail silently returning the original content.
-		$result = preg_replace_callback( self::PATTERN, array(
-			$this,
-			'link',
-		), $content ) ?: $content;
-
-		$this->entity_uri_service->reset_uris();
-
-		return $result;
-	}
-
-	/**
-	 * Get the entity match and replace it with a page link.
-	 *
-	 * @param array $matches An array of matches.
-	 *
-	 * @return string The replaced text with the link to the entity page.
-	 * @since 3.8.0
-	 *
-	 */
-	private function link( $matches ) {
-
-		// Get the entity itemid URI and label.
-		$css_class = $matches[2];
-		$uri       = $matches[3];
-		$label     = $matches[4];
-
-		// Get the entity post by URI.
-		$post = $this->entity_service->get_entity_post_by_uri( $uri );
-
-		// @todo: revise the `test-content-filter-service.php` before switching
-		// to the `entity_uri_service`. This is required, because the test injects
-		// itself as `entity_service` to mock the requests to get a post by
-		// entity uri.
-		//
-		// $post = $this->entity_uri_service->get_entity( $uri );
-
-		if ( null === $post ) {
-
-			// If the entity post is not found return the label w/o the markup
-			// around it.
-			//
-			// See https://github.com/insideout10/wordlift-plugin/issues/461.
-			return $label;
-		}
-
-		$no_link = - 1 < strpos( $css_class, 'wl-no-link' )
-		           // Do not link if already linked.
-		           || in_array( $post->ID, $this->entity_post_ids_linked_from_post_content );
-		$link    = - 1 < strpos( $css_class, 'wl-link' );
-
-		// Don't link if links are disabled and the entity is not link or the
-		// entity is do not link.
-		$dont_link = ( ! $this->is_link_by_default && ! $link ) || $no_link;
-
-		// Return the label if it's don't link.
-		if ( $dont_link ) {
-			return $label;
-		}
-
-		// Add the entity post id to the array of already linked entities, so that
-		// only the first entity occurrence is linked.
-		$this->entity_post_ids_linked_from_post_content[] = $post->ID;
-
-		// Get the link.
-		$href = Wordlift_Post_Adapter::get_production_permalink( $post->ID );
-
-		// Bail out if the `$href` has been reset.
-		if ( empty( $href ) ) {
-			return $label;
-		}
-
-		// Get an alternative title attribute.
-		$title_attribute = $this->get_title_attribute( $post->ID, $label );
-
-		// Return the link.
-		return "<a class='wl-entity-page-link' $title_attribute href='$href'>$label</a>";
-	}
-
-	/**
-	 * Get a `title` attribute with an alternative label for the link.
-	 *
-	 * If an alternative title isn't available an empty string is returned.
-	 *
-	 * @param int    $post_id The {@link WP_Post}'s id.
-	 * @param string $label The main link label.
-	 *
-	 * @return string A `title` attribute with an alternative label or an empty
-	 *                string if none available.
-	 * @since 3.15.0
-	 *
-	 */
-	private function get_title_attribute( $post_id, $label ) {
-
-		// Get an alternative title.
-		$title = $this->get_link_title( $post_id, $label );
-		if ( ! empty( $title ) ) {
-			return 'title="' . esc_attr( $title ) . '"';
-		}
-
-		return '';
-	}
-
-	/**
-	 * Get a string to be used as a title attribute in links to a post
-	 *
-	 * @param int    $post_id The post id of the post being linked.
-	 * @param string $ignore_label A label to ignore.
-	 *
-	 * @return string    The title to be used in the link. An empty string when
-	 *                    there is no alternative that is not the $ignore_label.
-	 * @since 3.15.0
-	 *
-	 */
-	function get_link_title( $post_id, $ignore_label ) {
-
-		// Get possible alternative labels we can select from.
-		$labels = $this->entity_service->get_alternative_labels( $post_id );
-
-		/*
+    /**
+     * The pattern to find entities in text.
+     *
+     * @since 3.8.0
+     */
+    const PATTERN = '/<(\\w+)[^<]*class="([^"]*)"\\sitemid=\"([^"]+)\"[^>]*>([^<]*)<\\/\\1>/i';
+
+    /**
+     * A {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.8.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
+
+    /**
+     * The {@link Wordlift_Configuration_Service} instance.
+     *
+     * @since  3.13.0
+     * @access private
+     * @var \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
+     */
+    private $configuration_service;
+
+    /**
+     * The `link by default` setting.
+     *
+     * @since  3.13.0
+     * @access private
+     * @var bool True if link by default is enabled otherwise false.
+     */
+    private $is_link_by_default;
+
+    private $entity_post_ids_linked_from_post_content = array();
+
+    /**
+     * The {@link Wordlift_Entity_Uri_Service} instance.
+     *
+     * @since  3.16.3
+     * @access private
+     * @var \Wordlift_Entity_Uri_Service $entity_uri_service The {@link Wordlift_Entity_Uri_Service} instance.
+     */
+    private $entity_uri_service;
+
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since 3.16.0
+     *
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * The {@link Wordlift_Content_Filter_Service} singleton instance.
+     *
+     * @since  3.14.2
+     * @access private
+     * @var \Wordlift_Content_Filter_Service $instance The {@link Wordlift_Content_Filter_Service} singleton instance.
+     */
+    private static $instance;
+
+    /**
+     * Create a {@link Wordlift_Content_Filter_Service} instance.
+     *
+     * @param \Wordlift_Entity_Service        $entity_service The {@link Wordlift_Entity_Service} instance.
+     * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
+     * @param \Wordlift_Entity_Uri_Service    $entity_uri_service The {@link Wordlift_Entity_Uri_Service} instance.
+     *
+     * @since 3.8.0
+     *
+     */
+    public function __construct( $entity_service, $configuration_service, $entity_uri_service ) {
+
+        $this->log = Wordlift_Log_Service::get_logger( get_class() );
+
+        $this->entity_service        = $entity_service;
+        $this->configuration_service = $configuration_service;
+        $this->entity_uri_service    = $entity_uri_service;
+
+        self::$instance = $this;
+
+    }
+
+    /**
+     * Get the {@link Wordlift_Content_Filter_Service} singleton instance.
+     *
+     * @return \Wordlift_Content_Filter_Service The {@link Wordlift_Content_Filter_Service} singleton instance.
+     * @since 3.14.2
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Mangle the content by adding links to the entity pages. This function is
+     * hooked to the 'the_content' WP's filter.
+     *
+     * @param string $content The content being filtered.
+     *
+     * @return string The filtered content.
+     * @since 3.8.0
+     *
+     */
+    public function the_content( $content ) {
+
+        $this->log->trace( "Filtering content [ " . ( is_singular() ? 'yes' : 'no' ) . " ]..." );
+
+        // Links should be added only on the front end and not for RSS.
+        if ( is_feed() ) {
+            return $content;
+        }
+
+        // Preload the `link by default` setting.
+        $this->is_link_by_default = $this->configuration_service->is_link_by_default();
+
+        // Reset the array of of entity post ids linked from the post content.
+        // This is used to avoid linking more the once the same post.
+        $this->entity_post_ids_linked_from_post_content = array();
+
+        // Preload URIs.
+        $matches = array();
+        preg_match_all( self::PATTERN, $content, $matches );
+
+        // Bail out if there are no URIs.
+        if ( empty( $matches[3] ) ) {
+            return $content;
+        }
+
+        // Preload the URIs.
+        $this->entity_uri_service->preload_uris( $matches[3] );
+
+        // Replace each match of the entity tag with the entity link. If an error
+        // occurs fail silently returning the original content.
+        $result = preg_replace_callback( self::PATTERN, array(
+            $this,
+            'link',
+        ), $content ) ?: $content;
+
+        $this->entity_uri_service->reset_uris();
+
+        return $result;
+    }
+
+    /**
+     * Get the entity match and replace it with a page link.
+     *
+     * @param array $matches An array of matches.
+     *
+     * @return string The replaced text with the link to the entity page.
+     * @since 3.8.0
+     *
+     */
+    private function link( $matches ) {
+
+        // Get the entity itemid URI and label.
+        $css_class = $matches[2];
+        $uri       = $matches[3];
+        $label     = $matches[4];
+
+        // Get the entity post by URI.
+        $post = $this->entity_service->get_entity_post_by_uri( $uri );
+
+        // @todo: revise the `test-content-filter-service.php` before switching
+        // to the `entity_uri_service`. This is required, because the test injects
+        // itself as `entity_service` to mock the requests to get a post by
+        // entity uri.
+        //
+        // $post = $this->entity_uri_service->get_entity( $uri );
+
+        if ( null === $post ) {
+
+            // If the entity post is not found return the label w/o the markup
+            // around it.
+            //
+            // See https://github.com/insideout10/wordlift-plugin/issues/461.
+            return $label;
+        }
+
+        $no_link = - 1 < strpos( $css_class, 'wl-no-link' )
+                    // Do not link if already linked.
+                   || in_array( $post->ID, $this->entity_post_ids_linked_from_post_content );
+        $link    = - 1 < strpos( $css_class, 'wl-link' );
+
+        // Don't link if links are disabled and the entity is not link or the
+        // entity is do not link.
+        $dont_link = ( ! $this->is_link_by_default && ! $link ) || $no_link;
+
+        // Return the label if it's don't link.
+        if ( $dont_link ) {
+            return $label;
+        }
+
+        // Add the entity post id to the array of already linked entities, so that
+        // only the first entity occurrence is linked.
+        $this->entity_post_ids_linked_from_post_content[] = $post->ID;
+
+        // Get the link.
+        $href = Wordlift_Post_Adapter::get_production_permalink( $post->ID );
+
+        // Bail out if the `$href` has been reset.
+        if ( empty( $href ) ) {
+            return $label;
+        }
+
+        // Get an alternative title attribute.
+        $title_attribute = $this->get_title_attribute( $post->ID, $label );
+
+        // Return the link.
+        return "<a class='wl-entity-page-link' $title_attribute href='$href'>$label</a>";
+    }
+
+    /**
+     * Get a `title` attribute with an alternative label for the link.
+     *
+     * If an alternative title isn't available an empty string is returned.
+     *
+     * @param int    $post_id The {@link WP_Post}'s id.
+     * @param string $label The main link label.
+     *
+     * @return string A `title` attribute with an alternative label or an empty
+     *                string if none available.
+     * @since 3.15.0
+     *
+     */
+    private function get_title_attribute( $post_id, $label ) {
+
+        // Get an alternative title.
+        $title = $this->get_link_title( $post_id, $label );
+        if ( ! empty( $title ) ) {
+            return 'title="' . esc_attr( $title ) . '"';
+        }
+
+        return '';
+    }
+
+    /**
+     * Get a string to be used as a title attribute in links to a post
+     *
+     * @param int    $post_id The post id of the post being linked.
+     * @param string $ignore_label A label to ignore.
+     *
+     * @return string    The title to be used in the link. An empty string when
+     *                    there is no alternative that is not the $ignore_label.
+     * @since 3.15.0
+     *
+     */
+    function get_link_title( $post_id, $ignore_label ) {
+
+        // Get possible alternative labels we can select from.
+        $labels = $this->entity_service->get_alternative_labels( $post_id );
+
+        /*
 		 * Since the original text might use an alternative label than the
 		 * Entity title, add the title itself which is not returned by the api.
 		 */
-		$labels[] = get_the_title( $post_id );
-
-		// Add some randomness to the label selection.
-		shuffle( $labels );
-
-		// Select the first label which is not to be ignored.
-		$title = '';
-		foreach ( $labels as $label ) {
-			if ( 0 !== strcasecmp( $label, $ignore_label ) ) {
-				$title = $label;
-				break;
-			}
-		}
-
-		return $title;
-	}
-
-	/**
-	 * Get the entity URIs (configured in the `itemid` attribute) contained in
-	 * the provided content.
-	 *
-	 * @param string $content The content.
-	 *
-	 * @return array An array of URIs.
-	 * @since 3.14.2
-	 *
-	 */
-	public function get_entity_uris( $content ) {
-
-		$matches = array();
-		preg_match_all( Wordlift_Content_Filter_Service::PATTERN, $content, $matches );
-
-		// We need to use `array_values` here in order to avoid further `json_encode`
-		// to turn it into an object (since if the 3rd match isn't found the index
-		// is not sequential.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/646.
-		return array_values( array_unique( $matches[3] ) );
-	}
+        $labels[] = get_the_title( $post_id );
+
+        // Add some randomness to the label selection.
+        shuffle( $labels );
+
+        // Select the first label which is not to be ignored.
+        $title = '';
+        foreach ( $labels as $label ) {
+            if ( 0 !== strcasecmp( $label, $ignore_label ) ) {
+                $title = $label;
+                break;
+            }
+        }
+
+        return $title;
+    }
+
+    /**
+     * Get the entity URIs (configured in the `itemid` attribute) contained in
+     * the provided content.
+     *
+     * @param string $content The content.
+     *
+     * @return array An array of URIs.
+     * @since 3.14.2
+     *
+     */
+    public function get_entity_uris( $content ) {
+
+        $matches = array();
+        preg_match_all( Wordlift_Content_Filter_Service::PATTERN, $content, $matches );
+
+        // We need to use `array_values` here in order to avoid further `json_encode`
+        // to turn it into an object (since if the 3rd match isn't found the index
+        // is not sequential.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/646.
+        return array_values( array_unique( $matches[3] ) );
+    }
 
 }

Spacing +34 added lines, -34 removed lines

@@ -91,9 +91,9 @@ 
 	 * @since 3.8.0
 	 *
 	 */
-	public function __construct( $entity_service, $configuration_service, $entity_uri_service ) {
+	public function __construct($entity_service, $configuration_service, $entity_uri_service) {
 
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
+		$this->log = Wordlift_Log_Service::get_logger(get_class());
 
 		$this->entity_service        = $entity_service;
 		$this->configuration_service = $configuration_service;
@@ -124,12 +124,12 @@ 
 	 * @since 3.8.0
 	 *
 	 */
-	public function the_content( $content ) {
+	public function the_content($content) {
 
-		$this->log->trace( "Filtering content [ " . ( is_singular() ? 'yes' : 'no' ) . " ]..." );
+		$this->log->trace("Filtering content [ ".(is_singular() ? 'yes' : 'no')." ]...");
 
 		// Links should be added only on the front end and not for RSS.
-		if ( is_feed() ) {
+		if (is_feed()) {
 			return $content;
 		}
 
@@ -142,22 +142,22 @@ 
 
 		// Preload URIs.
 		$matches = array();
-		preg_match_all( self::PATTERN, $content, $matches );
+		preg_match_all(self::PATTERN, $content, $matches);
 
 		// Bail out if there are no URIs.
-		if ( empty( $matches[3] ) ) {
+		if (empty($matches[3])) {
 			return $content;
 		}
 
 		// Preload the URIs.
-		$this->entity_uri_service->preload_uris( $matches[3] );
+		$this->entity_uri_service->preload_uris($matches[3]);
 
 		// Replace each match of the entity tag with the entity link. If an error
 		// occurs fail silently returning the original content.
-		$result = preg_replace_callback( self::PATTERN, array(
+		$result = preg_replace_callback(self::PATTERN, array(
 			$this,
 			'link',
-		), $content ) ?: $content;
+		), $content) ?: $content;
 
 		$this->entity_uri_service->reset_uris();
 
@@ -173,7 +173,7 @@ 
 	 * @since 3.8.0
 	 *
 	 */
-	private function link( $matches ) {
+	private function link($matches) {
 
 		// Get the entity itemid URI and label.
 		$css_class = $matches[2];
@@ -181,7 +181,7 @@ 
 		$label     = $matches[4];
 
 		// Get the entity post by URI.
-		$post = $this->entity_service->get_entity_post_by_uri( $uri );
+		$post = $this->entity_service->get_entity_post_by_uri($uri);
 
 		// @todo: revise the `test-content-filter-service.php` before switching
 		// to the `entity_uri_service`. This is required, because the test injects
@@ -190,7 +190,7 @@ 
 		//
 		// $post = $this->entity_uri_service->get_entity( $uri );
 
-		if ( null === $post ) {
+		if (null === $post) {
 
 			// If the entity post is not found return the label w/o the markup
 			// around it.
@@ -199,17 +199,17 @@ 
 			return $label;
 		}
 
-		$no_link = - 1 < strpos( $css_class, 'wl-no-link' )
+		$no_link = - 1 < strpos($css_class, 'wl-no-link')
 		           // Do not link if already linked.
-		           || in_array( $post->ID, $this->entity_post_ids_linked_from_post_content );
-		$link    = - 1 < strpos( $css_class, 'wl-link' );
+		           || in_array($post->ID, $this->entity_post_ids_linked_from_post_content);
+		$link    = - 1 < strpos($css_class, 'wl-link');
 
 		// Don't link if links are disabled and the entity is not link or the
 		// entity is do not link.
-		$dont_link = ( ! $this->is_link_by_default && ! $link ) || $no_link;
+		$dont_link = ( ! $this->is_link_by_default && ! $link) || $no_link;
 
 		// Return the label if it's don't link.
-		if ( $dont_link ) {
+		if ($dont_link) {
 			return $label;
 		}
 
@@ -218,15 +218,15 @@ 
 		$this->entity_post_ids_linked_from_post_content[] = $post->ID;
 
 		// Get the link.
-		$href = Wordlift_Post_Adapter::get_production_permalink( $post->ID );
+		$href = Wordlift_Post_Adapter::get_production_permalink($post->ID);
 
 		// Bail out if the `$href` has been reset.
-		if ( empty( $href ) ) {
+		if (empty($href)) {
 			return $label;
 		}
 
 		// Get an alternative title attribute.
-		$title_attribute = $this->get_title_attribute( $post->ID, $label );
+		$title_attribute = $this->get_title_attribute($post->ID, $label);
 
 		// Return the link.
 		return "<a class='wl-entity-page-link' $title_attribute href='$href'>$label</a>";
@@ -245,12 +245,12 @@ 
 	 * @since 3.15.0
 	 *
 	 */
-	private function get_title_attribute( $post_id, $label ) {
+	private function get_title_attribute($post_id, $label) {
 
 		// Get an alternative title.
-		$title = $this->get_link_title( $post_id, $label );
-		if ( ! empty( $title ) ) {
-			return 'title="' . esc_attr( $title ) . '"';
+		$title = $this->get_link_title($post_id, $label);
+		if ( ! empty($title)) {
+			return 'title="'.esc_attr($title).'"';
 		}
 
 		return '';
@@ -267,24 +267,24 @@ 
 	 * @since 3.15.0
 	 *
 	 */
-	function get_link_title( $post_id, $ignore_label ) {
+	function get_link_title($post_id, $ignore_label) {
 
 		// Get possible alternative labels we can select from.
-		$labels = $this->entity_service->get_alternative_labels( $post_id );
+		$labels = $this->entity_service->get_alternative_labels($post_id);
 
 		/*
 		 * Since the original text might use an alternative label than the
 		 * Entity title, add the title itself which is not returned by the api.
 		 */
-		$labels[] = get_the_title( $post_id );
+		$labels[] = get_the_title($post_id);
 
 		// Add some randomness to the label selection.
-		shuffle( $labels );
+		shuffle($labels);
 
 		// Select the first label which is not to be ignored.
 		$title = '';
-		foreach ( $labels as $label ) {
-			if ( 0 !== strcasecmp( $label, $ignore_label ) ) {
+		foreach ($labels as $label) {
+			if (0 !== strcasecmp($label, $ignore_label)) {
 				$title = $label;
 				break;
 			}
@@ -303,17 +303,17 @@ 
 	 * @since 3.14.2
 	 *
 	 */
-	public function get_entity_uris( $content ) {
+	public function get_entity_uris($content) {
 
 		$matches = array();
-		preg_match_all( Wordlift_Content_Filter_Service::PATTERN, $content, $matches );
+		preg_match_all(Wordlift_Content_Filter_Service::PATTERN, $content, $matches);
 
 		// We need to use `array_values` here in order to avoid further `json_encode`
 		// to turn it into an object (since if the 3rd match isn't found the index
 		// is not sequential.
 		//
 		// See https://github.com/insideout10/wordlift-plugin/issues/646.
-		return array_values( array_unique( $matches[3] ) );
+		return array_values(array_unique($matches[3]));
 	}
 
 }

src/cli/class-wordlift-push-reference-data-command.php

Indentation +144 added lines, -144 removed lines

@@ -16,131 +16,131 @@ 
  */
 class Wordlift_Push_Reference_Data_Command {
 
-	/**
-	 * The {@link Wordlift_Relation_Service} instance.
-	 *
-	 * @since  3.18.0
-	 * @access private
-	 * @var \Wordlift_Relation_Service $relation_service The {@link Wordlift_Relation_Service} instance.
-	 */
-	private $relation_service;
-
-	/**
-	 * The {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.18.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
-
-	/**
-	 * The {@link Wordlift_Sparql_Service} instance.
-	 *
-	 * @since  3.18.0
-	 * @access private
-	 * @var \Wordlift_Sparql_Service $sparql_service The {@link Wordlift_Sparql_Service} instance.
-	 */
-	private $sparql_service;
-
-	/**
-	 * The {@link Wordlift_Configuration_Service} instance.
-	 *
-	 * @since  3.18.0
-	 * @access private
-	 * @var \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
-	 */
-	private $configuration_service;
-	/**
-	 * @var Wordlift_Entity_Type_Service
-	 */
-	private $entity_type_service;
-
-	/**
-	 * Wordlift_Push_Reference_Data_Command constructor.
-	 *
-	 * @param \Wordlift_Relation_Service      $relation_service The {@link Wordlift_Relation_Service} instance.
-	 * @param \Wordlift_Entity_Service        $entity_service The {@link Wordlift_Entity_Service} instance.
-	 * @param \Wordlift_Sparql_Service        $sparql_service The {@link Wordlift_Sparql_Service} instance.
-	 * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
-	 * @param \Wordlift_Entity_Type_Service   $entity_type_service The {@link Wordlift_Entity_Type_Service} instance.
-	 *
-	 * @since 3.18.0
-	 *
-	 */
-	public function __construct( $relation_service, $entity_service, $sparql_service, $configuration_service, $entity_type_service ) {
-
-		$this->relation_service      = $relation_service;
-		$this->entity_service        = $entity_service;
-		$this->sparql_service        = $sparql_service;
-		$this->configuration_service = $configuration_service;
-		$this->entity_type_service   = $entity_type_service;
-
-	}
-
-	public function __invoke( $args ) {
-
-		$relations    = $this->relation_service->find_all_grouped_by_subject_id();
-		$progress_bar = \WP_CLI\Utils\make_progress_bar( 'Processing...', count( $relations ) );
-
-		foreach ( $relations as $relation ) {
-			$progress_bar->tick();
-
-			// Get the post.
-			$post = get_post( $relation->subject_id );
-
-			// Bail out if the post isn't found.
-			if ( null == $post ) {
-				continue;
-			}
-
-			// Bail out if it's an entity: we're only interested in articles
-			// *referencing* entities.
-			if ( $this->entity_service->is_entity( $post->ID ) ) {
-				continue;
-			}
-
-			// Get the article URI.
-			$uri = $this->entity_service->get_uri( $post->ID );
-
-			// Prepare the DELETE query to delete existing data.
-			$query = self::get_delete_query( $uri )
-			         . $this->get_insert_query( $post, $uri, explode( ',', $relation->object_ids ) );
-
-			$this->sparql_service->execute( $query, false );
-
-		}
-
-		$progress_bar->finish();
-
-	}
-
-	private static function get_delete_query( $uri ) {
-
-		return Wordlift_Query_Builder
-			       ::new_instance()
-			       ->delete()->statement( $uri, Wordlift_Query_Builder::DCTERMS_REFERENCES_URI, '?o' )
-			       ->build()
-		       . Wordlift_Query_Builder
-			       ::new_instance()
-			       ->delete()->statement( $uri, Wordlift_Query_Builder::RDFS_TYPE_URI, '?o' )
-			       ->build()
-		       . Wordlift_Query_Builder
-			       ::new_instance()
-			       ->delete()->statement( $uri, Wordlift_Query_Builder::SCHEMA_HEADLINE_URI, '?o' )
-			       ->build()
-		       . Wordlift_Query_Builder
-			       ::new_instance()
-			       ->delete()->statement( $uri, Wordlift_Query_Builder::SCHEMA_URL_URI, '?o' )
-			       ->build();
-	}
-
-	private function get_insert_query( $post, $uri, $object_ids ) {
-
-		$language_code = $this->configuration_service->get_language_code();
-		$type          = $this->entity_type_service->get( $post->ID );
-
-		/*
+    /**
+     * The {@link Wordlift_Relation_Service} instance.
+     *
+     * @since  3.18.0
+     * @access private
+     * @var \Wordlift_Relation_Service $relation_service The {@link Wordlift_Relation_Service} instance.
+     */
+    private $relation_service;
+
+    /**
+     * The {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.18.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
+
+    /**
+     * The {@link Wordlift_Sparql_Service} instance.
+     *
+     * @since  3.18.0
+     * @access private
+     * @var \Wordlift_Sparql_Service $sparql_service The {@link Wordlift_Sparql_Service} instance.
+     */
+    private $sparql_service;
+
+    /**
+     * The {@link Wordlift_Configuration_Service} instance.
+     *
+     * @since  3.18.0
+     * @access private
+     * @var \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
+     */
+    private $configuration_service;
+    /**
+     * @var Wordlift_Entity_Type_Service
+     */
+    private $entity_type_service;
+
+    /**
+     * Wordlift_Push_Reference_Data_Command constructor.
+     *
+     * @param \Wordlift_Relation_Service      $relation_service The {@link Wordlift_Relation_Service} instance.
+     * @param \Wordlift_Entity_Service        $entity_service The {@link Wordlift_Entity_Service} instance.
+     * @param \Wordlift_Sparql_Service        $sparql_service The {@link Wordlift_Sparql_Service} instance.
+     * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
+     * @param \Wordlift_Entity_Type_Service   $entity_type_service The {@link Wordlift_Entity_Type_Service} instance.
+     *
+     * @since 3.18.0
+     *
+     */
+    public function __construct( $relation_service, $entity_service, $sparql_service, $configuration_service, $entity_type_service ) {
+
+        $this->relation_service      = $relation_service;
+        $this->entity_service        = $entity_service;
+        $this->sparql_service        = $sparql_service;
+        $this->configuration_service = $configuration_service;
+        $this->entity_type_service   = $entity_type_service;
+
+    }
+
+    public function __invoke( $args ) {
+
+        $relations    = $this->relation_service->find_all_grouped_by_subject_id();
+        $progress_bar = \WP_CLI\Utils\make_progress_bar( 'Processing...', count( $relations ) );
+
+        foreach ( $relations as $relation ) {
+            $progress_bar->tick();
+
+            // Get the post.
+            $post = get_post( $relation->subject_id );
+
+            // Bail out if the post isn't found.
+            if ( null == $post ) {
+                continue;
+            }
+
+            // Bail out if it's an entity: we're only interested in articles
+            // *referencing* entities.
+            if ( $this->entity_service->is_entity( $post->ID ) ) {
+                continue;
+            }
+
+            // Get the article URI.
+            $uri = $this->entity_service->get_uri( $post->ID );
+
+            // Prepare the DELETE query to delete existing data.
+            $query = self::get_delete_query( $uri )
+                        . $this->get_insert_query( $post, $uri, explode( ',', $relation->object_ids ) );
+
+            $this->sparql_service->execute( $query, false );
+
+        }
+
+        $progress_bar->finish();
+
+    }
+
+    private static function get_delete_query( $uri ) {
+
+        return Wordlift_Query_Builder
+                    ::new_instance()
+                    ->delete()->statement( $uri, Wordlift_Query_Builder::DCTERMS_REFERENCES_URI, '?o' )
+                    ->build()
+                . Wordlift_Query_Builder
+                    ::new_instance()
+                    ->delete()->statement( $uri, Wordlift_Query_Builder::RDFS_TYPE_URI, '?o' )
+                    ->build()
+                . Wordlift_Query_Builder
+                    ::new_instance()
+                    ->delete()->statement( $uri, Wordlift_Query_Builder::SCHEMA_HEADLINE_URI, '?o' )
+                    ->build()
+                . Wordlift_Query_Builder
+                    ::new_instance()
+                    ->delete()->statement( $uri, Wordlift_Query_Builder::SCHEMA_URL_URI, '?o' )
+                    ->build();
+    }
+
+    private function get_insert_query( $post, $uri, $object_ids ) {
+
+        $language_code = $this->configuration_service->get_language_code();
+        $type          = $this->entity_type_service->get( $post->ID );
+
+        /*
 		 * When inserting the schema:url property in the triple store, we want to use the production
 		 * URL, i.e. we must take into consideration that the current URL is a staging one and that
 		 * 3rd parties may want to update the URL with a production one.
@@ -149,24 +149,24 @@ 
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/850.
 		 */
-		$builder = Wordlift_Query_Builder
-			::new_instance()
-			->insert()
-			->statement( $uri, Wordlift_Query_Builder::SCHEMA_HEADLINE_URI, $post->post_title, Wordlift_Query_Builder::OBJECT_VALUE, null, $language_code )
-			->statement( $uri, Wordlift_Query_Builder::RDFS_TYPE_URI, $type['uri'] );
-
-		$permalink = Wordlift_Post_Adapter::get_production_permalink( $post->ID );
-		if ( ! empty( $permalink ) ) {
-			$builder->statement( $uri, Wordlift_Query_Builder::SCHEMA_URL_URI, $permalink );
-		}
-
-		$entity_service = $this->entity_service;
-		array_walk( $object_ids, function ( $item ) use ( $entity_service, $builder, $uri ) {
-			$object_uri = $entity_service->get_uri( $item );
-			$builder->statement( $uri, Wordlift_Query_Builder::DCTERMS_REFERENCES_URI, $object_uri );
-		} );
-
-		return $builder->build();
-	}
+        $builder = Wordlift_Query_Builder
+            ::new_instance()
+            ->insert()
+            ->statement( $uri, Wordlift_Query_Builder::SCHEMA_HEADLINE_URI, $post->post_title, Wordlift_Query_Builder::OBJECT_VALUE, null, $language_code )
+            ->statement( $uri, Wordlift_Query_Builder::RDFS_TYPE_URI, $type['uri'] );
+
+        $permalink = Wordlift_Post_Adapter::get_production_permalink( $post->ID );
+        if ( ! empty( $permalink ) ) {
+            $builder->statement( $uri, Wordlift_Query_Builder::SCHEMA_URL_URI, $permalink );
+        }
+
+        $entity_service = $this->entity_service;
+        array_walk( $object_ids, function ( $item ) use ( $entity_service, $builder, $uri ) {
+            $object_uri = $entity_service->get_uri( $item );
+            $builder->statement( $uri, Wordlift_Query_Builder::DCTERMS_REFERENCES_URI, $object_uri );
+        } );
+
+        return $builder->build();
+    }
 
 }

Spacing +26 added lines, -26 removed lines

@@ -68,7 +68,7 @@ 
 	 * @since 3.18.0
 	 *
 	 */
-	public function __construct( $relation_service, $entity_service, $sparql_service, $configuration_service, $entity_type_service ) {
+	public function __construct($relation_service, $entity_service, $sparql_service, $configuration_service, $entity_type_service) {
 
 		$this->relation_service      = $relation_service;
 		$this->entity_service        = $entity_service;
@@ -78,36 +78,36 @@ 
 
 	}
 
-	public function __invoke( $args ) {
+	public function __invoke($args) {
 
 		$relations    = $this->relation_service->find_all_grouped_by_subject_id();
-		$progress_bar = \WP_CLI\Utils\make_progress_bar( 'Processing...', count( $relations ) );
+		$progress_bar = \WP_CLI\Utils\make_progress_bar('Processing...', count($relations));
 
-		foreach ( $relations as $relation ) {
+		foreach ($relations as $relation) {
 			$progress_bar->tick();
 
 			// Get the post.
-			$post = get_post( $relation->subject_id );
+			$post = get_post($relation->subject_id);
 
 			// Bail out if the post isn't found.
-			if ( null == $post ) {
+			if (null == $post) {
 				continue;
 			}
 
 			// Bail out if it's an entity: we're only interested in articles
 			// *referencing* entities.
-			if ( $this->entity_service->is_entity( $post->ID ) ) {
+			if ($this->entity_service->is_entity($post->ID)) {
 				continue;
 			}
 
 			// Get the article URI.
-			$uri = $this->entity_service->get_uri( $post->ID );
+			$uri = $this->entity_service->get_uri($post->ID);
 
 			// Prepare the DELETE query to delete existing data.
-			$query = self::get_delete_query( $uri )
-			         . $this->get_insert_query( $post, $uri, explode( ',', $relation->object_ids ) );
+			$query = self::get_delete_query($uri)
+			         . $this->get_insert_query($post, $uri, explode(',', $relation->object_ids));
 
-			$this->sparql_service->execute( $query, false );
+			$this->sparql_service->execute($query, false);
 
 		}
 
@@ -115,30 +115,30 @@ 
 
 	}
 
-	private static function get_delete_query( $uri ) {
+	private static function get_delete_query($uri) {
 
 		return Wordlift_Query_Builder
 			       ::new_instance()
-			       ->delete()->statement( $uri, Wordlift_Query_Builder::DCTERMS_REFERENCES_URI, '?o' )
+			       ->delete()->statement($uri, Wordlift_Query_Builder::DCTERMS_REFERENCES_URI, '?o')
 			       ->build()
 		       . Wordlift_Query_Builder
 			       ::new_instance()
-			       ->delete()->statement( $uri, Wordlift_Query_Builder::RDFS_TYPE_URI, '?o' )
+			       ->delete()->statement($uri, Wordlift_Query_Builder::RDFS_TYPE_URI, '?o')
 			       ->build()
 		       . Wordlift_Query_Builder
 			       ::new_instance()
-			       ->delete()->statement( $uri, Wordlift_Query_Builder::SCHEMA_HEADLINE_URI, '?o' )
+			       ->delete()->statement($uri, Wordlift_Query_Builder::SCHEMA_HEADLINE_URI, '?o')
 			       ->build()
 		       . Wordlift_Query_Builder
 			       ::new_instance()
-			       ->delete()->statement( $uri, Wordlift_Query_Builder::SCHEMA_URL_URI, '?o' )
+			       ->delete()->statement($uri, Wordlift_Query_Builder::SCHEMA_URL_URI, '?o')
 			       ->build();
 	}
 
-	private function get_insert_query( $post, $uri, $object_ids ) {
+	private function get_insert_query($post, $uri, $object_ids) {
 
 		$language_code = $this->configuration_service->get_language_code();
-		$type          = $this->entity_type_service->get( $post->ID );
+		$type          = $this->entity_type_service->get($post->ID);
 
 		/*
 		 * When inserting the schema:url property in the triple store, we want to use the production
@@ -152,18 +152,18 @@ 
 		$builder = Wordlift_Query_Builder
 			::new_instance()
 			->insert()
-			->statement( $uri, Wordlift_Query_Builder::SCHEMA_HEADLINE_URI, $post->post_title, Wordlift_Query_Builder::OBJECT_VALUE, null, $language_code )
-			->statement( $uri, Wordlift_Query_Builder::RDFS_TYPE_URI, $type['uri'] );
+			->statement($uri, Wordlift_Query_Builder::SCHEMA_HEADLINE_URI, $post->post_title, Wordlift_Query_Builder::OBJECT_VALUE, null, $language_code)
+			->statement($uri, Wordlift_Query_Builder::RDFS_TYPE_URI, $type['uri']);
 
-		$permalink = Wordlift_Post_Adapter::get_production_permalink( $post->ID );
-		if ( ! empty( $permalink ) ) {
-			$builder->statement( $uri, Wordlift_Query_Builder::SCHEMA_URL_URI, $permalink );
+		$permalink = Wordlift_Post_Adapter::get_production_permalink($post->ID);
+		if ( ! empty($permalink)) {
+			$builder->statement($uri, Wordlift_Query_Builder::SCHEMA_URL_URI, $permalink);
 		}
 
 		$entity_service = $this->entity_service;
-		array_walk( $object_ids, function ( $item ) use ( $entity_service, $builder, $uri ) {
-			$object_uri = $entity_service->get_uri( $item );
-			$builder->statement( $uri, Wordlift_Query_Builder::DCTERMS_REFERENCES_URI, $object_uri );
+		array_walk($object_ids, function($item) use ($entity_service, $builder, $uri) {
+			$object_uri = $entity_service->get_uri($item);
+			$builder->statement($uri, Wordlift_Query_Builder::DCTERMS_REFERENCES_URI, $object_uri);
 		} );
 
 		return $builder->build();