insideout10 / wordlift-plugin

src/includes/class-wordlift-property-service.php

Indentation +113 added lines, -113 removed lines

@@ -14,118 +14,118 @@
  */
 abstract class Wordlift_Property_Service {
 
-	protected static $instance;
-
-	public function __construct() {
-
-		static::$instance = $this;
-	}
-
-	/**
-	 * Get the field singleton.
-	 *
-	 * @since 3.6.0
-	 * @return \Wordlift_Schema_Url_Property_Service The singleton instance.
-	 */
-	public static function get_instance() {
-
-		return static::$instance;
-	}
-
-	/**
-	 * Get the value for the specified post/entity.
-	 *
-	 * @since 3.6.0
-	 *
-	 * @param int $post_id The post id.
-	 *
-	 * @return mixed
-	 */
-	abstract public function get( $post_id );
-
-	/**
-	 * Sanitize the provided value.
-	 *
-	 * @since 3.6.0
-	 *
-	 * @param mixed $value The value to sanitize.
-	 *
-	 * @return mixed|NULL The sanitized value or NULL avoid saving this value (see {@link Wl_Metabox_Field}).
-	 */
-	abstract public function sanitize( $value );
-
-	/**
-	 * The RDF predicate for the property.
-	 *
-	 * @since 3.6.0
-	 * @return string The RDF predicate.
-	 */
-	abstract public function get_rdf_predicate();
-
-	/**
-	 * The RDF data type.
-	 *
-	 * @since 3.6.0
-	 * @return string The RDF data type.
-	 */
-	abstract public function get_rdf_data_type();
-
-	/**
-	 * The internal data type.
-	 *
-	 * @since 3.6.0
-	 * @return string The internal data type.
-	 */
-	abstract public function get_data_type();
-
-	/**
-	 * The cardinality.
-	 *
-	 * @since 3.6.0
-	 * @return mixed The cardinality.
-	 */
-	abstract public function get_cardinality();
-
-	/**
-	 * The metabox field class name.
-	 *
-	 * @since 3.6.0
-	 * @return string The metabox field class name.
-	 */
-	abstract public function get_metabox_class();
-
-	/**
-	 * The untranslated metabox field label.
-	 *
-	 * @since 3.6.0
-	 * @return string The untranslated metabox field label.
-	 */
-	abstract public function get_metabox_label();
-
-	/**
-	 * The definition of the property returned as a compatible array.
-	 *
-	 * @deprecated
-	 *
-	 * @since 3.6.0
-	 * @return array An array of property definitions.
-	 */
-	public function get_compat_definition() {
-
-		return array(
-			'type'        => $this->get_data_type(),
-			'predicate'   => $this->get_rdf_predicate(),
-			'export_type' => $this->get_rdf_data_type(),
-			'constraints' => array(
-				'cardinality' => $this->get_cardinality(),
-			),
-			// Use the standard metabox for these URI (the URI metabox creates local entities).
-			'metabox'     => array(
-				'class' => $this->get_metabox_class(),
-				'label' => $this->get_metabox_label(),
-			),
-			'sanitize'    => array( $this, 'sanitize' ),
-		);
-	}
+    protected static $instance;
+
+    public function __construct() {
+
+        static::$instance = $this;
+    }
+
+    /**
+     * Get the field singleton.
+     *
+     * @since 3.6.0
+     * @return \Wordlift_Schema_Url_Property_Service The singleton instance.
+     */
+    public static function get_instance() {
+
+        return static::$instance;
+    }
+
+    /**
+     * Get the value for the specified post/entity.
+     *
+     * @since 3.6.0
+     *
+     * @param int $post_id The post id.
+     *
+     * @return mixed
+     */
+    abstract public function get( $post_id );
+
+    /**
+     * Sanitize the provided value.
+     *
+     * @since 3.6.0
+     *
+     * @param mixed $value The value to sanitize.
+     *
+     * @return mixed|NULL The sanitized value or NULL avoid saving this value (see {@link Wl_Metabox_Field}).
+     */
+    abstract public function sanitize( $value );
+
+    /**
+     * The RDF predicate for the property.
+     *
+     * @since 3.6.0
+     * @return string The RDF predicate.
+     */
+    abstract public function get_rdf_predicate();
+
+    /**
+     * The RDF data type.
+     *
+     * @since 3.6.0
+     * @return string The RDF data type.
+     */
+    abstract public function get_rdf_data_type();
+
+    /**
+     * The internal data type.
+     *
+     * @since 3.6.0
+     * @return string The internal data type.
+     */
+    abstract public function get_data_type();
+
+    /**
+     * The cardinality.
+     *
+     * @since 3.6.0
+     * @return mixed The cardinality.
+     */
+    abstract public function get_cardinality();
+
+    /**
+     * The metabox field class name.
+     *
+     * @since 3.6.0
+     * @return string The metabox field class name.
+     */
+    abstract public function get_metabox_class();
+
+    /**
+     * The untranslated metabox field label.
+     *
+     * @since 3.6.0
+     * @return string The untranslated metabox field label.
+     */
+    abstract public function get_metabox_label();
+
+    /**
+     * The definition of the property returned as a compatible array.
+     *
+     * @deprecated
+     *
+     * @since 3.6.0
+     * @return array An array of property definitions.
+     */
+    public function get_compat_definition() {
+
+        return array(
+            'type'        => $this->get_data_type(),
+            'predicate'   => $this->get_rdf_predicate(),
+            'export_type' => $this->get_rdf_data_type(),
+            'constraints' => array(
+                'cardinality' => $this->get_cardinality(),
+            ),
+            // Use the standard metabox for these URI (the URI metabox creates local entities).
+            'metabox'     => array(
+                'class' => $this->get_metabox_class(),
+                'label' => $this->get_metabox_label(),
+            ),
+            'sanitize'    => array( $this, 'sanitize' ),
+        );
+    }
 
 }

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
 	 *
 	 * @return mixed
 	 */
-	abstract public function get( $post_id );
+	abstract public function get($post_id);
 
 	/**
 	 * Sanitize the provided value.
@@ -52,7 +52,7 @@ 
 	 *
 	 * @return mixed|NULL The sanitized value or NULL avoid saving this value (see {@link Wl_Metabox_Field}).
 	 */
