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 +108 added lines, -108 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,11 +164,11 @@ 
      *
      * @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) {
             $taxonomy_config = array(
                 'taxonomy' => $post_taxonomy->name,
                 'hide_empty' => false
@@ -176,7 +176,7 @@ 
 
             $total_terms = wp_count_terms($taxonomy_config);
 
-            $post_taxonomy_terms = get_terms( $taxonomy_config );
+            $post_taxonomy_terms = get_terms($taxonomy_config);
 
             if ($total_terms) {
                 $group_taxonomy = array('parentValue' => 'post_taxonomy', 'group_name' => $post_taxonomy->label, 'group_options' => array());
@@ -184,7 +184,7 @@ 
                 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',
                         )
@@ -197,7 +197,7 @@ 
 
                         array_push($group_taxonomy['group_options'],
                             array(
-                                'label' => ' -- ' . $child_term->name,
+                                'label' => ' -- '.$child_term->name,
                                 'value' => $child_term->slug,
                                 'taxonomy' => 'post_taxonomy',
                             )
@@ -218,21 +218,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')
 				);
 			}
 
@@ -247,43 +247,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'),
 		);
 	}
 
@@ -294,23 +294,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'),
 			);
 		}
 	}
@@ -322,23 +322,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')
 			);
 		}
 	}
@@ -350,7 +350,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();
@@ -364,11 +364,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;
@@ -383,27 +383,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);
 		}
 	}
 
@@ -416,14 +416,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(
@@ -431,18 +431,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);
 		}
 
 	}
@@ -455,11 +455,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;
@@ -474,31 +474,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);
 		}
 	}
 
@@ -509,33 +509,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,
             'api_source' => 'post_taxonomy'
         );
@@ -280,14 +280,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,
 			),
 		);
@@ -301,12 +301,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;
@@ -318,13 +318,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;
 	}
@@ -336,7 +336,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'];
@@ -386,14 +386,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/dataset/background/class-sync-background-process-stopped-state.php

Doc Comments +3 added lines

@@ -9,6 +9,9 @@
 	 */
 	private $context;
 
