insideout10 / wordlift-plugin

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( $post_id, $property );
+    public function get_data( $post_id, $property );
 
 }

Spacing +1 added lines, -1 removed lines

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

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

Indentation +33 added lines, -33 removed lines

@@ -11,38 +11,38 @@
 
 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()
-		);
-	}
-
-	public function get_data( $post_id, $property_data ) {
-		switch ( $property_data['field_type'] ) {
-			case Jsonld_Converter::FIELD_TYPE_ACF:
-				return $this->data_sources['acf']->get_data( $post_id, $property_data );
-			case Jsonld_Converter::FIELD_TYPE_CUSTOM_FIELD:
-				return $this->data_sources['meta']->get_data( $post_id, $property_data );
-			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()
+        );
+    }
+
+    public function get_data( $post_id, $property_data ) {
+        switch ( $property_data['field_type'] ) {
+            case Jsonld_Converter::FIELD_TYPE_ACF:
+                return $this->data_sources['acf']->get_data( $post_id, $property_data );
+            case Jsonld_Converter::FIELD_TYPE_CUSTOM_FIELD:
+                return $this->data_sources['meta']->get_data( $post_id, $property_data );
+            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

@@ -22,12 +22,12 @@ 
 		);
 	}
 
-	public function get_data( $post_id, $property_data ) {
-		switch ( $property_data['field_type'] ) {
+	public function get_data($post_id, $property_data) {
+		switch ($property_data['field_type']) {
 			case Jsonld_Converter::FIELD_TYPE_ACF:
-				return $this->data_sources['acf']->get_data( $post_id, $property_data );
+				return $this->data_sources['acf']->get_data($post_id, $property_data);
 			case Jsonld_Converter::FIELD_TYPE_CUSTOM_FIELD:
-				return $this->data_sources['meta']->get_data( $post_id, $property_data );
+				return $this->data_sources['meta']->get_data($post_id, $property_data);
 			default:
 				return $property_data['field_name'];
 		}
@@ -38,7 +38,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-meta-data-source.php

Indentation +8 added lines, -8 removed lines

@@ -13,14 +13,14 @@
  */
 class Meta_Data_Source implements Abstract_Data_Source {
 
-	public function get_data( $post_id, $property ) {
+    public function get_data( $post_id, $property ) {
 
-		$value = $property['field_name'];
+        $value = $property['field_name'];
 
-		if ( get_queried_object() instanceof \WP_Term ) {
-			return array_map( 'wp_strip_all_tags', get_term_meta( get_queried_object_id(), $value ) );
-		} else {
-			return array_map( 'wp_strip_all_tags', get_post_meta( $post_id, $value ) );
-		}
-	}
+        if ( get_queried_object() instanceof \WP_Term ) {
+            return array_map( 'wp_strip_all_tags', get_term_meta( get_queried_object_id(), $value ) );
+        } else {
+            return array_map( 'wp_strip_all_tags', get_post_meta( $post_id, $value ) );
+        }
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -13,14 +13,14 @@
  */
 class Meta_Data_Source implements Abstract_Data_Source {
 
-	public function get_data( $post_id, $property ) {
+	public function get_data($post_id, $property) {
 
 		$value = $property['field_name'];
 
-		if ( get_queried_object() instanceof \WP_Term ) {
-			return array_map( 'wp_strip_all_tags', get_term_meta( get_queried_object_id(), $value ) );
+		if (get_queried_object() instanceof \WP_Term) {
+			return array_map('wp_strip_all_tags', get_term_meta(get_queried_object_id(), $value));
 		} else {
-			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));
 		}
 	}
 }

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

Indentation +49 added lines, -49 removed lines

@@ -8,58 +8,58 @@
 
 class Acf_Data_Source implements Abstract_Data_Source {
 
-	public function get_data( $post_id, $property_data ) {
+    public function get_data( $post_id, $property_data ) {
 
-		if ( ! function_exists( 'get_field' ) || ! function_exists( 'get_field_object' ) ) {
-			return array();
-		}
+        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 );
+    }
 
-	/**
-	 * Gets data from acf, format the data if it is a repeater field.
-	 *
-	 * @param $field_name
-	 * @param $post_id
-	 *
-	 * @return array|mixed
-	 */
-	private function get_data_for_acf_field( $field_name, $post_id ) {
-		if ( get_queried_object() instanceof \WP_Term ) {
-			// Data fetching method for term is different.
-			$term       = get_queried_object();
-			$field_data = get_field_object( $field_name, $term );
-			$data       = get_field( $field_name, $term );
-		} else {
-			$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,
-			 * so each repeater item should be checked if there is a single sub field.
-			 */
-			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' );
+    /**
+     * Gets data from acf, format the data if it is a repeater field.
+     *
+     * @param $field_name
+     * @param $post_id
+     *
+     * @return array|mixed
+     */
+    private function get_data_for_acf_field( $field_name, $post_id ) {
+        if ( get_queried_object() instanceof \WP_Term ) {
+            // Data fetching method for term is different.
+            $term       = get_queried_object();
+            $field_data = get_field_object( $field_name, $term );
+            $data       = get_field( $field_name, $term );
+        } else {
+            $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,
+             * so each repeater item should be checked if there is a single sub field.
+             */
+            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 );
-			}
-		}
+                // re-index all the values.
+                return array_values( $repeater_formatted_data );
+            }
+        }
 
-		// Return normal acf data if it is not a repeater field.
-		return $data;
-	}
+        // Return normal acf data if it is not a repeater field.
+        return $data;
+    }
 }

Spacing +19 added lines, -19 removed lines

@@ -8,13 +8,13 @@ 
 
 class Acf_Data_Source implements Abstract_Data_Source {
 
-	public function get_data( $post_id, $property_data ) {
+	public function get_data($post_id, $property_data) {
 
-		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);
 	}
 
 	/**
@@ -25,37 +25,37 @@ 
 	 *
 	 * @return array|mixed
 	 */
-	private function get_data_for_acf_field( $field_name, $post_id ) {
-		if ( get_queried_object() instanceof \WP_Term ) {
+	private function get_data_for_acf_field($field_name, $post_id) {
+		if (get_queried_object() instanceof \WP_Term) {
 			// Data fetching method for term is different.
 			$term       = get_queried_object();
-			$field_data = get_field_object( $field_name, $term );
-			$data       = get_field( $field_name, $term );
+			$field_data = get_field_object($field_name, $term);
+			$data       = get_field($field_name, $term);
 		} else {
-			$field_data = get_field_object( $field_name, $post_id );
-			$data       = get_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,
 			 * so each repeater item should be checked if there is a single sub field.
 			 */
-			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);
 			}
 		}
 

src/wordlift/mappings/pages/class-edit-mappings-page.php

Indentation +350 added lines, -350 removed lines

@@ -24,356 +24,356 @@
  */
 class Edit_Mappings_Page extends Wordlift_Admin_Page {
 
-	/** Instance to store the registry class.
-	 * @var Mappings_Transform_Functions_Registry { @link Mappings_Transform_Functions_Registry instance}
-	 */
-	public $transform_function_registry;
-
-	/**
-	 * Edit_Mappings_Page constructor.
-	 *
-	 * @param $transform_function_registry Mappings_Transform_Functions_Registry { @link Mappings_Transform_Functions_Registry instance }
-	 */
-	public function __construct( $transform_function_registry ) {
-		parent::__construct();
-		$this->transform_function_registry = $transform_function_registry;
-	}
-
-	public function render() {
-		// Render all the settings when this method is called, because the partial page is loaded after
-		// this method.
-		// Load the UI dependencies.
-		$edit_mapping_settings = $this->get_ui_settings_array();
-		// Supply the settings to js client.
-		wp_localize_script( 'wl-mappings-edit', 'wl_edit_mappings_config', $edit_mapping_settings );
-
-		parent::render();
-	}
-
-	/**
-	 * Load the text settings needed for the edit_mappings_page.
-	 * @param array $edit_mapping_settings Key value pair of settings used by edit mappings page.
-	 *
-	 * @return array Adding text settings to the main settings array.
-	 */
-	private function load_text_settings_for_edit_mapping_page( array $edit_mapping_settings ) {
-		$edit_mapping_settings['wl_add_mapping_text']     = __( 'Add Mapping', 'wordlift' );
-		$edit_mapping_settings['wl_edit_mapping_text']    = __( 'Edit Mapping', 'wordlift' );
-		$edit_mapping_settings['wl_edit_mapping_no_item'] = __( 'Unable to find the mapping item', 'wordlift' );
-		$edit_mapping_settings['page']                    = 'wl_edit_mapping';
-		return $edit_mapping_settings;
-	}
-
-	/**
-	 * The base class {@link Wordlift_Admin_Page} will add the admin page to the WordLift menu.
-	 *
-	 * We don't want this page to be in the menu though. Therefore we override the `parent_slug` so that WordPress won't
-	 * show it there.
-	 *
-	 * @return null return null to avoid this page to be displayed in WordLift's menu.
-	 */
-	protected function get_parent_slug() {
-		return null;
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_page_title() {
-
-		return __( 'Edit Mappings', 'wordlift' );
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_menu_title() {
-
-		return __( 'Edit Mappings', 'wordlift' );
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_menu_slug() {
-
-		return 'wl_edit_mapping';
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_partial_name() {
-		return 'wordlift-admin-mappings-edit.php';
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function enqueue_scripts() {
-
-		// Enqueue the script.
-		Scripts_Helper::enqueue_based_on_wordpress_version(
-			'wl-mappings-edit',
-			plugin_dir_url( dirname( dirname( dirname( __FILE__ ) ) ) ) . 'js/dist/mappings-edit',
-			array( 'react', 'react-dom', 'wp-polyfill' ),
-			true
-		);
-
-		// Enqueue the style.
-		wp_enqueue_style(
-			'wl-mappings-edit',
-			plugin_dir_url( dirname( dirname( dirname( __FILE__ ) ) ) ) . 'js/dist/mappings-edit.css',
-			Wordlift::get_instance()->get_version()
-		);
-	}
-
-	/**
-	 * Returns field name options based on the chosen field type.
-	 * if string is returned a text field would be shown to user, if an array of options is returned
-	 * then the select box would be shown to user.
-	 *
-	 * @return array Array of the options.
-	 */
-	public static function get_all_field_name_options() {
-
-		$options = array(
-			array(
-				'field_type' => Wordlift\Mappings\Jsonld_Converter::FIELD_TYPE_TEXT_FIELD,
-				'value'      => '',
-				'label'      => __( 'Fixed Text', 'wordlift' ),
-			),
-			// @@todo maybe it makes sense to move this one as well to Wordlift/Mappings/Custom_Fields_Mappings.
-			array(
-				'field_type' => Wordlift\Mappings\Jsonld_Converter::FIELD_TYPE_CUSTOM_FIELD,
-				'value'      => '',
-				'label'      => __( 'Custom Field', 'wordlift' ),
-			),
-		);
-
-		/**
-		 * Allow 3rd parties to add field types.
-		 *
-		 * @param array An array of Field Types.
-		 *
-		 * @return array An array of Field Types.
-		 *
-		 * @since 3.25.0
-		 */
-		return apply_filters( 'wl_mappings_field_types', $options );
-	}
-
-	/**
-	 * @since 3.25.0
-	 * Load dependencies required for js client.
-	 * @return array An Array containing key value pairs of settings.
-	 */
-	public function get_ui_settings_array() {
-		// Create ui settings array to be used by js client.
-		$edit_mapping_settings                               = array();
-		$edit_mapping_settings                               = $this->load_rest_settings( $edit_mapping_settings );
-		$edit_mapping_settings = $this->load_text_settings_for_edit_mapping_page( $edit_mapping_settings );
-		$edit_mapping_settings['wl_transform_function_options'] = $this->transform_function_registry->get_options();
-		$edit_mapping_settings = $this->load_field_type_and_name_options( $edit_mapping_settings );
-		// Load logic field options.
-		$edit_mapping_settings = $this->load_logic_field_options( $edit_mapping_settings );
-		$edit_mapping_settings = $this->load_rule_field_options( $edit_mapping_settings );
-
-		return $edit_mapping_settings;
-	}
-
-	/**
-	 * Returns post type, post category, or any other post taxonomies
-	 * @return array An array of select options
-	 */
-	private static function get_post_taxonomies_and_terms() {
-		$taxonomy_options = array();
-		$term_options     = array();
-		$taxonomies       = get_object_taxonomies( 'post', 'objects' );
-
-		foreach ( $taxonomies as $taxonomy ) {
-			array_push(
-				$taxonomy_options,
-				array(
-					'label'      => $taxonomy->label,
-					'value'      => $taxonomy->name,
-					'api_source' => 'taxonomy'
-				)
-			);
-		}
-		// Post type is also included in the list of taxonomies, so get the post type and merge with options.
-		$post_type_array =  self::get_post_type_key_and_value();
-		$post_type_option = $post_type_array['post_type_option_name'];
-		// Get also the list of post types from the post_type_array.
-		$post_type_option_values = $post_type_array['post_type_option_values'];
-		// Merge the post type option and post types in the taxonomy options
-		array_push( $taxonomy_options, $post_type_option );
-		$term_options = array_merge( $term_options, $post_type_option_values );
-		return array(
-			'taxonomy_options' => $taxonomy_options,
-			'term_options' => $term_options
-		);
-	}
-
-	/**
-	 * Return post type option and post type option values.
-	 *
-	 * @return array Array of post_type_option and post_type_option_values.
-	 */
-	private static function get_post_type_key_and_value() {
-		$post_type_option_name   = array(
-			'label'      => __( 'Post type', 'wordlift' ),
-			'value'      => Wordlift\Mappings\Validators\Post_Type_Rule_Validator::POST_TYPE,
-			// Left empty since post types are provided locally.
-			'api_source' => '',
-		);
-		$post_type_option_values = array();
-		$post_types              = get_post_types(
-			array(),
-			'objects'
-		);
-		foreach ( $post_types as $post_type ) {
-			array_push(
-				$post_type_option_values,
-				array(
-					'label'        => $post_type->label,
-					'value'        => $post_type->name,
-					'parent_value' => 'post_type',
-				)
-			);
-		}
-		return array(
-			'post_type_option_name' =>  $post_type_option_name,
-			'post_type_option_values' => $post_type_option_values
-		);
-	}
-
-	/**
-	 * This function loads the equal to, not equal to operator to the edit mapping settings.
-	 *
-	 * @param array $edit_mapping_settings
-	 * @return array Loads the logic field options to the $edit_mapping_settings.
-	 */
-	private function load_logic_field_options( array $edit_mapping_settings ) {
-		$edit_mapping_settings['wl_logic_field_options'] = array(
-			array(
-				'label' => __( 'is equal to', 'wordlift' ),
-				'value' => Wordlift\Mappings\Validators\Rule_Validator::IS_EQUAL_TO,
-			),
-			array(
-				'label' => __( 'is not equal to', 'wordlift' ),
-				'value' => Wordlift\Mappings\Validators\Rule_Validator::IS_NOT_EQUAL_TO,
-			),
-		);
-
-		return $edit_mapping_settings;
-	}
-
-	/**
-	 * Validates the nonce posted by client and then assign the mapping id which should be edited.
-	 *
-	 * @param array $edit_mapping_settings
-	 * @return array Edit mapping settings array with the mapping id if the nonce is valid.
-	 */
-	private function validate_nonce_and_assign_mapping_id( array $edit_mapping_settings ) {
+    /** Instance to store the registry class.
+     * @var Mappings_Transform_Functions_Registry { @link Mappings_Transform_Functions_Registry instance}
+     */
+    public $transform_function_registry;
+
+    /**
+     * Edit_Mappings_Page constructor.
+     *
+     * @param $transform_function_registry Mappings_Transform_Functions_Registry { @link Mappings_Transform_Functions_Registry instance }
+     */
+    public function __construct( $transform_function_registry ) {
+        parent::__construct();
+        $this->transform_function_registry = $transform_function_registry;
+    }
+
+    public function render() {
+        // Render all the settings when this method is called, because the partial page is loaded after
+        // this method.
+        // Load the UI dependencies.
+        $edit_mapping_settings = $this->get_ui_settings_array();
+        // Supply the settings to js client.
+        wp_localize_script( 'wl-mappings-edit', 'wl_edit_mappings_config', $edit_mapping_settings );
+
+        parent::render();
+    }
+
+    /**
+     * Load the text settings needed for the edit_mappings_page.
+     * @param array $edit_mapping_settings Key value pair of settings used by edit mappings page.
+     *
+     * @return array Adding text settings to the main settings array.
+     */
+    private function load_text_settings_for_edit_mapping_page( array $edit_mapping_settings ) {
+        $edit_mapping_settings['wl_add_mapping_text']     = __( 'Add Mapping', 'wordlift' );
+        $edit_mapping_settings['wl_edit_mapping_text']    = __( 'Edit Mapping', 'wordlift' );
+        $edit_mapping_settings['wl_edit_mapping_no_item'] = __( 'Unable to find the mapping item', 'wordlift' );
+        $edit_mapping_settings['page']                    = 'wl_edit_mapping';
+        return $edit_mapping_settings;
+    }
+
+    /**
+     * The base class {@link Wordlift_Admin_Page} will add the admin page to the WordLift menu.
+     *
+     * We don't want this page to be in the menu though. Therefore we override the `parent_slug` so that WordPress won't
+     * show it there.
+     *
+     * @return null return null to avoid this page to be displayed in WordLift's menu.
+     */
+    protected function get_parent_slug() {
+        return null;
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_page_title() {
+
+        return __( 'Edit Mappings', 'wordlift' );
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_menu_title() {
+
+        return __( 'Edit Mappings', 'wordlift' );
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_menu_slug() {
+
+        return 'wl_edit_mapping';
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_partial_name() {
+        return 'wordlift-admin-mappings-edit.php';
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function enqueue_scripts() {
+
+        // Enqueue the script.
+        Scripts_Helper::enqueue_based_on_wordpress_version(
+            'wl-mappings-edit',
+            plugin_dir_url( dirname( dirname( dirname( __FILE__ ) ) ) ) . 'js/dist/mappings-edit',
+            array( 'react', 'react-dom', 'wp-polyfill' ),
+            true
+        );
+
+        // Enqueue the style.
+        wp_enqueue_style(
+            'wl-mappings-edit',
+            plugin_dir_url( dirname( dirname( dirname( __FILE__ ) ) ) ) . 'js/dist/mappings-edit.css',
+            Wordlift::get_instance()->get_version()
+        );
+    }
+
+    /**
+     * Returns field name options based on the chosen field type.
+     * if string is returned a text field would be shown to user, if an array of options is returned
+     * then the select box would be shown to user.
+     *
+     * @return array Array of the options.
+     */
+    public static function get_all_field_name_options() {
+
+        $options = array(
+            array(
+                'field_type' => Wordlift\Mappings\Jsonld_Converter::FIELD_TYPE_TEXT_FIELD,
+                'value'      => '',
+                'label'      => __( 'Fixed Text', 'wordlift' ),
+            ),
+            // @@todo maybe it makes sense to move this one as well to Wordlift/Mappings/Custom_Fields_Mappings.
+            array(
+                'field_type' => Wordlift\Mappings\Jsonld_Converter::FIELD_TYPE_CUSTOM_FIELD,
+                'value'      => '',
+                'label'      => __( 'Custom Field', 'wordlift' ),
+            ),
+        );
+
+        /**
+         * Allow 3rd parties to add field types.
+         *
+         * @param array An array of Field Types.
+         *
+         * @return array An array of Field Types.
+         *
+         * @since 3.25.0
+         */
+        return apply_filters( 'wl_mappings_field_types', $options );
+    }
+
+    /**
+     * @since 3.25.0
+     * Load dependencies required for js client.
+     * @return array An Array containing key value pairs of settings.
+     */
+    public function get_ui_settings_array() {
+        // Create ui settings array to be used by js client.
+        $edit_mapping_settings                               = array();
+        $edit_mapping_settings                               = $this->load_rest_settings( $edit_mapping_settings );
+        $edit_mapping_settings = $this->load_text_settings_for_edit_mapping_page( $edit_mapping_settings );
+        $edit_mapping_settings['wl_transform_function_options'] = $this->transform_function_registry->get_options();
+        $edit_mapping_settings = $this->load_field_type_and_name_options( $edit_mapping_settings );
+        // Load logic field options.
+        $edit_mapping_settings = $this->load_logic_field_options( $edit_mapping_settings );
+        $edit_mapping_settings = $this->load_rule_field_options( $edit_mapping_settings );
+
+        return $edit_mapping_settings;
+    }
+
+    /**
+     * Returns post type, post category, or any other post taxonomies
+     * @return array An array of select options
+     */
+    private static function get_post_taxonomies_and_terms() {
+        $taxonomy_options = array();
+        $term_options     = array();
+        $taxonomies       = get_object_taxonomies( 'post', 'objects' );
+
+        foreach ( $taxonomies as $taxonomy ) {
+            array_push(
+                $taxonomy_options,
+                array(
+                    'label'      => $taxonomy->label,
+                    'value'      => $taxonomy->name,
+                    'api_source' => 'taxonomy'
+                )
+            );
+        }
+        // Post type is also included in the list of taxonomies, so get the post type and merge with options.
+        $post_type_array =  self::get_post_type_key_and_value();
+        $post_type_option = $post_type_array['post_type_option_name'];
+        // Get also the list of post types from the post_type_array.
+        $post_type_option_values = $post_type_array['post_type_option_values'];
+        // Merge the post type option and post types in the taxonomy options
+        array_push( $taxonomy_options, $post_type_option );
+        $term_options = array_merge( $term_options, $post_type_option_values );
+        return array(
+            'taxonomy_options' => $taxonomy_options,
+            'term_options' => $term_options
+        );
+    }
+
+    /**
+     * Return post type option and post type option values.
+     *
+     * @return array Array of post_type_option and post_type_option_values.
+     */
+    private static function get_post_type_key_and_value() {
+        $post_type_option_name   = array(
+            'label'      => __( 'Post type', 'wordlift' ),
+            'value'      => Wordlift\Mappings\Validators\Post_Type_Rule_Validator::POST_TYPE,
+            // Left empty since post types are provided locally.
+            'api_source' => '',
+        );
+        $post_type_option_values = array();
+        $post_types              = get_post_types(
+            array(),
+            'objects'
+        );
+        foreach ( $post_types as $post_type ) {
+            array_push(
+                $post_type_option_values,
+                array(
+                    'label'        => $post_type->label,
+                    'value'        => $post_type->name,
+                    'parent_value' => 'post_type',
+                )
+            );
+        }
+        return array(
+            'post_type_option_name' =>  $post_type_option_name,
+            'post_type_option_values' => $post_type_option_values
+        );
+    }
+
+    /**
+     * This function loads the equal to, not equal to operator to the edit mapping settings.
+     *
+     * @param array $edit_mapping_settings
+     * @return array Loads the logic field options to the $edit_mapping_settings.
+     */
+    private function load_logic_field_options( array $edit_mapping_settings ) {
+        $edit_mapping_settings['wl_logic_field_options'] = array(
+            array(
+                'label' => __( 'is equal to', 'wordlift' ),
+                'value' => Wordlift\Mappings\Validators\Rule_Validator::IS_EQUAL_TO,
+            ),
+            array(
+                'label' => __( 'is not equal to', 'wordlift' ),
+                'value' => Wordlift\Mappings\Validators\Rule_Validator::IS_NOT_EQUAL_TO,
+            ),
+        );
+
+        return $edit_mapping_settings;
+    }
+
+    /**
+     * Validates the nonce posted by client and then assign the mapping id which should be edited.
+     *
+     * @param array $edit_mapping_settings
+     * @return array Edit mapping settings array with the mapping id if the nonce is valid.
+     */
+    private function validate_nonce_and_assign_mapping_id( array $edit_mapping_settings ) {
         // We verify the nonce before making to load the edit mapping page for the wl_edit_mapping_id
-		if ( isset( $_REQUEST['_wl_edit_mapping_nonce'] )
-		     && wp_verify_nonce( $_REQUEST['_wl_edit_mapping_nonce'], 'wl-edit-mapping-nonce' ) ) {
-			// We're using `INPUT_GET` here because this is a link from the UI, i.e. no POST.
-			$edit_mapping_settings['wl_edit_mapping_id'] = (int) filter_var( $_REQUEST['wl_edit_mapping_id'], FILTER_VALIDATE_INT );
-		}
-
-		return $edit_mapping_settings;
-	}
-
-	/**
-	 * Load the rest settings required for the edit_mappings js client.
-	 *
-	 * @param array $edit_mapping_settings
-	 * @return array
-	 */
-	private function load_rest_settings( array $edit_mapping_settings ) {
-		$edit_mapping_settings['rest_url']                   = get_rest_url(
-			null,
-			WL_REST_ROUTE_DEFAULT_NAMESPACE . Mappings_REST_Controller::MAPPINGS_NAMESPACE
-		);
-		$edit_mapping_settings['wl_edit_mapping_rest_nonce'] = wp_create_nonce( 'wp_rest' );
-		$edit_mapping_settings                               = $this->validate_nonce_and_assign_mapping_id( $edit_mapping_settings );
-
-		return $edit_mapping_settings;
-	}
-
-	/**
-	 * Load the rule field options in to the settings.
-	 *
-	 * @param array $edit_mapping_settings
-	 *
-	 * @return array Return the settings.
- 	 */
-	private function load_rule_field_options( array $edit_mapping_settings ) {
-		// Load the rule field options.
-		$rule_field_data                                    = self::get_post_taxonomies_and_terms();
-		$edit_mapping_settings['wl_rule_field_one_options'] = $rule_field_data['taxonomy_options'];
-		$edit_mapping_settings['wl_rule_field_two_options'] = $rule_field_data['term_options'];
-
-		/**
-		 * Allow 3rd parties to add ui options.
-		 *
-		 * @param array Array of Rule field one options where each item is in format
-		 *
-		 *  array ( 'label' => string, 'value' => string, 'api_source'=>string);
-		 *
-		 *  Leave api_source empty string to ensure didnt fetch rule field two options
-		 *  from api.
-		 *
-		 * @return array Array of Rule field one options
-		 *
-		 * @since 3.27.0
-		 */
-		$edit_mapping_settings['wl_rule_field_one_options'] = apply_filters(
-			'wl_mappings_rule_field_one_options',
-			$edit_mapping_settings['wl_rule_field_one_options']
-		);
-
-		/**
-		 * Allow 3rd parties to add rule field two options.
-		 *
-		 * @param array Array of Rule field two option where each item is in format
-		 *
-		 * array ( 'label' => string, 'value' => string, 'parent_value' => string );
-		 *
-		 * where parent_value is the value of the parent option in the rule_field_one_option.
-		 *
-		 * @since 3.27.0
-		 */
-		$edit_mapping_settings['wl_rule_field_two_options'] = apply_filters(
-			'wl_mappings_rule_field_two_options',
-			$edit_mapping_settings['wl_rule_field_two_options']
-		);
-
-		return $edit_mapping_settings;
-	}
-
-	/**
-	 * Load field type and field name options to the settings array.
-	 * @param array $edit_mapping_settings
-	 *
-	 * @return array
-	 */
-	private function load_field_type_and_name_options( array $edit_mapping_settings ) {
-		$all_field_name_options  = self::get_all_field_name_options();
-		$all_field_types_options = array_map( function ( $item ) {
-			return array(
-				'label' => $item['label'],
-				'value' => $item['field_type'],
-			);
-		}, $all_field_name_options );
-
-		$edit_mapping_settings['wl_field_type_options'] = $all_field_types_options;
-		// Add wl_edit_field_name_options.
-		$edit_mapping_settings['wl_field_name_options'] = $all_field_name_options;
-
-		return $edit_mapping_settings;
-	}
+        if ( isset( $_REQUEST['_wl_edit_mapping_nonce'] )
+             && wp_verify_nonce( $_REQUEST['_wl_edit_mapping_nonce'], 'wl-edit-mapping-nonce' ) ) {
+            // We're using `INPUT_GET` here because this is a link from the UI, i.e. no POST.
+            $edit_mapping_settings['wl_edit_mapping_id'] = (int) filter_var( $_REQUEST['wl_edit_mapping_id'], FILTER_VALIDATE_INT );
+        }
+
+        return $edit_mapping_settings;
+    }
+
+    /**
+     * Load the rest settings required for the edit_mappings js client.
+     *
+     * @param array $edit_mapping_settings
+     * @return array
+     */
+    private function load_rest_settings( array $edit_mapping_settings ) {
+        $edit_mapping_settings['rest_url']                   = get_rest_url(
+            null,
+            WL_REST_ROUTE_DEFAULT_NAMESPACE . Mappings_REST_Controller::MAPPINGS_NAMESPACE
+        );
+        $edit_mapping_settings['wl_edit_mapping_rest_nonce'] = wp_create_nonce( 'wp_rest' );
+        $edit_mapping_settings                               = $this->validate_nonce_and_assign_mapping_id( $edit_mapping_settings );
+
+        return $edit_mapping_settings;
+    }
+
+    /**
+     * Load the rule field options in to the settings.
+     *
+     * @param array $edit_mapping_settings
+     *
+     * @return array Return the settings.
+     */
+    private function load_rule_field_options( array $edit_mapping_settings ) {
+        // Load the rule field options.
+        $rule_field_data                                    = self::get_post_taxonomies_and_terms();
+        $edit_mapping_settings['wl_rule_field_one_options'] = $rule_field_data['taxonomy_options'];
+        $edit_mapping_settings['wl_rule_field_two_options'] = $rule_field_data['term_options'];
+
+        /**
+         * Allow 3rd parties to add ui options.
+         *
+         * @param array Array of Rule field one options where each item is in format
+         *
+         *  array ( 'label' => string, 'value' => string, 'api_source'=>string);
+         *
+         *  Leave api_source empty string to ensure didnt fetch rule field two options
+         *  from api.
+         *
+         * @return array Array of Rule field one options
+         *
+         * @since 3.27.0
+         */
+        $edit_mapping_settings['wl_rule_field_one_options'] = apply_filters(
+            'wl_mappings_rule_field_one_options',
+            $edit_mapping_settings['wl_rule_field_one_options']
+        );
+
+        /**
+         * Allow 3rd parties to add rule field two options.
+         *
+         * @param array Array of Rule field two option where each item is in format
+         *
+         * array ( 'label' => string, 'value' => string, 'parent_value' => string );
+         *
+         * where parent_value is the value of the parent option in the rule_field_one_option.
+         *
+         * @since 3.27.0
+         */
+        $edit_mapping_settings['wl_rule_field_two_options'] = apply_filters(
+            'wl_mappings_rule_field_two_options',
+            $edit_mapping_settings['wl_rule_field_two_options']
+        );
+
+        return $edit_mapping_settings;
+    }
+
+    /**
+     * Load field type and field name options to the settings array.
+     * @param array $edit_mapping_settings
+     *
+     * @return array
+     */
+    private function load_field_type_and_name_options( array $edit_mapping_settings ) {
+        $all_field_name_options  = self::get_all_field_name_options();
+        $all_field_types_options = array_map( function ( $item ) {
+            return array(
+                'label' => $item['label'],
+                'value' => $item['field_type'],
+            );
+        }, $all_field_name_options );
+
+        $edit_mapping_settings['wl_field_type_options'] = $all_field_types_options;
+        // Add wl_edit_field_name_options.
+        $edit_mapping_settings['wl_field_name_options'] = $all_field_name_options;
+
+        return $edit_mapping_settings;
+    }
 
 }

Spacing +43 added lines, -43 removed lines

@@ -34,7 +34,7 @@ 
 	 *
 	 * @param $transform_function_registry Mappings_Transform_Functions_Registry { @link Mappings_Transform_Functions_Registry instance }
 	 */
-	public function __construct( $transform_function_registry ) {
+	public function __construct($transform_function_registry) {
 		parent::__construct();
 		$this->transform_function_registry = $transform_function_registry;
 	}
@@ -45,7 +45,7 @@ 
 		// Load the UI dependencies.
 		$edit_mapping_settings = $this->get_ui_settings_array();
 		// Supply the settings to js client.
-		wp_localize_script( 'wl-mappings-edit', 'wl_edit_mappings_config', $edit_mapping_settings );
+		wp_localize_script('wl-mappings-edit', 'wl_edit_mappings_config', $edit_mapping_settings);
 
 		parent::render();
 	}
@@ -56,10 +56,10 @@ 
 	 *
 	 * @return array Adding text settings to the main settings array.
 	 */
-	private function load_text_settings_for_edit_mapping_page( array $edit_mapping_settings ) {
-		$edit_mapping_settings['wl_add_mapping_text']     = __( 'Add Mapping', 'wordlift' );
-		$edit_mapping_settings['wl_edit_mapping_text']    = __( 'Edit Mapping', 'wordlift' );
-		$edit_mapping_settings['wl_edit_mapping_no_item'] = __( 'Unable to find the mapping item', 'wordlift' );
+	private function load_text_settings_for_edit_mapping_page(array $edit_mapping_settings) {
+		$edit_mapping_settings['wl_add_mapping_text']     = __('Add Mapping', 'wordlift');
+		$edit_mapping_settings['wl_edit_mapping_text']    = __('Edit Mapping', 'wordlift');
+		$edit_mapping_settings['wl_edit_mapping_no_item'] = __('Unable to find the mapping item', 'wordlift');
 		$edit_mapping_settings['page']                    = 'wl_edit_mapping';
 		return $edit_mapping_settings;
 	}
@@ -81,7 +81,7 @@ 
 	 */
 	public function get_page_title() {
 
-		return __( 'Edit Mappings', 'wordlift' );
+		return __('Edit Mappings', 'wordlift');
 	}
 
 	/**
@@ -89,7 +89,7 @@ 
 	 */
 	public function get_menu_title() {
 
-		return __( 'Edit Mappings', 'wordlift' );
+		return __('Edit Mappings', 'wordlift');
 	}
 
 	/**
@@ -115,15 +115,15 @@ 
 		// Enqueue the script.
 		Scripts_Helper::enqueue_based_on_wordpress_version(
 			'wl-mappings-edit',
-			plugin_dir_url( dirname( dirname( dirname( __FILE__ ) ) ) ) . 'js/dist/mappings-edit',
-			array( 'react', 'react-dom', 'wp-polyfill' ),
+			plugin_dir_url(dirname(dirname(dirname(__FILE__)))).'js/dist/mappings-edit',
+			array('react', 'react-dom', 'wp-polyfill'),
 			true
 		);
 
 		// Enqueue the style.
 		wp_enqueue_style(
 			'wl-mappings-edit',
-			plugin_dir_url( dirname( dirname( dirname( __FILE__ ) ) ) ) . 'js/dist/mappings-edit.css',
+			plugin_dir_url(dirname(dirname(dirname(__FILE__)))).'js/dist/mappings-edit.css',
 			Wordlift::get_instance()->get_version()
 		);
 	}
@@ -141,13 +141,13 @@ 
 			array(
 				'field_type' => Wordlift\Mappings\Jsonld_Converter::FIELD_TYPE_TEXT_FIELD,
 				'value'      => '',
-				'label'      => __( 'Fixed Text', 'wordlift' ),
+				'label'      => __('Fixed Text', 'wordlift'),
 			),
 			// @@todo maybe it makes sense to move this one as well to Wordlift/Mappings/Custom_Fields_Mappings.
 			array(
 				'field_type' => Wordlift\Mappings\Jsonld_Converter::FIELD_TYPE_CUSTOM_FIELD,
 				'value'      => '',
-				'label'      => __( 'Custom Field', 'wordlift' ),
+				'label'      => __('Custom Field', 'wordlift'),
 			),
 		);
 
@@ -160,7 +160,7 @@ 
 		 *
 		 * @since 3.25.0
 		 */
-		return apply_filters( 'wl_mappings_field_types', $options );
+		return apply_filters('wl_mappings_field_types', $options);
 	}
 
 	/**
@@ -171,13 +171,13 @@ 
 	public function get_ui_settings_array() {
 		// Create ui settings array to be used by js client.
 		$edit_mapping_settings                               = array();
-		$edit_mapping_settings                               = $this->load_rest_settings( $edit_mapping_settings );
-		$edit_mapping_settings = $this->load_text_settings_for_edit_mapping_page( $edit_mapping_settings );
+		$edit_mapping_settings                               = $this->load_rest_settings($edit_mapping_settings);
+		$edit_mapping_settings = $this->load_text_settings_for_edit_mapping_page($edit_mapping_settings);
 		$edit_mapping_settings['wl_transform_function_options'] = $this->transform_function_registry->get_options();
-		$edit_mapping_settings = $this->load_field_type_and_name_options( $edit_mapping_settings );
+		$edit_mapping_settings = $this->load_field_type_and_name_options($edit_mapping_settings);
 		// Load logic field options.
-		$edit_mapping_settings = $this->load_logic_field_options( $edit_mapping_settings );
-		$edit_mapping_settings = $this->load_rule_field_options( $edit_mapping_settings );
+		$edit_mapping_settings = $this->load_logic_field_options($edit_mapping_settings);
+		$edit_mapping_settings = $this->load_rule_field_options($edit_mapping_settings);
 
 		return $edit_mapping_settings;
 	}
@@ -189,9 +189,9 @@ 
 	private static function get_post_taxonomies_and_terms() {
 		$taxonomy_options = array();
 		$term_options     = array();
-		$taxonomies       = get_object_taxonomies( 'post', 'objects' );
+		$taxonomies       = get_object_taxonomies('post', 'objects');
 
-		foreach ( $taxonomies as $taxonomy ) {
+		foreach ($taxonomies as $taxonomy) {
 			array_push(
 				$taxonomy_options,
 				array(
@@ -202,13 +202,13 @@ 
 			);
 		}
 		// Post type is also included in the list of taxonomies, so get the post type and merge with options.
-		$post_type_array =  self::get_post_type_key_and_value();
+		$post_type_array = self::get_post_type_key_and_value();
 		$post_type_option = $post_type_array['post_type_option_name'];
 		// Get also the list of post types from the post_type_array.
 		$post_type_option_values = $post_type_array['post_type_option_values'];
 		// Merge the post type option and post types in the taxonomy options
-		array_push( $taxonomy_options, $post_type_option );
-		$term_options = array_merge( $term_options, $post_type_option_values );
+		array_push($taxonomy_options, $post_type_option);
+		$term_options = array_merge($term_options, $post_type_option_values);
 		return array(
 			'taxonomy_options' => $taxonomy_options,
 			'term_options' => $term_options
@@ -221,8 +221,8 @@ 
 	 * @return array Array of post_type_option and post_type_option_values.
 	 */
 	private static function get_post_type_key_and_value() {
-		$post_type_option_name   = array(
-			'label'      => __( 'Post type', 'wordlift' ),
+		$post_type_option_name = array(
+			'label'      => __('Post type', 'wordlift'),
 			'value'      => Wordlift\Mappings\Validators\Post_Type_Rule_Validator::POST_TYPE,
 			// Left empty since post types are provided locally.
 			'api_source' => '',
@@ -232,7 +232,7 @@ 
 			array(),
 			'objects'
 		);
-		foreach ( $post_types as $post_type ) {
+		foreach ($post_types as $post_type) {
 			array_push(
 				$post_type_option_values,
 				array(
@@ -254,14 +254,14 @@ 
 	 * @param array $edit_mapping_settings
 	 * @return array Loads the logic field options to the $edit_mapping_settings.
 	 */
-	private function load_logic_field_options( array $edit_mapping_settings ) {
+	private function load_logic_field_options(array $edit_mapping_settings) {
 		$edit_mapping_settings['wl_logic_field_options'] = array(
 			array(
-				'label' => __( 'is equal to', 'wordlift' ),
+				'label' => __('is equal to', 'wordlift'),
 				'value' => Wordlift\Mappings\Validators\Rule_Validator::IS_EQUAL_TO,
 			),
 			array(
-				'label' => __( 'is not equal to', 'wordlift' ),
+				'label' => __('is not equal to', 'wordlift'),
 				'value' => Wordlift\Mappings\Validators\Rule_Validator::IS_NOT_EQUAL_TO,
 			),
 		);
@@ -275,12 +275,12 @@ 
 	 * @param array $edit_mapping_settings
 	 * @return array Edit mapping settings array with the mapping id if the nonce is valid.
 	 */
-	private function validate_nonce_and_assign_mapping_id( array $edit_mapping_settings ) {
+	private function validate_nonce_and_assign_mapping_id(array $edit_mapping_settings) {
         // We verify the nonce before making to load the edit mapping page for the wl_edit_mapping_id
-		if ( isset( $_REQUEST['_wl_edit_mapping_nonce'] )
-		     && wp_verify_nonce( $_REQUEST['_wl_edit_mapping_nonce'], 'wl-edit-mapping-nonce' ) ) {
+		if (isset($_REQUEST['_wl_edit_mapping_nonce'])
+		     && wp_verify_nonce($_REQUEST['_wl_edit_mapping_nonce'], 'wl-edit-mapping-nonce')) {
 			// We're using `INPUT_GET` here because this is a link from the UI, i.e. no POST.
-			$edit_mapping_settings['wl_edit_mapping_id'] = (int) filter_var( $_REQUEST['wl_edit_mapping_id'], FILTER_VALIDATE_INT );
+			$edit_mapping_settings['wl_edit_mapping_id'] = (int) filter_var($_REQUEST['wl_edit_mapping_id'], FILTER_VALIDATE_INT);
 		}
 
 		return $edit_mapping_settings;
@@ -292,13 +292,13 @@ 
 	 * @param array $edit_mapping_settings
 	 * @return array
 	 */
-	private function load_rest_settings( array $edit_mapping_settings ) {
-		$edit_mapping_settings['rest_url']                   = get_rest_url(
+	private function load_rest_settings(array $edit_mapping_settings) {
+		$edit_mapping_settings['rest_url'] = get_rest_url(
 			null,
-			WL_REST_ROUTE_DEFAULT_NAMESPACE . Mappings_REST_Controller::MAPPINGS_NAMESPACE
+			WL_REST_ROUTE_DEFAULT_NAMESPACE.Mappings_REST_Controller::MAPPINGS_NAMESPACE
 		);
-		$edit_mapping_settings['wl_edit_mapping_rest_nonce'] = wp_create_nonce( 'wp_rest' );
-		$edit_mapping_settings                               = $this->validate_nonce_and_assign_mapping_id( $edit_mapping_settings );
+		$edit_mapping_settings['wl_edit_mapping_rest_nonce'] = wp_create_nonce('wp_rest');
+		$edit_mapping_settings                               = $this->validate_nonce_and_assign_mapping_id($edit_mapping_settings);
 
 		return $edit_mapping_settings;
 	}
@@ -310,7 +310,7 @@ 
 	 *
 	 * @return array Return the settings.
  	 */
-	private function load_rule_field_options( array $edit_mapping_settings ) {
+	private function load_rule_field_options(array $edit_mapping_settings) {
 		// Load the rule field options.
 		$rule_field_data                                    = self::get_post_taxonomies_and_terms();
 		$edit_mapping_settings['wl_rule_field_one_options'] = $rule_field_data['taxonomy_options'];
@@ -360,14 +360,14 @@ 
 	 *
 	 * @return array
 	 */
-	private function load_field_type_and_name_options( array $edit_mapping_settings ) {
+	private function load_field_type_and_name_options(array $edit_mapping_settings) {
 		$all_field_name_options  = self::get_all_field_name_options();
-		$all_field_types_options = array_map( function ( $item ) {
+		$all_field_types_options = array_map(function($item) {
 			return array(
 				'label' => $item['label'],
 				'value' => $item['field_type'],
 			);
-		}, $all_field_name_options );
+		}, $all_field_name_options);
 
 		$edit_mapping_settings['wl_field_type_options'] = $all_field_types_options;
 		// Add wl_edit_field_name_options.

src/wordlift/mappings/class-taxonomy-option.php

Indentation +39 added lines, -39 removed lines

@@ -14,58 +14,58 @@
  */
 class Taxonomy_Option {
 
-	/**
-	 * This value is used in rule field one and rule field two
-	 * options are linked to one by this value
-	 */
-	const PARENT_VALUE = 'taxonomy';
+    /**
+     * This value is used in rule field one and rule field two
+     * options are linked to one by this value
+     */
+    const PARENT_VALUE = 'taxonomy';
 
-	public function add_taxonomy_option() {
-		$this->add_rule_field_one_options();
-		$this->add_rule_field_two_options();
-	}
+    public function add_taxonomy_option() {
+        $this->add_rule_field_one_options();
+        $this->add_rule_field_two_options();
+    }
 
-	private function add_rule_field_one_options() {
+    private function add_rule_field_one_options() {
 
-		add_filter( 'wl_mappings_rule_field_one_options', function ( $rule_field_one_options ) {
+        add_filter( 'wl_mappings_rule_field_one_options', function ( $rule_field_one_options ) {
 
-			$rule_field_one_options[] = array(
-				'label'      => __( 'Taxonomy', 'wordlift' ),
-				'value'      => Taxonomy_Option::PARENT_VALUE,
-				// Left empty since these values are provided locally, not needed to be fetched from
-				// api.
-				'api_source' => ''
-			);
+            $rule_field_one_options[] = array(
+                'label'      => __( 'Taxonomy', 'wordlift' ),
+                'value'      => Taxonomy_Option::PARENT_VALUE,
+                // Left empty since these values are provided locally, not needed to be fetched from
+                // api.
+                'api_source' => ''
+            );
 
-			return $rule_field_one_options;
+            return $rule_field_one_options;
 
-		} );
-	}
+        } );
+    }
 
-	private function add_rule_field_two_options() {
+    private function add_rule_field_two_options() {
 
-		add_filter( 'wl_mappings_rule_field_two_options', function ( $rule_field_two_options ) {
+        add_filter( 'wl_mappings_rule_field_two_options', function ( $rule_field_two_options ) {
 
-			$taxonomies = get_object_taxonomies( 'post', 'objects' );
+            $taxonomies = get_object_taxonomies( 'post', 'objects' );
 
-			$taxonomy_options = array();
+            $taxonomy_options = array();
 
-			foreach ( $taxonomies as $item ) {
-				/**
-				 * $item Taxonomy
-				 */
-				$taxonomy_options[] = array(
-					'label'        => __( $item->label, 'wordlift' ),
-					'value'        => $item->name,
-					// The value of parent option on rule field one.
-					'parent_value' => Taxonomy_Option::PARENT_VALUE
-				);
-			}
+            foreach ( $taxonomies as $item ) {
+                /**
+                 * $item Taxonomy
+                 */
+                $taxonomy_options[] = array(
+                    'label'        => __( $item->label, 'wordlift' ),
+                    'value'        => $item->name,
+                    // The value of parent option on rule field one.
+                    'parent_value' => Taxonomy_Option::PARENT_VALUE
+                );
+            }
 
-			return array_merge( $rule_field_two_options, $taxonomy_options );
+            return array_merge( $rule_field_two_options, $taxonomy_options );
 
-		} );
+        } );
 
-	}
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -27,10 +27,10 @@ 
 
 	private function add_rule_field_one_options() {
 
-		add_filter( 'wl_mappings_rule_field_one_options', function ( $rule_field_one_options ) {
+		add_filter('wl_mappings_rule_field_one_options', function($rule_field_one_options) {
 
 			$rule_field_one_options[] = array(
-				'label'      => __( 'Taxonomy', 'wordlift' ),
+				'label'      => __('Taxonomy', 'wordlift'),
 				'value'      => Taxonomy_Option::PARENT_VALUE,
 				// Left empty since these values are provided locally, not needed to be fetched from
 				// api.
@@ -44,25 +44,25 @@ 
 
 	private function add_rule_field_two_options() {
 
-		add_filter( 'wl_mappings_rule_field_two_options', function ( $rule_field_two_options ) {
+		add_filter('wl_mappings_rule_field_two_options', function($rule_field_two_options) {
 
-			$taxonomies = get_object_taxonomies( 'post', 'objects' );
+			$taxonomies = get_object_taxonomies('post', 'objects');
 
 			$taxonomy_options = array();
 
-			foreach ( $taxonomies as $item ) {
+			foreach ($taxonomies as $item) {
 				/**
 				 * $item Taxonomy
 				 */
 				$taxonomy_options[] = array(
-					'label'        => __( $item->label, 'wordlift' ),
+					'label'        => __($item->label, 'wordlift'),
 					'value'        => $item->name,
 					// The value of parent option on rule field one.
 					'parent_value' => Taxonomy_Option::PARENT_VALUE
 				);
 			}
 
-			return array_merge( $rule_field_two_options, $taxonomy_options );
+			return array_merge($rule_field_two_options, $taxonomy_options);
 
 		} );
 

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

Indentation +42 added lines, -42 removed lines

@@ -15,56 +15,56 @@
  */
 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( $post_id, $operator, $operand_1, $taxonomy ) {
-		/*
+    public function is_valid( $post_id, $operator, $operand_1, $taxonomy ) {
+        /*
 		 * post_id should not be used since we validate this for term pages.
 		 */
-		$current_term = get_queried_object();
-		// 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 );
-		}
+        $current_term = get_queried_object();
+        // 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 +12 added lines, -12 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,42 +28,42 @@ 
 	 *
 	 * @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( $post_id, $operator, $operand_1, $taxonomy ) {
+	public function is_valid($post_id, $operator, $operand_1, $taxonomy) {
 		/*
 		 * post_id should not be used since we validate this for term pages.
 		 */
 		$current_term = get_queried_object();
 		// 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 ) {
+		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-rule-groups-validator.php

Indentation +40 added lines, -40 removed lines

@@ -19,54 +19,54 @@
  */
 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 $post_id The post id.
-	 * @param array $rule_groups An array of rules' groups.
-	 *
-	 * @return bool Whether the post passes at least one rule group.
-	 */
-	public function is_valid( $post_id, $rule_groups ) {
+    /**
+     * Check whether the specified post passes at least one group of rules.
+     *
+     * @param int $post_id The post id.
+     * @param array $rule_groups An array of rules' groups.
+     *
+     * @return bool Whether the post passes at least one rule group.
+     */
+    public function is_valid( $post_id, $rule_groups ) {
 
-		// 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( $post_id, $rule_logic_field, $rule_field_one, $rule_field_two ) ) {
-					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( $post_id, $rule_logic_field, $rule_field_one, $rule_field_two ) ) {
+                    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;
+    }
 
 }

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

Indentation +188 added lines, -188 removed lines

@@ -18,193 +18,193 @@
  * @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 );
-		add_filter( 'wl_term_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               = Data_Source_Factory::get_instance()->get_data( $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;
-	}
-
-	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.
+     * 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 );
+        add_filter( 'wl_term_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               = Data_Source_Factory::get_instance()->get_data( $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;
+    }
+
+    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 +39 added lines, -39 removed lines

@@ -49,15 +49,15 @@ 
 	 * @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_term_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);
+		add_filter('wl_term_jsonld_array', array($this, 'wl_post_jsonld_array'), 11, 3);
 	}
 
 	/**
@@ -77,13 +77,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,
 		);
 	}
@@ -98,27 +98,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;
 	}
@@ -133,16 +133,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               = Data_Source_Factory::get_instance()->get_data( $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               = Data_Source_Factory::get_instance()->get_data($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;
@@ -156,17 +156,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;
 
 			/**
@@ -174,34 +174,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;