-	abstract public function sanitize( $value );
+	abstract public function sanitize($value);
 
 	/**
 	 * The RDF predicate for the property.
@@ -124,7 +124,7 @@ 
 				'class' => $this->get_metabox_class(),
 				'label' => $this->get_metabox_label(),
 			),
-			'sanitize'    => array( $this, 'sanitize' ),
+			'sanitize'    => array($this, 'sanitize'),
 		);
 	}
 

src/includes/class-wordlift-user-service.php

Indentation +323 added lines, -323 removed lines

@@ -19,336 +19,336 @@
  */
 class Wordlift_User_Service {
 
-	/**
-	 * The meta key where the user's URI is stored.
-	 *
-	 * @since 3.1.7
-	 */
-	const URI_META_KEY = '_wl_uri';
-
-	/**
-	 * The user meta key where the deny entity edit flag is stored.
-	 *
-	 * @since 3.14.0
-	 */
-	const DENY_ENTITY_CREATE_META_KEY = '_wl_deny_entity_create';
-
-	/**
-	 * The meta key holding the entity id representing a {@link WP_User}.
-	 *
-	 * @since 3.14.0
-	 */
-	const ENTITY_META_KEY = '_wl_entity';
-
-	/**
-	 * The Log service.
-	 *
-	 * @since  3.1.7
-	 * @access private
-	 * @var \Wordlift_Log_Service $log_service The Log service.
-	 */
-	private $log_service;
-
-	/**
-	 * Create an instance of the User service.
-	 *
-	 * @since 3.1.7
-	 */
-	protected function __construct() {
-
-		$this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_User_Service' );
-
-		add_filter( 'user_has_cap', array( $this, 'has_cap' ), 10, 3 );
-
-	}
-
-	/**
-	 * The singleton instance of the User service.
-	 *
-	 * @since  3.1.7
-	 * @access private
-	 * @var \Wordlift_User_Service $user_service The singleton instance of the User service.
-	 */
-	private static $instance = null;
-
-	/**
-	 * Get the singleton instance of the User service.
-	 *
-	 * @return \Wordlift_User_Service The singleton instance of the User service.
-	 * @since 3.1.7
-	 */
-	public static function get_instance() {
-
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self();
-		}
-
-		return self::$instance;
-	}
-
-	/**
-	 * Get the URI for a user.
-	 *
-	 * @param int $user_id The user id
-	 *
-	 * @return false|string The user's URI or false in case of failure.
-	 * @since 3.1.7
-	 */
-	public function get_uri( $user_id ) {
-
-		// Try to get the URI stored in the user's meta and return it if available.
-		$dataset_uri = wl_configuration_get_redlink_dataset_uri();
-		$user_uri    = $this->_get_uri( $user_id );
-		if ( ! empty( $dataset_uri ) && ! empty( $user_uri ) && 0 === strpos( $user_uri, $dataset_uri ) ) {
-			return $user_uri;
-		}
-
-		// Try to build an URI, return false in case of failure.
-		$user_uri = $this->_build_uri( $user_id );
-		if ( false === $user_uri ) {
-			return false;
-		}
-
-		// Store the URI for future requests (we need a "permanent" URI).
-		$this->_set_uri( $user_id, $user_uri );
-
-		return $user_uri;
-	}
-
-	/**
-	 * Set the `id` of the entity representing a {@link WP_User}.
-	 *
-	 * If the `id` is set to 0 (or less) then the meta is deleted.
-	 *
-	 * @param int $user_id The {@link WP_User}.
-	 * @param int $value The entity {@link WP_Post} `id`.
-	 *
-	 * @return bool|int  Meta ID if the key didn't exist, true on successful update, false on failure.
-	 * @since 3.14.0
-	 */
-	public function set_entity( $user_id, $value ) {
-
-		return 0 < $value
-			? update_user_meta( $user_id, self::ENTITY_META_KEY, $value )
-			: delete_user_meta( $user_id, self::ENTITY_META_KEY );
-	}
-
-	/**
-	 * Get the {@link WP_Post} `id` of the entity representing a {@link WP_User}.
-	 *
-	 * @param int $user_id The {@link WP_User}'s `id`.
-	 *
-	 * @return string|false The entity {@link WP_Post} `id` or an empty string if not set or false if the object id is invalid
-	 * @since 3.14.0
-	 */
-	public function get_entity( $user_id ) {
-
-		return get_user_meta( $user_id, self::ENTITY_META_KEY, true );
-	}
-
-	/**
-	 * Get the user's URI stored in the user's meta.
-	 *
-	 * @param int $user_id The user id.
-	 *
-	 * @return false|string The user's URI or false if not found.
-	 * @since 3.1.7
-	 */
-	// phpcs:ignore PSR2.Methods.MethodDeclaration.Underscore
-	private function _get_uri( $user_id ) {
-
-		$user_uri = get_user_meta( $user_id, self::URI_META_KEY, true );
-
-		if ( empty( $user_uri ) ) {
-			return false;
-		}
-
-		return $user_uri;
-	}
-
-	/**
-	 * Build an URI for a user.
-	 *
-	 * @param int $user_id The user's id.
-	 *
-	 * @return false|string The user's URI or false in case of failure.
-	 * @since 3.1.7
-	 */
-	// phpcs:ignore PSR2.Methods.MethodDeclaration.Underscore
-	private function _build_uri( $user_id ) {
-
-		// Get the user, return false in case of failure.
-		$user = get_userdata( $user_id );
-		if ( false === $user ) {
-			return false;
-		};
-
-		// If the nicename is not set, return a failure.
-		if ( empty( $user->user_nicename ) ) {
-			return false;
-		}
-
-		/**
-		 * @since 3.27.7 changed `user` to `author` to avoid potential clashes with CPTs ( `author` is reserved
-		 *  https://developer.wordpress.org/reference/functions/register_post_type/#reserved-post-types )
-		 */
-		return untrailingslashit( wl_configuration_get_redlink_dataset_uri() ) . "/author/$user->user_nicename";
-	}
-
-	/**
-	 * Store the URI in user's meta.
-	 *
-	 * @param int    $user_id The user's id.
-	 * @param string $user_uri The user's uri.
-	 *
-	 * @return int|bool Meta ID if the key didn't exist, true on successful update, false on failure.
-	 * @since 3.1.7
-	 */
-	// phpcs:ignore PSR2.Methods.MethodDeclaration.Underscore
-	private function _set_uri( $user_id, $user_uri ) {
-
-		return update_user_meta( $user_id, self::URI_META_KEY, $user_uri );
-	}
-
-	/**
-	 * Mark an editor user as denied from editing entities.
-	 * Does nothing if the user is not an editor
-	 *
-	 * @param integer $user_id The ID of the user
-	 *
-	 * @since 3.14.0
-	 */
-	public function deny_editor_entity_create( $user_id ) {
-
-		// Bail out if the user is not an editor.
-		if ( ! $this->is_editor( $user_id ) ) {
-			return;
-		}
-
-		// The user explicitly do not have the capability.
-		update_user_option( $user_id, self::DENY_ENTITY_CREATE_META_KEY, 'yes' );
-
-	}
-
-	/**
-	 * Remove the "deny entity editing" mark from an editor user.
-	 * Does nothing if the user is not an editor
-	 *
-	 * @param integer $user_id The ID of the user
-	 *
-	 * @since 3.14.0
-	 */
-	public function allow_editor_entity_create( $user_id ) {
-
-		// Bail out if the user is not an editor.
-		if ( ! $this->is_editor( $user_id ) ) {
-			return;
-		}
-
-		// The user explicitly do not have the capability.
-		delete_user_option( $user_id, self::DENY_ENTITY_CREATE_META_KEY );
-
-	}
-
-	/**
-	 * Get whether the 'deny editor entity editing' flag is set.
-	 *
-	 * @param int $user_id The {@link WP_User} `id`.
-	 *
-	 * @return int bool True if editing is denied otherwise false.
-	 * @since 3.14.0
-	 */
-	public function is_deny_editor_entity_create( $user_id ) {
-
-		return 'yes' === get_user_option( self::DENY_ENTITY_CREATE_META_KEY, $user_id );
-	}
-
-	/**
-	 * Check whether the {@link WP_User} with the specified `id` is an editor,
-	 * i.e. has the `editor` role.
-	 *
-	 * @param int $user_id The {@link WP_User} `id`.
-	 *
-	 * @return bool True if the {@link WP_User} is an editor otherwise false.
-	 * @since 3.14.0
-	 */
-	public function is_editor( $user_id ) {
-
-		// Get the user.
-		$user = get_user_by( 'id', $user_id );
-
-		// Return true, if the user is found and has the `editor` role.
-		return is_a( $user, 'WP_User' ) && in_array( 'editor', (array) $user->roles, true );
-	}
-
-	/**
-	 * Check if an editor can create entities.
-	 *
-	 * @param int $user_id The user id of the user being checked.
-	 *
-	 * @return bool    false if it is an editor that is denied from edit entities, true otherwise.
-	 * @since 3.14.0
-	 */
-	public function editor_can_create_entities( $user_id ) {
-
-		// Return true if not an editor.
-		if ( ! $this->is_editor( $user_id ) ) {
-			return true;
-		}
-
-		// Check if the user explicitly denied.
-		return ! $this->is_deny_editor_entity_create( $user_id );
-	}
-
-	/**
-	 * Filter capabilities of user.
-	 *
-	 * Deny the capability of managing and editing entities for some users.
-	 *
-	 * @param array $allcaps All the capabilities of the user
-	 * @param array $cap [0] Required capability
-	 * @param array $args [0] Requested capability
-	 *                       [1] User ID
-	 *                       [2] Associated object ID
-	 *
-	 * @return array The capabilities array.
-	 * @since 3.14.0
-	 */
-	public function has_cap( $allcaps, $cap, $args ) {
-		/*
+    /**
+     * The meta key where the user's URI is stored.
+     *
+     * @since 3.1.7
+     */
+    const URI_META_KEY = '_wl_uri';
+
+    /**
+     * The user meta key where the deny entity edit flag is stored.
+     *
+     * @since 3.14.0
+     */
+    const DENY_ENTITY_CREATE_META_KEY = '_wl_deny_entity_create';
+
+    /**
+     * The meta key holding the entity id representing a {@link WP_User}.
+     *
+     * @since 3.14.0
+     */
+    const ENTITY_META_KEY = '_wl_entity';
+
+    /**
+     * The Log service.
+     *
+     * @since  3.1.7
+     * @access private
+     * @var \Wordlift_Log_Service $log_service The Log service.
+     */
+    private $log_service;
+
+    /**
+     * Create an instance of the User service.
+     *
+     * @since 3.1.7
+     */
+    protected function __construct() {
+
+        $this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_User_Service' );
+
+        add_filter( 'user_has_cap', array( $this, 'has_cap' ), 10, 3 );
+
+    }
+
+    /**
+     * The singleton instance of the User service.
+     *
+     * @since  3.1.7
+     * @access private
+     * @var \Wordlift_User_Service $user_service The singleton instance of the User service.
+     */
+    private static $instance = null;
+
+    /**
+     * Get the singleton instance of the User service.
+     *
+     * @return \Wordlift_User_Service The singleton instance of the User service.
+     * @since 3.1.7
+     */
+    public static function get_instance() {
+
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self();
+        }
+
+        return self::$instance;
+    }
+
+    /**
+     * Get the URI for a user.
+     *
+     * @param int $user_id The user id
+     *
+     * @return false|string The user's URI or false in case of failure.
+     * @since 3.1.7
+     */
+    public function get_uri( $user_id ) {
+
+        // Try to get the URI stored in the user's meta and return it if available.
+        $dataset_uri = wl_configuration_get_redlink_dataset_uri();
+        $user_uri    = $this->_get_uri( $user_id );
+        if ( ! empty( $dataset_uri ) && ! empty( $user_uri ) && 0 === strpos( $user_uri, $dataset_uri ) ) {
+            return $user_uri;
+        }
+
+        // Try to build an URI, return false in case of failure.
+        $user_uri = $this->_build_uri( $user_id );
+        if ( false === $user_uri ) {
+            return false;
+        }
+
+        // Store the URI for future requests (we need a "permanent" URI).
+        $this->_set_uri( $user_id, $user_uri );
+
+        return $user_uri;
+    }
+
+    /**
+     * Set the `id` of the entity representing a {@link WP_User}.
+     *
+     * If the `id` is set to 0 (or less) then the meta is deleted.
+     *
+     * @param int $user_id The {@link WP_User}.
+     * @param int $value The entity {@link WP_Post} `id`.
+     *
+     * @return bool|int  Meta ID if the key didn't exist, true on successful update, false on failure.
+     * @since 3.14.0
+     */
+    public function set_entity( $user_id, $value ) {
+
+        return 0 < $value
+            ? update_user_meta( $user_id, self::ENTITY_META_KEY, $value )
+            : delete_user_meta( $user_id, self::ENTITY_META_KEY );
+    }
+
+    /**
+     * Get the {@link WP_Post} `id` of the entity representing a {@link WP_User}.
+     *
+     * @param int $user_id The {@link WP_User}'s `id`.
+     *
+     * @return string|false The entity {@link WP_Post} `id` or an empty string if not set or false if the object id is invalid
+     * @since 3.14.0
+     */
+    public function get_entity( $user_id ) {
+
+        return get_user_meta( $user_id, self::ENTITY_META_KEY, true );
+    }
+
+    /**
+     * Get the user's URI stored in the user's meta.
+     *
+     * @param int $user_id The user id.
+     *
+     * @return false|string The user's URI or false if not found.
+     * @since 3.1.7
+     */
+    // phpcs:ignore PSR2.Methods.MethodDeclaration.Underscore
+    private function _get_uri( $user_id ) {
+
+        $user_uri = get_user_meta( $user_id, self::URI_META_KEY, true );
+
+        if ( empty( $user_uri ) ) {
+            return false;
+        }
+
+        return $user_uri;
+    }
+
+    /**
+     * Build an URI for a user.
+     *
+     * @param int $user_id The user's id.
+     *
+     * @return false|string The user's URI or false in case of failure.
+     * @since 3.1.7
+     */
+    // phpcs:ignore PSR2.Methods.MethodDeclaration.Underscore
+    private function _build_uri( $user_id ) {
+
+        // Get the user, return false in case of failure.
+        $user = get_userdata( $user_id );
+        if ( false === $user ) {
+            return false;
+        };
+
+        // If the nicename is not set, return a failure.
+        if ( empty( $user->user_nicename ) ) {
+            return false;
+        }
+
+        /**
+         * @since 3.27.7 changed `user` to `author` to avoid potential clashes with CPTs ( `author` is reserved
+         *  https://developer.wordpress.org/reference/functions/register_post_type/#reserved-post-types )
+         */
+        return untrailingslashit( wl_configuration_get_redlink_dataset_uri() ) . "/author/$user->user_nicename";
+    }
+
+    /**
+     * Store the URI in user's meta.
+     *
+     * @param int    $user_id The user's id.
+     * @param string $user_uri The user's uri.
+     *
+     * @return int|bool Meta ID if the key didn't exist, true on successful update, false on failure.
+     * @since 3.1.7
+     */
+    // phpcs:ignore PSR2.Methods.MethodDeclaration.Underscore
+    private function _set_uri( $user_id, $user_uri ) {
+
+        return update_user_meta( $user_id, self::URI_META_KEY, $user_uri );
+    }
+
+    /**
+     * Mark an editor user as denied from editing entities.
+     * Does nothing if the user is not an editor
+     *
+     * @param integer $user_id The ID of the user
+     *
+     * @since 3.14.0
+     */
+    public function deny_editor_entity_create( $user_id ) {
+
+        // Bail out if the user is not an editor.
+        if ( ! $this->is_editor( $user_id ) ) {
+            return;
+        }
+
+        // The user explicitly do not have the capability.
+        update_user_option( $user_id, self::DENY_ENTITY_CREATE_META_KEY, 'yes' );
+
+    }
+
+    /**
+     * Remove the "deny entity editing" mark from an editor user.
+     * Does nothing if the user is not an editor
+     *
+     * @param integer $user_id The ID of the user
+     *
+     * @since 3.14.0
+     */
+    public function allow_editor_entity_create( $user_id ) {
+
+        // Bail out if the user is not an editor.
+        if ( ! $this->is_editor( $user_id ) ) {
+            return;
+        }
+
+        // The user explicitly do not have the capability.
+        delete_user_option( $user_id, self::DENY_ENTITY_CREATE_META_KEY );
+
+    }
+
+    /**
+     * Get whether the 'deny editor entity editing' flag is set.
+     *
+     * @param int $user_id The {@link WP_User} `id`.
+     *
+     * @return int bool True if editing is denied otherwise false.
+     * @since 3.14.0
+     */
+    public function is_deny_editor_entity_create( $user_id ) {
+
+        return 'yes' === get_user_option( self::DENY_ENTITY_CREATE_META_KEY, $user_id );
+    }
+
+    /**
+     * Check whether the {@link WP_User} with the specified `id` is an editor,
+     * i.e. has the `editor` role.
+     *
+     * @param int $user_id The {@link WP_User} `id`.
+     *
+     * @return bool True if the {@link WP_User} is an editor otherwise false.
+     * @since 3.14.0
+     */
+    public function is_editor( $user_id ) {
+
+        // Get the user.
+        $user = get_user_by( 'id', $user_id );
+
+        // Return true, if the user is found and has the `editor` role.
+        return is_a( $user, 'WP_User' ) && in_array( 'editor', (array) $user->roles, true );
+    }
+
+    /**
+     * Check if an editor can create entities.
+     *
+     * @param int $user_id The user id of the user being checked.
+     *
+     * @return bool    false if it is an editor that is denied from edit entities, true otherwise.
+     * @since 3.14.0
+     */
+    public function editor_can_create_entities( $user_id ) {
+
+        // Return true if not an editor.
+        if ( ! $this->is_editor( $user_id ) ) {
+            return true;
+        }
+
+        // Check if the user explicitly denied.
+        return ! $this->is_deny_editor_entity_create( $user_id );
+    }
+
+    /**
+     * Filter capabilities of user.
+     *
+     * Deny the capability of managing and editing entities for some users.
+     *
+     * @param array $allcaps All the capabilities of the user
+     * @param array $cap [0] Required capability
+     * @param array $args [0] Requested capability
+     *                       [1] User ID
+     *                       [2] Associated object ID
+     *
+     * @return array The capabilities array.
+     * @since 3.14.0
+     */
+    public function has_cap( $allcaps, $cap, $args ) {
+        /*
 		 * For entity management/editing related capabilities
 		 * check that an editor was not explicitly denied (in user profile)
 		 * the capability.
 		 */
 
-		/*
+        /*
 		 * Need protection against the case of edit_user and likes which do not
 		 * require a capability, just request one.
 		 */
-		if ( empty( $cap ) || ! isset( $cap[0] ) ) {
-			return $allcaps;
-		}
-
-		if (
-			( 'edit_wordlift_entity' === $cap[0] ) ||
-			( 'edit_wordlift_entities' === $cap[0] ) ||
-			( 'edit_others_wordlift_entities' === $cap[0] ) ||
-			( 'publish_wordlift_entities' === $cap[0] ) ||
-			( 'read_private_wordlift_entities' === $cap[0] ) ||
-			( 'delete_wordlift_entity' === $cap[0] ) ||
-			( 'delete_wordlift_entities' === $cap[0] ) ||
-			( 'delete_others_wordlift_entities' === $cap[0] ) ||
-			( 'delete_published_wordlift_entities' === $cap[0] ) ||
-			( 'delete_private_wordlift_entities' === $cap[0] )
-		) {
-			$user_id = $args[1];
-
-			if ( ! $this->editor_can_create_entities( $user_id ) ) {
-				$allcaps[ $cap[0] ] = false;
-			}
-		}
-
-		return $allcaps;
-	}
+        if ( empty( $cap ) || ! isset( $cap[0] ) ) {
+            return $allcaps;
+        }
+
+        if (
+            ( 'edit_wordlift_entity' === $cap[0] ) ||
+            ( 'edit_wordlift_entities' === $cap[0] ) ||
+            ( 'edit_others_wordlift_entities' === $cap[0] ) ||
+            ( 'publish_wordlift_entities' === $cap[0] ) ||
+            ( 'read_private_wordlift_entities' === $cap[0] ) ||
+            ( 'delete_wordlift_entity' === $cap[0] ) ||
+            ( 'delete_wordlift_entities' === $cap[0] ) ||
+            ( 'delete_others_wordlift_entities' === $cap[0] ) ||
+            ( 'delete_published_wordlift_entities' === $cap[0] ) ||
+            ( 'delete_private_wordlift_entities' === $cap[0] )
+        ) {
+            $user_id = $args[1];
+
+            if ( ! $this->editor_can_create_entities( $user_id ) ) {
+                $allcaps[ $cap[0] ] = false;
+            }
+        }
+
+        return $allcaps;
+    }
 
 }

Spacing +52 added lines, -52 removed lines

@@ -56,9 +56,9 @@ 
 	 */
 	protected function __construct() {
 
-		$this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_User_Service' );
+		$this->log_service = Wordlift_Log_Service::get_logger('Wordlift_User_Service');
 
-		add_filter( 'user_has_cap', array( $this, 'has_cap' ), 10, 3 );
+		add_filter('user_has_cap', array($this, 'has_cap'), 10, 3);
 
 	}
 
@@ -79,7 +79,7 @@ 
 	 */
 	public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -94,23 +94,23 @@ 
 	 * @return false|string The user's URI or false in case of failure.
 	 * @since 3.1.7
 	 */
