insideout10 / wordlift-plugin

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

Indentation +25 added lines, -25 removed lines

@@ -8,29 +8,29 @@
 // phpcs:ignore WordPress.WP.CapitalPDangit.MisspelledClassName
 abstract class Abstract_Wordpress_Content_Legacy_Service extends Abstract_Wordpress_Content_Service {
 
-	private $expected_object_type;
-	private $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 ) )
-		);
-
-		$result = call_user_func( $this->get_meta_fn, $content_id->get_id(), 'entity_url', true );
-		return $result ? $result : null;
-	}
-
-	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function delete( $content_id ) {
-		// do nothing, WP deletes the post meta for us.
-	}
+    private $expected_object_type;
+    private $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 ) )
+        );
+
+        $result = call_user_func( $this->get_meta_fn, $content_id->get_id(), 'entity_url', true );
+        return $result ? $result : null;
+    }
+
+    // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+    public function delete( $content_id ) {
+        // do nothing, WP deletes the post meta for us.
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -11,26 +11,26 @@
 	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 ) {
+	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 ) )
+			sprintf('`content_id` must be of type `%s`.', Object_Type_Enum::to_string($this->expected_object_type))
 		);
 
-		$result = call_user_func( $this->get_meta_fn, $content_id->get_id(), 'entity_url', true );
+		$result = call_user_func($this->get_meta_fn, $content_id->get_id(), 'entity_url', true);
 		return $result ? $result : null;
 	}
 
 	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	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-dataset-content-service-hooks.php

Indentation +60 added lines, -60 removed lines

@@ -8,73 +8,73 @@
 // phpcs:ignore WordPress.WP.CapitalPDangit.MisspelledClassName
 class Wordpress_Dataset_Content_Service_Hooks {
 
-	public static function register() {
-		add_action( 'wp_insert_post', array( get_called_class(), 'insert_post' ) );
-		add_action( 'after_delete_post', array( get_called_class(), 'after_delete_post' ) );
-		add_action( 'created_term', array( get_called_class(), 'created_term' ) );
-		add_action( 'delete_term', array( get_called_class(), 'delete_term' ) );
-		add_action( 'user_register', array( get_called_class(), 'user_register' ) );
-		add_action( 'deleted_user', array( get_called_class(), 'deleted_user' ) );
-	}
+    public static function register() {
+        add_action( 'wp_insert_post', array( get_called_class(), 'insert_post' ) );
+        add_action( 'after_delete_post', array( get_called_class(), 'after_delete_post' ) );
+        add_action( 'created_term', array( get_called_class(), 'created_term' ) );
+        add_action( 'delete_term', array( get_called_class(), 'delete_term' ) );
+        add_action( 'user_register', array( get_called_class(), 'user_register' ) );
+        add_action( 'deleted_user', array( get_called_class(), 'deleted_user' ) );
+    }
 
-	/**
-	 * @throws Exception in case of error.
-	 */
-	public static function insert_post( $post_id ) {
-		if ( ! wp_is_post_revision( $post_id ) ) {
-			self::set_entity_id( Wordpress_Content_Id::create_post( $post_id ) );
-		}
-	}
+    /**
+     * @throws Exception in case of error.
+     */
+    public static function insert_post( $post_id ) {
+        if ( ! wp_is_post_revision( $post_id ) ) {
+            self::set_entity_id( Wordpress_Content_Id::create_post( $post_id ) );
+        }
+    }
 
-	public static function after_delete_post( $post_id ) {
-		self::delete( Wordpress_Content_Id::create_post( $post_id ) );
-	}
+    public static function after_delete_post( $post_id ) {
+        self::delete( Wordpress_Content_Id::create_post( $post_id ) );
+    }
 
-	/**
-	 * @throws Exception in case of error.
-	 */
-	public static function created_term( $term_id ) {
-		self::set_entity_id( Wordpress_Content_Id::create_term( $term_id ) );
-	}
+    /**
+     * @throws Exception in case of error.
+     */
+    public static function created_term( $term_id ) {
+        self::set_entity_id( Wordpress_Content_Id::create_term( $term_id ) );
+    }
 
-	public static function delete_term( $term_id ) {
-		self::delete( Wordpress_Content_Id::create_term( $term_id ) );
-	}
+    public static function delete_term( $term_id ) {
+        self::delete( Wordpress_Content_Id::create_term( $term_id ) );
+    }
 
-	/**
-	 * @throws Exception in case of error.
-	 */
-	public static function user_register( $user_id ) {
-		self::set_entity_id( Wordpress_Content_Id::create_user( $user_id ) );
-	}
+    /**
+     * @throws Exception in case of error.
+     */
+    public static function user_register( $user_id ) {
+        self::set_entity_id( Wordpress_Content_Id::create_user( $user_id ) );
+    }
 
-	public static function deleted_user( $user_id ) {
-		self::delete( Wordpress_Content_Id::create_user( $user_id ) );
-	}
+    public static function deleted_user( $user_id ) {
+        self::delete( Wordpress_Content_Id::create_user( $user_id ) );
+    }
 
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 *
-	 * @return void
-	 * @throws Exception in case of error.
-	 */
-	private static function set_entity_id( $content_id ) {
-		$content_service = Wordpress_Content_Service::get_instance();
-		try {
-			$existing_id = $content_service->get_entity_id( $content_id );
-			if ( empty( $existing_id ) ) {
-				$rel_uri = Entity_Uri_Generator::create_uri( $content_id->get_type(), $content_id->get_id() );
-				$content_service->set_entity_id( $content_id, $rel_uri );
-			}
-		// phpcs:ignore Generic.CodeAnalysis.EmptyStatement.DetectedCatch
-		} catch ( Exception $e ) {
-			// Don't report.
-		}
-	}
+    /**
+     * @param Wordpress_Content_Id $content_id
+     *
+     * @return void
+     * @throws Exception in case of error.
+     */
+    private static function set_entity_id( $content_id ) {
+        $content_service = Wordpress_Content_Service::get_instance();
+        try {
+            $existing_id = $content_service->get_entity_id( $content_id );
+            if ( empty( $existing_id ) ) {
+                $rel_uri = Entity_Uri_Generator::create_uri( $content_id->get_type(), $content_id->get_id() );
+                $content_service->set_entity_id( $content_id, $rel_uri );
+            }
+        // phpcs:ignore Generic.CodeAnalysis.EmptyStatement.DetectedCatch
+        } catch ( Exception $e ) {
+            // Don't report.
+        }
+    }
 
-	private static function delete( $content_id ) {
-		$content_service = Wordpress_Content_Service::get_instance();
-		$content_service->delete( $content_id );
-	}
+    private static function delete( $content_id ) {
+        $content_service = Wordpress_Content_Service::get_instance();
+        $content_service->delete( $content_id );
+    }
 
 }