+	/**
+	 * @param Sync_Background_Process $context
+	 */
 	function __construct( $context ) {
 		parent::__construct( Sync_Background_Process::STATE_STOPPED );
 

Indentation +18 added lines, -18 removed lines

@@ -4,30 +4,30 @@
 
 class Sync_Background_Process_Stopped_State extends Abstract_Sync_Background_Process_State {
 
-	/**
-	 * @var Sync_Background_Process
-	 */
-	private $context;
+    /**
+     * @var Sync_Background_Process
+     */
+    private $context;
 
-	function __construct( $context ) {
-		parent::__construct( Sync_Background_Process::STATE_STOPPED );
+    function __construct( $context ) {
+        parent::__construct( Sync_Background_Process::STATE_STOPPED );
 
-		$this->context = $context;
-	}
+        $this->context = $context;
+    }
 
-	function enter() {
-		$this->context->set_state( Sync_Background_Process::STATE_STOPPED );
-	}
+    function enter() {
+        $this->context->set_state( Sync_Background_Process::STATE_STOPPED );
+    }
 
-	function leave() {
-		$this->context->set_state( null );
-	}
+    function leave() {
+        $this->context->set_state( null );
+    }
 
-	function task( $item ) {
+    function task( $item ) {
 
-		$this->context->cancel_process();
+        $this->context->cancel_process();
 
-		return false;
-	}
+        return false;
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -9,21 +9,21 @@
 	 */
 	private $context;
 
-	function __construct( $context ) {
-		parent::__construct( Sync_Background_Process::STATE_STOPPED );
+	function __construct($context) {
+		parent::__construct(Sync_Background_Process::STATE_STOPPED);
 
 		$this->context = $context;
 	}
 
 	function enter() {
-		$this->context->set_state( Sync_Background_Process::STATE_STOPPED );
+		$this->context->set_state(Sync_Background_Process::STATE_STOPPED);
 	}
 
 	function leave() {
-		$this->context->set_state( null );
+		$this->context->set_state(null);
 	}
 
-	function task( $item ) {
+	function task($item) {
 
 		$this->context->cancel_process();
 

src/wordlift/cache/class-ttl-cache.php

Indentation +157 added lines, -157 removed lines

@@ -17,177 +17,177 @@
 // @@todo: add a hook to clear the cached files now and then.
 class Ttl_Cache {
 
-	/**
-	 * The cache name.
-	 *
-	 * @var string $name The cache name.
-	 * @access private
-	 * @since 3.21.2
-	 */
-	private $name;
-
-	/**
-	 * The TTL of cached responses in seconds.
-	 *
-	 * @var int $ttl The TTL in seconds.
-	 * @access private
-	 * @since 3.21.2
-	 */
-	private $ttl;
-
-	/**
-	 * The cache dir where the cached data is written.
-	 *
-	 * @since 3.21.2
-	 * @access private
-	 * @var string $cache_dir The cache dir where the cached responses are written.
-	 */
-	private $cache_dir;
-
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @var Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 * @access private
-	 * @since 3.21.2
-	 */
-	private $log;
-
-	/**
-	 * @var array
-	 */
-	private static $caches = array();
-
-	/**
-	 * Create a {@link Ttl_Cache} with the specified TTL, default 900 secs.
-	 *
-	 * @param string $name The cache name.
-	 * @param int $ttl The cache TTL, default 900 secs.
-	 *
-	 * @since 3.21.2
-	 */
-	public function __construct( $name, $ttl = 900 ) {
-
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
-
-		$this->name = $name;
-		$this->ttl  = $ttl;
-
-		$this->cache_dir = self::get_cache_folder() . DIRECTORY_SEPARATOR . md5( $name );
-
-		$this->log->trace( "Creating the cache folder {$this->cache_dir}..." );
-		wp_mkdir_p( $this->cache_dir );
-
-		self::$caches[ $name ] = $this;
-
-	}
-
-	/**
-	 * Get the root cache folder.
-	 *
-	 * This is useful to introduce a cache cleaning procedure which will scan and delete older stale cache files.
-	 *
-	 * @return string The root cache folder.
-	 * @since 3.22.5
-	 */
-	public static function get_cache_folder() {
-
-		// Get the temp dir and add the directory separator if missing.
-		$temp_dir = get_temp_dir();
-		if ( DIRECTORY_SEPARATOR !== substr( $temp_dir, - strlen( DIRECTORY_SEPARATOR ) ) ) {
-			$temp_dir .= DIRECTORY_SEPARATOR;
-		}
-
-		return $temp_dir . 'wl.cache' . DIRECTORY_SEPARATOR . md5( home_url() );
-	}
-
-	/**
-	 * Get the cached data for the specified key.
-	 *
-	 * @param mixed $key A serializable key.
-	 *
-	 * @return mixed|null
-	 * @since 3.21.2
-	 */
-	public function get( $key ) {
-
-		$filename = $this->get_filename( $key );
-
-		// If the cache file exists and it's not too old, then return it.
-		if ( file_exists( $filename ) && $this->ttl >= time() - filemtime( $filename ) ) {
-			$this->log->trace( "Cache HIT.\n" );
-
-			return json_decode( file_get_contents( $filename ), true );
-		}
-
-		$this->log->trace( "Cache MISS, filename $filename.\n" );
-
-		return null;
-	}
-
-	public function put( $key, $data ) {
-
-		$filename = $this->get_filename( $key );
-
-		// Cache.
-		if ( file_exists( $filename ) ) {
-			@unlink( $filename );
-		}
-		@file_put_contents( $filename, wp_json_encode( $data ) );
+    /**
+     * The cache name.
+     *
+     * @var string $name The cache name.
+     * @access private
+     * @since 3.21.2
+     */
+    private $name;
+
+    /**
+     * The TTL of cached responses in seconds.
+     *
+     * @var int $ttl The TTL in seconds.
+     * @access private
+     * @since 3.21.2
+     */
+    private $ttl;
+
+    /**
+     * The cache dir where the cached data is written.
+     *
+     * @since 3.21.2
+     * @access private
+     * @var string $cache_dir The cache dir where the cached responses are written.
+     */
+    private $cache_dir;
+
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @var Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     * @access private
+     * @since 3.21.2
+     */
+    private $log;
+
+    /**
+     * @var array
+     */
+    private static $caches = array();
+
+    /**
+     * Create a {@link Ttl_Cache} with the specified TTL, default 900 secs.
+     *
+     * @param string $name The cache name.
+     * @param int $ttl The cache TTL, default 900 secs.
+     *
+     * @since 3.21.2
+     */
+    public function __construct( $name, $ttl = 900 ) {
+
+        $this->log = Wordlift_Log_Service::get_logger( get_class() );
+
+        $this->name = $name;
+        $this->ttl  = $ttl;
+
+        $this->cache_dir = self::get_cache_folder() . DIRECTORY_SEPARATOR . md5( $name );
+
+        $this->log->trace( "Creating the cache folder {$this->cache_dir}..." );
+        wp_mkdir_p( $this->cache_dir );
+
+        self::$caches[ $name ] = $this;
+
+    }
+
+    /**
+     * Get the root cache folder.
+     *
+     * This is useful to introduce a cache cleaning procedure which will scan and delete older stale cache files.
+     *
+     * @return string The root cache folder.
+     * @since 3.22.5
+     */
+    public static function get_cache_folder() {
+
+        // Get the temp dir and add the directory separator if missing.
+        $temp_dir = get_temp_dir();
+        if ( DIRECTORY_SEPARATOR !== substr( $temp_dir, - strlen( DIRECTORY_SEPARATOR ) ) ) {
+            $temp_dir .= DIRECTORY_SEPARATOR;
+        }
+
+        return $temp_dir . 'wl.cache' . DIRECTORY_SEPARATOR . md5( home_url() );
+    }
+
+    /**
+     * Get the cached data for the specified key.
+     *
+     * @param mixed $key A serializable key.
+     *
+     * @return mixed|null
+     * @since 3.21.2
+     */
+    public function get( $key ) {
+
+        $filename = $this->get_filename( $key );
+
+        // If the cache file exists and it's not too old, then return it.
+        if ( file_exists( $filename ) && $this->ttl >= time() - filemtime( $filename ) ) {
+            $this->log->trace( "Cache HIT.\n" );
+
+            return json_decode( file_get_contents( $filename ), true );
+        }
+
+        $this->log->trace( "Cache MISS, filename $filename.\n" );
+
+        return null;
+    }
+
+    public function put( $key, $data ) {
+
+        $filename = $this->get_filename( $key );
+
+        // Cache.
+        if ( file_exists( $filename ) ) {
+            @unlink( $filename );
+        }
+        @file_put_contents( $filename, wp_json_encode( $data ) );
 
-	}
-
-	public function delete( $key ) {
+    }
+
+    public function delete( $key ) {
 
-		$filename = $this->get_filename( $key );
+        $filename = $this->get_filename( $key );
 
-		// Delete.
-		if ( file_exists( $filename ) ) {
-			@unlink( $filename );
-		}
+        // Delete.
+        if ( file_exists( $filename ) ) {
+            @unlink( $filename );
+        }
 
-	}
+    }
 
-	public function flush() {
+    public function flush() {
 
-		$files = glob( $this->cache_dir . DIRECTORY_SEPARATOR . '*' );
-		foreach ( $files as $file ) { // iterate files
-			if ( is_file( $file ) ) {
-				@unlink( $file );
-			}
-		}
+        $files = glob( $this->cache_dir . DIRECTORY_SEPARATOR . '*' );
+        foreach ( $files as $file ) { // iterate files
+            if ( is_file( $file ) ) {
+                @unlink( $file );
+            }
+        }
 
-	}
+    }
 
-	public static function flush_all() {
+    public static function flush_all() {
 
-		/** @var Ttl_Cache $cache */
-		foreach ( self::$caches as $cache ) {
-			$cache->flush();
-		}
+        /** @var Ttl_Cache $cache */
+        foreach ( self::$caches as $cache ) {
+            $cache->flush();
+        }
 
-	}
+    }
 
-	/**
-	 * Get the full path for the given `$hash`. The file is not checked for its existence.
-	 *
-	 * @param string $hash A file hash.
-	 *
-	 * @return string The full path to the file.
-	 * @since 3.21.2
-	 */
-	private function get_path( $hash ) {
+    /**
+     * Get the full path for the given `$hash`. The file is not checked for its existence.
+     *
+     * @param string $hash A file hash.
+     *
+     * @return string The full path to the file.
+     * @since 3.21.2
+     */
+    private function get_path( $hash ) {
 
-		return $this->cache_dir . DIRECTORY_SEPARATOR . $hash;
-	}
+        return $this->cache_dir . DIRECTORY_SEPARATOR . $hash;
+    }
 
-	private function get_filename( $key ) {
+    private function get_filename( $key ) {
 
-		// Create a hash and a path to the cache file.
-		$hash     = md5( json_encode( $key ) );
-		$filename = $this->get_path( $hash );
+        // Create a hash and a path to the cache file.
+        $hash     = md5( json_encode( $key ) );
+        $filename = $this->get_path( $hash );
 
-		return $filename;
-	}
+        return $filename;
+    }
 
 }

Spacing +33 added lines, -33 removed lines

@@ -66,19 +66,19 @@ 
 	 *
 	 * @since 3.21.2
 	 */
-	public function __construct( $name, $ttl = 900 ) {
+	public function __construct($name, $ttl = 900) {
 
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
+		$this->log = Wordlift_Log_Service::get_logger(get_class());
 
 		$this->name = $name;
 		$this->ttl  = $ttl;
 
-		$this->cache_dir = self::get_cache_folder() . DIRECTORY_SEPARATOR . md5( $name );
+		$this->cache_dir = self::get_cache_folder().DIRECTORY_SEPARATOR.md5($name);
 
-		$this->log->trace( "Creating the cache folder {$this->cache_dir}..." );
-		wp_mkdir_p( $this->cache_dir );
+		$this->log->trace("Creating the cache folder {$this->cache_dir}...");
+		wp_mkdir_p($this->cache_dir);
 
-		self::$caches[ $name ] = $this;
+		self::$caches[$name] = $this;
 
 	}
 
@@ -94,11 +94,11 @@ 
 
 		// Get the temp dir and add the directory separator if missing.
 		$temp_dir = get_temp_dir();
-		if ( DIRECTORY_SEPARATOR !== substr( $temp_dir, - strlen( DIRECTORY_SEPARATOR ) ) ) {
+		if (DIRECTORY_SEPARATOR !== substr($temp_dir, - strlen(DIRECTORY_SEPARATOR))) {
 			$temp_dir .= DIRECTORY_SEPARATOR;
 		}
 
-		return $temp_dir . 'wl.cache' . DIRECTORY_SEPARATOR . md5( home_url() );
+		return $temp_dir.'wl.cache'.DIRECTORY_SEPARATOR.md5(home_url());
 	}
 
 	/**
@@ -109,51 +109,51 @@ 
 	 * @return mixed|null
 	 * @since 3.21.2
 	 */
-	public function get( $key ) {
+	public function get($key) {
 
-		$filename = $this->get_filename( $key );
+		$filename = $this->get_filename($key);
 
 		// If the cache file exists and it's not too old, then return it.
-		if ( file_exists( $filename ) && $this->ttl >= time() - filemtime( $filename ) ) {
-			$this->log->trace( "Cache HIT.\n" );
+		if (file_exists($filename) && $this->ttl >= time() - filemtime($filename)) {
+			$this->log->trace("Cache HIT.\n");
 
-			return json_decode( file_get_contents( $filename ), true );
+			return json_decode(file_get_contents($filename), true);
 		}
 
-		$this->log->trace( "Cache MISS, filename $filename.\n" );
+		$this->log->trace("Cache MISS, filename $filename.\n");
 
 		return null;
 	}
 
-	public function put( $key, $data ) {
+	public function put($key, $data) {
 
-		$filename = $this->get_filename( $key );
+		$filename = $this->get_filename($key);
 
 		// Cache.
-		if ( file_exists( $filename ) ) {
-			@unlink( $filename );
+		if (file_exists($filename)) {
+			@unlink($filename);
 		}
-		@file_put_contents( $filename, wp_json_encode( $data ) );
+		@file_put_contents($filename, wp_json_encode($data));
 
 	}
 
-	public function delete( $key ) {
+	public function delete($key) {
 
-		$filename = $this->get_filename( $key );
+		$filename = $this->get_filename($key);
 
 		// Delete.
-		if ( file_exists( $filename ) ) {
-			@unlink( $filename );
+		if (file_exists($filename)) {
+			@unlink($filename);
 		}
 
 	}
 
 	public function flush() {
 
-		$files = glob( $this->cache_dir . DIRECTORY_SEPARATOR . '*' );
-		foreach ( $files as $file ) { // iterate files
-			if ( is_file( $file ) ) {
-				@unlink( $file );
+		$files = glob($this->cache_dir.DIRECTORY_SEPARATOR.'*');
+		foreach ($files as $file) { // iterate files
+			if (is_file($file)) {
+				@unlink($file);
 			}
 		}
 
@@ -162,7 +162,7 @@ 
 	public static function flush_all() {
 
 		/** @var Ttl_Cache $cache */
-		foreach ( self::$caches as $cache ) {
+		foreach (self::$caches as $cache) {
 			$cache->flush();
 		}
 
@@ -176,16 +176,16 @@ 
 	 * @return string The full path to the file.
 	 * @since 3.21.2
 	 */
-	private function get_path( $hash ) {
+	private function get_path($hash) {
 
-		return $this->cache_dir . DIRECTORY_SEPARATOR . $hash;
+		return $this->cache_dir.DIRECTORY_SEPARATOR.$hash;
 	}
 
-	private function get_filename( $key ) {
+	private function get_filename($key) {
 
 		// Create a hash and a path to the cache file.
-		$hash     = md5( json_encode( $key ) );
-		$filename = $this->get_path( $hash );
+		$hash     = md5(json_encode($key));
+		$filename = $this->get_path($hash);
 
 		return $filename;
 	}

src/wordlift/dataset/index.php

Indentation +19 added lines, -19 removed lines

@@ -12,31 +12,31 @@
 use Wordlift\Jsonld\Jsonld_Service;
 
 if ( ! defined( 'ABSPATH' ) ) {
-	exit;
+    exit;
 }
 // Register the Dataset JSON Endpoint. The `$api_service` variable must be defined in the calling file (wordlift.php).
 if ( apply_filters( 'wl_feature__enable__dataset-ng', false ) ) {
 
-	$sync_object_adapter_factory = new Sync_Object_Adapter_Factory();
-	$sync_service                = new Sync_Service( $api_service, $sync_object_adapter_factory, Jsonld_Service::get_instance(), Wordlift_Entity_Service::get_instance() );
-	new Sync_Post_Hooks( $sync_service, $sync_object_adapter_factory );
-	new Sync_User_Hooks( $sync_service );
+    $sync_object_adapter_factory = new Sync_Object_Adapter_Factory();
+    $sync_service                = new Sync_Service( $api_service, $sync_object_adapter_factory, Jsonld_Service::get_instance(), Wordlift_Entity_Service::get_instance() );
+    new Sync_Post_Hooks( $sync_service, $sync_object_adapter_factory );
+    new Sync_User_Hooks( $sync_service );
 
-	/**
-	 * @since 3.28.0
-	 * @see https://github.com/insideout10/wordlift-plugin/issues/1186
-	 */
-	new Sync_Hooks_Entity_Relation( Wordlift_Entity_Service::get_instance() );
+    /**
+     * @since 3.28.0
+     * @see https://github.com/insideout10/wordlift-plugin/issues/1186
+     */
+    new Sync_Hooks_Entity_Relation( Wordlift_Entity_Service::get_instance() );
 
-	if ( apply_filters( 'wl_feature__enable__wordpress-ontology', false ) ) {
-		new Sync_Hooks_Wordpress_Ontology();
-	}
+    if ( apply_filters( 'wl_feature__enable__wordpress-ontology', false ) ) {
+        new Sync_Hooks_Wordpress_Ontology();
+    }
 
-	if ( apply_filters( 'wl_feature__enable__sync-background', false ) ) {
-		// Set up the sync background process.
-		$sync_background_process = new Sync_Background_Process( $sync_service, $sync_object_adapter_factory );
-		new Sync_Background_Process_Wpjson_Endpoint( $sync_background_process );
-		new Sync_Page();
-	}
+    if ( apply_filters( 'wl_feature__enable__sync-background', false ) ) {
+        // Set up the sync background process.
+        $sync_background_process = new Sync_Background_Process( $sync_service, $sync_object_adapter_factory );
+        new Sync_Background_Process_Wpjson_Endpoint( $sync_background_process );
+        new Sync_Page();
+    }
 
 }

Spacing +10 added lines, -10 removed lines

@@ -11,31 +11,31 @@
 use Wordlift\Dataset\Sync_User_Hooks;
 use Wordlift\Jsonld\Jsonld_Service;
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 // Register the Dataset JSON Endpoint. The `$api_service` variable must be defined in the calling file (wordlift.php).
-if ( apply_filters( 'wl_feature__enable__dataset-ng', false ) ) {
+if (apply_filters('wl_feature__enable__dataset-ng', false)) {
 
 	$sync_object_adapter_factory = new Sync_Object_Adapter_Factory();
-	$sync_service                = new Sync_Service( $api_service, $sync_object_adapter_factory, Jsonld_Service::get_instance(), Wordlift_Entity_Service::get_instance() );
-	new Sync_Post_Hooks( $sync_service, $sync_object_adapter_factory );
-	new Sync_User_Hooks( $sync_service );
+	$sync_service                = new Sync_Service($api_service, $sync_object_adapter_factory, Jsonld_Service::get_instance(), Wordlift_Entity_Service::get_instance());
+	new Sync_Post_Hooks($sync_service, $sync_object_adapter_factory);
+	new Sync_User_Hooks($sync_service);
 
 	/**
 	 * @since 3.28.0
 	 * @see https://github.com/insideout10/wordlift-plugin/issues/1186
 	 */
-	new Sync_Hooks_Entity_Relation( Wordlift_Entity_Service::get_instance() );
+	new Sync_Hooks_Entity_Relation(Wordlift_Entity_Service::get_instance());
 
-	if ( apply_filters( 'wl_feature__enable__wordpress-ontology', false ) ) {
+	if (apply_filters('wl_feature__enable__wordpress-ontology', false)) {
 		new Sync_Hooks_Wordpress_Ontology();
 	}
 
-	if ( apply_filters( 'wl_feature__enable__sync-background', false ) ) {
+	if (apply_filters('wl_feature__enable__sync-background', false)) {
 		// Set up the sync background process.
-		$sync_background_process = new Sync_Background_Process( $sync_service, $sync_object_adapter_factory );
-		new Sync_Background_Process_Wpjson_Endpoint( $sync_background_process );
+		$sync_background_process = new Sync_Background_Process($sync_service, $sync_object_adapter_factory);
+		new Sync_Background_Process_Wpjson_Endpoint($sync_background_process);
 		new Sync_Page();
 	}
 

src/wordlift/dataset/class-sync-hooks-wordpress-ontology.php

Indentation +29 added lines, -29 removed lines

@@ -6,47 +6,47 @@
 
 class Sync_Hooks_Wordpress_Ontology {
 
-	const HTTP_PURL_ORG_WORDPRESS_1_0 = 'http://purl.org/wordpress/1.0/';
+    const HTTP_PURL_ORG_WORDPRESS_1_0 = 'http://purl.org/wordpress/1.0/';
 
-	public function __construct() {
-		add_filter( 'wl_dataset__sync_service__sync_item__jsonld', array( $this, 'jsonld' ), 10, 3 );
-	}
+    public function __construct() {
+        add_filter( 'wl_dataset__sync_service__sync_item__jsonld', array( $this, 'jsonld' ), 10, 3 );
+    }
 
-	public function jsonld( $jsonld, $type, $object_id ) {
+    public function jsonld( $jsonld, $type, $object_id ) {
 
-		$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'id' ] = $object_id;
+        $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'id' ] = $object_id;
 
-		switch ( $type ) {
+        switch ( $type ) {
 
-			case Object_Type_Enum::TERM:
-				$term = get_term( $object_id );
+            case Object_Type_Enum::TERM:
+                $term = get_term( $object_id );
 
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'contentType' ] = 'term';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'name' ]        = $term->name;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'description' ] = $term->description;
-				break;
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'contentType' ] = 'term';
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'name' ]        = $term->name;
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'description' ] = $term->description;
+                break;
 
-			case Object_Type_Enum::USER:
-				$user = get_userdata( $object_id );
+            case Object_Type_Enum::USER:
+                $user = get_userdata( $object_id );
 
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'contentType' ] = 'user';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'displayName' ] = $user->display_name;
-				break;
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'contentType' ] = 'user';
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'displayName' ] = $user->display_name;
+                break;
 
-			case Object_Type_Enum::POST:
-				$post = get_post( $object_id );
+            case Object_Type_Enum::POST:
+                $post = get_post( $object_id );
 
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'contentType' ] = 'post';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'postType' ]    = $post->post_type;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'postTitle' ]   = $post->post_title;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'postStatus' ]  = $post->post_status;
-				break;
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'contentType' ] = 'post';
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'postType' ]    = $post->post_type;
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'postTitle' ]   = $post->post_title;
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'postStatus' ]  = $post->post_status;
+                break;
 
-			default:
+            default:
 
-		}
+        }
 
-		return $jsonld;
-	}
+        return $jsonld;
+    }
 
 }

Spacing +16 added lines, -16 removed lines

@@ -9,37 +9,37 @@
 	const HTTP_PURL_ORG_WORDPRESS_1_0 = 'http://purl.org/wordpress/1.0/';
 
 	public function __construct() {
-		add_filter( 'wl_dataset__sync_service__sync_item__jsonld', array( $this, 'jsonld' ), 10, 3 );
+		add_filter('wl_dataset__sync_service__sync_item__jsonld', array($this, 'jsonld'), 10, 3);
 	}
 
-	public function jsonld( $jsonld, $type, $object_id ) {
+	public function jsonld($jsonld, $type, $object_id) {
 
-		$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'id' ] = $object_id;
+		$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'id'] = $object_id;
 
-		switch ( $type ) {
+		switch ($type) {
 
 			case Object_Type_Enum::TERM:
-				$term = get_term( $object_id );
+				$term = get_term($object_id);
 
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'contentType' ] = 'term';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'name' ]        = $term->name;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'description' ] = $term->description;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'contentType'] = 'term';
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'name']        = $term->name;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'description'] = $term->description;
 				break;
 
 			case Object_Type_Enum::USER:
-				$user = get_userdata( $object_id );
+				$user = get_userdata($object_id);
 
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'contentType' ] = 'user';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'displayName' ] = $user->display_name;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'contentType'] = 'user';
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'displayName'] = $user->display_name;
 				break;
 
 			case Object_Type_Enum::POST:
-				$post = get_post( $object_id );
+				$post = get_post($object_id);
 
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'contentType' ] = 'post';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'postType' ]    = $post->post_type;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'postTitle' ]   = $post->post_title;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'postStatus' ]  = $post->post_status;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'contentType'] = 'post';
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'postType']    = $post->post_type;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'postTitle']   = $post->post_title;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'postStatus']  = $post->post_status;
 				break;
 
 			default:

src/wordlift/dataset/class-sync-object-adapter-factory.php

Indentation +29 added lines, -29 removed lines

@@ -6,34 +6,34 @@
 
 class Sync_Object_Adapter_Factory {
 
-	/**
-	 * @param int $type One of Object_Type_Enum::POST, Object_Type_Enum::USER, ...
-	 * @param int $object_id The object id.
-	 *
-	 * @return Sync_Object_Adapter
-	 * @throws \Exception
-	 */
-	function create( $type, $object_id ) {
-
-		switch ( $type ) {
-			case Object_Type_Enum::POST:
-				return new Sync_Post_Adapter( $object_id );
-			case Object_Type_Enum::USER:
-				return new Sync_User_Adapter( $object_id );
-			case Object_Type_Enum::TERM:
-				return new Sync_Term_Adapter( $object_id );
-			default:
-				throw new \Exception( "Unsupported type $type." );
-		}
-
-	}
-
-	function create_many( $type, $object_ids ) {
-		$that = $this;
-
-		return array_map( function ( $item ) use ( $type, $that ) {
-			return $that->create( $type, $item );
-		}, (array) $object_ids );
-	}
+    /**
+     * @param int $type One of Object_Type_Enum::POST, Object_Type_Enum::USER, ...
+     * @param int $object_id The object id.
+     *
+     * @return Sync_Object_Adapter
+     * @throws \Exception
+     */
+    function create( $type, $object_id ) {
+
+        switch ( $type ) {
+            case Object_Type_Enum::POST:
+                return new Sync_Post_Adapter( $object_id );
+            case Object_Type_Enum::USER:
+                return new Sync_User_Adapter( $object_id );
+            case Object_Type_Enum::TERM:
+                return new Sync_Term_Adapter( $object_id );
+            default:
+                throw new \Exception( "Unsupported type $type." );
+        }
+
+    }
+
+    function create_many( $type, $object_ids ) {
+        $that = $this;
+
+        return array_map( function ( $item ) use ( $type, $that ) {
+            return $that->create( $type, $item );
+        }, (array) $object_ids );
+    }
 
 }

