insideout10 / wordlift-plugin

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

Indentation +139 added lines, -139 removed lines

@@ -16,144 +16,144 @@
 use Wordlift\Mappings\Validators\Rule_Groups_Validator;
 
 final class Mappings_Validator {
-	const TRASH_CATEGORY = 'trash';
-	const ACTIVE_CATEGORY = 'active';
-
-	/**
-	 * The {@link Mappings_DBO} instance to test.
-	 *
-	 * @since  3.25.0
-	 * @access private
-	 * @var Mappings_DBO $dbo The {@link Mappings_DBO} instance to test.
-	 */
-	private $dbo;
-
-	/**
-	 * @var Rule_Groups_Validator
-	 */
-	private $rule_groups_validator;
-
-	/**
-	 * Constructor for Wordlift_Mapping_Validator.
-	 *
-	 * @param Mappings_DBO $dbo The {@link Mappings_DBO} instance.
-	 * @param Rule_Groups_Validator $rule_groups_validator
-	 */
-	public function __construct( $dbo, $rule_groups_validator ) {
-
-		$this->dbo                   = $dbo;
-		$this->rule_groups_validator = $rule_groups_validator;
-
-	}
-
-	/**
-	 * This method is used to filter properties based on presence
-	 * of certain key values.
-	 *
-	 * @param $items array Array of properties.
-	 *
-	 * @return array
-	 */
-	private static function filter_properties_for_required_keys( $items ) {
-		return array_filter(
-			$items,
-			function ( $item ) {
-				/**
-				 * Since the properties might also be passed
-				 * by external plugins, we might need to check if
-				 * they have correct data format.
-				 */
-				if ( ! array_key_exists( 'property_name', $item ) ||
-				     ! array_key_exists( 'field_type', $item ) ||
-				     ! array_key_exists( 'field_name', $item ) ||
-				     ! array_key_exists( 'transform_function', $item )
-				) {
-					// If these keys doesnt exist, then dont process.
-					return false;
-				} else {
-					// If the keys exist, then filter it.
-					return true;
-				}
-			}
-		);
-	}
-
-	/**
-	 * Validates two values based on the passed logic
-	 * a single rule passes the user defined logic.
-	 *
-	 * @param string $key The key which every object has mapped to our value.
-	 * @param array $items The array of items.
-	 * @param string $status The value which the items should have.
-	 *
-	 * @return array
-	 */
-	private static function get_property_item_by_status( $key, $items, $status ) {
-		return array_filter(
-			$items,
-			function ( $item ) use ( $key, $status ) {
-				return $item[ $key ] === (string) $status;
-			}
-		);
-	}
-
-	/**
-	 * Validates a post id with the list of active mapping items and check if
-	 * a mapping can be applied.
-	 *
-	 * @param int $identifier The post id.
-	 *
-	 * @param string $type Post or term.
-	 *
-	 * @return array
-	 */
-	public function validate( $identifier, $type ) {
-		// Reset the valid property items before making the validation.
-		$properties = array();
-
-		// Filter registered properties
-		$filter_registered_properties = array();
-
-		// Get active mappings.
-		$mappings = $this->dbo->get_active_mappings();
-		/**
-		 * Apply this filter to get mappings from external plugins.
-		 *
-		 * @param $mappings array Array of mappings from database.
-		 * @param $identifier int The post id.
-		 */
-		$mappings = apply_filters( 'wl_mappings_post', $mappings, $identifier );
-
-		// Get all active rule groups for the mapping items.
-		foreach ( $mappings as $mapping ) {
-			if ( array_key_exists( 'mapping_id', $mapping ) ) {
-				$rule_groups          = $this->dbo->get_rule_groups_by_mapping( (int) $mapping['mapping_id'] );
-				$should_apply_mapping = $this->rule_groups_validator->is_valid( $identifier, $rule_groups, $type );
-				if ( $should_apply_mapping ) {
-					$mapping_item_properties = $this->dbo->get_properties( $mapping['mapping_id'] );
-					$properties              = array_merge( $properties, $mapping_item_properties );
-				}
-			} else {
-				/**
-				 * This is a programmatically defined mapping,
-				 * so we will have the rule groups and the properties in the array keys
-				 */
-				if ( array_key_exists( 'properties', $mapping ) &&
-				     is_array( $mapping['properties'] ) ) {
-					$filter_registered_properties = array_merge( $filter_registered_properties, $mapping['properties'] );
-				}
-			}
-		}
-		// Filter all registered properties based on required key values.
-		$filter_registered_properties = self::filter_properties_for_required_keys( $filter_registered_properties );
-		$active_properties            = self::get_property_item_by_status(
-			'property_status',
-			$properties,
-			self::ACTIVE_CATEGORY
-		);
-
-		// Merge ui defined properties with filter registered properties.
-		return array_merge( $active_properties, $filter_registered_properties );
-	}
+    const TRASH_CATEGORY = 'trash';
+    const ACTIVE_CATEGORY = 'active';
+
+    /**
+     * The {@link Mappings_DBO} instance to test.
+     *
+     * @since  3.25.0
+     * @access private
+     * @var Mappings_DBO $dbo The {@link Mappings_DBO} instance to test.
+     */
+    private $dbo;
+
+    /**
+     * @var Rule_Groups_Validator
+     */
+    private $rule_groups_validator;
+
+    /**
+     * Constructor for Wordlift_Mapping_Validator.
+     *
+     * @param Mappings_DBO $dbo The {@link Mappings_DBO} instance.
+     * @param Rule_Groups_Validator $rule_groups_validator
+     */
+    public function __construct( $dbo, $rule_groups_validator ) {
+
+        $this->dbo                   = $dbo;
+        $this->rule_groups_validator = $rule_groups_validator;
+
+    }
+
+    /**
+     * This method is used to filter properties based on presence
+     * of certain key values.
+     *
+     * @param $items array Array of properties.
+     *
+     * @return array
+     */
+    private static function filter_properties_for_required_keys( $items ) {
+        return array_filter(
+            $items,
+            function ( $item ) {
+                /**
+                 * Since the properties might also be passed
+                 * by external plugins, we might need to check if
+                 * they have correct data format.
+                 */
+                if ( ! array_key_exists( 'property_name', $item ) ||
+                     ! array_key_exists( 'field_type', $item ) ||
+                     ! array_key_exists( 'field_name', $item ) ||
+                     ! array_key_exists( 'transform_function', $item )
+                ) {
+                    // If these keys doesnt exist, then dont process.
+                    return false;
+                } else {
+                    // If the keys exist, then filter it.
+                    return true;
+                }
+            }
+        );
+    }
+
+    /**
+     * Validates two values based on the passed logic
+     * a single rule passes the user defined logic.
+     *
+     * @param string $key The key which every object has mapped to our value.
+     * @param array $items The array of items.
+     * @param string $status The value which the items should have.
+     *
+     * @return array
+     */
+    private static function get_property_item_by_status( $key, $items, $status ) {
+        return array_filter(
+            $items,
+            function ( $item ) use ( $key, $status ) {
+                return $item[ $key ] === (string) $status;
+            }
+        );
+    }
+
+    /**
+     * Validates a post id with the list of active mapping items and check if
+     * a mapping can be applied.
+     *
+     * @param int $identifier The post id.
+     *
+     * @param string $type Post or term.
+     *
+     * @return array
+     */
+    public function validate( $identifier, $type ) {
+        // Reset the valid property items before making the validation.
+        $properties = array();
+
+        // Filter registered properties
+        $filter_registered_properties = array();
+
+        // Get active mappings.
+        $mappings = $this->dbo->get_active_mappings();
+        /**
+         * Apply this filter to get mappings from external plugins.
+         *
+         * @param $mappings array Array of mappings from database.
+         * @param $identifier int The post id.
+         */
+        $mappings = apply_filters( 'wl_mappings_post', $mappings, $identifier );
+
+        // Get all active rule groups for the mapping items.
+        foreach ( $mappings as $mapping ) {
+            if ( array_key_exists( 'mapping_id', $mapping ) ) {
+                $rule_groups          = $this->dbo->get_rule_groups_by_mapping( (int) $mapping['mapping_id'] );
+                $should_apply_mapping = $this->rule_groups_validator->is_valid( $identifier, $rule_groups, $type );
+                if ( $should_apply_mapping ) {
+                    $mapping_item_properties = $this->dbo->get_properties( $mapping['mapping_id'] );
+                    $properties              = array_merge( $properties, $mapping_item_properties );
+                }
+            } else {
+                /**
+                 * This is a programmatically defined mapping,
+                 * so we will have the rule groups and the properties in the array keys
+                 */
+                if ( array_key_exists( 'properties', $mapping ) &&
+                     is_array( $mapping['properties'] ) ) {
+                    $filter_registered_properties = array_merge( $filter_registered_properties, $mapping['properties'] );
+                }
+            }
+        }
+        // Filter all registered properties based on required key values.
+        $filter_registered_properties = self::filter_properties_for_required_keys( $filter_registered_properties );
+        $active_properties            = self::get_property_item_by_status(
+            'property_status',
+            $properties,
+            self::ACTIVE_CATEGORY
+        );
+
+        // Merge ui defined properties with filter registered properties.
+        return array_merge( $active_properties, $filter_registered_properties );
+    }
 
 }