Spacing +27 added lines, -27 removed lines

@@ -9,47 +9,47 @@ 
 class Wordpress_Dataset_Content_Service_Hooks {
 
 	public static function register() {
-		add_action( 'wp_insert_post', array( get_called_class(), 'insert_post' ) );
-		add_action( 'after_delete_post', array( get_called_class(), 'after_delete_post' ) );
-		add_action( 'created_term', array( get_called_class(), 'created_term' ) );
-		add_action( 'delete_term', array( get_called_class(), 'delete_term' ) );
-		add_action( 'user_register', array( get_called_class(), 'user_register' ) );
-		add_action( 'deleted_user', array( get_called_class(), 'deleted_user' ) );
+		add_action('wp_insert_post', array(get_called_class(), 'insert_post'));
+		add_action('after_delete_post', array(get_called_class(), 'after_delete_post'));
+		add_action('created_term', array(get_called_class(), 'created_term'));
+		add_action('delete_term', array(get_called_class(), 'delete_term'));
+		add_action('user_register', array(get_called_class(), 'user_register'));
+		add_action('deleted_user', array(get_called_class(), 'deleted_user'));
 	}
 
 	/**
 	 * @throws Exception in case of error.
 	 */
-	public static function insert_post( $post_id ) {
-		if ( ! wp_is_post_revision( $post_id ) ) {
-			self::set_entity_id( Wordpress_Content_Id::create_post( $post_id ) );
+	public static function insert_post($post_id) {
+		if ( ! wp_is_post_revision($post_id)) {
+			self::set_entity_id(Wordpress_Content_Id::create_post($post_id));
 		}
 	}
 
-	public static function after_delete_post( $post_id ) {
-		self::delete( Wordpress_Content_Id::create_post( $post_id ) );
+	public static function after_delete_post($post_id) {
+		self::delete(Wordpress_Content_Id::create_post($post_id));
 	}
 
 	/**
 	 * @throws Exception in case of error.
 	 */
-	public static function created_term( $term_id ) {
-		self::set_entity_id( Wordpress_Content_Id::create_term( $term_id ) );
+	public static function created_term($term_id) {
+		self::set_entity_id(Wordpress_Content_Id::create_term($term_id));
 	}
 
-	public static function delete_term( $term_id ) {
-		self::delete( Wordpress_Content_Id::create_term( $term_id ) );
+	public static function delete_term($term_id) {
+		self::delete(Wordpress_Content_Id::create_term($term_id));
 	}
 
 	/**
 	 * @throws Exception in case of error.
 	 */
-	public static function user_register( $user_id ) {
-		self::set_entity_id( Wordpress_Content_Id::create_user( $user_id ) );
+	public static function user_register($user_id) {
+		self::set_entity_id(Wordpress_Content_Id::create_user($user_id));
 	}
 
-	public static function deleted_user( $user_id ) {
-		self::delete( Wordpress_Content_Id::create_user( $user_id ) );
+	public static function deleted_user($user_id) {
+		self::delete(Wordpress_Content_Id::create_user($user_id));
 	}
 
 	/**
@@ -58,23 +58,23 @@ 
 	 * @return void
 	 * @throws Exception in case of error.
 	 */
-	private static function set_entity_id( $content_id ) {
+	private static function set_entity_id($content_id) {
 		$content_service = Wordpress_Content_Service::get_instance();
 		try {
-			$existing_id = $content_service->get_entity_id( $content_id );
-			if ( empty( $existing_id ) ) {
-				$rel_uri = Entity_Uri_Generator::create_uri( $content_id->get_type(), $content_id->get_id() );
-				$content_service->set_entity_id( $content_id, $rel_uri );
+			$existing_id = $content_service->get_entity_id($content_id);
+			if (empty($existing_id)) {
+				$rel_uri = Entity_Uri_Generator::create_uri($content_id->get_type(), $content_id->get_id());
+				$content_service->set_entity_id($content_id, $rel_uri);
 			}
 		// phpcs:ignore Generic.CodeAnalysis.EmptyStatement.DetectedCatch
-		} catch ( Exception $e ) {
+		} catch (Exception $e) {
 			// Don't report.
 		}
 	}
 
-	private static function delete( $content_id ) {
+	private static function delete($content_id) {
 		$content_service = Wordpress_Content_Service::get_instance();
-		$content_service->delete( $content_id );
+		$content_service->delete($content_id);
 	}
 
 }

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

Indentation +107 added lines, -107 removed lines

@@ -9,111 +9,111 @@
 // phpcs:ignore WordPress.WP.CapitalPDangit.MisspelledClassName
 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;
-	}
-
-	public 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 in case of error.
-	 */
-	public 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;
-	}
-
-	public function get_entity_id( $content_id ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( ! $delegate->supports( $content_id ) ) {
-				continue;
-			}
-			$uri = $delegate->get_entity_id( $content_id );
-			if ( $uri ) {
-				return $uri;
-			}
-		}
-
-		return null;
-	}
-
-	public 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' );
-	}
-
-	public function supports( $content_id ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) ) {
-				return true;
-			}
-		}
-
-		return false;
-	}
-
-	public 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;
+    }
+
+    public 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 in case of error.
+     */
+    public 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;
+    }
+
+    public function get_entity_id( $content_id ) {
+        foreach ( $this->delegates as $delegate ) {
+            if ( ! $delegate->supports( $content_id ) ) {
+                continue;
+            }
+            $uri = $delegate->get_entity_id( $content_id );
+            if ( $uri ) {
+                return $uri;
+            }
+        }
+
+        return null;
+    }
+
+    public 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' );
+    }
+
+    public function supports( $content_id ) {
+        foreach ( $this->delegates as $delegate ) {
+            if ( $delegate->supports( $content_id ) ) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
+    public function delete( $content_id ) {
+        foreach ( $this->delegates as $delegate ) {
+            if ( $delegate->supports( $content_id ) ) {
+                $delegate->delete( $content_id );
+                break;
+            }
+        }
+    }
 }

Spacing +31 added lines, -31 removed lines

@@ -26,12 +26,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();
 		}