Spacing +10 added lines, -10 removed lines

@@ -13,27 +13,27 @@
 	 * @return Sync_Object_Adapter
 	 * @throws \Exception
 	 */
-	function create( $type, $object_id ) {
+	function create($type, $object_id) {
 
-		switch ( $type ) {
+		switch ($type) {
 			case Object_Type_Enum::POST:
-				return new Sync_Post_Adapter( $object_id );
+				return new Sync_Post_Adapter($object_id);
 			case Object_Type_Enum::USER:
-				return new Sync_User_Adapter( $object_id );
+				return new Sync_User_Adapter($object_id);
 			case Object_Type_Enum::TERM:
-				return new Sync_Term_Adapter( $object_id );
+				return new Sync_Term_Adapter($object_id);
 			default:
-				throw new \Exception( "Unsupported type $type." );
+				throw new \Exception("Unsupported type $type.");
 		}
 
 	}
 
-	function create_many( $type, $object_ids ) {
+	function create_many($type, $object_ids) {
 		$that = $this;
 
-		return array_map( function ( $item ) use ( $type, $that ) {
-			return $that->create( $type, $item );
-		}, (array) $object_ids );
+		return array_map(function($item) use ($type, $that) {
+			return $that->create($type, $item);
+		}, (array) $object_ids);
 	}
 
 }

src/wordlift/dataset/class-sync-post-adapter.php

Indentation +29 added lines, -29 removed lines

@@ -5,34 +5,34 @@
 use Wordlift\Object_Type_Enum;
 
 class Sync_Post_Adapter extends Abstract_Sync_Object_Adapter {
-	/**
-	 * @var int
-	 */
-	private $post_id;
-
-	/**
-	 * Sync_Term_Adapter constructor.
-	 *
-	 * @param int $post_id
-	 *
-	 * @throws \Exception
-	 */
-	function __construct( $post_id ) {
-		parent::__construct( Object_Type_Enum::POST, $post_id );
-
-		$this->post_id = $post_id;
-	}
-
-	function is_published() {
-		return ( 'publish' === get_post_status( $this->post_id ) );
-	}
-
-	function is_public() {
-		// Check if the post type is public.
-		$post_type     = get_post_type( $this->post_id );
-		$post_type_obj = get_post_type_object( $post_type );
-
-		return $post_type_obj->public;
-	}
+    /**
+     * @var int
+     */
+    private $post_id;
+
+    /**
+     * Sync_Term_Adapter constructor.
+     *
+     * @param int $post_id
+     *
+     * @throws \Exception
+     */
+    function __construct( $post_id ) {
+        parent::__construct( Object_Type_Enum::POST, $post_id );
+
+        $this->post_id = $post_id;
+    }
+
+    function is_published() {
+        return ( 'publish' === get_post_status( $this->post_id ) );
+    }
+
+    function is_public() {
+        // Check if the post type is public.
+        $post_type     = get_post_type( $this->post_id );
+        $post_type_obj = get_post_type_object( $post_type );
+
+        return $post_type_obj->public;
+    }
 
 }

src/wordlift/dataset/class-sync-page.php

Indentation +28 added lines, -28 removed lines

@@ -4,39 +4,39 @@ 
 
 class Sync_Page {
 
-	/**
-	 * Sync_Page constructor.
-	 */
-	public function __construct() {
+    /**
+     * Sync_Page constructor.
+     */
+    public function __construct() {
 
-		add_action( 'admin_menu', array( $this, 'admin_menu' ) );
+        add_action( 'admin_menu', array( $this, 'admin_menu' ) );
 
-	}
+    }
 
-	public function admin_menu() {
+    public function admin_menu() {
 
-		add_submenu_page( 'wl_admin_menu', __( 'Synchronize Dataset', 'wordlift' ), __( 'Synchronize Dataset', 'wordlift' ), 'manage_options', 'wl_dataset_sync', array(
-			$this,
-			'render'
-		) );
+        add_submenu_page( 'wl_admin_menu', __( 'Synchronize Dataset', 'wordlift' ), __( 'Synchronize Dataset', 'wordlift' ), 'manage_options', 'wl_dataset_sync', array(
+            $this,
+            'render'
+        ) );
 
-	}
+    }
 
-	public function render() {
+    public function render() {
 
-		wp_enqueue_style(
-			'wl-tasks-page',
-			plugin_dir_url( dirname( __FILE__ ) ) . 'tasks/admin/assets/tasks-page.css',
-			array(),
-			\Wordlift::get_instance()->get_version(),
-			'all' );
-		wp_enqueue_script(
-			'wl-dataset-sync-page',
-			plugin_dir_url( __FILE__ ) . 'assets/sync-page.js',
-			array( 'wp-api' ),
-			\Wordlift::get_instance()->get_version() );
+        wp_enqueue_style(
+            'wl-tasks-page',
+            plugin_dir_url( dirname( __FILE__ ) ) . 'tasks/admin/assets/tasks-page.css',
+            array(),
+            \Wordlift::get_instance()->get_version(),
+            'all' );
+        wp_enqueue_script(
+            'wl-dataset-sync-page',
+            plugin_dir_url( __FILE__ ) . 'assets/sync-page.js',
+            array( 'wp-api' ),
+            \Wordlift::get_instance()->get_version() );
 
-		?>
+        ?>
         <div class="wrap">
             <h2><?php esc_html_e( 'Synchronize Dataset', 'wordlift' ); ?></h2>
 
@@ -46,12 +46,12 @@ 
             </div>
 
             <button id="wl-start-btn" type="button" class="button button-large button-primary"><?php
-				esc_html_e( 'Start', 'wordlift-framework' ); ?></button>
+                esc_html_e( 'Start', 'wordlift-framework' ); ?></button>
             <button id="wl-stop-btn" type="button" class="button button-large button-primary hidden"><?php
-				esc_html_e( 'Stop', 'wordlift-framework' ); ?></button>
+                esc_html_e( 'Stop', 'wordlift-framework' ); ?></button>
 
         </div>
 		<?php
-	}
+    }
 
 }
\ No newline at end of file

Spacing +9 added lines, -9 removed lines

@@ -9,16 +9,16 @@ 
 	 */
 	public function __construct() {
 
-		add_action( 'admin_menu', array( $this, 'admin_menu' ) );
+		add_action('admin_menu', array($this, 'admin_menu'));
 
 	}
 
 	public function admin_menu() {
 
-		add_submenu_page( 'wl_admin_menu', __( 'Synchronize Dataset', 'wordlift' ), __( 'Synchronize Dataset', 'wordlift' ), 'manage_options', 'wl_dataset_sync', array(
+		add_submenu_page('wl_admin_menu', __('Synchronize Dataset', 'wordlift'), __('Synchronize Dataset', 'wordlift'), 'manage_options', 'wl_dataset_sync', array(
 			$this,
 			'render'
-		) );
+		));
 
 	}
 
@@ -26,19 +26,19 @@ 
 
 		wp_enqueue_style(
 			'wl-tasks-page',
-			plugin_dir_url( dirname( __FILE__ ) ) . 'tasks/admin/assets/tasks-page.css',
+			plugin_dir_url(dirname(__FILE__)).'tasks/admin/assets/tasks-page.css',
 			array(),
 			\Wordlift::get_instance()->get_version(),
 			'all' );
 		wp_enqueue_script(
 			'wl-dataset-sync-page',
-			plugin_dir_url( __FILE__ ) . 'assets/sync-page.js',
-			array( 'wp-api' ),
+			plugin_dir_url(__FILE__).'assets/sync-page.js',
+			array('wp-api'),
 			\Wordlift::get_instance()->get_version() );
 
 		?>
         <div class="wrap">
-            <h2><?php esc_html_e( 'Synchronize Dataset', 'wordlift' ); ?></h2>
+            <h2><?php esc_html_e('Synchronize Dataset', 'wordlift'); ?></h2>
 
             <div class="wl-task__progress" style="border: 1px solid #23282D; height: 20px; margin: 8px 0;">
                 <div class="wl-task__progress__bar"
@@ -46,9 +46,9 @@ 
             </div>
 
             <button id="wl-start-btn" type="button" class="button button-large button-primary"><?php
-				esc_html_e( 'Start', 'wordlift-framework' ); ?></button>
+				esc_html_e('Start', 'wordlift-framework'); ?></button>
             <button id="wl-stop-btn" type="button" class="button button-large button-primary hidden"><?php
-				esc_html_e( 'Stop', 'wordlift-framework' ); ?></button>
+				esc_html_e('Stop', 'wordlift-framework'); ?></button>
 
         </div>
 		<?php