insideout10 / wordlift-plugin

src/wordlift/jsonld/class-jsonld-endpoint.php

Indentation +165 added lines, -165 removed lines

@@ -21,157 +21,157 @@ 
  */
 class Jsonld_Endpoint {
 
-	/**
-	 * The {@link Wordlift_Jsonld_Service} instance.
-	 *
-	 * @var Wordlift_Jsonld_Service The {@link Wordlift_Jsonld_Service} instance.
-	 */
-	private $jsonld_service;
-	/**
-	 * @var \Wordlift_Entity_Uri_Service
-	 */
-	private $entity_uri_service;
-
-	/**
-	 * Jsonld_Endpoint constructor.
-	 *
-	 * @param Jsonld_Service $jsonld_service
-	 * @param \Wordlift_Entity_Uri_Service $entity_uri_service
-	 */
-	public function __construct( $jsonld_service, $entity_uri_service ) {
-
-		$this->jsonld_service     = $jsonld_service;
-		$this->entity_uri_service = $entity_uri_service;
-
-		// PHP 5.3 compatibility.
-		$that = $this;
-		add_action( 'rest_api_init', function () use ( $that ) {
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<id>\d+)', array(
-				'methods'             => WP_REST_Server::READABLE,
-				'callback'            => array( $that, 'jsonld_using_post_id' ),
-				'permission_callback' => '__return_true',
-				'args'                => array(
-					'id' => array(
-						'validate_callback' => function ( $param, $request, $key ) {
-							return is_numeric( $param );
-						},
-						'sanitize_callback' => 'absint',
-					),
-				)
-			) );
-
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/http/(?P<item_id>.*)', array(
-				'methods'             => 'GET',
-				'callback'            => array( $that, 'jsonld_using_item_id' ),
-				'permission_callback' => '__return_true',
-			) );
-
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/post-meta/(?P<meta_key>[^/]+)', array(
-				'methods'             => 'GET',
-				'callback'            => array( $that, 'jsonld_using_post_meta' ),
-				'permission_callback' => '__return_true',
-			) );
-
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/meta/(?P<meta_key>[^/]+)', array(
-				'methods'             => 'GET',
-				'callback'            => array( $that, 'jsonld_using_meta' ),
-				'permission_callback' => '__return_true',
-			) );
-
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<post_type>.*)/(?P<post_name>.*)', array(
-				'methods'             => 'GET',
-				'callback'            => array( $that, 'jsonld_using_get_page_by_path' ),
-				'permission_callback' => '__return_true',
-			) );
-
-		} );
-
-	}
-
-	/**
-	 * Callback for the JSON-LD request.
-	 *
-	 * @param array $request {
-	 *  The request array.
-	 *
-	 * @type int $id The post id.
-	 * }
-	 *
-	 * @return WP_REST_Response
-	 * @throws \Exception
-	 */
-	public function jsonld_using_post_id( $request ) {
-
-		$post_id = $request['id'];
-		$type    = ( 0 === $post_id ? Object_Type_Enum::HOMEPAGE : Object_Type_Enum::POST );
-
-		// Send the generated JSON-LD.
-		$data = $this->jsonld_service->get( $type, $post_id, Jsonld_Context_Enum::REST );
-
-		return Jsonld_Response_Helper::to_response( $data );
-	}
-
-	/**
-	 * Provide a JSON-LD given the itemId.
-	 *
-	 * @param array $request {
-	 *  The request array.
-	 *
-	 * @type string $item_id The entity item id.
-	 * }
-	 *
-	 * @return WP_REST_Response
-	 * @throws \Exception
-	 */
-	public function jsonld_using_item_id( $request ) {
-
-		$item_id = 'http://' . $request['item_id'];
-		$post    = $this->entity_uri_service->get_entity( $item_id );
-
-		if ( ! is_a( $post, 'WP_Post' ) ) {
-			return new WP_REST_Response( esc_html( "$item_id not found." ), 404, array( 'Content-Type' => 'text/html' ) );
-		}
-
-		return $this->jsonld_using_post_id( array( 'id' => $post->ID, ) );
-	}
-
-	public function jsonld_using_get_page_by_path( $request ) {
-
-		$post_name = $request['post_name'];
-		$post_type = $request['post_type'];
-
-		global $wpdb;
-
-		$sql = "
+    /**
+     * The {@link Wordlift_Jsonld_Service} instance.
+     *
+     * @var Wordlift_Jsonld_Service The {@link Wordlift_Jsonld_Service} instance.
+     */
+    private $jsonld_service;
+    /**
+     * @var \Wordlift_Entity_Uri_Service
+     */
+    private $entity_uri_service;
+
+    /**
+     * Jsonld_Endpoint constructor.
+     *
+     * @param Jsonld_Service $jsonld_service
+     * @param \Wordlift_Entity_Uri_Service $entity_uri_service
+     */
+    public function __construct( $jsonld_service, $entity_uri_service ) {
+
+        $this->jsonld_service     = $jsonld_service;
+        $this->entity_uri_service = $entity_uri_service;
+
+        // PHP 5.3 compatibility.
+        $that = $this;
+        add_action( 'rest_api_init', function () use ( $that ) {
+            register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<id>\d+)', array(
+                'methods'             => WP_REST_Server::READABLE,
+                'callback'            => array( $that, 'jsonld_using_post_id' ),
+                'permission_callback' => '__return_true',
+                'args'                => array(
+                    'id' => array(
+                        'validate_callback' => function ( $param, $request, $key ) {
+                            return is_numeric( $param );
+                        },
+                        'sanitize_callback' => 'absint',
+                    ),
+                )
+            ) );
+
+            register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/http/(?P<item_id>.*)', array(
+                'methods'             => 'GET',
+                'callback'            => array( $that, 'jsonld_using_item_id' ),
+                'permission_callback' => '__return_true',
+            ) );
+
+            register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/post-meta/(?P<meta_key>[^/]+)', array(
+                'methods'             => 'GET',
+                'callback'            => array( $that, 'jsonld_using_post_meta' ),
+                'permission_callback' => '__return_true',
+            ) );
+
+            register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/meta/(?P<meta_key>[^/]+)', array(
+                'methods'             => 'GET',
+                'callback'            => array( $that, 'jsonld_using_meta' ),
+                'permission_callback' => '__return_true',
+            ) );
+
+            register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<post_type>.*)/(?P<post_name>.*)', array(
+                'methods'             => 'GET',
+                'callback'            => array( $that, 'jsonld_using_get_page_by_path' ),
+                'permission_callback' => '__return_true',
+            ) );
+
+        } );
+
+    }
+
+    /**
+     * Callback for the JSON-LD request.
+     *
+     * @param array $request {
+     *  The request array.
+     *
+     * @type int $id The post id.
+     * }
+     *
+     * @return WP_REST_Response
+     * @throws \Exception
+     */
+    public function jsonld_using_post_id( $request ) {
+
+        $post_id = $request['id'];
+        $type    = ( 0 === $post_id ? Object_Type_Enum::HOMEPAGE : Object_Type_Enum::POST );
+
+        // Send the generated JSON-LD.
+        $data = $this->jsonld_service->get( $type, $post_id, Jsonld_Context_Enum::REST );
+
+        return Jsonld_Response_Helper::to_response( $data );
+    }
+
+    /**
+     * Provide a JSON-LD given the itemId.
+     *
+     * @param array $request {
+     *  The request array.
+     *
+     * @type string $item_id The entity item id.
+     * }
+     *
+     * @return WP_REST_Response
+     * @throws \Exception
+     */
+    public function jsonld_using_item_id( $request ) {
+
+        $item_id = 'http://' . $request['item_id'];
+        $post    = $this->entity_uri_service->get_entity( $item_id );
+
+        if ( ! is_a( $post, 'WP_Post' ) ) {
+            return new WP_REST_Response( esc_html( "$item_id not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+        }
+
+        return $this->jsonld_using_post_id( array( 'id' => $post->ID, ) );
+    }
+
+    public function jsonld_using_get_page_by_path( $request ) {
+
+        $post_name = $request['post_name'];
+        $post_type = $request['post_type'];
+
+        global $wpdb;
+
+        $sql = "
 			SELECT ID
 			FROM $wpdb->posts
 			WHERE post_name = %s
 			 AND post_type = %s
 		";
 
-		$post_id = $wpdb->get_var( $wpdb->prepare( $sql, $post_name, $post_type ) );
+        $post_id = $wpdb->get_var( $wpdb->prepare( $sql, $post_name, $post_type ) );
 
-		if ( is_null( $post_id ) ) {
-			return new WP_REST_Response( esc_html( "$post_name of type $post_type not found." ), 404, array( 'Content-Type' => 'text/html' ) );
-		}
+        if ( is_null( $post_id ) ) {
+            return new WP_REST_Response( esc_html( "$post_name of type $post_type not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+        }
 
-		return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
-	}
+        return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
+    }
 
-	/**
-	 * @param WP_REST_Request $request
-	 *
-	 * @return WP_REST_Response
-	 * @throws \Exception
-	 */
-	public function jsonld_using_post_meta( $request ) {
+    /**
+     * @param WP_REST_Request $request
+     *
+     * @return WP_REST_Response
+     * @throws \Exception
+     */
+    public function jsonld_using_post_meta( $request ) {
 
-		$meta_key   = $request['meta_key'];
-		$meta_value = urldecode( current( $request->get_query_params( 'meta_value' ) ) );
+        $meta_key   = $request['meta_key'];
+        $meta_value = urldecode( current( $request->get_query_params( 'meta_value' ) ) );
 
-		global $wpdb;
+        global $wpdb;
 
-		$sql = "
+        $sql = "
 			SELECT post_id AS ID
 			FROM $wpdb->postmeta
 			WHERE meta_key = %s
@@ -179,24 +179,24 @@ 
 			LIMIT 1
 		";
 
-		$post_id = $wpdb->get_var( $wpdb->prepare( $sql, $meta_key, $meta_value ) );
+        $post_id = $wpdb->get_var( $wpdb->prepare( $sql, $meta_key, $meta_value ) );
 
-		if ( is_null( $post_id ) ) {
-			return new WP_REST_Response( esc_html( "Post with meta key $meta_key and value $meta_value not found." ), 404, array( 'Content-Type' => 'text/html' ) );
-		}
+        if ( is_null( $post_id ) ) {
+            return new WP_REST_Response( esc_html( "Post with meta key $meta_key and value $meta_value not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+        }
 
-		return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
-	}
+        return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
+    }
 
-	public function jsonld_using_meta( $request ) {
+    public function jsonld_using_meta( $request ) {
 
-		global $wpdb;
+        global $wpdb;
 
-		$meta_key   = $request['meta_key'];
-		$meta_value = urldecode( current( $request->get_query_params( 'meta_value' ) ) );
+        $meta_key   = $request['meta_key'];
+        $meta_value = urldecode( current( $request->get_query_params( 'meta_value' ) ) );
 
-		$results = $wpdb->get_results( $wpdb->prepare(
-			"
+        $results = $wpdb->get_results( $wpdb->prepare(
+            "
 			SELECT pm.post_id AS id, %s AS type
 			 FROM {$wpdb->postmeta} pm
 			 	INNER JOIN {$wpdb->posts} p
@@ -207,23 +207,23 @@ 
 			 FROM {$wpdb->termmeta}
 			 WHERE meta_key = %s AND meta_value = %s
 			",
-			Object_Type_Enum::POST,
-			$meta_key,
-			$meta_value,
-			Object_Type_Enum::TERM,
-			$meta_key,
-			$meta_value
-		) );
+            Object_Type_Enum::POST,
+            $meta_key,
+            $meta_value,
+            Object_Type_Enum::TERM,
+            $meta_key,
+            $meta_value
+        ) );
 
-		$jsonld_service = $this->jsonld_service;
+        $jsonld_service = $this->jsonld_service;
 
-		$data = array_reduce( $results, function ( $carry, $result ) use ( $jsonld_service ) {
-			$jsonld = $jsonld_service->get( $result->type, $result->id, Jsonld_Context_Enum::REST );
+        $data = array_reduce( $results, function ( $carry, $result ) use ( $jsonld_service ) {
+            $jsonld = $jsonld_service->get( $result->type, $result->id, Jsonld_Context_Enum::REST );
 
-			return array_merge( $carry, $jsonld );
-		}, array() );
+            return array_merge( $carry, $jsonld );
+        }, array() );
 
-		return Jsonld_Response_Helper::to_response( $data );
-	}
+        return Jsonld_Response_Helper::to_response( $data );
+    }
 
 }

Spacing +49 added lines, -49 removed lines

@@ -38,51 +38,51 @@ 
 	 * @param Jsonld_Service $jsonld_service
 	 * @param \Wordlift_Entity_Uri_Service $entity_uri_service
 	 */
-	public function __construct( $jsonld_service, $entity_uri_service ) {
+	public function __construct($jsonld_service, $entity_uri_service) {
 
 		$this->jsonld_service     = $jsonld_service;
 		$this->entity_uri_service = $entity_uri_service;
 
 		// PHP 5.3 compatibility.
 		$that = $this;
-		add_action( 'rest_api_init', function () use ( $that ) {
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<id>\d+)', array(
+		add_action('rest_api_init', function() use ($that) {
+			register_rest_route(WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<id>\d+)', array(
 				'methods'             => WP_REST_Server::READABLE,
-				'callback'            => array( $that, 'jsonld_using_post_id' ),
+				'callback'            => array($that, 'jsonld_using_post_id'),
 				'permission_callback' => '__return_true',
 				'args'                => array(
 					'id' => array(
-						'validate_callback' => function ( $param, $request, $key ) {
-							return is_numeric( $param );
+						'validate_callback' => function($param, $request, $key) {
+							return is_numeric($param);
 						},
 						'sanitize_callback' => 'absint',
 					),
 				)
-			) );
+			));
 
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/http/(?P<item_id>.*)', array(
+			register_rest_route(WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/http/(?P<item_id>.*)', array(
 				'methods'             => 'GET',
-				'callback'            => array( $that, 'jsonld_using_item_id' ),
+				'callback'            => array($that, 'jsonld_using_item_id'),
 				'permission_callback' => '__return_true',
-			) );
+			));
 
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/post-meta/(?P<meta_key>[^/]+)', array(
+			register_rest_route(WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/post-meta/(?P<meta_key>[^/]+)', array(
 				'methods'             => 'GET',
-				'callback'            => array( $that, 'jsonld_using_post_meta' ),
+				'callback'            => array($that, 'jsonld_using_post_meta'),
 				'permission_callback' => '__return_true',
-			) );
+			));
 
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/meta/(?P<meta_key>[^/]+)', array(
+			register_rest_route(WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/meta/(?P<meta_key>[^/]+)', array(
 				'methods'             => 'GET',
-				'callback'            => array( $that, 'jsonld_using_meta' ),
+				'callback'            => array($that, 'jsonld_using_meta'),
 				'permission_callback' => '__return_true',
-			) );
+			));
 