@@ -39,16 +39,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;
 	}
 
-	public function get_by_entity_id( $uri ) {
-		foreach ( $this->delegates as $delegate ) {
-			$content = $delegate->get_by_entity_id_or_same_as( $uri );
-			if ( isset( $content ) ) {
+	public 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;
 			}
 		}
@@ -61,10 +61,10 @@ 
 	 *
 	 * @throws Exception in case of error.
 	 */
-	public 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 ) ) {
+	public 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;
 			}
 		}
@@ -72,13 +72,13 @@ 
 		return null;
 	}
 
-	public function get_entity_id( $content_id ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( ! $delegate->supports( $content_id ) ) {
+	public function get_entity_id($content_id) {
+		foreach ($this->delegates as $delegate) {
+			if ( ! $delegate->supports($content_id)) {
 				continue;
 			}
-			$uri = $delegate->get_entity_id( $content_id );
-			if ( $uri ) {
+			$uri = $delegate->get_entity_id($content_id);
+			if ($uri) {
 				return $uri;
 			}
 		}
@@ -86,21 +86,21 @@ 
 		return null;
 	}
 
-	public function set_entity_id( $content_id, $uri ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) ) {
-				$delegate->set_entity_id( $content_id, $uri );
+	public 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');
 	}
 
-	public function supports( $content_id ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) ) {
+	public function supports($content_id) {
+		foreach ($this->delegates as $delegate) {
+			if ($delegate->supports($content_id)) {
 				return true;
 			}
 		}
@@ -108,10 +108,10 @@ 
 		return false;
 	}
 
-	public function delete( $content_id ) {
-		foreach ( $this->delegates as $delegate ) {
-			if ( $delegate->supports( $content_id ) ) {
-				$delegate->delete( $content_id );
+	public function delete($content_id) {
+		foreach ($this->delegates as $delegate) {
+			if ($delegate->supports($content_id)) {
+				$delegate->delete($content_id);
 				break;
 			}
 		}

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

Indentation +27 added lines, -27 removed lines

@@ -13,32 +13,32 @@
 // phpcs:ignore WordPress.WP.CapitalPDangit.MisspelledClassName
 class Wordpress_Post_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 Content_Service
-	 */
-	public static function get_instance() {
-
-		if ( ! isset( self::$instance ) ) {
-
-			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			if ( ! apply_filters( 'wl_feature__enable__rel-item-id', false ) ) {
-				self::$instance = Wordpress_Post_Content_Legacy_Service::get_instance();
-			} else {
-				// Migrate `entity_url` from post-meta to wl_entities.
-				$content_migration = new Content_Migration();
-				$content_migration->migrate();
-
-				// Create the post content service that uses wl_entities.
-				self::$instance = Wordpress_Post_Content_Table_Service::get_instance();
-			}
-		}
-
-		return self::$instance;
-	}
+    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 Content_Service
+     */
+    public static function get_instance() {
+
+        if ( ! isset( self::$instance ) ) {
+
+            // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+            if ( ! apply_filters( 'wl_feature__enable__rel-item-id', false ) ) {
+                self::$instance = Wordpress_Post_Content_Legacy_Service::get_instance();
+            } else {
+                // Migrate `entity_url` from post-meta to wl_entities.
+                $content_migration = new Content_Migration();
+                $content_migration->migrate();
+
+                // Create the post content service that uses wl_entities.
+                self::$instance = Wordpress_Post_Content_Table_Service::get_instance();
+            }
+        }
+
+        return self::$instance;
+    }
 
 }

Spacing +2 added lines, -2 removed lines

@@ -23,10 +23,10 @@
 	 */
 	public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 
 			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			if ( ! apply_filters( 'wl_feature__enable__rel-item-id', false ) ) {
+			if ( ! apply_filters('wl_feature__enable__rel-item-id', false)) {
 				self::$instance = Wordpress_Post_Content_Legacy_Service::get_instance();
 			} else {
 				// Migrate `entity_url` from post-meta to wl_entities.

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

Indentation +101 added lines, -101 removed lines

@@ -9,47 +9,47 @@ 
 // phpcs:ignore WordPress.WP.CapitalPDangit.MisspelledClassName
 class Wordpress_Term_Content_Legacy_Service extends Abstract_Wordpress_Content_Legacy_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_Term_Content_Legacy_Service
-	 */
-	public static function get_instance() {
-
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self( Object_Type_Enum::TERM, 'get_term_meta' );
-		}
-
-		return self::$instance;
-	}
-
-	/**
-	 * @param string $uri The entity id, relative or absolute.
-	 *
-	 * @return Wordpress_Content|null The term encapsulate within a {@link Wordpress_Content} structure or null.
-	 * @throws Exception in case of error. when the uri is not within the dataset URI scope.
-	 */
-	public 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.' );
-
-		$abs_uri = $this->make_absolute( $uri );
-
-		Assertions::starts_with(
-			$abs_uri,
-			$this->get_dataset_uri(),
-			sprintf( 'URI `%s` must be within the dataset URI scope, `%s` provided.', $abs_uri, $this->get_dataset_uri() )
-		);
-
-		global $wpdb;
-
-		$term_id = $wpdb->get_var(
-			$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_Term_Content_Legacy_Service
+     */
+    public static function get_instance() {
+
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self( Object_Type_Enum::TERM, 'get_term_meta' );
+        }
+
+        return self::$instance;
+    }
+
+    /**
+     * @param string $uri The entity id, relative or absolute.
+     *
+     * @return Wordpress_Content|null The term encapsulate within a {@link Wordpress_Content} structure or null.
+     * @throws Exception in case of error. when the uri is not within the dataset URI scope.
+     */
+    public 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.' );
+
+        $abs_uri = $this->make_absolute( $uri );
+
+        Assertions::starts_with(
+            $abs_uri,
+            $this->get_dataset_uri(),
+            sprintf( 'URI `%s` must be within the dataset URI scope, `%s` provided.', $abs_uri, $this->get_dataset_uri() )
+        );
+
+        global $wpdb;
+
+        $term_id = $wpdb->get_var(
+            $wpdb->prepare(
+                "
 			SELECT t.term_id 
 			FROM $wpdb->terms AS t 
 			INNER JOIN $wpdb->termmeta AS tm
@@ -57,70 +57,70 @@ 
 			WHERE tm.meta_key = 'entity_url' AND tm.meta_value = %s
 			LIMIT 1
 		",
-				$abs_uri
-			)
-		);
-
-		if ( isset( $term_id ) ) {
-			return new Wordpress_Content( get_term( $term_id ) );
-		}
-
-		return null;
-	}
-
-	/**
-	 * @throws Exception in case of error. when `$uri` is not a string.
-	 */
-	public 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 );
-		}
-
-		global $wpdb;
-
-		$term_id = $wpdb->get_var(
-			$wpdb->prepare(
-				"
+                $abs_uri
+            )
+        );
+
+        if ( isset( $term_id ) ) {
+            return new Wordpress_Content( get_term( $term_id ) );
+        }
+
+        return null;
+    }
+
+    /**
+     * @throws Exception in case of error. when `$uri` is not a string.
+     */
+    public 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 );
+        }
+
+        global $wpdb;
+
+        $term_id = $wpdb->get_var(
+            $wpdb->prepare(
+                "
 			SELECT tm.term_id
 			FROM $wpdb->termmeta tm
 			WHERE tm.meta_key IN ( 'entity_url', 'entity_same_as' ) AND tm.meta_value = %s
 			LIMIT 1
 		",
-				$uri
-			)
-		);
-
-		if ( isset( $term_id ) ) {
-			return new Wordpress_Content( get_term( $term_id ) );
-		}
-
-		return null;
-	}
-
-	public function set_entity_id( $content_id, $uri ) {
-		Assertions::equals( $content_id->get_type(), Object_Type_Enum::TERM, 'Content must be of `term` type.' );
-		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.' );
-		}
-
-		$abs_url = $this->make_absolute( $uri );
-
-		update_term_meta( $content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url );
-	}
-
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 *
-	 * @return bool
-	 */
-	public function supports( $content_id ) {
-		return $content_id->get_type() === Object_Type_Enum::TERM;
-	}
+                $uri
+            )
+        );
+
+        if ( isset( $term_id ) ) {
+            return new Wordpress_Content( get_term( $term_id ) );
+        }
+
+        return null;
+    }
+
+    public function set_entity_id( $content_id, $uri ) {
+        Assertions::equals( $content_id->get_type(), Object_Type_Enum::TERM, 'Content must be of `term` type.' );
+        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.' );
+        }
+
+        $abs_url = $this->make_absolute( $uri );
+
+        update_term_meta( $content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url );
+    }
+
+    /**
+     * @param Wordpress_Content_Id $content_id
+     *
+     * @return bool
+     */
+    public function supports( $content_id ) {
+        return $content_id->get_type() === Object_Type_Enum::TERM;
+    }
 
 }