Spacing +24 added lines, -24 removed lines

@@ -39,7 +39,7 @@ 
 	 * @param Mappings_DBO $dbo The {@link Mappings_DBO} instance.
 	 * @param Rule_Groups_Validator $rule_groups_validator
 	 */
-	public function __construct( $dbo, $rule_groups_validator ) {
+	public function __construct($dbo, $rule_groups_validator) {
 
 		$this->dbo                   = $dbo;
 		$this->rule_groups_validator = $rule_groups_validator;
@@ -54,19 +54,19 @@ 
 	 *
 	 * @return array
 	 */
-	private static function filter_properties_for_required_keys( $items ) {
+	private static function filter_properties_for_required_keys($items) {
 		return array_filter(
 			$items,
-			function ( $item ) {
+			function($item) {
 				/**
 				 * Since the properties might also be passed
 				 * by external plugins, we might need to check if
 				 * they have correct data format.
 				 */
-				if ( ! array_key_exists( 'property_name', $item ) ||
-				     ! array_key_exists( 'field_type', $item ) ||
-				     ! array_key_exists( 'field_name', $item ) ||
-				     ! array_key_exists( 'transform_function', $item )
+				if ( ! array_key_exists('property_name', $item) ||
+				     ! array_key_exists('field_type', $item) ||
+				     ! array_key_exists('field_name', $item) ||
+				     ! array_key_exists('transform_function', $item)
 				) {
 					// If these keys doesnt exist, then dont process.
 					return false;
@@ -88,11 +88,11 @@ 
 	 *
 	 * @return array
 	 */
-	private static function get_property_item_by_status( $key, $items, $status ) {
+	private static function get_property_item_by_status($key, $items, $status) {
 		return array_filter(
 			$items,
-			function ( $item ) use ( $key, $status ) {
-				return $item[ $key ] === (string) $status;
+			function($item) use ($key, $status) {
+				return $item[$key] === (string) $status;
 			}
 		);
 	}
@@ -107,7 +107,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function validate( $identifier, $type ) {
+	public function validate($identifier, $type) {
 		// Reset the valid property items before making the validation.
 		$properties = array();
 
@@ -122,30 +122,30 @@ 
 		 * @param $mappings array Array of mappings from database.
 		 * @param $identifier int The post id.
 		 */
-		$mappings = apply_filters( 'wl_mappings_post', $mappings, $identifier );
+		$mappings = apply_filters('wl_mappings_post', $mappings, $identifier);
 
 		// Get all active rule groups for the mapping items.
-		foreach ( $mappings as $mapping ) {
-			if ( array_key_exists( 'mapping_id', $mapping ) ) {
-				$rule_groups          = $this->dbo->get_rule_groups_by_mapping( (int) $mapping['mapping_id'] );
-				$should_apply_mapping = $this->rule_groups_validator->is_valid( $identifier, $rule_groups, $type );
-				if ( $should_apply_mapping ) {
-					$mapping_item_properties = $this->dbo->get_properties( $mapping['mapping_id'] );
-					$properties              = array_merge( $properties, $mapping_item_properties );
+		foreach ($mappings as $mapping) {
+			if (array_key_exists('mapping_id', $mapping)) {
+				$rule_groups          = $this->dbo->get_rule_groups_by_mapping((int) $mapping['mapping_id']);
+				$should_apply_mapping = $this->rule_groups_validator->is_valid($identifier, $rule_groups, $type);
+				if ($should_apply_mapping) {
+					$mapping_item_properties = $this->dbo->get_properties($mapping['mapping_id']);
+					$properties              = array_merge($properties, $mapping_item_properties);
 				}
 			} else {
 				/**
 				 * This is a programmatically defined mapping,
 				 * so we will have the rule groups and the properties in the array keys
 				 */
-				if ( array_key_exists( 'properties', $mapping ) &&
-				     is_array( $mapping['properties'] ) ) {
-					$filter_registered_properties = array_merge( $filter_registered_properties, $mapping['properties'] );
+				if (array_key_exists('properties', $mapping) &&
+				     is_array($mapping['properties'])) {
+					$filter_registered_properties = array_merge($filter_registered_properties, $mapping['properties']);
 				}
 			}
 		}
 		// Filter all registered properties based on required key values.
-		$filter_registered_properties = self::filter_properties_for_required_keys( $filter_registered_properties );
+		$filter_registered_properties = self::filter_properties_for_required_keys($filter_registered_properties);
 		$active_properties            = self::get_property_item_by_status(
 			'property_status',
 			$properties,
@@ -153,7 +153,7 @@ 
 		);
 
 		// Merge ui defined properties with filter registered properties.
-		return array_merge( $active_properties, $filter_registered_properties );
+		return array_merge($active_properties, $filter_registered_properties);
 	}
 
 }

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

Indentation +232 added lines, -232 removed lines

@@ -18,237 +18,237 @@
  * @since 3.25.0
  */
 class Jsonld_Converter {
-	/**
-	 * Enumerations for the field types.
-	 */
-	const FIELD_TYPE_TEXT_FIELD = 'text';
-	const FIELD_TYPE_CUSTOM_FIELD = 'custom_field';
-	const FIELD_TYPE_ACF = 'acf';
-
-	/**
-	 * Mappings can be applied to either post
-	 * or term, the below is used to specify whether it is a post or a term.
-	 */
-	const POST = 'post';
-	const TERM = 'term';
-	/**
-	 * 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, 2 );
-
-		// This is wrong: `wl_term_jsonld_array` will provide a term ID not a post ID (that `wl_post_jsonld_array` expects).
-		add_filter( 'wl_term_jsonld_array', array( $this, 'wl_term_jsonld_array' ), 11, 2 );
-	}
-
-	/**
-	 * Hook to `wl_term_jsonld_array`.
-	 *
-	 * Receive the JSON-LD and the references in the array along with the term 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 $term_id The Term ID.
-	 *
-	 * @return array An array with the updated JSON-LD and references.
-	 */
-	public function wl_term_jsonld_array( $value, $term_id ) {
-		$jsonld     = $value['jsonld'];
-		$references = $value['references'];
-
-		return array(
-			'jsonld'     => $this->build_jsonld( $jsonld, $term_id, $references, self::TERM ),
-			'references' => $references,
-		);
-	}
-
-
-	/**
-	 * 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->build_jsonld( $jsonld, $post_id, $references, self::POST ),
-			'references' => $references,
-		);
-	}
-
-	/**
-	 * Returns JSON-LD data after applying transformation functions.
-	 *
-	 * @param array $jsonld The JSON-LD structure.
-	 * @param int $identifier The {@link WP_Post} id or {@link \WP_Term} id.
-	 * @param array $references An array of post references.
-	 *
-	 * @param string $type Post or term.
-	 *
-	 * @return array the new refactored array structure.
-	 * @since 3.25.0
-	 */
-	private function build_jsonld( $jsonld, $identifier, &$references, $type ) {
-
-		// @@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( $identifier, $type );
-		$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, $identifier, $references, $type );
-			if ( false !== $property_transformed_data ) {
-				$jsonld[ $property['property_name'] ] = $property_transformed_data;
-			}
-		}
-
-		$jsonld = $this->process_nested_properties( $nested_properties, $jsonld, $identifier, $references, $type );
-
-		return $jsonld;
-	}
-
-	/**
-	 * Get the property data by applying the transformation function
-	 *
-	 * @param $property
-	 * @param $jsonld
-	 * @param $post_id
-	 * @param $references
-	 *
-	 * @param $type
-	 *
-	 * @return array|bool|null
-	 */
-	public function get_property_data( $property, $jsonld, $post_id, &$references, $type ) {
-		$transform_instance = $this->transform_functions_registry->get_transform_function( $property['transform_function'] );
-		$data               = Data_Source_Factory::get_instance()->get_data( $post_id, $property, $type );
-		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
-	 *
-	 * @param $post_id
-	 * @param $references
-	 * @param string $type Post or term.
-	 *
-	 * @return array
-	 */
-	public function process_nested_properties( $nested_properties, $jsonld, $post_id, &$references, $type ) {
-		foreach ( $nested_properties as $property ) {
-			$property_data = $this->get_property_data( $property, $jsonld, $post_id, $references, $type );
-			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;
-	}
-
-	private function make_single( $value ) {
-
-		$values = (array) $value;
-
-		if ( empty( $values ) ) {
-			return false;
-		}
-
-		if ( 1 === count( $values ) && 0 === key( $values ) ) {
-			return current( $values );
-		}
-
-		return $values;
-	}
+    /**
+     * Enumerations for the field types.
+     */
+    const FIELD_TYPE_TEXT_FIELD = 'text';
+    const FIELD_TYPE_CUSTOM_FIELD = 'custom_field';
+    const FIELD_TYPE_ACF = 'acf';
+
+    /**
+     * Mappings can be applied to either post
+     * or term, the below is used to specify whether it is a post or a term.
+     */
+    const POST = 'post';
+    const TERM = 'term';
+    /**
+     * 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, 2 );
+
+        // This is wrong: `wl_term_jsonld_array` will provide a term ID not a post ID (that `wl_post_jsonld_array` expects).
+        add_filter( 'wl_term_jsonld_array', array( $this, 'wl_term_jsonld_array' ), 11, 2 );
+    }
+
+    /**
+     * Hook to `wl_term_jsonld_array`.
+     *
+     * Receive the JSON-LD and the references in the array along with the term 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 $term_id The Term ID.
+     *
+     * @return array An array with the updated JSON-LD and references.
+     */
+    public function wl_term_jsonld_array( $value, $term_id ) {
+        $jsonld     = $value['jsonld'];
+        $references = $value['references'];
+
+        return array(
+            'jsonld'     => $this->build_jsonld( $jsonld, $term_id, $references, self::TERM ),
+            'references' => $references,
+        );
+    }
+
+
+    /**
+     * 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->build_jsonld( $jsonld, $post_id, $references, self::POST ),
+            'references' => $references,
+        );
+    }
+
+    /**
+     * Returns JSON-LD data after applying transformation functions.
+     *
+     * @param array $jsonld The JSON-LD structure.
+     * @param int $identifier The {@link WP_Post} id or {@link \WP_Term} id.
+     * @param array $references An array of post references.
+     *
+     * @param string $type Post or term.
+     *
+     * @return array the new refactored array structure.
+     * @since 3.25.0
+     */
+    private function build_jsonld( $jsonld, $identifier, &$references, $type ) {
+
+        // @@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( $identifier, $type );
+        $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, $identifier, $references, $type );
+            if ( false !== $property_transformed_data ) {
+                $jsonld[ $property['property_name'] ] = $property_transformed_data;
+            }
+        }
+
+        $jsonld = $this->process_nested_properties( $nested_properties, $jsonld, $identifier, $references, $type );
+
+        return $jsonld;
+    }
+
+    /**
+     * Get the property data by applying the transformation function
+     *
+     * @param $property
+     * @param $jsonld
+     * @param $post_id
+     * @param $references
+     *
+     * @param $type
+     *
+     * @return array|bool|null
+     */
+    public function get_property_data( $property, $jsonld, $post_id, &$references, $type ) {
+        $transform_instance = $this->transform_functions_registry->get_transform_function( $property['transform_function'] );
+        $data               = Data_Source_Factory::get_instance()->get_data( $post_id, $property, $type );
+        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
+     *
+     * @param $post_id
+     * @param $references
+     * @param string $type Post or term.
+     *
+     * @return array
+     */
+    public function process_nested_properties( $nested_properties, $jsonld, $post_id, &$references, $type ) {
+        foreach ( $nested_properties as $property ) {
+            $property_data = $this->get_property_data( $property, $jsonld, $post_id, $references, $type );
+            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;
+    }
+
+    private function make_single( $value ) {
+
+        $values = (array) $value;
+
+        if ( empty( $values ) ) {
+            return false;
+        }
+
+        if ( 1 === count( $values ) && 0 === key( $values ) ) {
+            return current( $values );
+        }
+
+        return $values;
+    }
 
 }

Spacing +41 added lines, -41 removed lines

@@ -55,17 +55,17 @@ 
 	 * @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, 2 );
+		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, 2);
 
 		// This is wrong: `wl_term_jsonld_array` will provide a term ID not a post ID (that `wl_post_jsonld_array` expects).
-		add_filter( 'wl_term_jsonld_array', array( $this, 'wl_term_jsonld_array' ), 11, 2 );
+		add_filter('wl_term_jsonld_array', array($this, 'wl_term_jsonld_array'), 11, 2);
 	}
 
 	/**
@@ -85,12 +85,12 @@ 
 	 *
 	 * @return array An array with the updated JSON-LD and references.
 	 */
-	public function wl_term_jsonld_array( $value, $term_id ) {
+	public function wl_term_jsonld_array($value, $term_id) {
 		$jsonld     = $value['jsonld'];
 		$references = $value['references'];
 
 		return array(
-			'jsonld'     => $this->build_jsonld( $jsonld, $term_id, $references, self::TERM ),
+			'jsonld'     => $this->build_jsonld($jsonld, $term_id, $references, self::TERM),
 			'references' => $references,
 		);
 	}
@@ -113,13 +113,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->build_jsonld( $jsonld, $post_id, $references, self::POST ),
+			'jsonld'     => $this->build_jsonld($jsonld, $post_id, $references, self::POST),
 			'references' => $references,
 		);
 	}
@@ -136,27 +136,27 @@ 
 	 * @return array the new refactored array structure.
 	 * @since 3.25.0
 	 */
-	private function build_jsonld( $jsonld, $identifier, &$references, $type ) {
+	private function build_jsonld($jsonld, $identifier, &$references, $type) {
 
 		// @@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( $identifier, $type );
+		$properties        = $this->validator->validate($identifier, $type);
 		$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, $identifier, $references, $type );
-			if ( false !== $property_transformed_data ) {
-				$jsonld[ $property['property_name'] ] = $property_transformed_data;
+			$property_transformed_data = $this->get_property_data($property, $jsonld, $identifier, $references, $type);
+			if (false !== $property_transformed_data) {
+				$jsonld[$property['property_name']] = $property_transformed_data;
 			}
 		}
 
-		$jsonld = $this->process_nested_properties( $nested_properties, $jsonld, $identifier, $references, $type );
+		$jsonld = $this->process_nested_properties($nested_properties, $jsonld, $identifier, $references, $type);
 
 		return $jsonld;
 	}
@@ -173,16 +173,16 @@ 
 	 *
 	 * @return array|bool|null
 	 */
-	public function get_property_data( $property, $jsonld, $post_id, &$references, $type ) {
-		$transform_instance = $this->transform_functions_registry->get_transform_function( $property['transform_function'] );
-		$data               = Data_Source_Factory::get_instance()->get_data( $post_id, $property, $type );
-		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, $type) {
+		$transform_instance = $this->transform_functions_registry->get_transform_function($property['transform_function']);
+		$data               = Data_Source_Factory::get_instance()->get_data($post_id, $property, $type);
+		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;
@@ -200,17 +200,17 @@ 
 	 *
 	 * @return array
 	 */
-	public function process_nested_properties( $nested_properties, $jsonld, $post_id, &$references, $type ) {
-		foreach ( $nested_properties as $property ) {
-			$property_data = $this->get_property_data( $property, $jsonld, $post_id, $references, $type );
-			if ( false === $property_data ) {
+	public function process_nested_properties($nested_properties, $jsonld, $post_id, &$references, $type) {
+		foreach ($nested_properties as $property) {
+			$property_data = $this->get_property_data($property, $jsonld, $post_id, $references, $type);
+			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;
 
 			/**
@@ -218,34 +218,34 @@ 
 			 * 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;
 	}
 
-	private function make_single( $value ) {
+	private function make_single($value) {
 
 		$values = (array) $value;
 
-		if ( empty( $values ) ) {
+		if (empty($values)) {
 			return false;
 		}
 
-		if ( 1 === count( $values ) && 0 === key( $values ) ) {
-			return current( $values );
+		if (1 === count($values) && 0 === key($values)) {
+			return current($values);
 		}
 
 		return $values;

src/wordlift/mappings/validators/class-rule-validator.php

Indentation +20 added lines, -20 removed lines

@@ -19,27 +19,27 @@
  */
 interface Rule_Validator {
 
-	const IS_EQUAL_TO = '===';
-	const IS_NOT_EQUAL_TO = '!==';
+    const IS_EQUAL_TO = '===';
+    const IS_NOT_EQUAL_TO = '!==';
 
-	/**
-	 * Get the validator label.
-	 *
-	 * @return string The validator label.
-	 */
-	public function get_label();
+    /**
+     * Get the validator label.
+     *
+     * @return string The validator label.
+     */
+    public function get_label();
 
-	/**
-	 * Test whether a post passes a validation.
-	 *
-	 * @param int $identifier The post id or term id.
-	 * @param string $operator The operator.
-	 * @param string $operand_1 The first operand.
-	 * @param string $operand_2 The second operand.
-	 * @param string $type The type is either post or term.
-	 *
-	 * @return bool Whether the post passes or not the validation.
-	 */
-	public function is_valid( $identifier, $operator, $operand_1, $operand_2, $type );
+    /**
+     * Test whether a post passes a validation.
+     *
+     * @param int $identifier The post id or term id.
+     * @param string $operator The operator.
+     * @param string $operand_1 The first operand.
+     * @param string $operand_2 The second operand.
+     * @param string $type The type is either post or term.
+     *
+     * @return bool Whether the post passes or not the validation.
+     */
+    public function is_valid( $identifier, $operator, $operand_1, $operand_2, $type );
 
 }

Spacing +1 added lines, -1 removed lines

@@ -40,6 +40,6 @@
 	 *
 	 * @return bool Whether the post passes or not the validation.
 	 */
-	public function is_valid( $identifier, $operator, $operand_1, $operand_2, $type );
+	public function is_valid($identifier, $operator, $operand_1, $operand_2, $type);
 
 }

src/wordlift/mappings/validators/class-rule-groups-validator.php

Indentation +41 added lines, -41 removed lines

@@ -19,55 +19,55 @@
  */
 class Rule_Groups_Validator {
 
-	/**
-	 * The {@link Rule_Validators_Registry} instance.
-	 *
-	 * @var Rule_Validators_Registry $rule_validators_registry The {@link Rule_Validators_Registry} instance.
-	 */
-	private $rule_validators_registry;
+    /**
+     * The {@link Rule_Validators_Registry} instance.
+     *
+     * @var Rule_Validators_Registry $rule_validators_registry The {@link Rule_Validators_Registry} instance.
+     */
+    private $rule_validators_registry;
 
-	/**
-	 * Rule_Groups_Validator constructor.
-	 *
-	 * @param Rule_Validators_Registry $rule_validators_registry
-	 */
-	public function __construct( $rule_validators_registry ) {
+    /**
+     * Rule_Groups_Validator constructor.
+     *
+     * @param Rule_Validators_Registry $rule_validators_registry
+     */
+    public function __construct( $rule_validators_registry ) {
 
-		$this->rule_validators_registry = $rule_validators_registry;
+        $this->rule_validators_registry = $rule_validators_registry;
 
-	}
+    }
 
-	/**
-	 * Check whether the specified post passes at least one group of rules.
-	 *
-	 * @param int $identifier The post id.
-	 * @param array $rule_groups An array of rules' groups.
-	 * @param $type string Post or term
-	 *
-	 * @return bool Whether the post passes at least one rule group.
-	 */
-	public function is_valid( $identifier, $rule_groups, $type ) {
+    /**
+     * Check whether the specified post passes at least one group of rules.
+     *
+     * @param int $identifier The post id.
+     * @param array $rule_groups An array of rules' groups.
+     * @param $type string Post or term
+     *
+     * @return bool Whether the post passes at least one rule group.
+     */
+    public function is_valid( $identifier, $rule_groups, $type ) {
 
-		// Validate each group. Return true as soon as one group is validated (all rules).
-		foreach ( (array) $rule_groups as $rule_group ) {
-			foreach ( $rule_group['rules'] as $rule ) {
-				$rule_field_one   = $rule['rule_field_one'];
-				$rule_logic_field = $rule['rule_logic_field'];
-				$rule_field_two   = $rule['rule_field_two'];
+        // Validate each group. Return true as soon as one group is validated (all rules).
+        foreach ( (array) $rule_groups as $rule_group ) {
+            foreach ( $rule_group['rules'] as $rule ) {
+                $rule_field_one   = $rule['rule_field_one'];
+                $rule_logic_field = $rule['rule_logic_field'];
+                $rule_field_two   = $rule['rule_field_two'];
 
-				$rule_validator = $this->rule_validators_registry->get_rule_validator( $rule_field_one );
-				// Skip to the next Rule Group if a rule isn't valid.
-				if ( ! $rule_validator->is_valid( $identifier, $rule_logic_field, $rule_field_one, $rule_field_two, $type ) ) {
-					continue 2;
-				}
+                $rule_validator = $this->rule_validators_registry->get_rule_validator( $rule_field_one );
+                // Skip to the next Rule Group if a rule isn't valid.
+                if ( ! $rule_validator->is_valid( $identifier, $rule_logic_field, $rule_field_one, $rule_field_two, $type ) ) {
+                    continue 2;
+                }
 
-			}
+            }
 
-			// If we got here it means that all the rules have been validated (or the rules' group has no rules).
-			return true;
-		}
+            // If we got here it means that all the rules have been validated (or the rules' group has no rules).
+            return true;
+        }
 
-		return false;
-	}
+        return false;
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -31,7 +31,7 @@ 
 	 *
 	 * @param Rule_Validators_Registry $rule_validators_registry
 	 */
-	public function __construct( $rule_validators_registry ) {
+	public function __construct($rule_validators_registry) {
 
 		$this->rule_validators_registry = $rule_validators_registry;
 
@@ -46,18 +46,18 @@ 
 	 *
 	 * @return bool Whether the post passes at least one rule group.
 	 */
-	public function is_valid( $identifier, $rule_groups, $type ) {
+	public function is_valid($identifier, $rule_groups, $type) {
 
 		// Validate each group. Return true as soon as one group is validated (all rules).
-		foreach ( (array) $rule_groups as $rule_group ) {
-			foreach ( $rule_group['rules'] as $rule ) {
+		foreach ((array) $rule_groups as $rule_group) {
+			foreach ($rule_group['rules'] as $rule) {
 				$rule_field_one   = $rule['rule_field_one'];
 				$rule_logic_field = $rule['rule_logic_field'];
 				$rule_field_two   = $rule['rule_field_two'];
 
-				$rule_validator = $this->rule_validators_registry->get_rule_validator( $rule_field_one );
+				$rule_validator = $this->rule_validators_registry->get_rule_validator($rule_field_one);
 				// Skip to the next Rule Group if a rule isn't valid.
-				if ( ! $rule_validator->is_valid( $identifier, $rule_logic_field, $rule_field_one, $rule_field_two, $type ) ) {
+				if ( ! $rule_validator->is_valid($identifier, $rule_logic_field, $rule_field_one, $rule_field_two, $type)) {
 					continue 2;
 				}
 

src/wordlift/mappings/validators/class-taxonomy-term-rule-validator.php

Indentation +41 added lines, -41 removed lines

@@ -15,53 +15,53 @@
  */
 class Taxonomy_Term_Rule_Validator implements Rule_Validator {
 
-	const TAXONOMY = 'taxonomy';
+    const TAXONOMY = 'taxonomy';
 
-	public function __construct() {
-		add_filter( 'wl_mappings_rule_validators', array( $this, 'wl_mappings_rule_validators' ) );
-	}
+    public function __construct() {
+        add_filter( 'wl_mappings_rule_validators', array( $this, 'wl_mappings_rule_validators' ) );
+    }
 
-	/**
-	 * Hook to `wl_mappings_rule_validators` to register ourselves.
-	 *
-	 * @param array $value An array with validators.
-	 *
-	 * @return array An array with validators plus ours.
-	 */
-	public function wl_mappings_rule_validators( $value ) {
-		$value[ self::TAXONOMY ] = $this;
+    /**
+     * Hook to `wl_mappings_rule_validators` to register ourselves.
+     *
+     * @param array $value An array with validators.
+     *
+     * @return array An array with validators plus ours.
+     */
+    public function wl_mappings_rule_validators( $value ) {
+        $value[ self::TAXONOMY ] = $this;
 
-		return $value;
-	}
+        return $value;
+    }
 
 
-	public function get_label() {
-		return __( 'TaxonomyTerm', 'wordlift' );
-	}
+    public function get_label() {
+        return __( 'TaxonomyTerm', 'wordlift' );
+    }
 
-	public function is_valid( $identifier, $operator, $operand_1, $taxonomy, $type ) {
-		$current_term = get_term( $identifier );
-		// If it is not a term page, then return false for two operators.
-		if ( ! $current_term instanceof \WP_Term ) {
-			return false;
-		}
-		$terms = get_terms( $taxonomy, array( 'get' => 'all' ) );
-		$terms = array_map( function ( $term ) {
-			/**
-			 * @var $term \WP_Term
-			 */
-			return $term->term_id;
-		}, $terms );
-		if ( $operator === Rule_Validator::IS_EQUAL_TO ) {
-			// if we dont have term id, then skip the flow.
-			// If we are in term page, then we need to check if the current
-			// term belongs to the taxonomy
-			return in_array( $current_term->term_id, $terms );
-		}
+    public function is_valid( $identifier, $operator, $operand_1, $taxonomy, $type ) {
+        $current_term = get_term( $identifier );
+        // If it is not a term page, then return false for two operators.
+        if ( ! $current_term instanceof \WP_Term ) {
+            return false;
+        }
+        $terms = get_terms( $taxonomy, array( 'get' => 'all' ) );
+        $terms = array_map( function ( $term ) {
+            /**
+             * @var $term \WP_Term
+             */
+            return $term->term_id;
+        }, $terms );
+        if ( $operator === Rule_Validator::IS_EQUAL_TO ) {
+            // if we dont have term id, then skip the flow.
+            // If we are in term page, then we need to check if the current
+            // term belongs to the taxonomy
+            return in_array( $current_term->term_id, $terms );
+        }
 
-		if ( $operator === Rule_Validator::IS_NOT_EQUAL_TO ) {
-			return ! in_array( $current_term->term_id, $terms );
-		}
+        if ( $operator === Rule_Validator::IS_NOT_EQUAL_TO ) {
+            return ! in_array( $current_term->term_id, $terms );
+        }
 
-	}
+    }
 }

Spacing +14 added lines, -14 removed lines

@@ -18,7 +18,7 @@ 
 	const TAXONOMY = 'taxonomy';
 
 	public function __construct() {
-		add_filter( 'wl_mappings_rule_validators', array( $this, 'wl_mappings_rule_validators' ) );
+		add_filter('wl_mappings_rule_validators', array($this, 'wl_mappings_rule_validators'));
 	}
 
 	/**
@@ -28,39 +28,39 @@ 
 	 *
 	 * @return array An array with validators plus ours.
 	 */
-	public function wl_mappings_rule_validators( $value ) {
-		$value[ self::TAXONOMY ] = $this;
+	public function wl_mappings_rule_validators($value) {
+		$value[self::TAXONOMY] = $this;
 
 		return $value;
 	}
 
 
 	public function get_label() {
-		return __( 'TaxonomyTerm', 'wordlift' );
+		return __('TaxonomyTerm', 'wordlift');
 	}
 
-	public function is_valid( $identifier, $operator, $operand_1, $taxonomy, $type ) {
-		$current_term = get_term( $identifier );
+	public function is_valid($identifier, $operator, $operand_1, $taxonomy, $type) {
+		$current_term = get_term($identifier);
 		// If it is not a term page, then return false for two operators.
-		if ( ! $current_term instanceof \WP_Term ) {
+		if ( ! $current_term instanceof \WP_Term) {
 			return false;
 		}
-		$terms = get_terms( $taxonomy, array( 'get' => 'all' ) );
-		$terms = array_map( function ( $term ) {
+		$terms = get_terms($taxonomy, array('get' => 'all'));
+		$terms = array_map(function($term) {
 			/**
 			 * @var $term \WP_Term
 			 */
 			return $term->term_id;
-		}, $terms );
-		if ( $operator === Rule_Validator::IS_EQUAL_TO ) {
+		}, $terms);
+		if ($operator === Rule_Validator::IS_EQUAL_TO) {
 			// if we dont have term id, then skip the flow.
 			// If we are in term page, then we need to check if the current
 			// term belongs to the taxonomy
-			return in_array( $current_term->term_id, $terms );
+			return in_array($current_term->term_id, $terms);
 		}
 
-		if ( $operator === Rule_Validator::IS_NOT_EQUAL_TO ) {
-			return ! in_array( $current_term->term_id, $terms );
+		if ($operator === Rule_Validator::IS_NOT_EQUAL_TO) {
+			return ! in_array($current_term->term_id, $terms);
 		}
 
 	}

src/wordlift/mappings/validators/class-post-type-rule-validator.php

Indentation +58 added lines, -58 removed lines

@@ -15,63 +15,63 @@
  * @package Wordlift\Mappings\Validators
  */
 class Post_Type_Rule_Validator implements Rule_Validator {
-	/**
-	 * @since 3.25.0
-	 * Enum for the post type rule validator.
-	 */
-	const POST_TYPE = 'post_type';
-	/**
-	 * Post_Type_Rule_Validator constructor.
-	 *
-	 * When initializing the class hooks to `wl_mappings_rule_validators`.
-	 */
-	public function __construct() {
-
-		add_filter( 'wl_mappings_rule_validators', array( $this, 'wl_mappings_rule_validators' ) );
-
-	}
-
-	/**
-	 * Hook to `wl_mappings_rule_validators` to register ourselves.
-	 *
-	 * @param array $value An array with validators.
-	 *
-	 * @return array An array with validators plus ours.
-	 */
-	public function wl_mappings_rule_validators( $value ) {
-
-		$value[self::POST_TYPE] = $this;
-
-		return $value;
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_label() {
-		return __( 'Post Type', 'wordlift' );
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function is_valid( $identifier, $operator, $operand_1, $operand_2, $type ) {
-
-		// Get the post type and then check whether it matches or not according to the operator.
-		$post_type = get_post_type( $identifier );
-
-		switch ( $operator ) {
-			case Rule_Validator::IS_NOT_EQUAL_TO:
-				return $post_type !== $operand_2;
-
-			case Rule_Validator::IS_EQUAL_TO:
-				return $post_type === $operand_2;
-
-			default:
-
-		}
-
-		return false;
-	}
+    /**
+     * @since 3.25.0
+     * Enum for the post type rule validator.
+     */
+    const POST_TYPE = 'post_type';
+    /**
+     * Post_Type_Rule_Validator constructor.
+     *
+     * When initializing the class hooks to `wl_mappings_rule_validators`.
+     */
+    public function __construct() {
+
+        add_filter( 'wl_mappings_rule_validators', array( $this, 'wl_mappings_rule_validators' ) );
+
+    }
+
+    /**
+     * Hook to `wl_mappings_rule_validators` to register ourselves.
+     *
+     * @param array $value An array with validators.
+     *
+     * @return array An array with validators plus ours.
+     */
+    public function wl_mappings_rule_validators( $value ) {
+
+        $value[self::POST_TYPE] = $this;
+
+        return $value;
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_label() {
+        return __( 'Post Type', 'wordlift' );
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function is_valid( $identifier, $operator, $operand_1, $operand_2, $type ) {
+
+        // Get the post type and then check whether it matches or not according to the operator.
+        $post_type = get_post_type( $identifier );
+
+        switch ( $operator ) {
+            case Rule_Validator::IS_NOT_EQUAL_TO:
+                return $post_type !== $operand_2;
+
+            case Rule_Validator::IS_EQUAL_TO:
+                return $post_type === $operand_2;
+
+            default:
+
+        }
+
+        return false;
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -27,7 +27,7 @@ 
 	 */
 	public function __construct() {
 
-		add_filter( 'wl_mappings_rule_validators', array( $this, 'wl_mappings_rule_validators' ) );
+		add_filter('wl_mappings_rule_validators', array($this, 'wl_mappings_rule_validators'));
 
 	}
 
@@ -38,7 +38,7 @@ 
 	 *
 	 * @return array An array with validators plus ours.
 	 */
-	public function wl_mappings_rule_validators( $value ) {
+	public function wl_mappings_rule_validators($value) {
 
 		$value[self::POST_TYPE] = $this;
 
@@ -49,18 +49,18 @@ 
 	 * {@inheritdoc}
 	 */
 	public function get_label() {
-		return __( 'Post Type', 'wordlift' );
+		return __('Post Type', 'wordlift');
 	}
 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function is_valid( $identifier, $operator, $operand_1, $operand_2, $type ) {
+	public function is_valid($identifier, $operator, $operand_1, $operand_2, $type) {
 
 		// Get the post type and then check whether it matches or not according to the operator.
-		$post_type = get_post_type( $identifier );
+		$post_type = get_post_type($identifier);
 
-		switch ( $operator ) {
+		switch ($operator) {
 			case Rule_Validator::IS_NOT_EQUAL_TO:
 				return $post_type !== $operand_2;
 

src/wordlift/mappings/validators/class-taxonomy-rule-validator.php

Indentation +24 added lines, -24 removed lines

@@ -16,28 +16,28 @@
  */
 class Taxonomy_Rule_Validator implements Rule_Validator {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function is_valid( $identifier, $operator, $operand_1, $operand_2, $type ) {
-
-		$taxonomy  = $operand_1;
-		$term_slug = $operand_2;
-
-		$is_object_in_term = is_object_in_term( $identifier, $taxonomy, $term_slug );
-
-		if ( is_wp_error( $is_object_in_term ) ) {
-			return false;
-		}
-
-		return ( $is_object_in_term && self::IS_EQUAL_TO === $operator )
-		       || ( ! $is_object_in_term && self::IS_NOT_EQUAL_TO === $operator );
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_label() {
-		return __( 'Taxonomy', 'wordlift' );
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function is_valid( $identifier, $operator, $operand_1, $operand_2, $type ) {
+
+        $taxonomy  = $operand_1;
+        $term_slug = $operand_2;
+
+        $is_object_in_term = is_object_in_term( $identifier, $taxonomy, $term_slug );
+
+        if ( is_wp_error( $is_object_in_term ) ) {
+            return false;
+        }
+
+        return ( $is_object_in_term && self::IS_EQUAL_TO === $operator )
+               || ( ! $is_object_in_term && self::IS_NOT_EQUAL_TO === $operator );
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_label() {
+        return __( 'Taxonomy', 'wordlift' );
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -19,25 +19,25 @@
 	/**
 	 * {@inheritdoc}
 	 */
-	public function is_valid( $identifier, $operator, $operand_1, $operand_2, $type ) {
+	public function is_valid($identifier, $operator, $operand_1, $operand_2, $type) {
 
 		$taxonomy  = $operand_1;
 		$term_slug = $operand_2;
 
-		$is_object_in_term = is_object_in_term( $identifier, $taxonomy, $term_slug );
+		$is_object_in_term = is_object_in_term($identifier, $taxonomy, $term_slug);
 
-		if ( is_wp_error( $is_object_in_term ) ) {
+		if (is_wp_error($is_object_in_term)) {
 			return false;
 		}
 
-		return ( $is_object_in_term && self::IS_EQUAL_TO === $operator )
-		       || ( ! $is_object_in_term && self::IS_NOT_EQUAL_TO === $operator );
+		return ($is_object_in_term && self::IS_EQUAL_TO === $operator)
+		       || ( ! $is_object_in_term && self::IS_NOT_EQUAL_TO === $operator);
 	}
 
 	/**
 	 * {@inheritdoc}
 	 */
 	public function get_label() {
-		return __( 'Taxonomy', 'wordlift' );
+		return __('Taxonomy', 'wordlift');
 	}
 }

src/wordlift/mappings/data-source/class-data-source-factory.php

Indentation +40 added lines, -40 removed lines

@@ -11,45 +11,45 @@
 
 class Data_Source_Factory {
 
-	private static $instance = null;
-
-	private $data_sources = array();
-
-	public function __construct() {
-		$this->data_sources = array(
-			'acf'  => new Acf_Data_Source(),
-			'meta' => new Meta_Data_Source()
-		);
-	}
-
-	/**
-	 * @param $identifier int post id or term id based on type.
-	 * @param $property_data array
-	 * @param $type string post or term.
-	 *
-	 * @return mixed
-	 */
-	public function get_data( $identifier, $property_data, $type ) {
-		switch ( $property_data['field_type'] ) {
-			case Jsonld_Converter::FIELD_TYPE_ACF:
-				return $this->data_sources['acf']->get_data( $identifier, $property_data, $type );
-			case Jsonld_Converter::FIELD_TYPE_CUSTOM_FIELD:
-				return $this->data_sources['meta']->get_data( $identifier, $property_data, $type );
-			default:
-				return $property_data['field_name'];
-		}
-	}
-
-
-	/**
-	 * @return Data_Source_Factory
-	 */
-	public static function get_instance() {
-		if ( self::$instance === null ) {
-			self::$instance = new Data_Source_Factory();
-		}
-
-		return self::$instance;
-	}
+    private static $instance = null;
+
+    private $data_sources = array();
+
+    public function __construct() {
+        $this->data_sources = array(
+            'acf'  => new Acf_Data_Source(),
+            'meta' => new Meta_Data_Source()
+        );
+    }
+
+    /**
+     * @param $identifier int post id or term id based on type.
+     * @param $property_data array
+     * @param $type string post or term.
+     *
+     * @return mixed
+     */
+    public function get_data( $identifier, $property_data, $type ) {
+        switch ( $property_data['field_type'] ) {
+            case Jsonld_Converter::FIELD_TYPE_ACF:
+                return $this->data_sources['acf']->get_data( $identifier, $property_data, $type );
+            case Jsonld_Converter::FIELD_TYPE_CUSTOM_FIELD:
+                return $this->data_sources['meta']->get_data( $identifier, $property_data, $type );
+            default:
+                return $property_data['field_name'];
+        }
+    }
+
+
+    /**
+     * @return Data_Source_Factory
+     */
+    public static function get_instance() {
+        if ( self::$instance === null ) {
+            self::$instance = new Data_Source_Factory();
+        }
+
+        return self::$instance;
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -29,12 +29,12 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function get_data( $identifier, $property_data, $type ) {
-		switch ( $property_data['field_type'] ) {
+	public function get_data($identifier, $property_data, $type) {
+		switch ($property_data['field_type']) {
 			case Jsonld_Converter::FIELD_TYPE_ACF:
-				return $this->data_sources['acf']->get_data( $identifier, $property_data, $type );
+				return $this->data_sources['acf']->get_data($identifier, $property_data, $type);
 			case Jsonld_Converter::FIELD_TYPE_CUSTOM_FIELD:
-				return $this->data_sources['meta']->get_data( $identifier, $property_data, $type );
+				return $this->data_sources['meta']->get_data($identifier, $property_data, $type);
 			default:
 				return $property_data['field_name'];
 		}
@@ -45,7 +45,7 @@ 
 	 * @return Data_Source_Factory
 	 */
 	public static function get_instance() {
-		if ( self::$instance === null ) {
+		if (self::$instance === null) {
 			self::$instance = new Data_Source_Factory();
 		}
 

src/wordlift/mappings/data-source/class-abstract-data-source.php

Indentation +1 added lines, -1 removed lines

@@ -13,6 +13,6 @@
  */
 interface Abstract_Data_Source {
 
-	public function get_data( $identifier, $property, $type );
+    public function get_data( $identifier, $property, $type );
 
 }

Spacing +1 added lines, -1 removed lines

@@ -13,6 +13,6 @@
  */
 interface Abstract_Data_Source {
 
-	public function get_data( $identifier, $property, $type );
+	public function get_data($identifier, $property, $type);
 
 }