insideout10 / wordlift-plugin

src/wordlift/mappings/class-mappings-rest-controller.php

Indentation +458 added lines, -458 removed lines

@@ -13,135 +13,135 @@ 
  * @subpackage Wordlift/includes/sync-mappings
  */
 class Mappings_REST_Controller {
-	// Namespace for CRUD mappings.
-	const MAPPINGS_NAMESPACE = '/mappings';
-
-	/**
-	 * Registers route on rest api initialisation.
-	 */
-	public static function register_routes() {
-
-		add_action( 'rest_api_init', 'Wordlift\Mappings\Mappings_REST_Controller::register_route_callback' );
-
-	}
-
-	/**
-	 * Get a single mapping item by its mapping_id
-	 *
-	 * @param WP_REST_Request $request {@link WP_REST_Request instance}.
-	 *
-	 * @return array
-	 */
-	public static function get_mapping_item( $request ) {
-		$dbo             = new Mappings_DBO();
-		$mapping_id      = $request['id'];
-		$mapping_id_data = array();
-		$rule_groups     = $dbo->get_rule_groups_by_mapping( $mapping_id );
-		$properties      = $dbo->get_properties( $mapping_id );
-		$mapping_row     = $dbo->get_mapping_item_data( $mapping_id );
-
-		$mapping_id_data['mapping_id']      = $mapping_id;
-		$mapping_id_data['property_list']   = $properties;
-		$mapping_id_data['rule_group_list'] = $rule_groups;
-		$mapping_id_data['mapping_title']   = $mapping_row['mapping_title'];
-
-		return $mapping_id_data;
-	}
-
-	/**
-	 * Register route call back function, called when rest api gets initialised
-	 *
-	 * @return void
-	 */
-	public static function register_route_callback() {
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/mappings',
-			array(
-				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::insert_or_update_mapping_item',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
-		// Get list of mapping items.
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/mappings',
-			array(
-				'methods'             => WP_REST_Server::READABLE,
-				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::list_mapping_items',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
-
-		// Delete mapping items by id.
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'mappings',
-			array(
-				'methods'             => WP_REST_Server::DELETABLE,
-				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::delete_mapping_items',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
-
-		// Get single mapping item route.
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'mappings/(?P<id>\d+)',
-			array(
-				'methods'             => WP_REST_Server::READABLE,
-				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::get_mapping_item',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
-
-		// Update mapping items.
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'mappings',
-			array(
-				'methods'             => WP_REST_Server::EDITABLE,
-				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::update_mapping_items',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
-
-		// Clone mapping items.
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'mappings/clone',
-			array(
-				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::clone_mapping_items',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
-
-		// Register rest endpoint to get the terms.
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'mappings/get_terms',
-			array(
-				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::get_terms_for_the_posted_taxonomy',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
+    // Namespace for CRUD mappings.
+    const MAPPINGS_NAMESPACE = '/mappings';
+
+    /**
+     * Registers route on rest api initialisation.
+     */
+    public static function register_routes() {
+
+        add_action( 'rest_api_init', 'Wordlift\Mappings\Mappings_REST_Controller::register_route_callback' );
+
+    }
+
+    /**
+     * Get a single mapping item by its mapping_id
+     *
+     * @param WP_REST_Request $request {@link WP_REST_Request instance}.
+     *
+     * @return array
+     */
+    public static function get_mapping_item( $request ) {
+        $dbo             = new Mappings_DBO();
+        $mapping_id      = $request['id'];
+        $mapping_id_data = array();
+        $rule_groups     = $dbo->get_rule_groups_by_mapping( $mapping_id );
+        $properties      = $dbo->get_properties( $mapping_id );
+        $mapping_row     = $dbo->get_mapping_item_data( $mapping_id );
+
+        $mapping_id_data['mapping_id']      = $mapping_id;
+        $mapping_id_data['property_list']   = $properties;
+        $mapping_id_data['rule_group_list'] = $rule_groups;
+        $mapping_id_data['mapping_title']   = $mapping_row['mapping_title'];
+
+        return $mapping_id_data;
+    }
+
+    /**
+     * Register route call back function, called when rest api gets initialised
+     *
+     * @return void
+     */
+    public static function register_route_callback() {
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/mappings',
+            array(
+                'methods'             => WP_REST_Server::CREATABLE,
+                'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::insert_or_update_mapping_item',
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
+        // Get list of mapping items.
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/mappings',
+            array(
+                'methods'             => WP_REST_Server::READABLE,
+                'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::list_mapping_items',
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
+
+        // Delete mapping items by id.
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            'mappings',
+            array(
+                'methods'             => WP_REST_Server::DELETABLE,
+                'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::delete_mapping_items',
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
+
+        // Get single mapping item route.
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            'mappings/(?P<id>\d+)',
+            array(
+                'methods'             => WP_REST_Server::READABLE,
+                'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::get_mapping_item',
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
+
+        // Update mapping items.
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            'mappings',
+            array(
+                'methods'             => WP_REST_Server::EDITABLE,
+                'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::update_mapping_items',
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
+
+        // Clone mapping items.
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            'mappings/clone',
+            array(
+                'methods'             => WP_REST_Server::CREATABLE,
+                'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::clone_mapping_items',
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
+
+        // Register rest endpoint to get the terms.
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            'mappings/get_terms',
+            array(
+                'methods'             => WP_REST_Server::CREATABLE,
+                'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::get_terms_for_the_posted_taxonomy',
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
 
         // Register rest endpoint to get the terms.
         register_rest_route(
@@ -155,7 +155,7 @@ 
                 },
             )
         );
-	}
+    }
 
     /**
      * Get the taxonomy & terms
@@ -206,334 +206,334 @@ 
         return $taxonomy_terms;
     }
 
-	/**
-	 * Get the terms for the posted taxonomy name.
-	 *
-	 * @param WP_REST_Request $request {@link WP_REST_Request instance}.
-	 *
-	 * @return array The array of the terms for the taxonomy.
-	 */
-	public static function get_terms_for_the_posted_taxonomy( $request ) {
-		$post_data = $request->get_params();
-		if ( ! array_key_exists( 'taxonomy', $post_data ) ) {
-			return array(
-				'status'  => 'failure',
-				'message' => __( 'Request not valid, must post a taxonomy to get terms', 'wordlift' )
-			);
-		} else {
-			$taxonomy = $post_data['taxonomy'];
-			$terms    = get_terms( $taxonomy, array( 'hide_empty' => false, ) );
-			if ( is_wp_error( $terms ) ) {
-				// Return error response, if the taxonomy is not valid.
-				return array(
-					'status'  => 'failure',
-					'message' => __( 'Request not valid, must post a valid taxonomy', 'wordlift' )
-				);
-			}
-
-			return $terms;
-		}
-	}
-
-	/**
-	 * Clone posted mapping items.
-	 *
-	 * @param WP_REST_Request $request {@link WP_REST_Request instance}.
-	 *
-	 * @return array
-	 */
-	public static function clone_mapping_items( $request ) {
-		$dbo           = new Mappings_DBO();
-		$post_data     = (array) $request->get_params();
-		$mapping_items = (array) $post_data['mapping_items'];
-		foreach ( $mapping_items as $mapping_item ) {
-			$mapping_id = (int) $mapping_item['mapping_id'];
-			// Clone the current mapping item.
-			$cloned_mapping_id = $dbo->insert_mapping_item( $mapping_item['mapping_title'] );
-			// Clone all the rule groups.
-			$rule_groups_to_be_cloned = $dbo->get_rule_groups_by_mapping( $mapping_id );
-			// Clone all the properties.
-			$properties_to_be_cloned = $dbo->get_properties( $mapping_id );
-			foreach ( $properties_to_be_cloned as $property ) {
-				// Assign a new mapping id.
-				$property['mapping_id'] = $cloned_mapping_id;
-				// Removing this property id, since a new id needed to be created for
-				// new property.
-				unset( $property['property_id'] );
-				$dbo->insert_or_update_property( $property );
-			}
-			// Loop through the rule groups and insert them in table with the mapping id.
-			foreach ( $rule_groups_to_be_cloned as $rule_group ) {
-				$cloned_rule_group_id = $dbo->insert_rule_group( $cloned_mapping_id );
-				$original_rules       = (array) $rule_group['rules'];
-				// Now we need to insert these rules for the cloned rule group id.
-				foreach ( $original_rules as $clone_rule ) {
-					// We should replace only rule group id in the cloned rules.
-					$clone_rule['rule_group_id'] = (int) $cloned_rule_group_id;
-					unset( $clone_rule['rule_id'] );
-					$dbo->insert_or_update_rule_item( $clone_rule );
-				}
-			}
-		}
-
-		return array(
-			'status'  => 'success',
-			'message' => __( 'Successfully cloned mapping items', 'wordlift' ),
-		);
-	}
-
-	/**
-	 * Update posted mapping items.
-	 *
-	 * @param WP_REST_Request $request {@link WP_REST_Request instance}.
-	 *
-	 * @return array
-	 */
-	public static function update_mapping_items( $request ) {
-		$dbo       = new Mappings_DBO();
-		$post_data = $request->get_params();
-		if ( array_key_exists( 'mapping_items', $post_data ) ) {
-			$mapping_items = (array) $post_data['mapping_items'];
-			foreach ( $mapping_items as $mapping_item ) {
-				$dbo->insert_or_update_mapping_item( $mapping_item );
-			}
-
-			return array(
-				'status'  => 'success',
-				'message' => __( 'Mapping items successfully updated', 'wordlift' ),
-			);
-		} else {
-			return array(
-				'status'  => 'failure',
-				'message' => __( 'Unable to update mapping item', 'wordlift' ),
-			);
-		}
-	}
-
-	/**
-	 * Delete mapping items by mapping id
-	 *
-	 * @param WP_REST_Request $request {@link WP_REST_Request instance}.
-	 *
-	 * @return array
-	 */
-	public static function delete_mapping_items( $request ) {
-		$dbo       = new Mappings_DBO();
-		$post_data = $request->get_params();
-		if ( array_key_exists( 'mapping_items', $post_data ) ) {
-			$mapping_items = (array) $post_data['mapping_items'];
-			foreach ( $mapping_items as $mapping_item ) {
-				$dbo->delete_mapping_item( $mapping_item['mapping_id'] );
-			}
-
-			return array(
-				'status'  => 'success',
-				'message' => __( 'successfully deleted mapping items', 'wordlift' )
-			);
-		} else {
-			return array(
-				'status'  => 'failure',
-				'message' => __( 'Unable to delete mapping items', 'wordlift' )
-			);
-		}
-	}
-
-	/**
-	 * Get all mapping items
-	 *
-	 * @param WP_REST_Request $request {@link WP_REST_Request instance}.
-	 *
-	 * @return array
-	 */
-	public static function list_mapping_items( $request ) {
-		$dbo = new Mappings_DBO();
-
-		return $dbo->get_mappings();
-	}
-
-	/**
-	 * Returns a array of rule ids for the rule group id
-	 *
-	 * @param Object $dbo Instance of {@link Mappings_DBO } class.
-	 * @param int $rule_group_id Primary key of rule group table.
-	 *
-	 * @return array A list of rule ids.
-	 */
-	private static function get_rule_ids( $dbo, $rule_group_id ) {
-		$rule_rows_in_db = $dbo->get_rules_by_rule_group( $rule_group_id );
-		$rule_ids        = array();
-		foreach ( $rule_rows_in_db as $rule_row ) {
-			array_push( $rule_ids, (int) $rule_row['rule_id'] );
-		}
-
-		return $rule_ids;
-	}
-
-	/**
-	 * Insert or update mapping item depends on data
-	 *
-	 * @param Object $dbo Instance of {@link Mappings_DBO } class.
-	 * @param int $rule_group_id Refers to a rule group which this rule belongs to.
-	 * @param array $rule_list Array of rule  items.
-	 *
-	 * @return void
-	 */
-	private static function save_rules( $dbo, $rule_group_id, $rule_list ) {
-		$rule_ids = self::get_rule_ids( $dbo, $rule_group_id );
-		foreach ( $rule_list as $rule ) {
-			// Some rules may not have rule group id, because they are inserted
-			// in ui, so lets add them any way.
-			$rule['rule_group_id'] = $rule_group_id;
-			$dbo->insert_or_update_rule_item( $rule );
-			if ( array_key_exists( 'rule_id', $rule ) ) {
-				$index_to_be_removed = array_search(
-					(int) $rule['rule_id'],
-					$rule_ids,
-					true
-				);
-				if ( false !== $index_to_be_removed ) {
-					unset( $rule_ids[ $index_to_be_removed ] );
-				}
-			}
-		}
-		foreach ( $rule_ids as $rule_id ) {
-			// Delete all the rule ids which are not posted.
-			$dbo->delete_rule_item( $rule_id );
-		}
-	}
-
-	/**
-	 * Insert or update rule group list based on data
-	 *
-	 * @param Object $dbo Instance of {@link Mappings_DBO } class.
-	 * @param int $mapping_id Primary key of mapping table.
-	 * @param array $property_list { Array of property items }.
-	 *
-	 * @return void
-	 */
-	private static function save_property_list( $dbo, $mapping_id, $property_list ) {
-		$properties_needed_to_be_deleted = $dbo->get_properties( $mapping_id );
-		$property_ids                    = array();
-		foreach ( $properties_needed_to_be_deleted as $property ) {
-			array_push( $property_ids, (int) $property['property_id'] );
-		}
-		foreach ( $property_list as $property ) {
-			if ( array_key_exists( 'property_id', $property ) ) {
-				// Remove the id from the list of property ids needed to be deleted
-				// because it is posted.
-				$index_to_be_removed = array_search(
-					(int) $property['property_id'],
-					$property_ids,
-					true
-				);
-				if ( false !== $index_to_be_removed ) {
-					unset( $property_ids[ $index_to_be_removed ] );
-				}
-			}
-			// Add mapping id to property data.
-			$property['mapping_id'] = $mapping_id;
-			$dbo->insert_or_update_property( $property );
-
-		}
-		// At the end remove all the property ids which are not posted.
-		foreach ( $property_ids as $property_id ) {
-			$dbo->delete_property( $property_id );
-		}
-
-	}
-
-	/**
-	 * Returns a array of rule group ids for the mapping id
-	 *
-	 * @param Object $dbo Instance of {@link Mappings_DBO } class.
-	 * @param int $mapping_id Primary key of mapping table.
-	 *
-	 * @return array $rule_group_ids A list of rule group ids.
-	 */
-	private static function get_rule_group_ids( $dbo, $mapping_id ) {
-		$rule_group_rows = $dbo->get_rule_group_list( $mapping_id );
-		$rule_group_ids  = array();
-		foreach ( $rule_group_rows as $rule_group_row ) {
-			array_push( $rule_group_ids, (int) $rule_group_row['rule_group_id'] );
-		}
-
-		return $rule_group_ids;
-	}
-
-	/**
-	 * Insert or update rule group list
-	 *
-	 * @param Object $dbo Instance of {@link Mappings_DBO } class.
-	 * @param int $mapping_id Primary key of mapping table.
-	 * @param array $rule_group_list { Array of rule group items }.
-	 *
-	 * @return void
-	 */
-	private static function save_rule_group_list( $dbo, $mapping_id, $rule_group_list ) {
-		// The rule groups not posted should be deleted.
-		$rule_group_ids = self::get_rule_group_ids( $dbo, $mapping_id );
-		// Loop through rule group list and save the rule group.
-		foreach ( $rule_group_list as $rule_group ) {
-			if ( array_key_exists( 'rule_group_id', $rule_group ) ) {
-				$rule_group_id = $rule_group['rule_group_id'];
-			} else {
-				// New rule group, should create new rule group id.
-				$rule_group_id = $dbo->insert_rule_group( $mapping_id );
-			}
-			$index_to_be_removed = array_search(
-				(int) $rule_group_id,
-				$rule_group_ids,
-				true
-			);
-			if ( false !== $index_to_be_removed ) {
-				unset( $rule_group_ids[ $index_to_be_removed ] );
-			}
-			self::save_rules( $dbo, $rule_group_id, $rule_group['rules'] );
-		}
-
-		// Remove all the rule groups which are not posted.
-		foreach ( $rule_group_ids as $rule_group_id ) {
-			$dbo->delete_rule_group_item( $rule_group_id );
-		}
-	}
-
-	/**
-	 * Insert or update mapping item depends on data
-	 *
-	 * @param WP_REST_Request $request {@link WP_REST_Request instance}.
-	 *
-	 * @return array
-	 */
-	public static function insert_or_update_mapping_item( $request ) {
-		$post_data = $request->get_params() === null ? array() : $request->get_params();
-		$dbo       = new Mappings_DBO();
-		// check if valid object is posted.
-		if ( array_key_exists( 'mapping_title', $post_data ) &&
-		     array_key_exists( 'rule_group_list', $post_data ) &&
-		     array_key_exists( 'property_list', $post_data ) ) {
-			// Do validation, remove all incomplete data.
-			$mapping_item = array();
-			if ( array_key_exists( 'mapping_id', $post_data ) ) {
-				$mapping_item['mapping_id'] = $post_data['mapping_id'];
-			}
-			$mapping_item['mapping_title'] = $post_data['mapping_title'];
-			// lets save the mapping item.
-			$mapping_id = $dbo->insert_or_update_mapping_item( $mapping_item );
-			self::save_rule_group_list( $dbo, $mapping_id, $post_data['rule_group_list'] );
-			self::save_property_list( $dbo, $mapping_id, $post_data['property_list'] );
-
-			return array(
-				'status'     => 'success',
-				'message'    => __( 'Successfully saved mapping item', 'wordlift' ),
-				'mapping_id' => (int) $mapping_id,
-			);
-		} else {
-			return array(
-				'status'  => 'error',
-				'message' => __( 'Unable to save mapping item', 'wordlift' ),
-			);
-		}
-	}
+    /**
+     * Get the terms for the posted taxonomy name.
+     *
+     * @param WP_REST_Request $request {@link WP_REST_Request instance}.
+     *
+     * @return array The array of the terms for the taxonomy.
+     */
+    public static function get_terms_for_the_posted_taxonomy( $request ) {
+        $post_data = $request->get_params();
+        if ( ! array_key_exists( 'taxonomy', $post_data ) ) {
+            return array(
+                'status'  => 'failure',
+                'message' => __( 'Request not valid, must post a taxonomy to get terms', 'wordlift' )
+            );
+        } else {
+            $taxonomy = $post_data['taxonomy'];
+            $terms    = get_terms( $taxonomy, array( 'hide_empty' => false, ) );
+            if ( is_wp_error( $terms ) ) {
+                // Return error response, if the taxonomy is not valid.
+                return array(
+                    'status'  => 'failure',
+                    'message' => __( 'Request not valid, must post a valid taxonomy', 'wordlift' )
+                );
+            }
+
+            return $terms;
+        }
+    }
+
+    /**
+     * Clone posted mapping items.
+     *
+     * @param WP_REST_Request $request {@link WP_REST_Request instance}.
+     *
+     * @return array
+     */
+    public static function clone_mapping_items( $request ) {
+        $dbo           = new Mappings_DBO();
+        $post_data     = (array) $request->get_params();
+        $mapping_items = (array) $post_data['mapping_items'];
+        foreach ( $mapping_items as $mapping_item ) {
+            $mapping_id = (int) $mapping_item['mapping_id'];
+            // Clone the current mapping item.
+            $cloned_mapping_id = $dbo->insert_mapping_item( $mapping_item['mapping_title'] );
+            // Clone all the rule groups.
+            $rule_groups_to_be_cloned = $dbo->get_rule_groups_by_mapping( $mapping_id );
+            // Clone all the properties.
+            $properties_to_be_cloned = $dbo->get_properties( $mapping_id );
+            foreach ( $properties_to_be_cloned as $property ) {
+                // Assign a new mapping id.
+                $property['mapping_id'] = $cloned_mapping_id;
+                // Removing this property id, since a new id needed to be created for
+                // new property.
+                unset( $property['property_id'] );
+                $dbo->insert_or_update_property( $property );
+            }
+            // Loop through the rule groups and insert them in table with the mapping id.
+            foreach ( $rule_groups_to_be_cloned as $rule_group ) {
+                $cloned_rule_group_id = $dbo->insert_rule_group( $cloned_mapping_id );
+                $original_rules       = (array) $rule_group['rules'];
+                // Now we need to insert these rules for the cloned rule group id.
+                foreach ( $original_rules as $clone_rule ) {
+                    // We should replace only rule group id in the cloned rules.
+                    $clone_rule['rule_group_id'] = (int) $cloned_rule_group_id;
+                    unset( $clone_rule['rule_id'] );
+                    $dbo->insert_or_update_rule_item( $clone_rule );
+                }
+            }
+        }
+
+        return array(
+            'status'  => 'success',
+            'message' => __( 'Successfully cloned mapping items', 'wordlift' ),
+        );
+    }
+
+    /**
+     * Update posted mapping items.
+     *
+     * @param WP_REST_Request $request {@link WP_REST_Request instance}.
+     *
+     * @return array
+     */
+    public static function update_mapping_items( $request ) {
+        $dbo       = new Mappings_DBO();
+        $post_data = $request->get_params();
+        if ( array_key_exists( 'mapping_items', $post_data ) ) {
+            $mapping_items = (array) $post_data['mapping_items'];
+            foreach ( $mapping_items as $mapping_item ) {
+                $dbo->insert_or_update_mapping_item( $mapping_item );
+            }
+
+            return array(
+                'status'  => 'success',
+                'message' => __( 'Mapping items successfully updated', 'wordlift' ),
+            );
+        } else {
+            return array(
+                'status'  => 'failure',
+                'message' => __( 'Unable to update mapping item', 'wordlift' ),
+            );
+        }
+    }
+
+    /**
+     * Delete mapping items by mapping id
+     *
+     * @param WP_REST_Request $request {@link WP_REST_Request instance}.
+     *
+     * @return array
+     */
+    public static function delete_mapping_items( $request ) {
+        $dbo       = new Mappings_DBO();
+        $post_data = $request->get_params();
+        if ( array_key_exists( 'mapping_items', $post_data ) ) {
+            $mapping_items = (array) $post_data['mapping_items'];
+            foreach ( $mapping_items as $mapping_item ) {
+                $dbo->delete_mapping_item( $mapping_item['mapping_id'] );
+            }
+
+            return array(
+                'status'  => 'success',
+                'message' => __( 'successfully deleted mapping items', 'wordlift' )
+            );
+        } else {
+            return array(
+                'status'  => 'failure',
+                'message' => __( 'Unable to delete mapping items', 'wordlift' )
+            );
+        }
+    }
+
+    /**
+     * Get all mapping items
+     *
+     * @param WP_REST_Request $request {@link WP_REST_Request instance}.
+     *
+     * @return array
+     */
+    public static function list_mapping_items( $request ) {
+        $dbo = new Mappings_DBO();
+
+        return $dbo->get_mappings();
+    }
+
+    /**
+     * Returns a array of rule ids for the rule group id
+     *
+     * @param Object $dbo Instance of {@link Mappings_DBO } class.
+     * @param int $rule_group_id Primary key of rule group table.
+     *
+     * @return array A list of rule ids.
+     */
+    private static function get_rule_ids( $dbo, $rule_group_id ) {
+        $rule_rows_in_db = $dbo->get_rules_by_rule_group( $rule_group_id );
+        $rule_ids        = array();
+        foreach ( $rule_rows_in_db as $rule_row ) {
+            array_push( $rule_ids, (int) $rule_row['rule_id'] );
+        }
+
+        return $rule_ids;
+    }
+
+    /**
+     * Insert or update mapping item depends on data
+     *
+     * @param Object $dbo Instance of {@link Mappings_DBO } class.
+     * @param int $rule_group_id Refers to a rule group which this rule belongs to.
+     * @param array $rule_list Array of rule  items.
+     *
+     * @return void
+     */
+    private static function save_rules( $dbo, $rule_group_id, $rule_list ) {
+        $rule_ids = self::get_rule_ids( $dbo, $rule_group_id );
+        foreach ( $rule_list as $rule ) {
+            // Some rules may not have rule group id, because they are inserted
+            // in ui, so lets add them any way.
+            $rule['rule_group_id'] = $rule_group_id;
+            $dbo->insert_or_update_rule_item( $rule );
+            if ( array_key_exists( 'rule_id', $rule ) ) {
+                $index_to_be_removed = array_search(
+                    (int) $rule['rule_id'],
+                    $rule_ids,
+                    true
+                );
+                if ( false !== $index_to_be_removed ) {
+                    unset( $rule_ids[ $index_to_be_removed ] );
+                }
+            }
+        }
+        foreach ( $rule_ids as $rule_id ) {
+            // Delete all the rule ids which are not posted.
+            $dbo->delete_rule_item( $rule_id );
+        }
+    }
+
+    /**
+     * Insert or update rule group list based on data
+     *
+     * @param Object $dbo Instance of {@link Mappings_DBO } class.
+     * @param int $mapping_id Primary key of mapping table.
+     * @param array $property_list { Array of property items }.
+     *
+     * @return void
+     */
+    private static function save_property_list( $dbo, $mapping_id, $property_list ) {
+        $properties_needed_to_be_deleted = $dbo->get_properties( $mapping_id );
+        $property_ids                    = array();
+        foreach ( $properties_needed_to_be_deleted as $property ) {
+            array_push( $property_ids, (int) $property['property_id'] );
+        }
+        foreach ( $property_list as $property ) {
+            if ( array_key_exists( 'property_id', $property ) ) {
+                // Remove the id from the list of property ids needed to be deleted
+                // because it is posted.
+                $index_to_be_removed = array_search(
+                    (int) $property['property_id'],
+                    $property_ids,
+                    true
+                );
+                if ( false !== $index_to_be_removed ) {
+                    unset( $property_ids[ $index_to_be_removed ] );
+                }
+            }
+            // Add mapping id to property data.
+            $property['mapping_id'] = $mapping_id;
+            $dbo->insert_or_update_property( $property );
+
+        }
+        // At the end remove all the property ids which are not posted.
+        foreach ( $property_ids as $property_id ) {
+            $dbo->delete_property( $property_id );
+        }
+
+    }
+
+    /**
+     * Returns a array of rule group ids for the mapping id
+     *
+     * @param Object $dbo Instance of {@link Mappings_DBO } class.
+     * @param int $mapping_id Primary key of mapping table.
+     *
+     * @return array $rule_group_ids A list of rule group ids.
+     */
+    private static function get_rule_group_ids( $dbo, $mapping_id ) {
+        $rule_group_rows = $dbo->get_rule_group_list( $mapping_id );
+        $rule_group_ids  = array();
+        foreach ( $rule_group_rows as $rule_group_row ) {
+            array_push( $rule_group_ids, (int) $rule_group_row['rule_group_id'] );
+        }
+
+        return $rule_group_ids;
+    }
+
+    /**
+     * Insert or update rule group list
+     *
+     * @param Object $dbo Instance of {@link Mappings_DBO } class.
+     * @param int $mapping_id Primary key of mapping table.
+     * @param array $rule_group_list { Array of rule group items }.
+     *
+     * @return void
+     */
+    private static function save_rule_group_list( $dbo, $mapping_id, $rule_group_list ) {
+        // The rule groups not posted should be deleted.
+        $rule_group_ids = self::get_rule_group_ids( $dbo, $mapping_id );
+        // Loop through rule group list and save the rule group.
+        foreach ( $rule_group_list as $rule_group ) {
+            if ( array_key_exists( 'rule_group_id', $rule_group ) ) {
+                $rule_group_id = $rule_group['rule_group_id'];
+            } else {
+                // New rule group, should create new rule group id.
+                $rule_group_id = $dbo->insert_rule_group( $mapping_id );
+            }
+            $index_to_be_removed = array_search(
+                (int) $rule_group_id,
+                $rule_group_ids,
+                true
+            );
+            if ( false !== $index_to_be_removed ) {
+                unset( $rule_group_ids[ $index_to_be_removed ] );
+            }
+            self::save_rules( $dbo, $rule_group_id, $rule_group['rules'] );
+        }
+
+        // Remove all the rule groups which are not posted.
+        foreach ( $rule_group_ids as $rule_group_id ) {
+            $dbo->delete_rule_group_item( $rule_group_id );
+        }
+    }
+
+    /**
+     * Insert or update mapping item depends on data
+     *
+     * @param WP_REST_Request $request {@link WP_REST_Request instance}.
+     *
+     * @return array
+     */
+    public static function insert_or_update_mapping_item( $request ) {
+        $post_data = $request->get_params() === null ? array() : $request->get_params();
+        $dbo       = new Mappings_DBO();
+        // check if valid object is posted.
+        if ( array_key_exists( 'mapping_title', $post_data ) &&
+             array_key_exists( 'rule_group_list', $post_data ) &&
+             array_key_exists( 'property_list', $post_data ) ) {
+            // Do validation, remove all incomplete data.
+            $mapping_item = array();
+            if ( array_key_exists( 'mapping_id', $post_data ) ) {
+                $mapping_item['mapping_id'] = $post_data['mapping_id'];
+            }
+            $mapping_item['mapping_title'] = $post_data['mapping_title'];
+            // lets save the mapping item.
+            $mapping_id = $dbo->insert_or_update_mapping_item( $mapping_item );
+            self::save_rule_group_list( $dbo, $mapping_id, $post_data['rule_group_list'] );
+            self::save_property_list( $dbo, $mapping_id, $post_data['property_list'] );
+
+            return array(
+                'status'     => 'success',
+                'message'    => __( 'Successfully saved mapping item', 'wordlift' ),
+                'mapping_id' => (int) $mapping_id,
+            );
+        } else {
+            return array(
+                'status'  => 'error',
+                'message' => __( 'Unable to save mapping item', 'wordlift' ),
+            );
+        }
+    }
 }
 
 Mappings_REST_Controller::register_routes();

Spacing +113 added lines, -113 removed lines

@@ -21,7 +21,7 @@ 
 	 */
 	public static function register_routes() {
 
-		add_action( 'rest_api_init', 'Wordlift\Mappings\Mappings_REST_Controller::register_route_callback' );
+		add_action('rest_api_init', 'Wordlift\Mappings\Mappings_REST_Controller::register_route_callback');
 
 	}
 
@@ -32,13 +32,13 @@ 
 	 *
 	 * @return array
 	 */
-	public static function get_mapping_item( $request ) {
+	public static function get_mapping_item($request) {
 		$dbo             = new Mappings_DBO();
 		$mapping_id      = $request['id'];
 		$mapping_id_data = array();
-		$rule_groups     = $dbo->get_rule_groups_by_mapping( $mapping_id );
-		$properties      = $dbo->get_properties( $mapping_id );
-		$mapping_row     = $dbo->get_mapping_item_data( $mapping_id );
+		$rule_groups     = $dbo->get_rule_groups_by_mapping($mapping_id);
+		$properties      = $dbo->get_properties($mapping_id);
+		$mapping_row     = $dbo->get_mapping_item_data($mapping_id);
 
 		$mapping_id_data['mapping_id']      = $mapping_id;
 		$mapping_id_data['property_list']   = $properties;
@@ -60,8 +60,8 @@ 
 			array(
 				'methods'             => WP_REST_Server::CREATABLE,
 				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::insert_or_update_mapping_item',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -72,8 +72,8 @@ 
 			array(
 				'methods'             => WP_REST_Server::READABLE,
 				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::list_mapping_items',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -85,8 +85,8 @@ 
 			array(
 				'methods'             => WP_REST_Server::DELETABLE,
 				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::delete_mapping_items',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -98,8 +98,8 @@ 
 			array(
 				'methods'             => WP_REST_Server::READABLE,
 				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::get_mapping_item',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -111,8 +111,8 @@ 
 			array(
 				'methods'             => WP_REST_Server::EDITABLE,
 				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::update_mapping_items',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -124,8 +124,8 @@ 
 			array(
 				'methods'             => WP_REST_Server::CREATABLE,
 				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::clone_mapping_items',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -137,8 +137,8 @@ 
 			array(
 				'methods'             => WP_REST_Server::CREATABLE,
 				'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::get_terms_for_the_posted_taxonomy',
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -150,8 +150,8 @@ 
             array(
                 'methods'             => WP_REST_Server::CREATABLE,
                 'callback'            => 'Wordlift\Mappings\Mappings_REST_Controller::get_taxonomy_terms_for_the_posted_taxonomy',
-                'permission_callback' => function () {
-                    return current_user_can( 'manage_options' );
+                'permission_callback' => function() {
+                    return current_user_can('manage_options');
                 },
             )
         );
@@ -164,23 +164,23 @@ 
      *
      * @return array The array of the taxonomies & terms.
      */
-    public static function get_taxonomy_terms_for_the_posted_taxonomy( $request ) {
+    public static function get_taxonomy_terms_for_the_posted_taxonomy($request) {
         $taxonomy_terms = array();
-        $post_taxonomies = get_taxonomies( array(), 'objects' );
+        $post_taxonomies = get_taxonomies(array(), 'objects');
 
-        foreach ( $post_taxonomies as $post_taxonomy ) {
+        foreach ($post_taxonomies as $post_taxonomy) {
 
-            $group_taxonomy = array( 'parentValue' => 'post_taxonomy', 'group_name' => $post_taxonomy->label, 'group_options' => array() );
+            $group_taxonomy = array('parentValue' => 'post_taxonomy', 'group_name' => $post_taxonomy->label, 'group_options' => array());
 
-            $post_taxonomy_terms = get_terms( array(
+            $post_taxonomy_terms = get_terms(array(
                 'taxonomy' => $post_taxonomy->name,
                 'hide_empty' => true
-            ) );
+            ));
 
-            foreach ( $post_taxonomy_terms as $post_taxonomy_term ) {
+            foreach ($post_taxonomy_terms as $post_taxonomy_term) {
                 array_push($group_taxonomy['group_options'],
                     array(
-                        'label'        => ' - ' . $post_taxonomy_term->name,
+                        'label'        => ' - '.$post_taxonomy_term->name,
                         'value'        => $post_taxonomy_term->slug,
                         'taxonomy'    => 'post_taxonomy',
                     )
@@ -188,12 +188,12 @@ 
 
                 $post_term_children = get_term_children($post_taxonomy_term->term_id, $post_taxonomy->name);
 
-                foreach ( $post_term_children as $post_term_child ) {
-                    $child_term =  get_term_by('id', $post_term_child, $post_taxonomy->name);
+                foreach ($post_term_children as $post_term_child) {
+                    $child_term = get_term_by('id', $post_term_child, $post_taxonomy->name);
 
                     array_push($group_taxonomy['group_options'],
                         array(
-                            'label'        => ' -- ' . $child_term->name,
+                            'label'        => ' -- '.$child_term->name,
                             'value'        => $child_term->slug,
                             'taxonomy'    => 'post_taxonomy',
                         )
@@ -213,21 +213,21 @@ 
 	 *
 	 * @return array The array of the terms for the taxonomy.
 	 */
-	public static function get_terms_for_the_posted_taxonomy( $request ) {
+	public static function get_terms_for_the_posted_taxonomy($request) {
 		$post_data = $request->get_params();
-		if ( ! array_key_exists( 'taxonomy', $post_data ) ) {
+		if ( ! array_key_exists('taxonomy', $post_data)) {
 			return array(
 				'status'  => 'failure',
-				'message' => __( 'Request not valid, must post a taxonomy to get terms', 'wordlift' )
+				'message' => __('Request not valid, must post a taxonomy to get terms', 'wordlift')
 			);
 		} else {
 			$taxonomy = $post_data['taxonomy'];
-			$terms    = get_terms( $taxonomy, array( 'hide_empty' => false, ) );
-			if ( is_wp_error( $terms ) ) {
+			$terms    = get_terms($taxonomy, array('hide_empty' => false,));
+			if (is_wp_error($terms)) {
 				// Return error response, if the taxonomy is not valid.
 				return array(
 					'status'  => 'failure',
-					'message' => __( 'Request not valid, must post a valid taxonomy', 'wordlift' )
+					'message' => __('Request not valid, must post a valid taxonomy', 'wordlift')
 				);
 			}
 
@@ -242,43 +242,43 @@ 
 	 *
 	 * @return array
 	 */
-	public static function clone_mapping_items( $request ) {
+	public static function clone_mapping_items($request) {
 		$dbo           = new Mappings_DBO();
 		$post_data     = (array) $request->get_params();
 		$mapping_items = (array) $post_data['mapping_items'];
-		foreach ( $mapping_items as $mapping_item ) {
+		foreach ($mapping_items as $mapping_item) {
 			$mapping_id = (int) $mapping_item['mapping_id'];
 			// Clone the current mapping item.
-			$cloned_mapping_id = $dbo->insert_mapping_item( $mapping_item['mapping_title'] );
+			$cloned_mapping_id = $dbo->insert_mapping_item($mapping_item['mapping_title']);
 			// Clone all the rule groups.
-			$rule_groups_to_be_cloned = $dbo->get_rule_groups_by_mapping( $mapping_id );
+			$rule_groups_to_be_cloned = $dbo->get_rule_groups_by_mapping($mapping_id);
 			// Clone all the properties.
-			$properties_to_be_cloned = $dbo->get_properties( $mapping_id );
-			foreach ( $properties_to_be_cloned as $property ) {
+			$properties_to_be_cloned = $dbo->get_properties($mapping_id);
+			foreach ($properties_to_be_cloned as $property) {
 				// Assign a new mapping id.
 				$property['mapping_id'] = $cloned_mapping_id;
 				// Removing this property id, since a new id needed to be created for
 				// new property.
-				unset( $property['property_id'] );
-				$dbo->insert_or_update_property( $property );
+				unset($property['property_id']);
+				$dbo->insert_or_update_property($property);
 			}
 			// Loop through the rule groups and insert them in table with the mapping id.
-			foreach ( $rule_groups_to_be_cloned as $rule_group ) {
-				$cloned_rule_group_id = $dbo->insert_rule_group( $cloned_mapping_id );
+			foreach ($rule_groups_to_be_cloned as $rule_group) {
+				$cloned_rule_group_id = $dbo->insert_rule_group($cloned_mapping_id);
 				$original_rules       = (array) $rule_group['rules'];
 				// Now we need to insert these rules for the cloned rule group id.
-				foreach ( $original_rules as $clone_rule ) {
+				foreach ($original_rules as $clone_rule) {
 					// We should replace only rule group id in the cloned rules.
 					$clone_rule['rule_group_id'] = (int) $cloned_rule_group_id;
-					unset( $clone_rule['rule_id'] );
-					$dbo->insert_or_update_rule_item( $clone_rule );
+					unset($clone_rule['rule_id']);
+					$dbo->insert_or_update_rule_item($clone_rule);
 				}
 			}
 		}
 
 		return array(
 			'status'  => 'success',
-			'message' => __( 'Successfully cloned mapping items', 'wordlift' ),
+			'message' => __('Successfully cloned mapping items', 'wordlift'),
 		);
 	}
 
@@ -289,23 +289,23 @@ 
 	 *
 	 * @return array
 	 */
-	public static function update_mapping_items( $request ) {
+	public static function update_mapping_items($request) {
 		$dbo       = new Mappings_DBO();
 		$post_data = $request->get_params();
-		if ( array_key_exists( 'mapping_items', $post_data ) ) {
+		if (array_key_exists('mapping_items', $post_data)) {
 			$mapping_items = (array) $post_data['mapping_items'];
-			foreach ( $mapping_items as $mapping_item ) {
-				$dbo->insert_or_update_mapping_item( $mapping_item );
+			foreach ($mapping_items as $mapping_item) {
+				$dbo->insert_or_update_mapping_item($mapping_item);
 			}
 
 			return array(
 				'status'  => 'success',
-				'message' => __( 'Mapping items successfully updated', 'wordlift' ),
+				'message' => __('Mapping items successfully updated', 'wordlift'),
 			);
 		} else {
 			return array(
 				'status'  => 'failure',
-				'message' => __( 'Unable to update mapping item', 'wordlift' ),
+				'message' => __('Unable to update mapping item', 'wordlift'),
 			);
 		}
 	}
@@ -317,23 +317,23 @@ 
 	 *
 	 * @return array
 	 */
-	public static function delete_mapping_items( $request ) {
+	public static function delete_mapping_items($request) {
 		$dbo       = new Mappings_DBO();
 		$post_data = $request->get_params();
-		if ( array_key_exists( 'mapping_items', $post_data ) ) {
+		if (array_key_exists('mapping_items', $post_data)) {
 			$mapping_items = (array) $post_data['mapping_items'];
-			foreach ( $mapping_items as $mapping_item ) {
-				$dbo->delete_mapping_item( $mapping_item['mapping_id'] );
+			foreach ($mapping_items as $mapping_item) {
+				$dbo->delete_mapping_item($mapping_item['mapping_id']);
 			}
 
 			return array(
 				'status'  => 'success',
-				'message' => __( 'successfully deleted mapping items', 'wordlift' )
+				'message' => __('successfully deleted mapping items', 'wordlift')
 			);
 		} else {
 			return array(
 				'status'  => 'failure',
-				'message' => __( 'Unable to delete mapping items', 'wordlift' )
+				'message' => __('Unable to delete mapping items', 'wordlift')
 			);
 		}
 	}
@@ -345,7 +345,7 @@ 
 	 *
 	 * @return array
 	 */
-	public static function list_mapping_items( $request ) {
+	public static function list_mapping_items($request) {
 		$dbo = new Mappings_DBO();
 
 		return $dbo->get_mappings();
@@ -359,11 +359,11 @@ 
 	 *
 	 * @return array A list of rule ids.
 	 */
-	private static function get_rule_ids( $dbo, $rule_group_id ) {
-		$rule_rows_in_db = $dbo->get_rules_by_rule_group( $rule_group_id );
+	private static function get_rule_ids($dbo, $rule_group_id) {
+		$rule_rows_in_db = $dbo->get_rules_by_rule_group($rule_group_id);
 		$rule_ids        = array();
-		foreach ( $rule_rows_in_db as $rule_row ) {
-			array_push( $rule_ids, (int) $rule_row['rule_id'] );
+		foreach ($rule_rows_in_db as $rule_row) {
+			array_push($rule_ids, (int) $rule_row['rule_id']);
 		}
 
 		return $rule_ids;
@@ -378,27 +378,27 @@ 
 	 *
 	 * @return void
 	 */
-	private static function save_rules( $dbo, $rule_group_id, $rule_list ) {
-		$rule_ids = self::get_rule_ids( $dbo, $rule_group_id );
-		foreach ( $rule_list as $rule ) {
+	private static function save_rules($dbo, $rule_group_id, $rule_list) {
+		$rule_ids = self::get_rule_ids($dbo, $rule_group_id);
+		foreach ($rule_list as $rule) {
 			// Some rules may not have rule group id, because they are inserted
 			// in ui, so lets add them any way.
 			$rule['rule_group_id'] = $rule_group_id;
-			$dbo->insert_or_update_rule_item( $rule );
-			if ( array_key_exists( 'rule_id', $rule ) ) {
+			$dbo->insert_or_update_rule_item($rule);
+			if (array_key_exists('rule_id', $rule)) {
 				$index_to_be_removed = array_search(
 					(int) $rule['rule_id'],
 					$rule_ids,
 					true
 				);
-				if ( false !== $index_to_be_removed ) {
-					unset( $rule_ids[ $index_to_be_removed ] );
+				if (false !== $index_to_be_removed) {
+					unset($rule_ids[$index_to_be_removed]);
 				}
 			}
 		}
-		foreach ( $rule_ids as $rule_id ) {
+		foreach ($rule_ids as $rule_id) {
 			// Delete all the rule ids which are not posted.
-			$dbo->delete_rule_item( $rule_id );
+			$dbo->delete_rule_item($rule_id);
 		}
 	}
 
@@ -411,14 +411,14 @@ 
 	 *
 	 * @return void
 	 */
-	private static function save_property_list( $dbo, $mapping_id, $property_list ) {
-		$properties_needed_to_be_deleted = $dbo->get_properties( $mapping_id );
+	private static function save_property_list($dbo, $mapping_id, $property_list) {
+		$properties_needed_to_be_deleted = $dbo->get_properties($mapping_id);
 		$property_ids                    = array();
-		foreach ( $properties_needed_to_be_deleted as $property ) {
-			array_push( $property_ids, (int) $property['property_id'] );
+		foreach ($properties_needed_to_be_deleted as $property) {
+			array_push($property_ids, (int) $property['property_id']);
 		}
-		foreach ( $property_list as $property ) {
-			if ( array_key_exists( 'property_id', $property ) ) {
+		foreach ($property_list as $property) {
+			if (array_key_exists('property_id', $property)) {
 				// Remove the id from the list of property ids needed to be deleted
 				// because it is posted.
 				$index_to_be_removed = array_search(
@@ -426,18 +426,18 @@ 
 					$property_ids,
 					true
 				);
-				if ( false !== $index_to_be_removed ) {
-					unset( $property_ids[ $index_to_be_removed ] );
+				if (false !== $index_to_be_removed) {
+					unset($property_ids[$index_to_be_removed]);
 				}
 			}
 			// Add mapping id to property data.
 			$property['mapping_id'] = $mapping_id;
-			$dbo->insert_or_update_property( $property );
+			$dbo->insert_or_update_property($property);
 
 		}
 		// At the end remove all the property ids which are not posted.
-		foreach ( $property_ids as $property_id ) {
-			$dbo->delete_property( $property_id );
+		foreach ($property_ids as $property_id) {
+			$dbo->delete_property($property_id);
 		}
 
 	}
@@ -450,11 +450,11 @@ 
 	 *
 	 * @return array $rule_group_ids A list of rule group ids.
 	 */
-	private static function get_rule_group_ids( $dbo, $mapping_id ) {
-		$rule_group_rows = $dbo->get_rule_group_list( $mapping_id );
+	private static function get_rule_group_ids($dbo, $mapping_id) {
+		$rule_group_rows = $dbo->get_rule_group_list($mapping_id);
 		$rule_group_ids  = array();
-		foreach ( $rule_group_rows as $rule_group_row ) {
-			array_push( $rule_group_ids, (int) $rule_group_row['rule_group_id'] );
+		foreach ($rule_group_rows as $rule_group_row) {
+			array_push($rule_group_ids, (int) $rule_group_row['rule_group_id']);
 		}
 
 		return $rule_group_ids;
@@ -469,31 +469,31 @@ 
 	 *
 	 * @return void
 	 */
-	private static function save_rule_group_list( $dbo, $mapping_id, $rule_group_list ) {
+	private static function save_rule_group_list($dbo, $mapping_id, $rule_group_list) {
 		// The rule groups not posted should be deleted.
-		$rule_group_ids = self::get_rule_group_ids( $dbo, $mapping_id );
+		$rule_group_ids = self::get_rule_group_ids($dbo, $mapping_id);
 		// Loop through rule group list and save the rule group.
-		foreach ( $rule_group_list as $rule_group ) {
-			if ( array_key_exists( 'rule_group_id', $rule_group ) ) {
+		foreach ($rule_group_list as $rule_group) {
+			if (array_key_exists('rule_group_id', $rule_group)) {
 				$rule_group_id = $rule_group['rule_group_id'];
 			} else {
 				// New rule group, should create new rule group id.
-				$rule_group_id = $dbo->insert_rule_group( $mapping_id );
+				$rule_group_id = $dbo->insert_rule_group($mapping_id);
 			}
 			$index_to_be_removed = array_search(
 				(int) $rule_group_id,
 				$rule_group_ids,
 				true
 			);
-			if ( false !== $index_to_be_removed ) {
-				unset( $rule_group_ids[ $index_to_be_removed ] );
+			if (false !== $index_to_be_removed) {
+				unset($rule_group_ids[$index_to_be_removed]);
 			}
-			self::save_rules( $dbo, $rule_group_id, $rule_group['rules'] );
+			self::save_rules($dbo, $rule_group_id, $rule_group['rules']);
 		}
 
 		// Remove all the rule groups which are not posted.
-		foreach ( $rule_group_ids as $rule_group_id ) {
-			$dbo->delete_rule_group_item( $rule_group_id );
+		foreach ($rule_group_ids as $rule_group_id) {
+			$dbo->delete_rule_group_item($rule_group_id);
 		}
 	}
 
@@ -504,33 +504,33 @@ 
 	 *
 	 * @return array
 	 */
-	public static function insert_or_update_mapping_item( $request ) {
+	public static function insert_or_update_mapping_item($request) {
 		$post_data = $request->get_params() === null ? array() : $request->get_params();
 		$dbo       = new Mappings_DBO();
 		// check if valid object is posted.
-		if ( array_key_exists( 'mapping_title', $post_data ) &&
-		     array_key_exists( 'rule_group_list', $post_data ) &&
-		     array_key_exists( 'property_list', $post_data ) ) {
+		if (array_key_exists('mapping_title', $post_data) &&
+		     array_key_exists('rule_group_list', $post_data) &&
+		     array_key_exists('property_list', $post_data)) {
 			// Do validation, remove all incomplete data.
 			$mapping_item = array();
-			if ( array_key_exists( 'mapping_id', $post_data ) ) {
+			if (array_key_exists('mapping_id', $post_data)) {
 				$mapping_item['mapping_id'] = $post_data['mapping_id'];
 			}
 			$mapping_item['mapping_title'] = $post_data['mapping_title'];
 			// lets save the mapping item.
-			$mapping_id = $dbo->insert_or_update_mapping_item( $mapping_item );
-			self::save_rule_group_list( $dbo, $mapping_id, $post_data['rule_group_list'] );
-			self::save_property_list( $dbo, $mapping_id, $post_data['property_list'] );
+			$mapping_id = $dbo->insert_or_update_mapping_item($mapping_item);
+			self::save_rule_group_list($dbo, $mapping_id, $post_data['rule_group_list']);
+			self::save_property_list($dbo, $mapping_id, $post_data['property_list']);
 
 			return array(
 				'status'     => 'success',
-				'message'    => __( 'Successfully saved mapping item', 'wordlift' ),
+				'message'    => __('Successfully saved mapping item', 'wordlift'),
 				'mapping_id' => (int) $mapping_id,
 			);
 		} else {
 			return array(
 				'status'  => 'error',
-				'message' => __( 'Unable to save mapping item', 'wordlift' ),
+				'message' => __('Unable to save mapping item', 'wordlift'),
 			);
 		}
 	}

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

Indentation +351 added lines, -351 removed lines

@@ -24,235 +24,235 @@ 
  */
 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'];
+    /** 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'];
 
         $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
-		);
-	}
+        // 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.
@@ -275,132 +275,132 @@ 
         );
     }
 
-	/**
-	 * 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 ) {
+    /**
+     * 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 +46 added lines, -46 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,17 +202,17 @@ 
 			);
 		}
 		// 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'];
 
-        $post_taxonomy_array =  self::get_post_taxonomy_key_and_value();
+        $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 );
+		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
@@ -225,8 +225,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' => '',
@@ -237,7 +237,7 @@ 
 			'objects'
 		);
 
-		foreach ( $post_types as $post_type ) {
+		foreach ($post_types as $post_type) {
 			array_push(
 				$post_type_option_values,
 				array(
@@ -261,8 +261,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,
             // Left empty since post types are provided locally.
             'api_source' => 'taxonomy'
@@ -281,14 +281,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,
 			),
 		);
@@ -302,12 +302,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;
@@ -319,13 +319,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;
 	}
@@ -337,7 +337,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'];
@@ -387,14 +387,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/validators/class-post-taxonomy-term-rule-validator.php

Indentation +43 added lines, -43 removed lines

@@ -17,48 +17,48 @@ 
  * @package Wordlift\Mappings\Validators
  */
 class Post_Taxonomy_Term_Rule_Validator implements Rule_Validator {
-	/**
-	 * @since 3.25.0
-	 * Enum for the post type rule validator.
-	 */
-	const POST_TAXONOMY = 'post_taxonomy';
-
-	/**
-	 * 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_TAXONOMY ] = $this;
-
-		return $value;
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_label() {
-		return __( 'Post Taxonomy', 'wordlift' );
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function is_valid ( $identifier, $operator, $operand_1, $operand_2, $type ) {
+    /**
+     * @since 3.25.0
+     * Enum for the post type rule validator.
+     */
+    const POST_TAXONOMY = 'post_taxonomy';
+
+    /**
+     * 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_TAXONOMY ] = $this;
+
+        return $value;
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function get_label() {
+        return __( 'Post Taxonomy', 'wordlift' );
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function is_valid ( $identifier, $operator, $operand_1, $operand_2, $type ) {
         $taxonomy  = $operand_1;
         $term_slug = $operand_2;
 
@@ -88,5 +88,5 @@ 
         }
 
         return true;
-	}
+    }
 }

Spacing +15 added lines, -15 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_TAXONOMY ] = $this;
+		$value[self::POST_TAXONOMY] = $this;
 
 		return $value;
 	}
@@ -52,39 +52,39 @@ 
 	 * {@inheritdoc}
 	 */
 	public function get_label() {
-		return __( 'Post Taxonomy', 'wordlift' );
+		return __('Post Taxonomy', 'wordlift');
 	}
 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function is_valid ( $identifier, $operator, $operand_1, $operand_2, $type ) {
+	public function is_valid($identifier, $operator, $operand_1, $operand_2, $type) {
         $taxonomy  = $operand_1;
         $term_slug = $operand_2;
 
-        $current_term = get_term( $identifier );
-        $terms = get_terms( $taxonomy, array( 'get' => 'all' ) );
+        $current_term = get_term($identifier);
+        $terms = get_terms($taxonomy, array('get' => 'all'));
 
-        $is_object_in_term = is_object_in_term( $identifier, $taxonomy, $term_slug );
+        $is_object_in_term = is_object_in_term($identifier, $taxonomy, $term_slug);
 
-        $taxonomy = get_taxonomy( $term_slug );
+        $taxonomy = get_taxonomy($term_slug);
 
-        if (!in_array('post', $taxonomy->object_type )) {
+        if ( ! in_array('post', $taxonomy->object_type)) {
             return false;
         }
 
-        if ( is_wp_error( $is_object_in_term ) ) {
+        if (is_wp_error($is_object_in_term)) {
             return false;
         }
 
-        if ( $operator === Rule_Validator::IS_EQUAL_TO ) {
+        if ($operator === Rule_Validator::IS_EQUAL_TO) {
             // 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);
         }
 
         return true;