Spacing +25 added lines, -25 removed lines

@@ -19,8 +19,8 @@ 
 	 */
 	public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self( Object_Type_Enum::TERM, 'get_term_meta' );
+		if ( ! isset(self::$instance)) {
+			self::$instance = new self(Object_Type_Enum::TERM, 'get_term_meta');
 		}
 
 		return self::$instance;
@@ -32,17 +32,17 @@ 
 	 * @return Wordpress_Content|null The term encapsulate within a {@link Wordpress_Content} structure or null.
 	 * @throws Exception in case of error. when the uri is not within the dataset URI scope.
 	 */
-	public 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.' );
+	public 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.');
 
-		$abs_uri = $this->make_absolute( $uri );
+		$abs_uri = $this->make_absolute($uri);
 
 		Assertions::starts_with(
 			$abs_uri,
 			$this->get_dataset_uri(),
-			sprintf( 'URI `%s` must be within the dataset URI scope, `%s` provided.', $abs_uri, $this->get_dataset_uri() )
+			sprintf('URI `%s` must be within the dataset URI scope, `%s` provided.', $abs_uri, $this->get_dataset_uri())
 		);
 
 		global $wpdb;
@@ -61,8 +61,8 @@ 
 			)
 		);
 
-		if ( isset( $term_id ) ) {
-			return new Wordpress_Content( get_term( $term_id ) );
+		if (isset($term_id)) {
+			return new Wordpress_Content(get_term($term_id));
 		}
 
 		return null;
@@ -71,13 +71,13 @@ 
 	/**
 	 * @throws Exception in case of error. when `$uri` is not a string.
 	 */
-	public 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.' );
+	public 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);
 		}
 
 		global $wpdb;
