insideout10 / wordlift-plugin

src/wordlift/mappings/class-jsonld-converter.php

Indentation +254 added lines, -254 removed lines

@@ -16,259 +16,259 @@
  * @since 3.25.0
  */
 class Jsonld_Converter {
-	/**
-	 * Enumerations for the field types.
-	 * Enumerations for the field types.
-	 */
-	const FIELD_TYPE_TEXT_FIELD = 'text';
-	const FIELD_TYPE_CUSTOM_FIELD = 'custom_field';
-	const FIELD_TYPE_ACF = 'acf';
-	/**
-	 * The {@link Mappings_Validator} instance to test.
-	 *
-	 * @since  3.25.0
-	 * @access private
-	 * @var Mappings_Validator $validator The {@link Mappings_Validator} instance.
-	 */
-	private $validator;
-
-	/**
-	 * The {@link Mappings_Transform_Functions_Registry} instance.
-	 *
-	 * @since  3.25.0
-	 * @access private
-	 * @var Mappings_Transform_Functions_Registry $transform_functions_registry The {@link Mappings_Transform_Functions_Registry} instance.
-	 */
-	private $transform_functions_registry;
-
-	/**
-	 * Initialize all dependencies required.
-	 *
-	 * @param Mappings_Validator $validator A {@link Mappings_Validator} instance.
-	 * @param Mappings_Transform_Functions_Registry $transform_functions_registry
-	 */
-	public function __construct( $validator, $transform_functions_registry ) {
-
-		$this->validator                    = $validator;
-		$this->transform_functions_registry = $transform_functions_registry;
-
-		// Hook to refactor the JSON-LD.
-		add_filter( 'wl_post_jsonld_array', array( $this, 'wl_post_jsonld_array' ), 11, 2 );
-		add_filter( 'wl_entity_jsonld_array', array( $this, 'wl_post_jsonld_array' ), 11, 3 );
-
-	}
-
-	/**
-	 * Hook to `wl_post_jsonld_array` and `wl_entity_jsonld_array`.
-	 *
-	 * Receive the JSON-LD and the references in the array along with the post ID and transform them according to
-	 * the configuration.
-	 *
-	 * @param array $value {
-	 *      The array containing the JSON-LD and the references.
-	 *
-	 * @type array $jsonld The JSON-LD array.
-	 * @type int[] $references An array of post ID referenced by the JSON-LD (will be expanded by the converter).
-	 * }
-	 *
-	 * @param int $post_id The post ID.
-	 *
-	 * @return array An array with the updated JSON-LD and references.
-	 */
-	public function wl_post_jsonld_array( $value, $post_id ) {
-
-		$jsonld     = $value['jsonld'];
-		$references = $value['references'];
-
-		return array(
-			'jsonld'     => $this->wl_post_jsonld( $jsonld, $post_id, $references ),
-			'references' => $references,
-		);
-	}
-
-	/**
-	 * Returns JSON-LD data after applying transformation functions.
-	 *
-	 * @param array $jsonld The JSON-LD structure.
-	 * @param int $post_id The {@link WP_Post} id.
-	 * @param array $references An array of post references.
-	 *
-	 * @return array the new refactored array structure.
-	 * @since 3.25.0
-	 */
-	private function wl_post_jsonld( $jsonld, $post_id, &$references ) {
-
-		// @@todo I think there's an issue here with the Validator, because you're changing the instance state and the
-		// instance may be reused afterwards.
-
-		$properties        = $this->validator->validate( $post_id );
-		$nested_properties = array();
-
-		foreach ( $properties as $property ) {
-			// If the property has the character '/' in the property name then it is a nested property.
-			if ( strpos( $property['property_name'], '/' ) !== false ) {
-				$nested_properties[] = $property;
-				continue;
-			}
-			$property_transformed_data = $this->get_property_data( $property, $jsonld, $post_id, $references );
-			if ( false !== $property_transformed_data ) {
-				$jsonld[ $property['property_name'] ] = $property_transformed_data;
-			}
-		}
-
-		$jsonld = $this->process_nested_properties( $nested_properties, $jsonld, $post_id, $references );
-
-		return $jsonld;
-	}
-
-	/**
-	 * Get the property data by applying the transformation function
-	 *
-	 * @param $property
-	 * @param $jsonld
-	 * @param $post_id
-	 * @param $references
-	 *
-	 * @return array|bool|null
-	 */
-	public function get_property_data( $property, $jsonld, $post_id, &$references ) {
-		$transform_instance = $this->transform_functions_registry->get_transform_function( $property['transform_function'] );
-		$data               = $this->get_data_from_data_source( $post_id, $property );
-		if ( null !== $transform_instance ) {
-			$transform_data = $transform_instance->transform_data( $data, $jsonld, $references, $post_id );
-			if ( null !== $transform_data ) {
-				return $this->make_single( $transform_data );
-			}
-		} else {
-			return $this->make_single( $data );
-		}
-
-		return false;
-	}
-
-	/**
-	 * Process all the nested properties.
-	 *
-	 * @param $nested_properties array
-	 * @param $jsonld array
-	 *
-	 * @return array
-	 */
-	public function process_nested_properties( $nested_properties, $jsonld, $post_id, &$references ) {
-		foreach ( $nested_properties as $property ) {
-			$property_data = $this->get_property_data( $property, $jsonld, $post_id, $references );
-			if ( false === $property_data ) {
-				// No need to create nested levels.
-				continue;
-			}
-
-			$keys = explode( '/', $property['property_name'] );
-			// end is the last level of the nested property.
-			$end                      = array_pop( $keys );
-			$current_property_pointer = &$jsonld;
-
-			/**
-			 * Once we find all the nested levels from the property name
-			 * loop through it and create associative array if the levels
-			 * didnt exist.
-			 */
-			while ( count( $keys ) > 0 ) {
-				$key = array_shift( $keys );
-				if ( $key === "" ) {
-					continue;
-				}
-				if ( ! array_key_exists( $key, $current_property_pointer ) ) {
-					$current_property_pointer[ $key ] = array();
-				}
-				// We are setting the pointer to the current key, so that at the end
-				// we can add the data at last level.
-				$current_property_pointer = &$current_property_pointer[ $key ];
-			}
-			$current_property_pointer[ $end ] = $property_data;
-		}
-
-		return $jsonld;
-	}
-
-
-	/**
-	 * Returns data from data source.
-	 *
-	 * @param int $post_id Id of the post.
-	 * @param array $property_data The property data for the post_id.
-	 *
-	 * @return array Returns key, value array, if the value is not found, then it returns null.
-	 */
-	final public function get_data_from_data_source( $post_id, $property_data ) {
-		$value = $property_data['field_name'];
-
-		// Do 1 to 1 mapping and return result.
-		switch ( $property_data['field_type'] ) {
-			case self::FIELD_TYPE_ACF:
-				if ( ! function_exists( 'get_field' ) || ! function_exists( 'get_field_object' ) ) {
-					return array();
-				}
-
-				return $this->get_data_for_acf_field( $property_data['field_name'], $post_id );
-
-			case self::FIELD_TYPE_CUSTOM_FIELD:
-
-				return array_map( 'wp_strip_all_tags', get_post_meta( $post_id, $value ) );
-
-			default:
-				return $value;
-		}
-
-	}
-
-	/**
-	 * Gets data from acf, format the data if it is a repeater field.
-	 *
-	 * @param $field_name
-	 * @param $post_id
-	 *
-	 * @return array|mixed
-	 */
-	public function get_data_for_acf_field( $field_name, $post_id ) {
-		$field_data = get_field_object( $field_name, $post_id );
-		$data       = get_field( $field_name, $post_id );
-
-		// only process if it is a repeater field, else return the data.
-		if ( is_array( $field_data ) && array_key_exists( 'type', $field_data )
-		     && $field_data['type'] === 'repeater' ) {
-			// check if we have only one sub field, currently we only support one subfield.
-			if ( is_array( $data ) && count( $data ) > 0 && count( array_keys( $data[0] ) === 1 ) ) {
-				$repeater_formatted_data = array();
-				foreach ( $data as $item ) {
-					$repeater_formatted_data = array_merge( $repeater_formatted_data, array_values( $item ) );
-				}
-				// Remove non unique values.
-				$repeater_formatted_data = array_unique( $repeater_formatted_data );
-				// Remove empty values
-				$repeater_formatted_data = array_filter( $repeater_formatted_data, 'strlen' );
-
-				// re-index all the values.
-				return array_values( $repeater_formatted_data );
-			}
-		}
-
-		// Return normal acf data if it is not a repeater field.
-		return $data;
-	}
-
-	private function make_single( $value ) {
-
-		$values = (array) $value;
-
-		if ( empty( $values ) ) {
-			return null;
-		}
-
-		if ( 1 === count( $values ) && 0 === key( $values ) ) {
-			return current( $values );
-		}
-
-		return $values;
-	}
+    /**
+     * Enumerations for the field types.
+     * Enumerations for the field types.
+     */
+    const FIELD_TYPE_TEXT_FIELD = 'text';
+    const FIELD_TYPE_CUSTOM_FIELD = 'custom_field';
+    const FIELD_TYPE_ACF = 'acf';
+    /**
+     * The {@link Mappings_Validator} instance to test.
+     *
+     * @since  3.25.0
+     * @access private
+     * @var Mappings_Validator $validator The {@link Mappings_Validator} instance.
+     */
+    private $validator;
+
+    /**
+     * The {@link Mappings_Transform_Functions_Registry} instance.
+     *
+     * @since  3.25.0
+     * @access private
+     * @var Mappings_Transform_Functions_Registry $transform_functions_registry The {@link Mappings_Transform_Functions_Registry} instance.
+     */
+    private $transform_functions_registry;
+
+    /**
+     * Initialize all dependencies required.
+     *
+     * @param Mappings_Validator $validator A {@link Mappings_Validator} instance.
+     * @param Mappings_Transform_Functions_Registry $transform_functions_registry
+     */
+    public function __construct( $validator, $transform_functions_registry ) {
+
+        $this->validator                    = $validator;
+        $this->transform_functions_registry = $transform_functions_registry;
+
+        // Hook to refactor the JSON-LD.
+        add_filter( 'wl_post_jsonld_array', array( $this, 'wl_post_jsonld_array' ), 11, 2 );
+        add_filter( 'wl_entity_jsonld_array', array( $this, 'wl_post_jsonld_array' ), 11, 3 );
+
+    }
+
+    /**
+     * Hook to `wl_post_jsonld_array` and `wl_entity_jsonld_array`.
+     *
+     * Receive the JSON-LD and the references in the array along with the post ID and transform them according to
+     * the configuration.
+     *
+     * @param array $value {
+     *      The array containing the JSON-LD and the references.
+     *
+     * @type array $jsonld The JSON-LD array.
+     * @type int[] $references An array of post ID referenced by the JSON-LD (will be expanded by the converter).
+     * }
+     *
+     * @param int $post_id The post ID.
+     *
+     * @return array An array with the updated JSON-LD and references.
+     */
+    public function wl_post_jsonld_array( $value, $post_id ) {
+
+        $jsonld     = $value['jsonld'];
+        $references = $value['references'];
+
+        return array(
+            'jsonld'     => $this->wl_post_jsonld( $jsonld, $post_id, $references ),
+            'references' => $references,
+        );
+    }
+
+    /**
+     * Returns JSON-LD data after applying transformation functions.
+     *
+     * @param array $jsonld The JSON-LD structure.
+     * @param int $post_id The {@link WP_Post} id.
+     * @param array $references An array of post references.
+     *
+     * @return array the new refactored array structure.
+     * @since 3.25.0
+     */
+    private function wl_post_jsonld( $jsonld, $post_id, &$references ) {
+
+        // @@todo I think there's an issue here with the Validator, because you're changing the instance state and the
+        // instance may be reused afterwards.
+
+        $properties        = $this->validator->validate( $post_id );
+        $nested_properties = array();
+
+        foreach ( $properties as $property ) {
+            // If the property has the character '/' in the property name then it is a nested property.
+            if ( strpos( $property['property_name'], '/' ) !== false ) {
+                $nested_properties[] = $property;
+                continue;
+            }
+            $property_transformed_data = $this->get_property_data( $property, $jsonld, $post_id, $references );
+            if ( false !== $property_transformed_data ) {
+                $jsonld[ $property['property_name'] ] = $property_transformed_data;
+            }
+        }
+
+        $jsonld = $this->process_nested_properties( $nested_properties, $jsonld, $post_id, $references );
+
+        return $jsonld;
+    }
+
+    /**
+     * Get the property data by applying the transformation function
+     *
+     * @param $property
+     * @param $jsonld
+     * @param $post_id
+     * @param $references
+     *
+     * @return array|bool|null
+     */
+    public function get_property_data( $property, $jsonld, $post_id, &$references ) {
+        $transform_instance = $this->transform_functions_registry->get_transform_function( $property['transform_function'] );
+        $data               = $this->get_data_from_data_source( $post_id, $property );
+        if ( null !== $transform_instance ) {
+            $transform_data = $transform_instance->transform_data( $data, $jsonld, $references, $post_id );
+            if ( null !== $transform_data ) {
+                return $this->make_single( $transform_data );
+            }
+        } else {
+            return $this->make_single( $data );
+        }
+
+        return false;
+    }
+
+    /**
+     * Process all the nested properties.
+     *
+     * @param $nested_properties array
+     * @param $jsonld array
+     *
+     * @return array
+     */
+    public function process_nested_properties( $nested_properties, $jsonld, $post_id, &$references ) {
+        foreach ( $nested_properties as $property ) {
+            $property_data = $this->get_property_data( $property, $jsonld, $post_id, $references );
+            if ( false === $property_data ) {
+                // No need to create nested levels.
+                continue;
+            }
+
+            $keys = explode( '/', $property['property_name'] );
+            // end is the last level of the nested property.
+            $end                      = array_pop( $keys );
+            $current_property_pointer = &$jsonld;
+
+            /**
+             * Once we find all the nested levels from the property name
+             * loop through it and create associative array if the levels
+             * didnt exist.
+             */
+            while ( count( $keys ) > 0 ) {
+                $key = array_shift( $keys );
+                if ( $key === "" ) {
+                    continue;
+                }
+                if ( ! array_key_exists( $key, $current_property_pointer ) ) {
+                    $current_property_pointer[ $key ] = array();
+                }
+                // We are setting the pointer to the current key, so that at the end
+                // we can add the data at last level.
+                $current_property_pointer = &$current_property_pointer[ $key ];
+            }
+            $current_property_pointer[ $end ] = $property_data;
+        }
+
+        return $jsonld;
+    }
+
+
+    /**
+     * Returns data from data source.
+     *
+     * @param int $post_id Id of the post.
+     * @param array $property_data The property data for the post_id.
+     *
+     * @return array Returns key, value array, if the value is not found, then it returns null.
+     */
+    final public function get_data_from_data_source( $post_id, $property_data ) {
+        $value = $property_data['field_name'];
+
+        // Do 1 to 1 mapping and return result.
+        switch ( $property_data['field_type'] ) {
+            case self::FIELD_TYPE_ACF:
+                if ( ! function_exists( 'get_field' ) || ! function_exists( 'get_field_object' ) ) {
+                    return array();
+                }
+
+                return $this->get_data_for_acf_field( $property_data['field_name'], $post_id );
+
+            case self::FIELD_TYPE_CUSTOM_FIELD:
+
+                return array_map( 'wp_strip_all_tags', get_post_meta( $post_id, $value ) );
+
+            default:
+                return $value;
+        }
+
+    }
+
+    /**
+     * Gets data from acf, format the data if it is a repeater field.
+     *
+     * @param $field_name
+     * @param $post_id
+     *
+     * @return array|mixed
+     */
+    public function get_data_for_acf_field( $field_name, $post_id ) {
+        $field_data = get_field_object( $field_name, $post_id );
+        $data       = get_field( $field_name, $post_id );
+
+        // only process if it is a repeater field, else return the data.
+        if ( is_array( $field_data ) && array_key_exists( 'type', $field_data )
+             && $field_data['type'] === 'repeater' ) {
+            // check if we have only one sub field, currently we only support one subfield.
+            if ( is_array( $data ) && count( $data ) > 0 && count( array_keys( $data[0] ) === 1 ) ) {
+                $repeater_formatted_data = array();
+                foreach ( $data as $item ) {
+                    $repeater_formatted_data = array_merge( $repeater_formatted_data, array_values( $item ) );
+                }
+                // Remove non unique values.
+                $repeater_formatted_data = array_unique( $repeater_formatted_data );
+                // Remove empty values
+                $repeater_formatted_data = array_filter( $repeater_formatted_data, 'strlen' );
+
+                // re-index all the values.
+                return array_values( $repeater_formatted_data );
+            }
+        }
+
+        // Return normal acf data if it is not a repeater field.
+        return $data;
+    }
+
+    private function make_single( $value ) {
+
+        $values = (array) $value;
+
+        if ( empty( $values ) ) {
+            return null;
+        }
+
+        if ( 1 === count( $values ) && 0 === key( $values ) ) {
+            return current( $values );
+        }
+
+        return $values;
+    }
 
 }