-			register_rest_route( WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<post_type>.*)/(?P<post_name>.*)', array(
+			register_rest_route(WL_REST_ROUTE_DEFAULT_NAMESPACE, '/jsonld/(?P<post_type>.*)/(?P<post_name>.*)', array(
 				'methods'             => 'GET',
-				'callback'            => array( $that, 'jsonld_using_get_page_by_path' ),
+				'callback'            => array($that, 'jsonld_using_get_page_by_path'),
 				'permission_callback' => '__return_true',
-			) );
+			));
 
 		} );
 
@@ -100,15 +100,15 @@ 
 	 * @return WP_REST_Response
 	 * @throws \Exception
 	 */
-	public function jsonld_using_post_id( $request ) {
+	public function jsonld_using_post_id($request) {
 
 		$post_id = $request['id'];
-		$type    = ( 0 === $post_id ? Object_Type_Enum::HOMEPAGE : Object_Type_Enum::POST );
+		$type    = (0 === $post_id ? Object_Type_Enum::HOMEPAGE : Object_Type_Enum::POST);
 
 		// Send the generated JSON-LD.
-		$data = $this->jsonld_service->get( $type, $post_id, Jsonld_Context_Enum::REST );
+		$data = $this->jsonld_service->get($type, $post_id, Jsonld_Context_Enum::REST);
 
-		return Jsonld_Response_Helper::to_response( $data );
+		return Jsonld_Response_Helper::to_response($data);
 	}
 
 	/**
@@ -123,19 +123,19 @@ 
 	 * @return WP_REST_Response
 	 * @throws \Exception
 	 */
-	public function jsonld_using_item_id( $request ) {
+	public function jsonld_using_item_id($request) {
 
-		$item_id = 'http://' . $request['item_id'];
-		$post    = $this->entity_uri_service->get_entity( $item_id );
+		$item_id = 'http://'.$request['item_id'];
+		$post    = $this->entity_uri_service->get_entity($item_id);
 
-		if ( ! is_a( $post, 'WP_Post' ) ) {
-			return new WP_REST_Response( esc_html( "$item_id not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+		if ( ! is_a($post, 'WP_Post')) {
+			return new WP_REST_Response(esc_html("$item_id not found."), 404, array('Content-Type' => 'text/html'));
 		}
 
-		return $this->jsonld_using_post_id( array( 'id' => $post->ID, ) );
+		return $this->jsonld_using_post_id(array('id' => $post->ID,));
 	}
 
-	public function jsonld_using_get_page_by_path( $request ) {
+	public function jsonld_using_get_page_by_path($request) {
 
 		$post_name = $request['post_name'];
 		$post_type = $request['post_type'];
@@ -149,13 +149,13 @@ 
 			 AND post_type = %s
 		";
 
-		$post_id = $wpdb->get_var( $wpdb->prepare( $sql, $post_name, $post_type ) );
+		$post_id = $wpdb->get_var($wpdb->prepare($sql, $post_name, $post_type));
 
-		if ( is_null( $post_id ) ) {
-			return new WP_REST_Response( esc_html( "$post_name of type $post_type not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+		if (is_null($post_id)) {
+			return new WP_REST_Response(esc_html("$post_name of type $post_type not found."), 404, array('Content-Type' => 'text/html'));
 		}
 
-		return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
+		return $this->jsonld_using_post_id(array('id' => $post_id,));
 	}
 
 	/**
@@ -164,10 +164,10 @@ 
 	 * @return WP_REST_Response
 	 * @throws \Exception
 	 */
-	public function jsonld_using_post_meta( $request ) {
+	public function jsonld_using_post_meta($request) {
 
 		$meta_key   = $request['meta_key'];
-		$meta_value = urldecode( current( $request->get_query_params( 'meta_value' ) ) );
+		$meta_value = urldecode(current($request->get_query_params('meta_value')));
 
 		global $wpdb;
 
@@ -179,23 +179,23 @@ 
 			LIMIT 1
 		";
 
-		$post_id = $wpdb->get_var( $wpdb->prepare( $sql, $meta_key, $meta_value ) );
+		$post_id = $wpdb->get_var($wpdb->prepare($sql, $meta_key, $meta_value));
 
-		if ( is_null( $post_id ) ) {
-			return new WP_REST_Response( esc_html( "Post with meta key $meta_key and value $meta_value not found." ), 404, array( 'Content-Type' => 'text/html' ) );
+		if (is_null($post_id)) {
+			return new WP_REST_Response(esc_html("Post with meta key $meta_key and value $meta_value not found."), 404, array('Content-Type' => 'text/html'));
 		}
 
-		return $this->jsonld_using_post_id( array( 'id' => $post_id, ) );
+		return $this->jsonld_using_post_id(array('id' => $post_id,));
 	}
 
-	public function jsonld_using_meta( $request ) {
+	public function jsonld_using_meta($request) {
 
 		global $wpdb;
 
 		$meta_key   = $request['meta_key'];
-		$meta_value = urldecode( current( $request->get_query_params( 'meta_value' ) ) );
+		$meta_value = urldecode(current($request->get_query_params('meta_value')));
 
-		$results = $wpdb->get_results( $wpdb->prepare(
+		$results = $wpdb->get_results($wpdb->prepare(
 			"
 			SELECT pm.post_id AS id, %s AS type
 			 FROM {$wpdb->postmeta} pm
@@ -213,17 +213,17 @@ 
 			Object_Type_Enum::TERM,
 			$meta_key,
 			$meta_value
-		) );
+		));
 
 		$jsonld_service = $this->jsonld_service;
 
-		$data = array_reduce( $results, function ( $carry, $result ) use ( $jsonld_service ) {
-			$jsonld = $jsonld_service->get( $result->type, $result->id, Jsonld_Context_Enum::REST );
+		$data = array_reduce($results, function($carry, $result) use ($jsonld_service) {
+			$jsonld = $jsonld_service->get($result->type, $result->id, Jsonld_Context_Enum::REST);
 
-			return array_merge( $carry, $jsonld );
-		}, array() );
+			return array_merge($carry, $jsonld);
+		}, array());
 
-		return Jsonld_Response_Helper::to_response( $data );
+		return Jsonld_Response_Helper::to_response($data);
 	}
 
 }

src/wordlift/jsonld/class-jsonld-context-enum.php

Indentation +8 added lines, -8 removed lines

@@ -4,13 +4,13 @@
 
 class Jsonld_Context_Enum {
 
-	const UNKNOWN = 0;
-	/** Web Page output (typically in the head). */
-	const PAGE = 1;
-	/** Carousel. */
-	const CAROUSEL = 2;
-	const FAQ = 3;
-	/** REST services. */
-	const REST = 3;
+    const UNKNOWN = 0;
+    /** Web Page output (typically in the head). */
+    const PAGE = 1;
+    /** Carousel. */
+    const CAROUSEL = 2;
+    const FAQ = 3;
+    /** REST services. */
+    const REST = 3;
 
 }

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/shortcodes/class-wordlift-products-navigator-shortcode-rest.php

Indentation +232 added lines, -232 removed lines

@@ -4,208 +4,208 @@ 
 
 class Wordlift_Products_Navigator_Shortcode_REST extends Wordlift_Shortcode_REST {
 
-	const CACHE_TTL = 3600; // 1 hour
-
-	public function __construct() {
-		parent::__construct(
-			'/products-navigator',
-			array(
-				'post_id' => array(
-					'description' => __( 'Post ID for which Navigator has to be queried', 'wordlift' ),
-					'type'        => 'integer',
-					'required'    => true
-				),
-				'uniqid'  => array(
-					'description' => __( 'Navigator uniqueid', 'wordlift' ),
-					'type'        => 'string',
-					'required'    => true
-				),
-				'limit'   => array(
-					'default'           => 4,
-					'type'              => 'integer',
-					'sanitize_callback' => 'absint'
-				),
-				'offset'  => array(
-					'default'           => 0,
-					'type'              => 'integer',
-					'sanitize_callback' => 'absint'
-				),
-				'sort'    => array(
-					'default'           => 'ID DESC',
-					'sanitize_callback' => 'sanitize_sql_orderby'
-				),
-				'amp'     => array(
-					'sanitize_callback' => 'rest_sanitize_boolean'
-				)
-			)
-		);
-	}
-
-	public function get_data( $request ) {
-
-		// Sanitize and set defaults
-		$navigator_length = $request['limit'];
-		$navigator_offset = $request['offset'];
-		$order_by         = $request['sort'];
-		$post_id          = $request['post_id'];
-		$navigator_id     = $request['uniqid'];
-		$amp              = $request['amp'];
-
-		$post = get_post( $post_id );
-
-		// Post ID has to match an existing item
-		if ( null === $post ) {
-			return new WP_Error( 'rest_invalid_post_id', __( 'Invalid post_id', 'wordlift' ), array( 'status' => 404 ) );
-		}
-
-		// Determine navigator type and call respective get_*_results
-		if ( get_post_type( $post_id ) === Wordlift_Entity_Service::TYPE_NAME ) {
-			$referencing_posts = $this->get_entity_results( $post_id, array(
-				'ID',
-				'post_title',
-			), $order_by, $navigator_length, $navigator_offset );
-		} else {
-			$referencing_posts = $this->get_post_results( $post_id, array(
-				'ID',
-				'post_title',
-			), $order_by, $navigator_length, $navigator_offset );
-		}
-
-		// Fetch directly referencing posts excluding referencing posts via entities
-		$directly_referencing_posts = $this->get_directly_referencing_posts( $post_id, array_map( function ( $referencing_post ) {
-			return $referencing_post->ID;
-		}, $referencing_posts ) );
-
-		// Combine directly referencing posts and referencing posts via entities
-		$referencing_posts = array_merge( $directly_referencing_posts, $referencing_posts );
-
-		// loop over them and take the first one which is not already in the $related_posts
-		$results = array();
-		foreach ( $referencing_posts as $referencing_post ) {
-			$serialized_entity = wl_serialize_entity( $referencing_post->entity_id );
-			$product           = wc_get_product( $referencing_post->ID );
-
-			$result = array(
-				'product' => array(
-					'id'              => $referencing_post->ID,
-					'permalink'       => get_permalink( $referencing_post->ID ),
-					'title'           => $referencing_post->post_title,
-					'thumbnail'       => get_the_post_thumbnail_url( $referencing_post, 'medium' ),
-					'regular_price'   => $product->get_regular_price(),
-					'sale_price'      => $product->get_sale_price(),
-					'price'           => $product->get_price(),
-					'currency_symbol' => get_woocommerce_currency_symbol(),
-					'discount_pc'     => ( $product->get_sale_price() && ( $product->get_regular_price() > 0 ) ) ? round( 1 - ( $product->get_sale_price() / $product->get_regular_price() ), 2 ) * 100 : 0,
-					'average_rating'  => $product->get_average_rating(),
-					'rating_count'    => $product->get_rating_count(),
-					'rating_html'     => wc_get_rating_html( $product->get_average_rating(), $product->get_rating_count() )
-				),
-				'entity'  => array(
-					'id'        => $referencing_post->entity_id,
-					'label'     => $serialized_entity['label'],
-					'mainType'  => $serialized_entity['mainType'],
-					'permalink' => get_permalink( $referencing_post->entity_id ),
-				),
-			);
-
-			$results[] = $result;
-
-		}
-
-		if ( count( $results ) < $navigator_length ) {
-			$results = apply_filters( 'wl_products_navigator_data_placeholder', $results, $navigator_id, $navigator_offset, $navigator_length );
-		}
-
-
-		// Add filler posts if needed
-		$filler_count = $navigator_length - count( $results );
-		if ( $filler_count > 0 ) {
-			$referencing_post_ids = array_map( function ( $p ) {
-				return $p->ID;
-			}, $referencing_posts );
-			/**
-			 * @since 3.28.0
-			 * Filler posts are fetched using this util.
-			 */
-			$filler_posts_util    = new Filler_Posts_Util( $post_id, 'product' );
-			$post_ids_to_be_excluded = array_merge( array( $post_id ), $referencing_post_ids );
-			$filler_posts            = $filler_posts_util->get_product_navigator_response( $filler_count, $post_ids_to_be_excluded );
-			$results                 = array_merge( $results, $filler_posts );
-		}
-
-		// Apply filters after fillers are added
-		foreach ( $results as $result_index => $result ) {
-			$results[ $result_index ]['product'] = apply_filters( 'wl_products_navigator_data_post', $result['product'], intval( $result['product']['id'] ), $navigator_id );
-			$results[ $result_index ]['entity'] = apply_filters( 'wl_products_navigator_data_entity', $result['entity'], intval( $result['entity']['id'] ), $navigator_id );
-		}
-
-		$results = apply_filters( 'wl_products_navigator_results', $results, $navigator_id );
-
-		return $amp ? array(
-			'items' => array(
-				array( 'values' => $results ),
-			),
-		) : $results;
-
-	}
-
-	private function get_directly_referencing_posts( $post_id, $referencing_post_ids ) {
-
-		$directly_referencing_post_ids = Wordlift_Entity_Service::get_instance()->get_related_entities( $post_id );
-
-		$post__in = array_diff( $directly_referencing_post_ids, $referencing_post_ids );
-
-		$directly_referencing_posts = get_posts( array(
-			'meta_query'          => array(
-				array(
-					'key' => '_thumbnail_id'
-				),
-				array(
-					'key'   => '_stock_status',
-					'value' => 'instock'
-				)
-			),
-			'post__in'            => $post__in,
-			'post_type'           => 'product',
-			'ignore_sticky_posts' => 1
-		) );
-
-		$results = array();
-
-		foreach ( $directly_referencing_posts as $post ) {
-			$result             = new stdClass();
-			$result->ID         = $post->ID;
-			$result->post_title = $post->post_title;
-			$result->entity_id  = $post->ID;
-			$results[]          = $result;
-		}
-
-		return $results;
-	}
-
-	private function get_entity_results(
-		$post_id,
-		$fields = array(
-			'ID',
-			'post_title',
-		),
-		$order_by = 'ID DESC',
-		$limit = 10,
-		$offset = 0
-	) {
-		global $wpdb;
-
-		$select = implode( ', ', array_map( function ( $item ) {
-			return "p.$item AS $item";
-		}, (array) $fields ) );
-
-		$order_by = implode( ', ', array_map( function ( $item ) {
-			return "p.$item";
-		}, (array) $order_by ) );
-
-		/** @noinspection SqlNoDataSourceInspection */
-		return $wpdb->get_results(
-			$wpdb->prepare( <<<EOF
+    const CACHE_TTL = 3600; // 1 hour
+
+    public function __construct() {
+        parent::__construct(
+            '/products-navigator',
+            array(
+                'post_id' => array(
+                    'description' => __( 'Post ID for which Navigator has to be queried', 'wordlift' ),
+                    'type'        => 'integer',
+                    'required'    => true
+                ),
+                'uniqid'  => array(
+                    'description' => __( 'Navigator uniqueid', 'wordlift' ),
+                    'type'        => 'string',
+                    'required'    => true
+                ),
+                'limit'   => array(
+                    'default'           => 4,
+                    'type'              => 'integer',
+                    'sanitize_callback' => 'absint'
+                ),
+                'offset'  => array(
+                    'default'           => 0,
+                    'type'              => 'integer',
+                    'sanitize_callback' => 'absint'
+                ),
+                'sort'    => array(
+                    'default'           => 'ID DESC',
+                    'sanitize_callback' => 'sanitize_sql_orderby'
+                ),
+                'amp'     => array(
+                    'sanitize_callback' => 'rest_sanitize_boolean'
+                )
+            )
+        );
+    }
+
+    public function get_data( $request ) {
+
+        // Sanitize and set defaults
+        $navigator_length = $request['limit'];
+        $navigator_offset = $request['offset'];
+        $order_by         = $request['sort'];
+        $post_id          = $request['post_id'];
+        $navigator_id     = $request['uniqid'];
+        $amp              = $request['amp'];
+
+        $post = get_post( $post_id );
+
+        // Post ID has to match an existing item
+        if ( null === $post ) {
+            return new WP_Error( 'rest_invalid_post_id', __( 'Invalid post_id', 'wordlift' ), array( 'status' => 404 ) );
+        }
+
+        // Determine navigator type and call respective get_*_results
+        if ( get_post_type( $post_id ) === Wordlift_Entity_Service::TYPE_NAME ) {
+            $referencing_posts = $this->get_entity_results( $post_id, array(
+                'ID',
+                'post_title',
+            ), $order_by, $navigator_length, $navigator_offset );
+        } else {
+            $referencing_posts = $this->get_post_results( $post_id, array(
+                'ID',
+                'post_title',
+            ), $order_by, $navigator_length, $navigator_offset );
+        }
+
+        // Fetch directly referencing posts excluding referencing posts via entities
+        $directly_referencing_posts = $this->get_directly_referencing_posts( $post_id, array_map( function ( $referencing_post ) {
+            return $referencing_post->ID;
+        }, $referencing_posts ) );
+
+        // Combine directly referencing posts and referencing posts via entities
+        $referencing_posts = array_merge( $directly_referencing_posts, $referencing_posts );
+
+        // loop over them and take the first one which is not already in the $related_posts
+        $results = array();
+        foreach ( $referencing_posts as $referencing_post ) {
+            $serialized_entity = wl_serialize_entity( $referencing_post->entity_id );
+            $product           = wc_get_product( $referencing_post->ID );
+
+            $result = array(
+                'product' => array(
+                    'id'              => $referencing_post->ID,
+                    'permalink'       => get_permalink( $referencing_post->ID ),
+                    'title'           => $referencing_post->post_title,
+                    'thumbnail'       => get_the_post_thumbnail_url( $referencing_post, 'medium' ),
+                    'regular_price'   => $product->get_regular_price(),
+                    'sale_price'      => $product->get_sale_price(),
+                    'price'           => $product->get_price(),
+                    'currency_symbol' => get_woocommerce_currency_symbol(),
+                    'discount_pc'     => ( $product->get_sale_price() && ( $product->get_regular_price() > 0 ) ) ? round( 1 - ( $product->get_sale_price() / $product->get_regular_price() ), 2 ) * 100 : 0,
+                    'average_rating'  => $product->get_average_rating(),
+                    'rating_count'    => $product->get_rating_count(),
+                    'rating_html'     => wc_get_rating_html( $product->get_average_rating(), $product->get_rating_count() )
+                ),
+                'entity'  => array(
+                    'id'        => $referencing_post->entity_id,
+                    'label'     => $serialized_entity['label'],
+                    'mainType'  => $serialized_entity['mainType'],
+                    'permalink' => get_permalink( $referencing_post->entity_id ),
+                ),
+            );
+
+            $results[] = $result;
+
+        }
+
+        if ( count( $results ) < $navigator_length ) {
+            $results = apply_filters( 'wl_products_navigator_data_placeholder', $results, $navigator_id, $navigator_offset, $navigator_length );
+        }
+
+
+        // Add filler posts if needed
+        $filler_count = $navigator_length - count( $results );
+        if ( $filler_count > 0 ) {
+            $referencing_post_ids = array_map( function ( $p ) {
+                return $p->ID;
+            }, $referencing_posts );
+            /**
+             * @since 3.28.0
+             * Filler posts are fetched using this util.
+             */
+            $filler_posts_util    = new Filler_Posts_Util( $post_id, 'product' );
+            $post_ids_to_be_excluded = array_merge( array( $post_id ), $referencing_post_ids );
+            $filler_posts            = $filler_posts_util->get_product_navigator_response( $filler_count, $post_ids_to_be_excluded );
+            $results                 = array_merge( $results, $filler_posts );
+        }
+
+        // Apply filters after fillers are added
+        foreach ( $results as $result_index => $result ) {
+            $results[ $result_index ]['product'] = apply_filters( 'wl_products_navigator_data_post', $result['product'], intval( $result['product']['id'] ), $navigator_id );
+            $results[ $result_index ]['entity'] = apply_filters( 'wl_products_navigator_data_entity', $result['entity'], intval( $result['entity']['id'] ), $navigator_id );
+        }
+
+        $results = apply_filters( 'wl_products_navigator_results', $results, $navigator_id );
+
+        return $amp ? array(
+            'items' => array(
+                array( 'values' => $results ),
+            ),
+        ) : $results;
+
+    }
+
+    private function get_directly_referencing_posts( $post_id, $referencing_post_ids ) {
+
+        $directly_referencing_post_ids = Wordlift_Entity_Service::get_instance()->get_related_entities( $post_id );
+
+        $post__in = array_diff( $directly_referencing_post_ids, $referencing_post_ids );
+
+        $directly_referencing_posts = get_posts( array(
+            'meta_query'          => array(
+                array(
+                    'key' => '_thumbnail_id'
+                ),
+                array(
+                    'key'   => '_stock_status',
+                    'value' => 'instock'
+                )
+            ),
+            'post__in'            => $post__in,
+            'post_type'           => 'product',
+            'ignore_sticky_posts' => 1
+        ) );
+
+        $results = array();
+
+        foreach ( $directly_referencing_posts as $post ) {
+            $result             = new stdClass();
+            $result->ID         = $post->ID;
+            $result->post_title = $post->post_title;
+            $result->entity_id  = $post->ID;
+            $results[]          = $result;
+        }
+
+        return $results;
+    }
+
+    private function get_entity_results(
+        $post_id,
+        $fields = array(
+            'ID',
+            'post_title',
+        ),
+        $order_by = 'ID DESC',
+        $limit = 10,
+        $offset = 0
+    ) {
+        global $wpdb;
+
+        $select = implode( ', ', array_map( function ( $item ) {
+            return "p.$item AS $item";
+        }, (array) $fields ) );
+
+        $order_by = implode( ', ', array_map( function ( $item ) {
+            return "p.$item";
+        }, (array) $order_by ) );
+
+        /** @noinspection SqlNoDataSourceInspection */
+        return $wpdb->get_results(
+            $wpdb->prepare( <<<EOF
 SELECT %4\$s, p2.ID as entity_id
  FROM {$wpdb->prefix}wl_relation_instances r1
 	-- get the ID of the post entity in common between the object and the subject 2. 
@@ -238,33 +238,33 @@ 
  LIMIT %2\$d
  OFFSET %3\$d
 EOF
-				, $post_id, $limit, $offset, $select, $order_by )
-		);
-	}
-
-	private function get_post_results(
-		$post_id,
-		$fields = array(
-			'ID',
-			'post_title',
-		),
-		$order_by = 'ID DESC',
-		$limit = 10,
-		$offset = 0
-	) {
-		global $wpdb;
-
-		$select = implode( ', ', array_map( function ( $item ) {
-			return "p.$item AS $item";
-		}, (array) $fields ) );
-
-		$order_by = implode( ', ', array_map( function ( $item ) {
-			return "p.$item";
-		}, (array) $order_by ) );
-
-		/** @noinspection SqlNoDataSourceInspection */
-		return $wpdb->get_results(
-			$wpdb->prepare( <<<EOF
+                , $post_id, $limit, $offset, $select, $order_by )
+        );
+    }
+
+    private function get_post_results(
+        $post_id,
+        $fields = array(
+            'ID',
+            'post_title',
+        ),
+        $order_by = 'ID DESC',
+        $limit = 10,
+        $offset = 0
+    ) {
+        global $wpdb;
+
+        $select = implode( ', ', array_map( function ( $item ) {
+            return "p.$item AS $item";
+        }, (array) $fields ) );
+
+        $order_by = implode( ', ', array_map( function ( $item ) {
+            return "p.$item";
+        }, (array) $order_by ) );
+
+        /** @noinspection SqlNoDataSourceInspection */
+        return $wpdb->get_results(
+            $wpdb->prepare( <<<EOF
 SELECT %4\$s, p2.ID as entity_id
  FROM {$wpdb->prefix}wl_relation_instances r1
     INNER JOIN {$wpdb->prefix}wl_relation_instances r2
@@ -300,8 +300,8 @@ 
  LIMIT %2\$d
  OFFSET %3\$d
 EOF
-				, $post_id, $limit, $offset, $select, $order_by )
-		);
-	}
+                , $post_id, $limit, $offset, $select, $order_by )
+        );
+    }
 
 }

Spacing +55 added lines, -55 removed lines

@@ -11,12 +11,12 @@ 
 			'/products-navigator',
 			array(
 				'post_id' => array(
-					'description' => __( 'Post ID for which Navigator has to be queried', 'wordlift' ),
+					'description' => __('Post ID for which Navigator has to be queried', 'wordlift'),
 					'type'        => 'integer',
 					'required'    => true
 				),
 				'uniqid'  => array(
-					'description' => __( 'Navigator uniqueid', 'wordlift' ),
+					'description' => __('Navigator uniqueid', 'wordlift'),
 					'type'        => 'string',
 					'required'    => true
 				),
@@ -41,7 +41,7 @@ 
 		);
 	}
 
-	public function get_data( $request ) {
+	public function get_data($request) {
 
 		// Sanitize and set defaults
 		$navigator_length = $request['limit'];
@@ -51,60 +51,60 @@ 
 		$navigator_id     = $request['uniqid'];
 		$amp              = $request['amp'];
 
-		$post = get_post( $post_id );
+		$post = get_post($post_id);
 
 		// Post ID has to match an existing item
-		if ( null === $post ) {
-			return new WP_Error( 'rest_invalid_post_id', __( 'Invalid post_id', 'wordlift' ), array( 'status' => 404 ) );
+		if (null === $post) {
+			return new WP_Error('rest_invalid_post_id', __('Invalid post_id', 'wordlift'), array('status' => 404));
 		}
 
 		// Determine navigator type and call respective get_*_results
-		if ( get_post_type( $post_id ) === Wordlift_Entity_Service::TYPE_NAME ) {
-			$referencing_posts = $this->get_entity_results( $post_id, array(
+		if (get_post_type($post_id) === Wordlift_Entity_Service::TYPE_NAME) {
+			$referencing_posts = $this->get_entity_results($post_id, array(
 				'ID',
 				'post_title',
-			), $order_by, $navigator_length, $navigator_offset );
+			), $order_by, $navigator_length, $navigator_offset);
 		} else {
-			$referencing_posts = $this->get_post_results( $post_id, array(
+			$referencing_posts = $this->get_post_results($post_id, array(
 				'ID',
 				'post_title',
-			), $order_by, $navigator_length, $navigator_offset );
+			), $order_by, $navigator_length, $navigator_offset);
 		}
 
 		// Fetch directly referencing posts excluding referencing posts via entities
-		$directly_referencing_posts = $this->get_directly_referencing_posts( $post_id, array_map( function ( $referencing_post ) {
+		$directly_referencing_posts = $this->get_directly_referencing_posts($post_id, array_map(function($referencing_post) {
 			return $referencing_post->ID;
-		}, $referencing_posts ) );
+		}, $referencing_posts));
 
 		// Combine directly referencing posts and referencing posts via entities
-		$referencing_posts = array_merge( $directly_referencing_posts, $referencing_posts );
+		$referencing_posts = array_merge($directly_referencing_posts, $referencing_posts);
 
 		// loop over them and take the first one which is not already in the $related_posts
 		$results = array();
-		foreach ( $referencing_posts as $referencing_post ) {
-			$serialized_entity = wl_serialize_entity( $referencing_post->entity_id );
-			$product           = wc_get_product( $referencing_post->ID );
+		foreach ($referencing_posts as $referencing_post) {
+			$serialized_entity = wl_serialize_entity($referencing_post->entity_id);
+			$product           = wc_get_product($referencing_post->ID);
 
 			$result = array(
 				'product' => array(
 					'id'              => $referencing_post->ID,
-					'permalink'       => get_permalink( $referencing_post->ID ),
+					'permalink'       => get_permalink($referencing_post->ID),
 					'title'           => $referencing_post->post_title,
-					'thumbnail'       => get_the_post_thumbnail_url( $referencing_post, 'medium' ),
+					'thumbnail'       => get_the_post_thumbnail_url($referencing_post, 'medium'),
 					'regular_price'   => $product->get_regular_price(),
 					'sale_price'      => $product->get_sale_price(),
 					'price'           => $product->get_price(),
 					'currency_symbol' => get_woocommerce_currency_symbol(),
-					'discount_pc'     => ( $product->get_sale_price() && ( $product->get_regular_price() > 0 ) ) ? round( 1 - ( $product->get_sale_price() / $product->get_regular_price() ), 2 ) * 100 : 0,
+					'discount_pc'     => ($product->get_sale_price() && ($product->get_regular_price() > 0)) ? round(1 - ($product->get_sale_price() / $product->get_regular_price()), 2) * 100 : 0,
 					'average_rating'  => $product->get_average_rating(),
 					'rating_count'    => $product->get_rating_count(),
-					'rating_html'     => wc_get_rating_html( $product->get_average_rating(), $product->get_rating_count() )
+					'rating_html'     => wc_get_rating_html($product->get_average_rating(), $product->get_rating_count())
 				),
 				'entity'  => array(
 					'id'        => $referencing_post->entity_id,
 					'label'     => $serialized_entity['label'],
 					'mainType'  => $serialized_entity['mainType'],
-					'permalink' => get_permalink( $referencing_post->entity_id ),
+					'permalink' => get_permalink($referencing_post->entity_id),
 				),
 			);
 
@@ -112,50 +112,50 @@ 
 
 		}
 
-		if ( count( $results ) < $navigator_length ) {
-			$results = apply_filters( 'wl_products_navigator_data_placeholder', $results, $navigator_id, $navigator_offset, $navigator_length );
+		if (count($results) < $navigator_length) {
+			$results = apply_filters('wl_products_navigator_data_placeholder', $results, $navigator_id, $navigator_offset, $navigator_length);
 		}
 
 
 		// Add filler posts if needed
-		$filler_count = $navigator_length - count( $results );
-		if ( $filler_count > 0 ) {
-			$referencing_post_ids = array_map( function ( $p ) {
+		$filler_count = $navigator_length - count($results);
+		if ($filler_count > 0) {
+			$referencing_post_ids = array_map(function($p) {
 				return $p->ID;
-			}, $referencing_posts );
+			}, $referencing_posts);
 			/**
 			 * @since 3.28.0
 			 * Filler posts are fetched using this util.
 			 */
-			$filler_posts_util    = new Filler_Posts_Util( $post_id, 'product' );
-			$post_ids_to_be_excluded = array_merge( array( $post_id ), $referencing_post_ids );
-			$filler_posts            = $filler_posts_util->get_product_navigator_response( $filler_count, $post_ids_to_be_excluded );
-			$results                 = array_merge( $results, $filler_posts );
+			$filler_posts_util = new Filler_Posts_Util($post_id, 'product');
+			$post_ids_to_be_excluded = array_merge(array($post_id), $referencing_post_ids);
+			$filler_posts            = $filler_posts_util->get_product_navigator_response($filler_count, $post_ids_to_be_excluded);
+			$results                 = array_merge($results, $filler_posts);
 		}
 
 		// Apply filters after fillers are added
-		foreach ( $results as $result_index => $result ) {
-			$results[ $result_index ]['product'] = apply_filters( 'wl_products_navigator_data_post', $result['product'], intval( $result['product']['id'] ), $navigator_id );
-			$results[ $result_index ]['entity'] = apply_filters( 'wl_products_navigator_data_entity', $result['entity'], intval( $result['entity']['id'] ), $navigator_id );
+		foreach ($results as $result_index => $result) {
+			$results[$result_index]['product'] = apply_filters('wl_products_navigator_data_post', $result['product'], intval($result['product']['id']), $navigator_id);
+			$results[$result_index]['entity'] = apply_filters('wl_products_navigator_data_entity', $result['entity'], intval($result['entity']['id']), $navigator_id);
 		}
 
-		$results = apply_filters( 'wl_products_navigator_results', $results, $navigator_id );
+		$results = apply_filters('wl_products_navigator_results', $results, $navigator_id);
 
 		return $amp ? array(
 			'items' => array(
-				array( 'values' => $results ),
+				array('values' => $results),
 			),
 		) : $results;
 
 	}
 
-	private function get_directly_referencing_posts( $post_id, $referencing_post_ids ) {
+	private function get_directly_referencing_posts($post_id, $referencing_post_ids) {
 
-		$directly_referencing_post_ids = Wordlift_Entity_Service::get_instance()->get_related_entities( $post_id );
+		$directly_referencing_post_ids = Wordlift_Entity_Service::get_instance()->get_related_entities($post_id);
 
-		$post__in = array_diff( $directly_referencing_post_ids, $referencing_post_ids );
+		$post__in = array_diff($directly_referencing_post_ids, $referencing_post_ids);
 
-		$directly_referencing_posts = get_posts( array(
+		$directly_referencing_posts = get_posts(array(
 			'meta_query'          => array(
 				array(
 					'key' => '_thumbnail_id'
@@ -168,11 +168,11 @@ 
 			'post__in'            => $post__in,
 			'post_type'           => 'product',
 			'ignore_sticky_posts' => 1
-		) );
+		));
 
 		$results = array();
 
-		foreach ( $directly_referencing_posts as $post ) {
+		foreach ($directly_referencing_posts as $post) {
 			$result             = new stdClass();
 			$result->ID         = $post->ID;
 			$result->post_title = $post->post_title;
@@ -195,17 +195,17 @@ 
 	) {
 		global $wpdb;
 
-		$select = implode( ', ', array_map( function ( $item ) {
+		$select = implode(', ', array_map(function($item) {
 			return "p.$item AS $item";
-		}, (array) $fields ) );
+		}, (array) $fields));
 
-		$order_by = implode( ', ', array_map( function ( $item ) {
+		$order_by = implode(', ', array_map(function($item) {
 			return "p.$item";
-		}, (array) $order_by ) );
+		}, (array) $order_by));
 
 		/** @noinspection SqlNoDataSourceInspection */
 		return $wpdb->get_results(
-			$wpdb->prepare( <<<EOF
+			$wpdb->prepare(<<<EOF
 SELECT %4\$s, p2.ID as entity_id
  FROM {$wpdb->prefix}wl_relation_instances r1
 	-- get the ID of the post entity in common between the object and the subject 2. 
@@ -238,7 +238,7 @@ 
  LIMIT %2\$d
  OFFSET %3\$d
 EOF
-				, $post_id, $limit, $offset, $select, $order_by )
+				, $post_id, $limit, $offset, $select, $order_by)
 		);
 	}
 
@@ -254,17 +254,17 @@ 
 	) {
 		global $wpdb;
 
-		$select = implode( ', ', array_map( function ( $item ) {
+		$select = implode(', ', array_map(function($item) {
 			return "p.$item AS $item";
-		}, (array) $fields ) );
+		}, (array) $fields));
 
-		$order_by = implode( ', ', array_map( function ( $item ) {
+		$order_by = implode(', ', array_map(function($item) {
 			return "p.$item";
-		}, (array) $order_by ) );
+		}, (array) $order_by));
 
 		/** @noinspection SqlNoDataSourceInspection */
 		return $wpdb->get_results(
-			$wpdb->prepare( <<<EOF
+			$wpdb->prepare(<<<EOF
 SELECT %4\$s, p2.ID as entity_id
  FROM {$wpdb->prefix}wl_relation_instances r1
     INNER JOIN {$wpdb->prefix}wl_relation_instances r2
@@ -300,7 +300,7 @@ 
  LIMIT %2\$d
  OFFSET %3\$d
 EOF
-				, $post_id, $limit, $offset, $select, $order_by )
+				, $post_id, $limit, $offset, $select, $order_by)
 		);
 	}
 

src/wordlift/widgets/navigator/filler-posts/class-same-category-filler-posts.php

Indentation +6 added lines, -6 removed lines

@@ -14,12 +14,12 @@
  */
 class Same_Category_Filler_Posts extends Filler_Posts {
 
-	function get_posts( $filler_count, $post_ids_to_be_excluded ) {
+    function get_posts( $filler_count, $post_ids_to_be_excluded ) {
 
-		$current_post_categories = wp_get_post_categories( $this->post_id );
-		$post_type = $this->alternate_post_type ?: get_post_types();
+        $current_post_categories = wp_get_post_categories( $this->post_id );
+        $post_type = $this->alternate_post_type ?: get_post_types();
 
-		return get_posts( array( 'category__in' => $current_post_categories, 'post_type' => $post_type )
-		                  + $this->get_posts_config( $filler_count, $post_ids_to_be_excluded ) );
-	}
+        return get_posts( array( 'category__in' => $current_post_categories, 'post_type' => $post_type )
+                          + $this->get_posts_config( $filler_count, $post_ids_to_be_excluded ) );
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -14,12 +14,12 @@
  */
 class Same_Category_Filler_Posts extends Filler_Posts {
 
-	function get_posts( $filler_count, $post_ids_to_be_excluded ) {
+	function get_posts($filler_count, $post_ids_to_be_excluded) {
 
-		$current_post_categories = wp_get_post_categories( $this->post_id );
+		$current_post_categories = wp_get_post_categories($this->post_id);
 		$post_type = $this->alternate_post_type ?: get_post_types();
 
-		return get_posts( array( 'category__in' => $current_post_categories, 'post_type' => $post_type )
-		                  + $this->get_posts_config( $filler_count, $post_ids_to_be_excluded ) );
+		return get_posts(array('category__in' => $current_post_categories, 'post_type' => $post_type)
+		                  + $this->get_posts_config($filler_count, $post_ids_to_be_excluded));
 	}
 }

src/wordlift/widgets/navigator/filler-posts/class-same-post-type-same-category-posts.php

Indentation +9 added lines, -9 removed lines

@@ -8,14 +8,14 @@
 class Same_Post_Type_Same_Category_Posts extends Filler_Posts {
 
 
-	function get_posts( $filler_count, $post_ids_to_be_excluded ) {
-		$current_post_categories = wp_get_post_categories( $this->post_id );
-		$post_type = $this->alternate_post_type ?: get_post_type( $this->post_id );
+    function get_posts( $filler_count, $post_ids_to_be_excluded ) {
+        $current_post_categories = wp_get_post_categories( $this->post_id );
+        $post_type = $this->alternate_post_type ?: get_post_type( $this->post_id );
 
-		return get_posts( array(
-			                  'category__in' => $current_post_categories,
-			                  'post_type'    => $post_type
-		                  )
-		                  + $this->get_posts_config( $filler_count, $post_ids_to_be_excluded ) );
-	}
+        return get_posts( array(
+                                'category__in' => $current_post_categories,
+                                'post_type'    => $post_type
+                            )
+                          + $this->get_posts_config( $filler_count, $post_ids_to_be_excluded ) );
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -8,14 +8,14 @@
 class Same_Post_Type_Same_Category_Posts extends Filler_Posts {
 
 
-	function get_posts( $filler_count, $post_ids_to_be_excluded ) {
-		$current_post_categories = wp_get_post_categories( $this->post_id );
-		$post_type = $this->alternate_post_type ?: get_post_type( $this->post_id );
+	function get_posts($filler_count, $post_ids_to_be_excluded) {
+		$current_post_categories = wp_get_post_categories($this->post_id);
+		$post_type = $this->alternate_post_type ?: get_post_type($this->post_id);
 
-		return get_posts( array(
+		return get_posts(array(
 			                  'category__in' => $current_post_categories,
 			                  'post_type'    => $post_type
 		                  )
-		                  + $this->get_posts_config( $filler_count, $post_ids_to_be_excluded ) );
+		                  + $this->get_posts_config($filler_count, $post_ids_to_be_excluded));
 	}
 }

src/wordlift/widgets/navigator/filler-posts/class-same-post-type-filler-posts.php

Indentation +8 added lines, -8 removed lines

@@ -7,15 +7,15 @@
  */
 class Same_Post_Type_Filler_Posts extends Filler_Posts {
 
-	function get_posts( $filler_count, $post_ids_to_be_excluded ) {
+    function get_posts( $filler_count, $post_ids_to_be_excluded ) {
 
-		$post_type = $this->alternate_post_type ?: get_post_type( $this->post_id );
+        $post_type = $this->alternate_post_type ?: get_post_type( $this->post_id );
 
-		if ( $post_type === 'entity' ) {
-			$post_type = 'post';
-		}
+        if ( $post_type === 'entity' ) {
+            $post_type = 'post';
+        }
 
-		return get_posts( array( 'post_type' => $post_type )
-		                  + $this->get_posts_config( $filler_count, $post_ids_to_be_excluded ) );
-	}
+        return get_posts( array( 'post_type' => $post_type )
+                          + $this->get_posts_config( $filler_count, $post_ids_to_be_excluded ) );
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -7,15 +7,15 @@
  */
 class Same_Post_Type_Filler_Posts extends Filler_Posts {
 
-	function get_posts( $filler_count, $post_ids_to_be_excluded ) {
+	function get_posts($filler_count, $post_ids_to_be_excluded) {
 
-		$post_type = $this->alternate_post_type ?: get_post_type( $this->post_id );
+		$post_type = $this->alternate_post_type ?: get_post_type($this->post_id);
 
-		if ( $post_type === 'entity' ) {
+		if ($post_type === 'entity') {
 			$post_type = 'post';
 		}
 
-		return get_posts( array( 'post_type' => $post_type )
-		                  + $this->get_posts_config( $filler_count, $post_ids_to_be_excluded ) );
+		return get_posts(array('post_type' => $post_type)
+		                  + $this->get_posts_config($filler_count, $post_ids_to_be_excluded));
 	}
 }

src/wordlift/widgets/navigator/filler-posts/class-filler-posts.php

Indentation +49 added lines, -49 removed lines

@@ -7,55 +7,55 @@
  */
 abstract class Filler_Posts {
 
-	/**
-	 * @var int
-	 */
-	public $filler_count;
-
-	/**
-	 * @var array<int>
-	 */
-	public $post_ids_to_be_excluded;
-
-	/**
-	 * @var $post_id int
-	 */
-	protected $post_id;
-
-	/**
-	 * Filler_Posts constructor.
-	 *
-	 * @param $post_id
-	 */
-	public function __construct( $post_id, $alternate_post_type = null ) {
-
-		$this->post_id = $post_id;
-		$this->alternate_post_type = $alternate_post_type;
-
-	}
-
-	protected function get_posts_config($filler_count, $post_ids_to_be_excluded) {
-
-		return array(
-			'meta_query'          => array(
-				array(
-					'key' => '_thumbnail_id'
-				)
-			),
-			'numberposts'         => $filler_count,
-			'post__not_in'        => $post_ids_to_be_excluded,
-			'ignore_sticky_posts' => 1
-		);
-
-	}
-
-	/**
-	 * @param $filler_count
-	 * @param $post_ids_to_be_excluded
-	 *
-	 * @return array<\WP_Post>
-	 */
-	abstract function get_posts( $filler_count, $post_ids_to_be_excluded );
+    /**
+     * @var int
+     */
+    public $filler_count;
+
+    /**
+     * @var array<int>
+     */
+    public $post_ids_to_be_excluded;
+
+    /**
+     * @var $post_id int
+     */
+    protected $post_id;
+
+    /**
+     * Filler_Posts constructor.
+     *
+     * @param $post_id
+     */
+    public function __construct( $post_id, $alternate_post_type = null ) {
+
+        $this->post_id = $post_id;
+        $this->alternate_post_type = $alternate_post_type;
+
+    }
+
+    protected function get_posts_config($filler_count, $post_ids_to_be_excluded) {
+
+        return array(
+            'meta_query'          => array(
+                array(
+                    'key' => '_thumbnail_id'
+                )
+            ),
+            'numberposts'         => $filler_count,
+            'post__not_in'        => $post_ids_to_be_excluded,
+            'ignore_sticky_posts' => 1
+        );
+
+    }
+
+    /**
+     * @param $filler_count
+     * @param $post_ids_to_be_excluded
+     *
+     * @return array<\WP_Post>
+     */
+    abstract function get_posts( $filler_count, $post_ids_to_be_excluded );
 
 
 }

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
 	 *
 	 * @param $post_id
 	 */
-	public function __construct( $post_id, $alternate_post_type = null ) {
+	public function __construct($post_id, $alternate_post_type = null) {
 
 		$this->post_id = $post_id;
 		$this->alternate_post_type = $alternate_post_type;
@@ -55,7 +55,7 @@ 
 	 *
 	 * @return array<\WP_Post>
 	 */
-	abstract function get_posts( $filler_count, $post_ids_to_be_excluded );
+	abstract function get_posts($filler_count, $post_ids_to_be_excluded);
 
 
 }

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);
 	}
 
 }