@@ -94,24 +94,24 @@ 
 			)
 		);
 
-		if ( isset( $term_id ) ) {
-			return new Wordpress_Content( get_term( $term_id ) );
+		if (isset($term_id)) {
+			return new Wordpress_Content(get_term($term_id));
 		}
 
 		return null;
 	}
 
-	public function set_entity_id( $content_id, $uri ) {
-		Assertions::equals( $content_id->get_type(), Object_Type_Enum::TERM, 'Content must be of `term` type.' );
-		Assertions::not_empty( $uri, "`uri` can't be empty" );
+	public function set_entity_id($content_id, $uri) {
+		Assertions::equals($content_id->get_type(), Object_Type_Enum::TERM, 'Content must be of `term` type.');
+		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.');
 		}
 
-		$abs_url = $this->make_absolute( $uri );
+		$abs_url = $this->make_absolute($uri);
 
-		update_term_meta( $content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url );
+		update_term_meta($content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url);
 	}
 
 	/**
@@ -119,7 +119,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public function supports( $content_id ) {
+	public function supports($content_id) {
 		return $content_id->get_type() === Object_Type_Enum::TERM;
 	}
 

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

Indentation +76 added lines, -76 removed lines

@@ -9,37 +9,37 @@ 
 // phpcs:ignore WordPress.WP.CapitalPDangit.MisspelledClassName
 class Wordpress_User_Content_Legacy_Service extends Abstract_Wordpress_Content_Legacy_Service {
 
-	private static $instance = null;
+    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_User_Content_Legacy_Service
-	 */
-	public static function get_instance() {
+    /**
+     * The singleton instance. We use this only to provide this instance to those classes where we have no access to
+     * the constructor.
+     *
+     * @return Wordpress_User_Content_Legacy_Service
+     */
+    public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self( Object_Type_Enum::USER, 'get_user_meta' );
-		}
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self( Object_Type_Enum::USER, 'get_user_meta' );
+        }
 
-		return self::$instance;
-	}
+        return self::$instance;
+    }
 
-	public 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.' );
+    public 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.' );
 
-		$abs_uri = $this->make_absolute( $uri );
+        $abs_uri = $this->make_absolute( $uri );
 
-		Assertions::starts_with( $abs_uri, $this->get_dataset_uri(), '`uri` must be within the dataset URI scope.' );
+        Assertions::starts_with( $abs_uri, $this->get_dataset_uri(), '`uri` must be within the dataset URI scope.' );
 
-		global $wpdb;
+        global $wpdb;
 
-		$user_id = $wpdb->get_var(
-			$wpdb->prepare(
-				"
+        $user_id = $wpdb->get_var(
+            $wpdb->prepare(
+                "
 			SELECT u.ID 
 			FROM $wpdb->users AS u
 			INNER JOIN $wpdb->usermeta AS um
@@ -47,34 +47,34 @@ 
 			WHERE um.meta_key = 'entity_url' AND um.meta_value = %s
 			LIMIT 1
 		",
-				$abs_uri
-			)
-		);
-
-		if ( isset( $user_id ) ) {
-			return new Wordpress_Content( get_userdata( $user_id ) );
-		}
-
-		return null;
-	}
-
-	/**
-	 * @throws Exception in case of error. when `$uri` is not a string.
-	 */
-	public 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 );
-		}
-
-		global $wpdb;
-
-		$user_id = $wpdb->get_var(
-			$wpdb->prepare(
-				"
+                $abs_uri
+            )
+        );
+
+        if ( isset( $user_id ) ) {
+            return new Wordpress_Content( get_userdata( $user_id ) );
+        }
+
+        return null;
+    }
+
+    /**
+     * @throws Exception in case of error. when `$uri` is not a string.
+     */
+    public 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 );
+        }
+
+        global $wpdb;
+
+        $user_id = $wpdb->get_var(
+            $wpdb->prepare(
+                "
 			SELECT u.ID 
 			FROM $wpdb->users AS u
 			INNER JOIN $wpdb->usermeta AS um
@@ -82,38 +82,38 @@ 
 			WHERE um.meta_key IN ( 'entity_url', 'entity_same_as' ) AND um.meta_value = %s
 			LIMIT 1
 		",
-				$uri
-			)
-		);
+                $uri
+            )
+        );
 
-		if ( isset( $user_id ) ) {
-			return new Wordpress_Content( get_userdata( $user_id ) );
-		}
+        if ( isset( $user_id ) ) {
+            return new Wordpress_Content( get_userdata( $user_id ) );
+        }
 
-		return null;
-	}
+        return null;
+    }
 
