insideout10 / wordlift-plugin

src/wordlift/class-assertions.php

Indentation +79 added lines, -79 removed lines

@@ -15,84 +15,84 @@
 
 class Assertions {
 
-	/**
-	 * Asserts that the provided value is of the specified type.
-	 *
-	 * @param mixed  $value The value to test.
-	 * @param string $type The expected type.
-	 *
-	 * @throws Exception when an error occurs.
-	 */
-	public static function assert_of_type( $value, $type ) {
-
-		// Check for nulls.
-		if ( null === $value ) {
-			throw new Exception( 'Value can`t be null.' );
-		}
-
-		// Check for type.
-		if ( get_class( $value ) !== $type ) {
-			throw new Exception( "Value must be a $type." );
-		}
-
-	}
-
-	/**
-	 * @throws Exception when the argument is not a string.
-	 */
-	public static function is_string( $arg, $message = 'Value must be a string' ) {
-		if ( ! is_string( $arg ) ) {
-			throw new Exception( $message );
-		}
-	}
-
-	/**
-	 * @throws Exception when actual doesn't match expected.
-	 */
-	public static function equals( $actual, $expected, $message = 'Values do not match' ) {
-		if ( $actual !== $expected ) {
-			throw new Exception( $message );
-		}
-	}
-
-	/**
-	 * @throws Exception when the value doesn't match the pattern.
-	 */
-	public static function matches( $value, $pattern, $message = "Value doesn't match" ) {
-		if ( 1 !== preg_match( $pattern, $value ) ) {
-			throw new Exception( $message );
-		}
-
-	}
-
-	/**
-	 * @throws Exception when the value doesn't start with the provided scope.
-	 */
-	public static function starts_with( $value, $scope, $message = "Value doesn't start with provided scope" ) {
-		if ( 0 !== strpos( $value, $scope ) ) {
-			throw new Exception( $message );
-		}
-	}
-
-	/**
-	 * @throws Exception when the value is not of the specified type.
-	 */
-	public static function is_a( $value, $type, $message = 'Value is not of the required type' ) {
-		if ( ! is_a( $value, $type ) ) {
-			throw new Exception( $message );
-		}
-	}
-
-	public static function is_set( $value, $message = 'Value is not set' ) {
-		if ( ! isset( $value ) ) {
-			throw new Exception( $message );
-		}
-	}
-
-	public static function not_empty( $value, $message = "Value can't be empty" ) {
-		if ( empty( $value ) ) {
-			throw new Exception( $message );
-		}
-	}
+    /**
+     * Asserts that the provided value is of the specified type.
+     *
+     * @param mixed  $value The value to test.
+     * @param string $type The expected type.
+     *
+     * @throws Exception when an error occurs.
+     */
+    public static function assert_of_type( $value, $type ) {
+
+        // Check for nulls.
+        if ( null === $value ) {
+            throw new Exception( 'Value can`t be null.' );
+        }
+
+        // Check for type.
+        if ( get_class( $value ) !== $type ) {
+            throw new Exception( "Value must be a $type." );
+        }
+
+    }
+
+    /**
+     * @throws Exception when the argument is not a string.
+     */
+    public static function is_string( $arg, $message = 'Value must be a string' ) {
+        if ( ! is_string( $arg ) ) {
+            throw new Exception( $message );
+        }
+    }
+
+    /**
+     * @throws Exception when actual doesn't match expected.
+     */
+    public static function equals( $actual, $expected, $message = 'Values do not match' ) {
+        if ( $actual !== $expected ) {
+            throw new Exception( $message );
+        }
+    }
+
+    /**
+     * @throws Exception when the value doesn't match the pattern.
+     */
+    public static function matches( $value, $pattern, $message = "Value doesn't match" ) {
+        if ( 1 !== preg_match( $pattern, $value ) ) {
+            throw new Exception( $message );
+        }
+
+    }
+
+    /**
+     * @throws Exception when the value doesn't start with the provided scope.
+     */
+    public static function starts_with( $value, $scope, $message = "Value doesn't start with provided scope" ) {
+        if ( 0 !== strpos( $value, $scope ) ) {
+            throw new Exception( $message );
+        }
+    }
+
+    /**
+     * @throws Exception when the value is not of the specified type.
+     */
+    public static function is_a( $value, $type, $message = 'Value is not of the required type' ) {
+        if ( ! is_a( $value, $type ) ) {
+            throw new Exception( $message );
+        }
+    }
+
+    public static function is_set( $value, $message = 'Value is not set' ) {
+        if ( ! isset( $value ) ) {
+            throw new Exception( $message );
+        }
+    }
+
+    public static function not_empty( $value, $message = "Value can't be empty" ) {
+        if ( empty( $value ) ) {
+            throw new Exception( $message );
+        }
+    }
 
 }

