insideout10 / wordlift-plugin

src/wordlift/content/wordpress/class-wordpress-post-content-table-service.php

Indentation +133 added lines, -133 removed lines

@@ -15,158 +15,158 @@
 
 class Wordpress_Post_Content_Table_Service extends Abstract_Wordpress_Content_Service {
 
-	private static $instance = null;
-
-	/**
-	 * The singleton instance. We use this only to provide this instance to those classes where we have no access to
-	 * the constructor.
-	 *
-	 * @return Wordpress_Post_Content_Table_Service
-	 */
-	public static function get_instance() {
-
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self();
-		}
-
-		return self::$instance;
-	}
-
-	/**
-	 * @param string $uri An absolute or relative URI. When absolute it must be within the dataset URI scope.
-	 *
-	 * @return Wordpress_Content|null
-	 * @throws Exception when the URI is not within the dataset URI.
-	 */
-	function get_by_entity_id( $uri ) {
-		Assertions::is_string( $uri, '`uri` must be a string.' );
-		Assertions::not_empty( $uri, '`uri` cannot be empty.' );
-		Assertions::not_empty( $this->get_dataset_uri(), '`dataset_uri` cannot be empty.' );
-
-		if ( $this->is_absolute( $uri ) && ! $this->is_internal( $uri ) ) {
-			throw new Exception( '`uri` must be within the dataset URI scope.' );
-		}
-
-		$rel_uri = $this->make_relative( $uri );
-
-		global $wpdb;
-		$row = $wpdb->get_row( $wpdb->prepare( "
+    private static $instance = null;
+
+    /**
+     * The singleton instance. We use this only to provide this instance to those classes where we have no access to
+     * the constructor.
+     *
+     * @return Wordpress_Post_Content_Table_Service
+     */
+    public static function get_instance() {
+
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self();
+        }
+
+        return self::$instance;
+    }
+
+    /**
+     * @param string $uri An absolute or relative URI. When absolute it must be within the dataset URI scope.
+     *
+     * @return Wordpress_Content|null
+     * @throws Exception when the URI is not within the dataset URI.
+     */
+    function get_by_entity_id( $uri ) {
+        Assertions::is_string( $uri, '`uri` must be a string.' );
+        Assertions::not_empty( $uri, '`uri` cannot be empty.' );
+        Assertions::not_empty( $this->get_dataset_uri(), '`dataset_uri` cannot be empty.' );
+
+        if ( $this->is_absolute( $uri ) && ! $this->is_internal( $uri ) ) {
+            throw new Exception( '`uri` must be within the dataset URI scope.' );
+        }
+
+        $rel_uri = $this->make_relative( $uri );
+
+        global $wpdb;
+        $row = $wpdb->get_row( $wpdb->prepare( "
 			SELECT content_type, content_id
 			FROM {$wpdb->prefix}wl_entities
 			WHERE rel_uri = %s
 		", $rel_uri ) );
 
-		if ( ! isset( $row ) || Object_Type_Enum::POST !== (int) $row->content_type ) {
-			return null;
-		}
-
-		return new Wordpress_Content( get_post( $row->content_id ) );
-	}
-
-	/**
-	 * @throws Exception when `$uri` is not a string.
-	 */
-	function get_by_entity_id_or_same_as( $uri ) {
-		Assertions::is_string( $uri, '`uri` must be a string.' );
-		Assertions::not_empty( '`uri` cannot be empty.' );
-
-		// If it's a relative URI, or it's an internal URI, look in entity ID.
-		if ( ! $this->is_absolute( $uri ) || $this->is_internal( $uri ) ) {
-			return $this->get_by_entity_id( $uri );
-		}
-
-		// Look in sameAs.
-		$query_args = array(
-			// See https://github.com/insideout10/wordlift-plugin/issues/654.
-			'ignore_sticky_posts' => 1,
-			'posts_per_page'      => 1,
-			'post_status'         => 'any',
-			'post_type'           => Wordlift_Entity_Service::valid_entity_post_types(),
-			'meta_query'          => array(
-				array(
-					'key'     => Wordlift_Schema_Service::FIELD_SAME_AS,
-					'value'   => $uri,
-					'compare' => '=',
-				)
-			),
-		);
-
-		$posts = get_posts( $query_args );
-
-		// Get the current post or allow 3rd parties to provide a replacement.
-		$post = current( $posts ) ?: apply_filters( 'wl_content_service__post__not_found', null, $uri );
-
-		if ( is_a( $post, 'WP_Post' ) ) {
-			return new Wordpress_Content( current( $posts ) );
-		}
-
-		return null;
-	}
-
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 *
-	 * @return string|null The entity ID.
-	 * @throws Exception
-	 */
-	function get_entity_id( $content_id ) {
-		Assertions::equals( $content_id->get_type(), Object_Type_Enum::POST, '`content_id` must be of type post.' );
-
-		global $wpdb;
-		$rel_uri = $wpdb->get_var( $wpdb->prepare( "
+        if ( ! isset( $row ) || Object_Type_Enum::POST !== (int) $row->content_type ) {
+            return null;
+        }
+
+        return new Wordpress_Content( get_post( $row->content_id ) );
+    }
+
+    /**
+     * @throws Exception when `$uri` is not a string.
+     */
+    function get_by_entity_id_or_same_as( $uri ) {
+        Assertions::is_string( $uri, '`uri` must be a string.' );
+        Assertions::not_empty( '`uri` cannot be empty.' );
+
+        // If it's a relative URI, or it's an internal URI, look in entity ID.
+        if ( ! $this->is_absolute( $uri ) || $this->is_internal( $uri ) ) {
+            return $this->get_by_entity_id( $uri );
+        }
+
+        // Look in sameAs.
+        $query_args = array(
+            // See https://github.com/insideout10/wordlift-plugin/issues/654.
+            'ignore_sticky_posts' => 1,
+            'posts_per_page'      => 1,
+            'post_status'         => 'any',
+            'post_type'           => Wordlift_Entity_Service::valid_entity_post_types(),
+            'meta_query'          => array(
+                array(
+                    'key'     => Wordlift_Schema_Service::FIELD_SAME_AS,
+                    'value'   => $uri,
+                    'compare' => '=',
+                )
+            ),
+        );
+
+        $posts = get_posts( $query_args );
+
+        // Get the current post or allow 3rd parties to provide a replacement.
+        $post = current( $posts ) ?: apply_filters( 'wl_content_service__post__not_found', null, $uri );
+
+        if ( is_a( $post, 'WP_Post' ) ) {
+            return new Wordpress_Content( current( $posts ) );
+        }
+
+        return null;
+    }
+
+    /**
+     * @param Wordpress_Content_Id $content_id
+     *
+     * @return string|null The entity ID.
+     * @throws Exception
+     */
+    function get_entity_id( $content_id ) {
+        Assertions::equals( $content_id->get_type(), Object_Type_Enum::POST, '`content_id` must be of type post.' );
+
+        global $wpdb;
+        $rel_uri = $wpdb->get_var( $wpdb->prepare( "
 			SELECT rel_uri
 			FROM {$wpdb->prefix}wl_entities
 			WHERE content_id = %d AND content_type = %d
 		", $content_id->get_id(), $content_id->get_type() ) );
 
-		return $rel_uri ? $this->make_absolute( $rel_uri ) : null;
-	}
+        return $rel_uri ? $this->make_absolute( $rel_uri ) : null;
+    }
 
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 * @param string $uri
-	 *
-	 * @return void
-	 * @throws Exception
-	 */
-	function set_entity_id( $content_id, $uri ) {
-		Assertions::equals( $content_id->get_type(), Object_Type_Enum::POST, '`content_id` must be of type post.' );
-		Assertions::not_empty( $uri, "`uri` can't be empty" );
+    /**
+     * @param Wordpress_Content_Id $content_id
+     * @param string $uri
+     *
+     * @return void
+     * @throws Exception
+     */
+    function set_entity_id( $content_id, $uri ) {
+        Assertions::equals( $content_id->get_type(), Object_Type_Enum::POST, '`content_id` must be of type post.' );
+        Assertions::not_empty( $uri, "`uri` can't be empty" );
 
-		if ( $this->is_absolute( $uri ) && ! $this->is_internal( $uri ) ) {
-			throw new Exception( '`uri` must be within the dataset URI scope.' );
-		}
+        if ( $this->is_absolute( $uri ) && ! $this->is_internal( $uri ) ) {
+            throw new Exception( '`uri` must be within the dataset URI scope.' );
+        }
 
-		$rel_url = $this->make_relative( $uri );
+        $rel_url = $this->make_relative( $uri );
 
-		global $wpdb;
-		$wpdb->query( $wpdb->prepare( "
+        global $wpdb;
+        $wpdb->query( $wpdb->prepare( "
 			INSERT INTO {$wpdb->prefix}wl_entities( content_id, content_type, rel_uri, rel_uri_hash )
 			VALUES( %d, %d, %s, SHA1( %s ) )
 			ON DUPLICATE KEY UPDATE rel_uri = VALUES( rel_uri ), rel_uri_hash = SHA1( VALUES( rel_uri ) );
 		", $content_id->get_id(), $content_id->get_type(), $rel_url, $rel_url ) );
-	}
-
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 *
-	 * @return bool
-	 */
-	function supports( $content_id ) {
-		return $content_id->get_type() === Object_Type_Enum::POST;
-	}
-
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 *
-	 * @return void
-	 */
-	function delete( $content_id ) {
-		global $wpdb;
-
-		$wpdb->query( $wpdb->prepare( "
+    }
+
+    /**
+     * @param Wordpress_Content_Id $content_id
+     *
+     * @return bool
+     */
+    function supports( $content_id ) {
+        return $content_id->get_type() === Object_Type_Enum::POST;
+    }
+
+    /**
+     * @param Wordpress_Content_Id $content_id
+     *
+     * @return void
+     */
+    function delete( $content_id ) {
+        global $wpdb;
+
+        $wpdb->query( $wpdb->prepare( "
 			DELETE FROM {$wpdb->prefix}wl_entities
 			WHERE content_id = %d AND content_type = %d
 		", $content_id->get_id(), $content_id->get_type() ) );
-	}
+    }
 }
\ No newline at end of file

Spacing +38 added lines, -38 removed lines

@@ -25,7 +25,7 @@ 
 	 */
 	public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -38,41 +38,41 @@ 
 	 * @return Wordpress_Content|null
 	 * @throws Exception when the URI is not within the dataset URI.
 	 */
-	function get_by_entity_id( $uri ) {
-		Assertions::is_string( $uri, '`uri` must be a string.' );
-		Assertions::not_empty( $uri, '`uri` cannot be empty.' );
-		Assertions::not_empty( $this->get_dataset_uri(), '`dataset_uri` cannot be empty.' );
+	function get_by_entity_id($uri) {
+		Assertions::is_string($uri, '`uri` must be a string.');
+		Assertions::not_empty($uri, '`uri` cannot be empty.');
+		Assertions::not_empty($this->get_dataset_uri(), '`dataset_uri` cannot be empty.');
 
-		if ( $this->is_absolute( $uri ) && ! $this->is_internal( $uri ) ) {
-			throw new Exception( '`uri` must be within the dataset URI scope.' );
+		if ($this->is_absolute($uri) && ! $this->is_internal($uri)) {
+			throw new Exception('`uri` must be within the dataset URI scope.');
 		}
 
-		$rel_uri = $this->make_relative( $uri );
+		$rel_uri = $this->make_relative($uri);
 
 		global $wpdb;
-		$row = $wpdb->get_row( $wpdb->prepare( "
+		$row = $wpdb->get_row($wpdb->prepare("
 			SELECT content_type, content_id
 			FROM {$wpdb->prefix}wl_entities
 			WHERE rel_uri = %s
-		", $rel_uri ) );
+		", $rel_uri));
 
-		if ( ! isset( $row ) || Object_Type_Enum::POST !== (int) $row->content_type ) {
+		if ( ! isset($row) || Object_Type_Enum::POST !== (int) $row->content_type) {
 			return null;
 		}
 
-		return new Wordpress_Content( get_post( $row->content_id ) );
+		return new Wordpress_Content(get_post($row->content_id));
 	}
 
 	/**
 	 * @throws Exception when `$uri` is not a string.
 	 */
-	function get_by_entity_id_or_same_as( $uri ) {
-		Assertions::is_string( $uri, '`uri` must be a string.' );
-		Assertions::not_empty( '`uri` cannot be empty.' );
+	function get_by_entity_id_or_same_as($uri) {
+		Assertions::is_string($uri, '`uri` must be a string.');
+		Assertions::not_empty('`uri` cannot be empty.');
 
 		// If it's a relative URI, or it's an internal URI, look in entity ID.
-		if ( ! $this->is_absolute( $uri ) || $this->is_internal( $uri ) ) {
-			return $this->get_by_entity_id( $uri );
+		if ( ! $this->is_absolute($uri) || $this->is_internal($uri)) {
+			return $this->get_by_entity_id($uri);
 		}
 
 		// Look in sameAs.
@@ -91,13 +91,13 @@ 
 			),
 		);
 
-		$posts = get_posts( $query_args );
+		$posts = get_posts($query_args);
 
 		// Get the current post or allow 3rd parties to provide a replacement.
-		$post = current( $posts ) ?: apply_filters( 'wl_content_service__post__not_found', null, $uri );
+		$post = current($posts) ?: apply_filters('wl_content_service__post__not_found', null, $uri);
 
-		if ( is_a( $post, 'WP_Post' ) ) {
-			return new Wordpress_Content( current( $posts ) );
+		if (is_a($post, 'WP_Post')) {
+			return new Wordpress_Content(current($posts));
 		}
 
 		return null;
@@ -109,17 +109,17 @@ 
 	 * @return string|null The entity ID.
 	 * @throws Exception
 	 */
-	function get_entity_id( $content_id ) {
-		Assertions::equals( $content_id->get_type(), Object_Type_Enum::POST, '`content_id` must be of type post.' );
+	function get_entity_id($content_id) {
+		Assertions::equals($content_id->get_type(), Object_Type_Enum::POST, '`content_id` must be of type post.');
 
 		global $wpdb;
-		$rel_uri = $wpdb->get_var( $wpdb->prepare( "
+		$rel_uri = $wpdb->get_var($wpdb->prepare("
 			SELECT rel_uri
 			FROM {$wpdb->prefix}wl_entities
 			WHERE content_id = %d AND content_type = %d
-		", $content_id->get_id(), $content_id->get_type() ) );
+		", $content_id->get_id(), $content_id->get_type()));
 
-		return $rel_uri ? $this->make_absolute( $rel_uri ) : null;
+		return $rel_uri ? $this->make_absolute($rel_uri) : null;
 	}
 
 	/**
@@ -129,22 +129,22 @@ 
 	 * @return void
 	 * @throws Exception
 	 */
-	function set_entity_id( $content_id, $uri ) {
-		Assertions::equals( $content_id->get_type(), Object_Type_Enum::POST, '`content_id` must be of type post.' );
-		Assertions::not_empty( $uri, "`uri` can't be empty" );
+	function set_entity_id($content_id, $uri) {
+		Assertions::equals($content_id->get_type(), Object_Type_Enum::POST, '`content_id` must be of type post.');
+		Assertions::not_empty($uri, "`uri` can't be empty");
 
-		if ( $this->is_absolute( $uri ) && ! $this->is_internal( $uri ) ) {
-			throw new Exception( '`uri` must be within the dataset URI scope.' );
+		if ($this->is_absolute($uri) && ! $this->is_internal($uri)) {
+			throw new Exception('`uri` must be within the dataset URI scope.');
 		}
 
-		$rel_url = $this->make_relative( $uri );
+		$rel_url = $this->make_relative($uri);
 
 		global $wpdb;
-		$wpdb->query( $wpdb->prepare( "
+		$wpdb->query($wpdb->prepare("
 			INSERT INTO {$wpdb->prefix}wl_entities( content_id, content_type, rel_uri, rel_uri_hash )
 			VALUES( %d, %d, %s, SHA1( %s ) )
 			ON DUPLICATE KEY UPDATE rel_uri = VALUES( rel_uri ), rel_uri_hash = SHA1( VALUES( rel_uri ) );
-		", $content_id->get_id(), $content_id->get_type(), $rel_url, $rel_url ) );
+		", $content_id->get_id(), $content_id->get_type(), $rel_url, $rel_url));
 	}
 
 	/**
@@ -152,7 +152,7 @@ 
 	 *
 	 * @return bool
 	 */
-	function supports( $content_id ) {
+	function supports($content_id) {
 		return $content_id->get_type() === Object_Type_Enum::POST;
 	}
 
@@ -161,12 +161,12 @@ 
 	 *
 	 * @return void
 	 */
-	function delete( $content_id ) {
+	function delete($content_id) {
 		global $wpdb;
 
-		$wpdb->query( $wpdb->prepare( "
+		$wpdb->query($wpdb->prepare("
 			DELETE FROM {$wpdb->prefix}wl_entities
 			WHERE content_id = %d AND content_type = %d
-		", $content_id->get_id(), $content_id->get_type() ) );
+		", $content_id->get_id(), $content_id->get_type()));
 	}
 }
\ No newline at end of file

src/wordlift/content/wordpress/class-abstract-wordpress-content-legacy-service.php

Indentation +15 added lines, -15 removed lines

@@ -8,24 +8,24 @@
 abstract class Abstract_Wordpress_Content_Legacy_Service extends Abstract_Wordpress_Content_Service {
 
 
-	private $expected_object_type;
-	private $get_meta_fn;
+    private $expected_object_type;
+    private $get_meta_fn;
 
-	protected function __construct( $expected_object_type, $get_meta_fn ) {
-		parent::__construct();
+    protected function __construct( $expected_object_type, $get_meta_fn ) {
+        parent::__construct();
 
-		$this->expected_object_type = $expected_object_type;
-		$this->get_meta_fn          = $get_meta_fn;
-	}
+        $this->expected_object_type = $expected_object_type;
+        $this->get_meta_fn          = $get_meta_fn;
+    }
 
-	public function get_entity_id( $content_id ) {
-		Assertions::equals( $content_id->get_type(), $this->expected_object_type,
-			sprintf( '`content_id` must be of type `%s`.', Object_Type_Enum::to_string( $this->expected_object_type ) ) );
+    public function get_entity_id( $content_id ) {
+        Assertions::equals( $content_id->get_type(), $this->expected_object_type,
+            sprintf( '`content_id` must be of type `%s`.', Object_Type_Enum::to_string( $this->expected_object_type ) ) );
 
-		return call_user_func( $this->get_meta_fn, $content_id->get_id(), 'entity_url', true ) ?: null;
-	}
+        return call_user_func( $this->get_meta_fn, $content_id->get_id(), 'entity_url', true ) ?: null;
+    }
 
-	public function delete( $content_id ) {
-		// do nothing, WP deletes the post meta for us.
-	}
+    public function delete( $content_id ) {
+        // do nothing, WP deletes the post meta for us.
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -11,21 +11,21 @@
 	private $expected_object_type;
 	private $get_meta_fn;
 
-	protected function __construct( $expected_object_type, $get_meta_fn ) {
+	protected function __construct($expected_object_type, $get_meta_fn) {
 		parent::__construct();
 
 		$this->expected_object_type = $expected_object_type;
 		$this->get_meta_fn          = $get_meta_fn;
 	}
 
-	public function get_entity_id( $content_id ) {
-		Assertions::equals( $content_id->get_type(), $this->expected_object_type,
-			sprintf( '`content_id` must be of type `%s`.', Object_Type_Enum::to_string( $this->expected_object_type ) ) );
+	public function get_entity_id($content_id) {
+		Assertions::equals($content_id->get_type(), $this->expected_object_type,
+			sprintf('`content_id` must be of type `%s`.', Object_Type_Enum::to_string($this->expected_object_type)));
 
-		return call_user_func( $this->get_meta_fn, $content_id->get_id(), 'entity_url', true ) ?: null;
+		return call_user_func($this->get_meta_fn, $content_id->get_id(), 'entity_url', true) ?: null;
 	}
 
-	public function delete( $content_id ) {
+	public function delete($content_id) {
 		// do nothing, WP deletes the post meta for us.
 	}
 }

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

Indentation +125 added lines, -125 removed lines

@@ -8,62 +8,62 @@ 
 
 class Wordpress_Permalink_Content_Service implements Content_Service {
 
-	private static $instance = null;
-
-	protected function __construct() {
-	}
-
-	/**
-	 * The singleton instance.
-	 *
-	 * @return Content_Service
-	 */
-	public static function get_instance() {
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self();
-		}
-
-		return self::$instance;
-	}
-
-	/**
-	 * @param $uri string In the form https://example.org/path/post#post/1
-	 *
-	 * @return Wordpress_Content|null
-	 */
-	function get_by_entity_id( $uri ) {
-		if ( ! preg_match( '@.*#(\w+)/(\d+)@', $uri, $matches ) ) {
-			return null;
-		}
-
-		$type_name = $matches[1];
-		$id        = $matches[2];
-		switch ( Object_Type_Enum::from_string( $type_name ) ) {
-			case Object_Type_Enum::POST:
-				return new Wordpress_Content( get_post( $id ) );
-			case Object_Type_Enum::TERM:
-				return new Wordpress_Content( get_term( $id ) );
-			case Object_Type_Enum::USER:
-				return new Wordpress_Content( get_user_by( 'ID', $id ) );
-		}
-
-		return null;
-	}
-
-	/**
-	 * Get a
-	 * @throws Exception
-	 */
-	function get_by_entity_id_or_same_as( $uri ) {
-		// If the URL is in the local site URL, then try to find a corresponding post.
-		if ( 0 === strpos( $uri, site_url() ) ) {
-			return $this->get_by_entity_id( $uri );
-		}
-
-		// Otherwise look in sameAs.
-		global $wpdb;
-
-		$row = $wpdb->get_row( $wpdb->prepare( "
+    private static $instance = null;
+
+    protected function __construct() {
+    }
+
+    /**
+     * The singleton instance.
+     *
+     * @return Content_Service
+     */
+    public static function get_instance() {
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self();
+        }
+
+        return self::$instance;
+    }
+
+    /**
+     * @param $uri string In the form https://example.org/path/post#post/1
+     *
+     * @return Wordpress_Content|null
+     */
+    function get_by_entity_id( $uri ) {
+        if ( ! preg_match( '@.*#(\w+)/(\d+)@', $uri, $matches ) ) {
+            return null;
+        }
+
+        $type_name = $matches[1];
+        $id        = $matches[2];
+        switch ( Object_Type_Enum::from_string( $type_name ) ) {
+            case Object_Type_Enum::POST:
+                return new Wordpress_Content( get_post( $id ) );
+            case Object_Type_Enum::TERM:
+                return new Wordpress_Content( get_term( $id ) );
+            case Object_Type_Enum::USER:
+                return new Wordpress_Content( get_user_by( 'ID', $id ) );
+        }
+
+        return null;
+    }
+
+    /**
+     * Get a
+     * @throws Exception
+     */
+    function get_by_entity_id_or_same_as( $uri ) {
+        // If the URL is in the local site URL, then try to find a corresponding post.
+        if ( 0 === strpos( $uri, site_url() ) ) {
+            return $this->get_by_entity_id( $uri );
+        }
+
+        // Otherwise look in sameAs.
+        global $wpdb;
+
+        $row = $wpdb->get_row( $wpdb->prepare( "
 			SELECT content_type, content_id
 			FROM (
 			    SELECT %d AS content_type, post_id AS content_id
@@ -80,73 +80,73 @@ 
 			) _tmp_same_as 
 			LIMIT 1
 		",
-			Object_Type_Enum::POST, $uri,
-			Object_Type_Enum::TERM, $uri,
-			Object_Type_Enum::USER, $uri ) );
-
-		if ( ! isset( $row ) ) {
-			return null;
-		}
-
-		switch ( (int) $row->content_type ) {
-			case Object_Type_Enum::POST:
-				return new Wordpress_Content( get_post( $row->content_id ) );
-			case Object_Type_Enum::TERM:
-				return new Wordpress_Content( get_term( $row->content_id ) );
-			case Object_Type_Enum::USER:
-				return new Wordpress_Content( get_user_by( 'ID', $row->content_id ) );
-			default:
-				return null;
-		}
-
-	}
-
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 *
-	 * @return string|void|null
-	 */
-	function get_entity_id( $content_id ) {
-		$type = $content_id->get_type();
-		$id   = $content_id->get_id();
-
-		switch ( $type ) {
-			case Object_Type_Enum::POST:
-				$base_uri = get_permalink( $id );
-				break;
-			case Object_Type_Enum::TERM:
-				$base_uri = get_term_link( $id );
-				break;
-			case Object_Type_Enum::USER:
-				$base_uri = get_author_posts_url( $id );
-				break;
-			default:
-				return null;
-		}
-
-		$type_name = Object_Type_Enum::to_string( $type );
-
-		return "$base_uri#$type_name/$id";
-	}
-
-	function set_entity_id( $content_id, $uri ) {
-		throw new Exception( 'Not supported' );
-	}
-
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 *
-	 * @return bool|void
-	 */
-	function supports( $content_id ) {
-		return in_array( $content_id->get_type(), array(
-			Object_Type_Enum::POST,
-			Object_Type_Enum::TERM,
-			Object_Type_Enum::USER,
-		) );
-	}
-
-	function delete( $content_id ) {
-		// Ignore, we don't store any data.
-	}
+            Object_Type_Enum::POST, $uri,
+            Object_Type_Enum::TERM, $uri,
+            Object_Type_Enum::USER, $uri ) );
+
+        if ( ! isset( $row ) ) {
+            return null;
+        }
+
+        switch ( (int) $row->content_type ) {
+            case Object_Type_Enum::POST:
+                return new Wordpress_Content( get_post( $row->content_id ) );
+            case Object_Type_Enum::TERM:
+                return new Wordpress_Content( get_term( $row->content_id ) );
+            case Object_Type_Enum::USER:
+                return new Wordpress_Content( get_user_by( 'ID', $row->content_id ) );
+            default:
+                return null;
+        }
+
+    }
+
+    /**
+     * @param Wordpress_Content_Id $content_id
+     *
+     * @return string|void|null
+     */
+    function get_entity_id( $content_id ) {
+        $type = $content_id->get_type();
+        $id   = $content_id->get_id();
+
+        switch ( $type ) {
+            case Object_Type_Enum::POST:
+                $base_uri = get_permalink( $id );
+                break;
+            case Object_Type_Enum::TERM:
+                $base_uri = get_term_link( $id );
+                break;
+            case Object_Type_Enum::USER:
+                $base_uri = get_author_posts_url( $id );
+                break;
+            default:
+                return null;
+        }
+
+        $type_name = Object_Type_Enum::to_string( $type );
+
+        return "$base_uri#$type_name/$id";
+    }
+
+    function set_entity_id( $content_id, $uri ) {
+        throw new Exception( 'Not supported' );
+    }
+
+    /**
+     * @param Wordpress_Content_Id $content_id
+     *
+     * @return bool|void
+     */
+    function supports( $content_id ) {
+        return in_array( $content_id->get_type(), array(
+            Object_Type_Enum::POST,
+            Object_Type_Enum::TERM,
+            Object_Type_Enum::USER,
+        ) );
+    }
+
+    function delete( $content_id ) {
+        // Ignore, we don't store any data.
+    }
 }

Spacing +29 added lines, -29 removed lines

@@ -19,7 +19,7 @@ 
 	 * @return Content_Service
 	 */
 	public static function get_instance() {
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -31,20 +31,20 @@ 
 	 *
 	 * @return Wordpress_Content|null
 	 */
-	function get_by_entity_id( $uri ) {
-		if ( ! preg_match( '@.*#(\w+)/(\d+)@', $uri, $matches ) ) {
+	function get_by_entity_id($uri) {
+		if ( ! preg_match('@.*#(\w+)/(\d+)@', $uri, $matches)) {
 			return null;
 		}
 
 		$type_name = $matches[1];
 		$id        = $matches[2];
-		switch ( Object_Type_Enum::from_string( $type_name ) ) {
+		switch (Object_Type_Enum::from_string($type_name)) {
 			case Object_Type_Enum::POST:
-				return new Wordpress_Content( get_post( $id ) );
+				return new Wordpress_Content(get_post($id));
 			case Object_Type_Enum::TERM:
-				return new Wordpress_Content( get_term( $id ) );
+				return new Wordpress_Content(get_term($id));
 			case Object_Type_Enum::USER:
-				return new Wordpress_Content( get_user_by( 'ID', $id ) );
+				return new Wordpress_Content(get_user_by('ID', $id));
 		}
 
 		return null;
@@ -54,16 +54,16 @@ 
 	 * Get a
 	 * @throws Exception
 	 */
-	function get_by_entity_id_or_same_as( $uri ) {
+	function get_by_entity_id_or_same_as($uri) {
 		// If the URL is in the local site URL, then try to find a corresponding post.
-		if ( 0 === strpos( $uri, site_url() ) ) {
-			return $this->get_by_entity_id( $uri );
+		if (0 === strpos($uri, site_url())) {
+			return $this->get_by_entity_id($uri);
 		}
 
 		// Otherwise look in sameAs.
 		global $wpdb;
 
-		$row = $wpdb->get_row( $wpdb->prepare( "
+		$row = $wpdb->get_row($wpdb->prepare("
 			SELECT content_type, content_id
 			FROM (
 			    SELECT %d AS content_type, post_id AS content_id
@@ -82,19 +82,19 @@ 
 		",
 			Object_Type_Enum::POST, $uri,
 			Object_Type_Enum::TERM, $uri,
-			Object_Type_Enum::USER, $uri ) );
+			Object_Type_Enum::USER, $uri));
 
-		if ( ! isset( $row ) ) {
+		if ( ! isset($row)) {
 			return null;
 		}
 
-		switch ( (int) $row->content_type ) {
+		switch ((int) $row->content_type) {
 			case Object_Type_Enum::POST:
-				return new Wordpress_Content( get_post( $row->content_id ) );
+				return new Wordpress_Content(get_post($row->content_id));
 			case Object_Type_Enum::TERM:
-				return new Wordpress_Content( get_term( $row->content_id ) );
+				return new Wordpress_Content(get_term($row->content_id));
 			case Object_Type_Enum::USER:
-				return new Wordpress_Content( get_user_by( 'ID', $row->content_id ) );
+				return new Wordpress_Content(get_user_by('ID', $row->content_id));
 			default:
 				return null;
 		}
@@ -106,31 +106,31 @@ 
 	 *
 	 * @return string|void|null
 	 */
-	function get_entity_id( $content_id ) {
+	function get_entity_id($content_id) {
 		$type = $content_id->get_type();
 		$id   = $content_id->get_id();
 
-		switch ( $type ) {
+		switch ($type) {
 			case Object_Type_Enum::POST:
-				$base_uri = get_permalink( $id );
+				$base_uri = get_permalink($id);
 				break;
 			case Object_Type_Enum::TERM:
-				$base_uri = get_term_link( $id );
+				$base_uri = get_term_link($id);
 				break;
 			case Object_Type_Enum::USER:
-				$base_uri = get_author_posts_url( $id );
+				$base_uri = get_author_posts_url($id);
 				break;
 			default:
 				return null;
 		}
 
-		$type_name = Object_Type_Enum::to_string( $type );
+		$type_name = Object_Type_Enum::to_string($type);
 
 		return "$base_uri#$type_name/$id";
 	}
 
-	function set_entity_id( $content_id, $uri ) {
-		throw new Exception( 'Not supported' );
+	function set_entity_id($content_id, $uri) {
+		throw new Exception('Not supported');
 	}
 
 	/**
@@ -138,15 +138,15 @@ 
 	 *
 	 * @return bool|void
 	 */
-	function supports( $content_id ) {
-		return in_array( $content_id->get_type(), array(
+	function supports($content_id) {
+		return in_array($content_id->get_type(), array(
 			Object_Type_Enum::POST,
 			Object_Type_Enum::TERM,
 			Object_Type_Enum::USER,
-		) );
+		));
 	}
 
-	function delete( $content_id ) {
+	function delete($content_id) {
 		// Ignore, we don't store any data.
 	}
 }

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

Indentation +102 added lines, -102 removed lines

@@ -8,106 +8,106 @@
 
 class Wordpress_Dataset_Content_Service implements Content_Service {
 
-	/**
-	 * @var Content_Service[]
-	 */
-	private $delegates = array();
-
-	protected function __construct() {
-
-	}
-
-	private static $instance = null;
-
-	/**
-	 * The singleton instance.
-	 *
-	 * @return Content_Service
-	 */
-	public static function get_instance() {
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self();
-
-			self::$instance->register_delegate( Wordpress_Post_Content_Service::get_instance() );
-			self::$instance->register_delegate( Wordpress_Term_Content_Legacy_Service::get_instance() );
-			self::$instance->register_delegate( Wordpress_User_Content_Legacy_Service::get_instance() );
-
-			Wordpress_Dataset_Content_Service_Hooks::register();
-		}
-
-		return self::$instance;
-	}
-
-	public function register_delegate( $delegate ) {
-		Assertions::is_a( $delegate, 'Wordlift\Content\Content_Service', 'A `delegate` must implement the `Wordlift\Content\Content_Service` interface.' );
-
-		$this->delegates[] = $delegate;
-	}
-
-	function get_by_entity_id( $uri ) {
-		foreach ( $this->delegates as $delegate ) {
-			$content = $delegate->get_by_entity_id_or_same_as( $uri );
-			if ( isset( $content ) ) {
-				return $content;
-			}
-		}
-
-		return null;
-	}
-
-	/**
-	 * Get a
-	 * @throws Exception
-	 */
-	function get_by_entity_id_or_same_as( $uri ) {
-		foreach ( $this->delegates as $delegate ) {
-			$content = $delegate->get_by_entity_id_or_same_as( $uri );
-			if ( isset( $content ) ) {
-				return $content;
-			}
-		}
-
-		return null;
-	}
-
-	function get_entity_id( $content_id ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) && $uri = $delegate->get_entity_id( $content_id ) ) {
-				return $uri;
-			}
-		}
-
-		return null;
-	}
-
-	function set_entity_id( $content_id, $uri ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) ) {
-				$delegate->set_entity_id( $content_id, $uri );
-
-				return;
-			}
-		}
-
-		throw new Exception( 'Not supported' );
-	}
-
-	function supports( $content_id ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) ) {
-				return true;
-			}
-		}
-
-		return false;
-	}
-
-	function delete( $content_id ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) ) {
-				$delegate->delete( $content_id );
-				break;
-			}
-		}
-	}
+    /**
+     * @var Content_Service[]
+     */
+    private $delegates = array();
+
+    protected function __construct() {
+
+    }
+
+    private static $instance = null;
+
+    /**
+     * The singleton instance.
+     *
+     * @return Content_Service
+     */
+    public static function get_instance() {
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self();
+
+            self::$instance->register_delegate( Wordpress_Post_Content_Service::get_instance() );
+            self::$instance->register_delegate( Wordpress_Term_Content_Legacy_Service::get_instance() );
+            self::$instance->register_delegate( Wordpress_User_Content_Legacy_Service::get_instance() );
+
+            Wordpress_Dataset_Content_Service_Hooks::register();
+        }
+
+        return self::$instance;
+    }
+
+    public function register_delegate( $delegate ) {
+        Assertions::is_a( $delegate, 'Wordlift\Content\Content_Service', 'A `delegate` must implement the `Wordlift\Content\Content_Service` interface.' );
+
+        $this->delegates[] = $delegate;
+    }
+
+    function get_by_entity_id( $uri ) {
+        foreach ( $this->delegates as $delegate ) {
+            $content = $delegate->get_by_entity_id_or_same_as( $uri );
+            if ( isset( $content ) ) {
+                return $content;
+            }
+        }
+
+        return null;
+    }
+
+    /**
+     * Get a
+     * @throws Exception
+     */
+    function get_by_entity_id_or_same_as( $uri ) {
+        foreach ( $this->delegates as $delegate ) {
+            $content = $delegate->get_by_entity_id_or_same_as( $uri );
+            if ( isset( $content ) ) {
+                return $content;
+            }
+        }
+
+        return null;
+    }
+
+    function get_entity_id( $content_id ) {
+        foreach ( $this->delegates as $delegate ) {
+            if ( $delegate->supports( $content_id ) && $uri = $delegate->get_entity_id( $content_id ) ) {
+                return $uri;
+            }
+        }
+
+        return null;
+    }
+
+    function set_entity_id( $content_id, $uri ) {
+        foreach ( $this->delegates as $delegate ) {
+            if ( $delegate->supports( $content_id ) ) {
+                $delegate->set_entity_id( $content_id, $uri );
+
+                return;
+            }
+        }
+
+        throw new Exception( 'Not supported' );
+    }
+
+    function supports( $content_id ) {
+        foreach ( $this->delegates as $delegate ) {
+            if ( $delegate->supports( $content_id ) ) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
+    function delete( $content_id ) {
+        foreach ( $this->delegates as $delegate ) {
+            if ( $delegate->supports( $content_id ) ) {
+                $delegate->delete( $content_id );
+                break;
+            }
+        }
+    }
 }

Spacing +29 added lines, -29 removed lines

@@ -25,12 +25,12 @@ 
 	 * @return Content_Service
 	 */
 	public static function get_instance() {
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			self::$instance = new self();
 
-			self::$instance->register_delegate( Wordpress_Post_Content_Service::get_instance() );
-			self::$instance->register_delegate( Wordpress_Term_Content_Legacy_Service::get_instance() );
-			self::$instance->register_delegate( Wordpress_User_Content_Legacy_Service::get_instance() );
+			self::$instance->register_delegate(Wordpress_Post_Content_Service::get_instance());
+			self::$instance->register_delegate(Wordpress_Term_Content_Legacy_Service::get_instance());
+			self::$instance->register_delegate(Wordpress_User_Content_Legacy_Service::get_instance());
 
 			Wordpress_Dataset_Content_Service_Hooks::register();
 		}
@@ -38,16 +38,16 @@ 
 		return self::$instance;
 	}
 
-	public function register_delegate( $delegate ) {
-		Assertions::is_a( $delegate, 'Wordlift\Content\Content_Service', 'A `delegate` must implement the `Wordlift\Content\Content_Service` interface.' );
+	public function register_delegate($delegate) {
+		Assertions::is_a($delegate, 'Wordlift\Content\Content_Service', 'A `delegate` must implement the `Wordlift\Content\Content_Service` interface.');
 
 		$this->delegates[] = $delegate;
 	}
 
-	function get_by_entity_id( $uri ) {
-		foreach ( $this->delegates as $delegate ) {
-			$content = $delegate->get_by_entity_id_or_same_as( $uri );
-			if ( isset( $content ) ) {
+	function get_by_entity_id($uri) {
+		foreach ($this->delegates as $delegate) {
+			$content = $delegate->get_by_entity_id_or_same_as($uri);
+			if (isset($content)) {
 				return $content;
 			}
 		}
@@ -59,10 +59,10 @@ 
 	 * Get a
 	 * @throws Exception
 	 */
-	function get_by_entity_id_or_same_as( $uri ) {
-		foreach ( $this->delegates as $delegate ) {
-			$content = $delegate->get_by_entity_id_or_same_as( $uri );
-			if ( isset( $content ) ) {
+	function get_by_entity_id_or_same_as($uri) {
+		foreach ($this->delegates as $delegate) {
+			$content = $delegate->get_by_entity_id_or_same_as($uri);
+			if (isset($content)) {
 				return $content;
 			}
 		}
@@ -70,9 +70,9 @@ 
 		return null;
 	}
 
-	function get_entity_id( $content_id ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) && $uri = $delegate->get_entity_id( $content_id ) ) {
+	function get_entity_id($content_id) {
+		foreach ($this->delegates as $delegate) {
+			if ($delegate->supports($content_id) && $uri = $delegate->get_entity_id($content_id)) {
 				return $uri;
 			}
 		}
@@ -80,21 +80,21 @@ 
 		return null;
 	}
 
-	function set_entity_id( $content_id, $uri ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) ) {
-				$delegate->set_entity_id( $content_id, $uri );
+	function set_entity_id($content_id, $uri) {
+		foreach ($this->delegates as $delegate) {
+			if ($delegate->supports($content_id)) {
+				$delegate->set_entity_id($content_id, $uri);
 
 				return;
 			}
 		}
 
-		throw new Exception( 'Not supported' );
+		throw new Exception('Not supported');
 	}
 
-	function supports( $content_id ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) ) {
+	function supports($content_id) {
+		foreach ($this->delegates as $delegate) {
+			if ($delegate->supports($content_id)) {
 				return true;
 			}
 		}
@@ -102,10 +102,10 @@ 
 		return false;
 	}
 
-	function delete( $content_id ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) ) {
-				$delegate->delete( $content_id );
+	function delete($content_id) {
+		foreach ($this->delegates as $delegate) {
+			if ($delegate->supports($content_id)) {
+				$delegate->delete($content_id);
 				break;
 			}
 		}

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

Indentation +50 added lines, -50 removed lines

@@ -10,59 +10,59 @@
 
 interface Content_Service {
 
-	/**
-	 * Get an {@link Content} given a URI.
-	 *
-	 * @param string $uri The URI.
-	 *
-	 * @return Content|null The found {@link Content} or null if not found.
-	 * @throws Exception if the URI is absolute and not within the dataset URI scope or the dataset URI isn't set.
-	 *
-	 */
-	function get_by_entity_id( $uri );
+    /**
+     * Get an {@link Content} given a URI.
+     *
+     * @param string $uri The URI.
+     *
+     * @return Content|null The found {@link Content} or null if not found.
+     * @throws Exception if the URI is absolute and not within the dataset URI scope or the dataset URI isn't set.
+     *
+     */
+    function get_by_entity_id( $uri );
 
-	/**
-	 * Get an {@link Content} given a URI. The search is performed also in sameAs.
-	 *
-	 * @param string $uri The URI.
-	 *
-	 * @return Content|null The found {@link Content} or null if not found.
-	 */
-	function get_by_entity_id_or_same_as( $uri );
+    /**
+     * Get an {@link Content} given a URI. The search is performed also in sameAs.
+     *
+     * @param string $uri The URI.
+     *
+     * @return Content|null The found {@link Content} or null if not found.
+     */
+    function get_by_entity_id_or_same_as( $uri );
 
-	/**
-	 * Get the {@link Content}'s URI given an {@link Content_Id}.
-	 *
-	 * @param Content_Id $content_id An {@link Content_Id}.
-	 *
-	 * @return string|null An absolute URI or null if not found.
-	 */
-	function get_entity_id( $content_id );
+    /**
+     * Get the {@link Content}'s URI given an {@link Content_Id}.
+     *
+     * @param Content_Id $content_id An {@link Content_Id}.
+     *
+     * @return string|null An absolute URI or null if not found.
+     */
+    function get_entity_id( $content_id );
 
-	/**
-	 * Set the {@link Content}'s URI for the specified {@link Content_Id}.
-	 *
-	 * @param Content_Id $content_id An {@link Content_Id}.
-	 * @param string $uri The URI.
-	 *
-	 * @return void
-	 * @throws Exception if the URI is absolute and not within the dataset URI scope or the dataset URI isn't set.
-	 */
-	function set_entity_id( $content_id, $uri );
+    /**
+     * Set the {@link Content}'s URI for the specified {@link Content_Id}.
+     *
+     * @param Content_Id $content_id An {@link Content_Id}.
+     * @param string $uri The URI.
+     *
+     * @return void
+     * @throws Exception if the URI is absolute and not within the dataset URI scope or the dataset URI isn't set.
+     */
+    function set_entity_id( $content_id, $uri );
 
-	/**
-	 * Whether the {@link Content_Service} supports the provided {@link Content_Id}.
-	 *
-	 * @param Content_Id $content_id
-	 *
-	 * @return bool
-	 */
-	function supports( $content_id );
+    /**
+     * Whether the {@link Content_Service} supports the provided {@link Content_Id}.
+     *
+     * @param Content_Id $content_id
+     *
+     * @return bool
+     */
+    function supports( $content_id );
 
-	/**
-	 * Delete the content with the specified ID.
-	 *
-	 * @param Content_Id $content_id
-	 */
-	function delete( $content_id );
+    /**
+     * Delete the content with the specified ID.
+     *
+     * @param Content_Id $content_id
+     */
+    function delete( $content_id );
 }

Spacing +6 added lines, -6 removed lines

@@ -19,7 +19,7 @@ 
 	 * @throws Exception if the URI is absolute and not within the dataset URI scope or the dataset URI isn't set.
 	 *
 	 */
-	function get_by_entity_id( $uri );
+	function get_by_entity_id($uri);
 
 	/**
 	 * Get an {@link Content} given a URI. The search is performed also in sameAs.
@@ -28,7 +28,7 @@ 
 	 *
 	 * @return Content|null The found {@link Content} or null if not found.
 	 */
-	function get_by_entity_id_or_same_as( $uri );
+	function get_by_entity_id_or_same_as($uri);
 
 	/**
 	 * Get the {@link Content}'s URI given an {@link Content_Id}.
@@ -37,7 +37,7 @@ 
 	 *
 	 * @return string|null An absolute URI or null if not found.
 	 */
-	function get_entity_id( $content_id );
+	function get_entity_id($content_id);
 
 	/**
 	 * Set the {@link Content}'s URI for the specified {@link Content_Id}.
@@ -48,7 +48,7 @@ 
 	 * @return void
 	 * @throws Exception if the URI is absolute and not within the dataset URI scope or the dataset URI isn't set.
 	 */
-	function set_entity_id( $content_id, $uri );
+	function set_entity_id($content_id, $uri);
 
 	/**
 	 * Whether the {@link Content_Service} supports the provided {@link Content_Id}.
@@ -57,12 +57,12 @@ 
 	 *
 	 * @return bool
 	 */
-	function supports( $content_id );
+	function supports($content_id);
 
 	/**
 	 * Delete the content with the specified ID.
 	 *
 	 * @param Content_Id $content_id
 	 */
-	function delete( $content_id );
+	function delete($content_id);
 }

src/wordlift.php

Indentation +126 added lines, -126 removed lines

@@ -33,7 +33,7 @@ 
 
 // If this file is called directly, abort.
 if ( ! defined( 'WPINC' ) ) {
-	die;
+    die;
 }
 
 /**
@@ -42,7 +42,7 @@ 
  *
  */
 if ( ! apply_filters( 'wl_is_enabled', true ) ) {
-	return;
+    return;
 }
 
 require_once plugin_dir_path( __FILE__ ) . 'vendor/autoload.php';
@@ -72,29 +72,29 @@ 
  */
 function activate_wordlift() {
 
-	$log = Wordlift_Log_Service::get_logger( 'activate_wordlift' );
-
-	$log->info( 'Activating WordLift...' );
-
-	require_once plugin_dir_path( __FILE__ ) . 'includes/class-wordlift-activator.php';
-	Wordlift_Activator::activate();
-
-	/**
-	 * Tell the {@link Wordlift_Http_Api} class that we're activating, to let it run activation tasks.
-	 *
-	 * @see https://github.com/insideout10/wordlift-plugin/issues/820 related issue.
-	 * @since 3.19.2
-	 */
-	Wordlift_Http_Api::activate();
-
-	// Ensure the post type is registered before flushing the rewrite rules.
-	Wordlift_Entity_Post_Type_Service::get_instance()->register();
-	flush_rewrite_rules();
-	/**
-	 * @since 3.27.7
-	 * @see https://github.com/insideout10/wordlift-plugin/issues/1214
-	 */
-	Top_Entities::activate();
+    $log = Wordlift_Log_Service::get_logger( 'activate_wordlift' );
+
+    $log->info( 'Activating WordLift...' );
+
+    require_once plugin_dir_path( __FILE__ ) . 'includes/class-wordlift-activator.php';
+    Wordlift_Activator::activate();
+
+    /**
+     * Tell the {@link Wordlift_Http_Api} class that we're activating, to let it run activation tasks.
+     *
+     * @see https://github.com/insideout10/wordlift-plugin/issues/820 related issue.
+     * @since 3.19.2
+     */
+    Wordlift_Http_Api::activate();
+
+    // Ensure the post type is registered before flushing the rewrite rules.
+    Wordlift_Entity_Post_Type_Service::get_instance()->register();
+    flush_rewrite_rules();
+    /**
+     * @since 3.27.7
+     * @see https://github.com/insideout10/wordlift-plugin/issues/1214
+     */
+    Top_Entities::activate();
 }
 
 /**
@@ -103,21 +103,21 @@ 
  */
 function deactivate_wordlift() {
 
-	require_once plugin_dir_path( __FILE__ ) . 'includes/class-wordlift-deactivator.php';
-	Wordlift_Deactivator::deactivate();
-	Wordlift_Http_Api::deactivate();
-	Ttl_Cache_Cleaner::deactivate();
-	/**
-	 * @since 3.27.7
-	 * @see https://github.com/insideout10/wordlift-plugin/issues/1214
-	 */
-	Top_Entities::deactivate();
-	/**
-	 * @since 3.27.8
-	 * Remove notification flag on deactivation.
-	 */
-	Key_Validation_Notice::remove_notification_flag();
-	flush_rewrite_rules();
+    require_once plugin_dir_path( __FILE__ ) . 'includes/class-wordlift-deactivator.php';
+    Wordlift_Deactivator::deactivate();
+    Wordlift_Http_Api::deactivate();
+    Ttl_Cache_Cleaner::deactivate();
+    /**
+     * @since 3.27.7
+     * @see https://github.com/insideout10/wordlift-plugin/issues/1214
+     */
+    Top_Entities::deactivate();
+    /**
+     * @since 3.27.8
+     * Remove notification flag on deactivation.
+     */
+    Key_Validation_Notice::remove_notification_flag();
+    flush_rewrite_rules();
 
 }
 
@@ -140,70 +140,70 @@ 
  * @since    1.0.0
  */
 function run_wordlift() {
-	/**
-	 * Filter: wl_feature__enable__widgets.
-	 *
-	 * @param bool whether the widgets needed to be registered, defaults to true.
-	 *
-	 * @return bool
-	 * @since 3.27.6
-	 */
-	if ( apply_filters( 'wl_feature__enable__widgets', true ) ) {
-		add_action( 'widgets_init', 'wl_register_chord_widget' );
-		add_action( 'widgets_init', 'wl_register_geo_widget' );
-		add_action( 'widgets_init', 'wl_register_timeline_widget' );
-	}
-	add_filter( 'widget_text', 'do_shortcode' );
-
-
-	/**
-	 * Filter: wl_feature__enable__analysis
-	 *
-	 * @param bool Whether to send api request to analysis or not
-	 *
-	 * @return bool
-	 * @since 3.27.6
-	 */
-	if ( apply_filters( 'wl_feature__enable__analysis', true ) ) {
-		add_action( 'wp_ajax_wl_analyze', 'wl_ajax_analyze_action' );
-	} else {
-		add_action( 'wp_ajax_wl_analyze', 'wl_ajax_analyze_disabled_action' );
-	}
-
-	$plugin = new Wordlift();
-	$plugin->run();
-
-	// Initialize the TTL Cache Cleaner.
-	new Ttl_Cache_Cleaner();
-
-	// Load the new Post Adapter.
-	new Post_Adapter();
-
-	// Load the API Data Hooks.
-	new Api_Data_Hooks();
-
-	add_action( 'plugins_loaded', function () {
-		// All features from registry should be initialized here.
-		$features_registry = Features_Registry::get_instance();
-		$features_registry->initialize_all_features();
-	}, 5 );
-
-	add_action( 'plugins_loaded', function () use ( $plugin ) {
-
-		new Wordlift_Products_Navigator_Shortcode_REST();
-
-		// Register the Dataset module, requires `$api_service`.
-		require_once plugin_dir_path( __FILE__ ) . 'wordlift/dataset/index.php';
-		require_once plugin_dir_path( __FILE__ ) . 'wordlift/shipping-data/index.php';
-
-		/*
+    /**
+     * Filter: wl_feature__enable__widgets.
+     *
+     * @param bool whether the widgets needed to be registered, defaults to true.
+     *
+     * @return bool
+     * @since 3.27.6
+     */
+    if ( apply_filters( 'wl_feature__enable__widgets', true ) ) {
+        add_action( 'widgets_init', 'wl_register_chord_widget' );
+        add_action( 'widgets_init', 'wl_register_geo_widget' );
+        add_action( 'widgets_init', 'wl_register_timeline_widget' );
+    }
+    add_filter( 'widget_text', 'do_shortcode' );
+
+
+    /**
+     * Filter: wl_feature__enable__analysis
+     *
+     * @param bool Whether to send api request to analysis or not
+     *
+     * @return bool
+     * @since 3.27.6
+     */
+    if ( apply_filters( 'wl_feature__enable__analysis', true ) ) {
+        add_action( 'wp_ajax_wl_analyze', 'wl_ajax_analyze_action' );
+    } else {
+        add_action( 'wp_ajax_wl_analyze', 'wl_ajax_analyze_disabled_action' );
+    }
+
+    $plugin = new Wordlift();
+    $plugin->run();
+
+    // Initialize the TTL Cache Cleaner.
+    new Ttl_Cache_Cleaner();
+
+    // Load the new Post Adapter.
+    new Post_Adapter();
+
+    // Load the API Data Hooks.
+    new Api_Data_Hooks();
+
+    add_action( 'plugins_loaded', function () {
+        // All features from registry should be initialized here.
+        $features_registry = Features_Registry::get_instance();
+        $features_registry->initialize_all_features();
+    }, 5 );
+
+    add_action( 'plugins_loaded', function () use ( $plugin ) {
+
+        new Wordlift_Products_Navigator_Shortcode_REST();
+
+        // Register the Dataset module, requires `$api_service`.
+        require_once plugin_dir_path( __FILE__ ) . 'wordlift/dataset/index.php';
+        require_once plugin_dir_path( __FILE__ ) . 'wordlift/shipping-data/index.php';
+
+        /*
 		 * Require the Entity annotation cleanup module.
 		 *
 		 * @since 3.34.6
 		 */
-		require_once plugin_dir_path( __FILE__ ) . 'wordlift/cleanup/index.php';
+        require_once plugin_dir_path( __FILE__ ) . 'wordlift/cleanup/index.php';
 
-	} );
+    } );
 
 }
 
@@ -217,50 +217,50 @@ 
  */
 function wordlift_plugin_autoload_register() {
 
-	spl_autoload_register( function ( $class_name ) {
+    spl_autoload_register( function ( $class_name ) {
 
-		// Bail out if these are not our classes.
-		if ( 0 !== strpos( $class_name, 'Wordlift\\' ) ) {
-			return false;
-		}
+        // Bail out if these are not our classes.
+        if ( 0 !== strpos( $class_name, 'Wordlift\\' ) ) {
+            return false;
+        }
 
-		$class_name_lc = strtolower( str_replace( '_', '-', $class_name ) );
+        $class_name_lc = strtolower( str_replace( '_', '-', $class_name ) );
 
-		preg_match( '|^(?:(.*)\\\\)?(.+?)$|', $class_name_lc, $matches );
+        preg_match( '|^(?:(.*)\\\\)?(.+?)$|', $class_name_lc, $matches );
 
-		$path = str_replace( '\\', DIRECTORY_SEPARATOR, $matches[1] );
-		$file = 'class-' . $matches[2] . '.php';
+        $path = str_replace( '\\', DIRECTORY_SEPARATOR, $matches[1] );
+        $file = 'class-' . $matches[2] . '.php';
 
-		$full_path = plugin_dir_path( __FILE__ ) . $path . DIRECTORY_SEPARATOR . $file;
+        $full_path = plugin_dir_path( __FILE__ ) . $path . DIRECTORY_SEPARATOR . $file;
 
-		if ( ! file_exists( $full_path ) ) {
-			echo esc_html( "Class $class_name not found at $full_path." );
+        if ( ! file_exists( $full_path ) ) {
+            echo esc_html( "Class $class_name not found at $full_path." );
 
-			return false;
-		}
+            return false;
+        }
 
-		require_once $full_path;
+        require_once $full_path;
 
-		return true;
-	} );
+        return true;
+    } );
 
 }
 
 function wl_block_categories( $categories, $post ) {
-	return array_merge(
-		$categories,
-		array(
-			array(
-				'slug'  => 'wordlift',
-				'title' => __( 'WordLift', 'wordlift' ),
-			),
-		)
-	);
+    return array_merge(
+        $categories,
+        array(
+            array(
+                'slug'  => 'wordlift',
+                'title' => __( 'WordLift', 'wordlift' ),
+            ),
+        )
+    );
 }
 
 add_filter( 'block_categories', 'wl_block_categories', 10, 2 );
 
 // Temporary fix for a typo in WooCommerce Extension.
 add_filter( 'wl_feature__enable__dataset', function ( $value ) {
-	return apply_filters( 'wl_features__enable__dataset', $value );
+    return apply_filters( 'wl_features__enable__dataset', $value );
 } );

Spacing +42 added lines, -42 removed lines

@@ -32,7 +32,7 @@ 
 use Wordlift\Post\Post_Adapter;
 
 // If this file is called directly, abort.
-if ( ! defined( 'WPINC' ) ) {
+if ( ! defined('WPINC')) {
 	die;
 }
 
@@ -41,11 +41,11 @@ 
  * @since 3.33.6
  *
  */
-if ( ! apply_filters( 'wl_is_enabled', true ) ) {
+if ( ! apply_filters('wl_is_enabled', true)) {
 	return;
 }
 
-require_once plugin_dir_path( __FILE__ ) . 'vendor/autoload.php';
+require_once plugin_dir_path(__FILE__).'vendor/autoload.php';
 
 /*
 	 * We introduce the WordLift autoloader, since we start using classes in namespaces, i.e. Wordlift\Http.
@@ -56,15 +56,15 @@ 
 
 
 // Include WordLift constants.
-require_once( 'wordlift_constants.php' );
+require_once('wordlift_constants.php');
 
 // Load modules.
-require_once( 'modules/core/wordlift_core.php' );
+require_once('modules/core/wordlift_core.php');
 
-require_once( 'deprecations.php' );
+require_once('deprecations.php');
 
 // Load early to enable/disable features.
-require_once plugin_dir_path( __FILE__ ) . 'wordlift/features/index.php';
+require_once plugin_dir_path(__FILE__).'wordlift/features/index.php';
 
 /**
  * The code that runs during plugin activation.
@@ -72,11 +72,11 @@ 
  */
 function activate_wordlift() {
 
-	$log = Wordlift_Log_Service::get_logger( 'activate_wordlift' );
+	$log = Wordlift_Log_Service::get_logger('activate_wordlift');
 
-	$log->info( 'Activating WordLift...' );
+	$log->info('Activating WordLift...');
 
-	require_once plugin_dir_path( __FILE__ ) . 'includes/class-wordlift-activator.php';
+	require_once plugin_dir_path(__FILE__).'includes/class-wordlift-activator.php';
 	Wordlift_Activator::activate();
 
 	/**
@@ -103,7 +103,7 @@ 
  */
 function deactivate_wordlift() {
 
-	require_once plugin_dir_path( __FILE__ ) . 'includes/class-wordlift-deactivator.php';
+	require_once plugin_dir_path(__FILE__).'includes/class-wordlift-deactivator.php';
 	Wordlift_Deactivator::deactivate();
 	Wordlift_Http_Api::deactivate();
 	Ttl_Cache_Cleaner::deactivate();
@@ -121,14 +121,14 @@ 
 
 }
 
-register_activation_hook( __FILE__, 'activate_wordlift' );
-register_deactivation_hook( __FILE__, 'deactivate_wordlift' );
+register_activation_hook(__FILE__, 'activate_wordlift');
+register_deactivation_hook(__FILE__, 'deactivate_wordlift');
 
 /**
  * The core plugin class that is used to define internationalization,
  * admin-specific hooks, and public-facing site hooks.
  */
-require plugin_dir_path( __FILE__ ) . 'includes/class-wordlift.php';
+require plugin_dir_path(__FILE__).'includes/class-wordlift.php';
 
 /**
  * Begins execution of the plugin.
@@ -148,12 +148,12 @@ 
 	 * @return bool
 	 * @since 3.27.6
 	 */
-	if ( apply_filters( 'wl_feature__enable__widgets', true ) ) {
-		add_action( 'widgets_init', 'wl_register_chord_widget' );
-		add_action( 'widgets_init', 'wl_register_geo_widget' );
-		add_action( 'widgets_init', 'wl_register_timeline_widget' );
+	if (apply_filters('wl_feature__enable__widgets', true)) {
+		add_action('widgets_init', 'wl_register_chord_widget');
+		add_action('widgets_init', 'wl_register_geo_widget');
+		add_action('widgets_init', 'wl_register_timeline_widget');
 	}
-	add_filter( 'widget_text', 'do_shortcode' );
+	add_filter('widget_text', 'do_shortcode');
 
 
 	/**
@@ -164,10 +164,10 @@ 
 	 * @return bool
 	 * @since 3.27.6
 	 */
-	if ( apply_filters( 'wl_feature__enable__analysis', true ) ) {
-		add_action( 'wp_ajax_wl_analyze', 'wl_ajax_analyze_action' );
+	if (apply_filters('wl_feature__enable__analysis', true)) {
+		add_action('wp_ajax_wl_analyze', 'wl_ajax_analyze_action');
 	} else {
-		add_action( 'wp_ajax_wl_analyze', 'wl_ajax_analyze_disabled_action' );
+		add_action('wp_ajax_wl_analyze', 'wl_ajax_analyze_disabled_action');
 	}
 
 	$plugin = new Wordlift();
@@ -182,26 +182,26 @@ 
 	// Load the API Data Hooks.
 	new Api_Data_Hooks();
 
-	add_action( 'plugins_loaded', function () {
+	add_action('plugins_loaded', function() {
 		// All features from registry should be initialized here.
 		$features_registry = Features_Registry::get_instance();
 		$features_registry->initialize_all_features();
-	}, 5 );
+	}, 5);
 
-	add_action( 'plugins_loaded', function () use ( $plugin ) {
+	add_action('plugins_loaded', function() use ($plugin) {
 
 		new Wordlift_Products_Navigator_Shortcode_REST();
 
 		// Register the Dataset module, requires `$api_service`.
-		require_once plugin_dir_path( __FILE__ ) . 'wordlift/dataset/index.php';
-		require_once plugin_dir_path( __FILE__ ) . 'wordlift/shipping-data/index.php';
+		require_once plugin_dir_path(__FILE__).'wordlift/dataset/index.php';
+		require_once plugin_dir_path(__FILE__).'wordlift/shipping-data/index.php';
 
 		/*
 		 * Require the Entity annotation cleanup module.
 		 *
 		 * @since 3.34.6
 		 */
-		require_once plugin_dir_path( __FILE__ ) . 'wordlift/cleanup/index.php';
+		require_once plugin_dir_path(__FILE__).'wordlift/cleanup/index.php';
 
 	} );
 
@@ -217,24 +217,24 @@ 
  */
 function wordlift_plugin_autoload_register() {
 
-	spl_autoload_register( function ( $class_name ) {
+	spl_autoload_register(function($class_name) {
 
 		// Bail out if these are not our classes.
-		if ( 0 !== strpos( $class_name, 'Wordlift\\' ) ) {
+		if (0 !== strpos($class_name, 'Wordlift\\')) {
 			return false;
 		}
 
-		$class_name_lc = strtolower( str_replace( '_', '-', $class_name ) );
+		$class_name_lc = strtolower(str_replace('_', '-', $class_name));
 
-		preg_match( '|^(?:(.*)\\\\)?(.+?)$|', $class_name_lc, $matches );
+		preg_match('|^(?:(.*)\\\\)?(.+?)$|', $class_name_lc, $matches);
 
-		$path = str_replace( '\\', DIRECTORY_SEPARATOR, $matches[1] );
-		$file = 'class-' . $matches[2] . '.php';
+		$path = str_replace('\\', DIRECTORY_SEPARATOR, $matches[1]);
+		$file = 'class-'.$matches[2].'.php';
 
-		$full_path = plugin_dir_path( __FILE__ ) . $path . DIRECTORY_SEPARATOR . $file;
+		$full_path = plugin_dir_path(__FILE__).$path.DIRECTORY_SEPARATOR.$file;
 
-		if ( ! file_exists( $full_path ) ) {
-			echo esc_html( "Class $class_name not found at $full_path." );
+		if ( ! file_exists($full_path)) {
+			echo esc_html("Class $class_name not found at $full_path.");
 
 			return false;
 		}
@@ -246,21 +246,21 @@ 
 
 }
 
-function wl_block_categories( $categories, $post ) {
+function wl_block_categories($categories, $post) {
 	return array_merge(
 		$categories,
 		array(
 			array(
 				'slug'  => 'wordlift',
-				'title' => __( 'WordLift', 'wordlift' ),
+				'title' => __('WordLift', 'wordlift'),
 			),
 		)
 	);
 }
 
-add_filter( 'block_categories', 'wl_block_categories', 10, 2 );
+add_filter('block_categories', 'wl_block_categories', 10, 2);
 
 // Temporary fix for a typo in WooCommerce Extension.
-add_filter( 'wl_feature__enable__dataset', function ( $value ) {
-	return apply_filters( 'wl_features__enable__dataset', $value );
+add_filter('wl_feature__enable__dataset', function($value) {
+	return apply_filters('wl_features__enable__dataset', $value);
 } );

src/wordlift/task/class-all-posts-task.php

Indentation +37 added lines, -37 removed lines

@@ -11,46 +11,46 @@
 
 class All_Posts_Task implements Task {
 
-	private $callable;
-
-	public function __construct( $callable ) {
-		$this->callable = $callable;
-	}
-
-	public function starting() {
-		global $wpdb;
-
-		// Try to get the count from transient, or load it from database.
-		if ( false === ( $count = get_transient( '_wl_task__all_posts_task__count' ) ) ) {
-			$count = $wpdb->get_var( "SELECT COUNT( 1 ) FROM $wpdb->posts" );
-			set_transient( '_wl_task__all_posts_task__count', $count, HOUR_IN_SECONDS );
-		}
-
-		return $count;
-	}
-
-	/**
-	 * @param $value mixed The incoming value.
-	 * @param $args {
-	 *
-	 * @type int $offset The index of the current item.
-	 * @type int $count The total number of items as provided by the `starting` function.
-	 * @type int $batch_size The number of items to process within this call.
-	 * }
-	 *
-	 * @return void
-	 */
-	public function tick( $value, $args ) {
-		global $wpdb;
-
-		$ids = $wpdb->get_col( $wpdb->prepare( "
+    private $callable;
+
+    public function __construct( $callable ) {
+        $this->callable = $callable;
+    }
+
+    public function starting() {
+        global $wpdb;
+
+        // Try to get the count from transient, or load it from database.
+        if ( false === ( $count = get_transient( '_wl_task__all_posts_task__count' ) ) ) {
+            $count = $wpdb->get_var( "SELECT COUNT( 1 ) FROM $wpdb->posts" );
+            set_transient( '_wl_task__all_posts_task__count', $count, HOUR_IN_SECONDS );
+        }
+
+        return $count;
+    }
+
+    /**
+     * @param $value mixed The incoming value.
+     * @param $args {
+     *
+     * @type int $offset The index of the current item.
+     * @type int $count The total number of items as provided by the `starting` function.
+     * @type int $batch_size The number of items to process within this call.
+     * }
+     *
+     * @return void
+     */
+    public function tick( $value, $args ) {
+        global $wpdb;
+
+        $ids = $wpdb->get_col( $wpdb->prepare( "
 			SELECT ID FROM wp_posts ORDER BY ID LIMIT %d,%d;
 		", $args['offset'], $args['batch_size'] ) );
 
-		foreach ( $ids as $id ) {
-			call_user_func( $this->callable, $id );
-		}
+        foreach ( $ids as $id ) {
+            call_user_func( $this->callable, $id );
+        }
 
-	}
+    }
 
 }

Spacing +9 added lines, -9 removed lines

@@ -13,7 +13,7 @@ 
 
 	private $callable;
 
-	public function __construct( $callable ) {
+	public function __construct($callable) {
 		$this->callable = $callable;
 	}
 
@@ -21,9 +21,9 @@ 
 		global $wpdb;
 
 		// Try to get the count from transient, or load it from database.
-		if ( false === ( $count = get_transient( '_wl_task__all_posts_task__count' ) ) ) {
-			$count = $wpdb->get_var( "SELECT COUNT( 1 ) FROM $wpdb->posts" );
-			set_transient( '_wl_task__all_posts_task__count', $count, HOUR_IN_SECONDS );
+		if (false === ($count = get_transient('_wl_task__all_posts_task__count'))) {
+			$count = $wpdb->get_var("SELECT COUNT( 1 ) FROM $wpdb->posts");
+			set_transient('_wl_task__all_posts_task__count', $count, HOUR_IN_SECONDS);
 		}
 
 		return $count;
@@ -40,15 +40,15 @@ 
 	 *
 	 * @return void
 	 */
-	public function tick( $value, $args ) {
+	public function tick($value, $args) {
 		global $wpdb;
 
-		$ids = $wpdb->get_col( $wpdb->prepare( "
+		$ids = $wpdb->get_col($wpdb->prepare("
 			SELECT ID FROM wp_posts ORDER BY ID LIMIT %d,%d;
-		", $args['offset'], $args['batch_size'] ) );
+		", $args['offset'], $args['batch_size']));
 
-		foreach ( $ids as $id ) {
-			call_user_func( $this->callable, $id );
+		foreach ($ids as $id) {
+			call_user_func($this->callable, $id);
 		}
 
 	}

src/wordlift/task/background/class-background-task-route.php

Indentation +66 added lines, -66 removed lines

@@ -8,71 +8,71 @@
 
 class Background_Task_Route {
 
-	const VERSION_STRING = 'wordlift/v1';
-
-	/**
-	 * @var Background_Task
-	 */
-	private $background_task;
-
-	/**
-	 * @var string
-	 */
-	private $route_name;
-
-	public function __construct( $background_task, $route_name ) {
-		$this->background_task = $background_task;
-		$this->route_name      = $route_name;
-	}
-
-	public static function create( $task, $route_name ) {
-		$route = new self( $task, $route_name );
-
-		add_action( 'rest_api_init', array( $route, 'register' ) );
-
-		return $route;
-	}
-
-	/**
-	 * @throws Exception if the input value is invalid.
-	 */
-	public function register() {
-		Assertions::starts_with( $this->route_name, '/', 'The route name must start with a slash.' );
-
-		register_rest_route( self::VERSION_STRING, $this->route_name, array(
-			'methods'             => WP_REST_Server::CREATABLE,
-			'callback'            => array( $this->background_task, 'start' ),
-			'permission_callback' => array( $this, 'permission_callback' ),
-		) );
-
-		register_rest_route( self::VERSION_STRING, $this->route_name, array(
-			'methods'             => WP_REST_Server::READABLE,
-			'callback'            => array( $this->background_task, 'get_info' ),
-			'permission_callback' => array( $this, 'permission_callback' ),
-		) );
-
-		register_rest_route( self::VERSION_STRING, $this->route_name, array(
-			'methods'             => WP_REST_Server::DELETABLE,
-			'callback'            => array( $this->background_task, 'stop' ),
-			'permission_callback' => array( $this, 'permission_callback' ),
-		) );
-
-		register_rest_route( self::VERSION_STRING, $this->route_name, array(
-			'methods'             => 'PUT',
-			'callback'            => array( $this->background_task, 'resume' ),
-			'permission_callback' => array( $this, 'permission_callback' ),
-		) );
-
-	}
-
-	public function permission_callback() {
-		$user = wp_get_current_user();
-
-		return is_super_admin( $user->ID ) || in_array( 'administrator', (array) $user->roles );
-	}
-
-	public function get_rest_path() {
-		return self::VERSION_STRING . $this->route_name;
-	}
+    const VERSION_STRING = 'wordlift/v1';
+
+    /**
+     * @var Background_Task
+     */
+    private $background_task;
+
+    /**
+     * @var string
+     */
+    private $route_name;
+
+    public function __construct( $background_task, $route_name ) {
+        $this->background_task = $background_task;
+        $this->route_name      = $route_name;
+    }
+
+    public static function create( $task, $route_name ) {
+        $route = new self( $task, $route_name );
+
+        add_action( 'rest_api_init', array( $route, 'register' ) );
+
+        return $route;
+    }
+
+    /**
+     * @throws Exception if the input value is invalid.
+     */
+    public function register() {
+        Assertions::starts_with( $this->route_name, '/', 'The route name must start with a slash.' );
+
+        register_rest_route( self::VERSION_STRING, $this->route_name, array(
+            'methods'             => WP_REST_Server::CREATABLE,
+            'callback'            => array( $this->background_task, 'start' ),
+            'permission_callback' => array( $this, 'permission_callback' ),
+        ) );
+
+        register_rest_route( self::VERSION_STRING, $this->route_name, array(
+            'methods'             => WP_REST_Server::READABLE,
+            'callback'            => array( $this->background_task, 'get_info' ),
+            'permission_callback' => array( $this, 'permission_callback' ),
+        ) );
+
+        register_rest_route( self::VERSION_STRING, $this->route_name, array(
+            'methods'             => WP_REST_Server::DELETABLE,
+            'callback'            => array( $this->background_task, 'stop' ),
+            'permission_callback' => array( $this, 'permission_callback' ),
+        ) );
+
+        register_rest_route( self::VERSION_STRING, $this->route_name, array(
+            'methods'             => 'PUT',
+            'callback'            => array( $this->background_task, 'resume' ),
+            'permission_callback' => array( $this, 'permission_callback' ),
+        ) );
+
+    }
+
+    public function permission_callback() {
+        $user = wp_get_current_user();
+
+        return is_super_admin( $user->ID ) || in_array( 'administrator', (array) $user->roles );
+    }
+
+    public function get_rest_path() {
+        return self::VERSION_STRING . $this->route_name;
+    }
 
 }

Spacing +23 added lines, -23 removed lines

@@ -20,15 +20,15 @@ 
 	 */
 	private $route_name;
 
-	public function __construct( $background_task, $route_name ) {
+	public function __construct($background_task, $route_name) {
 		$this->background_task = $background_task;
 		$this->route_name      = $route_name;
 	}
 
-	public static function create( $task, $route_name ) {
-		$route = new self( $task, $route_name );
+	public static function create($task, $route_name) {
+		$route = new self($task, $route_name);
 
-		add_action( 'rest_api_init', array( $route, 'register' ) );
+		add_action('rest_api_init', array($route, 'register'));
 
 		return $route;
 	}
@@ -37,42 +37,42 @@ 
 	 * @throws Exception if the input value is invalid.
 	 */
 	public function register() {
-		Assertions::starts_with( $this->route_name, '/', 'The route name must start with a slash.' );
+		Assertions::starts_with($this->route_name, '/', 'The route name must start with a slash.');
 
-		register_rest_route( self::VERSION_STRING, $this->route_name, array(
+		register_rest_route(self::VERSION_STRING, $this->route_name, array(
 			'methods'             => WP_REST_Server::CREATABLE,
-			'callback'            => array( $this->background_task, 'start' ),
-			'permission_callback' => array( $this, 'permission_callback' ),
-		) );
+			'callback'            => array($this->background_task, 'start'),
+			'permission_callback' => array($this, 'permission_callback'),
+		));
 
-		register_rest_route( self::VERSION_STRING, $this->route_name, array(
+		register_rest_route(self::VERSION_STRING, $this->route_name, array(
 			'methods'             => WP_REST_Server::READABLE,
-			'callback'            => array( $this->background_task, 'get_info' ),
-			'permission_callback' => array( $this, 'permission_callback' ),
-		) );
+			'callback'            => array($this->background_task, 'get_info'),
+			'permission_callback' => array($this, 'permission_callback'),
+		));
 
-		register_rest_route( self::VERSION_STRING, $this->route_name, array(
+		register_rest_route(self::VERSION_STRING, $this->route_name, array(
 			'methods'             => WP_REST_Server::DELETABLE,
-			'callback'            => array( $this->background_task, 'stop' ),
-			'permission_callback' => array( $this, 'permission_callback' ),
-		) );
+			'callback'            => array($this->background_task, 'stop'),
+			'permission_callback' => array($this, 'permission_callback'),
+		));
 
-		register_rest_route( self::VERSION_STRING, $this->route_name, array(
+		register_rest_route(self::VERSION_STRING, $this->route_name, array(
 			'methods'             => 'PUT',
-			'callback'            => array( $this->background_task, 'resume' ),
-			'permission_callback' => array( $this, 'permission_callback' ),
-		) );
+			'callback'            => array($this->background_task, 'resume'),
+			'permission_callback' => array($this, 'permission_callback'),
+		));
 
 	}
 
 	public function permission_callback() {
 		$user = wp_get_current_user();
 
-		return is_super_admin( $user->ID ) || in_array( 'administrator', (array) $user->roles );
+		return is_super_admin($user->ID) || in_array('administrator', (array) $user->roles);
 	}
 
 	public function get_rest_path() {
-		return self::VERSION_STRING . $this->route_name;
+		return self::VERSION_STRING.$this->route_name;
 	}
 
 }

src/wordlift/task/background/class-background-task-page.php

Indentation +60 added lines, -60 removed lines

@@ -8,63 +8,63 @@ 
 
 class Background_Task_Page {
 
-	private $title;
-
-	private $menu_slug;
-
-	/**
-	 * @var Background_Task_Route $background_task_route
-	 */
-	private $background_task_route;
-
-	/**
-	 * @throws Exception if one or more parameters are invalid.
-	 */
-	public function __construct( $title, $menu_slug, $background_task_route ) {
-		Assertions::not_empty( $title, '`$title` cannot be empty.' );
-		Assertions::not_empty( $menu_slug, '`$menu_slug` cannot be empty.' );
-		Assertions::is_a( $background_task_route, 'Wordlift\Task\Background\Background_Task_Route', '`Background_Task_Route` must be of type `Wordlift\Task\Background\Background_Route`.' );
-
-		add_action( 'admin_menu', array( $this, 'admin_menu' ) );
-
-		$this->title                 = $title;
-		$this->menu_slug             = $menu_slug;
-		$this->background_task_route = $background_task_route;
-	}
-
-	/**
-	 * @throws Exception if one or more parameters are invalid.
-	 */
-	public static function create( $title, $menu_slug, $background_route ) {
-		return new self( $title, $menu_slug, $background_route );
-	}
-
-	public function admin_menu() {
-
-		add_submenu_page( 'wl_admin_menu', $this->title, $this->title, 'manage_options', $this->menu_slug, array(
-			$this,
-			'render'
-		) );
-
-	}
-
-	public function render() {
-
-		wp_enqueue_style(
-			'wl-task-page',
-			plugin_dir_url( __FILE__ ) . 'assets/task-page.css',
-			array(),
-			Wordlift::get_instance()->get_version(),
-			'all' );
-
-		wp_enqueue_script(
-			'wl-task-page',
-			plugin_dir_url( __FILE__ ) . 'assets/task-page.js',
-			array( 'wp-api' ),
-			Wordlift::get_instance()->get_version() );
-
-		wp_localize_script( 'wl-task-page', '_wlTaskPageSettings', array( 'rest_path' => $this->background_task_route->get_rest_path() ) );
-		?>
+    private $title;
+
+    private $menu_slug;
+
+    /**
+     * @var Background_Task_Route $background_task_route
+     */
+    private $background_task_route;
+
+    /**
+     * @throws Exception if one or more parameters are invalid.
+     */
+    public function __construct( $title, $menu_slug, $background_task_route ) {
+        Assertions::not_empty( $title, '`$title` cannot be empty.' );
+        Assertions::not_empty( $menu_slug, '`$menu_slug` cannot be empty.' );
+        Assertions::is_a( $background_task_route, 'Wordlift\Task\Background\Background_Task_Route', '`Background_Task_Route` must be of type `Wordlift\Task\Background\Background_Route`.' );
+
+        add_action( 'admin_menu', array( $this, 'admin_menu' ) );
+
+        $this->title                 = $title;
+        $this->menu_slug             = $menu_slug;
+        $this->background_task_route = $background_task_route;
+    }
+
+    /**
+     * @throws Exception if one or more parameters are invalid.
+     */
+    public static function create( $title, $menu_slug, $background_route ) {
+        return new self( $title, $menu_slug, $background_route );
+    }
+
+    public function admin_menu() {
+
+        add_submenu_page( 'wl_admin_menu', $this->title, $this->title, 'manage_options', $this->menu_slug, array(
+            $this,
+            'render'
+        ) );
+
+    }
+
+    public function render() {
+
+        wp_enqueue_style(
+            'wl-task-page',
+            plugin_dir_url( __FILE__ ) . 'assets/task-page.css',
+            array(),
+            Wordlift::get_instance()->get_version(),
+            'all' );
+
+        wp_enqueue_script(
+            'wl-task-page',
+            plugin_dir_url( __FILE__ ) . 'assets/task-page.js',
+            array( 'wp-api' ),
+            Wordlift::get_instance()->get_version() );
+
+        wp_localize_script( 'wl-task-page', '_wlTaskPageSettings', array( 'rest_path' => $this->background_task_route->get_rest_path() ) );
+        ?>
         <div class="wrap">
             <h2><?php echo esc_html( $this->title ); ?></h2>
 
@@ -74,12 +74,12 @@ 
             </div>
 
             <button id="wl-start-btn" type="button" class="button button-large button-primary"><?php
-				esc_html_e( 'Start', 'wordlift-framework' ); ?></button>
+                esc_html_e( 'Start', 'wordlift-framework' ); ?></button>
             <button id="wl-stop-btn" type="button" class="button button-large button-primary hidden"><?php
-				esc_html_e( 'Stop', 'wordlift-framework' ); ?></button>
+                esc_html_e( 'Stop', 'wordlift-framework' ); ?></button>
 
         </div>
 		<?php
-	}
+    }
 
 }
\ No newline at end of file

Spacing +16 added lines, -16 removed lines

@@ -20,12 +20,12 @@ 
 	/**
 	 * @throws Exception if one or more parameters are invalid.
 	 */
-	public function __construct( $title, $menu_slug, $background_task_route ) {
-		Assertions::not_empty( $title, '`$title` cannot be empty.' );
-		Assertions::not_empty( $menu_slug, '`$menu_slug` cannot be empty.' );
-		Assertions::is_a( $background_task_route, 'Wordlift\Task\Background\Background_Task_Route', '`Background_Task_Route` must be of type `Wordlift\Task\Background\Background_Route`.' );
+	public function __construct($title, $menu_slug, $background_task_route) {
+		Assertions::not_empty($title, '`$title` cannot be empty.');
+		Assertions::not_empty($menu_slug, '`$menu_slug` cannot be empty.');
+		Assertions::is_a($background_task_route, 'Wordlift\Task\Background\Background_Task_Route', '`Background_Task_Route` must be of type `Wordlift\Task\Background\Background_Route`.');
 
-		add_action( 'admin_menu', array( $this, 'admin_menu' ) );
+		add_action('admin_menu', array($this, 'admin_menu'));
 
 		$this->title                 = $title;
 		$this->menu_slug             = $menu_slug;
@@ -35,16 +35,16 @@ 
 	/**
 	 * @throws Exception if one or more parameters are invalid.
 	 */
-	public static function create( $title, $menu_slug, $background_route ) {
-		return new self( $title, $menu_slug, $background_route );
+	public static function create($title, $menu_slug, $background_route) {
+		return new self($title, $menu_slug, $background_route);
 	}
 
 	public function admin_menu() {
 
-		add_submenu_page( 'wl_admin_menu', $this->title, $this->title, 'manage_options', $this->menu_slug, array(
+		add_submenu_page('wl_admin_menu', $this->title, $this->title, 'manage_options', $this->menu_slug, array(
 			$this,
 			'render'
-		) );
+		));
 
 	}
 
@@ -52,21 +52,21 @@ 
 
 		wp_enqueue_style(
 			'wl-task-page',
-			plugin_dir_url( __FILE__ ) . 'assets/task-page.css',
+			plugin_dir_url(__FILE__).'assets/task-page.css',
 			array(),
 			Wordlift::get_instance()->get_version(),
 			'all' );
 
 		wp_enqueue_script(
 			'wl-task-page',
-			plugin_dir_url( __FILE__ ) . 'assets/task-page.js',
-			array( 'wp-api' ),
+			plugin_dir_url(__FILE__).'assets/task-page.js',
+			array('wp-api'),
 			Wordlift::get_instance()->get_version() );
 
-		wp_localize_script( 'wl-task-page', '_wlTaskPageSettings', array( 'rest_path' => $this->background_task_route->get_rest_path() ) );
+		wp_localize_script('wl-task-page', '_wlTaskPageSettings', array('rest_path' => $this->background_task_route->get_rest_path()));
 		?>
         <div class="wrap">
-            <h2><?php echo esc_html( $this->title ); ?></h2>
+            <h2><?php echo esc_html($this->title); ?></h2>
 
             <div class="wl-task__progress" style="border: 1px solid #23282D; height: 20px; margin: 8px 0;">
                 <div class="wl-task__progress__bar"
@@ -74,9 +74,9 @@ 
             </div>
 
             <button id="wl-start-btn" type="button" class="button button-large button-primary"><?php
-				esc_html_e( 'Start', 'wordlift-framework' ); ?></button>
+				esc_html_e('Start', 'wordlift-framework'); ?></button>
             <button id="wl-stop-btn" type="button" class="button button-large button-primary hidden"><?php
-				esc_html_e( 'Stop', 'wordlift-framework' ); ?></button>
+				esc_html_e('Stop', 'wordlift-framework'); ?></button>
 
         </div>
 		<?php