-	public function set_entity_id( $content_id, $uri ) {
-		Assertions::equals( $content_id->get_type(), Object_Type_Enum::USER, '`content_id` must be of type user.' );
-		Assertions::not_empty( $uri, "`uri` can't be empty" );
+    public function set_entity_id( $content_id, $uri ) {
+        Assertions::equals( $content_id->get_type(), Object_Type_Enum::USER, '`content_id` must be of type user.' );
+        Assertions::not_empty( $uri, "`uri` can't be empty" );
 
-		if ( $this->is_absolute( $uri ) && ! $this->is_internal( $uri ) ) {
+        if ( $this->is_absolute( $uri ) && ! $this->is_internal( $uri ) ) {
 
-			throw new Exception( '`uri` must be within the dataset URI scope.' );
-		}
+            throw new Exception( '`uri` must be within the dataset URI scope.' );
+        }
 
-		$abs_url = $this->make_absolute( $uri );
+        $abs_url = $this->make_absolute( $uri );
 
-		update_user_meta( $content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url );
-	}
+        update_user_meta( $content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url );
+    }
 
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 *
-	 * @return bool
-	 */
-	public function supports( $content_id ) {
-		return $content_id->get_type() === Object_Type_Enum::USER;
-	}
+    /**
+     * @param Wordpress_Content_Id $content_id
+     *
+     * @return bool
+     */
+    public function supports( $content_id ) {
+        return $content_id->get_type() === Object_Type_Enum::USER;
+    }
 
 }

Spacing +25 added lines, -25 removed lines

@@ -19,21 +19,21 @@ 
 	 */
 	public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self( Object_Type_Enum::USER, 'get_user_meta' );
+		if ( ! isset(self::$instance)) {
+			self::$instance = new self(Object_Type_Enum::USER, 'get_user_meta');
 		}
 
 		return self::$instance;
 	}
 
-	public 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.' );
+	public 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.');
 
-		$abs_uri = $this->make_absolute( $uri );
+		$abs_uri = $this->make_absolute($uri);
 
-		Assertions::starts_with( $abs_uri, $this->get_dataset_uri(), '`uri` must be within the dataset URI scope.' );
+		Assertions::starts_with($abs_uri, $this->get_dataset_uri(), '`uri` must be within the dataset URI scope.');
 
 		global $wpdb;
 
@@ -51,8 +51,8 @@ 
 			)
 		);
 
-		if ( isset( $user_id ) ) {
-			return new Wordpress_Content( get_userdata( $user_id ) );
+		if (isset($user_id)) {
+			return new Wordpress_Content(get_userdata($user_id));
 		}
 
 		return null;
@@ -61,13 +61,13 @@ 
 	/**
 	 * @throws Exception in case of error. when `$uri` is not a string.
 	 */
-	public 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.' );
+	public 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);
 		}
 
 		global $wpdb;
@@ -86,25 +86,25 @@ 
 			)
 		);
 
-		if ( isset( $user_id ) ) {
-			return new Wordpress_Content( get_userdata( $user_id ) );
+		if (isset($user_id)) {
+			return new Wordpress_Content(get_userdata($user_id));
 		}
 
 		return null;
 	}
 
-	public function set_entity_id( $content_id, $uri ) {
-		Assertions::equals( $content_id->get_type(), Object_Type_Enum::USER, '`content_id` must be of type user.' );
-		Assertions::not_empty( $uri, "`uri` can't be empty" );
+	public function set_entity_id($content_id, $uri) {
+		Assertions::equals($content_id->get_type(), Object_Type_Enum::USER, '`content_id` must be of type user.');
+		Assertions::not_empty($uri, "`uri` can't be empty");
 
-		if ( $this->is_absolute( $uri ) && ! $this->is_internal( $uri ) ) {
+		if ($this->is_absolute($uri) && ! $this->is_internal($uri)) {
 
-			throw new Exception( '`uri` must be within the dataset URI scope.' );
+			throw new Exception('`uri` must be within the dataset URI scope.');
 		}
 
-		$abs_url = $this->make_absolute( $uri );
+		$abs_url = $this->make_absolute($uri);
 
-		update_user_meta( $content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url );
+		update_user_meta($content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url);
 	}
 
 	/**
@@ -112,7 +112,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public function supports( $content_id ) {
+	public function supports($content_id) {
 		return $content_id->get_type() === Object_Type_Enum::USER;
 	}
 

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

Indentation +127 added lines, -127 removed lines

@@ -11,132 +11,132 @@
 // phpcs:ignore WordPress.WP.CapitalPDangit.MisspelledClassName
 class Wordpress_Post_Content_Legacy_Service extends Abstract_Wordpress_Content_Legacy_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_Legacy_Service
-	 */
-	public static function get_instance() {
-
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self( Object_Type_Enum::POST, 'get_post_meta' );
-		}
-
-		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 in case of error. when the URI is not within the dataset URI.
-	 */
-	public 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.' );
-
-		$abs_uri = $this->make_absolute( $uri );
-
-		Assertions::starts_with( $abs_uri, $this->get_dataset_uri(), '`uri` must start with dataset 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'     => WL_ENTITY_URL_META_NAME,
-					'value'   => $abs_uri,
-					'compare' => '=',
-				),
-			),
-		);
-
-		$posts = get_posts( $query_args );
-
-		// Get the current post or allow 3rd parties to provide a replacement.
-		$post = current( $posts );
-		$post = $post ? $post : apply_filters( 'wl_content_service__post__not_found', null, $uri );
-
-		if ( is_a( $post, 'WP_Post' ) ) {
-			return new Wordpress_Content( current( $posts ) );
-		}
-
-		return null;
-	}
-
-	/**
-	 * @throws Exception in case of error. when `$uri` is not a string.
-	 */
-	public function get_by_entity_id_or_same_as( $uri ) {
-		// ATM we're too strict with the assertions here so we return null if the uri isn't provided.
-		// Assertions::is_string( $uri, '`uri` must be a string.' );
-		// Assertions::not_empty( '`uri` cannot be empty.' );
-		if ( ! is_string( $uri ) || empty( $uri ) ) {
-			return null;
-		}
-
-		// 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 );
-		$post = $post ? $post : apply_filters( 'wl_content_service__post__not_found', null, $uri );
-
-		if ( is_a( $post, '\WP_Post' ) ) {
-			return new Wordpress_Content( current( $posts ) );
-		}
-
-		return null;
-	}
-
-	public 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.' );
-		}
-
-		$abs_url = $this->make_absolute( $uri );
-
-		update_post_meta( $content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url );
-	}
-
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 *
-	 * @return bool
-	 */
-	public function supports( $content_id ) {
-		return $content_id->get_type() === Object_Type_Enum::POST;
-	}
+    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_Legacy_Service
+     */
+    public static function get_instance() {
+
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self( Object_Type_Enum::POST, 'get_post_meta' );
+        }
+
+        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 in case of error. when the URI is not within the dataset URI.
+     */
+    public 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.' );
+
+        $abs_uri = $this->make_absolute( $uri );
+
+        Assertions::starts_with( $abs_uri, $this->get_dataset_uri(), '`uri` must start with dataset 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'     => WL_ENTITY_URL_META_NAME,
+                    'value'   => $abs_uri,
+                    'compare' => '=',
+                ),
+            ),
+        );
+
+        $posts = get_posts( $query_args );
+
+        // Get the current post or allow 3rd parties to provide a replacement.
+        $post = current( $posts );
+        $post = $post ? $post : apply_filters( 'wl_content_service__post__not_found', null, $uri );
+
+        if ( is_a( $post, 'WP_Post' ) ) {
+            return new Wordpress_Content( current( $posts ) );
+        }
+
+        return null;
+    }
+
+    /**
+     * @throws Exception in case of error. when `$uri` is not a string.
+     */
+    public function get_by_entity_id_or_same_as( $uri ) {
+        // ATM we're too strict with the assertions here so we return null if the uri isn't provided.
+        // Assertions::is_string( $uri, '`uri` must be a string.' );
+        // Assertions::not_empty( '`uri` cannot be empty.' );
+        if ( ! is_string( $uri ) || empty( $uri ) ) {
+            return null;
+        }
+
+        // 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 );
+        $post = $post ? $post : apply_filters( 'wl_content_service__post__not_found', null, $uri );
+
+        if ( is_a( $post, '\WP_Post' ) ) {
+            return new Wordpress_Content( current( $posts ) );
+        }
+
+        return null;
+    }
+
+    public 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.' );
+        }
+
+        $abs_url = $this->make_absolute( $uri );
+
+        update_post_meta( $content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url );
+    }
+
+    /**
+     * @param Wordpress_Content_Id $content_id
+     *
+     * @return bool
+     */
+    public function supports( $content_id ) {
+        return $content_id->get_type() === Object_Type_Enum::POST;
+    }
 
 }