Spacing +54 added lines, -54 removed lines

@@ -47,14 +47,14 @@ 
 	 * @param Mappings_Validator $validator A {@link Mappings_Validator} instance.
 	 * @param Mappings_Transform_Functions_Registry $transform_functions_registry
 	 */
-	public function __construct( $validator, $transform_functions_registry ) {
+	public function __construct($validator, $transform_functions_registry) {
 
 		$this->validator                    = $validator;
 		$this->transform_functions_registry = $transform_functions_registry;
 
 		// Hook to refactor the JSON-LD.
-		add_filter( 'wl_post_jsonld_array', array( $this, 'wl_post_jsonld_array' ), 11, 2 );
-		add_filter( 'wl_entity_jsonld_array', array( $this, 'wl_post_jsonld_array' ), 11, 3 );
+		add_filter('wl_post_jsonld_array', array($this, 'wl_post_jsonld_array'), 11, 2);
+		add_filter('wl_entity_jsonld_array', array($this, 'wl_post_jsonld_array'), 11, 3);
 
 	}
 
@@ -75,13 +75,13 @@ 
 	 *
 	 * @return array An array with the updated JSON-LD and references.
 	 */
-	public function wl_post_jsonld_array( $value, $post_id ) {
+	public function wl_post_jsonld_array($value, $post_id) {
 
 		$jsonld     = $value['jsonld'];
 		$references = $value['references'];
 
 		return array(
-			'jsonld'     => $this->wl_post_jsonld( $jsonld, $post_id, $references ),
+			'jsonld'     => $this->wl_post_jsonld($jsonld, $post_id, $references),
 			'references' => $references,
 		);
 	}
@@ -96,27 +96,27 @@ 
 	 * @return array the new refactored array structure.
 	 * @since 3.25.0
 	 */
-	private function wl_post_jsonld( $jsonld, $post_id, &$references ) {
+	private function wl_post_jsonld($jsonld, $post_id, &$references) {
 
 		// @@todo I think there's an issue here with the Validator, because you're changing the instance state and the
 		// instance may be reused afterwards.
 
-		$properties        = $this->validator->validate( $post_id );
+		$properties        = $this->validator->validate($post_id);
 		$nested_properties = array();
 
-		foreach ( $properties as $property ) {
+		foreach ($properties as $property) {
 			// If the property has the character '/' in the property name then it is a nested property.
-			if ( strpos( $property['property_name'], '/' ) !== false ) {
+			if (strpos($property['property_name'], '/') !== false) {
 				$nested_properties[] = $property;
 				continue;
 			}
-			$property_transformed_data = $this->get_property_data( $property, $jsonld, $post_id, $references );
-			if ( false !== $property_transformed_data ) {
-				$jsonld[ $property['property_name'] ] = $property_transformed_data;
+			$property_transformed_data = $this->get_property_data($property, $jsonld, $post_id, $references);
+			if (false !== $property_transformed_data) {
+				$jsonld[$property['property_name']] = $property_transformed_data;
 			}
 		}
 
-		$jsonld = $this->process_nested_properties( $nested_properties, $jsonld, $post_id, $references );
+		$jsonld = $this->process_nested_properties($nested_properties, $jsonld, $post_id, $references);
 
 		return $jsonld;
 	}
@@ -131,16 +131,16 @@ 
 	 *
 	 * @return array|bool|null
 	 */
-	public function get_property_data( $property, $jsonld, $post_id, &$references ) {
-		$transform_instance = $this->transform_functions_registry->get_transform_function( $property['transform_function'] );
-		$data               = $this->get_data_from_data_source( $post_id, $property );
-		if ( null !== $transform_instance ) {
-			$transform_data = $transform_instance->transform_data( $data, $jsonld, $references, $post_id );
-			if ( null !== $transform_data ) {
-				return $this->make_single( $transform_data );
+	public function get_property_data($property, $jsonld, $post_id, &$references) {
+		$transform_instance = $this->transform_functions_registry->get_transform_function($property['transform_function']);
+		$data               = $this->get_data_from_data_source($post_id, $property);
+		if (null !== $transform_instance) {
+			$transform_data = $transform_instance->transform_data($data, $jsonld, $references, $post_id);
+			if (null !== $transform_data) {
+				return $this->make_single($transform_data);
 			}
 		} else {
-			return $this->make_single( $data );
+			return $this->make_single($data);
 		}
 
 		return false;
@@ -154,17 +154,17 @@ 
 	 *
 	 * @return array
 	 */
-	public function process_nested_properties( $nested_properties, $jsonld, $post_id, &$references ) {
-		foreach ( $nested_properties as $property ) {
-			$property_data = $this->get_property_data( $property, $jsonld, $post_id, $references );
-			if ( false === $property_data ) {
+	public function process_nested_properties($nested_properties, $jsonld, $post_id, &$references) {
+		foreach ($nested_properties as $property) {
+			$property_data = $this->get_property_data($property, $jsonld, $post_id, $references);
+			if (false === $property_data) {
 				// No need to create nested levels.
 				continue;
 			}
 
-			$keys = explode( '/', $property['property_name'] );
+			$keys = explode('/', $property['property_name']);
 			// end is the last level of the nested property.
-			$end                      = array_pop( $keys );
+			$end                      = array_pop($keys);
 			$current_property_pointer = &$jsonld;
 
 			/**
@@ -172,19 +172,19 @@ 
 			 * loop through it and create associative array if the levels
 			 * didnt exist.
 			 */
-			while ( count( $keys ) > 0 ) {
-				$key = array_shift( $keys );
-				if ( $key === "" ) {
+			while (count($keys) > 0) {
+				$key = array_shift($keys);
+				if ($key === "") {
 					continue;
 				}
-				if ( ! array_key_exists( $key, $current_property_pointer ) ) {
-					$current_property_pointer[ $key ] = array();
+				if ( ! array_key_exists($key, $current_property_pointer)) {
+					$current_property_pointer[$key] = array();
 				}
 				// We are setting the pointer to the current key, so that at the end
 				// we can add the data at last level.
-				$current_property_pointer = &$current_property_pointer[ $key ];
+				$current_property_pointer = &$current_property_pointer[$key];
 			}
-			$current_property_pointer[ $end ] = $property_data;
+			$current_property_pointer[$end] = $property_data;
 		}
 
 		return $jsonld;
@@ -199,21 +199,21 @@ 
 	 *
 	 * @return array Returns key, value array, if the value is not found, then it returns null.
 	 */
-	final public function get_data_from_data_source( $post_id, $property_data ) {
+	final public function get_data_from_data_source($post_id, $property_data) {
 		$value = $property_data['field_name'];
 
 		// Do 1 to 1 mapping and return result.
-		switch ( $property_data['field_type'] ) {
+		switch ($property_data['field_type']) {
 			case self::FIELD_TYPE_ACF:
-				if ( ! function_exists( 'get_field' ) || ! function_exists( 'get_field_object' ) ) {
+				if ( ! function_exists('get_field') || ! function_exists('get_field_object')) {
 					return array();
 				}
 
-				return $this->get_data_for_acf_field( $property_data['field_name'], $post_id );
+				return $this->get_data_for_acf_field($property_data['field_name'], $post_id);
 
 			case self::FIELD_TYPE_CUSTOM_FIELD:
 
-				return array_map( 'wp_strip_all_tags', get_post_meta( $post_id, $value ) );
+				return array_map('wp_strip_all_tags', get_post_meta($post_id, $value));
 
 			default:
 				return $value;
@@ -229,26 +229,26 @@ 
 	 *
 	 * @return array|mixed
 	 */
-	public function get_data_for_acf_field( $field_name, $post_id ) {
-		$field_data = get_field_object( $field_name, $post_id );
-		$data       = get_field( $field_name, $post_id );
+	public function get_data_for_acf_field($field_name, $post_id) {
+		$field_data = get_field_object($field_name, $post_id);
+		$data       = get_field($field_name, $post_id);
 
 		// only process if it is a repeater field, else return the data.
-		if ( is_array( $field_data ) && array_key_exists( 'type', $field_data )
-		     && $field_data['type'] === 'repeater' ) {
+		if (is_array($field_data) && array_key_exists('type', $field_data)
+		     && $field_data['type'] === 'repeater') {
 			// check if we have only one sub field, currently we only support one subfield.
-			if ( is_array( $data ) && count( $data ) > 0 && count( array_keys( $data[0] ) === 1 ) ) {
+			if (is_array($data) && count($data) > 0 && count(array_keys($data[0]) === 1)) {
 				$repeater_formatted_data = array();
-				foreach ( $data as $item ) {
-					$repeater_formatted_data = array_merge( $repeater_formatted_data, array_values( $item ) );
+				foreach ($data as $item) {
+					$repeater_formatted_data = array_merge($repeater_formatted_data, array_values($item));
 				}
 				// Remove non unique values.
-				$repeater_formatted_data = array_unique( $repeater_formatted_data );
+				$repeater_formatted_data = array_unique($repeater_formatted_data);
 				// Remove empty values
-				$repeater_formatted_data = array_filter( $repeater_formatted_data, 'strlen' );
+				$repeater_formatted_data = array_filter($repeater_formatted_data, 'strlen');
 
 				// re-index all the values.
-				return array_values( $repeater_formatted_data );
+				return array_values($repeater_formatted_data);
 			}
 		}
 
@@ -256,16 +256,16 @@ 
 		return $data;
 	}
 
-	private function make_single( $value ) {
+	private function make_single($value) {
 
 		$values = (array) $value;
 
-		if ( empty( $values ) ) {
+		if (empty($values)) {
 			return null;
 		}
 
-		if ( 1 === count( $values ) && 0 === key( $values ) ) {
-			return current( $values );
+		if (1 === count($values) && 0 === key($values)) {
+			return current($values);
 		}
 
 		return $values;

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

Indentation +582 added lines, -582 removed lines

@@ -16,506 +16,506 @@ 
  */
 class Wordlift_Entity_Service {
 
-	/**
-	 * The Log service.
-	 *
-	 * @since  3.2.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log The Log service.
-	 */
-	private $log;
-
-	/**
-	 * The UI service.
-	 *
-	 * @since  3.2.0
-	 * @access private
-	 * @var \Wordlift_UI_Service $ui_service The UI service.
-	 */
-	private $ui_service;
-
-	/**
-	 * The {@link Wordlift_Relation_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Relation_Service $relation_service The {@link Wordlift_Relation_Service} instance.
-	 */
-	private $relation_service;
-
-	/**
-	 * The {@link Wordlift_Entity_Uri_Service} instance.
-	 *
-	 * @since  3.16.3
-	 * @access private
-	 * @var \Wordlift_Entity_Uri_Service $entity_uri_service The {@link Wordlift_Entity_Uri_Service} instance.
-	 */
-	private $entity_uri_service;
-
-	/**
-	 * The entity post type name.
-	 *
-	 * @since 3.1.0
-	 */
-	const TYPE_NAME = 'entity';
-
-	/**
-	 * The alternative label meta key.
-	 *
-	 * @since 3.2.0
-	 */
-	const ALTERNATIVE_LABEL_META_KEY = '_wl_alt_label';
-
-	/**
-	 * The alternative label input template.
-	 *
-	 * @since 3.2.0
-	 */
-	// TODO: this should be moved to a class that deals with HTML code.
-	const ALTERNATIVE_LABEL_INPUT_TEMPLATE = '<div class="wl-alternative-label">
+    /**
+     * The Log service.
+     *
+     * @since  3.2.0
+     * @access private
+     * @var \Wordlift_Log_Service $log The Log service.
+     */
+    private $log;
+
+    /**
+     * The UI service.
+     *
+     * @since  3.2.0
+     * @access private
+     * @var \Wordlift_UI_Service $ui_service The UI service.
+     */
+    private $ui_service;
+
+    /**
+     * The {@link Wordlift_Relation_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Relation_Service $relation_service The {@link Wordlift_Relation_Service} instance.
+     */
+    private $relation_service;
+
+    /**
+     * The {@link Wordlift_Entity_Uri_Service} instance.
+     *
+     * @since  3.16.3
+     * @access private
+     * @var \Wordlift_Entity_Uri_Service $entity_uri_service The {@link Wordlift_Entity_Uri_Service} instance.
+     */
+    private $entity_uri_service;
+
+    /**
+     * The entity post type name.
+     *
+     * @since 3.1.0
+     */
+    const TYPE_NAME = 'entity';
+
+    /**
+     * The alternative label meta key.
+     *
+     * @since 3.2.0
+     */
+    const ALTERNATIVE_LABEL_META_KEY = '_wl_alt_label';
+
+    /**
+     * The alternative label input template.
+     *
+     * @since 3.2.0
+     */
+    // TODO: this should be moved to a class that deals with HTML code.
+    const ALTERNATIVE_LABEL_INPUT_TEMPLATE = '<div class="wl-alternative-label">
                 <label class="screen-reader-text" id="wl-alternative-label-prompt-text" for="wl-alternative-label">Enter alternative label here</label>
                 <input name="wl_alternative_label[]" size="30" value="%s" id="wl-alternative-label" type="text">
                 <button class="button wl-delete-button">%s</button>
                 </div>';
 
-	/**
-	 * A singleton instance of the Entity service.
-	 *
-	 * @since  3.2.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $instance A singleton instance of the Entity service.
-	 */
-	private static $instance;
-
-	/**
-	 * Create a Wordlift_Entity_Service instance.
-	 *
-	 * @param \Wordlift_UI_Service $ui_service The UI service.
-	 * @param \Wordlift_Relation_Service $relation_service The {@link Wordlift_Relation_Service} instance.
-	 * @param \Wordlift_Entity_Uri_Service $entity_uri_service The {@link Wordlift_Entity_Uri_Service} instance.
-	 *
-	 * @since 3.2.0
-	 *
-	 */
-	public function __construct( $ui_service, $relation_service, $entity_uri_service ) {
-
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Entity_Service' );
-
-		$this->ui_service         = $ui_service;
-		$this->relation_service   = $relation_service;
-		$this->entity_uri_service = $entity_uri_service;
-
-		// Set the singleton instance.
-		self::$instance = $this;
-	}
-
-	/**
-	 * Get the singleton instance of the Entity service.
-	 *
-	 * @return \Wordlift_Entity_Service The singleton instance of the Entity service.
-	 * @since 3.2.0
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Determines whether a post is an entity or not. Entity is in this context
-	 * something which is not an article.
-	 *
-	 * @param int $post_id A post id.
-	 *
-	 * @return bool Return true if the post is an entity otherwise false.
-	 * @since 3.1.0
-	 *
-	 */
-	public function is_entity( $post_id ) {
-
-		$terms = wp_get_object_terms( $post_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
-
-		if ( is_wp_error( $terms ) ) {
-			$this->log->error( "Cannot get the terms for post $post_id: " . $terms->get_error_message() );
-
-			return false;
-		}
-
-		if ( empty( $terms ) ) {
-			return false;
-		}
-
-		/*
+    /**
+     * A singleton instance of the Entity service.
+     *
+     * @since  3.2.0
+     * @access private
+     * @var \Wordlift_Entity_Service $instance A singleton instance of the Entity service.
+     */
+    private static $instance;
+
+    /**
+     * Create a Wordlift_Entity_Service instance.
+     *
+     * @param \Wordlift_UI_Service $ui_service The UI service.
+     * @param \Wordlift_Relation_Service $relation_service The {@link Wordlift_Relation_Service} instance.
+     * @param \Wordlift_Entity_Uri_Service $entity_uri_service The {@link Wordlift_Entity_Uri_Service} instance.
+     *
+     * @since 3.2.0
+     *
+     */
+    public function __construct( $ui_service, $relation_service, $entity_uri_service ) {
+
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Entity_Service' );
+
+        $this->ui_service         = $ui_service;
+        $this->relation_service   = $relation_service;
+        $this->entity_uri_service = $entity_uri_service;
+
+        // Set the singleton instance.
+        self::$instance = $this;
+    }
+
+    /**
+     * Get the singleton instance of the Entity service.
+     *
+     * @return \Wordlift_Entity_Service The singleton instance of the Entity service.
+     * @since 3.2.0
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Determines whether a post is an entity or not. Entity is in this context
+     * something which is not an article.
+     *
+     * @param int $post_id A post id.
+     *
+     * @return bool Return true if the post is an entity otherwise false.
+     * @since 3.1.0
+     *
+     */
+    public function is_entity( $post_id ) {
+
+        $terms = wp_get_object_terms( $post_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
+
+        if ( is_wp_error( $terms ) ) {
+            $this->log->error( "Cannot get the terms for post $post_id: " . $terms->get_error_message() );
+
+            return false;
+        }
+
+        if ( empty( $terms ) ) {
+            return false;
+        }
+
+        /*
 		 * We don't consider an `article` to be an entity.
 		 *
 		 * @since 3.20.0 At least one associated mustn't be an `article`.
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/835
 		 */
-		foreach ( $terms as $term ) {
-			if ( 'article' !== $term->slug ) {
-				return true;
-			}
-		}
-
-		return false;
-	}
-
-	/**
-	 * Get the proper classification scope for a given entity post
-	 *
-	 * @param integer $post_id An entity post id.
-	 *
-	 * @param string $default The default classification scope, `what` if not
-	 *                         provided.
-	 *
-	 * @return string Returns a classification scope (e.g. 'what').
-	 * @since 3.5.0
-	 *
-	 */
-	public function get_classification_scope_for( $post_id, $default = WL_WHAT_RELATION ) {
-
-		if ( false === $this->is_entity( $post_id ) ) {
-			return $default;
-		}
-
-		// Retrieve the entity type
-		$entity_type_arr = Wordlift_Entity_Type_Service::get_instance()->get( $post_id );
-		$entity_type     = str_replace( 'wl-', '', $entity_type_arr['css_class'] );
-		// Retrieve classification boxes configuration
-		$classification_boxes = unserialize( WL_CORE_POST_CLASSIFICATION_BOXES );
-		foreach ( $classification_boxes as $cb ) {
-			if ( in_array( $entity_type, $cb['registeredTypes'] ) ) {
-				return $cb['id'];
-			}
-		}
-
-		return $default;
-	}
-
-	/**
-	 * Check whether a {@link WP_Post} is used.
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return bool|null Null if it's not an entity, otherwise true if it's used.
-	 */
-	public function is_used( $post_id ) {
-
-		if ( false === $this->is_entity( $post_id ) ) {
-			return null;
-		}
-		// Retrieve the post
-		$entity = get_post( $post_id );
-
-		global $wpdb;
-		// Retrieve Wordlift relation instances table name
-		$table_name = wl_core_get_relation_instances_table_name();
-
-		// Check is it's referenced / related to another post / entity
-		$stmt = $wpdb->prepare(
-			"SELECT COUNT(*) FROM $table_name WHERE  object_id = %d",
-			$entity->ID
-		);
-
-		// Perform the query
-		$relation_instances = (int) $wpdb->get_var( $stmt );
-		// If there is at least one relation instance for the current entity, then it's used
-		if ( 0 < $relation_instances ) {
-			return true;
-		}
-
-		// Check if the entity uri is used as meta_value
-		$stmt = $wpdb->prepare(
-			"SELECT COUNT(*) FROM $wpdb->postmeta WHERE post_id != %d AND meta_value = %s",
-			$entity->ID,
-			wl_get_entity_uri( $entity->ID )
-		);
-		// Perform the query
-		$meta_instances = (int) $wpdb->get_var( $stmt );
-
-		// If there is at least one meta that refers the current entity uri, then current entity is used
-		if ( 0 < $meta_instances ) {
-			return true;
-		}
-
-		// If we are here, it means the current entity is not used at the moment
-		return false;
-	}
-
-	/**
-	 * Find entity posts by the entity URI. Entity as searched by their entity URI or same as.
-	 *
-	 * @param string $uri The entity URI.
-	 *
-	 * @return WP_Post|null A WP_Post instance or null if not found.
-	 * @deprecated in favor of Wordlift_Entity_Uri_Service->get_entity( $uri );
-	 *
-	 * @since      3.16.3 deprecated in favor of Wordlift_Entity_Uri_Service->get_entity( $uri );
-	 * @since      3.2.0
-	 *
-	 */
-	public function get_entity_post_by_uri( $uri ) {
-
-		return $this->entity_uri_service->get_entity( $uri );
-	}
-
-	/**
-	 * Fires once a post has been saved. This function uses the $_REQUEST, therefore
-	 * we check that the post we're saving is the current post.
-	 *
-	 * @see   https://github.com/insideout10/wordlift-plugin/issues/363
-	 *
-	 * @since 3.2.0
-	 *
-	 * @param int $post_id Post ID.
-	 * @param WP_Post $post Post object.
-	 * @param bool $update Whether this is an existing post being updated or not.
-	 */
-	public function save_post( $post_id, $post, $update ) {
-
-		// Avoid doing anything if post is autosave or a revision.
-		if ( wp_is_post_autosave( $post ) || wp_is_post_revision( $post ) ) {
-			return;
-		}
-
-		// We're setting the alternative label that have been provided via the UI
-		// (in fact we're using $_REQUEST), while save_post may be also called
-		// programmatically by some other function: we need to check therefore if
-		// the $post_id in the save_post call matches the post id set in the request.
-		//
-		// If this is not the current post being saved or if it's not an entity, return.
-		if ( ! isset( $_REQUEST['post_ID'] ) || $_REQUEST['post_ID'] != $post_id || ! $this->is_entity( $post_id ) ) {
-			return;
-		}
-
-		// Get the alt labels from the request (or empty array).
-		$alt_labels = isset( $_REQUEST['wl_alternative_label'] ) ? $_REQUEST['wl_alternative_label'] : array();
-
-		if ( ( ! empty( $_POST['content'] ) && ! empty( $_POST['post_content'] ) ) || isset( $_REQUEST['wl_alternative_label'] ) ) {
-			// This is via classic editor, so set the alternative labels.
-			$this->set_alternative_labels( $post_id, $alt_labels );
-		}
-
-
-	}
-
-	/**
-	 * Set the alternative labels.
-	 *
-	 * @param int $post_id The post id.
-	 * @param array $alt_labels An array of labels.
-	 *
-	 * @since 3.2.0
-	 *
-	 */
-	public function set_alternative_labels( $post_id, $alt_labels ) {
-
-		// Bail out if post id is not numeric. We add this check as we found a WP install that was sending a WP_Error
-		// instead of post id.
-		if ( ! is_numeric( $post_id ) ) {
-			return;
-		}
-
-		// Force $alt_labels to be an array
-		if ( ! is_array( $alt_labels ) ) {
-			$alt_labels = array( $alt_labels );
-		}
-
-		$this->log->debug( "Setting alternative labels [ post id :: $post_id ][ alt labels :: " . implode( ',', $alt_labels ) . " ]" );
-
-		// Delete all the existing alternate labels.
-		delete_post_meta( $post_id, self::ALTERNATIVE_LABEL_META_KEY );
-
-		// Set the alternative labels.
-		foreach ( $alt_labels as $alt_label ) {
-			if ( ! empty( $alt_label ) ) {
-				add_post_meta( $post_id, self::ALTERNATIVE_LABEL_META_KEY, $alt_label );
-			}
-		}
-
-	}
-
-	/**
-	 * Retrieve the alternate labels.
-	 *
-	 * @param int $post_id Post id.
-	 *
-	 * @return mixed An array  of alternative labels.
-	 * @since 3.2.0
-	 *
-	 */
-	public function get_alternative_labels( $post_id ) {
-
-		return get_post_meta( $post_id, self::ALTERNATIVE_LABEL_META_KEY );
-	}
-
-	/**
-	 * Retrieve the labels for an entity, i.e. the title + the synonyms.
-	 *
-	 * @param int $post_id The entity {@link WP_Post} id.
-	 *
-	 * @return array An array with the entity title and labels.
-	 * @since 3.12.0
-	 *
-	 */
-	public function get_labels( $post_id ) {
-
-		return array_merge( (array) get_the_title( $post_id ), $this->get_alternative_labels( $post_id ) );
-	}
-
-	/**
-	 * Fires before the permalink field in the edit form (this event is available in WP from 4.1.0).
-	 *
-	 * @param WP_Post $post Post object.
-	 *
-	 * @since 3.2.0
-	 *
-	 */
-	public function edit_form_before_permalink( $post ) {
-
-		// If it's not an entity, return.
-		if ( ! $this->is_entity( $post->ID ) ) {
-			return;
-		}
-
-		// Print the input template.
-		$this->ui_service->print_template( 'wl-tmpl-alternative-label-input', $this->get_alternative_label_input() );
-
-		// Print all the currently set alternative labels.
-		foreach ( $this->get_alternative_labels( $post->ID ) as $alt_label ) {
-
-			echo $this->get_alternative_label_input( $alt_label );
-
-		};
-
-		// Print the button.
-		$this->ui_service->print_button( 'wl-add-alternative-labels-button', __( 'Add more titles', 'wordlift' ) );
-
-	}
-
-	/**
-	 * Get the URI for the entity with the specified post id.
-	 *
-	 * @param int $post_id The entity post id.
-	 *
-	 * @return null|string The entity URI or NULL if not found or the dataset URI is not configured.
-	 * @since 3.6.0
-	 *
-	 */
-	public function get_uri( $post_id ) {
-
-		// If a null is given, nothing to do
-		if ( is_null( $post_id ) ) {
-			return null;
-		}
-
-		$dataset_uri = wl_configuration_get_redlink_dataset_uri();
-
-		// If the dataset uri is not properly configured, null is returned
-		if ( empty( $dataset_uri ) ) {
-			return null;
-		}
-
-		$uri = get_post_meta( $post_id, WL_ENTITY_URL_META_NAME, true );
-
-		/*
+        foreach ( $terms as $term ) {
+            if ( 'article' !== $term->slug ) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
+    /**
+     * Get the proper classification scope for a given entity post
+     *
+     * @param integer $post_id An entity post id.
+     *
+     * @param string $default The default classification scope, `what` if not
+     *                         provided.
+     *
+     * @return string Returns a classification scope (e.g. 'what').
+     * @since 3.5.0
+     *
+     */
+    public function get_classification_scope_for( $post_id, $default = WL_WHAT_RELATION ) {
+
+        if ( false === $this->is_entity( $post_id ) ) {
+            return $default;
+        }
+
+        // Retrieve the entity type
+        $entity_type_arr = Wordlift_Entity_Type_Service::get_instance()->get( $post_id );
+        $entity_type     = str_replace( 'wl-', '', $entity_type_arr['css_class'] );
+        // Retrieve classification boxes configuration
+        $classification_boxes = unserialize( WL_CORE_POST_CLASSIFICATION_BOXES );
+        foreach ( $classification_boxes as $cb ) {
+            if ( in_array( $entity_type, $cb['registeredTypes'] ) ) {
+                return $cb['id'];
+            }
+        }
+
+        return $default;
+    }
+
+    /**
+     * Check whether a {@link WP_Post} is used.
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return bool|null Null if it's not an entity, otherwise true if it's used.
+     */
+    public function is_used( $post_id ) {
+
+        if ( false === $this->is_entity( $post_id ) ) {
+            return null;
+        }
+        // Retrieve the post
+        $entity = get_post( $post_id );
+
+        global $wpdb;
+        // Retrieve Wordlift relation instances table name
+        $table_name = wl_core_get_relation_instances_table_name();
+
+        // Check is it's referenced / related to another post / entity
+        $stmt = $wpdb->prepare(
+            "SELECT COUNT(*) FROM $table_name WHERE  object_id = %d",
+            $entity->ID
+        );
+
+        // Perform the query
+        $relation_instances = (int) $wpdb->get_var( $stmt );
+        // If there is at least one relation instance for the current entity, then it's used
+        if ( 0 < $relation_instances ) {
+            return true;
+        }
+
+        // Check if the entity uri is used as meta_value
+        $stmt = $wpdb->prepare(
+            "SELECT COUNT(*) FROM $wpdb->postmeta WHERE post_id != %d AND meta_value = %s",
+            $entity->ID,
+            wl_get_entity_uri( $entity->ID )
+        );
+        // Perform the query
+        $meta_instances = (int) $wpdb->get_var( $stmt );
+
+        // If there is at least one meta that refers the current entity uri, then current entity is used
+        if ( 0 < $meta_instances ) {
+            return true;
+        }
+
+        // If we are here, it means the current entity is not used at the moment
+        return false;
+    }
+
+    /**
+     * Find entity posts by the entity URI. Entity as searched by their entity URI or same as.
+     *
+     * @param string $uri The entity URI.
+     *
+     * @return WP_Post|null A WP_Post instance or null if not found.
+     * @deprecated in favor of Wordlift_Entity_Uri_Service->get_entity( $uri );
+     *
+     * @since      3.16.3 deprecated in favor of Wordlift_Entity_Uri_Service->get_entity( $uri );
+     * @since      3.2.0
+     *
+     */
+    public function get_entity_post_by_uri( $uri ) {
+
+        return $this->entity_uri_service->get_entity( $uri );
+    }
+
+    /**
+     * Fires once a post has been saved. This function uses the $_REQUEST, therefore
+     * we check that the post we're saving is the current post.
+     *
+     * @see   https://github.com/insideout10/wordlift-plugin/issues/363
+     *
+     * @since 3.2.0
+     *
+     * @param int $post_id Post ID.
+     * @param WP_Post $post Post object.
+     * @param bool $update Whether this is an existing post being updated or not.
+     */
+    public function save_post( $post_id, $post, $update ) {
+
+        // Avoid doing anything if post is autosave or a revision.
+        if ( wp_is_post_autosave( $post ) || wp_is_post_revision( $post ) ) {
+            return;
+        }
+
+        // We're setting the alternative label that have been provided via the UI
+        // (in fact we're using $_REQUEST), while save_post may be also called
+        // programmatically by some other function: we need to check therefore if
+        // the $post_id in the save_post call matches the post id set in the request.
+        //
+        // If this is not the current post being saved or if it's not an entity, return.
+        if ( ! isset( $_REQUEST['post_ID'] ) || $_REQUEST['post_ID'] != $post_id || ! $this->is_entity( $post_id ) ) {
+            return;
+        }
+
+        // Get the alt labels from the request (or empty array).
+        $alt_labels = isset( $_REQUEST['wl_alternative_label'] ) ? $_REQUEST['wl_alternative_label'] : array();
+
+        if ( ( ! empty( $_POST['content'] ) && ! empty( $_POST['post_content'] ) ) || isset( $_REQUEST['wl_alternative_label'] ) ) {
+            // This is via classic editor, so set the alternative labels.
+            $this->set_alternative_labels( $post_id, $alt_labels );
+        }
+
+
+    }
+
+    /**
+     * Set the alternative labels.
+     *
+     * @param int $post_id The post id.
+     * @param array $alt_labels An array of labels.
+     *
+     * @since 3.2.0
+     *
+     */
+    public function set_alternative_labels( $post_id, $alt_labels ) {
+
+        // Bail out if post id is not numeric. We add this check as we found a WP install that was sending a WP_Error
+        // instead of post id.
+        if ( ! is_numeric( $post_id ) ) {
+            return;
+        }
+
+        // Force $alt_labels to be an array
+        if ( ! is_array( $alt_labels ) ) {
+            $alt_labels = array( $alt_labels );
+        }
+
+        $this->log->debug( "Setting alternative labels [ post id :: $post_id ][ alt labels :: " . implode( ',', $alt_labels ) . " ]" );
+
+        // Delete all the existing alternate labels.
+        delete_post_meta( $post_id, self::ALTERNATIVE_LABEL_META_KEY );
+
+        // Set the alternative labels.
+        foreach ( $alt_labels as $alt_label ) {
+            if ( ! empty( $alt_label ) ) {
+                add_post_meta( $post_id, self::ALTERNATIVE_LABEL_META_KEY, $alt_label );
+            }
+        }
+
+    }
+
+    /**
+     * Retrieve the alternate labels.
+     *
+     * @param int $post_id Post id.
+     *
+     * @return mixed An array  of alternative labels.
+     * @since 3.2.0
+     *
+     */
+    public function get_alternative_labels( $post_id ) {
+
+        return get_post_meta( $post_id, self::ALTERNATIVE_LABEL_META_KEY );
+    }
+
+    /**
+     * Retrieve the labels for an entity, i.e. the title + the synonyms.
+     *
+     * @param int $post_id The entity {@link WP_Post} id.
+     *
+     * @return array An array with the entity title and labels.
+     * @since 3.12.0
+     *
+     */
+    public function get_labels( $post_id ) {
+
+        return array_merge( (array) get_the_title( $post_id ), $this->get_alternative_labels( $post_id ) );
+    }
+
+    /**
+     * Fires before the permalink field in the edit form (this event is available in WP from 4.1.0).
+     *
+     * @param WP_Post $post Post object.
+     *
+     * @since 3.2.0
+     *
+     */
+    public function edit_form_before_permalink( $post ) {
+
+        // If it's not an entity, return.
+        if ( ! $this->is_entity( $post->ID ) ) {
+            return;
+        }
+
+        // Print the input template.
+        $this->ui_service->print_template( 'wl-tmpl-alternative-label-input', $this->get_alternative_label_input() );
+
+        // Print all the currently set alternative labels.
+        foreach ( $this->get_alternative_labels( $post->ID ) as $alt_label ) {
+
+            echo $this->get_alternative_label_input( $alt_label );
+
+        };
+
+        // Print the button.
+        $this->ui_service->print_button( 'wl-add-alternative-labels-button', __( 'Add more titles', 'wordlift' ) );
+
+    }
+
+    /**
+     * Get the URI for the entity with the specified post id.
+     *
+     * @param int $post_id The entity post id.
+     *
+     * @return null|string The entity URI or NULL if not found or the dataset URI is not configured.
+     * @since 3.6.0
+     *
+     */
+    public function get_uri( $post_id ) {
+
+        // If a null is given, nothing to do
+        if ( is_null( $post_id ) ) {
+            return null;
+        }
+
+        $dataset_uri = wl_configuration_get_redlink_dataset_uri();
+
+        // If the dataset uri is not properly configured, null is returned
+        if ( empty( $dataset_uri ) ) {
+            return null;
+        }
+
+        $uri = get_post_meta( $post_id, WL_ENTITY_URL_META_NAME, true );
+
+        /*
 		 * Consider the URI invalid if it doesn't start with the dataset URI.
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/996
 		 */
-		if ( 0 !== strpos( $uri, $dataset_uri ) ) {
-			$uri = null;
-		}
-
-		// Set the URI if it isn't set yet.
-		$post_status = get_post_status( $post_id );
-		if ( empty( $uri ) && 'auto-draft' !== $post_status && 'revision' !== $post_status ) {
-			$uri = wl_build_entity_uri( $post_id );
-			wl_set_entity_uri( $post_id, $uri );
-		}
-
-		return $uri;
-	}
-
-
-	/**
-	 * Get the alternative label input HTML code.
-	 *
-	 * @param string $value The input value.
-	 *
-	 * @return string The input HTML code.
-	 * @since 3.2.0
-	 *
-	 */
-	private function get_alternative_label_input( $value = '' ) {
-
-		return sprintf( self::ALTERNATIVE_LABEL_INPUT_TEMPLATE, esc_attr( $value ), __( 'Delete', 'wordlift' ) );
-	}
-
-	/**
-	 * Get the number of entity posts published in this blog.
-	 *
-	 * @return int The number of published entity posts.
-	 * @since 3.6.0
-	 *
-	 */
-	public function count() {
-		global $wpdb;
-
-		// Try to get the count from the transient.
-		$count = get_transient( '_wl_entity_service__count' );
-		if ( false !== $count ) {
-			return $count;
-		}
-
-		// Query the count.
-		$count = $wpdb->get_var( $wpdb->prepare(
-			"SELECT COUNT( DISTINCT( tr.object_id ) )"
-			. " FROM {$wpdb->term_relationships} tr"
-			. " INNER JOIN {$wpdb->term_taxonomy} tt"
-			. "  ON tt.taxonomy = %s AND tt.term_taxonomy_id = tr.term_taxonomy_id"
-			. " INNER JOIN {$wpdb->terms} t"
-			. "  ON t.term_id = tt.term_id AND t.name != %s",
-			Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
-			'article'
-		) );
-
-		// Store the count in cache.
-		set_transient( '_wl_entity_service__count', $count, 900 );
-
-		return $count;
-	}
-
-	/**
-	 * Add the entity filtering criterias to the arguments for a `get_posts`
-	 * call.
-	 *
-	 * @param array $args The arguments for a `get_posts` call.
-	 *
-	 * @return array The arguments for a `get_posts` call.
-	 * @since 3.15.0
-	 *
-	 */
-	public static function add_criterias( $args ) {
-
-		// Build an optimal tax-query.
-		$tax_query = array(
-			'relation' => 'AND',
-			array(
-				'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
-				'operator' => 'EXISTS',
-			),
-			array(
-				'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
-				'field'    => 'slug',
-				'terms'    => 'article',
-				'operator' => 'NOT IN',
-			),
-		);
-
-		return $args + array(
-				'post_type' => Wordlift_Entity_Service::valid_entity_post_types(),
-				/*
+        if ( 0 !== strpos( $uri, $dataset_uri ) ) {
+            $uri = null;
+        }
+
+        // Set the URI if it isn't set yet.
+        $post_status = get_post_status( $post_id );
+        if ( empty( $uri ) && 'auto-draft' !== $post_status && 'revision' !== $post_status ) {
+            $uri = wl_build_entity_uri( $post_id );
+            wl_set_entity_uri( $post_id, $uri );
+        }
+
+        return $uri;
+    }
+
+
+    /**
+     * Get the alternative label input HTML code.
+     *
+     * @param string $value The input value.
+     *
+     * @return string The input HTML code.
+     * @since 3.2.0
+     *
+     */
+    private function get_alternative_label_input( $value = '' ) {
+
+        return sprintf( self::ALTERNATIVE_LABEL_INPUT_TEMPLATE, esc_attr( $value ), __( 'Delete', 'wordlift' ) );
+    }
+
+    /**
+     * Get the number of entity posts published in this blog.
+     *
+     * @return int The number of published entity posts.
+     * @since 3.6.0
+     *
+     */
+    public function count() {
+        global $wpdb;
+
+        // Try to get the count from the transient.
+        $count = get_transient( '_wl_entity_service__count' );
+        if ( false !== $count ) {
+            return $count;
+        }
+
+        // Query the count.
+        $count = $wpdb->get_var( $wpdb->prepare(
+            "SELECT COUNT( DISTINCT( tr.object_id ) )"
+            . " FROM {$wpdb->term_relationships} tr"
+            . " INNER JOIN {$wpdb->term_taxonomy} tt"
+            . "  ON tt.taxonomy = %s AND tt.term_taxonomy_id = tr.term_taxonomy_id"
+            . " INNER JOIN {$wpdb->terms} t"
+            . "  ON t.term_id = tt.term_id AND t.name != %s",
+            Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
+            'article'
+        ) );
+
+        // Store the count in cache.
+        set_transient( '_wl_entity_service__count', $count, 900 );
+
+        return $count;
+    }
+
+    /**
+     * Add the entity filtering criterias to the arguments for a `get_posts`
+     * call.
+     *
+     * @param array $args The arguments for a `get_posts` call.
+     *
+     * @return array The arguments for a `get_posts` call.
+     * @since 3.15.0
+     *
+     */
+    public static function add_criterias( $args ) {
+
+        // Build an optimal tax-query.
+        $tax_query = array(
+            'relation' => 'AND',
+            array(
+                'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
+                'operator' => 'EXISTS',
+            ),
+            array(
+                'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
+                'field'    => 'slug',
+                'terms'    => 'article',
+                'operator' => 'NOT IN',
+            ),
+        );
+
+        return $args + array(
+                'post_type' => Wordlift_Entity_Service::valid_entity_post_types(),
+                /*
 				 * Ensure compatibility with Polylang.
 				 *
 				 * @see https://github.com/insideout10/wordlift-plugin/issues/855.
@@ -523,102 +523,102 @@ 
 				 *
 				 * @since 3.19.5
 				 */
-				'lang'      => '',
-				'tax_query' => $tax_query,
-			);
-	}
-
-	/**
-	 * Create a new entity.
-	 *
-	 * @param string $name The entity name.
-	 * @param string $type_uri The entity's type URI.
-	 * @param null $logo The entity logo id (or NULL if none).
-	 * @param string $status The post status, by default 'publish'.
-	 *
-	 * @return int|WP_Error The entity post id or a {@link WP_Error} in case the `wp_insert_post` call fails.
-	 * @since 3.9.0
-	 *
-	 */
-	public function create( $name, $type_uri, $logo = null, $status = 'publish' ) {
-
-		// Create an entity for the publisher.
-		$post_id = @wp_insert_post( array(
-			'post_type'    => self::TYPE_NAME,
-			'post_title'   => $name,
-			'post_status'  => $status,
-			'post_content' => '',
-		) );
-
-		// Return the error if any.
-		if ( is_wp_error( $post_id ) ) {
-			return $post_id;
-		}
-
-		// Set the entity logo.
-		if ( $logo && is_numeric( $logo ) ) {
-			set_post_thumbnail( $post_id, $logo );
-		}
-
-		// Set the entity type.
-		Wordlift_Entity_Type_Service::get_instance()->set( $post_id, $type_uri );
-
-		return $post_id;
-	}
-
-	/**
-	 * Get the entities related to the one with the specified id. By default only
-	 * published entities will be returned.
-	 *
-	 * @param int $id The post id.
-	 * @param string $post_status The target post status (default = publish).
-	 *
-	 * @return array An array of post ids.
-	 * @since 3.10.0
-	 *
-	 */
-	public function get_related_entities( $id, $post_status = 'publish' ) {
-
-		return $this->relation_service->get_objects( $id, 'ids', null, $post_status );
-	}
-
-	/**
-	 * Get the list of entities.
-	 *
-	 * @param array $params Custom parameters for WordPress' own {@link get_posts} function.
-	 *
-	 * @return array An array of entity posts.
-	 * @since 3.12.2
-	 *
-	 */
-	public function get( $params = array() ) {
-
-		// Set the defaults.
-		$defaults = array( 'post_type' => 'entity' );
-
-		// Merge the defaults with the provided parameters.
-		$args = wp_parse_args( $params, $defaults );
-
-		// Call the `get_posts` function.
-		return get_posts( $args );
-	}
-
-	/**
-	 * The list of post type names which can be used for entities
-	 *
-	 * Criteria is that the post type is public. The list of valid post types
-	 * can be overridden with a filter.
-	 *
-	 * @return array Array containing the names of the valid post types.
-	 * @since 3.15.0
-	 *
-	 */
-	static function valid_entity_post_types() {
-
-		// Ignore builtins in the call to avoid getting attachments.
-		$post_types = array( 'post', 'page', self::TYPE_NAME, 'product' );
-
-		return apply_filters( 'wl_valid_entity_post_types', $post_types );
-	}
+                'lang'      => '',
+                'tax_query' => $tax_query,
+            );
+    }
+
+    /**
+     * Create a new entity.
+     *
+     * @param string $name The entity name.
+     * @param string $type_uri The entity's type URI.
+     * @param null $logo The entity logo id (or NULL if none).
+     * @param string $status The post status, by default 'publish'.
+     *
+     * @return int|WP_Error The entity post id or a {@link WP_Error} in case the `wp_insert_post` call fails.
+     * @since 3.9.0
+     *
+     */
+    public function create( $name, $type_uri, $logo = null, $status = 'publish' ) {
+
+        // Create an entity for the publisher.
+        $post_id = @wp_insert_post( array(
+            'post_type'    => self::TYPE_NAME,
+            'post_title'   => $name,
+            'post_status'  => $status,
+            'post_content' => '',
+        ) );
+
+        // Return the error if any.
+        if ( is_wp_error( $post_id ) ) {
+            return $post_id;
+        }
+
+        // Set the entity logo.
+        if ( $logo && is_numeric( $logo ) ) {
+            set_post_thumbnail( $post_id, $logo );
+        }
+
+        // Set the entity type.
+        Wordlift_Entity_Type_Service::get_instance()->set( $post_id, $type_uri );
+
+        return $post_id;
+    }
+
+    /**
+     * Get the entities related to the one with the specified id. By default only
+     * published entities will be returned.
+     *
+     * @param int $id The post id.
+     * @param string $post_status The target post status (default = publish).
+     *
+     * @return array An array of post ids.
+     * @since 3.10.0
+     *
+     */
+    public function get_related_entities( $id, $post_status = 'publish' ) {
+
+        return $this->relation_service->get_objects( $id, 'ids', null, $post_status );
+    }
+
+    /**
+     * Get the list of entities.
+     *
+     * @param array $params Custom parameters for WordPress' own {@link get_posts} function.
+     *
+     * @return array An array of entity posts.
+     * @since 3.12.2
+     *
+     */
+    public function get( $params = array() ) {
+
+        // Set the defaults.
+        $defaults = array( 'post_type' => 'entity' );
+
+        // Merge the defaults with the provided parameters.
+        $args = wp_parse_args( $params, $defaults );
+
+        // Call the `get_posts` function.
+        return get_posts( $args );
+    }
+
+    /**
+     * The list of post type names which can be used for entities
+     *
+     * Criteria is that the post type is public. The list of valid post types
+     * can be overridden with a filter.
+     *
+     * @return array Array containing the names of the valid post types.
+     * @since 3.15.0
+     *
+     */
+    static function valid_entity_post_types() {
+
+        // Ignore builtins in the call to avoid getting attachments.
+        $post_types = array( 'post', 'page', self::TYPE_NAME, 'product' );
+
+        return apply_filters( 'wl_valid_entity_post_types', $post_types );
+    }
 
 }

Spacing +83 added lines, -83 removed lines

@@ -97,9 +97,9 @@ 
 	 * @since 3.2.0
 	 *
 	 */
-	public function __construct( $ui_service, $relation_service, $entity_uri_service ) {
+	public function __construct($ui_service, $relation_service, $entity_uri_service) {
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Entity_Service' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Entity_Service');
 
 		$this->ui_service         = $ui_service;
 		$this->relation_service   = $relation_service;
@@ -130,17 +130,17 @@ 
 	 * @since 3.1.0
 	 *
 	 */
-	public function is_entity( $post_id ) {
+	public function is_entity($post_id) {
 
-		$terms = wp_get_object_terms( $post_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
+		$terms = wp_get_object_terms($post_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME);
 
-		if ( is_wp_error( $terms ) ) {
-			$this->log->error( "Cannot get the terms for post $post_id: " . $terms->get_error_message() );
+		if (is_wp_error($terms)) {
+			$this->log->error("Cannot get the terms for post $post_id: ".$terms->get_error_message());
 
 			return false;
 		}
 
-		if ( empty( $terms ) ) {
+		if (empty($terms)) {
 			return false;
 		}
 
@@ -151,8 +151,8 @@ 
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/835
 		 */
-		foreach ( $terms as $term ) {
-			if ( 'article' !== $term->slug ) {
+		foreach ($terms as $term) {
+			if ('article' !== $term->slug) {
 				return true;
 			}
 		}
@@ -172,19 +172,19 @@ 
 	 * @since 3.5.0
 	 *
 	 */
-	public function get_classification_scope_for( $post_id, $default = WL_WHAT_RELATION ) {
+	public function get_classification_scope_for($post_id, $default = WL_WHAT_RELATION) {
 
-		if ( false === $this->is_entity( $post_id ) ) {
+		if (false === $this->is_entity($post_id)) {
 			return $default;
 		}
 
 		// Retrieve the entity type
-		$entity_type_arr = Wordlift_Entity_Type_Service::get_instance()->get( $post_id );
-		$entity_type     = str_replace( 'wl-', '', $entity_type_arr['css_class'] );
+		$entity_type_arr = Wordlift_Entity_Type_Service::get_instance()->get($post_id);
+		$entity_type     = str_replace('wl-', '', $entity_type_arr['css_class']);
 		// Retrieve classification boxes configuration
-		$classification_boxes = unserialize( WL_CORE_POST_CLASSIFICATION_BOXES );
-		foreach ( $classification_boxes as $cb ) {
-			if ( in_array( $entity_type, $cb['registeredTypes'] ) ) {
+		$classification_boxes = unserialize(WL_CORE_POST_CLASSIFICATION_BOXES);
+		foreach ($classification_boxes as $cb) {
+			if (in_array($entity_type, $cb['registeredTypes'])) {
 				return $cb['id'];
 			}
 		}
@@ -199,13 +199,13 @@ 
 	 *
 	 * @return bool|null Null if it's not an entity, otherwise true if it's used.
 	 */
-	public function is_used( $post_id ) {
+	public function is_used($post_id) {
 
-		if ( false === $this->is_entity( $post_id ) ) {
+		if (false === $this->is_entity($post_id)) {
 			return null;
 		}
 		// Retrieve the post
-		$entity = get_post( $post_id );
+		$entity = get_post($post_id);
 
 		global $wpdb;
 		// Retrieve Wordlift relation instances table name
@@ -218,9 +218,9 @@ 
 		);
 
 		// Perform the query
-		$relation_instances = (int) $wpdb->get_var( $stmt );
+		$relation_instances = (int) $wpdb->get_var($stmt);
 		// If there is at least one relation instance for the current entity, then it's used
-		if ( 0 < $relation_instances ) {
+		if (0 < $relation_instances) {
 			return true;
 		}
 
@@ -228,13 +228,13 @@ 
 		$stmt = $wpdb->prepare(
 			"SELECT COUNT(*) FROM $wpdb->postmeta WHERE post_id != %d AND meta_value = %s",
 			$entity->ID,
-			wl_get_entity_uri( $entity->ID )
+			wl_get_entity_uri($entity->ID)
 		);
 		// Perform the query
-		$meta_instances = (int) $wpdb->get_var( $stmt );
+		$meta_instances = (int) $wpdb->get_var($stmt);
 
 		// If there is at least one meta that refers the current entity uri, then current entity is used
-		if ( 0 < $meta_instances ) {
+		if (0 < $meta_instances) {
 			return true;
 		}
 
@@ -254,9 +254,9 @@ 
 	 * @since      3.2.0
 	 *
 	 */
-	public function get_entity_post_by_uri( $uri ) {
+	public function get_entity_post_by_uri($uri) {
 
-		return $this->entity_uri_service->get_entity( $uri );
+		return $this->entity_uri_service->get_entity($uri);
 	}
 
 	/**
@@ -271,10 +271,10 @@ 
 	 * @param WP_Post $post Post object.
 	 * @param bool $update Whether this is an existing post being updated or not.
 	 */
-	public function save_post( $post_id, $post, $update ) {
+	public function save_post($post_id, $post, $update) {
 
 		// Avoid doing anything if post is autosave or a revision.
-		if ( wp_is_post_autosave( $post ) || wp_is_post_revision( $post ) ) {
+		if (wp_is_post_autosave($post) || wp_is_post_revision($post)) {
 			return;
 		}
 
@@ -284,16 +284,16 @@ 
 		// the $post_id in the save_post call matches the post id set in the request.
 		//
 		// If this is not the current post being saved or if it's not an entity, return.
-		if ( ! isset( $_REQUEST['post_ID'] ) || $_REQUEST['post_ID'] != $post_id || ! $this->is_entity( $post_id ) ) {
+		if ( ! isset($_REQUEST['post_ID']) || $_REQUEST['post_ID'] != $post_id || ! $this->is_entity($post_id)) {
 			return;
 		}
 
 		// Get the alt labels from the request (or empty array).
-		$alt_labels = isset( $_REQUEST['wl_alternative_label'] ) ? $_REQUEST['wl_alternative_label'] : array();
+		$alt_labels = isset($_REQUEST['wl_alternative_label']) ? $_REQUEST['wl_alternative_label'] : array();
 
-		if ( ( ! empty( $_POST['content'] ) && ! empty( $_POST['post_content'] ) ) || isset( $_REQUEST['wl_alternative_label'] ) ) {
+		if (( ! empty($_POST['content']) && ! empty($_POST['post_content'])) || isset($_REQUEST['wl_alternative_label'])) {
 			// This is via classic editor, so set the alternative labels.
-			$this->set_alternative_labels( $post_id, $alt_labels );
+			$this->set_alternative_labels($post_id, $alt_labels);
 		}
 
 
@@ -308,28 +308,28 @@ 
 	 * @since 3.2.0
 	 *
 	 */
-	public function set_alternative_labels( $post_id, $alt_labels ) {
+	public function set_alternative_labels($post_id, $alt_labels) {
 
 		// Bail out if post id is not numeric. We add this check as we found a WP install that was sending a WP_Error
 		// instead of post id.
-		if ( ! is_numeric( $post_id ) ) {
+		if ( ! is_numeric($post_id)) {
 			return;
 		}
 
 		// Force $alt_labels to be an array
-		if ( ! is_array( $alt_labels ) ) {
-			$alt_labels = array( $alt_labels );
+		if ( ! is_array($alt_labels)) {
+			$alt_labels = array($alt_labels);
 		}
 
-		$this->log->debug( "Setting alternative labels [ post id :: $post_id ][ alt labels :: " . implode( ',', $alt_labels ) . " ]" );
+		$this->log->debug("Setting alternative labels [ post id :: $post_id ][ alt labels :: ".implode(',', $alt_labels)." ]");
 
 		// Delete all the existing alternate labels.
-		delete_post_meta( $post_id, self::ALTERNATIVE_LABEL_META_KEY );
+		delete_post_meta($post_id, self::ALTERNATIVE_LABEL_META_KEY);
 
 		// Set the alternative labels.
-		foreach ( $alt_labels as $alt_label ) {
-			if ( ! empty( $alt_label ) ) {
-				add_post_meta( $post_id, self::ALTERNATIVE_LABEL_META_KEY, $alt_label );
+		foreach ($alt_labels as $alt_label) {
+			if ( ! empty($alt_label)) {
+				add_post_meta($post_id, self::ALTERNATIVE_LABEL_META_KEY, $alt_label);
 			}
 		}
 
@@ -344,9 +344,9 @@ 
 	 * @since 3.2.0
 	 *
 	 */
-	public function get_alternative_labels( $post_id ) {
+	public function get_alternative_labels($post_id) {
 
-		return get_post_meta( $post_id, self::ALTERNATIVE_LABEL_META_KEY );
+		return get_post_meta($post_id, self::ALTERNATIVE_LABEL_META_KEY);
 	}
 
 	/**
@@ -358,9 +358,9 @@ 
 	 * @since 3.12.0
 	 *
 	 */
-	public function get_labels( $post_id ) {
+	public function get_labels($post_id) {
 
-		return array_merge( (array) get_the_title( $post_id ), $this->get_alternative_labels( $post_id ) );
+		return array_merge((array) get_the_title($post_id), $this->get_alternative_labels($post_id));
 	}
 
 	/**
@@ -371,25 +371,25 @@ 
 	 * @since 3.2.0
 	 *
 	 */
-	public function edit_form_before_permalink( $post ) {
+	public function edit_form_before_permalink($post) {
 
 		// If it's not an entity, return.
-		if ( ! $this->is_entity( $post->ID ) ) {
+		if ( ! $this->is_entity($post->ID)) {
 			return;
 		}
 
 		// Print the input template.
-		$this->ui_service->print_template( 'wl-tmpl-alternative-label-input', $this->get_alternative_label_input() );
+		$this->ui_service->print_template('wl-tmpl-alternative-label-input', $this->get_alternative_label_input());
 
 		// Print all the currently set alternative labels.
-		foreach ( $this->get_alternative_labels( $post->ID ) as $alt_label ) {
+		foreach ($this->get_alternative_labels($post->ID) as $alt_label) {
 
-			echo $this->get_alternative_label_input( $alt_label );
+			echo $this->get_alternative_label_input($alt_label);
 
 		};
 
 		// Print the button.
-		$this->ui_service->print_button( 'wl-add-alternative-labels-button', __( 'Add more titles', 'wordlift' ) );
+		$this->ui_service->print_button('wl-add-alternative-labels-button', __('Add more titles', 'wordlift'));
 
 	}
 
@@ -402,36 +402,36 @@ 
 	 * @since 3.6.0
 	 *
 	 */
-	public function get_uri( $post_id ) {
+	public function get_uri($post_id) {
 
 		// If a null is given, nothing to do
-		if ( is_null( $post_id ) ) {
+		if (is_null($post_id)) {
 			return null;
 		}
 
 		$dataset_uri = wl_configuration_get_redlink_dataset_uri();
 
 		// If the dataset uri is not properly configured, null is returned
-		if ( empty( $dataset_uri ) ) {
+		if (empty($dataset_uri)) {
 			return null;
 		}
 
-		$uri = get_post_meta( $post_id, WL_ENTITY_URL_META_NAME, true );
+		$uri = get_post_meta($post_id, WL_ENTITY_URL_META_NAME, true);
 
 		/*
 		 * Consider the URI invalid if it doesn't start with the dataset URI.
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/996
 		 */
-		if ( 0 !== strpos( $uri, $dataset_uri ) ) {
+		if (0 !== strpos($uri, $dataset_uri)) {
 			$uri = null;
 		}
 
 		// Set the URI if it isn't set yet.
-		$post_status = get_post_status( $post_id );
-		if ( empty( $uri ) && 'auto-draft' !== $post_status && 'revision' !== $post_status ) {
-			$uri = wl_build_entity_uri( $post_id );
-			wl_set_entity_uri( $post_id, $uri );
+		$post_status = get_post_status($post_id);
+		if (empty($uri) && 'auto-draft' !== $post_status && 'revision' !== $post_status) {
+			$uri = wl_build_entity_uri($post_id);
+			wl_set_entity_uri($post_id, $uri);
 		}
 
 		return $uri;
@@ -447,9 +447,9 @@ 
 	 * @since 3.2.0
 	 *
 	 */
-	private function get_alternative_label_input( $value = '' ) {
+	private function get_alternative_label_input($value = '') {
 
-		return sprintf( self::ALTERNATIVE_LABEL_INPUT_TEMPLATE, esc_attr( $value ), __( 'Delete', 'wordlift' ) );
+		return sprintf(self::ALTERNATIVE_LABEL_INPUT_TEMPLATE, esc_attr($value), __('Delete', 'wordlift'));
 	}
 
 	/**
@@ -463,13 +463,13 @@ 
 		global $wpdb;
 
 		// Try to get the count from the transient.
-		$count = get_transient( '_wl_entity_service__count' );
-		if ( false !== $count ) {
+		$count = get_transient('_wl_entity_service__count');
+		if (false !== $count) {
 			return $count;
 		}
 
 		// Query the count.
-		$count = $wpdb->get_var( $wpdb->prepare(
+		$count = $wpdb->get_var($wpdb->prepare(
 			"SELECT COUNT( DISTINCT( tr.object_id ) )"
 			. " FROM {$wpdb->term_relationships} tr"
 			. " INNER JOIN {$wpdb->term_taxonomy} tt"
@@ -478,10 +478,10 @@ 
 			. "  ON t.term_id = tt.term_id AND t.name != %s",
 			Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
 			'article'
-		) );
+		));
 
 		// Store the count in cache.
-		set_transient( '_wl_entity_service__count', $count, 900 );
+		set_transient('_wl_entity_service__count', $count, 900);
 
 		return $count;
 	}
@@ -496,7 +496,7 @@ 
 	 * @since 3.15.0
 	 *
 	 */
-	public static function add_criterias( $args ) {
+	public static function add_criterias($args) {
 
 		// Build an optimal tax-query.
 		$tax_query = array(
@@ -540,28 +540,28 @@ 
 	 * @since 3.9.0
 	 *
 	 */
-	public function create( $name, $type_uri, $logo = null, $status = 'publish' ) {
+	public function create($name, $type_uri, $logo = null, $status = 'publish') {
 
 		// Create an entity for the publisher.
-		$post_id = @wp_insert_post( array(
+		$post_id = @wp_insert_post(array(
 			'post_type'    => self::TYPE_NAME,
 			'post_title'   => $name,
 			'post_status'  => $status,
 			'post_content' => '',
-		) );
+		));
 
 		// Return the error if any.
-		if ( is_wp_error( $post_id ) ) {
+		if (is_wp_error($post_id)) {
 			return $post_id;
 		}
 
 		// Set the entity logo.
-		if ( $logo && is_numeric( $logo ) ) {
-			set_post_thumbnail( $post_id, $logo );
+		if ($logo && is_numeric($logo)) {
+			set_post_thumbnail($post_id, $logo);
 		}
 
 		// Set the entity type.
-		Wordlift_Entity_Type_Service::get_instance()->set( $post_id, $type_uri );
+		Wordlift_Entity_Type_Service::get_instance()->set($post_id, $type_uri);
 
 		return $post_id;
 	}
@@ -577,9 +577,9 @@ 
 	 * @since 3.10.0
 	 *
 	 */
-	public function get_related_entities( $id, $post_status = 'publish' ) {
+	public function get_related_entities($id, $post_status = 'publish') {
 
-		return $this->relation_service->get_objects( $id, 'ids', null, $post_status );
+		return $this->relation_service->get_objects($id, 'ids', null, $post_status);
 	}
 
 	/**
@@ -591,16 +591,16 @@ 
 	 * @since 3.12.2
 	 *
 	 */
-	public function get( $params = array() ) {
+	public function get($params = array()) {
 
 		// Set the defaults.
-		$defaults = array( 'post_type' => 'entity' );
+		$defaults = array('post_type' => 'entity');
 
 		// Merge the defaults with the provided parameters.
-		$args = wp_parse_args( $params, $defaults );
+		$args = wp_parse_args($params, $defaults);
 
 		// Call the `get_posts` function.
-		return get_posts( $args );
+		return get_posts($args);
 	}
 
 	/**
@@ -616,9 +616,9 @@ 
 	static function valid_entity_post_types() {
 
 		// Ignore builtins in the call to avoid getting attachments.
-		$post_types = array( 'post', 'page', self::TYPE_NAME, 'product' );
+		$post_types = array('post', 'page', self::TYPE_NAME, 'product');
 
-		return apply_filters( 'wl_valid_entity_post_types', $post_types );
+		return apply_filters('wl_valid_entity_post_types', $post_types);
 	}
 
 }

src/wordlift/mappings/class-acf-mappings.php

Indentation +77 added lines, -77 removed lines

@@ -15,82 +15,82 @@
  */
 class Acf_Mappings {
 
-	/**
-	 * Acf_Mappings constructor.
-	 *
-	 * Hooks `wl_mappings_field_types` to our own function to declare support for ACF fields.
-	 */
-	public function __construct() {
-
-		$that = $this;
-		add_action( 'plugins_loaded', function () use ( $that ) {
-			$that->add_acf_option_to_mappings_ui();
-		} );
-
-	}
-
-	private function add_acf_option_to_mappings_ui() {
-		// Bail out if ACF is not available.
-		if ( ! function_exists( 'acf_get_field_groups' ) ) {
-			return array();
-		}
-
-		add_filter( 'wl_mappings_field_types', array( $this, 'wl_mappings_field_types', ) );
-	}
-
-	/**
-	 * Hook to `wl_mappings_field_types` to declare support for ACF fields.
-	 *
-	 * @param array $field_types An array of field types.
-	 *
-	 * @return array The array with our ACF declaration.
-	 */
-	public function wl_mappings_field_types( $field_types ) {
-
-		$field_types[] = array(
-			'field_type' => 'acf',
-			'label'      => __( 'ACF', 'wordlift' ),
-			'value'      => self::get_acf_options(),
-		);
-
-		return $field_types;
-	}
-
-	/**
-	 * Returns array of acf options.
-	 *
-	 * The array is composed by all the ACF groups and their fields hierarchy.
-	 *
-	 * @return array ACF options array.
-	 */
-	private static function get_acf_options() {
-
-		$acf_options = array();
-
-		// Get all the ACF field groups.
-		$field_groups = acf_get_field_groups();
-
-		foreach ( $field_groups as $field_group ) {
-			$group_name = $field_group['title'];
-			$group_key  = $field_group['key'];
-
-			$group_fields = acf_get_fields( $group_key );
-
-			$group_options = array();
-			foreach ( $group_fields as $group_field ) {
-				$group_options[] = array(
-					'label' => $group_field['label'],
-					'value' => $group_field['key'],
-				);
-			}
-
-			$acf_options[] = array(
-				'group_name'    => $group_name,
-				'group_options' => $group_options,
-			);
-		}
-
-		return $acf_options;
-	}
+    /**
+     * Acf_Mappings constructor.
+     *
+     * Hooks `wl_mappings_field_types` to our own function to declare support for ACF fields.
+     */
+    public function __construct() {
+
+        $that = $this;
+        add_action( 'plugins_loaded', function () use ( $that ) {
+            $that->add_acf_option_to_mappings_ui();
+        } );
+
+    }
+
+    private function add_acf_option_to_mappings_ui() {
+        // Bail out if ACF is not available.
+        if ( ! function_exists( 'acf_get_field_groups' ) ) {
+            return array();
+        }
+
+        add_filter( 'wl_mappings_field_types', array( $this, 'wl_mappings_field_types', ) );
+    }
+
+    /**
+     * Hook to `wl_mappings_field_types` to declare support for ACF fields.
+     *
+     * @param array $field_types An array of field types.
+     *
+     * @return array The array with our ACF declaration.
+     */
+    public function wl_mappings_field_types( $field_types ) {
+
+        $field_types[] = array(
+            'field_type' => 'acf',
+            'label'      => __( 'ACF', 'wordlift' ),
+            'value'      => self::get_acf_options(),
+        );
+
+        return $field_types;
+    }
+
+    /**
+     * Returns array of acf options.
+     *
+     * The array is composed by all the ACF groups and their fields hierarchy.
+     *
+     * @return array ACF options array.
+     */
+    private static function get_acf_options() {
+
+        $acf_options = array();
+
+        // Get all the ACF field groups.
+        $field_groups = acf_get_field_groups();
+
+        foreach ( $field_groups as $field_group ) {
+            $group_name = $field_group['title'];
+            $group_key  = $field_group['key'];
+
+            $group_fields = acf_get_fields( $group_key );
+
+            $group_options = array();
+            foreach ( $group_fields as $group_field ) {
+                $group_options[] = array(
+                    'label' => $group_field['label'],
+                    'value' => $group_field['key'],
+                );
+            }
+
+            $acf_options[] = array(
+                'group_name'    => $group_name,
+                'group_options' => $group_options,
+            );
+        }
+
+        return $acf_options;
+    }
 
 }

Spacing +8 added lines, -8 removed lines

@@ -23,7 +23,7 @@ 
 	public function __construct() {
 
 		$that = $this;
-		add_action( 'plugins_loaded', function () use ( $that ) {
+		add_action('plugins_loaded', function() use ($that) {
 			$that->add_acf_option_to_mappings_ui();
 		} );
 
@@ -31,11 +31,11 @@ 
 
 	private function add_acf_option_to_mappings_ui() {
 		// Bail out if ACF is not available.
-		if ( ! function_exists( 'acf_get_field_groups' ) ) {
+		if ( ! function_exists('acf_get_field_groups')) {
 			return array();
 		}
 
-		add_filter( 'wl_mappings_field_types', array( $this, 'wl_mappings_field_types', ) );
+		add_filter('wl_mappings_field_types', array($this, 'wl_mappings_field_types',));
 	}
 
 	/**
@@ -45,11 +45,11 @@ 
 	 *
 	 * @return array The array with our ACF declaration.
 	 */
-	public function wl_mappings_field_types( $field_types ) {
+	public function wl_mappings_field_types($field_types) {
 
 		$field_types[] = array(
 			'field_type' => 'acf',
-			'label'      => __( 'ACF', 'wordlift' ),
+			'label'      => __('ACF', 'wordlift'),
 			'value'      => self::get_acf_options(),
 		);
 
@@ -70,14 +70,14 @@ 
 		// Get all the ACF field groups.
 		$field_groups = acf_get_field_groups();
 
-		foreach ( $field_groups as $field_group ) {
+		foreach ($field_groups as $field_group) {
 			$group_name = $field_group['title'];
 			$group_key  = $field_group['key'];
 
-			$group_fields = acf_get_fields( $group_key );
+			$group_fields = acf_get_fields($group_key);
 
 			$group_options = array();
-			foreach ( $group_fields as $group_field ) {
+			foreach ($group_fields as $group_field) {
 				$group_options[] = array(
 					'label' => $group_field['label'],
 					'value' => $group_field['key'],

src/wordlift/jsonld/class-jsonld-endpoint.php

Indentation +135 added lines, -135 removed lines

@@ -20,147 +20,147 @@ 
  */
 class Jsonld_Endpoint {
 
-	/**
-	 * The {@link Wordlift_Jsonld_Service} instance.
-	 *
-	 * @var Wordlift_Jsonld_Service The {@link Wordlift_Jsonld_Service} instance.
-	 */
-	private $jsonld_service;
-	/**
-	 * @var \Wordlift_Entity_Uri_Service
-	 */
-	private $entity_uri_service;
-
-	/**
-	 * Jsonld_Endpoint constructor.
-	 *
-	 * @param \Wordlift_Jsonld_Service $jsonld_service
-	 * @param \Wordlift_Entity_Uri_Service $entity_uri_service
-	 */
-	public function __construct( $jsonld_service, $entity_uri_service ) {
-
-		$this->jsonld_service     = $jsonld_service;
-		$this->entity_uri_service = $entity_uri_service;
-
-		// PHP 5.3 compatibility.
-		$that = $this;
-		add_action( 'rest_api_init', function () use ( $that ) {
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<id>\d+)', array(
-				'methods'  => WP_REST_Server::READABLE,
-				'callback' => array( $that, 'jsonld_using_post_id' ),
-				'args'     => array(
-					'id' => array(
-						'validate_callback' => function ( $param, $request, $key ) {
-							return is_numeric( $param );
-						},
-						'sanitize_callback' => 'absint',
-					),
-				)
-			) );
-
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/http/(?P<item_id>.*)', array(
-				'methods'  => 'GET',
-				'callback' => array( $that, 'jsonld_using_item_id' ),
-			) );
-
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/post-meta/(?P<meta_key>[^/]+)', array(
-				'methods'  => 'GET',
-				'callback' => array( $that, 'jsonld_using_post_meta' ),
-			) );
-
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<post_type>.*)/(?P<post_name>.*)', array(
-				'methods'  => 'GET',
-				'callback' => array( $that, 'jsonld_using_get_page_by_path' ),
-			) );
-
-		} );
-
-	}
-
-	/**
-	 * Callback for the JSON-LD request.
-	 *
-	 * @param array $request {
-	 *  The request array.
-	 *
-	 * @type int $id The post id.
-	 * }
-	 *
-	 * @return WP_REST_Response
-	 * @throws \Exception
-	 */
-	public function jsonld_using_post_id( $request ) {
-
-		$post_id     = $request['id'];
-		$is_homepage = ( 0 === $post_id );
-
-		// Send the generated JSON-LD.
-		$data = $this->jsonld_service->get_jsonld( $is_homepage, $post_id );
-
-		return Jsonld_Response_Helper::to_response( $data );
-	}
-
-	/**
-	 * Provide a JSON-LD given the itemId.
-	 *
-	 * @param array $request {
-	 *  The request array.
-	 *
-	 * @type string $item_id The entity item id.
-	 * }
-	 *
-	 * @return WP_REST_Response
-	 * @throws \Exception
-	 */
-	public function jsonld_using_item_id( $request ) {
-
-		$item_id = 'http://' . $request['item_id'];
-		$post    = $this->entity_uri_service->get_entity( $item_id );
-
-		if ( ! is_a( $post, 'WP_Post' ) ) {
-			return new WP_REST_Response( esc_html( "$item_id not found." ), 404, array( 'Content-Type' => 'text/html' ) );
-		}
-
-		return $this->jsonld_using_post_id( array( 'id' => $post->ID, ) );
-	}
-
-	public function jsonld_using_get_page_by_path( $request ) {
-
-		$post_name = $request['post_name'];
-		$post_type = $request['post_type'];
-
-		global $wpdb;
-
-		$sql = "
+    /**
+     * The {@link Wordlift_Jsonld_Service} instance.
+     *
+     * @var Wordlift_Jsonld_Service The {@link Wordlift_Jsonld_Service} instance.
+     */
+    private $jsonld_service;
+    /**
+     * @var \Wordlift_Entity_Uri_Service
+     */
+    private $entity_uri_service;
+
+    /**
+     * Jsonld_Endpoint constructor.
+     *
+     * @param \Wordlift_Jsonld_Service $jsonld_service
+     * @param \Wordlift_Entity_Uri_Service $entity_uri_service
+     */
+    public function __construct( $jsonld_service, $entity_uri_service ) {
+
+        $this->jsonld_service     = $jsonld_service;
+        $this->entity_uri_service = $entity_uri_service;
+
+        // PHP 5.3 compatibility.
+        $that = $this;
+        add_action( 'rest_api_init', function () use ( $that ) {
+            register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<id>\d+)', array(
+                'methods'  => WP_REST_Server::READABLE,
+                'callback' => array( $that, 'jsonld_using_post_id' ),
+                'args'     => array(
+                    'id' => array(
+                        'validate_callback' => function ( $param, $request, $key ) {
+                            return is_numeric( $param );
+                        },
+                        'sanitize_callback' => 'absint',
+                    ),
+                )
+            ) );
+
+            register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/http/(?P<item_id>.*)', array(
+                'methods'  => 'GET',
+                'callback' => array( $that, 'jsonld_using_item_id' ),
+            ) );
+
+            register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/post-meta/(?P<meta_key>[^/]+)', array(
+                'methods'  => 'GET',
+                'callback' => array( $that, 'jsonld_using_post_meta' ),
+            ) );
+
+            register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<post_type>.*)/(?P<post_name>.*)', array(
+                'methods'  => 'GET',
+                'callback' => array( $that, 'jsonld_using_get_page_by_path' ),
+            ) );
+
+        } );
+
+    }
+
+    /**
+     * Callback for the JSON-LD request.
+     *
+     * @param array $request {
+     *  The request array.
+     *
+     * @type int $id The post id.
+     * }
+     *
+     * @return WP_REST_Response
+     * @throws \Exception
+     */
+    public function jsonld_using_post_id( $request ) {
+
+        $post_id     = $request['id'];
+        $is_homepage = ( 0 === $post_id );
+
+        // Send the generated JSON-LD.
+        $data = $this->jsonld_service->get_jsonld( $is_homepage, $post_id );
+
+        return Jsonld_Response_Helper::to_response( $data );
+    }
+
+    /**
+     * Provide a JSON-LD given the itemId.
+     *
+     * @param array $request {
+     *  The request array.
+     *
+     * @type string $item_id The entity item id.
+     * }
+     *
+     * @return WP_REST_Response
+     * @throws \Exception
+     */
+    public function jsonld_using_item_id( $request ) {
+
+        $item_id = 'http://' . $request['item_id'];
+        $post    = $this->entity_uri_service->get_entity( $item_id );
+
+        if ( ! is_a( $post, 'WP_Post' ) ) {
+            return new WP_REST_Response( esc_html( "$item_id not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+        }
+
+        return $this->jsonld_using_post_id( array( 'id' => $post->ID, ) );
+    }
+
+    public function jsonld_using_get_page_by_path( $request ) {
+
+        $post_name = $request['post_name'];
+        $post_type = $request['post_type'];
+
+        global $wpdb;
+
+        $sql = "
 			SELECT ID
 			FROM $wpdb->posts
 			WHERE post_name = %s
 			 AND post_type = %s
 		";
 
-		$post_id = $wpdb->get_var( $wpdb->prepare( $sql, $post_name, $post_type ) );
+        $post_id = $wpdb->get_var( $wpdb->prepare( $sql, $post_name, $post_type ) );
 
-		if ( is_null( $post_id ) ) {
-			return new WP_REST_Response( esc_html( "$post_name of type $post_type not found." ), 404, array( 'Content-Type' => 'text/html' ) );
-		}
+        if ( is_null( $post_id ) ) {
+            return new WP_REST_Response( esc_html( "$post_name of type $post_type not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+        }
 
-		return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
-	}
+        return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
+    }
 
-	/**
-	 * @param WP_REST_Request $request
-	 *
-	 * @return WP_REST_Response
-	 * @throws \Exception
-	 */
-	public function jsonld_using_post_meta( $request ) {
+    /**
+     * @param WP_REST_Request $request
+     *
+     * @return WP_REST_Response
+     * @throws \Exception
+     */
+    public function jsonld_using_post_meta( $request ) {
 
-		$meta_key   = $request['meta_key'];
-		$meta_value = current( $request->get_query_params( 'meta_value' ) );
+        $meta_key   = $request['meta_key'];
+        $meta_value = current( $request->get_query_params( 'meta_value' ) );
 
-		global $wpdb;
+        global $wpdb;
 
-		$sql = "
+        $sql = "
 			SELECT post_id AS ID
 			FROM $wpdb->postmeta
 			WHERE meta_key = %s
@@ -168,13 +168,13 @@ 
 			LIMIT 1
 		";
 
-		$post_id = $wpdb->get_var( $wpdb->prepare( $sql, $meta_key, $meta_value ) );
+        $post_id = $wpdb->get_var( $wpdb->prepare( $sql, $meta_key, $meta_value ) );
 
-		if ( is_null( $post_id ) ) {
-			return new WP_REST_Response( esc_html( "Post with meta key $meta_key and value $meta_value not found." ), 404, array( 'Content-Type' => 'text/html' ) );
-		}
+        if ( is_null( $post_id ) ) {
+            return new WP_REST_Response( esc_html( "Post with meta key $meta_key and value $meta_value not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+        }
 
-		return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
-	}
+        return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
+    }
 
 }

Spacing +37 added lines, -37 removed lines

@@ -37,41 +37,41 @@ 
 	 * @param \Wordlift_Jsonld_Service $jsonld_service
 	 * @param \Wordlift_Entity_Uri_Service $entity_uri_service
 	 */
-	public function __construct( $jsonld_service, $entity_uri_service ) {
+	public function __construct($jsonld_service, $entity_uri_service) {
 
 		$this->jsonld_service     = $jsonld_service;
 		$this->entity_uri_service = $entity_uri_service;
 
 		// PHP 5.3 compatibility.
 		$that = $this;
-		add_action( 'rest_api_init', function () use ( $that ) {
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<id>\d+)', array(
+		add_action('rest_api_init', function() use ($that) {
+			register_rest_route(WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<id>\d+)', array(
 				'methods'  => WP_REST_Server::READABLE,
-				'callback' => array( $that, 'jsonld_using_post_id' ),
+				'callback' => array($that, 'jsonld_using_post_id'),
 				'args'     => array(
 					'id' => array(
-						'validate_callback' => function ( $param, $request, $key ) {
-							return is_numeric( $param );
+						'validate_callback' => function($param, $request, $key) {
+							return is_numeric($param);
 						},
 						'sanitize_callback' => 'absint',
 					),
 				)
-			) );
+			));
 
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/http/(?P<item_id>.*)', array(
+			register_rest_route(WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/http/(?P<item_id>.*)', array(
 				'methods'  => 'GET',
-				'callback' => array( $that, 'jsonld_using_item_id' ),
-			) );
+				'callback' => array($that, 'jsonld_using_item_id'),
+			));
 
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/post-meta/(?P<meta_key>[^/]+)', array(
+			register_rest_route(WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/post-meta/(?P<meta_key>[^/]+)', array(
 				'methods'  => 'GET',
-				'callback' => array( $that, 'jsonld_using_post_meta' ),
-			) );
+				'callback' => array($that, 'jsonld_using_post_meta'),
+			));
 
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<post_type>.*)/(?P<post_name>.*)', array(
+			register_rest_route(WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<post_type>.*)/(?P<post_name>.*)', array(
 				'methods'  => 'GET',
-				'callback' => array( $that, 'jsonld_using_get_page_by_path' ),
-			) );
+				'callback' => array($that, 'jsonld_using_get_page_by_path'),
+			));
 
 		} );
 
@@ -89,15 +89,15 @@ 
 	 * @return WP_REST_Response
 	 * @throws \Exception
 	 */
-	public function jsonld_using_post_id( $request ) {
+	public function jsonld_using_post_id($request) {
 
 		$post_id     = $request['id'];
-		$is_homepage = ( 0 === $post_id );
+		$is_homepage = (0 === $post_id);
 
 		// Send the generated JSON-LD.
-		$data = $this->jsonld_service->get_jsonld( $is_homepage, $post_id );
+		$data = $this->jsonld_service->get_jsonld($is_homepage, $post_id);
 
-		return Jsonld_Response_Helper::to_response( $data );
+		return Jsonld_Response_Helper::to_response($data);
 	}
 
 	/**
@@ -112,19 +112,19 @@ 
 	 * @return WP_REST_Response
 	 * @throws \Exception
 	 */
-	public function jsonld_using_item_id( $request ) {
+	public function jsonld_using_item_id($request) {
 
-		$item_id = 'http://' . $request['item_id'];
-		$post    = $this->entity_uri_service->get_entity( $item_id );
+		$item_id = 'http://'.$request['item_id'];
+		$post    = $this->entity_uri_service->get_entity($item_id);
 
-		if ( ! is_a( $post, 'WP_Post' ) ) {
-			return new WP_REST_Response( esc_html( "$item_id not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+		if ( ! is_a($post, 'WP_Post')) {
+			return new WP_REST_Response(esc_html("$item_id not found."), 404, array('Content-Type' => 'text/html'));
 		}
 
-		return $this->jsonld_using_post_id( array( 'id' => $post->ID, ) );
+		return $this->jsonld_using_post_id(array('id' => $post->ID,));
 	}
 
-	public function jsonld_using_get_page_by_path( $request ) {
+	public function jsonld_using_get_page_by_path($request) {
 
 		$post_name = $request['post_name'];
 		$post_type = $request['post_type'];
@@ -138,13 +138,13 @@ 
 			 AND post_type = %s
 		";
 
-		$post_id = $wpdb->get_var( $wpdb->prepare( $sql, $post_name, $post_type ) );
+		$post_id = $wpdb->get_var($wpdb->prepare($sql, $post_name, $post_type));
 
-		if ( is_null( $post_id ) ) {
-			return new WP_REST_Response( esc_html( "$post_name of type $post_type not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+		if (is_null($post_id)) {
+			return new WP_REST_Response(esc_html("$post_name of type $post_type not found."), 404, array('Content-Type' => 'text/html'));
 		}
 
-		return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
+		return $this->jsonld_using_post_id(array('id' => $post_id,));
 	}
 
 	/**
@@ -153,10 +153,10 @@ 
 	 * @return WP_REST_Response
 	 * @throws \Exception
 	 */
-	public function jsonld_using_post_meta( $request ) {
+	public function jsonld_using_post_meta($request) {
 
 		$meta_key   = $request['meta_key'];
-		$meta_value = current( $request->get_query_params( 'meta_value' ) );
+		$meta_value = current($request->get_query_params('meta_value'));
 
 		global $wpdb;
 
@@ -168,13 +168,13 @@ 
 			LIMIT 1
 		";
 
-		$post_id = $wpdb->get_var( $wpdb->prepare( $sql, $meta_key, $meta_value ) );
+		$post_id = $wpdb->get_var($wpdb->prepare($sql, $meta_key, $meta_value));
 
-		if ( is_null( $post_id ) ) {
-			return new WP_REST_Response( esc_html( "Post with meta key $meta_key and value $meta_value not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+		if (is_null($post_id)) {
+			return new WP_REST_Response(esc_html("Post with meta key $meta_key and value $meta_value not found."), 404, array('Content-Type' => 'text/html'));
 		}
 
-		return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
+		return $this->jsonld_using_post_id(array('id' => $post_id,));
 	}
 
 }