Spacing +26 added lines, -26 removed lines

@@ -23,16 +23,16 @@ 
 	 *
 	 * @throws Exception when an error occurs.
 	 */
-	public static function assert_of_type( $value, $type ) {
+	public static function assert_of_type($value, $type) {
 
 		// Check for nulls.
-		if ( null === $value ) {
-			throw new Exception( 'Value can`t be null.' );
+		if (null === $value) {
+			throw new Exception('Value can`t be null.');
 		}
 
 		// Check for type.
-		if ( get_class( $value ) !== $type ) {
-			throw new Exception( "Value must be a $type." );
+		if (get_class($value) !== $type) {
+			throw new Exception("Value must be a $type.");
 		}
 
 	}
@@ -40,27 +40,27 @@ 
 	/**
 	 * @throws Exception when the argument is not a string.
 	 */
-	public static function is_string( $arg, $message = 'Value must be a string' ) {
-		if ( ! is_string( $arg ) ) {
-			throw new Exception( $message );
+	public static function is_string($arg, $message = 'Value must be a string') {
+		if ( ! is_string($arg)) {
+			throw new Exception($message);
 		}
 	}
 
 	/**
 	 * @throws Exception when actual doesn't match expected.
 	 */
-	public static function equals( $actual, $expected, $message = 'Values do not match' ) {
-		if ( $actual !== $expected ) {
-			throw new Exception( $message );
+	public static function equals($actual, $expected, $message = 'Values do not match') {
+		if ($actual !== $expected) {
+			throw new Exception($message);
 		}
 	}
 
 	/**
 	 * @throws Exception when the value doesn't match the pattern.
 	 */
-	public static function matches( $value, $pattern, $message = "Value doesn't match" ) {
-		if ( 1 !== preg_match( $pattern, $value ) ) {
-			throw new Exception( $message );
+	public static function matches($value, $pattern, $message = "Value doesn't match") {
+		if (1 !== preg_match($pattern, $value)) {
+			throw new Exception($message);
 		}
 
 	}
@@ -68,30 +68,30 @@ 
 	/**
 	 * @throws Exception when the value doesn't start with the provided scope.
 	 */
-	public static function starts_with( $value, $scope, $message = "Value doesn't start with provided scope" ) {
-		if ( 0 !== strpos( $value, $scope ) ) {
-			throw new Exception( $message );
+	public static function starts_with($value, $scope, $message = "Value doesn't start with provided scope") {
+		if (0 !== strpos($value, $scope)) {
+			throw new Exception($message);
 		}
 	}
 
 	/**
 	 * @throws Exception when the value is not of the specified type.
 	 */
-	public static function is_a( $value, $type, $message = 'Value is not of the required type' ) {
-		if ( ! is_a( $value, $type ) ) {
-			throw new Exception( $message );
+	public static function is_a($value, $type, $message = 'Value is not of the required type') {
+		if ( ! is_a($value, $type)) {
+			throw new Exception($message);
 		}
 	}
 
-	public static function is_set( $value, $message = 'Value is not set' ) {
-		if ( ! isset( $value ) ) {
-			throw new Exception( $message );
+	public static function is_set($value, $message = 'Value is not set') {
+		if ( ! isset($value)) {
+			throw new Exception($message);
 		}
 	}
 
-	public static function not_empty( $value, $message = "Value can't be empty" ) {
-		if ( empty( $value ) ) {
-			throw new Exception( $message );
+	public static function not_empty($value, $message = "Value can't be empty") {
+		if (empty($value)) {
+			throw new Exception($message);
 		}
 	}
 

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

Indentation +64 added lines, -64 removed lines

@@ -22,69 +22,69 @@
  */
 class Admin_Mappings_Page extends Wordlift_Admin_Page {
 
-	/**
-	 * Provides script and js global values used by react component.
-	 */
-	public static function provide_ui_dependencies() {
-		// Create ui settings array to be used by js client.
-		$mapping_settings                          = array();
-		$mapping_settings['rest_url']              = get_rest_url(
-			null,
-			WL_REST_ROUTE_DEFAULT_NAMESPACE . Mappings_REST_Controller::MAPPINGS_NAMESPACE
-		);
-		$mapping_settings['wl_mapping_nonce']      = wp_create_nonce( 'wp_rest' );
-		$mapping_settings['wl_edit_mapping_nonce'] = wp_create_nonce( 'wl-edit-mapping-nonce' );
-		wp_localize_script( 'wl-mappings-admin', 'wlMappingsConfig', $mapping_settings );
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_page_title() {
-
-		return __( 'Mappings', 'wordlift' );
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_menu_title() {
-
-		return __( 'Mappings', 'wordlift' );
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_menu_slug() {
-
-		return 'wl_mappings_admin';
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_partial_name() {
-
-		return 'wordlift-admin-mappings-admin.php';
-	}
-
-	public function enqueue_scripts() {
-
-		Scripts_Helper::enqueue_based_on_wordpress_version(
-			'wl-mappings-admin',
-			plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . 'js/dist/mappings',
-			array( 'react', 'react-dom', 'wp-polyfill' ),
-			true
-		);
-
-		wp_enqueue_style(
-			'wl-mappings-admin',
-			plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . 'js/dist/mappings.css',
-			array(),
-			WORDLIFT_VERSION
-		);
-		self::provide_ui_dependencies();
-	}
+    /**
+     * Provides script and js global values used by react component.
+     */
+    public static function provide_ui_dependencies() {
+        // Create ui settings array to be used by js client.
+        $mapping_settings                          = array();
+        $mapping_settings['rest_url']              = get_rest_url(
+            null,
+            WL_REST_ROUTE_DEFAULT_NAMESPACE . Mappings_REST_Controller::MAPPINGS_NAMESPACE
+        );
+        $mapping_settings['wl_mapping_nonce']      = wp_create_nonce( 'wp_rest' );
+        $mapping_settings['wl_edit_mapping_nonce'] = wp_create_nonce( 'wl-edit-mapping-nonce' );
+        wp_localize_script( 'wl-mappings-admin', 'wlMappingsConfig', $mapping_settings );
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_page_title() {
+
+        return __( 'Mappings', 'wordlift' );
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_menu_title() {
+
+        return __( 'Mappings', 'wordlift' );
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_menu_slug() {
+
+        return 'wl_mappings_admin';
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_partial_name() {
+
+        return 'wordlift-admin-mappings-admin.php';
+    }
+
+    public function enqueue_scripts() {
+
+        Scripts_Helper::enqueue_based_on_wordpress_version(
+            'wl-mappings-admin',
+            plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . 'js/dist/mappings',
+            array( 'react', 'react-dom', 'wp-polyfill' ),
+            true
+        );
+
+        wp_enqueue_style(
+            'wl-mappings-admin',
+            plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . 'js/dist/mappings.css',
+            array(),
+            WORDLIFT_VERSION
+        );
+        self::provide_ui_dependencies();
+    }
 
 }

Spacing +9 added lines, -9 removed lines

@@ -30,11 +30,11 @@ 
 		$mapping_settings                          = array();
 		$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
 		);
-		$mapping_settings['wl_mapping_nonce']      = wp_create_nonce( 'wp_rest' );
-		$mapping_settings['wl_edit_mapping_nonce'] = wp_create_nonce( 'wl-edit-mapping-nonce' );
-		wp_localize_script( 'wl-mappings-admin', 'wlMappingsConfig', $mapping_settings );
+		$mapping_settings['wl_mapping_nonce']      = wp_create_nonce('wp_rest');
+		$mapping_settings['wl_edit_mapping_nonce'] = wp_create_nonce('wl-edit-mapping-nonce');
+		wp_localize_script('wl-mappings-admin', 'wlMappingsConfig', $mapping_settings);
 	}
 
 	/**
@@ -42,7 +42,7 @@ 
 	 */
 	public function get_page_title() {
 
-		return __( 'Mappings', 'wordlift' );
+		return __('Mappings', 'wordlift');
 	}
 
 	/**
@@ -50,7 +50,7 @@ 
 	 */
 	public function get_menu_title() {
 
-		return __( 'Mappings', 'wordlift' );
+		return __('Mappings', 'wordlift');
 	}
 
 	/**
@@ -73,14 +73,14 @@ 
 
 		Scripts_Helper::enqueue_based_on_wordpress_version(
 			'wl-mappings-admin',
-			plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . 'js/dist/mappings',
-			array( 'react', 'react-dom', 'wp-polyfill' ),
+			plugin_dir_url(dirname(dirname(__DIR__))).'js/dist/mappings',
+			array('react', 'react-dom', 'wp-polyfill'),
 			true
 		);
 
 		wp_enqueue_style(
 			'wl-mappings-admin',
-			plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . 'js/dist/mappings.css',
+			plugin_dir_url(dirname(dirname(__DIR__))).'js/dist/mappings.css',
 			array(),
 			WORDLIFT_VERSION
 		);

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

Indentation +392 added lines, -392 removed lines

@@ -24,397 +24,397 @@
  */
 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( __DIR__ ) ) ) . '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( __DIR__ ) ) ) . 'js/dist/mappings-edit.css',
-			array(),
-			WORDLIFT_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 );
-	}
-
-	/**
-	 * @return array An Array containing key value pairs of settings.
-	 * @since 3.25.0
-	 * Load dependencies required for js client.
-	 */
-	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'];
-
-		$post_taxonomy_array  = self::get_post_taxonomy_key_and_value();
-		$post_taxonomy_option = $post_taxonomy_array['post_taxonomy_option_name'];
-
-		// Merge the post type option and post types in the taxonomy options
-		array_push( $taxonomy_options, $post_type_option, $post_taxonomy_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,
-		);
-	}
-
-	/**
-	 * 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_taxonomy_key_and_value() {
-
-		$post_taxonomy_option_name   = array(
-			'label'      => __( 'Post Taxonomy', 'wordlift' ),
-			'value'      => Wordlift\Mappings\Validators\Post_Taxonomy_Term_Rule_Validator::POST_TAXONOMY,
-			'api_source' => 'post_taxonomy',
-		);
-		$post_taxonomy_option_values = array();
-
-		return array(
-			'post_taxonomy_option_name'   => $post_taxonomy_option_name,
-			'post_taxonomy_option_values' => $post_taxonomy_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( sanitize_text_field( wp_unslash( $_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.
-			$mapping_id                                  = isset( $_REQUEST['wl_edit_mapping_id'] ) ?
-				(int) filter_var( sanitize_text_field( wp_unslash( $_REQUEST['wl_edit_mapping_id'] ) ), FILTER_VALIDATE_INT ) : 0;
-			$edit_mapping_settings['wl_edit_mapping_id'] = $mapping_id;
-		}
-
-		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;
-	}
+    /** 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( __DIR__ ) ) ) . '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( __DIR__ ) ) ) . 'js/dist/mappings-edit.css',
+            array(),
+            WORDLIFT_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 );
+    }
+
+    /**
+     * @return array An Array containing key value pairs of settings.
+     * @since 3.25.0
+     * Load dependencies required for js client.
+     */
+    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'];
+
+        $post_taxonomy_array  = self::get_post_taxonomy_key_and_value();
+        $post_taxonomy_option = $post_taxonomy_array['post_taxonomy_option_name'];
+
+        // Merge the post type option and post types in the taxonomy options
+        array_push( $taxonomy_options, $post_type_option, $post_taxonomy_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,
+        );
+    }
+
+    /**
+     * 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_taxonomy_key_and_value() {
+
+        $post_taxonomy_option_name   = array(
+            'label'      => __( 'Post Taxonomy', 'wordlift' ),
+            'value'      => Wordlift\Mappings\Validators\Post_Taxonomy_Term_Rule_Validator::POST_TAXONOMY,
+            'api_source' => 'post_taxonomy',
+        );
+        $post_taxonomy_option_values = array();
+
+        return array(
+            'post_taxonomy_option_name'   => $post_taxonomy_option_name,
+            'post_taxonomy_option_values' => $post_taxonomy_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( sanitize_text_field( wp_unslash( $_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.
+            $mapping_id                                  = isset( $_REQUEST['wl_edit_mapping_id'] ) ?
+                (int) filter_var( sanitize_text_field( wp_unslash( $_REQUEST['wl_edit_mapping_id'] ) ), FILTER_VALIDATE_INT ) : 0;
+            $edit_mapping_settings['wl_edit_mapping_id'] = $mapping_id;
+        }
+
+        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 +44 added lines, -44 removed lines

@@ -35,7 +35,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;
 	}
@@ -46,7 +46,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();
 	}
@@ -58,10 +58,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;
@@ -84,7 +84,7 @@ 
 	 */
 	public function get_page_title() {
 
-		return __( 'Edit Mappings', 'wordlift' );
+		return __('Edit Mappings', 'wordlift');
 	}
 
 	/**
@@ -92,7 +92,7 @@ 
 	 */
 	public function get_menu_title() {
 
-		return __( 'Edit Mappings', 'wordlift' );
+		return __('Edit Mappings', 'wordlift');
 	}
 
 	/**
@@ -118,15 +118,15 @@ 
 		// Enqueue the script.
 		Scripts_Helper::enqueue_based_on_wordpress_version(
 			'wl-mappings-edit',
-			plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . 'js/dist/mappings-edit',
-			array( 'react', 'react-dom', 'wp-polyfill' ),
+			plugin_dir_url(dirname(dirname(__DIR__))).'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( __DIR__ ) ) ) . 'js/dist/mappings-edit.css',
+			plugin_dir_url(dirname(dirname(__DIR__))).'js/dist/mappings-edit.css',
 			array(),
 			WORDLIFT_VERSION
 		);
@@ -145,13 +145,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'),
 			),
 		);
 
@@ -164,7 +164,7 @@ 
 		 *
 		 * @since 3.25.0
 		 */
-		return apply_filters( 'wl_mappings_field_types', $options );
+		return apply_filters('wl_mappings_field_types', $options);
 	}
 
 	/**
@@ -175,13 +175,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;
 	}
@@ -194,9 +194,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(
@@ -216,8 +216,8 @@ 
 		$post_taxonomy_option = $post_taxonomy_array['post_taxonomy_option_name'];
 
 		// Merge the post type option and post types in the taxonomy options
-		array_push( $taxonomy_options, $post_type_option, $post_taxonomy_option );
-		$term_options = array_merge( $term_options, $post_type_option_values );
+		array_push($taxonomy_options, $post_type_option, $post_taxonomy_option);
+		$term_options = array_merge($term_options, $post_type_option_values);
 
 		return array(
 			'taxonomy_options' => $taxonomy_options,
@@ -231,8 +231,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' => '',
@@ -243,7 +243,7 @@ 
 			'objects'
 		);
 
-		foreach ( $post_types as $post_type ) {
+		foreach ($post_types as $post_type) {
 			array_push(
 				$post_type_option_values,
 				array(
@@ -267,8 +267,8 @@ 
 	 */
 	private static function get_post_taxonomy_key_and_value() {
 
-		$post_taxonomy_option_name   = array(
-			'label'      => __( 'Post Taxonomy', 'wordlift' ),
+		$post_taxonomy_option_name = array(
+			'label'      => __('Post Taxonomy', 'wordlift'),
 			'value'      => Wordlift\Mappings\Validators\Post_Taxonomy_Term_Rule_Validator::POST_TAXONOMY,
 			'api_source' => 'post_taxonomy',
 		);
@@ -287,14 +287,14 @@ 
 	 *
 	 * @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,
 			),
 		);
@@ -309,13 +309,13 @@ 
 	 *
 	 * @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( sanitize_text_field( wp_unslash( $_REQUEST['_wl_edit_mapping_nonce'] ) ), 'wl-edit-mapping-nonce' ) ) {
+		if (isset($_REQUEST['_wl_edit_mapping_nonce'])
+			 && wp_verify_nonce(sanitize_text_field(wp_unslash($_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.
-			$mapping_id                                  = isset( $_REQUEST['wl_edit_mapping_id'] ) ?
-				(int) filter_var( sanitize_text_field( wp_unslash( $_REQUEST['wl_edit_mapping_id'] ) ), FILTER_VALIDATE_INT ) : 0;
+			$mapping_id                                  = isset($_REQUEST['wl_edit_mapping_id']) ?
+				(int) filter_var(sanitize_text_field(wp_unslash($_REQUEST['wl_edit_mapping_id'])), FILTER_VALIDATE_INT) : 0;
 			$edit_mapping_settings['wl_edit_mapping_id'] = $mapping_id;
 		}
 
@@ -329,13 +329,13 @@ 
 	 *
 	 * @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;
 	}
@@ -347,7 +347,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'];
@@ -398,10 +398,10 @@ 
 	 *
 	 * @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 ) {
+			function($item) {
 				return array(
 					'label' => $item['label'],
 					'value' => $item['field_type'],

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

Indentation +65 added lines, -65 removed lines

@@ -17,70 +17,70 @@
  * @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 ) {
-		/**
-		 * If this is not done then this will validate for term
-		 * which causes a bug, so this rule validator would return false
-		 * when the current thing is not a post.
-		 */
-		if ( Jsonld_Converter::POST !== $type ) {
-			return false;
-		}
-		// 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 ) {
+        /**
+         * If this is not done then this will validate for term
+         * which causes a bug, so this rule validator would return false
+         * when the current thing is not a post.
+         */
+        if ( Jsonld_Converter::POST !== $type ) {
+            return false;
+        }
+        // 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 +8 added lines, -8 removed lines

@@ -30,7 +30,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'));
 
 	}
 
@@ -41,9 +41,9 @@ 
 	 *
 	 * @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;
+		$value[self::POST_TYPE] = $this;
 
 		return $value;
 	}
@@ -52,25 +52,25 @@ 
 	 * {@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) {
 		/**
 		 * If this is not done then this will validate for term
 		 * which causes a bug, so this rule validator would return false
 		 * when the current thing is not a post.
 		 */
-		if ( Jsonld_Converter::POST !== $type ) {
+		if (Jsonld_Converter::POST !== $type) {
 			return false;
 		}
 		// 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-rule-validators-registry.php

Indentation +38 added lines, -38 removed lines

@@ -18,43 +18,43 @@
  */
 class Rule_Validators_Registry {
 
-	/**
-	 * An array of {@link Rule_Validator}s.
-	 *
-	 * @var array An array of {@link Rule_Validator}s.
-	 */
-	private $rule_validators;
-
-	/**
-	 * Rule_Validators_Registry constructor.
-	 *
-	 * @param Rule_Validator $default The default rule validator.
-	 *
-	 * @throws Exception throws an exception if an invalid validator has been provided.
-	 */
-	public function __construct( $default ) {
-
-		// Check that a valid validator has been provided.
-		if ( ! ( $default instanceof Rule_Validator ) ) {
-			throw new Exception( 'An invalid Rule_Validator was provided as default validator.' );
-		}
-
-		// Allow 3rd parties to register other validators.
-		$this->rule_validators = apply_filters( 'wl_mappings_rule_validators', array( '__default__' => $default ) );
-
-	}
-
-	/**
-	 * Get a rule validator by its key.
-	 *
-	 * @param string $key A key uniquely identifying a validator.
-	 *
-	 * @return Rule_Validator A {@link Rule_Validator} instance or the default one when not found.
-	 */
-	public function get_rule_validator( $key ) {
-
-		return isset( $this->rule_validators[ $key ] )
-			? $this->rule_validators[ $key ] : $this->rule_validators['__default__'];
-	}
+    /**
+     * An array of {@link Rule_Validator}s.
+     *
+     * @var array An array of {@link Rule_Validator}s.
+     */
+    private $rule_validators;
+
+    /**
+     * Rule_Validators_Registry constructor.
+     *
+     * @param Rule_Validator $default The default rule validator.
+     *
+     * @throws Exception throws an exception if an invalid validator has been provided.
+     */
+    public function __construct( $default ) {
+
+        // Check that a valid validator has been provided.
+        if ( ! ( $default instanceof Rule_Validator ) ) {
+            throw new Exception( 'An invalid Rule_Validator was provided as default validator.' );
+        }
+
+        // Allow 3rd parties to register other validators.
+        $this->rule_validators = apply_filters( 'wl_mappings_rule_validators', array( '__default__' => $default ) );
+
+    }
+
+    /**
+     * Get a rule validator by its key.
+     *
+     * @param string $key A key uniquely identifying a validator.
+     *
+     * @return Rule_Validator A {@link Rule_Validator} instance or the default one when not found.
+     */
+    public function get_rule_validator( $key ) {
+
+        return isset( $this->rule_validators[ $key ] )
+            ? $this->rule_validators[ $key ] : $this->rule_validators['__default__'];
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -32,15 +32,15 @@ 
 	 *
 	 * @throws Exception throws an exception if an invalid validator has been provided.
 	 */
-	public function __construct( $default ) {
+	public function __construct($default) {
 
 		// Check that a valid validator has been provided.
-		if ( ! ( $default instanceof Rule_Validator ) ) {
-			throw new Exception( 'An invalid Rule_Validator was provided as default validator.' );
+		if ( ! ($default instanceof Rule_Validator)) {
+			throw new Exception('An invalid Rule_Validator was provided as default validator.');
 		}
 
 		// Allow 3rd parties to register other validators.
-		$this->rule_validators = apply_filters( 'wl_mappings_rule_validators', array( '__default__' => $default ) );
+		$this->rule_validators = apply_filters('wl_mappings_rule_validators', array('__default__' => $default));
 
 	}
 
@@ -51,10 +51,10 @@ 
 	 *
 	 * @return Rule_Validator A {@link Rule_Validator} instance or the default one when not found.
 	 */
-	public function get_rule_validator( $key ) {
+	public function get_rule_validator($key) {
 
-		return isset( $this->rule_validators[ $key ] )
-			? $this->rule_validators[ $key ] : $this->rule_validators['__default__'];
+		return isset($this->rule_validators[$key])
+			? $this->rule_validators[$key] : $this->rule_validators['__default__'];
 	}
 
 }

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

Indentation +47 added lines, -47 removed lines

@@ -15,60 +15,60 @@
  * @package Wordlift\Mappings\Validators
  */
 class Post_Taxonomy_Term_Rule_Validator implements Rule_Validator {
-	/**
-	 * @since 3.25.0
-	 * Enum for the post taxonomy type rule validator.
-	 */
-	const POST_TAXONOMY = 'post_taxonomy';
+    /**
+     * @since 3.25.0
+     * Enum for the post taxonomy type rule validator.
+     */
+    const POST_TAXONOMY = 'post_taxonomy';
 
-	/**
-	 * Post_Taxonomy_Term_Rule_Validator constructor.
-	 *
-	 * When initializing the class hooks to `wl_mappings_rule_validators`.
-	 */
-	public function __construct() {
+    /**
+     * Post_Taxonomy_Term_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' ) );
+        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 ) {
+    /**
+     * 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_TAXONOMY ] = $this;
+        $value[ self::POST_TAXONOMY ] = $this;
 
-		return $value;
-	}
+        return $value;
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_label() {
-		return __( 'Post Taxonomy', 'wordlift' );
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function get_label() {
+        return __( 'Post Taxonomy', 'wordlift' );
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function is_valid( $identifier, $operator, $operand_1, $operand_2, $type ) {
-		$taxonomy  = $operand_1;
-		$term_slug = $operand_2;
-		if ( get_post_type( $identifier ) !== 'post' ) {
-			return false;
-		}
-		$is_object_in_term = is_object_in_term( $identifier, $taxonomy, $term_slug );
-		if ( is_wp_error( $is_object_in_term ) ) {
-			return false;
-		}
+    /**
+     * {@inheritdoc}
+     */
+    // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+    public function is_valid( $identifier, $operator, $operand_1, $operand_2, $type ) {
+        $taxonomy  = $operand_1;
+        $term_slug = $operand_2;
+        if ( get_post_type( $identifier ) !== 'post' ) {
+            return false;
+        }
+        $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 );
-	}
+        return ( $is_object_in_term && self::IS_EQUAL_TO === $operator )
+               || ( ! $is_object_in_term && self::IS_NOT_EQUAL_TO === $operator );
+    }
 }

Spacing +10 added lines, -10 removed lines

@@ -28,7 +28,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'));
 
 	}
 
@@ -39,9 +39,9 @@ 
 	 *
 	 * @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_TAXONOMY ] = $this;
+		$value[self::POST_TAXONOMY] = $this;
 
 		return $value;
 	}
@@ -50,25 +50,25 @@ 
 	 * {@inheritdoc}
 	 */
 	public function get_label() {
-		return __( 'Post Taxonomy', 'wordlift' );
+		return __('Post Taxonomy', 'wordlift');
 	}
 
 	/**
 	 * {@inheritdoc}
 	 */
 	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	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;
-		if ( get_post_type( $identifier ) !== 'post' ) {
+		if (get_post_type($identifier) !== 'post') {
 			return false;
 		}
-		$is_object_in_term = is_object_in_term( $identifier, $taxonomy, $term_slug );
-		if ( is_wp_error( $is_object_in_term ) ) {
+		$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 );
+		return ($is_object_in_term && self::IS_EQUAL_TO === $operator)
+			   || ( ! $is_object_in_term && self::IS_NOT_EQUAL_TO === $operator);
 	}
 }

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

Indentation +41 added lines, -41 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                      $identifier The post id or term 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 or term 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 +47 added lines, -47 removed lines

@@ -17,58 +17,58 @@
  */
 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 ) {
-		if ( Jsonld_Converter::TERM !== $type ) {
-			return false;
-		}
-		$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 ( Rule_Validator::IS_EQUAL_TO === $operator ) {
-			// 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, true );
-		}
+    public function is_valid( $identifier, $operator, $operand_1, $taxonomy, $type ) {
+        if ( Jsonld_Converter::TERM !== $type ) {
+            return false;
+        }
+        $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 ( Rule_Validator::IS_EQUAL_TO === $operator ) {
+            // 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, true );
+        }
 
-		if ( Rule_Validator::IS_NOT_EQUAL_TO === $operator ) {
-			return ! in_array( $current_term->term_id, $terms, true );
-		}
+        if ( Rule_Validator::IS_NOT_EQUAL_TO === $operator ) {
+            return ! in_array( $current_term->term_id, $terms, true );
+        }
 
-	}
+    }
 }

Spacing +14 added lines, -14 removed lines

@@ -20,7 +20,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'));
 	}
 
 	/**
@@ -30,28 +30,28 @@ 
 	 *
 	 * @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 ) {
-		if ( Jsonld_Converter::TERM !== $type ) {
+	public function is_valid($identifier, $operator, $operand_1, $taxonomy, $type) {
+		if (Jsonld_Converter::TERM !== $type) {
 			return false;
 		}
-		$current_term = get_term( $identifier );
+		$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 = get_terms($taxonomy, array('get' => 'all'));
 		$terms = array_map(
-			function ( $term ) {
+			function($term) {
 				/**
 				 * @var $term \WP_Term
 				 */
@@ -59,15 +59,15 @@ 
 			},
 			$terms
 		);
-		if ( Rule_Validator::IS_EQUAL_TO === $operator ) {
+		if (Rule_Validator::IS_EQUAL_TO === $operator) {
 			// 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, true );
+			return in_array($current_term->term_id, $terms, true);
 		}
 
-		if ( Rule_Validator::IS_NOT_EQUAL_TO === $operator ) {
-			return ! in_array( $current_term->term_id, $terms, true );
+		if (Rule_Validator::IS_NOT_EQUAL_TO === $operator) {
+			return ! in_array($current_term->term_id, $terms, true);
 		}
 
 	}

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

Indentation +25 added lines, -25 removed lines

@@ -16,29 +16,29 @@
  */
 class Taxonomy_Rule_Validator implements Rule_Validator {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	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}
+     */
+    // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+    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

@@ -20,25 +20,25 @@
 	 * {@inheritdoc}
 	 */
 	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	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');
 	}
 }