Spacing +30 added lines, -30 removed lines

@@ -21,8 +21,8 @@ 
 	 */
 	public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self( Object_Type_Enum::POST, 'get_post_meta' );
+		if ( ! isset(self::$instance)) {
+			self::$instance = new self(Object_Type_Enum::POST, 'get_post_meta');
 		}
 
 		return self::$instance;
@@ -34,14 +34,14 @@ 
 	 * @return Wordpress_Content|null
 	 * @throws Exception in case of error. when the URI is not within the dataset URI.
 	 */
-	public 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.' );
+	public 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.');
 
-		$abs_uri = $this->make_absolute( $uri );
+		$abs_uri = $this->make_absolute($uri);
 
-		Assertions::starts_with( $abs_uri, $this->get_dataset_uri(), '`uri` must start with dataset URI.' );
+		Assertions::starts_with($abs_uri, $this->get_dataset_uri(), '`uri` must start with dataset URI.');
 
 		// Look in sameAs.
 		$query_args = array(
@@ -59,14 +59,14 @@ 
 			),
 		);
 
-		$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 );
-		$post = $post ? $post : apply_filters( 'wl_content_service__post__not_found', null, $uri );
+		$post = current($posts);
+		$post = $post ? $post : 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;
@@ -75,17 +75,17 @@ 
 	/**
 	 * @throws Exception in case of error. when `$uri` is not a string.
 	 */
-	public function get_by_entity_id_or_same_as( $uri ) {
+	public function get_by_entity_id_or_same_as($uri) {
 		// ATM we're too strict with the assertions here so we return null if the uri isn't provided.
 		// Assertions::is_string( $uri, '`uri` must be a string.' );
 		// Assertions::not_empty( '`uri` cannot be empty.' );
-		if ( ! is_string( $uri ) || empty( $uri ) ) {
+		if ( ! is_string($uri) || empty($uri)) {
 			return null;
 		}
 
 		// 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.
@@ -104,30 +104,30 @@ 
 			),
 		);
 
-		$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 );
-		$post = $post ? $post : apply_filters( 'wl_content_service__post__not_found', null, $uri );
+		$post = current($posts);
+		$post = $post ? $post : 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;
 	}
 
-	public 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" );
+	public 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.');
 		}
 
-		$abs_url = $this->make_absolute( $uri );
+		$abs_url = $this->make_absolute($uri);
 
