insideout10 / wordlift-plugin

src/wordlift/content/wordpress/class-wordpress-content.php

Indentation +85 added lines, -85 removed lines

@@ -10,89 +10,89 @@
 
 class Wordpress_Content implements Content {
 
-	/**
-	 * The actual content.
-	 *
-	 * @var WP_Post|WP_Term|WP_User $bag
-	 */
-	private $bag;
-
-	/**
-	 * Creates a WordPress content instance.
-	 *
-	 * @param WP_Post|WP_Term|WP_User $bag
-	 */
-	public function __construct( $bag ) {
-		$this->bag = $bag;
-	}
-
-	/**
-	 * The actual content.
-	 *
-	 * @return WP_Post|WP_Term|WP_User
-	 */
-	function get_bag() {
-		return $this->bag;
-	}
-
-	function get_id() {
-		if ( ! is_object( $this->bag ) ) {
-			return null;
-		}
-
-		switch ( get_class( $this->bag ) ) {
-			case 'WP_Post':
-			case 'WP_User':
-				return $this->bag->ID;
-			case 'WP_Term':
-				return $this->bag->term_id;
-		}
-
-		return null;
-	}
-
-	function get_object_type_enum() {
-		if ( ! is_object( $this->bag ) ) {
-			return null;
-		}
-
-		return Object_Type_Enum::from_wordpress_instance( $this->bag );
-	}
-
-	function get_permalink() {
-		if ( ! is_object( $this->bag ) ) {
-			return null;
-		}
-
-		switch ( get_class( $this->bag ) ) {
-			case 'WP_Post':
-				return get_permalink( $this->get_bag()->ID );
-			case 'WP_User':
-				return get_author_posts_url( $this->get_bag()->ID );
-			case 'WP_Term':
-				return get_term_link( $this->bag->term_id );
-		}
-
-		return null;
-	}
-
-	function get_edit_link() {
-		if ( ! is_object( $this->bag ) ) {
-			return null;
-		}
-
-		switch ( get_class( $this->bag ) ) {
-			case 'WP_Post':
-				// We need to return & character as &, by default context is set to display.
-				// so &  will be returned as & breaking header location redirects.
-				// By setting context to none we prevent this issue.
-				return get_edit_post_link( $this->get_bag()->ID, 'none' );
-			case 'WP_User':
-				return get_edit_user_link( $this->get_bag()->ID );
-			case 'WP_Term':
-				return get_edit_term_link( $this->bag->term_id );
-		}
-
-		return null;
-	}
+    /**
+     * The actual content.
+     *
+     * @var WP_Post|WP_Term|WP_User $bag
+     */
+    private $bag;
+
+    /**
+     * Creates a WordPress content instance.
+     *
+     * @param WP_Post|WP_Term|WP_User $bag
+     */
+    public function __construct( $bag ) {
+        $this->bag = $bag;
+    }
+
+    /**
+     * The actual content.
+     *
+     * @return WP_Post|WP_Term|WP_User
+     */
+    function get_bag() {
+        return $this->bag;
+    }
+
+    function get_id() {
+        if ( ! is_object( $this->bag ) ) {
+            return null;
+        }
+
+        switch ( get_class( $this->bag ) ) {
+            case 'WP_Post':
+            case 'WP_User':
+                return $this->bag->ID;
+            case 'WP_Term':
+                return $this->bag->term_id;
+        }
+
+        return null;
+    }
+
+    function get_object_type_enum() {
+        if ( ! is_object( $this->bag ) ) {
+            return null;
+        }
+
+        return Object_Type_Enum::from_wordpress_instance( $this->bag );
+    }
+
+    function get_permalink() {
+        if ( ! is_object( $this->bag ) ) {
+            return null;
+        }
+
+        switch ( get_class( $this->bag ) ) {
+            case 'WP_Post':
+                return get_permalink( $this->get_bag()->ID );
+            case 'WP_User':
+                return get_author_posts_url( $this->get_bag()->ID );
+            case 'WP_Term':
+                return get_term_link( $this->bag->term_id );
+        }
+
+        return null;
+    }
+
+    function get_edit_link() {
+        if ( ! is_object( $this->bag ) ) {
+            return null;
+        }
+
+        switch ( get_class( $this->bag ) ) {
+            case 'WP_Post':
+                // We need to return & character as &, by default context is set to display.
+                // so &  will be returned as & breaking header location redirects.
+                // By setting context to none we prevent this issue.
+                return get_edit_post_link( $this->get_bag()->ID, 'none' );
+            case 'WP_User':
+                return get_edit_user_link( $this->get_bag()->ID );
+            case 'WP_Term':
+                return get_edit_term_link( $this->bag->term_id );
+        }
+
+        return null;
+    }
 }
\ No newline at end of file

Spacing +15 added lines, -15 removed lines

@@ -22,7 +22,7 @@ 
 	 *
 	 * @param WP_Post|WP_Term|WP_User $bag
 	 */
