insideout10 / wordlift-plugin

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

Indentation +122 added lines, -122 removed lines

@@ -7,127 +7,127 @@
  */
 class Wordlift_Thumbnail_Service {
 
-	/**
-	 * The Thumbnail id meta key.
-	 *
-	 * @since 3.1.5
-	 */
-	const THUMBNAIL_ID_META_KEY = '_thumbnail_id';
-
-	/**
-	 * The predicate used in RDF to describe the thumbnail.
-	 *
-	 * @since 3.1.5
-	 */
-	const THUMBNAIL_RDF_PREDICATE = 'http://schema.org/image';
-
-	/**
-	 * The Log service.
-	 *
-	 * @since  3.1.5
-	 * @access private
-	 * @var \Wordlift_Log_Service The Log service.
-	 */
-	private $log_service;
-
-	/**
-	 * Create an instance of the Thumbnail service.
-	 *
-	 * @since 3.1.5
-	 */
-	public function __construct() {
-
-		$this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Thumbnail_Service' );
-
-	}
-
-	/**
-	 * Receive post meta events immediately after a post metadata has been deleted.
-	 *
-	 * @since 3.1.5
-	 *
-	 * @param array  $meta_ids    An array of deleted metadata entry IDs.
-	 * @param int    $object_id   Object ID.
-	 * @param string $meta_key    Meta key.
-	 * @param mixed  $_meta_value Meta value.
-	 */
-	public function deleted_post_meta( $meta_ids, $object_id, $meta_key, $_meta_value ) {
-
-		// Return if it's not the Thumbnail id meta key.
-		if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
-			return;
-		}
-
-		// Do not perform any action is the post is not published.
-		if ( 'publish' !== get_post_status( $object_id ) ) {
-			return;
-		}
-
-		// Get the post uri and return if it's null.
-		if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
-			return;
-		}
-
-		// Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
-		$query = sprintf( 'DELETE { <%s> <%s> ?o . } WHERE  { <%1$s> <%2$s> ?o . };', $uri, self::THUMBNAIL_RDF_PREDICATE );
-		if ( false === rl_execute_sparql_update_query( $query, false ) ) {
-
-			$this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: " . ( is_array( $meta_ids ) ? implode( ',', $meta_ids ) : $meta_ids ) . " ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . ( is_array( $_meta_value ) ? implode( ',', $_meta_value ) : $_meta_value ) . " ][ query :: $query ]" );
-
-		}
-
-	}
-
-	/**
-	 * Receive post meta events immediately after a post metadata has been added.
-	 *
-	 * @since 3.1.5
-	 *
-	 * @param int    $mid         The meta ID after successful update.
-	 * @param int    $object_id   Object ID.
-	 * @param string $meta_key    Meta key.
-	 * @param mixed  $_meta_value Meta value.
-	 */
-	public function added_or_updated_post_meta( $mid, $object_id, $meta_key, $_meta_value ) {
-
-		// $this->log_service->trace( "A post meta has been updated [ meta id :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ]" );
-
-		// Return if it's not the Thumbnail id meta key.
-		if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
-			return;
-		}
-
-		// The meta value must be the numeric id of the attachment. If it isn't, return.
-		if ( ! is_numeric( $_meta_value ) ) {
-			return;
-		}
-
-		// Do not perform any action is the post is not published.
-		if ( 'publish' !== get_post_status( $object_id ) ) {
-			return;
-		}
-
-		// Get the post uri and return if it's null.
-		if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
-			return;
-		}
-
-		// Don't pollute the cache.
-		wp_cache_delete( $object_id, 'post_meta' );
-
-		// Get the attachment url and return if not found.
-		if ( false === ( $attachment_url = wp_get_attachment_url( $_meta_value ) ) ) {
-			return;
-		};
-
-		// Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
-		$query = sprintf( 'DELETE { <%1$s> <%2$s> ?o . } WHERE  { <%1$s> <%2$s> ?o . }; INSERT DATA { <%1$s> <%2$s> <%3$s> . };', $uri, self::THUMBNAIL_RDF_PREDICATE, $attachment_url );
-		if ( false === rl_execute_sparql_update_query( $query, false ) ) {
-
-			$this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ][ query :: $query ]" );
-
-		}
-
-	}
+    /**
+     * The Thumbnail id meta key.
+     *
+     * @since 3.1.5
+     */
+    const THUMBNAIL_ID_META_KEY = '_thumbnail_id';
+
+    /**
+     * The predicate used in RDF to describe the thumbnail.
+     *
+     * @since 3.1.5
+     */
+    const THUMBNAIL_RDF_PREDICATE = 'http://schema.org/image';
+
+    /**
+     * The Log service.
+     *
+     * @since  3.1.5
+     * @access private
+     * @var \Wordlift_Log_Service The Log service.
+     */
+    private $log_service;
+
+    /**
+     * Create an instance of the Thumbnail service.
+     *
+     * @since 3.1.5
+     */
+    public function __construct() {
+
+        $this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Thumbnail_Service' );
+
+    }
+
+    /**
+     * Receive post meta events immediately after a post metadata has been deleted.
+     *
+     * @since 3.1.5
+     *
+     * @param array  $meta_ids    An array of deleted metadata entry IDs.
+     * @param int    $object_id   Object ID.
+     * @param string $meta_key    Meta key.
+     * @param mixed  $_meta_value Meta value.
+     */
+    public function deleted_post_meta( $meta_ids, $object_id, $meta_key, $_meta_value ) {
+
+        // Return if it's not the Thumbnail id meta key.
+        if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
+            return;
+        }
+
+        // Do not perform any action is the post is not published.
+        if ( 'publish' !== get_post_status( $object_id ) ) {
+            return;
+        }
+
+        // Get the post uri and return if it's null.
+        if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
+            return;
+        }
+
+        // Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
+        $query = sprintf( 'DELETE { <%s> <%s> ?o . } WHERE  { <%1$s> <%2$s> ?o . };', $uri, self::THUMBNAIL_RDF_PREDICATE );
+        if ( false === rl_execute_sparql_update_query( $query, false ) ) {
+
+            $this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: " . ( is_array( $meta_ids ) ? implode( ',', $meta_ids ) : $meta_ids ) . " ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . ( is_array( $_meta_value ) ? implode( ',', $_meta_value ) : $_meta_value ) . " ][ query :: $query ]" );
+
+        }
+
+    }
+
+    /**
+     * Receive post meta events immediately after a post metadata has been added.
+     *
+     * @since 3.1.5
+     *
+     * @param int    $mid         The meta ID after successful update.
+     * @param int    $object_id   Object ID.
+     * @param string $meta_key    Meta key.
+     * @param mixed  $_meta_value Meta value.
+     */
+    public function added_or_updated_post_meta( $mid, $object_id, $meta_key, $_meta_value ) {
+
+        // $this->log_service->trace( "A post meta has been updated [ meta id :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ]" );
+
+        // Return if it's not the Thumbnail id meta key.
+        if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
+            return;
+        }
+
+        // The meta value must be the numeric id of the attachment. If it isn't, return.
+        if ( ! is_numeric( $_meta_value ) ) {
+            return;
+        }
+
+        // Do not perform any action is the post is not published.
+        if ( 'publish' !== get_post_status( $object_id ) ) {
+            return;
+        }
+
+        // Get the post uri and return if it's null.
+        if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
+            return;
+        }
+
+        // Don't pollute the cache.
+        wp_cache_delete( $object_id, 'post_meta' );
+
+        // Get the attachment url and return if not found.
+        if ( false === ( $attachment_url = wp_get_attachment_url( $_meta_value ) ) ) {
+            return;
+        };
+
+        // Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
+        $query = sprintf( 'DELETE { <%1$s> <%2$s> ?o . } WHERE  { <%1$s> <%2$s> ?o . }; INSERT DATA { <%1$s> <%2$s> <%3$s> . };', $uri, self::THUMBNAIL_RDF_PREDICATE, $attachment_url );
+        if ( false === rl_execute_sparql_update_query( $query, false ) ) {
+
+            $this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ][ query :: $query ]" );
+
+        }
+
+    }
 
 }

Spacing +18 added lines, -18 removed lines

@@ -37,7 +37,7 @@ 
 	 */
 	public function __construct() {
 
-		$this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Thumbnail_Service' );
+		$this->log_service = Wordlift_Log_Service::get_logger('Wordlift_Thumbnail_Service');
 
 	}
 
@@ -51,28 +51,28 @@ 
 	 * @param string $meta_key    Meta key.
 	 * @param mixed  $_meta_value Meta value.
 	 */