-	public function get_uri( $user_id ) {
+	public function get_uri($user_id) {
 
 		// Try to get the URI stored in the user's meta and return it if available.
 		$dataset_uri = wl_configuration_get_redlink_dataset_uri();
-		$user_uri    = $this->_get_uri( $user_id );
-		if ( ! empty( $dataset_uri ) && ! empty( $user_uri ) && 0 === strpos( $user_uri, $dataset_uri ) ) {
+		$user_uri    = $this->_get_uri($user_id);
+		if ( ! empty($dataset_uri) && ! empty($user_uri) && 0 === strpos($user_uri, $dataset_uri)) {
 			return $user_uri;
 		}
 
 		// Try to build an URI, return false in case of failure.
-		$user_uri = $this->_build_uri( $user_id );
-		if ( false === $user_uri ) {
+		$user_uri = $this->_build_uri($user_id);
+		if (false === $user_uri) {
 			return false;
 		}
 
 		// Store the URI for future requests (we need a "permanent" URI).
-		$this->_set_uri( $user_id, $user_uri );
+		$this->_set_uri($user_id, $user_uri);
 
 		return $user_uri;
 	}
@@ -126,11 +126,11 @@ 
 	 * @return bool|int  Meta ID if the key didn't exist, true on successful update, false on failure.
 	 * @since 3.14.0
 	 */
-	public function set_entity( $user_id, $value ) {
+	public function set_entity($user_id, $value) {
 
 		return 0 < $value
-			? update_user_meta( $user_id, self::ENTITY_META_KEY, $value )
-			: delete_user_meta( $user_id, self::ENTITY_META_KEY );
+			? update_user_meta($user_id, self::ENTITY_META_KEY, $value)
+			: delete_user_meta($user_id, self::ENTITY_META_KEY);
 	}
 
 	/**
@@ -141,9 +141,9 @@ 
 	 * @return string|false The entity {@link WP_Post} `id` or an empty string if not set or false if the object id is invalid
 	 * @since 3.14.0
 	 */
-	public function get_entity( $user_id ) {
+	public function get_entity($user_id) {
 
-		return get_user_meta( $user_id, self::ENTITY_META_KEY, true );
+		return get_user_meta($user_id, self::ENTITY_META_KEY, true);
 	}
 
 	/**
@@ -155,11 +155,11 @@ 
 	 * @since 3.1.7
 	 */
 	// phpcs:ignore PSR2.Methods.MethodDeclaration.Underscore
-	private function _get_uri( $user_id ) {
+	private function _get_uri($user_id) {
 
-		$user_uri = get_user_meta( $user_id, self::URI_META_KEY, true );
+		$user_uri = get_user_meta($user_id, self::URI_META_KEY, true);
 
-		if ( empty( $user_uri ) ) {
+		if (empty($user_uri)) {
 			return false;
 		}
 
@@ -175,16 +175,16 @@ 
 	 * @since 3.1.7
 	 */
 	// phpcs:ignore PSR2.Methods.MethodDeclaration.Underscore
-	private function _build_uri( $user_id ) {
+	private function _build_uri($user_id) {
 
 		// Get the user, return false in case of failure.
-		$user = get_userdata( $user_id );
-		if ( false === $user ) {
+		$user = get_userdata($user_id);
+		if (false === $user) {
 			return false;
 		};
 
 		// If the nicename is not set, return a failure.
-		if ( empty( $user->user_nicename ) ) {
+		if (empty($user->user_nicename)) {
 			return false;
 		}
 
@@ -192,7 +192,7 @@ 
 		 * @since 3.27.7 changed `user` to `author` to avoid potential clashes with CPTs ( `author` is reserved
 		 *  https://developer.wordpress.org/reference/functions/register_post_type/#reserved-post-types )
 		 */
-		return untrailingslashit( wl_configuration_get_redlink_dataset_uri() ) . "/author/$user->user_nicename";
+		return untrailingslashit(wl_configuration_get_redlink_dataset_uri())."/author/$user->user_nicename";
 	}
 
 	/**
@@ -205,9 +205,9 @@ 
 	 * @since 3.1.7
 	 */
 	// phpcs:ignore PSR2.Methods.MethodDeclaration.Underscore
-	private function _set_uri( $user_id, $user_uri ) {
+	private function _set_uri($user_id, $user_uri) {
 
-		return update_user_meta( $user_id, self::URI_META_KEY, $user_uri );
+		return update_user_meta($user_id, self::URI_META_KEY, $user_uri);
 	}
 
 	/**
@@ -218,15 +218,15 @@ 
 	 *
 	 * @since 3.14.0
 	 */
-	public function deny_editor_entity_create( $user_id ) {
+	public function deny_editor_entity_create($user_id) {
 
 		// Bail out if the user is not an editor.
-		if ( ! $this->is_editor( $user_id ) ) {
+		if ( ! $this->is_editor($user_id)) {
 			return;
 		}
 
 		// The user explicitly do not have the capability.
-		update_user_option( $user_id, self::DENY_ENTITY_CREATE_META_KEY, 'yes' );
+		update_user_option($user_id, self::DENY_ENTITY_CREATE_META_KEY, 'yes');
 
 	}
 
@@ -238,15 +238,15 @@ 
 	 *
 	 * @since 3.14.0
 	 */
-	public function allow_editor_entity_create( $user_id ) {
+	public function allow_editor_entity_create($user_id) {
 
 		// Bail out if the user is not an editor.
-		if ( ! $this->is_editor( $user_id ) ) {
+		if ( ! $this->is_editor($user_id)) {
 			return;
 		}
 
 		// The user explicitly do not have the capability.
-		delete_user_option( $user_id, self::DENY_ENTITY_CREATE_META_KEY );
+		delete_user_option($user_id, self::DENY_ENTITY_CREATE_META_KEY);
 
 	}
 
@@ -258,9 +258,9 @@ 
 	 * @return int bool True if editing is denied otherwise false.
 	 * @since 3.14.0
 	 */
-	public function is_deny_editor_entity_create( $user_id ) {
+	public function is_deny_editor_entity_create($user_id) {
 
-		return 'yes' === get_user_option( self::DENY_ENTITY_CREATE_META_KEY, $user_id );
+		return 'yes' === get_user_option(self::DENY_ENTITY_CREATE_META_KEY, $user_id);
 	}
 
 	/**
@@ -272,13 +272,13 @@ 
 	 * @return bool True if the {@link WP_User} is an editor otherwise false.
 	 * @since 3.14.0
 	 */
-	public function is_editor( $user_id ) {
+	public function is_editor($user_id) {
 
 		// Get the user.
-		$user = get_user_by( 'id', $user_id );
+		$user = get_user_by('id', $user_id);
 
 		// Return true, if the user is found and has the `editor` role.
-		return is_a( $user, 'WP_User' ) && in_array( 'editor', (array) $user->roles, true );
+		return is_a($user, 'WP_User') && in_array('editor', (array) $user->roles, true);
 	}
 
 	/**
@@ -289,15 +289,15 @@ 
 	 * @return bool    false if it is an editor that is denied from edit entities, true otherwise.
 	 * @since 3.14.0
 	 */
-	public function editor_can_create_entities( $user_id ) {
+	public function editor_can_create_entities($user_id) {
 
 		// Return true if not an editor.
-		if ( ! $this->is_editor( $user_id ) ) {
+		if ( ! $this->is_editor($user_id)) {
 			return true;
 		}
 
 		// Check if the user explicitly denied.
-		return ! $this->is_deny_editor_entity_create( $user_id );
+		return ! $this->is_deny_editor_entity_create($user_id);
 	}
 
 	/**
@@ -314,7 +314,7 @@ 
 	 * @return array The capabilities array.
 	 * @since 3.14.0
 	 */
-	public function has_cap( $allcaps, $cap, $args ) {
+	public function has_cap($allcaps, $cap, $args) {
 		/*
 		 * For entity management/editing related capabilities
 		 * check that an editor was not explicitly denied (in user profile)
@@ -325,26 +325,26 @@ 
 		 * Need protection against the case of edit_user and likes which do not
 		 * require a capability, just request one.
 		 */
-		if ( empty( $cap ) || ! isset( $cap[0] ) ) {
+		if (empty($cap) || ! isset($cap[0])) {
 			return $allcaps;
 		}
 
 		if (
-			( 'edit_wordlift_entity' === $cap[0] ) ||
-			( 'edit_wordlift_entities' === $cap[0] ) ||
-			( 'edit_others_wordlift_entities' === $cap[0] ) ||
-			( 'publish_wordlift_entities' === $cap[0] ) ||
-			( 'read_private_wordlift_entities' === $cap[0] ) ||
-			( 'delete_wordlift_entity' === $cap[0] ) ||
-			( 'delete_wordlift_entities' === $cap[0] ) ||
-			( 'delete_others_wordlift_entities' === $cap[0] ) ||
-			( 'delete_published_wordlift_entities' === $cap[0] ) ||
-			( 'delete_private_wordlift_entities' === $cap[0] )
+			('edit_wordlift_entity' === $cap[0]) ||
+			('edit_wordlift_entities' === $cap[0]) ||
+			('edit_others_wordlift_entities' === $cap[0]) ||
+			('publish_wordlift_entities' === $cap[0]) ||
+			('read_private_wordlift_entities' === $cap[0]) ||
+			('delete_wordlift_entity' === $cap[0]) ||
+			('delete_wordlift_entities' === $cap[0]) ||
+			('delete_others_wordlift_entities' === $cap[0]) ||
+			('delete_published_wordlift_entities' === $cap[0]) ||
+			('delete_private_wordlift_entities' === $cap[0])
 		) {
 			$user_id = $args[1];
 
-			if ( ! $this->editor_can_create_entities( $user_id ) ) {
-				$allcaps[ $cap[0] ] = false;
+			if ( ! $this->editor_can_create_entities($user_id)) {
+				$allcaps[$cap[0]] = false;
 			}
 		}
 

src/includes/class-wordlift-sample-data-ajax-adapter.php

Indentation +41 added lines, -41 removed lines

@@ -17,59 +17,59 @@
  */
 class Wordlift_Sample_Data_Ajax_Adapter {
 
-	/**
-	 * The {@link Wordlift_Sample_Data_Service} instance.
-	 *
-	 * @since  3.12.0
-	 * @access private
-	 * @var \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
-	 */
-	private $sample_data_service;
+    /**
+     * The {@link Wordlift_Sample_Data_Service} instance.
+     *
+     * @since  3.12.0
+     * @access private
+     * @var \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
+     */
+    private $sample_data_service;
 
-	/**
-	 * Create a {@link Wordlift_Sample_Data_Ajax_Adapter} instance.
-	 *
-	 * @since 3.12.0
-	 *
-	 * @param \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
-	 */
-	public function __construct( $sample_data_service ) {
+    /**
+     * Create a {@link Wordlift_Sample_Data_Ajax_Adapter} instance.
+     *
+     * @since 3.12.0
+     *
+     * @param \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
+     */
+    public function __construct( $sample_data_service ) {
 
-		$this->sample_data_service = $sample_data_service;
+        $this->sample_data_service = $sample_data_service;
 
-	}
+    }
 
-	/**
-	 * Handle the `wl_sample_data_create` ajax action.
-	 *
-	 * @since 3.12.0
-	 */
-	public function create() {
+    /**
+     * Handle the `wl_sample_data_create` ajax action.
+     *
+     * @since 3.12.0
+     */
+    public function create() {
 
-		// Clean any potential garbage before us.
-		ob_clean();
+        // Clean any potential garbage before us.
+        ob_clean();
 
-		// Create the sample data.
-		$this->sample_data_service->create();
+        // Create the sample data.
+        $this->sample_data_service->create();
 
-		// Send success.
-		wp_send_json_success();
+        // Send success.
+        wp_send_json_success();
 
-	}
+    }
 
-	public function delete() {
+    public function delete() {
 
-		// Clean any potential garbage before us.
-		ob_clean();
+        // Clean any potential garbage before us.
+        ob_clean();
 
-		// Create the sample data.
-		$this->sample_data_service->delete();
+        // Create the sample data.
+        $this->sample_data_service->delete();
 
-		// Send success.
-		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@header( 'Content-Disposition: inline' );
-		wp_send_json_success();
+        // Send success.
+        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
+        @header( 'Content-Disposition: inline' );
+        wp_send_json_success();
 
-	}
+    }
 
 }

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
 	 *
 	 * @param \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
 	 */
-	public function __construct( $sample_data_service ) {
+	public function __construct($sample_data_service) {
 
 		$this->sample_data_service = $sample_data_service;
 
@@ -67,7 +67,7 @@ 
 
 		// Send success.
 		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@header( 'Content-Disposition: inline' );
+		@header('Content-Disposition: inline');
 		wp_send_json_success();
 
 	}

src/includes/class-wordlift-i18n.php

Indentation +33 added lines, -33 removed lines

@@ -26,38 +26,38 @@
  */
 class Wordlift_I18n {
 
-	/**
-	 * The domain specified for this plugin.
-	 *
-	 * @since    1.0.0
-	 * @access   private
-	 * @var      string    $domain    The domain identifier for this plugin.
-	 */
-	private $domain;
-
-	/**
-	 * Load the plugin text domain for translation.
-	 *
-	 * @since    1.0.0
-	 */
-	public function load_plugin_textdomain() {
-
-		load_plugin_textdomain(
-			$this->domain,
-			false,
-			dirname( dirname( plugin_basename( __FILE__ ) ) ) . '/languages/'
-		);
-
-	}
-
-	/**
-	 * Set the domain equal to that of the specified domain.
-	 *
-	 * @since    1.0.0
-	 * @param    string $domain    The domain that represents the locale of this plugin.
-	 */
-	public function set_domain( $domain ) {
-		$this->domain = $domain;
-	}
+    /**
+     * The domain specified for this plugin.
+     *
+     * @since    1.0.0
+     * @access   private
+     * @var      string    $domain    The domain identifier for this plugin.
+     */
+    private $domain;
+
+    /**
+     * Load the plugin text domain for translation.
+     *
+     * @since    1.0.0
+     */
+    public function load_plugin_textdomain() {
+
+        load_plugin_textdomain(
+            $this->domain,
+            false,
+            dirname( dirname( plugin_basename( __FILE__ ) ) ) . '/languages/'
+        );
+
+    }
+
+    /**
+     * Set the domain equal to that of the specified domain.
+     *
+     * @since    1.0.0
+     * @param    string $domain    The domain that represents the locale of this plugin.
+     */
+    public function set_domain( $domain ) {
+        $this->domain = $domain;
+    }
 
 }

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
 		load_plugin_textdomain(
 			$this->domain,
 			false,
-			dirname( dirname( plugin_basename( __FILE__ ) ) ) . '/languages/'
+			dirname(dirname(plugin_basename(__FILE__))).'/languages/'
 		);
 
 	}
@@ -56,7 +56,7 @@ 
 	 * @since    1.0.0
 	 * @param    string $domain    The domain that represents the locale of this plugin.
 	 */
-	public function set_domain( $domain ) {
+	public function set_domain($domain) {
 		$this->domain = $domain;
 	}
 

src/includes/class-wordlift-entity-post-to-jsonld-converter.php

Indentation +203 added lines, -203 removed lines

@@ -13,214 +13,214 @@
  */
 class Wordlift_Entity_Post_To_Jsonld_Converter extends Wordlift_Abstract_Post_To_Jsonld_Converter {
 
-	/**
-	 * The {@link Wordlift_Schemaorg_Property_Service} or null if not provided.
-	 *
-	 * @since 3.20.0
-	 * @access private
-	 * @var null|Wordlift_Schemaorg_Property_Service The {@link Wordlift_Schemaorg_Property_Service} or null if not provided.
-	 */
-	private $schemaorg_property_service;
-
-	/**
-	 * The {@link Wordlift_Post_To_Jsonld_Converter} is used to convert entities that also have the `article` term.
-	 *
-	 * @since 3.25.2
-	 * @var \Wordlift_Post_To_Jsonld_Converter $post_to_jsonld_converter The {@link Wordlift_Post_To_Jsonld_Converter} instance.
-	 */
-	private $post_to_jsonld_converter;
-
-	/**
-	 * Wordlift_Entity_To_Jsonld_Converter constructor.
-	 *
-	 * @param \Wordlift_Entity_Type_Service        $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
-	 * @param \Wordlift_User_Service               $user_service A {@link Wordlift_User_Service} instance.
-	 * @param \Wordlift_Attachment_Service         $attachment_service A {@link Wordlift_Attachment_Service} instance.
-	 * @param \Wordlift_Property_Getter            $property_getter A {@link Wordlift_Property_Getter} instance.
-	 * @param \Wordlift_Schemaorg_Property_Service $schemaorg_property_service A {@link Wordlift_Schemaorg_Property_Service} instance.
-	 * @param \Wordlift_Post_To_Jsonld_Converter   $post_to_jsonld_converter The {@link Wordlift_Post_To_Jsonld_Converter} instance.
-	 *
-	 * @since 3.8.0
-	 */
-	public function __construct( $entity_type_service, $user_service, $attachment_service, $property_getter, $schemaorg_property_service, $post_to_jsonld_converter ) {
-		parent::__construct( $entity_type_service, $user_service, $attachment_service, $property_getter );
-		$this->schemaorg_property_service = $schemaorg_property_service;
-		$this->post_to_jsonld_converter   = $post_to_jsonld_converter;
-	}
-
-	/**
-	 * Convert the provided {@link WP_Post} to a JSON-LD array. Any entity reference
-	 * found while processing the post is set in the $references array.
-	 *
-	 * @param int   $post_id The {@link WP_Post} id.
-	 * @param array $references An array of entity references.
-	 * @param array $references_infos
-	 *
-	 * @return array A JSON-LD array.
-	 * @since 3.8.0
-	 */
-	public function convert( $post_id, &$references = array(), &$references_infos = array() ) {
-
-		// Get the post instance.
-		$post = get_post( $post_id );
-		if ( null === $post ) {
-			// Post not found.
-			return null;
-		}
-
-		// Get the base JSON-LD and the list of entities referenced by this entity.
-		if ( has_term( 'article', Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME, $post_id ) ) {
-			$jsonld = $this->post_to_jsonld_converter->convert( $post_id, $references, $references_infos );
-		} else {
-			$jsonld = parent::convert( $post_id, $references, $references_infos );
-		}
-
-		// Get the entity name.
-		$jsonld['name'] = html_entity_decode( $post->post_title );
-
-		// 3.13.0, add alternate names.
-		$alternative_labels = Wordlift_Entity_Service::get_instance()->get_alternative_labels( $post_id );
-		if ( 0 < count( $alternative_labels ) ) {
-			$jsonld['alternateName'] = $alternative_labels;
-		}
-
-		// Get the entity `@type` with custom fields set by the Wordlift_Schema_Service.
-		//
-		// This allows us to gather the basic properties as defined by the `Thing` entity type.
-		// Get the configured type custom fields.
-		$custom_fields = $this->entity_type_service->get_custom_fields_for_post( $post_id );
-
-		if ( isset( $custom_fields ) ) {
-			$this->process_type_custom_fields( $jsonld, $custom_fields, $post, $references, $references_infos );
-		}
-
-		/*
+    /**
+     * The {@link Wordlift_Schemaorg_Property_Service} or null if not provided.
+     *
+     * @since 3.20.0
+     * @access private
+     * @var null|Wordlift_Schemaorg_Property_Service The {@link Wordlift_Schemaorg_Property_Service} or null if not provided.
+     */
+    private $schemaorg_property_service;
+
+    /**
+     * The {@link Wordlift_Post_To_Jsonld_Converter} is used to convert entities that also have the `article` term.
+     *
+     * @since 3.25.2
+     * @var \Wordlift_Post_To_Jsonld_Converter $post_to_jsonld_converter The {@link Wordlift_Post_To_Jsonld_Converter} instance.
+     */
+    private $post_to_jsonld_converter;
+
+    /**
+     * Wordlift_Entity_To_Jsonld_Converter constructor.
+     *
+     * @param \Wordlift_Entity_Type_Service        $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
+     * @param \Wordlift_User_Service               $user_service A {@link Wordlift_User_Service} instance.
+     * @param \Wordlift_Attachment_Service         $attachment_service A {@link Wordlift_Attachment_Service} instance.
+     * @param \Wordlift_Property_Getter            $property_getter A {@link Wordlift_Property_Getter} instance.
+     * @param \Wordlift_Schemaorg_Property_Service $schemaorg_property_service A {@link Wordlift_Schemaorg_Property_Service} instance.
+     * @param \Wordlift_Post_To_Jsonld_Converter   $post_to_jsonld_converter The {@link Wordlift_Post_To_Jsonld_Converter} instance.
+     *
+     * @since 3.8.0
+     */
+    public function __construct( $entity_type_service, $user_service, $attachment_service, $property_getter, $schemaorg_property_service, $post_to_jsonld_converter ) {
+        parent::__construct( $entity_type_service, $user_service, $attachment_service, $property_getter );
+        $this->schemaorg_property_service = $schemaorg_property_service;
+        $this->post_to_jsonld_converter   = $post_to_jsonld_converter;
+    }
+
+    /**
+     * Convert the provided {@link WP_Post} to a JSON-LD array. Any entity reference
+     * found while processing the post is set in the $references array.
+     *
+     * @param int   $post_id The {@link WP_Post} id.
+     * @param array $references An array of entity references.
+     * @param array $references_infos
+     *
+     * @return array A JSON-LD array.
+     * @since 3.8.0
+     */
+    public function convert( $post_id, &$references = array(), &$references_infos = array() ) {
+
+        // Get the post instance.
+        $post = get_post( $post_id );
+        if ( null === $post ) {
+            // Post not found.
+            return null;
+        }
+
+        // Get the base JSON-LD and the list of entities referenced by this entity.
+        if ( has_term( 'article', Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME, $post_id ) ) {
+            $jsonld = $this->post_to_jsonld_converter->convert( $post_id, $references, $references_infos );
+        } else {
+            $jsonld = parent::convert( $post_id, $references, $references_infos );
+        }
+
+        // Get the entity name.
+        $jsonld['name'] = html_entity_decode( $post->post_title );
+
+        // 3.13.0, add alternate names.
+        $alternative_labels = Wordlift_Entity_Service::get_instance()->get_alternative_labels( $post_id );
+        if ( 0 < count( $alternative_labels ) ) {
+            $jsonld['alternateName'] = $alternative_labels;
+        }
+
+        // Get the entity `@type` with custom fields set by the Wordlift_Schema_Service.
+        //
+        // This allows us to gather the basic properties as defined by the `Thing` entity type.
+        // Get the configured type custom fields.
+        $custom_fields = $this->entity_type_service->get_custom_fields_for_post( $post_id );
+
+        if ( isset( $custom_fields ) ) {
+            $this->process_type_custom_fields( $jsonld, $custom_fields, $post, $references, $references_infos );
+        }
+
+        /*
 		 * Get the properties attached to the post.
 		 *
 		 * @since 3.20.0 We attach properties directly to the posts.
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/835
 		 */
-		// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
-			$this->process_post_properties( $jsonld, $post_id );
-		}
-
-		/**
-		 * Call the `wl_post_jsonld_array` filter. This filter allows 3rd parties to also modify the references.
-		 *
-		 * @param array $value {
-		 *
-		 * @type array $jsonld The JSON-LD structure.
-		 * @type int[] $references An array of post IDs.
-		 * }
-		 * @since 3.25.0
-		 *
-		 * @see https://www.geeklab.info/2010/04/wordpress-pass-variables-by-reference-with-apply_filter/
-		 *
-		 * @api
-		 */
-		$ret_val    = apply_filters(
-			'wl_entity_jsonld_array',
-			array(
-				'jsonld'     => $jsonld,
-				'references' => $references,
-			),
-			$post_id
-		);
-		$jsonld     = $ret_val['jsonld'];
-		$references = $ret_val['references'];
-
-		/**
-		 * Call the `wl_entity_jsonld` filter.
-		 *
-		 * @param array $jsonld The JSON-LD structure.
-		 * @param int $post_id The {@link WP_Post} `id`.
-		 * @param array $references The array of referenced entities.
-		 *
-		 * @since 3.20.0
-		 *
-		 * @api
-		 */
-		return apply_filters( 'wl_entity_jsonld', $this->post_process( $jsonld ), $post_id, $references );
-	}
-
-	/**
-	 * Process the properties attached to the {@link WP_Post}.
-	 *
-	 * @param array $jsonld The JSON-LD array.
-	 * @param int   $post_id The target {@link WP_Post} id.
-	 *
-	 * @since 3.20.0
-	 */
-	private function process_post_properties( &$jsonld, $post_id ) {
-
-		// Get all the props.
-		$props = $this->schemaorg_property_service->get_all( $post_id );
-
-		// Process all the props.
-		foreach ( $props as $name => $instances ) {
-
-			// Get the values.
-			$values = array_map(
-				function ( $instance ) {
-					return $instance['value'];
-				},
-				$instances
-			);
-
-			// We might receive empty values, remove them.
-			$non_empty_values = array_filter(
-				$values,
-				function ( $value ) {
-					return ! empty( $value );
-				}
-			);
-
-			// Skip empty properties.
-			if ( empty( $non_empty_values ) ) {
-				continue;
-			}
-
-			// @@todo: need to handle maybe Numbers and URLs differently.
-			// Make an array a single value when possible.
-			$jsonld[ $name ] = self::make_one( $non_empty_values );
-		}
-
-	}
-
-	/**
-	 * Post process the generated JSON to reorganize values which are stored as 1st
-	 * level in WP but are really 2nd level.
-	 *
-	 * @param array $jsonld An array of JSON-LD properties and values.
-	 *
-	 * @return array The array remapped.
-	 * @since 3.8.0
-	 */
-	private function post_process( $jsonld ) {
-
-		foreach ( $jsonld as $key => $value ) {
-			if ( 'streetAddress' === $key || 'postalCode' === $key || 'addressLocality' === $key || 'addressRegion' === $key || 'addressCountry' === $key || 'postOfficeBoxNumber' === $key ) {
-				$jsonld['address']['@type'] = 'PostalAddress';
-				$jsonld['address'][ $key ]  = $value;
-				unset( $jsonld[ $key ] );
-			}
-
-			if ( 'latitude' === $key || 'longitude' === $key ) {
-				$jsonld['geo']['@type'] = 'GeoCoordinates';
-				$jsonld['geo'][ $key ]  = $value;
-				unset( $jsonld[ $key ] );
-			}
-
-			if ( 'calories' === $key ) {
-				$jsonld['nutrition']['@type'] = 'NutritionInformation';
-				$jsonld['nutrition'][ $key ]  = $value;
-				unset( $jsonld[ $key ] );
-			}
-		}
-
-		return $jsonld;
-	}
+        // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+        if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
+            $this->process_post_properties( $jsonld, $post_id );
+        }
+
+        /**
+         * Call the `wl_post_jsonld_array` filter. This filter allows 3rd parties to also modify the references.
+         *
+         * @param array $value {
+         *
+         * @type array $jsonld The JSON-LD structure.
+         * @type int[] $references An array of post IDs.
+         * }
+         * @since 3.25.0
+         *
+         * @see https://www.geeklab.info/2010/04/wordpress-pass-variables-by-reference-with-apply_filter/
+         *
+         * @api
+         */
+        $ret_val    = apply_filters(
+            'wl_entity_jsonld_array',
+            array(
+                'jsonld'     => $jsonld,
+                'references' => $references,
+            ),
+            $post_id
+        );
+        $jsonld     = $ret_val['jsonld'];
+        $references = $ret_val['references'];
+
+        /**
+         * Call the `wl_entity_jsonld` filter.
+         *
+         * @param array $jsonld The JSON-LD structure.
+         * @param int $post_id The {@link WP_Post} `id`.
+         * @param array $references The array of referenced entities.
+         *
+         * @since 3.20.0
+         *
+         * @api
+         */
+        return apply_filters( 'wl_entity_jsonld', $this->post_process( $jsonld ), $post_id, $references );
+    }
+
+    /**
+     * Process the properties attached to the {@link WP_Post}.
+     *
+     * @param array $jsonld The JSON-LD array.
+     * @param int   $post_id The target {@link WP_Post} id.
+     *
+     * @since 3.20.0
+     */
+    private function process_post_properties( &$jsonld, $post_id ) {
+
+        // Get all the props.
+        $props = $this->schemaorg_property_service->get_all( $post_id );
+
+        // Process all the props.
+        foreach ( $props as $name => $instances ) {
+
+            // Get the values.
+            $values = array_map(
+                function ( $instance ) {
+                    return $instance['value'];
+                },
+                $instances
+            );
+
+            // We might receive empty values, remove them.
+            $non_empty_values = array_filter(
+                $values,
+                function ( $value ) {
+                    return ! empty( $value );
+                }
+            );
+
+            // Skip empty properties.
+            if ( empty( $non_empty_values ) ) {
+                continue;
+            }
+
+            // @@todo: need to handle maybe Numbers and URLs differently.
+            // Make an array a single value when possible.
+            $jsonld[ $name ] = self::make_one( $non_empty_values );
+        }
+
+    }
+
+    /**
+     * Post process the generated JSON to reorganize values which are stored as 1st
+     * level in WP but are really 2nd level.
+     *
+     * @param array $jsonld An array of JSON-LD properties and values.
+     *
+     * @return array The array remapped.
+     * @since 3.8.0
+     */
+    private function post_process( $jsonld ) {
+
+        foreach ( $jsonld as $key => $value ) {
+            if ( 'streetAddress' === $key || 'postalCode' === $key || 'addressLocality' === $key || 'addressRegion' === $key || 'addressCountry' === $key || 'postOfficeBoxNumber' === $key ) {
+                $jsonld['address']['@type'] = 'PostalAddress';
+                $jsonld['address'][ $key ]  = $value;
+                unset( $jsonld[ $key ] );
+            }
+
+            if ( 'latitude' === $key || 'longitude' === $key ) {
+                $jsonld['geo']['@type'] = 'GeoCoordinates';
+                $jsonld['geo'][ $key ]  = $value;
+                unset( $jsonld[ $key ] );
+            }
+
+            if ( 'calories' === $key ) {
+                $jsonld['nutrition']['@type'] = 'NutritionInformation';
+                $jsonld['nutrition'][ $key ]  = $value;
+                unset( $jsonld[ $key ] );
+            }
+        }
+
+        return $jsonld;
+    }
 
 }

Spacing +37 added lines, -37 removed lines

@@ -42,8 +42,8 @@ 
 	 *
 	 * @since 3.8.0
 	 */
-	public function __construct( $entity_type_service, $user_service, $attachment_service, $property_getter, $schemaorg_property_service, $post_to_jsonld_converter ) {
-		parent::__construct( $entity_type_service, $user_service, $attachment_service, $property_getter );
+	public function __construct($entity_type_service, $user_service, $attachment_service, $property_getter, $schemaorg_property_service, $post_to_jsonld_converter) {
+		parent::__construct($entity_type_service, $user_service, $attachment_service, $property_getter);
 		$this->schemaorg_property_service = $schemaorg_property_service;
 		$this->post_to_jsonld_converter   = $post_to_jsonld_converter;
 	}
@@ -59,28 +59,28 @@ 
 	 * @return array A JSON-LD array.
 	 * @since 3.8.0
 	 */
-	public function convert( $post_id, &$references = array(), &$references_infos = array() ) {
+	public function convert($post_id, &$references = array(), &$references_infos = array()) {
 
 		// Get the post instance.
-		$post = get_post( $post_id );
-		if ( null === $post ) {
+		$post = get_post($post_id);
+		if (null === $post) {
 			// Post not found.
 			return null;
 		}
 
 		// Get the base JSON-LD and the list of entities referenced by this entity.
-		if ( has_term( 'article', Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME, $post_id ) ) {
-			$jsonld = $this->post_to_jsonld_converter->convert( $post_id, $references, $references_infos );
+		if (has_term('article', Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME, $post_id)) {
+			$jsonld = $this->post_to_jsonld_converter->convert($post_id, $references, $references_infos);
 		} else {
-			$jsonld = parent::convert( $post_id, $references, $references_infos );
+			$jsonld = parent::convert($post_id, $references, $references_infos);
 		}
 
 		// Get the entity name.
-		$jsonld['name'] = html_entity_decode( $post->post_title );
+		$jsonld['name'] = html_entity_decode($post->post_title);
 
 		// 3.13.0, add alternate names.
-		$alternative_labels = Wordlift_Entity_Service::get_instance()->get_alternative_labels( $post_id );
-		if ( 0 < count( $alternative_labels ) ) {
+		$alternative_labels = Wordlift_Entity_Service::get_instance()->get_alternative_labels($post_id);
+		if (0 < count($alternative_labels)) {
 			$jsonld['alternateName'] = $alternative_labels;
 		}
 
@@ -88,10 +88,10 @@ 
 		//
 		// This allows us to gather the basic properties as defined by the `Thing` entity type.
 		// Get the configured type custom fields.
-		$custom_fields = $this->entity_type_service->get_custom_fields_for_post( $post_id );
+		$custom_fields = $this->entity_type_service->get_custom_fields_for_post($post_id);
 
-		if ( isset( $custom_fields ) ) {
-			$this->process_type_custom_fields( $jsonld, $custom_fields, $post, $references, $references_infos );
+		if (isset($custom_fields)) {
+			$this->process_type_custom_fields($jsonld, $custom_fields, $post, $references, $references_infos);
 		}
 
 		/*
@@ -102,8 +102,8 @@ 
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/835
 		 */
 		// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
-			$this->process_post_properties( $jsonld, $post_id );
+		if (apply_filters('wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES)) {
+			$this->process_post_properties($jsonld, $post_id);
 		}
 
 		/**
@@ -120,7 +120,7 @@ 
 		 *
 		 * @api
 		 */
-		$ret_val    = apply_filters(
+		$ret_val = apply_filters(
 			'wl_entity_jsonld_array',
 			array(
 				'jsonld'     => $jsonld,
@@ -142,7 +142,7 @@ 
 		 *
 		 * @api
 		 */
-		return apply_filters( 'wl_entity_jsonld', $this->post_process( $jsonld ), $post_id, $references );
+		return apply_filters('wl_entity_jsonld', $this->post_process($jsonld), $post_id, $references);
 	}
 
 	/**
@@ -153,17 +153,17 @@ 
 	 *
 	 * @since 3.20.0
 	 */
-	private function process_post_properties( &$jsonld, $post_id ) {
+	private function process_post_properties(&$jsonld, $post_id) {
 
 		// Get all the props.
-		$props = $this->schemaorg_property_service->get_all( $post_id );
+		$props = $this->schemaorg_property_service->get_all($post_id);
 
 		// Process all the props.
-		foreach ( $props as $name => $instances ) {
+		foreach ($props as $name => $instances) {
 
 			// Get the values.
 			$values = array_map(
-				function ( $instance ) {
+				function($instance) {
 					return $instance['value'];
 				},
 				$instances
@@ -172,19 +172,19 @@ 
 			// We might receive empty values, remove them.
 			$non_empty_values = array_filter(
 				$values,
-				function ( $value ) {
-					return ! empty( $value );
+				function($value) {
+					return ! empty($value);
 				}
 			);
 
 			// Skip empty properties.
-			if ( empty( $non_empty_values ) ) {
+			if (empty($non_empty_values)) {
 				continue;
 			}
 
 			// @@todo: need to handle maybe Numbers and URLs differently.
 			// Make an array a single value when possible.
-			$jsonld[ $name ] = self::make_one( $non_empty_values );
+			$jsonld[$name] = self::make_one($non_empty_values);
 		}
 
 	}
@@ -198,25 +198,25 @@ 
 	 * @return array The array remapped.
 	 * @since 3.8.0
 	 */
-	private function post_process( $jsonld ) {
+	private function post_process($jsonld) {
 
-		foreach ( $jsonld as $key => $value ) {
-			if ( 'streetAddress' === $key || 'postalCode' === $key || 'addressLocality' === $key || 'addressRegion' === $key || 'addressCountry' === $key || 'postOfficeBoxNumber' === $key ) {
+		foreach ($jsonld as $key => $value) {
+			if ('streetAddress' === $key || 'postalCode' === $key || 'addressLocality' === $key || 'addressRegion' === $key || 'addressCountry' === $key || 'postOfficeBoxNumber' === $key) {
 				$jsonld['address']['@type'] = 'PostalAddress';
-				$jsonld['address'][ $key ]  = $value;
-				unset( $jsonld[ $key ] );
+				$jsonld['address'][$key]  = $value;
+				unset($jsonld[$key]);
 			}
 
-			if ( 'latitude' === $key || 'longitude' === $key ) {
+			if ('latitude' === $key || 'longitude' === $key) {
 				$jsonld['geo']['@type'] = 'GeoCoordinates';
-				$jsonld['geo'][ $key ]  = $value;
-				unset( $jsonld[ $key ] );
+				$jsonld['geo'][$key]  = $value;
+				unset($jsonld[$key]);
 			}
 
-			if ( 'calories' === $key ) {
+			if ('calories' === $key) {
 				$jsonld['nutrition']['@type'] = 'NutritionInformation';
-				$jsonld['nutrition'][ $key ]  = $value;
-				unset( $jsonld[ $key ] );
+				$jsonld['nutrition'][$key]  = $value;
+				unset($jsonld[$key]);
 			}
 		}
 

src/includes/class-wordlift-schema-url-property-service.php

Indentation +173 added lines, -173 removed lines

@@ -13,178 +13,178 @@
  */
 class Wordlift_Schema_Url_Property_Service extends Wordlift_Property_Service {
 
-	/**
-	 * The meta key used to store data for this property. We don't use wl_url to
-	 * avoid potential confusion about other URLs.
-	 *
-	 * @since 3.6.0
-	 */
-	const META_KEY = 'wl_schema_url';
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_rdf_predicate() {
-
-		return 'http://schema.org/url';
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_rdf_data_type() {
-
-		return 'xsd:anyURI';
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_data_type() {
-
-		return Wordlift_Schema_Service::DATA_TYPE_URI;
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_cardinality() {
-
-		return INF;
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_metabox_class() {
-
-		return 'Wl_Metabox_Field';
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_metabox_label() {
-
-		return __( 'Web Site(s)', 'wordlift' );
-	}
-
-	/**
-	 * Create a Wordlift_Schema_Url_Property_Service instance.
-	 *
-	 * @since 3.6.0
-	 */
-	public function __construct() {
-		parent::__construct();
-
-		// Finally listen for metadata requests for this field.
-		$this->add_filter_get_post_metadata();
-	}
-
-	/**
-	 * Get the schema:url value for the specified post/entity.
-	 *
-	 * @param int $post_id The post id.
-	 *
-	 * @return array|NULL The schema:url value or NULL if not set.
-	 * @since 3.6.0
-	 */
-	public function get( $post_id ) {
-
-		// Get the schema:url values set in WP.
-		$values = get_post_meta( $post_id, self::META_KEY, false );
-
-		// If the property has never been set, we set its default value the first
-		// time to <permalink>.
-		if ( empty( $values ) ) {
-			return array( '<permalink>' );
-		}
-
-		// If there's only one value and that value is empty, we return NULL, i.e.
-		// variable not set.
-		if ( 1 === count( $values ) && empty( $values[0] ) ) {
-			return null;
-		}
-
-		// Finally return whatever values the editor set.
-		return $values;
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function sanitize( $value ) {
-
-		// TODO: check that it's an URL or that is <permalink>
-
-		return $value;
-	}
-
-	/**
-	 * Get direct calls to read this meta and alter the response according to our
-	 * own strategy, i.e. if a value has never been set for this meta, then return
-	 * <permalink>.
-	 *
-	 * @param mixed  $value The original value.
-	 * @param int    $object_id The post id.
-	 * @param string $meta_key The meta key. We expect wl_schema_url or we return straight the value.
-	 * @param bool   $single Whether to return a single value.
-	 *
-	 * @return array|mixed|NULL|string
-	 * @since 3.6.0
-	 */
-	public function get_post_metadata( $value, $object_id, $meta_key, $single ) {
-
-		// It's not us, return the value.
-		if ( self::META_KEY !== $meta_key ) {
-			return $value;
-		}
-
-		$this->remove_filter_get_post_metadata();
-
-		$new_value = $this->get( $object_id );
-
-		$this->add_filter_get_post_metadata();
-
-		// If we must return a single value, but we don't have a value, then return an empty string.
-		if ( $single && ( $new_value === null || empty( $new_value ) ) ) {
-			return '';
-		}
-
-		// If we have a value and we need to return it as single, return the first value.
-		if ( $single ) {
-			return $new_value[0];
-		}
-
-		// Otherwise return the array.
-		return $new_value;
-	}
-
-	private function add_filter_get_post_metadata() {
-
-		add_filter(
-			'get_post_metadata',
-			array(
-				$this,
-				'get_post_metadata',
-			),
-			10,
-			4
-		);
-
-	}
-
-	private function remove_filter_get_post_metadata() {
-
-		remove_filter(
-			'get_post_metadata',
-			array(
-				$this,
-				'get_post_metadata',
-			),
-			10
-		);
-
-	}
+    /**
+     * The meta key used to store data for this property. We don't use wl_url to
+     * avoid potential confusion about other URLs.
+     *
+     * @since 3.6.0
+     */
+    const META_KEY = 'wl_schema_url';
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_rdf_predicate() {
+
+        return 'http://schema.org/url';
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_rdf_data_type() {
+
+        return 'xsd:anyURI';
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_data_type() {
+
+        return Wordlift_Schema_Service::DATA_TYPE_URI;
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_cardinality() {
+
+        return INF;
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_metabox_class() {
+
+        return 'Wl_Metabox_Field';
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_metabox_label() {
+
+        return __( 'Web Site(s)', 'wordlift' );
+    }
+
+    /**
+     * Create a Wordlift_Schema_Url_Property_Service instance.
+     *
+     * @since 3.6.0
+     */
+    public function __construct() {
+        parent::__construct();
+
+        // Finally listen for metadata requests for this field.
+        $this->add_filter_get_post_metadata();
+    }
+
+    /**
+     * Get the schema:url value for the specified post/entity.
+     *
+     * @param int $post_id The post id.
+     *
+     * @return array|NULL The schema:url value or NULL if not set.
+     * @since 3.6.0
+     */
+    public function get( $post_id ) {
+
+        // Get the schema:url values set in WP.
+        $values = get_post_meta( $post_id, self::META_KEY, false );
+
+        // If the property has never been set, we set its default value the first
+        // time to <permalink>.
+        if ( empty( $values ) ) {
+            return array( '<permalink>' );
+        }
+
+        // If there's only one value and that value is empty, we return NULL, i.e.
+        // variable not set.
+        if ( 1 === count( $values ) && empty( $values[0] ) ) {
+            return null;
+        }
+
+        // Finally return whatever values the editor set.
+        return $values;
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function sanitize( $value ) {
+
+        // TODO: check that it's an URL or that is <permalink>
+
+        return $value;
+    }
+
+    /**
+     * Get direct calls to read this meta and alter the response according to our
+     * own strategy, i.e. if a value has never been set for this meta, then return
+     * <permalink>.
+     *
+     * @param mixed  $value The original value.
+     * @param int    $object_id The post id.
+     * @param string $meta_key The meta key. We expect wl_schema_url or we return straight the value.
+     * @param bool   $single Whether to return a single value.
+     *
+     * @return array|mixed|NULL|string
+     * @since 3.6.0
+     */
+    public function get_post_metadata( $value, $object_id, $meta_key, $single ) {
+
+        // It's not us, return the value.
+        if ( self::META_KEY !== $meta_key ) {
+            return $value;
+        }
+
+        $this->remove_filter_get_post_metadata();
+
+        $new_value = $this->get( $object_id );
+
+        $this->add_filter_get_post_metadata();
+
+        // If we must return a single value, but we don't have a value, then return an empty string.
+        if ( $single && ( $new_value === null || empty( $new_value ) ) ) {
+            return '';
+        }
+
+        // If we have a value and we need to return it as single, return the first value.
+        if ( $single ) {
+            return $new_value[0];
+        }
+
+        // Otherwise return the array.
+        return $new_value;
+    }
+
+    private function add_filter_get_post_metadata() {
+
+        add_filter(
+            'get_post_metadata',
+            array(
+                $this,
+                'get_post_metadata',
+            ),
+            10,
+            4
+        );
+
+    }
+
+    private function remove_filter_get_post_metadata() {
+
+        remove_filter(
+            'get_post_metadata',
+            array(
+                $this,
+                'get_post_metadata',
+            ),
+            10
+        );
+
+    }
 
 }

Spacing +12 added lines, -12 removed lines

@@ -66,7 +66,7 @@ 
 	 */
 	public function get_metabox_label() {
 
-		return __( 'Web Site(s)', 'wordlift' );
+		return __('Web Site(s)', 'wordlift');
 	}
 
 	/**
@@ -89,20 +89,20 @@ 
 	 * @return array|NULL The schema:url value or NULL if not set.
 	 * @since 3.6.0
 	 */
-	public function get( $post_id ) {
+	public function get($post_id) {
 
 		// Get the schema:url values set in WP.
-		$values = get_post_meta( $post_id, self::META_KEY, false );
+		$values = get_post_meta($post_id, self::META_KEY, false);
 
 		// If the property has never been set, we set its default value the first
 		// time to <permalink>.
-		if ( empty( $values ) ) {
-			return array( '<permalink>' );
+		if (empty($values)) {
+			return array('<permalink>');
 		}
 
 		// If there's only one value and that value is empty, we return NULL, i.e.
 		// variable not set.
-		if ( 1 === count( $values ) && empty( $values[0] ) ) {
+		if (1 === count($values) && empty($values[0])) {
 			return null;
 		}
 
@@ -113,7 +113,7 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function sanitize( $value ) {
+	public function sanitize($value) {
 
 		// TODO: check that it's an URL or that is <permalink>
 
@@ -133,26 +133,26 @@ 
 	 * @return array|mixed|NULL|string
 	 * @since 3.6.0
 	 */
-	public function get_post_metadata( $value, $object_id, $meta_key, $single ) {
+	public function get_post_metadata($value, $object_id, $meta_key, $single) {
 
 		// It's not us, return the value.
-		if ( self::META_KEY !== $meta_key ) {
+		if (self::META_KEY !== $meta_key) {
 			return $value;
 		}
 
 		$this->remove_filter_get_post_metadata();
 
-		$new_value = $this->get( $object_id );
+		$new_value = $this->get($object_id);
 
 		$this->add_filter_get_post_metadata();
 
 		// If we must return a single value, but we don't have a value, then return an empty string.
-		if ( $single && ( $new_value === null || empty( $new_value ) ) ) {
+		if ($single && ($new_value === null || empty($new_value))) {
 			return '';
 		}
 
 		// If we have a value and we need to return it as single, return the first value.
-		if ( $single ) {
+		if ($single) {
 			return $new_value[0];
 		}
 

src/includes/class-wordlift-postid-to-jsonld-converter.php

Indentation +42 added lines, -42 removed lines

@@ -16,56 +16,56 @@
  */
 class Wordlift_Postid_To_Jsonld_Converter implements Wordlift_Post_Converter {
 
-	private $entity_post_to_jsonld_converter;
-	private $post_to_jsonld_converter;
+    private $entity_post_to_jsonld_converter;
+    private $post_to_jsonld_converter;
 
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.16.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.16.0
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
 
-	/**
-	 * Wordlift_Entity_To_Jsonld_Converter constructor.
-	 *
-	 * @param \Wordlift_Entity_Post_To_Jsonld_Converter $entity_post_to_jsonld_converter
-	 * @param \Wordlift_Post_To_Jsonld_Converter        $post_to_jsonld_converter
-	 *
-	 * @since 3.8.0
-	 */
-	public function __construct( $entity_post_to_jsonld_converter, $post_to_jsonld_converter ) {
+    /**
+     * Wordlift_Entity_To_Jsonld_Converter constructor.
+     *
+     * @param \Wordlift_Entity_Post_To_Jsonld_Converter $entity_post_to_jsonld_converter
+     * @param \Wordlift_Post_To_Jsonld_Converter        $post_to_jsonld_converter
+     *
+     * @since 3.8.0
+     */
+    public function __construct( $entity_post_to_jsonld_converter, $post_to_jsonld_converter ) {
 
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
+        $this->log = Wordlift_Log_Service::get_logger( get_class() );
 
-		$this->entity_post_to_jsonld_converter = $entity_post_to_jsonld_converter;
-		$this->post_to_jsonld_converter        = $post_to_jsonld_converter;
+        $this->entity_post_to_jsonld_converter = $entity_post_to_jsonld_converter;
+        $this->post_to_jsonld_converter        = $post_to_jsonld_converter;
 
-	}
+    }
 
-	/**
-	 * Convert a post to JSON-LD.
-	 *
-	 * @param string $id A post id (post or entity).
-	 * @param array  $references
-	 * @param array  $references_infos
-	 *
-	 * @return array|NULL A JSON-LD array representation of the post with the provided id, or NULL if not found.
-	 * @since 3.8.0
-	 */
-	public function convert( $id, &$references = array(), &$references_infos = array() ) {
+    /**
+     * Convert a post to JSON-LD.
+     *
+     * @param string $id A post id (post or entity).
+     * @param array  $references
+     * @param array  $references_infos
+     *
+     * @return array|NULL A JSON-LD array representation of the post with the provided id, or NULL if not found.
+     * @since 3.8.0
+     */
+    public function convert( $id, &$references = array(), &$references_infos = array() ) {
 
-		$this->log->trace( "Converting post $id..." );
+        $this->log->trace( "Converting post $id..." );
 
-		$entity_service = Wordlift_Entity_Service::get_instance();
+        $entity_service = Wordlift_Entity_Service::get_instance();
 
-		return $entity_service->is_entity( $id )
-			// Entity.
-			? $this->entity_post_to_jsonld_converter->convert( $id, $references, $references_infos )
-			// Post/Page.
-			: $this->post_to_jsonld_converter->convert( $id, $references );
-	}
+        return $entity_service->is_entity( $id )
+            // Entity.
+            ? $this->entity_post_to_jsonld_converter->convert( $id, $references, $references_infos )
+            // Post/Page.
+            : $this->post_to_jsonld_converter->convert( $id, $references );
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -36,9 +36,9 @@ 
 	 *
 	 * @since 3.8.0
 	 */
-	public function __construct( $entity_post_to_jsonld_converter, $post_to_jsonld_converter ) {
+	public function __construct($entity_post_to_jsonld_converter, $post_to_jsonld_converter) {
 
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
+		$this->log = Wordlift_Log_Service::get_logger(get_class());
 
 		$this->entity_post_to_jsonld_converter = $entity_post_to_jsonld_converter;
 		$this->post_to_jsonld_converter        = $post_to_jsonld_converter;
@@ -55,17 +55,17 @@ 
 	 * @return array|NULL A JSON-LD array representation of the post with the provided id, or NULL if not found.
 	 * @since 3.8.0
 	 */
-	public function convert( $id, &$references = array(), &$references_infos = array() ) {
+	public function convert($id, &$references = array(), &$references_infos = array()) {
 
-		$this->log->trace( "Converting post $id..." );
+		$this->log->trace("Converting post $id...");
 
 		$entity_service = Wordlift_Entity_Service::get_instance();
 
-		return $entity_service->is_entity( $id )
+		return $entity_service->is_entity($id)
 			// Entity.
-			? $this->entity_post_to_jsonld_converter->convert( $id, $references, $references_infos )
+			? $this->entity_post_to_jsonld_converter->convert($id, $references, $references_infos)
 			// Post/Page.
-			: $this->post_to_jsonld_converter->convert( $id, $references );
+			: $this->post_to_jsonld_converter->convert($id, $references);
 	}
 
 }

src/includes/class-wordlift-entity-post-type-service.php

Indentation +171 added lines, -171 removed lines

@@ -16,176 +16,176 @@
  */
 class Wordlift_Entity_Post_Type_Service {
 
-	/**
-	 * The entity post type.
-	 *
-	 * @since  3.6.0
-	 * @access private
-	 * @var string $post_type The entity post type.
-	 */
-	private $post_type;
-
-	/**
-	 * The entity type slug.
-	 *
-	 * @since  3.6.0
-	 * @access private
-	 * @var string $slug The entity type slug.
-	 */
-	private $slug;
-
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.16.3
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * A singleton instance of the entity type service.
-	 *
-	 * @since  3.6.0
-	 * @access private
-	 * @var Wordlift_Entity_Post_Type_Service
-	 */
-	private static $instance;
-
-	/**
-	 * Create an entity type service instance.
-	 *
-	 * @param string $post_type The post type, e.g. entity.
-	 * @param string $slug The entity type slug, if the slug is empty, the default slug will be used.
-	 *
-	 * @since 3.6.0
-	 */
-	public function __construct( $post_type, $slug ) {
-
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
-
-		$this->post_type = $post_type;
-
-		// We cannot assign an empty slug to the register_post_type function, therefore if the slug is empty we default
-		// to the type name.
-		$this->slug = $slug ? $slug : $post_type;
-
-		self::$instance = $this;
-
-	}
-
-	/**
-	 * Get the entity type service singleton instance.
-	 *
-	 * @return Wordlift_Entity_Post_Type_Service The entity type service singleton instance.
-	 * @since 3.6.0
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Get the entity type slug.
-	 *
-	 * @return string The entity type slug.
-	 * @since 3.6.0
-	 */
-	public function get_slug() {
-
-		return $this->slug;
-	}
-
-	/**
-	 * Get the entity post type name.
-	 *
-	 * @return string The entity post type.
-	 * @since 3.6.0
-	 */
-	public function get_post_type() {
-
-		return $this->post_type;
-	}
-
-	/**
-	 * Register the WordLift entity post type. This method is hooked to WordPress' init action.
-	 *
-	 * @since 3.6.0
-	 */
-	public function register() {
-
-		$this->log->debug( "Registering post type $this->post_type..." );
-
-		$labels = array(
-			'name'               => _x( 'Vocabulary', 'post type general name', 'wordlift' ),
-			'singular_name'      => _x( 'Entity', 'post type singular name', 'wordlift' ),
-			'add_new'            => _x( 'Add New Entity', 'entity', 'wordlift' ),
-			'add_new_item'       => __( 'Add New Entity', 'wordlift' ),
-			'edit_item'          => __( 'Edit Entity', 'wordlift' ),
-			'new_item'           => __( 'New Entity', 'wordlift' ),
-			'all_items'          => __( 'All Entities', 'wordlift' ),
-			'view_item'          => __( 'View Entity', 'wordlift' ),
-			'search_items'       => __( 'Search in Vocabulary', 'wordlift' ),
-			'not_found'          => __( 'No entities found', 'wordlift' ),
-			'not_found_in_trash' => __( 'No entities found in the Trash', 'wordlift' ),
-			'parent_item_colon'  => '',
-			'menu_name'          => __( 'Vocabulary', 'wordlift' ),
-		);
-
-		$args = array(
-			'labels'          => $labels,
-			'description'     => 'Holds our vocabulary (set of entities) and entity specific data',
-			'public'          => true,
-			'menu_position'   => 20,
-			// after the pages menu.
-			// Add support for 'authors' and 'revisions':
-			// * see https://github.com/insideout10/wordlift-plugin/issues/395
-			// * see https://github.com/insideout10/wordlift-plugin/issues/376
-			'supports'        => array(
-				'title',
-				'editor',
-				'thumbnail',
-				'excerpt',
-				'comments',
-				'author',
-				'revisions',
-				'custom-fields',
-			),
-			'has_archive'     => true,
-			'menu_icon'       => WP_CONTENT_URL . '/plugins/wordlift/images/svg/wl-vocabulary-icon.svg',
-			// Although we define our slug here, we further manage linking to entities using the Wordlift_Entity_Link_Service.
-			'rewrite'         => array( 'slug' => $this->slug ),
-			'capability_type' => array(
-				'wordlift_entity',
-				'wordlift_entities',
-			),
-			'capabilities'    => array(
-				'delete_posts' => 'delete_wordlift_entities',
-			),
-			'rest_base'       => 'entities',
-			'show_in_rest'    => true,
-			/**
-			 * Filter: wl_feature__enable__vocabulary.
-			 *
-			 * @param bool whether the vocabulary needs to be shown in menu.
-			 *
-			 * @return bool
-			 * @since 3.27.6
-			 */
-			'show_in_menu'    => apply_filters( 'wl_feature__enable__vocabulary', true ),
-		);
-
-		register_post_type( $this->post_type, $args );
-
-		// Enable WP's standard `category` taxonomy for entities.
-		//
-		// While this enables editors to bind entities to the WP posts' category
-		// taxonomy, in Wordlift_Category_Taxonomy_Service we also need to alter
-		// WP's main category query to include the `entity` post type.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/442
-		register_taxonomy_for_object_type( 'category', $this->post_type );
-
-	}
+    /**
+     * The entity post type.
+     *
+     * @since  3.6.0
+     * @access private
+     * @var string $post_type The entity post type.
+     */
+    private $post_type;
+
+    /**
+     * The entity type slug.
+     *
+     * @since  3.6.0
+     * @access private
+     * @var string $slug The entity type slug.
+     */
+    private $slug;
+
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.16.3
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * A singleton instance of the entity type service.
+     *
+     * @since  3.6.0
+     * @access private
+     * @var Wordlift_Entity_Post_Type_Service
+     */
+    private static $instance;
+
+    /**
+     * Create an entity type service instance.
+     *
+     * @param string $post_type The post type, e.g. entity.
+     * @param string $slug The entity type slug, if the slug is empty, the default slug will be used.
+     *
+     * @since 3.6.0
+     */
+    public function __construct( $post_type, $slug ) {
+
+        $this->log = Wordlift_Log_Service::get_logger( get_class() );
+
+        $this->post_type = $post_type;
+
+        // We cannot assign an empty slug to the register_post_type function, therefore if the slug is empty we default
+        // to the type name.
+        $this->slug = $slug ? $slug : $post_type;
+
+        self::$instance = $this;
+
+    }
+
+    /**
+     * Get the entity type service singleton instance.
+     *
+     * @return Wordlift_Entity_Post_Type_Service The entity type service singleton instance.
+     * @since 3.6.0
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Get the entity type slug.
+     *
+     * @return string The entity type slug.
+     * @since 3.6.0
+     */
+    public function get_slug() {
+
+        return $this->slug;
+    }
+
+    /**
+     * Get the entity post type name.
+     *
+     * @return string The entity post type.
+     * @since 3.6.0
+     */
+    public function get_post_type() {
+
+        return $this->post_type;
+    }
+
+    /**
+     * Register the WordLift entity post type. This method is hooked to WordPress' init action.
+     *
+     * @since 3.6.0
+     */
+    public function register() {
+
+        $this->log->debug( "Registering post type $this->post_type..." );
+
+        $labels = array(
+            'name'               => _x( 'Vocabulary', 'post type general name', 'wordlift' ),
+            'singular_name'      => _x( 'Entity', 'post type singular name', 'wordlift' ),
+            'add_new'            => _x( 'Add New Entity', 'entity', 'wordlift' ),
+            'add_new_item'       => __( 'Add New Entity', 'wordlift' ),
+            'edit_item'          => __( 'Edit Entity', 'wordlift' ),
+            'new_item'           => __( 'New Entity', 'wordlift' ),
+            'all_items'          => __( 'All Entities', 'wordlift' ),
+            'view_item'          => __( 'View Entity', 'wordlift' ),
+            'search_items'       => __( 'Search in Vocabulary', 'wordlift' ),
+            'not_found'          => __( 'No entities found', 'wordlift' ),
+            'not_found_in_trash' => __( 'No entities found in the Trash', 'wordlift' ),
+            'parent_item_colon'  => '',
+            'menu_name'          => __( 'Vocabulary', 'wordlift' ),
+        );
+
+        $args = array(
+            'labels'          => $labels,
+            'description'     => 'Holds our vocabulary (set of entities) and entity specific data',
+            'public'          => true,
+            'menu_position'   => 20,
+            // after the pages menu.
+            // Add support for 'authors' and 'revisions':
+            // * see https://github.com/insideout10/wordlift-plugin/issues/395
+            // * see https://github.com/insideout10/wordlift-plugin/issues/376
+            'supports'        => array(
+                'title',
+                'editor',
+                'thumbnail',
+                'excerpt',
+                'comments',
+                'author',
+                'revisions',
+                'custom-fields',
+            ),
+            'has_archive'     => true,
+            'menu_icon'       => WP_CONTENT_URL . '/plugins/wordlift/images/svg/wl-vocabulary-icon.svg',
+            // Although we define our slug here, we further manage linking to entities using the Wordlift_Entity_Link_Service.
+            'rewrite'         => array( 'slug' => $this->slug ),
+            'capability_type' => array(
+                'wordlift_entity',
+                'wordlift_entities',
+            ),
+            'capabilities'    => array(
+                'delete_posts' => 'delete_wordlift_entities',
+            ),
+            'rest_base'       => 'entities',
+            'show_in_rest'    => true,
+            /**
+             * Filter: wl_feature__enable__vocabulary.
+             *
+             * @param bool whether the vocabulary needs to be shown in menu.
+             *
+             * @return bool
+             * @since 3.27.6
+             */
+            'show_in_menu'    => apply_filters( 'wl_feature__enable__vocabulary', true ),
+        );
+
+        register_post_type( $this->post_type, $args );
+
+        // Enable WP's standard `category` taxonomy for entities.
+        //
+        // While this enables editors to bind entities to the WP posts' category
+        // taxonomy, in Wordlift_Category_Taxonomy_Service we also need to alter
+        // WP's main category query to include the `entity` post type.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/442
+        register_taxonomy_for_object_type( 'category', $this->post_type );
+
+    }
 
 }

Spacing +20 added lines, -20 removed lines

@@ -60,9 +60,9 @@ 
 	 *
 	 * @since 3.6.0
 	 */
-	public function __construct( $post_type, $slug ) {
+	public function __construct($post_type, $slug) {
 
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
+		$this->log = Wordlift_Log_Service::get_logger(get_class());
 
 		$this->post_type = $post_type;
 
@@ -114,22 +114,22 @@ 
 	 */
 	public function register() {
 
-		$this->log->debug( "Registering post type $this->post_type..." );
+		$this->log->debug("Registering post type $this->post_type...");
 
 		$labels = array(
-			'name'               => _x( 'Vocabulary', 'post type general name', 'wordlift' ),
-			'singular_name'      => _x( 'Entity', 'post type singular name', 'wordlift' ),
-			'add_new'            => _x( 'Add New Entity', 'entity', 'wordlift' ),
-			'add_new_item'       => __( 'Add New Entity', 'wordlift' ),
-			'edit_item'          => __( 'Edit Entity', 'wordlift' ),
-			'new_item'           => __( 'New Entity', 'wordlift' ),
-			'all_items'          => __( 'All Entities', 'wordlift' ),
-			'view_item'          => __( 'View Entity', 'wordlift' ),
-			'search_items'       => __( 'Search in Vocabulary', 'wordlift' ),
-			'not_found'          => __( 'No entities found', 'wordlift' ),
-			'not_found_in_trash' => __( 'No entities found in the Trash', 'wordlift' ),
+			'name'               => _x('Vocabulary', 'post type general name', 'wordlift'),
+			'singular_name'      => _x('Entity', 'post type singular name', 'wordlift'),
+			'add_new'            => _x('Add New Entity', 'entity', 'wordlift'),
+			'add_new_item'       => __('Add New Entity', 'wordlift'),
+			'edit_item'          => __('Edit Entity', 'wordlift'),
+			'new_item'           => __('New Entity', 'wordlift'),
+			'all_items'          => __('All Entities', 'wordlift'),
+			'view_item'          => __('View Entity', 'wordlift'),
+			'search_items'       => __('Search in Vocabulary', 'wordlift'),
+			'not_found'          => __('No entities found', 'wordlift'),
+			'not_found_in_trash' => __('No entities found in the Trash', 'wordlift'),
 			'parent_item_colon'  => '',
-			'menu_name'          => __( 'Vocabulary', 'wordlift' ),
+			'menu_name'          => __('Vocabulary', 'wordlift'),
 		);
 
 		$args = array(
@@ -152,9 +152,9 @@ 
 				'custom-fields',
 			),
 			'has_archive'     => true,
-			'menu_icon'       => WP_CONTENT_URL . '/plugins/wordlift/images/svg/wl-vocabulary-icon.svg',
+			'menu_icon'       => WP_CONTENT_URL.'/plugins/wordlift/images/svg/wl-vocabulary-icon.svg',
 			// Although we define our slug here, we further manage linking to entities using the Wordlift_Entity_Link_Service.
-			'rewrite'         => array( 'slug' => $this->slug ),
+			'rewrite'         => array('slug' => $this->slug),
 			'capability_type' => array(
 				'wordlift_entity',
 				'wordlift_entities',
@@ -172,10 +172,10 @@ 
 			 * @return bool
 			 * @since 3.27.6
 			 */
-			'show_in_menu'    => apply_filters( 'wl_feature__enable__vocabulary', true ),
+			'show_in_menu'    => apply_filters('wl_feature__enable__vocabulary', true),
 		);
 
-		register_post_type( $this->post_type, $args );
+		register_post_type($this->post_type, $args);
 
 		// Enable WP's standard `category` taxonomy for entities.
 		//
@@ -184,7 +184,7 @@ 
 		// WP's main category query to include the `entity` post type.
 		//
 		// See https://github.com/insideout10/wordlift-plugin/issues/442
-		register_taxonomy_for_object_type( 'category', $this->post_type );
+		register_taxonomy_for_object_type('category', $this->post_type);
 
 	}
 

src/includes/class-wordlift-publisher-service.php

Indentation +227 added lines, -227 removed lines

@@ -18,235 +18,235 @@
  */
 class Wordlift_Publisher_Service {
 
-	protected function __construct() {
-
-	}
-
-	private static $instance = null;
-
-	public static function get_instance() {
-
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self();
-		}
-
-		return self::$instance;
-	}
-
-	/**
-	 * Counts the number of potential publishers.
-	 *
-	 * @return int The number of potential publishers.
-	 * @since 3.11.0
-	 */
-	public function count() {
-
-		// Search for entities which are either a Person
-		// or Organization.
-
-		// Get only the ids as all we need is the count.
-		$entities = get_posts(
-			array(
-				'post_type'      => Wordlift_Entity_Service::valid_entity_post_types(),
-				'post_status'    => 'publish',
-				'posts_per_page' => - 1,
-				'tax_query'      => array(
-					array(
-						'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
-						'field'    => 'slug',
-						'terms'    => array( 'organization', 'person' ),
-					),
-				),
-				'fields'         => 'ids',
-			)
-		);
-
-		// Finally return the count.
-		return count( $entities );
-	}
-
-	/**
-	 * Search SQL filter for matching against post title only.
-	 *
-	 * @link    http://wordpress.stackexchange.com/a/11826/1685
-	 *
-	 * @since   3.15.0
-	 *
-	 * @param string   $search The search string.
-	 * @param WP_Query $wp_query The {@link WP_Query} instance.
-	 *
-	 * @return array|string An array of results.
-	 */
-	public function limit_search_to_title( $search, $wp_query ) {
-
-		// Bail out if the search or the `search_terms` haven't been set.
-		if ( empty( $search ) || empty( $wp_query->query_vars['search_terms'] ) ) {
-			return $search;
-		}
-
-		global $wpdb;
-
-		$query_vars = $wp_query->query_vars;
-		$percent    = ! empty( $query_vars['exact'] ) ? '' : '%';
-		$search     = array();
-
-		foreach ( (array) $query_vars['search_terms'] as $term ) {
-			$search[] = $wpdb->prepare( "$wpdb->posts.post_title LIKE %s", $percent . $wpdb->esc_like( $term ) . $percent );
-		}
-
-		if ( ! is_user_logged_in() ) {
-			$search[] = "$wpdb->posts.post_password = ''";
-		}
-
-		$search = ' AND ' . implode( ' AND ', $search );
-
-		return $search;
-	}
-
-	/**
-	 * Query WP for potential publishers, i.e. {@link WP_Post}s which are associated`
-	 * with `wl_entity_type` (taxonomy) terms of `Organization` or `Person`.
-	 *
-	 * @param string $filter The title filter.
-	 *
-	 * @return array An array of results in a select2 friendly format.
-	 * @since 3.11.0
-	 */
-	public function query( $filter = '' ) {
-
-		// Search for the filter in the titles only.
-		add_filter(
-			'posts_search',
-			array(
-				$this,
-				'limit_search_to_title',
-			),
-			10,
-			2
-		);
-
-		/*
+    protected function __construct() {
+
+    }
+
+    private static $instance = null;
+
+    public static function get_instance() {
+
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self();
+        }
+
+        return self::$instance;
+    }
+
+    /**
+     * Counts the number of potential publishers.
+     *
+     * @return int The number of potential publishers.
+     * @since 3.11.0
+     */
+    public function count() {
+
+        // Search for entities which are either a Person
+        // or Organization.
+
+        // Get only the ids as all we need is the count.
+        $entities = get_posts(
+            array(
+                'post_type'      => Wordlift_Entity_Service::valid_entity_post_types(),
+                'post_status'    => 'publish',
+                'posts_per_page' => - 1,
+                'tax_query'      => array(
+                    array(
+                        'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
+                        'field'    => 'slug',
+                        'terms'    => array( 'organization', 'person' ),
+                    ),
+                ),
+                'fields'         => 'ids',
+            )
+        );
+
+        // Finally return the count.
+        return count( $entities );
+    }
+
+    /**
+     * Search SQL filter for matching against post title only.
+     *
+     * @link    http://wordpress.stackexchange.com/a/11826/1685
+     *
+     * @since   3.15.0
+     *
+     * @param string   $search The search string.
+     * @param WP_Query $wp_query The {@link WP_Query} instance.
+     *
+     * @return array|string An array of results.
+     */
+    public function limit_search_to_title( $search, $wp_query ) {
+
+        // Bail out if the search or the `search_terms` haven't been set.
+        if ( empty( $search ) || empty( $wp_query->query_vars['search_terms'] ) ) {
+            return $search;
+        }
+
+        global $wpdb;
+
+        $query_vars = $wp_query->query_vars;
+        $percent    = ! empty( $query_vars['exact'] ) ? '' : '%';
+        $search     = array();
+
+        foreach ( (array) $query_vars['search_terms'] as $term ) {
+            $search[] = $wpdb->prepare( "$wpdb->posts.post_title LIKE %s", $percent . $wpdb->esc_like( $term ) . $percent );
+        }
+
+        if ( ! is_user_logged_in() ) {
+            $search[] = "$wpdb->posts.post_password = ''";
+        }
+
+        $search = ' AND ' . implode( ' AND ', $search );
+
+        return $search;
+    }
+
+    /**
+     * Query WP for potential publishers, i.e. {@link WP_Post}s which are associated`
+     * with `wl_entity_type` (taxonomy) terms of `Organization` or `Person`.
+     *
+     * @param string $filter The title filter.
+     *
+     * @return array An array of results in a select2 friendly format.
+     * @since 3.11.0
+     */
+    public function query( $filter = '' ) {
+
+        // Search for the filter in the titles only.
+        add_filter(
+            'posts_search',
+            array(
+                $this,
+                'limit_search_to_title',
+            ),
+            10,
+            2
+        );
+
+        /*
 		 * Search for entities which are either a Person
 		 * or Organization. Sort the results by title in ascending order.
 		 */
-		$entities = get_posts(
-			array(
-				'post_type'      => Wordlift_Entity_Service::valid_entity_post_types(),
-				'post_status'    => 'publish',
-				'posts_per_page' => - 1,
-				'tax_query'      => array(
-					array(
-						'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
-						'field'    => 'slug',
-						'terms'    => array( 'organization', 'person' ),
-					),
-				),
-				's'              => $filter,
-				'orderby'        => 'title',
-				'order'          => 'ASC',
-			)
-		);
-
-		// Remove the search filter added before the query.
-		remove_filter(
-			'posts_search',
-			array(
-				$this,
-				'limit_search_to_title',
-			),
-			10,
-			2
-		);
-
-		// Set a reference to ourselves to pass to the closure.
-		$publisher_service = $this;
-
-		// Map the results in a `Select2` compatible array.
-		return array_map(
-			function ( $entity ) use ( $publisher_service ) {
-				$type     = wp_get_post_terms( $entity->ID, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
-				$thumb_id = get_post_thumbnail_id( $entity->ID );
-
-				return array(
-					'id'            => $entity->ID,
-					'text'          => $entity->post_title,
-					'type'          => $type[0]->name,
-					'thumbnail_url' => $publisher_service->get_attachment_image_url( $thumb_id ),
-				);
-			},
-			$entities
-		);
-	}
-
-	/**
-	 * Get the thumbnail's URL.
-	 *
-	 * @param int    $attachment_id The attachment id.
-	 * @param string $size The attachment size (default = 'thumbnail').
-	 *
-	 * @return string|bool The image URL or false if not found.
-	 * @since 3.11.0
-	 */
-	public function get_attachment_image_url( $attachment_id, $size = 'thumbnail' ) {
-
-		$image = wp_get_attachment_image_src( $attachment_id, $size );
-
-		return isset( $image['0'] ) ? $image['0'] : false;
-	}
-
-	/**
-	 * Add additional instructions to featured image metabox
-	 * when the entity type is the publisher.
-	 *
-	 * @param string $content Current metabox content.
-	 *
-	 * @return string $content metabox content with additional instructions.
-	 * @since  3.19.0
-	 */
-	public function add_featured_image_instruction( $content ) {
-		// Get the current post ID.
-		$post_id = get_the_ID();
-
-		// Get the publisher id.
-		$publisher_id = Wordlift_Configuration_Service::get_instance()->get_publisher_id();
-
-		// Bail if for some reason the post id is not set.
-		if (
-			empty( $post_id ) ||
-			$post_id !== (int) $publisher_id
-		) {
-			return $content;
-		}
-
-		$terms = wp_get_post_terms(
-			$post_id, // The post id.
-			Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME, // The taxonomy slug.
-			array(
-				'fields' => 'slugs',
-				// We don't need all fields, but only slugs.
-			)
-		);
-
-		// Check that the entity type is "Organization".
-		if ( in_array( 'organization', $terms, true ) ) {
-			// Add the featured image description when the type is "Organization".
-
-			$link     = sprintf(
-				'<a target="_blank" href="%s">%s</a>',
-				esc_attr__( 'https://developers.google.com/search/docs/data-types/article#logo-guidelines', 'wordlift' ),
-				esc_html__( 'AMP logo guidelines', 'wordlift' )
-			);
-			$content .= sprintf(
-				'<p>'
-								/* translators: %s: AMP logo guidelines. */
-								 . esc_html_x( 'According to the %s, the logo should fit in a 60x600px rectangle, and either be exactly 60px high (preferred), or exactly 600px wide. For example, 450x45px would not be acceptable, even though it fits in the 600x60px rectangle. To comply with the guidelines, WordLift will automatically resize the Featured Image for structured data formats.', 'After "According to the" goes the link to the "AMP logo guidelines".', 'wordlift' )
-								 . '</p>',
-				$link
-			);
-		}
-
-		// Finally return the content.
-		return $content;
-	}
+        $entities = get_posts(
+            array(
+                'post_type'      => Wordlift_Entity_Service::valid_entity_post_types(),
+                'post_status'    => 'publish',
+                'posts_per_page' => - 1,
+                'tax_query'      => array(
+                    array(
+                        'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
+                        'field'    => 'slug',
+                        'terms'    => array( 'organization', 'person' ),
+                    ),
+                ),
+                's'              => $filter,
+                'orderby'        => 'title',
+                'order'          => 'ASC',
+            )
+        );
+
+        // Remove the search filter added before the query.
+        remove_filter(
+            'posts_search',
+            array(
+                $this,
+                'limit_search_to_title',
+            ),
+            10,
+            2
+        );
+
+        // Set a reference to ourselves to pass to the closure.
+        $publisher_service = $this;
+
+        // Map the results in a `Select2` compatible array.
+        return array_map(
+            function ( $entity ) use ( $publisher_service ) {
+                $type     = wp_get_post_terms( $entity->ID, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
+                $thumb_id = get_post_thumbnail_id( $entity->ID );
+
+                return array(
+                    'id'            => $entity->ID,
+                    'text'          => $entity->post_title,
+                    'type'          => $type[0]->name,
+                    'thumbnail_url' => $publisher_service->get_attachment_image_url( $thumb_id ),
+                );
+            },
+            $entities
+        );
+    }
+
+    /**
+     * Get the thumbnail's URL.
+     *
+     * @param int    $attachment_id The attachment id.
+     * @param string $size The attachment size (default = 'thumbnail').
+     *
+     * @return string|bool The image URL or false if not found.
+     * @since 3.11.0
+     */
+    public function get_attachment_image_url( $attachment_id, $size = 'thumbnail' ) {
+
+        $image = wp_get_attachment_image_src( $attachment_id, $size );
+
+        return isset( $image['0'] ) ? $image['0'] : false;
+    }
+
+    /**
+     * Add additional instructions to featured image metabox
+     * when the entity type is the publisher.
+     *
+     * @param string $content Current metabox content.
+     *
+     * @return string $content metabox content with additional instructions.
+     * @since  3.19.0
+     */
+    public function add_featured_image_instruction( $content ) {
+        // Get the current post ID.
+        $post_id = get_the_ID();
+
+        // Get the publisher id.
+        $publisher_id = Wordlift_Configuration_Service::get_instance()->get_publisher_id();
+
+        // Bail if for some reason the post id is not set.
+        if (
+            empty( $post_id ) ||
+            $post_id !== (int) $publisher_id
+        ) {
+            return $content;
+        }
+
+        $terms = wp_get_post_terms(
+            $post_id, // The post id.
+            Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME, // The taxonomy slug.
+            array(
+                'fields' => 'slugs',
+                // We don't need all fields, but only slugs.
+            )
+        );
+
+        // Check that the entity type is "Organization".
+        if ( in_array( 'organization', $terms, true ) ) {
+            // Add the featured image description when the type is "Organization".
+
+            $link     = sprintf(
+                '<a target="_blank" href="%s">%s</a>',
+                esc_attr__( 'https://developers.google.com/search/docs/data-types/article#logo-guidelines', 'wordlift' ),
+                esc_html__( 'AMP logo guidelines', 'wordlift' )
+            );
+            $content .= sprintf(
+                '<p>'
+                                /* translators: %s: AMP logo guidelines. */
+                                    . esc_html_x( 'According to the %s, the logo should fit in a 60x600px rectangle, and either be exactly 60px high (preferred), or exactly 600px wide. For example, 450x45px would not be acceptable, even though it fits in the 600x60px rectangle. To comply with the guidelines, WordLift will automatically resize the Featured Image for structured data formats.', 'After "According to the" goes the link to the "AMP logo guidelines".', 'wordlift' )
+                                    . '</p>',
+                $link
+            );
+        }
+
+        // Finally return the content.
+        return $content;
+    }
 
 }

Spacing +28 added lines, -28 removed lines

@@ -26,7 +26,7 @@ 
 
 	public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -49,12 +49,12 @@ 
 			array(
 				'post_type'      => Wordlift_Entity_Service::valid_entity_post_types(),
 				'post_status'    => 'publish',
-				'posts_per_page' => - 1,
+				'posts_per_page' => -1,
 				'tax_query'      => array(
 					array(
 						'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
 						'field'    => 'slug',
-						'terms'    => array( 'organization', 'person' ),
+						'terms'    => array('organization', 'person'),
 					),
 				),
 				'fields'         => 'ids',
@@ -62,7 +62,7 @@ 
 		);
 
 		// Finally return the count.
-		return count( $entities );
+		return count($entities);
 	}
 
 	/**
@@ -77,28 +77,28 @@ 
 	 *
 	 * @return array|string An array of results.
 	 */
-	public function limit_search_to_title( $search, $wp_query ) {
+	public function limit_search_to_title($search, $wp_query) {
 
 		// Bail out if the search or the `search_terms` haven't been set.
-		if ( empty( $search ) || empty( $wp_query->query_vars['search_terms'] ) ) {
+		if (empty($search) || empty($wp_query->query_vars['search_terms'])) {
 			return $search;
 		}
 
 		global $wpdb;
 
 		$query_vars = $wp_query->query_vars;
-		$percent    = ! empty( $query_vars['exact'] ) ? '' : '%';
+		$percent    = ! empty($query_vars['exact']) ? '' : '%';
 		$search     = array();
 
-		foreach ( (array) $query_vars['search_terms'] as $term ) {
-			$search[] = $wpdb->prepare( "$wpdb->posts.post_title LIKE %s", $percent . $wpdb->esc_like( $term ) . $percent );
+		foreach ((array) $query_vars['search_terms'] as $term) {
+			$search[] = $wpdb->prepare("$wpdb->posts.post_title LIKE %s", $percent.$wpdb->esc_like($term).$percent);
 		}
 
-		if ( ! is_user_logged_in() ) {
+		if ( ! is_user_logged_in()) {
 			$search[] = "$wpdb->posts.post_password = ''";
 		}
 
-		$search = ' AND ' . implode( ' AND ', $search );
+		$search = ' AND '.implode(' AND ', $search);
 
 		return $search;
 	}
@@ -112,7 +112,7 @@ 
 	 * @return array An array of results in a select2 friendly format.
 	 * @since 3.11.0
 	 */
-	public function query( $filter = '' ) {
+	public function query($filter = '') {
 
 		// Search for the filter in the titles only.
 		add_filter(
@@ -133,12 +133,12 @@ 
 			array(
 				'post_type'      => Wordlift_Entity_Service::valid_entity_post_types(),
 				'post_status'    => 'publish',
-				'posts_per_page' => - 1,
+				'posts_per_page' => -1,
 				'tax_query'      => array(
 					array(
 						'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
 						'field'    => 'slug',
-						'terms'    => array( 'organization', 'person' ),
+						'terms'    => array('organization', 'person'),
 					),
 				),
 				's'              => $filter,
@@ -163,15 +163,15 @@ 
 
 		// Map the results in a `Select2` compatible array.
 		return array_map(
-			function ( $entity ) use ( $publisher_service ) {
-				$type     = wp_get_post_terms( $entity->ID, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
-				$thumb_id = get_post_thumbnail_id( $entity->ID );
+			function($entity) use ($publisher_service) {
+				$type     = wp_get_post_terms($entity->ID, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME);
+				$thumb_id = get_post_thumbnail_id($entity->ID);
 
 				return array(
 					'id'            => $entity->ID,
 					'text'          => $entity->post_title,
 					'type'          => $type[0]->name,
-					'thumbnail_url' => $publisher_service->get_attachment_image_url( $thumb_id ),
+					'thumbnail_url' => $publisher_service->get_attachment_image_url($thumb_id),
 				);
 			},
 			$entities
@@ -187,11 +187,11 @@ 
 	 * @return string|bool The image URL or false if not found.
 	 * @since 3.11.0
 	 */
-	public function get_attachment_image_url( $attachment_id, $size = 'thumbnail' ) {
+	public function get_attachment_image_url($attachment_id, $size = 'thumbnail') {
 
-		$image = wp_get_attachment_image_src( $attachment_id, $size );
+		$image = wp_get_attachment_image_src($attachment_id, $size);
 
-		return isset( $image['0'] ) ? $image['0'] : false;
+		return isset($image['0']) ? $image['0'] : false;
 	}
 
 	/**
@@ -203,7 +203,7 @@ 
 	 * @return string $content metabox content with additional instructions.
 	 * @since  3.19.0
 	 */
-	public function add_featured_image_instruction( $content ) {
+	public function add_featured_image_instruction($content) {
 		// Get the current post ID.
 		$post_id = get_the_ID();
 
@@ -212,7 +212,7 @@ 
 
 		// Bail if for some reason the post id is not set.
 		if (
-			empty( $post_id ) ||
+			empty($post_id) ||
 			$post_id !== (int) $publisher_id
 		) {
 			return $content;
@@ -228,18 +228,18 @@ 
 		);
 
 		// Check that the entity type is "Organization".
-		if ( in_array( 'organization', $terms, true ) ) {
+		if (in_array('organization', $terms, true)) {
 			// Add the featured image description when the type is "Organization".
 
-			$link     = sprintf(
+			$link = sprintf(
 				'<a target="_blank" href="%s">%s</a>',
-				esc_attr__( 'https://developers.google.com/search/docs/data-types/article#logo-guidelines', 'wordlift' ),
-				esc_html__( 'AMP logo guidelines', 'wordlift' )
+				esc_attr__('https://developers.google.com/search/docs/data-types/article#logo-guidelines', 'wordlift'),
+				esc_html__('AMP logo guidelines', 'wordlift')
 			);
 			$content .= sprintf(
 				'<p>'
 								/* translators: %s: AMP logo guidelines. */
-								 . esc_html_x( 'According to the %s, the logo should fit in a 60x600px rectangle, and either be exactly 60px high (preferred), or exactly 600px wide. For example, 450x45px would not be acceptable, even though it fits in the 600x60px rectangle. To comply with the guidelines, WordLift will automatically resize the Featured Image for structured data formats.', 'After "According to the" goes the link to the "AMP logo guidelines".', 'wordlift' )
+								 . esc_html_x('According to the %s, the logo should fit in a 60x600px rectangle, and either be exactly 60px high (preferred), or exactly 600px wide. For example, 450x45px would not be acceptable, even though it fits in the 600x60px rectangle. To comply with the guidelines, WordLift will automatically resize the Featured Image for structured data formats.', 'After "According to the" goes the link to the "AMP logo guidelines".', 'wordlift')
 								 . '</p>',
 				$link
 			);