-		update_post_meta( $content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url );
+		update_post_meta($content_id->get_id(), WL_ENTITY_URL_META_NAME, $abs_url);
 	}
 
 	/**
@@ -135,7 +135,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public function supports( $content_id ) {
+	public function supports($content_id) {
 		return $content_id->get_type() === Object_Type_Enum::POST;
 	}
 

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

Indentation +165 added lines, -165 removed lines

@@ -16,184 +16,184 @@
 // phpcs:ignore WordPress.WP.CapitalPDangit.MisspelledClassName
 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 in case of error. when the URI is not within the dataset URI.
-	 */
-	public 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 in case of error. when the URI is not within the dataset URI.
+     */
+    public 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 in case of error. when `$uri` is not a string.
-	 */
-	public 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 );
-		$post = $post ? $post : 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 in case of error.
-	 */
-	public 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
+            )
+        );
+
+        if ( ! isset( $row ) || Object_Type_Enum::POST !== (int) $row->content_type ) {
+            return null;
+        }
+
+        return new Wordpress_Content( get_post( $row->content_id ) );
+    }
+
+    /**
+     * @throws Exception in case of error. when `$uri` is not a string.
+     */
+    public 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 );
+        $post = $post ? $post : 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 in case of error.
+     */
+    public 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;
-	}
-
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 * @param string               $uri
-	 *
-	 * @return void
-	 * @throws Exception in case of error.
-	 */
-	public 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.' );
-		}
-
-		$rel_url = $this->make_relative( $uri );
-
-		global $wpdb;
-		$wpdb->query(
-			$wpdb->prepare(
-				"
+                $content_id->get_id(),
+                $content_id->get_type()
+            )
+        );
+
+        return $rel_uri ? $this->make_absolute( $rel_uri ) : null;
+    }
+
+    /**
+     * @param Wordpress_Content_Id $content_id
+     * @param string               $uri
+     *
+     * @return void
+     * @throws Exception in case of error.
+     */
+    public 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.' );
+        }
+
+        $rel_url = $this->make_relative( $uri );
+
+        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
-	 */
-	public function supports( $content_id ) {
-		return $content_id->get_type() === Object_Type_Enum::POST;
-	}
-
-	/**
-	 * @param Wordpress_Content_Id $content_id
-	 *
-	 * @return void
-	 */
-	public function delete( $content_id ) {
-		global $wpdb;
-
-		$wpdb->query(
-			$wpdb->prepare(
-				"
+                $content_id->get_id(),
+                $content_id->get_type(),
+                $rel_url,
+                $rel_url
+            )
+        );
+    }
+
+    /**
+     * @param Wordpress_Content_Id $content_id
+     *
+     * @return bool
+     */
+    public function supports( $content_id ) {
+        return $content_id->get_type() === Object_Type_Enum::POST;
+    }
+
+    /**
+     * @param Wordpress_Content_Id $content_id
+     *
+     * @return void
+     */
+    public 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()
-			)
-		);
-	}
+                $content_id->get_id(),
+                $content_id->get_type()
+            )
+        );
+    }
 }

Spacing +31 added lines, -31 removed lines

@@ -26,7 +26,7 @@ 
 	 */
 	public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -39,16 +39,16 @@ 
 	 * @return Wordpress_Content|null
 	 * @throws Exception in case of error. when the URI is not within the dataset URI.
 	 */
-	public 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.' );
+	public 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(
@@ -62,23 +62,23 @@ 
 			)
 		);
 
-		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 in case of error. when `$uri` is not a string.
 	 */
-	public 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.' );
+	public 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.
@@ -97,14 +97,14 @@ 
 			),
 		);
 
-		$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 );
-		$post = $post ? $post : apply_filters( 'wl_content_service__post__not_found', null, $uri );
+		$post = current($posts);
+		$post = $post ? $post : 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;
@@ -116,8 +116,8 @@ 
 	 * @return string|null The entity ID.
 	 * @throws Exception in case of error.
 	 */
-	public function get_entity_id( $content_id ) {
-		Assertions::equals( $content_id->get_type(), Object_Type_Enum::POST, '`content_id` must be of type post.' );
+	public 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(
@@ -132,7 +132,7 @@ 
 			)
 		);
 
-		return $rel_uri ? $this->make_absolute( $rel_uri ) : null;
+		return $rel_uri ? $this->make_absolute($rel_uri) : null;
 	}
 
 	/**
@@ -142,15 +142,15 @@ 
 	 * @return void
 	 * @throws Exception in case of error.
 	 */
-	public 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" );
+	public 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(
@@ -173,7 +173,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public function supports( $content_id ) {
+	public function supports($content_id) {
 		return $content_id->get_type() === Object_Type_Enum::POST;
 	}
 
@@ -182,7 +182,7 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete( $content_id ) {
+	public function delete($content_id) {
 		global $wpdb;
 
 		$wpdb->query(

src/wordlift/content/class-content.php

Indentation +34 added lines, -34 removed lines

@@ -4,39 +4,39 @@
 
 interface Content {
 
-	/**
-	 * Get the actual content.
-	 *
-	 * @return mixed Get the actual content.
-	 */
-	public function get_bag();
-
-	/**
-	 * Get the content id.
-	 *
-	 * @return mixed
-	 */
-	public function get_id();
-
-	/**
-	 * Get the content type.
-	 *
-	 * @return mixed
-	 */
-	public function get_object_type_enum();
-
-	/**
-	 * Get the permalink.
-	 *
-	 * @return string
-	 */
-	public function get_permalink();
-
-	/**
-	 * Get the edit link.
-	 *
-	 * @return string
-	 */
-	public function get_edit_link();
+    /**
+     * Get the actual content.
+     *
+     * @return mixed Get the actual content.
+     */
+    public function get_bag();
+
+    /**
+     * Get the content id.
+     *
+     * @return mixed
+     */
+    public function get_id();
+
+    /**
+     * Get the content type.
+     *
+     * @return mixed
+     */
+    public function get_object_type_enum();
+
+    /**
+     * Get the permalink.
+     *
+     * @return string
+     */
+    public function get_permalink();
+
+    /**
+     * Get the edit link.
+     *
+     * @return string
+     */
+    public function get_edit_link();
 
 }