-	public function __construct( $bag ) {
+	public function __construct($bag) {
 		$this->bag = $bag;
 	}
 
@@ -36,11 +36,11 @@ 
 	}
 
 	function get_id() {
-		if ( ! is_object( $this->bag ) ) {
+		if ( ! is_object($this->bag)) {
 			return null;
 		}
 
-		switch ( get_class( $this->bag ) ) {
+		switch (get_class($this->bag)) {
 			case 'WP_Post':
 			case 'WP_User':
 				return $this->bag->ID;
@@ -52,45 +52,45 @@ 
 	}
 
 	function get_object_type_enum() {
-		if ( ! is_object( $this->bag ) ) {
+		if ( ! is_object($this->bag)) {
 			return null;
 		}
 
-		return Object_Type_Enum::from_wordpress_instance( $this->bag );
+		return Object_Type_Enum::from_wordpress_instance($this->bag);
 	}
 
 	function get_permalink() {
-		if ( ! is_object( $this->bag ) ) {
+		if ( ! is_object($this->bag)) {
 			return null;
 		}
 
-		switch ( get_class( $this->bag ) ) {
+		switch (get_class($this->bag)) {
 			case 'WP_Post':
-				return get_permalink( $this->get_bag()->ID );
+				return get_permalink($this->get_bag()->ID);
 			case 'WP_User':
-				return get_author_posts_url( $this->get_bag()->ID );
+				return get_author_posts_url($this->get_bag()->ID);
 			case 'WP_Term':
-				return get_term_link( $this->bag->term_id );
+				return get_term_link($this->bag->term_id);
 		}
 
 		return null;
 	}
 
 	function get_edit_link() {
-		if ( ! is_object( $this->bag ) ) {
+		if ( ! is_object($this->bag)) {
 			return null;
 		}
 
-		switch ( get_class( $this->bag ) ) {
+		switch (get_class($this->bag)) {
 			case 'WP_Post':
 				// We need to return & character as &, by default context is set to display.
 				// so &  will be returned as & breaking header location redirects.
 				// By setting context to none we prevent this issue.
-				return get_edit_post_link( $this->get_bag()->ID, 'none' );
+				return get_edit_post_link($this->get_bag()->ID, 'none');
 			case 'WP_User':
-				return get_edit_user_link( $this->get_bag()->ID );
+				return get_edit_user_link($this->get_bag()->ID);
 			case 'WP_Term':
-				return get_edit_term_link( $this->bag->term_id );
+				return get_edit_term_link($this->bag->term_id);
 		}
 
 		return null;

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

Indentation +256 added lines, -256 removed lines

@@ -23,261 +23,261 @@
  */
 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 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 $linked_entity_uris = 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;
-	/**
-	 * @var Object_Link_Provider
-	 */
-	private $object_link_provider;
-
-	/**
-	 * Create a {@link Wordlift_Content_Filter_Service} instance.
-	 *
-	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 * @param \Wordlift_Entity_Uri_Service $entity_uri_service The {@link Wordlift_Entity_Uri_Service} instance.
-	 *
-	 * @since 3.8.0
-	 *
-	 */
-	protected function __construct( $entity_service, $entity_uri_service ) {
-
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
-
-		$this->entity_service       = $entity_service;
-		$this->entity_uri_service   = $entity_uri_service;
-		$this->object_link_provider = Object_Link_Provider::get_instance();
-
-	}
-
-	private static $instance = null;
-
-	/**
-	 * 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() {
-
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self( Wordlift_Entity_Service::get_instance(), Wordlift_Entity_Uri_Service::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() || is_admin() || is_search() ) {
-			return $content;
-		}
-
-		// Preload the `link by default` setting.
-		$this->is_link_by_default = Wordlift_Configuration_Service::get_instance()->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->linked_entity_uris = 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;
-		}
-
-		// Replace each match of the entity tag with the entity link. If an error
-		// occurs fail silently returning the original content.
-		return preg_replace_callback( self::PATTERN, array(
-			$this,
-			'link',
-		), $content ) ?: $content;
-	}
-
-	/**
-	 * 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];
-
-
-		/**
-		 * If the entity is already linked, dont send query to the db.
-		 */
-		if ( $this->is_already_linked( $uri ) ) {
-			return $label;
-		}
-
-
-		$link = - 1 < strpos( $css_class, 'wl-link' );
-
-		// If the entity should not be linked and link by default is also disabled,
-		// then don't lookup the entity on the table.
-		if ( ! $this->is_link_by_default && ! $link ) {
-			return $label;
-		}
-
-		$content_service = Wordpress_Content_Service::get_instance();
-		$content         = $content_service->get_by_entity_id_or_same_as( $uri );
-
-		// If no content is found, return the label, that is _remove the annotation_.
-		if ( ! is_object( $content ) ) {
-			return $label;
-		}
-
-		$object_id   = $content->get_id();
-		$object_type = $content->get_object_type_enum();
-
-
-		$no_link = - 1 < strpos( $css_class, 'wl-no-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;
-		}
-
-		/**
-		 * @since 3.32.0
-		 * Object_ids are prefixed with object_type to prevent conflicts.
-		 */
-		$this->linked_entity_uris[] = $uri;
-
-		// Get the link.
-		$href = Wordlift_Post_Adapter::get_production_permalink( $object_id, $object_type );
-
-		// Bail out if the `$href` has been reset.
-		if ( empty( $href ) ) {
-			return $label;
-		}
-
-		return Link_Builder::create( $uri, $object_id )
-		                   ->label( $label )
-		                   ->href( $href )
-		                   ->generate_link();
-	}
-
-
-	/**
-	 * 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.
-	 * @deprecated 3.32.0 Use object link provider to get the link title for getting link
-	 * title for different types.
-	 * @since 3.15.0
-	 *
-	 * As of 3.32.0 this method is not used anywhere in the core, this should be removed
-	 * from tests and companion plugins.
-	 *
-	 */
-	function get_link_title( $post_id, $ignore_label, $object_type = Object_Type_Enum::POST ) {
-		return $this->object_link_provider->get_link_title( $post_id, $ignore_label, $object_type );
-	}
-
-	/**
-	 * 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] ) );
-	}
-
-
-	/**
-	 * @param $entity_uri
-	 *
-	 * @return bool
-	 */
-	private function is_already_linked( $entity_uri ) {
-		return in_array( $entity_uri, $this->linked_entity_uris );
-	}
+    /**
+     * 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 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 $linked_entity_uris = 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;
+    /**
+     * @var Object_Link_Provider
+     */
+    private $object_link_provider;
+
+    /**
+     * Create a {@link Wordlift_Content_Filter_Service} instance.
+     *
+     * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     * @param \Wordlift_Entity_Uri_Service $entity_uri_service The {@link Wordlift_Entity_Uri_Service} instance.
+     *
+     * @since 3.8.0
+     *
+     */
+    protected function __construct( $entity_service, $entity_uri_service ) {
+
+        $this->log = Wordlift_Log_Service::get_logger( get_class() );
+
+        $this->entity_service       = $entity_service;
+        $this->entity_uri_service   = $entity_uri_service;
+        $this->object_link_provider = Object_Link_Provider::get_instance();
+
+    }
+
+    private static $instance = null;
+
+    /**
+     * 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() {
+
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self( Wordlift_Entity_Service::get_instance(), Wordlift_Entity_Uri_Service::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() || is_admin() || is_search() ) {
+            return $content;
+        }
+
+        // Preload the `link by default` setting.
+        $this->is_link_by_default = Wordlift_Configuration_Service::get_instance()->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->linked_entity_uris = 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;
+        }
+
+        // Replace each match of the entity tag with the entity link. If an error
+        // occurs fail silently returning the original content.
+        return preg_replace_callback( self::PATTERN, array(
+            $this,
+            'link',
+        ), $content ) ?: $content;
+    }
+
+    /**
+     * 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];
+
+
+        /**
+         * If the entity is already linked, dont send query to the db.
+         */
+        if ( $this->is_already_linked( $uri ) ) {
+            return $label;
+        }
+
+
+        $link = - 1 < strpos( $css_class, 'wl-link' );
+
+        // If the entity should not be linked and link by default is also disabled,
+        // then don't lookup the entity on the table.
+        if ( ! $this->is_link_by_default && ! $link ) {
+            return $label;
+        }
+
+        $content_service = Wordpress_Content_Service::get_instance();
+        $content         = $content_service->get_by_entity_id_or_same_as( $uri );
+
+        // If no content is found, return the label, that is _remove the annotation_.
+        if ( ! is_object( $content ) ) {
+            return $label;
+        }
+
+        $object_id   = $content->get_id();
+        $object_type = $content->get_object_type_enum();
+
+
+        $no_link = - 1 < strpos( $css_class, 'wl-no-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;
+        }
+
+        /**
+         * @since 3.32.0
+         * Object_ids are prefixed with object_type to prevent conflicts.
+         */
+        $this->linked_entity_uris[] = $uri;
+
+        // Get the link.
+        $href = Wordlift_Post_Adapter::get_production_permalink( $object_id, $object_type );
+
+        // Bail out if the `$href` has been reset.
+        if ( empty( $href ) ) {
+            return $label;
+        }
+
+        return Link_Builder::create( $uri, $object_id )
+                            ->label( $label )
+                            ->href( $href )
+                            ->generate_link();
+    }
+
+
+    /**
+     * 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.
+     * @deprecated 3.32.0 Use object link provider to get the link title for getting link
+     * title for different types.
+     * @since 3.15.0
+     *
+     * As of 3.32.0 this method is not used anywhere in the core, this should be removed
+     * from tests and companion plugins.
+     *
+     */
+    function get_link_title( $post_id, $ignore_label, $object_type = Object_Type_Enum::POST ) {
+        return $this->object_link_provider->get_link_title( $post_id, $ignore_label, $object_type );
+    }
+
+    /**
+     * 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] ) );
+    }
+
+
+    /**
+     * @param $entity_uri
+     *
+     * @return bool
+     */
+    private function is_already_linked( $entity_uri ) {
+        return in_array( $entity_uri, $this->linked_entity_uris );
+    }
 
 }

Spacing +32 added lines, -32 removed lines

@@ -81,9 +81,9 @@ 
 	 * @since 3.8.0
 	 *
 	 */
-	protected function __construct( $entity_service, $entity_uri_service ) {
+	protected function __construct($entity_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->entity_uri_service   = $entity_uri_service;
@@ -101,8 +101,8 @@ 
 	 */
 	public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self( Wordlift_Entity_Service::get_instance(), Wordlift_Entity_Uri_Service::get_instance() );
+		if ( ! isset(self::$instance)) {
+			self::$instance = new self(Wordlift_Entity_Service::get_instance(), Wordlift_Entity_Uri_Service::get_instance());
 		}
 
 		return self::$instance;
@@ -118,11 +118,11 @@ 
 	 * @since 3.8.0
 	 *
 	 */
-	public function the_content( $content ) {
-		$this->log->trace( "Filtering content [ " . ( is_singular() ? 'yes' : 'no' ) . " ]..." );
+	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() || is_admin() || is_search() ) {
+		if (is_feed() || is_admin() || is_search()) {
 			return $content;
 		}
 
@@ -135,19 +135,19 @@ 
 
 		// 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;
 		}
 
 		// Replace each match of the entity tag with the entity link. If an error
 		// occurs fail silently returning the original content.
-		return preg_replace_callback( self::PATTERN, array(
+		return preg_replace_callback(self::PATTERN, array(
 			$this,
 			'link',
-		), $content ) ?: $content;
+		), $content) ?: $content;
 	}
 
 	/**
@@ -159,7 +159,7 @@ 
 	 * @since 3.8.0
 	 *
 	 */
-	private function link( $matches ) {
+	private function link($matches) {
 
 		// Get the entity itemid URI and label.
 		$css_class = $matches[2];
@@ -170,24 +170,24 @@ 
 		/**
 		 * If the entity is already linked, dont send query to the db.
 		 */
-		if ( $this->is_already_linked( $uri ) ) {
+		if ($this->is_already_linked($uri)) {
 			return $label;
 		}
 
 
-		$link = - 1 < strpos( $css_class, 'wl-link' );
+		$link = - 1 < strpos($css_class, 'wl-link');
 
 		// If the entity should not be linked and link by default is also disabled,
 		// then don't lookup the entity on the table.
-		if ( ! $this->is_link_by_default && ! $link ) {
+		if ( ! $this->is_link_by_default && ! $link) {
 			return $label;
 		}
 
 		$content_service = Wordpress_Content_Service::get_instance();
-		$content         = $content_service->get_by_entity_id_or_same_as( $uri );
+		$content         = $content_service->get_by_entity_id_or_same_as($uri);
 
 		// If no content is found, return the label, that is _remove the annotation_.
-		if ( ! is_object( $content ) ) {
+		if ( ! is_object($content)) {
 			return $label;
 		}
 
@@ -195,14 +195,14 @@ 
 		$object_type = $content->get_object_type_enum();
 
 
-		$no_link = - 1 < strpos( $css_class, 'wl-no-link' );
+		$no_link = - 1 < strpos($css_class, 'wl-no-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;
 		}
 
@@ -213,16 +213,16 @@ 
 		$this->linked_entity_uris[] = $uri;
 
 		// Get the link.
-		$href = Wordlift_Post_Adapter::get_production_permalink( $object_id, $object_type );
+		$href = Wordlift_Post_Adapter::get_production_permalink($object_id, $object_type);
 
 		// Bail out if the `$href` has been reset.
-		if ( empty( $href ) ) {
+		if (empty($href)) {
 			return $label;
 		}
 
-		return Link_Builder::create( $uri, $object_id )
-		                   ->label( $label )
-		                   ->href( $href )
+		return Link_Builder::create($uri, $object_id)
+		                   ->label($label)
+		                   ->href($href)
 		                   ->generate_link();
 	}
 
@@ -243,8 +243,8 @@ 
 	 * from tests and companion plugins.
 	 *
 	 */
-	function get_link_title( $post_id, $ignore_label, $object_type = Object_Type_Enum::POST ) {
-		return $this->object_link_provider->get_link_title( $post_id, $ignore_label, $object_type );
+	function get_link_title($post_id, $ignore_label, $object_type = Object_Type_Enum::POST) {
+		return $this->object_link_provider->get_link_title($post_id, $ignore_label, $object_type);
 	}
 
 	/**
@@ -257,17 +257,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]));
 	}
 
 
@@ -276,8 +276,8 @@ 
 	 *
 	 * @return bool
 	 */
-	private function is_already_linked( $entity_uri ) {
-		return in_array( $entity_uri, $this->linked_entity_uris );
+	private function is_already_linked($entity_uri) {
+		return in_array($entity_uri, $this->linked_entity_uris);
 	}
 
 }

src/wordlift/class-object-type-enum.php

Indentation +49 added lines, -49 removed lines

@@ -4,62 +4,62 @@
 
 class Object_Type_Enum {
 
-	const POST = 0;
-	const TERM = 1;
-	const HOMEPAGE = 2;
-	const USER = 3;
+    const POST = 0;
+    const TERM = 1;
+    const HOMEPAGE = 2;
+    const USER = 3;
 
-	// Enum constant to represent currently unknown
-	// value.
-	const UNKNOWN = 4;
+    // Enum constant to represent currently unknown
+    // value.
+    const UNKNOWN = 4;
 
-	public static function to_string( $object_type_enum ) {
-		switch ( $object_type_enum ) {
-			case 0:
-				return 'post';
-			case 1:
-				return 'term';
-			case 2:
-				return 'home';
-			case 3:
-				return 'user';
-			case 4:
-				return 'unkn';
-		}
+    public static function to_string( $object_type_enum ) {
+        switch ( $object_type_enum ) {
+            case 0:
+                return 'post';
+            case 1:
+                return 'term';
+            case 2:
+                return 'home';
+            case 3:
+                return 'user';
+            case 4:
+                return 'unkn';
+        }
 
-		return null;
-	}
+        return null;
+    }
 
-	public static function from_string( $object_type_name ) {
-		switch ( $object_type_name ) {
-			case 'post':
-				return 0;
-			case 'term':
-				return 1;
-			case 'home':
-				return 2;
-			case 'user':
-				return 3;
-		}
+    public static function from_string( $object_type_name ) {
+        switch ( $object_type_name ) {
+            case 'post':
+                return 0;
+            case 'term':
+                return 1;
+            case 'home':
+                return 2;
+            case 'user':
+                return 3;
+        }
 
-		return 4;
-	}
+        return 4;
+    }
 
-	public static function from_wordpress_instance( $instance ) {
-		if ( ! is_object( $instance ) ) {
-			return null;
-		}
+    public static function from_wordpress_instance( $instance ) {
+        if ( ! is_object( $instance ) ) {
+            return null;
+        }
 
-		switch ( get_class( $instance ) ) {
-			case 'WP_Post':
-				return self::POST;
-			case 'WP_Term':
-				return self::TERM;
-			case 'WP_User':
-				return self::USER;
-		}
+        switch ( get_class( $instance ) ) {
+            case 'WP_Post':
+                return self::POST;
+            case 'WP_Term':
+                return self::TERM;
+            case 'WP_User':
+                return self::USER;
+        }
 
-		return null;
-	}
+        return null;
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -13,8 +13,8 @@ 
 	// value.
 	const UNKNOWN = 4;
 
-	public static function to_string( $object_type_enum ) {
-		switch ( $object_type_enum ) {
+	public static function to_string($object_type_enum) {
+		switch ($object_type_enum) {
 			case 0:
 				return 'post';
 			case 1:
@@ -30,8 +30,8 @@ 
 		return null;
 	}
 
-	public static function from_string( $object_type_name ) {
-		switch ( $object_type_name ) {
+	public static function from_string($object_type_name) {
+		switch ($object_type_name) {
 			case 'post':
 				return 0;
 			case 'term':
@@ -45,12 +45,12 @@ 
 		return 4;
 	}
 
-	public static function from_wordpress_instance( $instance ) {
-		if ( ! is_object( $instance ) ) {
+	public static function from_wordpress_instance($instance) {
+		if ( ! is_object($instance)) {
 			return null;
 		}
 
-		switch ( get_class( $instance ) ) {
+		switch (get_class($instance)) {
 			case 'WP_Post':
 				return self::POST;
 			case 'WP_Term':

src/modules/food-kg/includes/admin/Ingredients_List_Table.php

Indentation +50 added lines, -50 removed lines

@@ -3,7 +3,7 @@ 
 namespace Wordlift\Modules\Food_Kg\Admin;
 
 if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once( ABSPATH . 'wp-admin/includes/class-wp-list-table.php' );
+    require_once( ABSPATH . 'wp-admin/includes/class-wp-list-table.php' );
 }
 
 use WP_List_Table;
@@ -18,35 +18,35 @@ 
 //			'ajax'     => false,
 //		] );
 //	}
-	public function prepare_items() {
-		global $wpdb; //This is used only if making any database queries
-
-		/**
-		 * First, lets decide how many records per page to show
-		 */
-		$per_page = 5;
-
-
-		/**
-		 * REQUIRED. Now we need to define our column headers. This includes a complete
-		 * array of columns to be displayed (slugs & titles), a list of columns
-		 * to keep hidden, and a list of columns that are sortable. Each of these
-		 * can be defined in another method (as we've done here) before being
-		 * used to build the value for our _column_headers property.
-		 */
-		$columns  = $this->get_columns();
-		$hidden   = array();
-		$sortable = $this->get_sortable_columns();
-
-		/**
-		 * REQUIRED. Finally, we build an array to be used by the class for column
-		 * headers. The $this->_column_headers property takes an array which contains
-		 * 3 other arrays. One for all columns, one for hidden columns, and one
-		 * for sortable columns.
-		 */
-		$this->_column_headers = array( $columns, $hidden, $sortable );
-
-		$sql = "
+    public function prepare_items() {
+        global $wpdb; //This is used only if making any database queries
+
+        /**
+         * First, lets decide how many records per page to show
+         */
+        $per_page = 5;
+
+
+        /**
+         * REQUIRED. Now we need to define our column headers. This includes a complete
+         * array of columns to be displayed (slugs & titles), a list of columns
+         * to keep hidden, and a list of columns that are sortable. Each of these
+         * can be defined in another method (as we've done here) before being
+         * used to build the value for our _column_headers property.
+         */
+        $columns  = $this->get_columns();
+        $hidden   = array();
+        $sortable = $this->get_sortable_columns();
+
+        /**
+         * REQUIRED. Finally, we build an array to be used by the class for column
+         * headers. The $this->_column_headers property takes an array which contains
+         * 3 other arrays. One for all columns, one for hidden columns, and one
+         * for sortable columns.
+         */
+        $this->_column_headers = array( $columns, $hidden, $sortable );
+
+        $sql = "
 			SELECT t.term_id, t.name, tm.meta_value
 			FROM $wpdb->term_taxonomy tt
 			INNER JOIN $wpdb->terms t
@@ -57,33 +57,33 @@ 
 			WHERE tt.taxonomy = %s;
 		";
 
-		$this->items = $wpdb->get_results( $wpdb->prepare( $sql, 'wprm_ingredient' ) );
-	}
+        $this->items = $wpdb->get_results( $wpdb->prepare( $sql, 'wprm_ingredient' ) );
+    }
 
 
-	public function no_items() {
-		_e( 'No ingredients found.', 'wordlift' );
-	}
+    public function no_items() {
+        _e( 'No ingredients found.', 'wordlift' );
+    }
 
-	public function get_columns() {
-		return [
-			'name'    => __( 'Name', 'wordlift' ),
-			'actions' => '',
-		];
-	}
+    public function get_columns() {
+        return [
+            'name'    => __( 'Name', 'wordlift' ),
+            'actions' => '',
+        ];
+    }
 
-	public function column_name( $item ) {
-		return sprintf( '<a href="%s">%s</a>', get_edit_term_link( $item->term_id ), $item->name );
-	}
+    public function column_name( $item ) {
+        return sprintf( '<a href="%s">%s</a>', get_edit_term_link( $item->term_id ), $item->name );
+    }
 
-	public function column_actions( $item ) {
+    public function column_actions( $item ) {
 
-		$url = admin_url(
-			sprintf( 'admin.php?page=wl_ingredients&modal_window=true&term_id=%d&TB_iframe=true', $item->term_id ) );
+        $url = admin_url(
+            sprintf( 'admin.php?page=wl_ingredients&modal_window=true&term_id=%d&TB_iframe=true', $item->term_id ) );
 
-		return sprintf( '<a href="%s" class="button alignright thickbox open-plugin-details-modal" data-title="%s" type="button">%s</a>',
-			$url, esc_attr( $item->name ), esc_html__( 'JSON-LD', 'wordlift' ) );
-	}
+        return sprintf( '<a href="%s" class="button alignright thickbox open-plugin-details-modal" data-title="%s" type="button">%s</a>',
+            $url, esc_attr( $item->name ), esc_html__( 'JSON-LD', 'wordlift' ) );
+    }
 
 }
 

Spacing +12 added lines, -12 removed lines

@@ -2,8 +2,8 @@ 
 
 namespace Wordlift\Modules\Food_Kg\Admin;
 
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once( ABSPATH . 'wp-admin/includes/class-wp-list-table.php' );
+if ( ! class_exists('WP_List_Table')) {
+	require_once(ABSPATH.'wp-admin/includes/class-wp-list-table.php');
 }
 
 use WP_List_Table;
@@ -44,7 +44,7 @@ 
 		 * 3 other arrays. One for all columns, one for hidden columns, and one
 		 * for sortable columns.
 		 */
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$sql = "
 			SELECT t.term_id, t.name, tm.meta_value
@@ -57,32 +57,32 @@ 
 			WHERE tt.taxonomy = %s;
 		";
 
-		$this->items = $wpdb->get_results( $wpdb->prepare( $sql, 'wprm_ingredient' ) );
+		$this->items = $wpdb->get_results($wpdb->prepare($sql, 'wprm_ingredient'));
 	}
 
 
 	public function no_items() {
-		_e( 'No ingredients found.', 'wordlift' );
+		_e('No ingredients found.', 'wordlift');
 	}
 
 	public function get_columns() {
 		return [
-			'name'    => __( 'Name', 'wordlift' ),
+			'name'    => __('Name', 'wordlift'),
 			'actions' => '',
 		];
 	}
 
-	public function column_name( $item ) {
-		return sprintf( '<a href="%s">%s</a>', get_edit_term_link( $item->term_id ), $item->name );
+	public function column_name($item) {
+		return sprintf('<a href="%s">%s</a>', get_edit_term_link($item->term_id), $item->name);
 	}
 
-	public function column_actions( $item ) {
+	public function column_actions($item) {
 
 		$url = admin_url(
-			sprintf( 'admin.php?page=wl_ingredients&modal_window=true&term_id=%d&TB_iframe=true', $item->term_id ) );
+			sprintf('admin.php?page=wl_ingredients&modal_window=true&term_id=%d&TB_iframe=true', $item->term_id) );
 
-		return sprintf( '<a href="%s" class="button alignright thickbox open-plugin-details-modal" data-title="%s" type="button">%s</a>',
-			$url, esc_attr( $item->name ), esc_html__( 'JSON-LD', 'wordlift' ) );
+		return sprintf('<a href="%s" class="button alignright thickbox open-plugin-details-modal" data-title="%s" type="button">%s</a>',
+			$url, esc_attr($item->name), esc_html__('JSON-LD', 'wordlift'));
 	}
 
 }

src/modules/food-kg/includes/admin/partials/ingredients.php

Indentation +5 added lines, -5 removed lines

@@ -7,9 +7,9 @@
     <h1><?php esc_attr_e( 'Ingredients', 'wordlift' ); ?></h1>
 	<?php
 
-	//Prepare Table of elements
-	$wp_list_table = new Ingredients_List_Table();
-	$wp_list_table->prepare_items();
-	$wp_list_table->display();
-	?>
+    //Prepare Table of elements
+    $wp_list_table = new Ingredients_List_Table();
+    $wp_list_table->prepare_items();
+    $wp_list_table->display();
+    ?>
 </div>

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
 ?>
 <div class="wrap">
-    <h1><?php esc_attr_e( 'Ingredients', 'wordlift' ); ?></h1>
+    <h1><?php esc_attr_e('Ingredients', 'wordlift'); ?></h1>
 	<?php
 
 	//Prepare Table of elements

src/modules/food-kg/includes/admin/partials/jsonld.php

Spacing +2 added lines, -2 removed lines

@@ -2,8 +2,8 @@
     <?php
 
     /** @var string $value */
-    $json = wp_json_encode( json_decode( $value ), JSON_PRETTY_PRINT );
-    echo esc_html( $json );
+    $json = wp_json_encode(json_decode($value), JSON_PRETTY_PRINT);
+    echo esc_html($json);
 
     ?>
 </textarea>
\ No newline at end of file

src/modules/food-kg/includes/admin/partials/main_ingredient.php

Indentation +5 added lines, -5 removed lines

@@ -7,9 +7,9 @@
     <h1><?php esc_attr_e( 'Main Ingredients', 'wordlift' ); ?></h1>
 	<?php
 
-	//Prepare Table of elements
-	$wp_list_table = new Main_Ingredient_List_Table();
-	$wp_list_table->prepare_items();
-	$wp_list_table->display();
-	?>
+    //Prepare Table of elements
+    $wp_list_table = new Main_Ingredient_List_Table();
+    $wp_list_table->prepare_items();
+    $wp_list_table->display();
+    ?>
 </div>

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
 ?>
 <div class="wrap">
-    <h1><?php esc_attr_e( 'Main Ingredients', 'wordlift' ); ?></h1>
+    <h1><?php esc_attr_e('Main Ingredients', 'wordlift'); ?></h1>
 	<?php
 
 	//Prepare Table of elements

src/modules/food-kg/includes/admin/Page.php

Indentation +33 added lines, -33 removed lines

@@ -4,38 +4,38 @@
 
 class Page {
 
-	/**
-	 * @var Page_Delegate
-	 */
-	private $delegate;
-
-	/**
-	 * @param Page_Delegate $full_page_delegate
-	 * @param Page_Delegate $modal_page_delegate
-	 */
-	public function __construct( $full_page_delegate, $modal_page_delegate ) {
-		$this->delegate = isset( $_GET['modal_window'] ) ? $modal_page_delegate : $full_page_delegate;
-	}
-
-	public function register_hooks() {
-		add_action( 'admin_menu', [ $this, 'admin_menu' ] );
-		add_action( 'admin_enqueue_scripts', [ $this, 'admin_enqueue_scripts' ] );
-	}
-
-	public function admin_menu() {
-		add_submenu_page( 'wl_admin_menu', __( 'Ingredients', 'wordlift' ), __( 'Ingredients', 'wordlift' ),
-			'manage_options', 'wl_ingredients', [ $this, 'render' ] );
-	}
-
-	public function render() {
-		$this->delegate->render();
-	}
-
-	public function admin_enqueue_scripts() {
-		// Check that we are on the right screen
-		if ( get_current_screen()->id === 'wordlift_page_wl_ingredients' ) {
-			$this->delegate->admin_enqueue_scripts();
-		}
-	}
+    /**
+     * @var Page_Delegate
+     */
+    private $delegate;
+
+    /**
+     * @param Page_Delegate $full_page_delegate
+     * @param Page_Delegate $modal_page_delegate
+     */
+    public function __construct( $full_page_delegate, $modal_page_delegate ) {
+        $this->delegate = isset( $_GET['modal_window'] ) ? $modal_page_delegate : $full_page_delegate;
+    }
+
+    public function register_hooks() {
+        add_action( 'admin_menu', [ $this, 'admin_menu' ] );
+        add_action( 'admin_enqueue_scripts', [ $this, 'admin_enqueue_scripts' ] );
+    }
+
+    public function admin_menu() {
+        add_submenu_page( 'wl_admin_menu', __( 'Ingredients', 'wordlift' ), __( 'Ingredients', 'wordlift' ),
+            'manage_options', 'wl_ingredients', [ $this, 'render' ] );
+    }
+
+    public function render() {
+        $this->delegate->render();
+    }
+
+    public function admin_enqueue_scripts() {
+        // Check that we are on the right screen
+        if ( get_current_screen()->id === 'wordlift_page_wl_ingredients' ) {
+            $this->delegate->admin_enqueue_scripts();
+        }
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -13,18 +13,18 @@ 
 	 * @param Page_Delegate $full_page_delegate
 	 * @param Page_Delegate $modal_page_delegate
 	 */
-	public function __construct( $full_page_delegate, $modal_page_delegate ) {
-		$this->delegate = isset( $_GET['modal_window'] ) ? $modal_page_delegate : $full_page_delegate;
+	public function __construct($full_page_delegate, $modal_page_delegate) {
+		$this->delegate = isset($_GET['modal_window']) ? $modal_page_delegate : $full_page_delegate;
 	}
 
 	public function register_hooks() {
-		add_action( 'admin_menu', [ $this, 'admin_menu' ] );
-		add_action( 'admin_enqueue_scripts', [ $this, 'admin_enqueue_scripts' ] );
+		add_action('admin_menu', [$this, 'admin_menu']);
+		add_action('admin_enqueue_scripts', [$this, 'admin_enqueue_scripts']);
 	}
 
 	public function admin_menu() {
-		add_submenu_page( 'wl_admin_menu', __( 'Ingredients', 'wordlift' ), __( 'Ingredients', 'wordlift' ),
-			'manage_options', 'wl_ingredients', [ $this, 'render' ] );
+		add_submenu_page('wl_admin_menu', __('Ingredients', 'wordlift'), __('Ingredients', 'wordlift'),
+			'manage_options', 'wl_ingredients', [$this, 'render']);
 	}
 
 	public function render() {
@@ -33,7 +33,7 @@ 
 
 	public function admin_enqueue_scripts() {
 		// Check that we are on the right screen
-		if ( get_current_screen()->id === 'wordlift_page_wl_ingredients' ) {
+		if (get_current_screen()->id === 'wordlift_page_wl_ingredients') {
 			$this->delegate->admin_enqueue_scripts();
 		}
 	}

src/modules/food-kg/includes/admin/Main_Ingredient_List_Table.php

Indentation +52 added lines, -52 removed lines

@@ -3,7 +3,7 @@ 
 namespace Wordlift\Modules\Food_Kg\Admin;
 
 if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once( ABSPATH . 'wp-admin/includes/class-wp-list-table.php' );
+    require_once( ABSPATH . 'wp-admin/includes/class-wp-list-table.php' );
 }
 
 use WP_List_Table;
@@ -11,35 +11,35 @@ 
 
 class Main_Ingredient_List_Table extends WP_List_Table {
 
-	public function prepare_items() {
-		global $wpdb; //This is used only if making any database queries
-
-		/**
-		 * First, lets decide how many records per page to show
-		 */
-		$per_page = 5;
-
-
-		/**
-		 * REQUIRED. Now we need to define our column headers. This includes a complete
-		 * array of columns to be displayed (slugs & titles), a list of columns
-		 * to keep hidden, and a list of columns that are sortable. Each of these
-		 * can be defined in another method (as we've done here) before being
-		 * used to build the value for our _column_headers property.
-		 */
-		$columns  = $this->get_columns();
-		$hidden   = array();
-		$sortable = $this->get_sortable_columns();
-
-		/**
-		 * REQUIRED. Finally, we build an array to be used by the class for column
-		 * headers. The $this->_column_headers property takes an array which contains
-		 * 3 other arrays. One for all columns, one for hidden columns, and one
-		 * for sortable columns.
-		 */
-		$this->_column_headers = array( $columns, $hidden, $sortable );
-
-		$sql = "
+    public function prepare_items() {
+        global $wpdb; //This is used only if making any database queries
+
+        /**
+         * First, lets decide how many records per page to show
+         */
+        $per_page = 5;
+
+
+        /**
+         * REQUIRED. Now we need to define our column headers. This includes a complete
+         * array of columns to be displayed (slugs & titles), a list of columns
+         * to keep hidden, and a list of columns that are sortable. Each of these
+         * can be defined in another method (as we've done here) before being
+         * used to build the value for our _column_headers property.
+         */
+        $columns  = $this->get_columns();
+        $hidden   = array();
+        $sortable = $this->get_sortable_columns();
+
+        /**
+         * REQUIRED. Finally, we build an array to be used by the class for column
+         * headers. The $this->_column_headers property takes an array which contains
+         * 3 other arrays. One for all columns, one for hidden columns, and one
+         * for sortable columns.
+         */
+        $this->_column_headers = array( $columns, $hidden, $sortable );
+
+        $sql = "
 			SELECT p.ID, p.post_title, pm.meta_value
 			FROM $wpdb->posts p
 			INNER JOIN $wpdb->postmeta pm
@@ -47,36 +47,36 @@ 
 					AND pm.meta_key = '_wl_main_ingredient_jsonld';
 		";
 
-		$this->items = $wpdb->get_results( $wpdb->prepare( $sql ) );
-	}
+        $this->items = $wpdb->get_results( $wpdb->prepare( $sql ) );
+    }
 
 
-	public function no_items() {
-		_e( 'No main ingredients found.', 'wordlift' );
-	}
+    public function no_items() {
+        _e( 'No main ingredients found.', 'wordlift' );
+    }
 
-	public function get_columns() {
-		return [
-			'post_title' => __( 'Name', 'wordlift' ),
-			'actions'    => '',
-		];
-	}
+    public function get_columns() {
+        return [
+            'post_title' => __( 'Name', 'wordlift' ),
+            'actions'    => '',
+        ];
+    }
 
-	public function column_post_title( $item ) {
-		$recipe   = WPRM_Recipe_Manager::get_recipe( $item->ID );
-		$edit_url = $recipe->parent_edit_url();
+    public function column_post_title( $item ) {
+        $recipe   = WPRM_Recipe_Manager::get_recipe( $item->ID );
+        $edit_url = $recipe->parent_edit_url();
 
-		return empty( $edit_url ) ? $item->post_title : sprintf( '<a href="%s">%s</a>', $recipe->parent_edit_url(), $item->post_title );
-	}
+        return empty( $edit_url ) ? $item->post_title : sprintf( '<a href="%s">%s</a>', $recipe->parent_edit_url(), $item->post_title );
+    }
 
-	public function column_actions( $item ) {
+    public function column_actions( $item ) {
 
-		$url = admin_url(
-			sprintf( 'admin.php?page=wl_ingredients&modal_window=true&id=%d&TB_iframe=true', $item->ID ) );
+        $url = admin_url(
+            sprintf( 'admin.php?page=wl_ingredients&modal_window=true&id=%d&TB_iframe=true', $item->ID ) );
 
-		return sprintf( '<a href="%s" class="button alignright thickbox open-plugin-details-modal" data-title="%s" type="button">%s</a>',
-			$url, esc_attr( $item->post_title ), esc_html__( 'JSON-LD', 'wordlift' ) );
-	}
+        return sprintf( '<a href="%s" class="button alignright thickbox open-plugin-details-modal" data-title="%s" type="button">%s</a>',
+            $url, esc_attr( $item->post_title ), esc_html__( 'JSON-LD', 'wordlift' ) );
+    }
 
 }
 

Spacing +13 added lines, -13 removed lines

@@ -2,8 +2,8 @@ 
 
 namespace Wordlift\Modules\Food_Kg\Admin;
 
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once( ABSPATH . 'wp-admin/includes/class-wp-list-table.php' );
+if ( ! class_exists('WP_List_Table')) {
+	require_once(ABSPATH.'wp-admin/includes/class-wp-list-table.php');
 }
 
 use WP_List_Table;
@@ -37,7 +37,7 @@ 
 		 * 3 other arrays. One for all columns, one for hidden columns, and one
 		 * for sortable columns.
 		 */
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$sql = "
 			SELECT p.ID, p.post_title, pm.meta_value
@@ -47,35 +47,35 @@ 
 					AND pm.meta_key = '_wl_main_ingredient_jsonld';
 		";
 
-		$this->items = $wpdb->get_results( $wpdb->prepare( $sql ) );
+		$this->items = $wpdb->get_results($wpdb->prepare($sql));
 	}
 
 
 	public function no_items() {
-		_e( 'No main ingredients found.', 'wordlift' );
+		_e('No main ingredients found.', 'wordlift');
 	}
 
 	public function get_columns() {
 		return [
-			'post_title' => __( 'Name', 'wordlift' ),
+			'post_title' => __('Name', 'wordlift'),
 			'actions'    => '',
 		];
 	}
 
-	public function column_post_title( $item ) {
-		$recipe   = WPRM_Recipe_Manager::get_recipe( $item->ID );
+	public function column_post_title($item) {
+		$recipe   = WPRM_Recipe_Manager::get_recipe($item->ID);
 		$edit_url = $recipe->parent_edit_url();
 
-		return empty( $edit_url ) ? $item->post_title : sprintf( '<a href="%s">%s</a>', $recipe->parent_edit_url(), $item->post_title );
+		return empty($edit_url) ? $item->post_title : sprintf('<a href="%s">%s</a>', $recipe->parent_edit_url(), $item->post_title);
 	}
 
-	public function column_actions( $item ) {
+	public function column_actions($item) {
 
 		$url = admin_url(
-			sprintf( 'admin.php?page=wl_ingredients&modal_window=true&id=%d&TB_iframe=true', $item->ID ) );
+			sprintf('admin.php?page=wl_ingredients&modal_window=true&id=%d&TB_iframe=true', $item->ID) );
 
-		return sprintf( '<a href="%s" class="button alignright thickbox open-plugin-details-modal" data-title="%s" type="button">%s</a>',
-			$url, esc_attr( $item->post_title ), esc_html__( 'JSON-LD', 'wordlift' ) );
+		return sprintf('<a href="%s" class="button alignright thickbox open-plugin-details-modal" data-title="%s" type="button">%s</a>',
+			$url, esc_attr($item->post_title), esc_html__('JSON-LD', 'wordlift'));
 	}
 
 }