-	public function deleted_post_meta( $meta_ids, $object_id, $meta_key, $_meta_value ) {
+	public function deleted_post_meta($meta_ids, $object_id, $meta_key, $_meta_value) {
 
 		// Return if it's not the Thumbnail id meta key.
-		if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
+		if (self::THUMBNAIL_ID_META_KEY !== $meta_key) {
 			return;
 		}
 
 		// Do not perform any action is the post is not published.
-		if ( 'publish' !== get_post_status( $object_id ) ) {
+		if ('publish' !== get_post_status($object_id)) {
 			return;
 		}
 
 		// Get the post uri and return if it's null.
-		if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
+		if (null === ($uri = wl_get_entity_uri($object_id))) {
 			return;
 		}
 
 		// Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
-		$query = sprintf( 'DELETE { <%s> <%s> ?o . } WHERE  { <%1$s> <%2$s> ?o . };', $uri, self::THUMBNAIL_RDF_PREDICATE );
-		if ( false === rl_execute_sparql_update_query( $query, false ) ) {
+		$query = sprintf('DELETE { <%s> <%s> ?o . } WHERE  { <%1$s> <%2$s> ?o . };', $uri, self::THUMBNAIL_RDF_PREDICATE);
+		if (false === rl_execute_sparql_update_query($query, false)) {
 
-			$this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: " . ( is_array( $meta_ids ) ? implode( ',', $meta_ids ) : $meta_ids ) . " ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . ( is_array( $_meta_value ) ? implode( ',', $_meta_value ) : $_meta_value ) . " ][ query :: $query ]" );
+			$this->log_service->error("An error occurred removing the post thumbnail [ meta ids :: ".(is_array($meta_ids) ? implode(',', $meta_ids) : $meta_ids)." ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: ".(is_array($_meta_value) ? implode(',', $_meta_value) : $_meta_value)." ][ query :: $query ]");
 
 		}
 
@@ -88,43 +88,43 @@ 
 	 * @param string $meta_key    Meta key.
 	 * @param mixed  $_meta_value Meta value.
 	 */
-	public function added_or_updated_post_meta( $mid, $object_id, $meta_key, $_meta_value ) {
+	public function added_or_updated_post_meta($mid, $object_id, $meta_key, $_meta_value) {
 
 		// $this->log_service->trace( "A post meta has been updated [ meta id :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ]" );
 
 		// Return if it's not the Thumbnail id meta key.
-		if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
+		if (self::THUMBNAIL_ID_META_KEY !== $meta_key) {
 			return;
 		}
 
 		// The meta value must be the numeric id of the attachment. If it isn't, return.
-		if ( ! is_numeric( $_meta_value ) ) {
+		if ( ! is_numeric($_meta_value)) {
 			return;
 		}
 
 		// Do not perform any action is the post is not published.
-		if ( 'publish' !== get_post_status( $object_id ) ) {
+		if ('publish' !== get_post_status($object_id)) {
 			return;
 		}
 
 		// Get the post uri and return if it's null.
-		if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
+		if (null === ($uri = wl_get_entity_uri($object_id))) {
 			return;
 		}
 
 		// Don't pollute the cache.
-		wp_cache_delete( $object_id, 'post_meta' );
+		wp_cache_delete($object_id, 'post_meta');
 
 		// Get the attachment url and return if not found.
-		if ( false === ( $attachment_url = wp_get_attachment_url( $_meta_value ) ) ) {
+		if (false === ($attachment_url = wp_get_attachment_url($_meta_value))) {
 			return;
 		};
 
 		// Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
-		$query = sprintf( 'DELETE { <%1$s> <%2$s> ?o . } WHERE  { <%1$s> <%2$s> ?o . }; INSERT DATA { <%1$s> <%2$s> <%3$s> . };', $uri, self::THUMBNAIL_RDF_PREDICATE, $attachment_url );
-		if ( false === rl_execute_sparql_update_query( $query, false ) ) {
+		$query = sprintf('DELETE { <%1$s> <%2$s> ?o . } WHERE  { <%1$s> <%2$s> ?o . }; INSERT DATA { <%1$s> <%2$s> <%3$s> . };', $uri, self::THUMBNAIL_RDF_PREDICATE, $attachment_url);
+		if (false === rl_execute_sparql_update_query($query, false)) {
 
-			$this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ][ query :: $query ]" );
+			$this->log_service->error("An error occurred removing the post thumbnail [ meta ids :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: ".var_export($_meta_value, true)." ][ query :: $query ]");
 
 		}
 

src/modules/core/wordlift_core_functions.php

Indentation +4 added lines, -4 removed lines

@@ -7,9 +7,9 @@
  * @param mixed $response The response to send to the client as JSON.
  */
 function wl_core_send_json( $response ) {
-	if ( ob_get_contents() ) {
-		ob_clean();
-	}
+    if ( ob_get_contents() ) {
+        ob_clean();
+    }
 
-	wp_send_json( $response );
+    wp_send_json( $response );
 }

Spacing +3 added lines, -3 removed lines

@@ -6,10 +6,10 @@
  *
  * @param mixed $response The response to send to the client as JSON.
  */
-function wl_core_send_json( $response ) {
-	if ( ob_get_contents() ) {
+function wl_core_send_json($response) {
+	if (ob_get_contents()) {
 		ob_clean();
 	}
 
-	wp_send_json( $response );
+	wp_send_json($response);
 }

src/modules/core/wordlift_core_schema_api.php

Indentation +78 added lines, -78 removed lines

@@ -17,34 +17,34 @@ 
  */
 function wl_schema_get_value( $post_id, $property_name ) {
 
-	// Property name must be defined.
-	if ( ! isset( $property_name ) || is_null( $property_name ) ) {
-		return null;
-	}
-
-	// store eventual schema name in  different variable
-	$property_schema_name = wl_build_full_schema_uri_from_schema_slug( $property_name );
-
-	// Establish entity id.
-	if ( is_null( $post_id ) || ! is_numeric( $post_id ) ) {
-		$post_id = get_the_ID();
-		if ( is_null( $post_id ) || ! is_numeric( $post_id ) ) {
-			return null;
-		}
-	}
-
-	// Get custom fields.
-	$term_mapping = wl_entity_taxonomy_get_custom_fields( $post_id );
-	// Search for the required meta value (by constant name or schema name)
-	foreach ( $term_mapping as $wl_constant => $property_info ) {
-		$found_constant  = ( $wl_constant == $property_name );
-		$found_predicate = ( isset( $property_info['predicate'] ) && $property_info['predicate'] == $property_schema_name );
-		if ( $found_constant || $found_predicate ) {
-			return get_post_meta( $post_id, $wl_constant );
-		}
-	}
-
-	return null;
+    // Property name must be defined.
+    if ( ! isset( $property_name ) || is_null( $property_name ) ) {
+        return null;
+    }
+
+    // store eventual schema name in  different variable
+    $property_schema_name = wl_build_full_schema_uri_from_schema_slug( $property_name );
+
+    // Establish entity id.
+    if ( is_null( $post_id ) || ! is_numeric( $post_id ) ) {
+        $post_id = get_the_ID();
+        if ( is_null( $post_id ) || ! is_numeric( $post_id ) ) {
+            return null;
+        }
+    }
+
+    // Get custom fields.
+    $term_mapping = wl_entity_taxonomy_get_custom_fields( $post_id );
+    // Search for the required meta value (by constant name or schema name)
+    foreach ( $term_mapping as $wl_constant => $property_info ) {
+        $found_constant  = ( $wl_constant == $property_name );
+        $found_predicate = ( isset( $property_info['predicate'] ) && $property_info['predicate'] == $property_schema_name );
+        if ( $found_constant || $found_predicate ) {
+            return get_post_meta( $post_id, $wl_constant );
+        }
+    }
+
+    return null;
 }
 
 /**
@@ -58,38 +58,38 @@ 
  */
 function wl_schema_set_value( $post_id, $property_name, $property_value ) {
 
-	// Some checks on the parameters
-	if ( ! is_numeric( $post_id ) || is_null( $property_name ) || empty( $property_value ) || is_null( $property_value ) ) {
-		return false;
-	}
+    // Some checks on the parameters
+    if ( ! is_numeric( $post_id ) || is_null( $property_name ) || empty( $property_value ) || is_null( $property_value ) ) {
+        return false;
+    }
 
-	// Build full schema uri if necessary
-	$property_name = wl_build_full_schema_uri_from_schema_slug( $property_name );
+    // Build full schema uri if necessary
+    $property_name = wl_build_full_schema_uri_from_schema_slug( $property_name );
 
-	// Get accepted properties
-	$accepted_fields = wl_entity_taxonomy_get_custom_fields( $post_id );
+    // Get accepted properties
+    $accepted_fields = wl_entity_taxonomy_get_custom_fields( $post_id );
 
-	// Find the name of the custom-field managing the schema property
-	foreach ( $accepted_fields as $wl_constant => $field ) {
-		if ( $field['predicate'] == $property_name ) {
+    // Find the name of the custom-field managing the schema property
+    foreach ( $accepted_fields as $wl_constant => $field ) {
+        if ( $field['predicate'] == $property_name ) {
 
-			// Deal with single values
-			if ( ! is_array( $property_value ) ) {
-				$property_value = array( $property_value );
-			}
+            // Deal with single values
+            if ( ! is_array( $property_value ) ) {
+                $property_value = array( $property_value );
+            }
 
-			// Delete present meta
-			delete_post_meta( $post_id, $wl_constant );
+            // Delete present meta
+            delete_post_meta( $post_id, $wl_constant );
 
-			foreach ( $property_value as $value ) {
-				add_post_meta( $post_id, $wl_constant, $value );
-			}
+            foreach ( $property_value as $value ) {
+                add_post_meta( $post_id, $wl_constant, $value );
+            }
 
-			return true;
-		}
-	}
+            return true;
+        }
+    }
 
-	return false;
+    return false;
 }
 
 /**
@@ -102,18 +102,18 @@ 
  */
 function wl_schema_get_types( $post_id ) {
 
-	// Some checks on the parameters
-	if ( ! is_numeric( $post_id ) ) {
-		return null;
-	}
+    // Some checks on the parameters
+    if ( ! is_numeric( $post_id ) ) {
+        return null;
+    }
 
-	$type = Wordlift_Entity_Type_Service::get_instance()->get( $post_id );
+    $type = Wordlift_Entity_Type_Service::get_instance()->get( $post_id );
 
-	if ( isset( $type['uri'] ) ) {
-		return array( $type['uri'] );
-	}
+    if ( isset( $type['uri'] ) ) {
+        return array( $type['uri'] );
+    }
 
-	return null;
+    return null;
 }
 
 /**
@@ -126,21 +126,21 @@ 
  */
 function wl_schema_set_types( $post_id, $type_names ) {
 
-	// Some checks on the parameters
-	if ( ! is_numeric( $post_id ) || empty( $type_names ) || is_null( $type_names ) ) {
-		return null;
-	}
+    // Some checks on the parameters
+    if ( ! is_numeric( $post_id ) || empty( $type_names ) || is_null( $type_names ) ) {
+        return null;
+    }
 
-	// TODO: support more than one type.
-	if ( is_array( $type_names ) ) {
-		$type_names = $type_names[0];
-	}
+    // TODO: support more than one type.
+    if ( is_array( $type_names ) ) {
+        $type_names = $type_names[0];
+    }
 
-	// Get the schema URI (e.g. http://schema.org/Thing)
-	$type_names = wl_build_full_schema_uri_from_schema_slug( $type_names );
+    // Get the schema URI (e.g. http://schema.org/Thing)
+    $type_names = wl_build_full_schema_uri_from_schema_slug( $type_names );
 
-	// Actually sets the taxonomy type
-	wl_set_entity_main_type( $post_id, $type_names );
+    // Actually sets the taxonomy type
+    wl_set_entity_main_type( $post_id, $type_names );
 
 }
 
@@ -153,11 +153,11 @@ 
  */
 function wl_build_full_schema_uri_from_schema_slug( $schema_name ) {
 
-	$schema_root_address = 'http://schema.org/';
+    $schema_root_address = 'http://schema.org/';
 
-	if ( strpos( $schema_name, $schema_root_address ) === false ) {   // === necessary
-		$schema_name = $schema_root_address . $schema_name;
-	}
+    if ( strpos( $schema_name, $schema_root_address ) === false ) {   // === necessary
+        $schema_name = $schema_root_address . $schema_name;
+    }
 
-	return $schema_name;
+    return $schema_name;
 }

Spacing +35 added lines, -35 removed lines

@@ -15,32 +15,32 @@ 
  *
  * @return array|null An array of values or NULL in case of no values (or error).
  */
-function wl_schema_get_value( $post_id, $property_name ) {
+function wl_schema_get_value($post_id, $property_name) {
 
 	// Property name must be defined.
-	if ( ! isset( $property_name ) || is_null( $property_name ) ) {
+	if ( ! isset($property_name) || is_null($property_name)) {
 		return null;
 	}
 
 	// store eventual schema name in  different variable
-	$property_schema_name = wl_build_full_schema_uri_from_schema_slug( $property_name );
+	$property_schema_name = wl_build_full_schema_uri_from_schema_slug($property_name);
 
 	// Establish entity id.
-	if ( is_null( $post_id ) || ! is_numeric( $post_id ) ) {
+	if (is_null($post_id) || ! is_numeric($post_id)) {
 		$post_id = get_the_ID();
-		if ( is_null( $post_id ) || ! is_numeric( $post_id ) ) {
+		if (is_null($post_id) || ! is_numeric($post_id)) {
 			return null;
 		}
 	}
 
 	// Get custom fields.
-	$term_mapping = wl_entity_taxonomy_get_custom_fields( $post_id );
+	$term_mapping = wl_entity_taxonomy_get_custom_fields($post_id);
 	// Search for the required meta value (by constant name or schema name)
-	foreach ( $term_mapping as $wl_constant => $property_info ) {
-		$found_constant  = ( $wl_constant == $property_name );
-		$found_predicate = ( isset( $property_info['predicate'] ) && $property_info['predicate'] == $property_schema_name );
-		if ( $found_constant || $found_predicate ) {
-			return get_post_meta( $post_id, $wl_constant );
+	foreach ($term_mapping as $wl_constant => $property_info) {
+		$found_constant  = ($wl_constant == $property_name);
+		$found_predicate = (isset($property_info['predicate']) && $property_info['predicate'] == $property_schema_name);
+		if ($found_constant || $found_predicate) {
+			return get_post_meta($post_id, $wl_constant);
 		}
 	}
 
@@ -56,33 +56,33 @@ 
  *
  * @return boolean The method returns true if everything went ok, an error string otherwise.
  */
-function wl_schema_set_value( $post_id, $property_name, $property_value ) {
+function wl_schema_set_value($post_id, $property_name, $property_value) {
 
 	// Some checks on the parameters
-	if ( ! is_numeric( $post_id ) || is_null( $property_name ) || empty( $property_value ) || is_null( $property_value ) ) {
+	if ( ! is_numeric($post_id) || is_null($property_name) || empty($property_value) || is_null($property_value)) {
 		return false;
 	}
 
 	// Build full schema uri if necessary
-	$property_name = wl_build_full_schema_uri_from_schema_slug( $property_name );
+	$property_name = wl_build_full_schema_uri_from_schema_slug($property_name);
 
 	// Get accepted properties
-	$accepted_fields = wl_entity_taxonomy_get_custom_fields( $post_id );
+	$accepted_fields = wl_entity_taxonomy_get_custom_fields($post_id);
 
 	// Find the name of the custom-field managing the schema property
-	foreach ( $accepted_fields as $wl_constant => $field ) {
-		if ( $field['predicate'] == $property_name ) {
+	foreach ($accepted_fields as $wl_constant => $field) {
+		if ($field['predicate'] == $property_name) {
 
 			// Deal with single values
-			if ( ! is_array( $property_value ) ) {
-				$property_value = array( $property_value );
+			if ( ! is_array($property_value)) {
+				$property_value = array($property_value);
 			}
 
 			// Delete present meta
-			delete_post_meta( $post_id, $wl_constant );
+			delete_post_meta($post_id, $wl_constant);
 
-			foreach ( $property_value as $value ) {
-				add_post_meta( $post_id, $wl_constant, $value );
+			foreach ($property_value as $value) {
+				add_post_meta($post_id, $wl_constant, $value);
 			}
 
 			return true;
@@ -100,17 +100,17 @@ 
  * @return array Array of type(s) (e.g. Type, for the http://schema.org/Type)
  *               or NULL in case of no values (or error).
  */
-function wl_schema_get_types( $post_id ) {
+function wl_schema_get_types($post_id) {
 
 	// Some checks on the parameters
-	if ( ! is_numeric( $post_id ) ) {
+	if ( ! is_numeric($post_id)) {
 		return null;
 	}
 
-	$type = Wordlift_Entity_Type_Service::get_instance()->get( $post_id );
+	$type = Wordlift_Entity_Type_Service::get_instance()->get($post_id);
 
-	if ( isset( $type['uri'] ) ) {
-		return array( $type['uri'] );
+	if (isset($type['uri'])) {
+		return array($type['uri']);
 	}
 
 	return null;
@@ -124,23 +124,23 @@ 
  *
  * @return boolean True if everything went ok, an error string otherwise.
  */
-function wl_schema_set_types( $post_id, $type_names ) {
+function wl_schema_set_types($post_id, $type_names) {
 
 	// Some checks on the parameters
-	if ( ! is_numeric( $post_id ) || empty( $type_names ) || is_null( $type_names ) ) {
+	if ( ! is_numeric($post_id) || empty($type_names) || is_null($type_names)) {
 		return null;
 	}
 
 	// TODO: support more than one type.
-	if ( is_array( $type_names ) ) {
+	if (is_array($type_names)) {
 		$type_names = $type_names[0];
 	}
 
 	// Get the schema URI (e.g. http://schema.org/Thing)
-	$type_names = wl_build_full_schema_uri_from_schema_slug( $type_names );
+	$type_names = wl_build_full_schema_uri_from_schema_slug($type_names);
 
 	// Actually sets the taxonomy type
-	wl_set_entity_main_type( $post_id, $type_names );
+	wl_set_entity_main_type($post_id, $type_names);
 
 }
 
@@ -151,12 +151,12 @@ 
  *
  * @return string The full schema uri (es. 'latitude' returns 'http://schema.org/latitude')
  */
-function wl_build_full_schema_uri_from_schema_slug( $schema_name ) {
+function wl_build_full_schema_uri_from_schema_slug($schema_name) {
 
 	$schema_root_address = 'http://schema.org/';
 
-	if ( strpos( $schema_name, $schema_root_address ) === false ) {   // === necessary
-		$schema_name = $schema_root_address . $schema_name;
+	if (strpos($schema_name, $schema_root_address) === false) {   // === necessary
+		$schema_name = $schema_root_address.$schema_name;
 	}
 
 	return $schema_name;

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

Indentation +369 added lines, -369 removed lines

@@ -14,374 +14,374 @@
  */
 class Wordlift_Rating_Service {
 
-	/**
-	 * Entity rating max.
-	 *
-	 * @since 3.3.0
-	 */
-	const RATING_MAX = 7;
-
-	/**
-	 * Entity rating score meta key.
-	 *
-	 * @since 3.3.0
-	 */
-	const RATING_RAW_SCORE_META_KEY = '_wl_entity_rating_raw_score';
-
-	/**
-	 * Entity rating warnings meta key.
-	 *
-	 * @since 3.3.0
-	 */
-	const RATING_WARNINGS_META_KEY = '_wl_entity_rating_warnings';
-
-	/**
-	 * Entity warning has related post identifier.
-	 *
-	 * @since 3.3.0
-	 */
-	const RATING_WARNING_HAS_RELATED_POSTS = 'There are no related posts for the current entity.';
-
-	/**
-	 * Entity warning has content post identifier.
-	 *
-	 * @since 3.3.0
-	 */
-	const RATING_WARNING_HAS_CONTENT_POST = 'This entity has not description.';
-
-	/**
-	 * Entity warning has related entities identifier.
-	 *
-	 * @since 3.3.0
-	 */
-	const RATING_WARNING_HAS_RELATED_ENTITIES = 'There are no related entities for the current entity.';
-
-	/**
-	 * Entity warning is published identifier.
-	 *
-	 * @since 3.3.0
-	 */
-	const RATING_WARNING_IS_PUBLISHED = 'This entity is not published. It will not appear within analysis results.';
-
-	/**
-	 * Entity warning has thumbnail identifier.
-	 *
-	 * @since 3.3.0
-	 */
-	const RATING_WARNING_HAS_THUMBNAIL = 'This entity has no featured image yet.';
-
-	/**
-	 * Entity warning has same as identifier.
-	 *
-	 * @since 3.3.0
-	 */
-	const RATING_WARNING_HAS_SAME_AS = 'There are no sameAs configured for this entity.';
-
-	/**
-	 * Entity warning has completed metadata identifier.
-	 *
-	 * @since 3.3.0
-	 */
-	const RATING_WARNING_HAS_COMPLETED_METADATA = 'Schema.org metadata for this entity are not completed.';
-
-	/**
-	 * A {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access private
-	 * @var Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
-
-	/**
-	 *  A {@link Wordlift_Entity_Type_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access private
-	 * @var Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
-	 */
-	private $entity_type_service;
-
-	/**
-	 * The Notice service.
-	 *
-	 * @since  3.3.0
-	 * @access private
-	 * @var \Wordlift_Notice_Service $notice_service The Notice service.
-	 */
-	private $notice_service;
-
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access private
-	 * @var Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * Create a {@link Wordlift_Rating_Service} instance.
-	 *
-	 * @since 3.10.0
-	 *
-	 * @param \Wordlift_Entity_Service      $entity_service      A {@link Wordlift_Entity_Service} instance.
-	 * @param \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
-	 * @param \Wordlift_Notice_Service      $notice_service      A {@link Wordlift_Notice_Service} instance.
-	 */
-	public function __construct( $entity_service, $entity_type_service, $notice_service ) {
-
-		$this->entity_service      = $entity_service;
-		$this->entity_type_service = $entity_type_service;
-		$this->notice_service      = $notice_service;
-
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Rating_Service' );
-
-	}
-
-	/**
-	 * Set rating for a given entity.
-	 *
-	 * @since 3.3.0
-	 *
-	 * @param int $post_id The entity post id.
-	 *
-	 * @return array An array representing the rating obj.
-	 */
-	public function set_rating_for( $post_id ) {
-
-		if ( ! $this->entity_service->is_entity( $post_id ) ) {
-			return;
-		}
-
-		// Calculate rating for the given post.
-		$rating = $this->calculate_rating_for( $post_id );
-
-		// Store the rating on db as post meta. Please notice that RATING_RAW_SCORE_META_KEY
-		// is saved on a different meta to allow score sorting. Both meta are managed as unique.
-		//
-		// See https://codex.wordpress.org/Function_Reference/update_post_meta
-		update_post_meta( $post_id, self::RATING_RAW_SCORE_META_KEY, $rating['raw_score'] );
-		update_post_meta( $post_id, self::RATING_WARNINGS_META_KEY, $rating['warnings'] );
-
-		$this->log->trace( sprintf( "Rating set for [ post_id :: $post_id ] [ rating :: %s ]", $rating['raw_score'] ) );
-
-		// Finally returns the rating
-		return $rating;
-	}
-
-
-	/**
-	 * Get or calculate rating for a given entity
-	 *
-	 * @since 3.3.0
-	 *
-	 * @param int $post_id      The entity post id.
-	 * @param     $force_reload $warnings_needed If true, detailed warnings collection is provided with the rating obj.
-	 *
-	 * @return array An array representing the rating obj.
-	 */
-	public function get_rating_for( $post_id, $force_reload = false ) {
-
-		// If forced reload is required or rating is missing.
-		if ( $force_reload ) {
-			$this->log->trace( "Force rating reload [ post_id :: $post_id ]" );
-
-			return $this->set_rating_for( $post_id );
-		}
-
-		$current_raw_score = get_post_meta( $post_id, self::RATING_RAW_SCORE_META_KEY, true );
-
-		if ( ! is_numeric( $current_raw_score ) ) {
-			$this->log->trace( "Rating missing for [ post_id :: $post_id ] [ current_raw_score :: $current_raw_score ]" );
-
-			return $this->set_rating_for( $post_id );
-		}
-
-		$current_warnings = get_post_meta( $post_id, self::RATING_WARNINGS_META_KEY, true );
-
-		// Finally return score and warnings
-		return array(
-			'raw_score'           => $current_raw_score,
-			'traffic_light_score' => $this->convert_raw_score_to_traffic_light( $current_raw_score ),
-			'percentage_score'    => $this->convert_raw_score_to_percentage( $current_raw_score ),
-			'warnings'            => $current_warnings,
-		);
-
-	}
-
-	/**
-	 * Calculate rating for a given entity.
-	 *
-	 * Rating depends from following criteria:
-	 *
-	 * 1. Is the current entity related to at least 1 post?
-	 * 2. Is the current entity content post not empty?
-	 * 3. Is the current entity related to at least 1 entity?
-	 * 4. Is the entity published?
-	 * 5. There is a a thumbnail associated to the entity?
-	 * 6. Has the entity a sameas defined?
-	 * 7. Are all schema.org required metadata compiled?
-	 *
-	 * Each positive check means +1 in terms of rating score.
-	 *
-	 * @since 3.3.0
-	 *
-	 * @param int $post_id The entity post id.
-	 *
-	 * @return array An array representing the rating obj.
-	 */
-	private function calculate_rating_for( $post_id ) {
-
-		// If it's not an entity, return.
-		if ( ! $this->entity_service->is_entity( $post_id ) ) {
-			return array();
-		}
-		// Retrieve the post object.
-		$post = get_post( $post_id );
-
-		// Rating value.
-		$score = 0;
-
-		// Store warning messages.
-		$warnings = array();
-
-		// Is the current entity related to at least 1 post?
-		( 0 < count( wl_core_get_related_post_ids( $post->ID ) ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'There are no related posts for the current entity.', 'wordlift' ) );
-
-		// Is the post content not empty?
-		( ! empty( $post->post_content ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'This entity has not description.', 'wordlift' ) );
-
-		// Is the current entity related to at least 1 entity?
-		// Was the current entity already disambiguated?
-		( 0 < count( wl_core_get_related_entity_ids( $post->ID ) ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'There are no related entities for the current entity.', 'wordlift' ) );
-
-		// Is the entity published?
-		( 'publish' === get_post_status( $post->ID ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'This entity is not published. It will not appear within analysis results.', 'wordlift' ) );
-
-		// Has a thumbnail?
-		( has_post_thumbnail( $post->ID ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'This entity has no featured image yet.', 'wordlift' ) );
-
-		// Get all post meta keys for the current post
-		global $wpdb;
-
-		$query = $wpdb->prepare(
-			"SELECT DISTINCT (meta_key) FROM $wpdb->postmeta  WHERE post_id = %d", $post->ID
-		);
-
-		// Check intersection between available meta keys and expected ones
-		// arrays to detect missing values.
-		$available_meta_keys = $wpdb->get_col( $query );
-
-		// If each expected key is contained in available keys array ...
-		( in_array( Wordlift_Schema_Service::FIELD_SAME_AS, $available_meta_keys ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'There are no sameAs configured for this entity.', 'wordlift' ) );
-
-		$schema = $this->entity_type_service->get( $post_id );
-
-		$expected_meta_keys = ( null === $schema['custom_fields'] ) ?
-			array() :
-			array_keys( $schema['custom_fields'] );
-
-		$intersection = array_intersect( $expected_meta_keys, $available_meta_keys );
-		// If each expected key is contained in available keys array ...
-		( count( $intersection ) === count( $expected_meta_keys ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'Schema.org metadata for this entity are not completed.', 'wordlift' ) );
-
-		// Finally return score and warnings
-		return array(
-			'raw_score'           => $score,
-			'traffic_light_score' => $this->convert_raw_score_to_traffic_light( $score ),
-			'percentage_score'    => $this->convert_raw_score_to_percentage( $score ),
-			'warnings'            => $warnings,
-		);
-
-	}
-
-	/**
-	 * Get as rating as input and convert in a traffic-light rating
-	 *
-	 * @since 3.3.0
-	 *
-	 * @param int $score The rating score for a given entity.
-	 *
-	 * @return string The input HTML code.
-	 */
-	private function convert_raw_score_to_traffic_light( $score ) {
-		// RATING_MAX : $score = 3 : x
-		// See http://php.net/manual/en/function.round.php
-		$rating = round( ( $score * 3 ) / self::RATING_MAX, 0, PHP_ROUND_HALF_UP );
-
-		// If rating is 0, return 1, otherwise return rating
-		return ( 0 == $rating ) ? 1 : $rating;
-
-	}
-
-	/**
-	 * Get as rating as input and convert in a traffic-light rating
-	 *
-	 * @since 3.3.0
-	 *
-	 * @param int $score The rating score for a given entity.
-	 *
-	 * @return string The input HTML code.
-	 */
-	public function convert_raw_score_to_percentage( $score ) {
-
-		// RATING_MAX : $score = 100 : x
-		return round( ( $score * 100 ) / self::RATING_MAX, 0, PHP_ROUND_HALF_UP );
-	}
-
-
-	/**
-	 * Add admin notices for the current entity depending on the current rating.
-	 *
-	 * @since 3.3.0
-	 */
-	public function in_admin_header() {
-
-		// Return safely if get_current_screen() is not defined (yet)
-		if ( false === function_exists( 'get_current_screen' ) ) {
-			return;
-		}
-
-		$screen = get_current_screen();
-		// If there is any valid screen nothing to do
-		if ( null === $screen ) {
-			return;
-		}
-
-		// If you're not in the entity post edit page, return.
-		if ( Wordlift_Entity_Service::TYPE_NAME !== $screen->id ) {
-			return;
-		}
-		// Retrieve the current global post
-		global $post;
-		// If it's not an entity, return.
-		if ( ! $this->entity_service->is_entity( $post->ID ) ) {
-			return;
-		}
-		// Retrieve an updated rating for the current entity
-		$rating = $this->get_rating_for( $post->ID, true );
-
-		// If there is at least 1 warning
-		if ( isset( $rating['warnings'] ) && 0 < count( $rating['warnings'] ) ) {
-			// TODO - Pass Wordlift_Notice_Service trough the service constructor
-			$this->notice_service->add_suggestion( $rating['warnings'] );
-		}
-
-	}
+    /**
+     * Entity rating max.
+     *
+     * @since 3.3.0
+     */
+    const RATING_MAX = 7;
+
+    /**
+     * Entity rating score meta key.
+     *
+     * @since 3.3.0
+     */
+    const RATING_RAW_SCORE_META_KEY = '_wl_entity_rating_raw_score';
+
+    /**
+     * Entity rating warnings meta key.
+     *
+     * @since 3.3.0
+     */
+    const RATING_WARNINGS_META_KEY = '_wl_entity_rating_warnings';
+
+    /**
+     * Entity warning has related post identifier.
+     *
+     * @since 3.3.0
+     */
+    const RATING_WARNING_HAS_RELATED_POSTS = 'There are no related posts for the current entity.';
+
+    /**
+     * Entity warning has content post identifier.
+     *
+     * @since 3.3.0
+     */
+    const RATING_WARNING_HAS_CONTENT_POST = 'This entity has not description.';
+
+    /**
+     * Entity warning has related entities identifier.
+     *
+     * @since 3.3.0
+     */
+    const RATING_WARNING_HAS_RELATED_ENTITIES = 'There are no related entities for the current entity.';
+
+    /**
+     * Entity warning is published identifier.
+     *
+     * @since 3.3.0
+     */
+    const RATING_WARNING_IS_PUBLISHED = 'This entity is not published. It will not appear within analysis results.';
+
+    /**
+     * Entity warning has thumbnail identifier.
+     *
+     * @since 3.3.0
+     */
+    const RATING_WARNING_HAS_THUMBNAIL = 'This entity has no featured image yet.';
+
+    /**
+     * Entity warning has same as identifier.
+     *
+     * @since 3.3.0
+     */
+    const RATING_WARNING_HAS_SAME_AS = 'There are no sameAs configured for this entity.';
+
+    /**
+     * Entity warning has completed metadata identifier.
+     *
+     * @since 3.3.0
+     */
+    const RATING_WARNING_HAS_COMPLETED_METADATA = 'Schema.org metadata for this entity are not completed.';
+
+    /**
+     * A {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.10.0
+     * @access private
+     * @var Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
+
+    /**
+     *  A {@link Wordlift_Entity_Type_Service} instance.
+     *
+     * @since  3.10.0
+     * @access private
+     * @var Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
+     */
+    private $entity_type_service;
+
+    /**
+     * The Notice service.
+     *
+     * @since  3.3.0
+     * @access private
+     * @var \Wordlift_Notice_Service $notice_service The Notice service.
+     */
+    private $notice_service;
+
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.10.0
+     * @access private
+     * @var Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * Create a {@link Wordlift_Rating_Service} instance.
+     *
+     * @since 3.10.0
+     *
+     * @param \Wordlift_Entity_Service      $entity_service      A {@link Wordlift_Entity_Service} instance.
+     * @param \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
+     * @param \Wordlift_Notice_Service      $notice_service      A {@link Wordlift_Notice_Service} instance.
+     */
+    public function __construct( $entity_service, $entity_type_service, $notice_service ) {
+
+        $this->entity_service      = $entity_service;
+        $this->entity_type_service = $entity_type_service;
+        $this->notice_service      = $notice_service;
+
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Rating_Service' );
+
+    }
+
+    /**
+     * Set rating for a given entity.
+     *
+     * @since 3.3.0
+     *
+     * @param int $post_id The entity post id.
+     *
+     * @return array An array representing the rating obj.
+     */
+    public function set_rating_for( $post_id ) {
+
+        if ( ! $this->entity_service->is_entity( $post_id ) ) {
+            return;
+        }
+
+        // Calculate rating for the given post.
+        $rating = $this->calculate_rating_for( $post_id );
+
+        // Store the rating on db as post meta. Please notice that RATING_RAW_SCORE_META_KEY
+        // is saved on a different meta to allow score sorting. Both meta are managed as unique.
+        //
+        // See https://codex.wordpress.org/Function_Reference/update_post_meta
+        update_post_meta( $post_id, self::RATING_RAW_SCORE_META_KEY, $rating['raw_score'] );
+        update_post_meta( $post_id, self::RATING_WARNINGS_META_KEY, $rating['warnings'] );
+
+        $this->log->trace( sprintf( "Rating set for [ post_id :: $post_id ] [ rating :: %s ]", $rating['raw_score'] ) );
+
+        // Finally returns the rating
+        return $rating;
+    }
+
+
+    /**
+     * Get or calculate rating for a given entity
+     *
+     * @since 3.3.0
+     *
+     * @param int $post_id      The entity post id.
+     * @param     $force_reload $warnings_needed If true, detailed warnings collection is provided with the rating obj.
+     *
+     * @return array An array representing the rating obj.
+     */
+    public function get_rating_for( $post_id, $force_reload = false ) {
+
+        // If forced reload is required or rating is missing.
+        if ( $force_reload ) {
+            $this->log->trace( "Force rating reload [ post_id :: $post_id ]" );
+
+            return $this->set_rating_for( $post_id );
+        }
+
+        $current_raw_score = get_post_meta( $post_id, self::RATING_RAW_SCORE_META_KEY, true );
+
+        if ( ! is_numeric( $current_raw_score ) ) {
+            $this->log->trace( "Rating missing for [ post_id :: $post_id ] [ current_raw_score :: $current_raw_score ]" );
+
+            return $this->set_rating_for( $post_id );
+        }
+
+        $current_warnings = get_post_meta( $post_id, self::RATING_WARNINGS_META_KEY, true );
+
+        // Finally return score and warnings
+        return array(
+            'raw_score'           => $current_raw_score,
+            'traffic_light_score' => $this->convert_raw_score_to_traffic_light( $current_raw_score ),
+            'percentage_score'    => $this->convert_raw_score_to_percentage( $current_raw_score ),
+            'warnings'            => $current_warnings,
+        );
+
+    }
+
+    /**
+     * Calculate rating for a given entity.
+     *
+     * Rating depends from following criteria:
+     *
+     * 1. Is the current entity related to at least 1 post?
+     * 2. Is the current entity content post not empty?
+     * 3. Is the current entity related to at least 1 entity?
+     * 4. Is the entity published?
+     * 5. There is a a thumbnail associated to the entity?
+     * 6. Has the entity a sameas defined?
+     * 7. Are all schema.org required metadata compiled?
+     *
+     * Each positive check means +1 in terms of rating score.
+     *
+     * @since 3.3.0
+     *
+     * @param int $post_id The entity post id.
+     *
+     * @return array An array representing the rating obj.
+     */
+    private function calculate_rating_for( $post_id ) {
+
+        // If it's not an entity, return.
+        if ( ! $this->entity_service->is_entity( $post_id ) ) {
+            return array();
+        }
+        // Retrieve the post object.
+        $post = get_post( $post_id );
+
+        // Rating value.
+        $score = 0;
+
+        // Store warning messages.
+        $warnings = array();
+
+        // Is the current entity related to at least 1 post?
+        ( 0 < count( wl_core_get_related_post_ids( $post->ID ) ) ) ?
+            $score ++ :
+            array_push( $warnings, __( 'There are no related posts for the current entity.', 'wordlift' ) );
+
+        // Is the post content not empty?
+        ( ! empty( $post->post_content ) ) ?
+            $score ++ :
+            array_push( $warnings, __( 'This entity has not description.', 'wordlift' ) );
+
+        // Is the current entity related to at least 1 entity?
+        // Was the current entity already disambiguated?
+        ( 0 < count( wl_core_get_related_entity_ids( $post->ID ) ) ) ?
+            $score ++ :
+            array_push( $warnings, __( 'There are no related entities for the current entity.', 'wordlift' ) );
+
+        // Is the entity published?
+        ( 'publish' === get_post_status( $post->ID ) ) ?
+            $score ++ :
+            array_push( $warnings, __( 'This entity is not published. It will not appear within analysis results.', 'wordlift' ) );
+
+        // Has a thumbnail?
+        ( has_post_thumbnail( $post->ID ) ) ?
+            $score ++ :
+            array_push( $warnings, __( 'This entity has no featured image yet.', 'wordlift' ) );
+
+        // Get all post meta keys for the current post
+        global $wpdb;
+
+        $query = $wpdb->prepare(
+            "SELECT DISTINCT (meta_key) FROM $wpdb->postmeta  WHERE post_id = %d", $post->ID
+        );
+
+        // Check intersection between available meta keys and expected ones
+        // arrays to detect missing values.
+        $available_meta_keys = $wpdb->get_col( $query );
+
+        // If each expected key is contained in available keys array ...
+        ( in_array( Wordlift_Schema_Service::FIELD_SAME_AS, $available_meta_keys ) ) ?
+            $score ++ :
+            array_push( $warnings, __( 'There are no sameAs configured for this entity.', 'wordlift' ) );
+
+        $schema = $this->entity_type_service->get( $post_id );
+
+        $expected_meta_keys = ( null === $schema['custom_fields'] ) ?
+            array() :
+            array_keys( $schema['custom_fields'] );
+
+        $intersection = array_intersect( $expected_meta_keys, $available_meta_keys );
+        // If each expected key is contained in available keys array ...
+        ( count( $intersection ) === count( $expected_meta_keys ) ) ?
+            $score ++ :
+            array_push( $warnings, __( 'Schema.org metadata for this entity are not completed.', 'wordlift' ) );
+
+        // Finally return score and warnings
+        return array(
+            'raw_score'           => $score,
+            'traffic_light_score' => $this->convert_raw_score_to_traffic_light( $score ),
+            'percentage_score'    => $this->convert_raw_score_to_percentage( $score ),
+            'warnings'            => $warnings,
+        );
+
+    }
+
+    /**
+     * Get as rating as input and convert in a traffic-light rating
+     *
+     * @since 3.3.0
+     *
+     * @param int $score The rating score for a given entity.
+     *
+     * @return string The input HTML code.
+     */
+    private function convert_raw_score_to_traffic_light( $score ) {
+        // RATING_MAX : $score = 3 : x
+        // See http://php.net/manual/en/function.round.php
+        $rating = round( ( $score * 3 ) / self::RATING_MAX, 0, PHP_ROUND_HALF_UP );
+
+        // If rating is 0, return 1, otherwise return rating
+        return ( 0 == $rating ) ? 1 : $rating;
+
+    }
+
+    /**
+     * Get as rating as input and convert in a traffic-light rating
+     *
+     * @since 3.3.0
+     *
+     * @param int $score The rating score for a given entity.
+     *
+     * @return string The input HTML code.
+     */
+    public function convert_raw_score_to_percentage( $score ) {
+
+        // RATING_MAX : $score = 100 : x
+        return round( ( $score * 100 ) / self::RATING_MAX, 0, PHP_ROUND_HALF_UP );
+    }
+
+
+    /**
+     * Add admin notices for the current entity depending on the current rating.
+     *
+     * @since 3.3.0
+     */
+    public function in_admin_header() {
+
+        // Return safely if get_current_screen() is not defined (yet)
+        if ( false === function_exists( 'get_current_screen' ) ) {
+            return;
+        }
+
+        $screen = get_current_screen();
+        // If there is any valid screen nothing to do
+        if ( null === $screen ) {
+            return;
+        }
+
+        // If you're not in the entity post edit page, return.
+        if ( Wordlift_Entity_Service::TYPE_NAME !== $screen->id ) {
+            return;
+        }
+        // Retrieve the current global post
+        global $post;
+        // If it's not an entity, return.
+        if ( ! $this->entity_service->is_entity( $post->ID ) ) {
+            return;
+        }
+        // Retrieve an updated rating for the current entity
+        $rating = $this->get_rating_for( $post->ID, true );
+
+        // If there is at least 1 warning
+        if ( isset( $rating['warnings'] ) && 0 < count( $rating['warnings'] ) ) {
+            // TODO - Pass Wordlift_Notice_Service trough the service constructor
+            $this->notice_service->add_suggestion( $rating['warnings'] );
+        }
+
+    }
 
 }

Spacing +55 added lines, -63 removed lines

@@ -129,13 +129,13 @@ 
 	 * @param \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
 	 * @param \Wordlift_Notice_Service      $notice_service      A {@link Wordlift_Notice_Service} instance.
 	 */
-	public function __construct( $entity_service, $entity_type_service, $notice_service ) {
+	public function __construct($entity_service, $entity_type_service, $notice_service) {
 
 		$this->entity_service      = $entity_service;
 		$this->entity_type_service = $entity_type_service;
 		$this->notice_service      = $notice_service;
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Rating_Service' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Rating_Service');
 
 	}
 
@@ -148,23 +148,23 @@ 
 	 *
 	 * @return array An array representing the rating obj.
 	 */
-	public function set_rating_for( $post_id ) {
+	public function set_rating_for($post_id) {
 
-		if ( ! $this->entity_service->is_entity( $post_id ) ) {
+		if ( ! $this->entity_service->is_entity($post_id)) {
 			return;
 		}
 
 		// Calculate rating for the given post.
-		$rating = $this->calculate_rating_for( $post_id );
+		$rating = $this->calculate_rating_for($post_id);
 
 		// Store the rating on db as post meta. Please notice that RATING_RAW_SCORE_META_KEY
 		// is saved on a different meta to allow score sorting. Both meta are managed as unique.
 		//
 		// See https://codex.wordpress.org/Function_Reference/update_post_meta
-		update_post_meta( $post_id, self::RATING_RAW_SCORE_META_KEY, $rating['raw_score'] );
-		update_post_meta( $post_id, self::RATING_WARNINGS_META_KEY, $rating['warnings'] );
+		update_post_meta($post_id, self::RATING_RAW_SCORE_META_KEY, $rating['raw_score']);
+		update_post_meta($post_id, self::RATING_WARNINGS_META_KEY, $rating['warnings']);
 
-		$this->log->trace( sprintf( "Rating set for [ post_id :: $post_id ] [ rating :: %s ]", $rating['raw_score'] ) );
+		$this->log->trace(sprintf("Rating set for [ post_id :: $post_id ] [ rating :: %s ]", $rating['raw_score']));
 
 		// Finally returns the rating
 		return $rating;
@@ -181,30 +181,30 @@ 
 	 *
 	 * @return array An array representing the rating obj.
 	 */
-	public function get_rating_for( $post_id, $force_reload = false ) {
+	public function get_rating_for($post_id, $force_reload = false) {
 
 		// If forced reload is required or rating is missing.
-		if ( $force_reload ) {
-			$this->log->trace( "Force rating reload [ post_id :: $post_id ]" );
+		if ($force_reload) {
+			$this->log->trace("Force rating reload [ post_id :: $post_id ]");
 
-			return $this->set_rating_for( $post_id );
+			return $this->set_rating_for($post_id);
 		}
 
-		$current_raw_score = get_post_meta( $post_id, self::RATING_RAW_SCORE_META_KEY, true );
+		$current_raw_score = get_post_meta($post_id, self::RATING_RAW_SCORE_META_KEY, true);
 
-		if ( ! is_numeric( $current_raw_score ) ) {
-			$this->log->trace( "Rating missing for [ post_id :: $post_id ] [ current_raw_score :: $current_raw_score ]" );
+		if ( ! is_numeric($current_raw_score)) {
+			$this->log->trace("Rating missing for [ post_id :: $post_id ] [ current_raw_score :: $current_raw_score ]");
 
-			return $this->set_rating_for( $post_id );
+			return $this->set_rating_for($post_id);
 		}
 
-		$current_warnings = get_post_meta( $post_id, self::RATING_WARNINGS_META_KEY, true );
+		$current_warnings = get_post_meta($post_id, self::RATING_WARNINGS_META_KEY, true);
 
 		// Finally return score and warnings
 		return array(
 			'raw_score'           => $current_raw_score,
-			'traffic_light_score' => $this->convert_raw_score_to_traffic_light( $current_raw_score ),
-			'percentage_score'    => $this->convert_raw_score_to_percentage( $current_raw_score ),
+			'traffic_light_score' => $this->convert_raw_score_to_traffic_light($current_raw_score),
+			'percentage_score'    => $this->convert_raw_score_to_percentage($current_raw_score),
 			'warnings'            => $current_warnings,
 		);
 
@@ -231,14 +231,14 @@ 
 	 *
 	 * @return array An array representing the rating obj.
 	 */
-	private function calculate_rating_for( $post_id ) {
+	private function calculate_rating_for($post_id) {
 
 		// If it's not an entity, return.
-		if ( ! $this->entity_service->is_entity( $post_id ) ) {
+		if ( ! $this->entity_service->is_entity($post_id)) {
 			return array();
 		}
 		// Retrieve the post object.
-		$post = get_post( $post_id );
+		$post = get_post($post_id);
 
 		// Rating value.
 		$score = 0;
@@ -247,30 +247,25 @@ 
 		$warnings = array();
 
 		// Is the current entity related to at least 1 post?
-		( 0 < count( wl_core_get_related_post_ids( $post->ID ) ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'There are no related posts for the current entity.', 'wordlift' ) );
+		(0 < count(wl_core_get_related_post_ids($post->ID))) ?
+			$score++ : array_push($warnings, __('There are no related posts for the current entity.', 'wordlift'));
 
 		// Is the post content not empty?
-		( ! empty( $post->post_content ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'This entity has not description.', 'wordlift' ) );
+		( ! empty($post->post_content)) ?
+			$score++ : array_push($warnings, __('This entity has not description.', 'wordlift'));
 
 		// Is the current entity related to at least 1 entity?
 		// Was the current entity already disambiguated?
-		( 0 < count( wl_core_get_related_entity_ids( $post->ID ) ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'There are no related entities for the current entity.', 'wordlift' ) );
+		(0 < count(wl_core_get_related_entity_ids($post->ID))) ?
+			$score++ : array_push($warnings, __('There are no related entities for the current entity.', 'wordlift'));
 
 		// Is the entity published?
-		( 'publish' === get_post_status( $post->ID ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'This entity is not published. It will not appear within analysis results.', 'wordlift' ) );
+		('publish' === get_post_status($post->ID)) ?
+			$score++ : array_push($warnings, __('This entity is not published. It will not appear within analysis results.', 'wordlift'));
 
 		// Has a thumbnail?
-		( has_post_thumbnail( $post->ID ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'This entity has no featured image yet.', 'wordlift' ) );
+		(has_post_thumbnail($post->ID)) ?
+			$score++ : array_push($warnings, __('This entity has no featured image yet.', 'wordlift'));
 
 		// Get all post meta keys for the current post
 		global $wpdb;
@@ -281,30 +276,27 @@ 
 
 		// Check intersection between available meta keys and expected ones
 		// arrays to detect missing values.
-		$available_meta_keys = $wpdb->get_col( $query );
+		$available_meta_keys = $wpdb->get_col($query);
 
 		// If each expected key is contained in available keys array ...
-		( in_array( Wordlift_Schema_Service::FIELD_SAME_AS, $available_meta_keys ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'There are no sameAs configured for this entity.', 'wordlift' ) );
+		(in_array(Wordlift_Schema_Service::FIELD_SAME_AS, $available_meta_keys)) ?
+			$score++ : array_push($warnings, __('There are no sameAs configured for this entity.', 'wordlift'));
 
-		$schema = $this->entity_type_service->get( $post_id );
+		$schema = $this->entity_type_service->get($post_id);
 
-		$expected_meta_keys = ( null === $schema['custom_fields'] ) ?
-			array() :
-			array_keys( $schema['custom_fields'] );
+		$expected_meta_keys = (null === $schema['custom_fields']) ?
+			array() : array_keys($schema['custom_fields']);
 
-		$intersection = array_intersect( $expected_meta_keys, $available_meta_keys );
+		$intersection = array_intersect($expected_meta_keys, $available_meta_keys);
 		// If each expected key is contained in available keys array ...
-		( count( $intersection ) === count( $expected_meta_keys ) ) ?
-			$score ++ :
-			array_push( $warnings, __( 'Schema.org metadata for this entity are not completed.', 'wordlift' ) );
+		(count($intersection) === count($expected_meta_keys)) ?
+			$score++ : array_push($warnings, __('Schema.org metadata for this entity are not completed.', 'wordlift'));
 
 		// Finally return score and warnings
 		return array(
 			'raw_score'           => $score,
-			'traffic_light_score' => $this->convert_raw_score_to_traffic_light( $score ),
-			'percentage_score'    => $this->convert_raw_score_to_percentage( $score ),
+			'traffic_light_score' => $this->convert_raw_score_to_traffic_light($score),
+			'percentage_score'    => $this->convert_raw_score_to_percentage($score),
 			'warnings'            => $warnings,
 		);
 
@@ -319,13 +311,13 @@ 
 	 *
 	 * @return string The input HTML code.
 	 */
-	private function convert_raw_score_to_traffic_light( $score ) {
+	private function convert_raw_score_to_traffic_light($score) {
 		// RATING_MAX : $score = 3 : x
 		// See http://php.net/manual/en/function.round.php
-		$rating = round( ( $score * 3 ) / self::RATING_MAX, 0, PHP_ROUND_HALF_UP );
+		$rating = round(($score * 3) / self::RATING_MAX, 0, PHP_ROUND_HALF_UP);
 
 		// If rating is 0, return 1, otherwise return rating
-		return ( 0 == $rating ) ? 1 : $rating;
+		return (0 == $rating) ? 1 : $rating;
 
 	}
 
@@ -338,10 +330,10 @@ 
 	 *
 	 * @return string The input HTML code.
 	 */
-	public function convert_raw_score_to_percentage( $score ) {
+	public function convert_raw_score_to_percentage($score) {
 
 		// RATING_MAX : $score = 100 : x
-		return round( ( $score * 100 ) / self::RATING_MAX, 0, PHP_ROUND_HALF_UP );
+		return round(($score * 100) / self::RATING_MAX, 0, PHP_ROUND_HALF_UP);
 	}
 
 
@@ -353,33 +345,33 @@ 
 	public function in_admin_header() {
 
 		// Return safely if get_current_screen() is not defined (yet)
-		if ( false === function_exists( 'get_current_screen' ) ) {
+		if (false === function_exists('get_current_screen')) {
 			return;
 		}
 
 		$screen = get_current_screen();
 		// If there is any valid screen nothing to do
-		if ( null === $screen ) {
+		if (null === $screen) {
 			return;
 		}
 
 		// If you're not in the entity post edit page, return.
-		if ( Wordlift_Entity_Service::TYPE_NAME !== $screen->id ) {
+		if (Wordlift_Entity_Service::TYPE_NAME !== $screen->id) {
 			return;
 		}
 		// Retrieve the current global post
 		global $post;
 		// If it's not an entity, return.
-		if ( ! $this->entity_service->is_entity( $post->ID ) ) {
+		if ( ! $this->entity_service->is_entity($post->ID)) {
 			return;
 		}
 		// Retrieve an updated rating for the current entity
-		$rating = $this->get_rating_for( $post->ID, true );
+		$rating = $this->get_rating_for($post->ID, true);
 
 		// If there is at least 1 warning
-		if ( isset( $rating['warnings'] ) && 0 < count( $rating['warnings'] ) ) {
+		if (isset($rating['warnings']) && 0 < count($rating['warnings'])) {
 			// TODO - Pass Wordlift_Notice_Service trough the service constructor
-			$this->notice_service->add_suggestion( $rating['warnings'] );
+			$this->notice_service->add_suggestion($rating['warnings']);
 		}
 
 	}

src/includes/linked-data/storage/class-wordlift-post-author-storage.php

Indentation +50 added lines, -50 removed lines

@@ -18,63 +18,63 @@
  */
 class Wordlift_Post_Author_Storage extends Wordlift_Post_Property_Storage {
 
-	/**
-	 * The {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
+    /**
+     * The {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
 
-	/**
-	 * The {@link Wordlift_User_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_User_Service $user_service The {@link Wordlift_User_Service} instance.
-	 */
-	private $user_service;
+    /**
+     * The {@link Wordlift_User_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_User_Service $user_service The {@link Wordlift_User_Service} instance.
+     */
+    private $user_service;
 
-	/**
-	 * Create a {@link Wordlift_Post_Author_Storage} instance.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 * @param \Wordlift_User_Service   $user_service   The {@link Wordlift_User_Service} instance.
-	 */
-	public function __construct( $entity_service, $user_service ) {
-		parent::__construct( Wordlift_Post_Property_Storage::AUTHOR );
+    /**
+     * Create a {@link Wordlift_Post_Author_Storage} instance.
+     *
+     * @since 3.15.0
+     *
+     * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     * @param \Wordlift_User_Service   $user_service   The {@link Wordlift_User_Service} instance.
+     */
+    public function __construct( $entity_service, $user_service ) {
+        parent::__construct( Wordlift_Post_Property_Storage::AUTHOR );
 
-		$this->entity_service = $entity_service;
-		$this->user_service   = $user_service;
+        $this->entity_service = $entity_service;
+        $this->user_service   = $user_service;
 
-	}
+    }
 
-	/**
-	 * Get the property value.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array|string|null A single string, or an array of values or null
-	 *                           if the property isn't recognized.
-	 */
-	public function get( $post_id ) {
-		$author_id = parent::get( $post_id );
+    /**
+     * Get the property value.
+     *
+     * @since 3.15.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array|string|null A single string, or an array of values or null
+     *                           if the property isn't recognized.
+     */
+    public function get( $post_id ) {
+        $author_id = parent::get( $post_id );
 
-		// Get the entity bound to this user.
-		$entity_id = $this->user_service->get_entity( $author_id );
+        // Get the entity bound to this user.
+        $entity_id = $this->user_service->get_entity( $author_id );
 
-		// If there's no entity bound return a simple author structure.
-		if ( empty( $entity_id ) ) {
-			return $this->user_service->get_uri( $author_id );
-		}
+        // If there's no entity bound return a simple author structure.
+        if ( empty( $entity_id ) ) {
+            return $this->user_service->get_uri( $author_id );
+        }
 
-		// Return the entity URI.
-		return $this->entity_service->get_uri( $entity_id );
-	}
+        // Return the entity URI.
+        return $this->entity_service->get_uri( $entity_id );
+    }
 
 }

Spacing +8 added lines, -8 removed lines

@@ -44,8 +44,8 @@ 
 	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
 	 * @param \Wordlift_User_Service   $user_service   The {@link Wordlift_User_Service} instance.
 	 */
-	public function __construct( $entity_service, $user_service ) {
-		parent::__construct( Wordlift_Post_Property_Storage::AUTHOR );
+	public function __construct($entity_service, $user_service) {
+		parent::__construct(Wordlift_Post_Property_Storage::AUTHOR);
 
 		$this->entity_service = $entity_service;
 		$this->user_service   = $user_service;
@@ -62,19 +62,19 @@ 
 	 * @return array|string|null A single string, or an array of values or null
 	 *                           if the property isn't recognized.
 	 */
-	public function get( $post_id ) {
-		$author_id = parent::get( $post_id );
+	public function get($post_id) {
+		$author_id = parent::get($post_id);
 
 		// Get the entity bound to this user.
-		$entity_id = $this->user_service->get_entity( $author_id );
+		$entity_id = $this->user_service->get_entity($author_id);
 
 		// If there's no entity bound return a simple author structure.
-		if ( empty( $entity_id ) ) {
-			return $this->user_service->get_uri( $author_id );
+		if (empty($entity_id)) {
+			return $this->user_service->get_uri($author_id);
 		}
 
 		// Return the entity URI.
-		return $this->entity_service->get_uri( $entity_id );
+		return $this->entity_service->get_uri($entity_id);
 	}
 
 }

src/includes/linked-data/storage/class-wordlift-post-taxonomy-storage.php

Indentation +39 added lines, -39 removed lines

@@ -18,48 +18,48 @@
  */
 class Wordlift_Post_Taxonomy_Storage extends Wordlift_Storage {
 
-	/**
-	 * The taxonomy name.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var string $taxonomy The taxonomy name.
-	 */
-	private $taxonomy;
+    /**
+     * The taxonomy name.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var string $taxonomy The taxonomy name.
+     */
+    private $taxonomy;
 
-	/**
-	 * Create a {@link Wordlift_Post_Taxonomy_Storage} with the specified
-	 * taxonomy name.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param string $taxonomy The taxonomy name.
-	 */
-	public function __construct( $taxonomy ) {
+    /**
+     * Create a {@link Wordlift_Post_Taxonomy_Storage} with the specified
+     * taxonomy name.
+     *
+     * @since 3.15.0
+     *
+     * @param string $taxonomy The taxonomy name.
+     */
+    public function __construct( $taxonomy ) {
 
-		$this->taxonomy = $taxonomy;
-	}
+        $this->taxonomy = $taxonomy;
+    }
 
-	/**
-	 * Get the taxonomy's terms associated with the specified {@link WP_Post}.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array|WP_Error An array of terms or {@link WP_Error} in case of error.
-	 */
-	public function get( $post_id ) {
+    /**
+     * Get the taxonomy's terms associated with the specified {@link WP_Post}.
+     *
+     * @since 3.15.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array|WP_Error An array of terms or {@link WP_Error} in case of error.
+     */
+    public function get( $post_id ) {
 
-		return wp_get_post_terms( $post_id, $this->taxonomy, array(
-			'hide_empty' => false,
-			// Because of #334 (and the AAM plugin) we changed fields from 'id=>slug' to 'all'.
-			// An issue has been opened with the AAM plugin author as well.
-			//
-			// see https://github.com/insideout10/wordlift-plugin/issues/334
-			// see https://wordpress.org/support/topic/idslug-not-working-anymore?replies=1#post-8806863
-			'fields'     => 'all',
-		) );
-	}
+        return wp_get_post_terms( $post_id, $this->taxonomy, array(
+            'hide_empty' => false,
+            // Because of #334 (and the AAM plugin) we changed fields from 'id=>slug' to 'all'.
+            // An issue has been opened with the AAM plugin author as well.
+            //
+            // see https://github.com/insideout10/wordlift-plugin/issues/334
+            // see https://wordpress.org/support/topic/idslug-not-working-anymore?replies=1#post-8806863
+            'fields'     => 'all',
+        ) );
+    }
 
 }

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
 	 *
 	 * @param string $taxonomy The taxonomy name.
 	 */
-	public function __construct( $taxonomy ) {
+	public function __construct($taxonomy) {
 
 		$this->taxonomy = $taxonomy;
 	}
@@ -49,9 +49,9 @@ 
 	 *
 	 * @return array|WP_Error An array of terms or {@link WP_Error} in case of error.
 	 */
-	public function get( $post_id ) {
+	public function get($post_id) {
 
-		return wp_get_post_terms( $post_id, $this->taxonomy, array(
+		return wp_get_post_terms($post_id, $this->taxonomy, array(
 			'hide_empty' => false,
 			// Because of #334 (and the AAM plugin) we changed fields from 'id=>slug' to 'all'.
 			// An issue has been opened with the AAM plugin author as well.
@@ -59,7 +59,7 @@ 
 			// see https://github.com/insideout10/wordlift-plugin/issues/334
 			// see https://wordpress.org/support/topic/idslug-not-working-anymore?replies=1#post-8806863
 			'fields'     => 'all',
-		) );
+		));
 	}
 
 }

src/includes/linked-data/storage/class-wordlift-post-meta-storage.php

Indentation +36 added lines, -36 removed lines

@@ -18,41 +18,41 @@
  */
 class Wordlift_Post_Meta_Storage extends Wordlift_Storage {
 
-	/**
-	 * The meta key to load data from.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var string $meta_key The meta key to load data from.
-	 */
-	private $meta_key;
-
-	/**
-	 * Create a {@link Wordlift_Post_Meta_Storage} instance, by providing the
-	 * meta key the storage should read from.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param string $meta_key The post meta key.
-	 */
-	public function __construct( $meta_key ) {
-
-		$this->meta_key = $meta_key;
-
-	}
-
-	/**
-	 * Get the value for the specified meta key.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array An array of values (or an empty array if nothing is set).
-	 */
-	public function get( $post_id ) {
-
-		return get_post_meta( $post_id, $this->meta_key );
-	}
+    /**
+     * The meta key to load data from.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var string $meta_key The meta key to load data from.
+     */
+    private $meta_key;
+
+    /**
+     * Create a {@link Wordlift_Post_Meta_Storage} instance, by providing the
+     * meta key the storage should read from.
+     *
+     * @since 3.15.0
+     *
+     * @param string $meta_key The post meta key.
+     */
+    public function __construct( $meta_key ) {
+
+        $this->meta_key = $meta_key;
+
+    }
+
+    /**
+     * Get the value for the specified meta key.
+     *
+     * @since 3.15.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array An array of values (or an empty array if nothing is set).
+     */
+    public function get( $post_id ) {
+
+        return get_post_meta( $post_id, $this->meta_key );
+    }
 
 }

Spacing +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@ 
 	 *
 	 * @param string $meta_key The post meta key.
 	 */
-	public function __construct( $meta_key ) {
+	public function __construct($meta_key) {
 
 		$this->meta_key = $meta_key;
 
@@ -50,9 +50,9 @@ 
 	 *
 	 * @return array An array of values (or an empty array if nothing is set).
 	 */
-	public function get( $post_id ) {
+	public function get($post_id) {
 
-		return get_post_meta( $post_id, $this->meta_key );
+		return get_post_meta($post_id, $this->meta_key);
 	}
 
 }

src/includes/linked-data/storage/class-wordlift-post-related-storage.php

Spacing +6 added lines, -6 removed lines

@@ -34,7 +34,7 @@ 
 	 *
 	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
 	 */
-	public function __construct( $entity_service ) {
+	public function __construct($entity_service) {
 
 		$this->entity_service = $entity_service;
 
@@ -50,18 +50,18 @@ 
 	 * @return array|string|null A single string, or an array of values or null
 	 *                           if the property isn't recognized.
 	 */
-	public function get( $post_id ) {
+	public function get($post_id) {
 
 		// get related entities.
-		$related = wl_core_get_related_entity_ids( $post_id );
+		$related = wl_core_get_related_entity_ids($post_id);
 
 		// A reference to the entity service.
 		$entity_service = $this->entity_service;
 
 		// Map the related posts' ids to URIs.
-		return array_map( function ( $item ) use ( $entity_service ) {
-			return $entity_service->get_uri( $item );
-		}, $related );
+		return array_map(function($item) use ($entity_service) {
+			return $entity_service->get_uri($item);
+		}, $related);
 	}
 
 }

Indentation +41 added lines, -41 removed lines

@@ -21,53 +21,53 @@
  */
 class Wordlift_Post_Related_Storage extends Wordlift_Storage {
 
-	/**
-	 * The {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
+    /**
+     * The {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
 
-	/**
-	 * Create a {@link Wordlift_Post_Related_Storage} instance.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	public function __construct( $entity_service ) {
+    /**
+     * Create a {@link Wordlift_Post_Related_Storage} instance.
+     *
+     * @since 3.15.0
+     *
+     * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    public function __construct( $entity_service ) {
 
-		$this->entity_service = $entity_service;
+        $this->entity_service = $entity_service;
 
-	}
+    }
 
-	/**
-	 * Get the property value.
-	 *
-	 * There is no filter for entities or posts, the returned data here can
-	 * be used for `relations` and `references` according to the client.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array|string|null A single string, or an array of values or null
-	 *                           if the property isn't recognized.
-	 */
-	public function get( $post_id ) {
+    /**
+     * Get the property value.
+     *
+     * There is no filter for entities or posts, the returned data here can
+     * be used for `relations` and `references` according to the client.
+     *
+     * @since 3.15.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array|string|null A single string, or an array of values or null
+     *                           if the property isn't recognized.
+     */
+    public function get( $post_id ) {
 
-		// get related entities.
-		$related = wl_core_get_related_entity_ids( $post_id );
+        // get related entities.
+        $related = wl_core_get_related_entity_ids( $post_id );
 
-		// A reference to the entity service.
-		$entity_service = $this->entity_service;
+        // A reference to the entity service.
+        $entity_service = $this->entity_service;
 
-		// Map the related posts' ids to URIs.
-		return array_map( function ( $item ) use ( $entity_service ) {
-			return $entity_service->get_uri( $item );
-		}, $related );
-	}
+        // Map the related posts' ids to URIs.
+        return array_map( function ( $item ) use ( $entity_service ) {
+            return $entity_service->get_uri( $item );
+        }, $related );
+    }
 
 }

src/includes/linked-data/storage/class-wordlift-storage.php

Indentation +26 added lines, -26 removed lines

@@ -16,35 +16,35 @@
  */
 abstract class Wordlift_Storage {
 
-	/**
-	 * Get the values for the property of the {@link WP_Post}.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array
-	 */
-	abstract public function get( $post_id );
+    /**
+     * Get the values for the property of the {@link WP_Post}.
+     *
+     * @since 3.15.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array
+     */
+    abstract public function get( $post_id );
 
-	/**
-	 * Get the first value for a property.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return string The first property value or an empty string.
-	 */
-	public function get_first( $post_id ) {
+    /**
+     * Get the first value for a property.
+     *
+     * @since 3.15.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return string The first property value or an empty string.
+     */
+    public function get_first( $post_id ) {
 
-		$values = $this->get( $post_id );
+        $values = $this->get( $post_id );
 
-		if ( empty( $values ) ) {
-			return '';
-		}
+        if ( empty( $values ) ) {
+            return '';
+        }
 
-		return $values[0];
-	}
+        return $values[0];
+    }
 
 }

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
 	 *
 	 * @return array
 	 */
-	abstract public function get( $post_id );
+	abstract public function get($post_id);
 
 	/**
 	 * Get the first value for a property.
@@ -36,11 +36,11 @@ 
 	 *
 	 * @return string The first property value or an empty string.
 	 */
-	public function get_first( $post_id ) {
+	public function get_first($post_id) {
 
-		$values = $this->get( $post_id );
+		$values = $this->get($post_id);
 
-		if ( empty( $values ) ) {
+		if (empty($values)) {
 			return '';
 		}