insideout10 / wordlift-plugin

src/wordlift/dataset/index.php

Indentation +25 added lines, -25 removed lines

@@ -14,38 +14,38 @@
 use Wordlift\Jsonld\Jsonld_Service;
 
 if ( ! defined( 'ABSPATH' ) ) {
-	exit;
+    exit;
 }
 
 // Register the Dataset JSON Endpoint.
 // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
 if ( apply_filters( 'wl_feature__enable__dataset-ng', false ) ) {
 
-	$sync_object_adapter_factory = new Sync_Object_Adapter_Factory();
-	$sync_service                = new Sync_Service( Default_Api_Service::get_instance(), $sync_object_adapter_factory, Jsonld_Service::get_instance(), Wordlift_Entity_Service::get_instance() );
-	new Sync_Post_Hooks( $sync_service, $sync_object_adapter_factory );
-	new Sync_User_Hooks( $sync_service );
+    $sync_object_adapter_factory = new Sync_Object_Adapter_Factory();
+    $sync_service                = new Sync_Service( Default_Api_Service::get_instance(), $sync_object_adapter_factory, Jsonld_Service::get_instance(), Wordlift_Entity_Service::get_instance() );
+    new Sync_Post_Hooks( $sync_service, $sync_object_adapter_factory );
+    new Sync_User_Hooks( $sync_service );
 
-	// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-	if ( apply_filters( 'wl_feature__enable__no-vocabulary-terms', false ) ) {
-		new Sync_Term_Hooks( $sync_service, $sync_object_adapter_factory );
-	}
-	/**
-	 * @since 3.28.0
-	 * @see https://github.com/insideout10/wordlift-plugin/issues/1186
-	 */
-	new Sync_Hooks_Entity_Relation( Wordlift_Entity_Service::get_instance() );
+    // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+    if ( apply_filters( 'wl_feature__enable__no-vocabulary-terms', false ) ) {
+        new Sync_Term_Hooks( $sync_service, $sync_object_adapter_factory );
+    }
+    /**
+     * @since 3.28.0
+     * @see https://github.com/insideout10/wordlift-plugin/issues/1186
+     */
+    new Sync_Hooks_Entity_Relation( Wordlift_Entity_Service::get_instance() );
 
-	// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-	if ( apply_filters( 'wl_feature__enable__wordpress-ontology', false ) ) {
-		new Sync_Hooks_Wordpress_Ontology();
-	}
+    // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+    if ( apply_filters( 'wl_feature__enable__wordpress-ontology', false ) ) {
+        new Sync_Hooks_Wordpress_Ontology();
+    }
 
-	// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-	if ( apply_filters( 'wl_feature__enable__sync-background', false ) ) {
-		// Set up the sync background process.
-		$sync_background_process = new Sync_Background_Process( $sync_service, $sync_object_adapter_factory );
-		new Sync_Background_Process_Wpjson_Endpoint( $sync_background_process );
-		new Sync_Page();
-	}
+    // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+    if ( apply_filters( 'wl_feature__enable__sync-background', false ) ) {
+        // Set up the sync background process.
+        $sync_background_process = new Sync_Background_Process( $sync_service, $sync_object_adapter_factory );
+        new Sync_Background_Process_Wpjson_Endpoint( $sync_background_process );
+        new Sync_Page();
+    }
 }

Spacing +12 added lines, -12 removed lines

@@ -13,39 +13,39 @@
 use Wordlift\Dataset\Sync_User_Hooks;
 use Wordlift\Jsonld\Jsonld_Service;
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Register the Dataset JSON Endpoint.
 // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-if ( apply_filters( 'wl_feature__enable__dataset-ng', false ) ) {
+if (apply_filters('wl_feature__enable__dataset-ng', false)) {
 
 	$sync_object_adapter_factory = new Sync_Object_Adapter_Factory();
-	$sync_service                = new Sync_Service( Default_Api_Service::get_instance(), $sync_object_adapter_factory, Jsonld_Service::get_instance(), Wordlift_Entity_Service::get_instance() );
-	new Sync_Post_Hooks( $sync_service, $sync_object_adapter_factory );
-	new Sync_User_Hooks( $sync_service );
+	$sync_service                = new Sync_Service(Default_Api_Service::get_instance(), $sync_object_adapter_factory, Jsonld_Service::get_instance(), Wordlift_Entity_Service::get_instance());
+	new Sync_Post_Hooks($sync_service, $sync_object_adapter_factory);
+	new Sync_User_Hooks($sync_service);
 
 	// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-	if ( apply_filters( 'wl_feature__enable__no-vocabulary-terms', false ) ) {
-		new Sync_Term_Hooks( $sync_service, $sync_object_adapter_factory );
+	if (apply_filters('wl_feature__enable__no-vocabulary-terms', false)) {
+		new Sync_Term_Hooks($sync_service, $sync_object_adapter_factory);
 	}
 	/**
 	 * @since 3.28.0
 	 * @see https://github.com/insideout10/wordlift-plugin/issues/1186
 	 */
-	new Sync_Hooks_Entity_Relation( Wordlift_Entity_Service::get_instance() );
+	new Sync_Hooks_Entity_Relation(Wordlift_Entity_Service::get_instance());
 
 	// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-	if ( apply_filters( 'wl_feature__enable__wordpress-ontology', false ) ) {
+	if (apply_filters('wl_feature__enable__wordpress-ontology', false)) {
 		new Sync_Hooks_Wordpress_Ontology();
 	}
 
 	// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-	if ( apply_filters( 'wl_feature__enable__sync-background', false ) ) {
+	if (apply_filters('wl_feature__enable__sync-background', false)) {
 		// Set up the sync background process.
-		$sync_background_process = new Sync_Background_Process( $sync_service, $sync_object_adapter_factory );
-		new Sync_Background_Process_Wpjson_Endpoint( $sync_background_process );
+		$sync_background_process = new Sync_Background_Process($sync_service, $sync_object_adapter_factory);
+		new Sync_Background_Process_Wpjson_Endpoint($sync_background_process);
 		new Sync_Page();
 	}
 }

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

Indentation +60 added lines, -60 removed lines

@@ -7,65 +7,65 @@
 // phpcs:ignore WordPress.WP.CapitalPDangit.MisspelledClassName
 class Sync_Hooks_Wordpress_Ontology {
 
-	const HTTP_PURL_ORG_WORDPRESS_1_0 = 'http://purl.org/wordpress/1.0/';
-
-	public function __construct() {
-		add_filter( 'wl_dataset__sync_service__sync_item__jsonld', array( $this, 'jsonld' ), 10, 3 );
-	}
-
-	public function jsonld( $jsonld, $type, $object_id ) {
-
-		$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'id' ] = $object_id;
-
-		switch ( $type ) {
-
-			case Object_Type_Enum::TERM:
-				$term = get_term( $object_id );
-
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'type' ]        = 'term';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'name' ]        = $term->name;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'description' ] = $term->description;
-				break;
-
-			case Object_Type_Enum::USER:
-				$user = get_userdata( $object_id );
-
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'type' ]        = 'user';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'displayName' ] = $user->display_name;
-				break;
-
-			case Object_Type_Enum::POST:
-				$post = get_post( $object_id );
-
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'type' ]       = 'post';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'customType' ] = $post->post_type;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'title' ]      = $post->post_title;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'status' ]     = $post->post_status;
-				$content = has_blocks( $post ) ?
-					do_blocks( $post->post_content ) : do_shortcode( $post->post_content );
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'content' ]   = $content;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'permalink' ] = get_permalink( $post );
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'sticky' ]    = is_sticky( $post->ID );
-
-				$taxonomies = get_post_taxonomies( $post );
-				$_tmp_terms = array();
-				foreach ( $taxonomies as $taxonomy ) {
-					$terms = wp_get_post_terms( $post->ID, $taxonomy );
-					/** @var \WP_Term $term */
-					foreach ( $terms as $term ) {
-						$_tmp_terms[] = "$taxonomy:$term->name";
-					}
-				}
-				if ( ! empty( $_tmp_terms ) ) {
-					$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'terms' ] = $_tmp_terms;
-				}
-
-				break;
-
-			default:
-		}
-
-		return $jsonld;
-	}
+    const HTTP_PURL_ORG_WORDPRESS_1_0 = 'http://purl.org/wordpress/1.0/';
+
+    public function __construct() {
+        add_filter( 'wl_dataset__sync_service__sync_item__jsonld', array( $this, 'jsonld' ), 10, 3 );
+    }
+
+    public function jsonld( $jsonld, $type, $object_id ) {
+
+        $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'id' ] = $object_id;
+
+        switch ( $type ) {
+
+            case Object_Type_Enum::TERM:
+                $term = get_term( $object_id );
+
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'type' ]        = 'term';
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'name' ]        = $term->name;
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'description' ] = $term->description;
+                break;
+
+            case Object_Type_Enum::USER:
+                $user = get_userdata( $object_id );
+
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'type' ]        = 'user';
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'displayName' ] = $user->display_name;
+                break;
+
+            case Object_Type_Enum::POST:
+                $post = get_post( $object_id );
+
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'type' ]       = 'post';
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'customType' ] = $post->post_type;
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'title' ]      = $post->post_title;
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'status' ]     = $post->post_status;
+                $content = has_blocks( $post ) ?
+                    do_blocks( $post->post_content ) : do_shortcode( $post->post_content );
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'content' ]   = $content;
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'permalink' ] = get_permalink( $post );
+                $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'sticky' ]    = is_sticky( $post->ID );
+
+                $taxonomies = get_post_taxonomies( $post );
+                $_tmp_terms = array();
+                foreach ( $taxonomies as $taxonomy ) {
+                    $terms = wp_get_post_terms( $post->ID, $taxonomy );
+                    /** @var \WP_Term $term */
+                    foreach ( $terms as $term ) {
+                        $_tmp_terms[] = "$taxonomy:$term->name";
+                    }
+                }
+                if ( ! empty( $_tmp_terms ) ) {
+                    $jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'terms' ] = $_tmp_terms;
+                }
+
+                break;
+
+            default:
+        }
+
+        return $jsonld;
+    }
 
 }

Spacing +29 added lines, -29 removed lines

@@ -10,54 +10,54 @@
 	const HTTP_PURL_ORG_WORDPRESS_1_0 = 'http://purl.org/wordpress/1.0/';
 
 	public function __construct() {
-		add_filter( 'wl_dataset__sync_service__sync_item__jsonld', array( $this, 'jsonld' ), 10, 3 );
+		add_filter('wl_dataset__sync_service__sync_item__jsonld', array($this, 'jsonld'), 10, 3);
 	}
 
-	public function jsonld( $jsonld, $type, $object_id ) {
+	public function jsonld($jsonld, $type, $object_id) {
 
-		$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'id' ] = $object_id;
+		$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'id'] = $object_id;
 
-		switch ( $type ) {
+		switch ($type) {
 
 			case Object_Type_Enum::TERM:
-				$term = get_term( $object_id );
+				$term = get_term($object_id);
 
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'type' ]        = 'term';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'name' ]        = $term->name;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'description' ] = $term->description;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'type']        = 'term';
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'name']        = $term->name;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'description'] = $term->description;
 				break;
 
 			case Object_Type_Enum::USER:
-				$user = get_userdata( $object_id );
+				$user = get_userdata($object_id);
 
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'type' ]        = 'user';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'displayName' ] = $user->display_name;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'type']        = 'user';
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'displayName'] = $user->display_name;
 				break;
 
 			case Object_Type_Enum::POST:
-				$post = get_post( $object_id );
-
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'type' ]       = 'post';
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'customType' ] = $post->post_type;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'title' ]      = $post->post_title;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'status' ]     = $post->post_status;
-				$content = has_blocks( $post ) ?
-					do_blocks( $post->post_content ) : do_shortcode( $post->post_content );
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'content' ]   = $content;
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'permalink' ] = get_permalink( $post );
-				$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'sticky' ]    = is_sticky( $post->ID );
-
-				$taxonomies = get_post_taxonomies( $post );
+				$post = get_post($object_id);
+
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'type']       = 'post';
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'customType'] = $post->post_type;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'title']      = $post->post_title;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'status']     = $post->post_status;
+				$content = has_blocks($post) ?
+					do_blocks($post->post_content) : do_shortcode($post->post_content);
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'content']   = $content;
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'permalink'] = get_permalink($post);
+				$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'sticky']    = is_sticky($post->ID);
+
+				$taxonomies = get_post_taxonomies($post);
 				$_tmp_terms = array();
-				foreach ( $taxonomies as $taxonomy ) {
-					$terms = wp_get_post_terms( $post->ID, $taxonomy );
+				foreach ($taxonomies as $taxonomy) {
+					$terms = wp_get_post_terms($post->ID, $taxonomy);
 					/** @var \WP_Term $term */
-					foreach ( $terms as $term ) {
+					foreach ($terms as $term) {
 						$_tmp_terms[] = "$taxonomy:$term->name";
 					}
 				}
-				if ( ! empty( $_tmp_terms ) ) {
-					$jsonld[0][ self::HTTP_PURL_ORG_WORDPRESS_1_0 . 'terms' ] = $_tmp_terms;
+				if ( ! empty($_tmp_terms)) {
+					$jsonld[0][self::HTTP_PURL_ORG_WORDPRESS_1_0.'terms'] = $_tmp_terms;
 				}
 
 				break;

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

Indentation +32 added lines, -32 removed lines

@@ -6,37 +6,37 @@
 
 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 when an error occurs.
-	 */
-	public 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." );
-		}
-
-	}
-
-	public 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 when an error occurs.
+     */
+    public 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." );
+        }
+
+    }
+
+    public 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 +9 added lines, -9 removed lines

@@ -13,27 +13,27 @@
 	 * @return Sync_Object_Adapter
 	 * @throws \Exception when an error occurs.
 	 */
-	public function create( $type, $object_id ) {
+	public 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.");
 		}
 
 	}
 
-	public function create_many( $type, $object_ids ) {
+	public function create_many($type, $object_ids) {
 		$that = $this;
 
 		return array_map(
-			function ( $item ) use ( $type, $that ) {
-				return $that->create( $type, $item );
+			function($item) use ($type, $that) {
+				return $that->create($type, $item);
 			},
 			(array) $object_ids
 		);

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

Indentation +48 added lines, -48 removed lines

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

Spacing +9 added lines, -9 removed lines

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

src/wordlift/faq/class-faq-content-filter.php

Indentation +27 added lines, -27 removed lines

@@ -10,33 +10,33 @@
 namespace Wordlift\Faq;
 
 class Faq_Content_Filter {
-	/**
-	 * Constants used for replacing the tags in the html string.
-	 */
-	const FAQ_QUESTION_TAG_NAME = 'wl-faq-question';
-	const FAQ_ANSWER_TAG_NAME   = 'wl-faq-answer';
-	/**
-	 * Replaces all the html tags inserted by Faq highlighting code in the front end
-	 *
-	 * @param $content string Post content
-	 * @return string String after replacing all the opening and closing tags.
-	 */
-	public function remove_all_faq_question_and_answer_tags( $content ) {
-		/**
-		 * Replace all the question tags.
-		 */
-		$faq_question_closing_tag = '</' . self::FAQ_QUESTION_TAG_NAME . '>';
-		$content                  = preg_replace( '/<wl-faq-question class=".+?">/m', '', $content );
-		$content                  = str_replace( $faq_question_closing_tag, '', $content );
-		/**
-		 * Replace all the answer tags.
-		 */
-		$faq_answer_closing_tag = '</' . self::FAQ_ANSWER_TAG_NAME . '>';
-		$content                = preg_replace( '/<wl-faq-answer class=".+?">/m', '', $content );
-		$content                = str_replace( $faq_answer_closing_tag, '', $content );
+    /**
+     * Constants used for replacing the tags in the html string.
+     */
+    const FAQ_QUESTION_TAG_NAME = 'wl-faq-question';
+    const FAQ_ANSWER_TAG_NAME   = 'wl-faq-answer';
+    /**
+     * Replaces all the html tags inserted by Faq highlighting code in the front end
+     *
+     * @param $content string Post content
+     * @return string String after replacing all the opening and closing tags.
+     */
+    public function remove_all_faq_question_and_answer_tags( $content ) {
+        /**
+         * Replace all the question tags.
+         */
+        $faq_question_closing_tag = '</' . self::FAQ_QUESTION_TAG_NAME . '>';
+        $content                  = preg_replace( '/<wl-faq-question class=".+?">/m', '', $content );
+        $content                  = str_replace( $faq_question_closing_tag, '', $content );
+        /**
+         * Replace all the answer tags.
+         */
+        $faq_answer_closing_tag = '</' . self::FAQ_ANSWER_TAG_NAME . '>';
+        $content                = preg_replace( '/<wl-faq-answer class=".+?">/m', '', $content );
+        $content                = str_replace( $faq_answer_closing_tag, '', $content );
 
-		/** Return all the replaced content */
-		return $content;
-	}
+        /** Return all the replaced content */
+        return $content;
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -21,19 +21,19 @@
 	 * @param $content string Post content
 	 * @return string String after replacing all the opening and closing tags.
 	 */
-	public function remove_all_faq_question_and_answer_tags( $content ) {
+	public function remove_all_faq_question_and_answer_tags($content) {
 		/**
 		 * Replace all the question tags.
 		 */
-		$faq_question_closing_tag = '</' . self::FAQ_QUESTION_TAG_NAME . '>';
-		$content                  = preg_replace( '/<wl-faq-question class=".+?">/m', '', $content );
-		$content                  = str_replace( $faq_question_closing_tag, '', $content );
+		$faq_question_closing_tag = '</'.self::FAQ_QUESTION_TAG_NAME.'>';
+		$content                  = preg_replace('/<wl-faq-question class=".+?">/m', '', $content);
+		$content                  = str_replace($faq_question_closing_tag, '', $content);
 		/**
 		 * Replace all the answer tags.
 		 */
-		$faq_answer_closing_tag = '</' . self::FAQ_ANSWER_TAG_NAME . '>';
-		$content                = preg_replace( '/<wl-faq-answer class=".+?">/m', '', $content );
-		$content                = str_replace( $faq_answer_closing_tag, '', $content );
+		$faq_answer_closing_tag = '</'.self::FAQ_ANSWER_TAG_NAME.'>';
+		$content                = preg_replace('/<wl-faq-answer class=".+?">/m', '', $content);
+		$content                = str_replace($faq_answer_closing_tag, '', $content);
 
 		/** Return all the replaced content */
 		return $content;

src/wordlift/faq/class-faq-to-jsonld-converter.php

Indentation +71 added lines, -71 removed lines

@@ -17,80 +17,80 @@
  */
 class Faq_To_Jsonld_Converter {
 
-	const FAQ_JSONLD_TYPE = 'FAQPage';
+    const FAQ_JSONLD_TYPE = 'FAQPage';
 
-	public function __construct() {
-		add_filter( 'wl_post_jsonld', array( $this, 'get_jsonld_for_faq' ), 11, 2 );
-		add_filter( 'wl_entity_jsonld', array( $this, 'get_jsonld_for_faq' ), 11, 2 );
-	}
+    public function __construct() {
+        add_filter( 'wl_post_jsonld', array( $this, 'get_jsonld_for_faq' ), 11, 2 );
+        add_filter( 'wl_entity_jsonld', array( $this, 'get_jsonld_for_faq' ), 11, 2 );
+    }
 
-	/**
-	 * Set the FAQ type to the json ld array
-	 *
-	 * @param $jsonld array The jsonld array.
-	 *
-	 * @return array Returns the json ld array with the type set.
-	 */
-	public function set_faq_type( $jsonld ) {
-		if ( array_key_exists( '@type', $jsonld ) ) {
-			if ( is_string( $jsonld['@type'] ) ) {
-				// If a plain string is present, create an array with previous items.
-				$jsonld['@type'] = array( $jsonld['@type'], self::FAQ_JSONLD_TYPE );
-				return $jsonld;
-			}
-			// check if it is a array, then append the type.
-			if ( is_array( $jsonld['@type'] ) && ! in_array( self::FAQ_JSONLD_TYPE, $jsonld['@type'], true ) ) {
-				array_push( $jsonld['@type'], self::FAQ_JSONLD_TYPE );
-				return $jsonld;
-			}
-		} else {
-			$jsonld['@type'] = array( self::FAQ_JSONLD_TYPE );
-		}
-		return $jsonld;
-	}
-	/**
-	 * @param $post_id int The id of the post.
-	 *
-	 * @return array Get the converted jsonld data
-	 */
-	public function get_jsonld_for_faq( $jsonld, $post_id ) {
+    /**
+     * Set the FAQ type to the json ld array
+     *
+     * @param $jsonld array The jsonld array.
+     *
+     * @return array Returns the json ld array with the type set.
+     */
+    public function set_faq_type( $jsonld ) {
+        if ( array_key_exists( '@type', $jsonld ) ) {
+            if ( is_string( $jsonld['@type'] ) ) {
+                // If a plain string is present, create an array with previous items.
+                $jsonld['@type'] = array( $jsonld['@type'], self::FAQ_JSONLD_TYPE );
+                return $jsonld;
+            }
+            // check if it is a array, then append the type.
+            if ( is_array( $jsonld['@type'] ) && ! in_array( self::FAQ_JSONLD_TYPE, $jsonld['@type'], true ) ) {
+                array_push( $jsonld['@type'], self::FAQ_JSONLD_TYPE );
+                return $jsonld;
+            }
+        } else {
+            $jsonld['@type'] = array( self::FAQ_JSONLD_TYPE );
+        }
+        return $jsonld;
+    }
+    /**
+     * @param $post_id int The id of the post.
+     *
+     * @return array Get the converted jsonld data
+     */
+    public function get_jsonld_for_faq( $jsonld, $post_id ) {
 
-		$faq_items = get_post_meta( $post_id, Faq_Rest_Controller::FAQ_META_KEY );
-		/**
-		 * Apply the FAQ mapping only if the FAQ items are present.
-		 */
-		if ( count( $faq_items ) > 0 ) {
-			$faq_data = $this->get_faq_data( $faq_items );
-			// Merge the FAQ data with jsonld.
-			$jsonld = array_merge( $jsonld, $faq_data );
-			// check if the @type is set on json ld
-			$jsonld = $this->set_faq_type( $jsonld );
-		}
-		return $jsonld;
-	}
+        $faq_items = get_post_meta( $post_id, Faq_Rest_Controller::FAQ_META_KEY );
+        /**
+         * Apply the FAQ mapping only if the FAQ items are present.
+         */
+        if ( count( $faq_items ) > 0 ) {
+            $faq_data = $this->get_faq_data( $faq_items );
+            // Merge the FAQ data with jsonld.
+            $jsonld = array_merge( $jsonld, $faq_data );
+            // check if the @type is set on json ld
+            $jsonld = $this->set_faq_type( $jsonld );
+        }
+        return $jsonld;
+    }
 
-	/**
-	 * @param $faq_items array List of FAQ items extracted from the meta.
-	 *
-	 * @return array Associtative array of type, mainEntity.
-	 */
-	private function get_faq_data( $faq_items ) {
-		$jsonld_data               = array();
-		$jsonld_data['mainEntity'] = array();
-		foreach ( $faq_items as $faq_item ) {
-			if ( 0 === strlen( $faq_item['question'] ) || 0 === strlen( $faq_item['answer'] ) ) {
-				// Bail out if question or answer is not present
-				continue;
-			}
-			$faq_data                            = array();
-			$faq_data['@type']                   = 'Question';
-			$faq_data['name']                    = $faq_item['question'];
-			$faq_data['acceptedAnswer']          = array();
-			$faq_data['acceptedAnswer']['@type'] = 'Answer';
-			$faq_data['acceptedAnswer']['text']  = $faq_item['answer'];
-			array_push( $jsonld_data['mainEntity'], $faq_data );
-		}
+    /**
+     * @param $faq_items array List of FAQ items extracted from the meta.
+     *
+     * @return array Associtative array of type, mainEntity.
+     */
+    private function get_faq_data( $faq_items ) {
+        $jsonld_data               = array();
+        $jsonld_data['mainEntity'] = array();
+        foreach ( $faq_items as $faq_item ) {
+            if ( 0 === strlen( $faq_item['question'] ) || 0 === strlen( $faq_item['answer'] ) ) {
+                // Bail out if question or answer is not present
+                continue;
+            }
+            $faq_data                            = array();
+            $faq_data['@type']                   = 'Question';
+            $faq_data['name']                    = $faq_item['question'];
+            $faq_data['acceptedAnswer']          = array();
+            $faq_data['acceptedAnswer']['@type'] = 'Answer';
+            $faq_data['acceptedAnswer']['text']  = $faq_item['answer'];
+            array_push( $jsonld_data['mainEntity'], $faq_data );
+        }
 
-		return $jsonld_data;
-	}
+        return $jsonld_data;
+    }
 }

Spacing +19 added lines, -19 removed lines

@@ -20,8 +20,8 @@ 
 	const FAQ_JSONLD_TYPE = 'FAQPage';
 
 	public function __construct() {
-		add_filter( 'wl_post_jsonld', array( $this, 'get_jsonld_for_faq' ), 11, 2 );
-		add_filter( 'wl_entity_jsonld', array( $this, 'get_jsonld_for_faq' ), 11, 2 );
+		add_filter('wl_post_jsonld', array($this, 'get_jsonld_for_faq'), 11, 2);
+		add_filter('wl_entity_jsonld', array($this, 'get_jsonld_for_faq'), 11, 2);
 	}
 
 	/**
@@ -31,20 +31,20 @@ 
 	 *
 	 * @return array Returns the json ld array with the type set.
 	 */
-	public function set_faq_type( $jsonld ) {
-		if ( array_key_exists( '@type', $jsonld ) ) {
-			if ( is_string( $jsonld['@type'] ) ) {
+	public function set_faq_type($jsonld) {
+		if (array_key_exists('@type', $jsonld)) {
+			if (is_string($jsonld['@type'])) {
 				// If a plain string is present, create an array with previous items.
-				$jsonld['@type'] = array( $jsonld['@type'], self::FAQ_JSONLD_TYPE );
+				$jsonld['@type'] = array($jsonld['@type'], self::FAQ_JSONLD_TYPE);
 				return $jsonld;
 			}
 			// check if it is a array, then append the type.
-			if ( is_array( $jsonld['@type'] ) && ! in_array( self::FAQ_JSONLD_TYPE, $jsonld['@type'], true ) ) {
-				array_push( $jsonld['@type'], self::FAQ_JSONLD_TYPE );
+			if (is_array($jsonld['@type']) && ! in_array(self::FAQ_JSONLD_TYPE, $jsonld['@type'], true)) {
+				array_push($jsonld['@type'], self::FAQ_JSONLD_TYPE);
 				return $jsonld;
 			}
 		} else {
-			$jsonld['@type'] = array( self::FAQ_JSONLD_TYPE );
+			$jsonld['@type'] = array(self::FAQ_JSONLD_TYPE);
 		}
 		return $jsonld;
 	}
@@ -53,18 +53,18 @@ 
 	 *
 	 * @return array Get the converted jsonld data
 	 */
-	public function get_jsonld_for_faq( $jsonld, $post_id ) {
+	public function get_jsonld_for_faq($jsonld, $post_id) {
 
-		$faq_items = get_post_meta( $post_id, Faq_Rest_Controller::FAQ_META_KEY );
+		$faq_items = get_post_meta($post_id, Faq_Rest_Controller::FAQ_META_KEY);
 		/**
 		 * Apply the FAQ mapping only if the FAQ items are present.
 		 */
-		if ( count( $faq_items ) > 0 ) {
-			$faq_data = $this->get_faq_data( $faq_items );
+		if (count($faq_items) > 0) {
+			$faq_data = $this->get_faq_data($faq_items);
 			// Merge the FAQ data with jsonld.
-			$jsonld = array_merge( $jsonld, $faq_data );
+			$jsonld = array_merge($jsonld, $faq_data);
 			// check if the @type is set on json ld
-			$jsonld = $this->set_faq_type( $jsonld );
+			$jsonld = $this->set_faq_type($jsonld);
 		}
 		return $jsonld;
 	}
@@ -74,11 +74,11 @@ 
 	 *
 	 * @return array Associtative array of type, mainEntity.
 	 */
-	private function get_faq_data( $faq_items ) {
+	private function get_faq_data($faq_items) {
 		$jsonld_data               = array();
 		$jsonld_data['mainEntity'] = array();
-		foreach ( $faq_items as $faq_item ) {
-			if ( 0 === strlen( $faq_item['question'] ) || 0 === strlen( $faq_item['answer'] ) ) {
+		foreach ($faq_items as $faq_item) {
+			if (0 === strlen($faq_item['question']) || 0 === strlen($faq_item['answer'])) {
 				// Bail out if question or answer is not present
 				continue;
 			}
@@ -88,7 +88,7 @@ 
 			$faq_data['acceptedAnswer']          = array();
 			$faq_data['acceptedAnswer']['@type'] = 'Answer';
 			$faq_data['acceptedAnswer']['text']  = $faq_item['answer'];
-			array_push( $jsonld_data['mainEntity'], $faq_data );
+			array_push($jsonld_data['mainEntity'], $faq_data);
 		}
 
 		return $jsonld_data;

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

Indentation +229 added lines, -229 removed lines

@@ -17,233 +17,233 @@
  * @package Wordlift\FAQ
  */
 class Faq_Rest_Controller {
-	const FAQ_META_KEY = '_wl_faq';
-	/**
-	 * Enumerations to determine the field to be deleted on
-	 * the DELETE request in API.
-	 */
-	const QUESTION = 'question';
-	const ANSWER   = 'answer';
-
-	public static function register_routes() {
-		add_action( 'rest_api_init', 'Wordlift\FAQ\FAQ_Rest_Controller::register_route_callback' );
-	}
-
-	public static function register_route_callback() {
-		/**
-		 * Specifies the list of arguments to be present for update request.
-		 */
-		$post_id_validation_settings   = array(
-			'required'          => true,
-			// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-			'validate_callback' => function ( $param, $request, $key ) {
-				return is_numeric( $param );
-			},
-		);
-		$faq_items_validation_settings = array(
-			'required'          => true,
-			// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-			'validate_callback' => function ( $param, $request, $key ) {
-				return is_array( $param ) && count( $param ) > 0;
-			},
-		);
-		/**
-		 * Array of args to be present in order to
-		 * make create, update or delete request.
-		 */
-		$create_or_update_or_delete_args = array(
-			'post_id'   => $post_id_validation_settings,
-			'faq_items' => $faq_items_validation_settings,
-		);
-
-		/**
-		 * Rest route for creating new faq item.
-		 */
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/faq',
-			array(
-				'methods'             => \WP_REST_Server::CREATABLE,
-				'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::add_faq_items',
-				'permission_callback' => function () {
-					return current_user_can( 'publish_posts' );
-				},
-				'args'                => $create_or_update_or_delete_args,
-			)
-		);
-		/**
-		 * Rest route for updating faq items.
-		 */
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/faq',
-			array(
-				'methods'             => \WP_REST_Server::EDITABLE,
-				'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::update_faq_items',
-				'permission_callback' => function () {
-					return current_user_can( 'publish_posts' );
-				},
-				'args'                => $create_or_update_or_delete_args,
-			)
-		);
-		/**
-		 * Rest route for getting the faq items.
-		 */
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/faq/(?P<post_id>\d+)',
-			array(
-				'methods'             => \WP_REST_Server::READABLE,
-				'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::get_faq_items',
-				'permission_callback' => function () {
-					return current_user_can( 'publish_posts' );
-				},
-				'args'                => array( 'post_id' => $post_id_validation_settings ),
-			)
-		);
-		/**
-		 * Rest route for deleting faq item.
-		 */
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/faq',
-			array(
-				'methods'             => \WP_REST_Server::DELETABLE,
-				'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::delete_faq_items',
-				'permission_callback' => function () {
-					return current_user_can( 'publish_posts' );
-				},
-				'args'                => $create_or_update_or_delete_args,
-			)
-		);
-	}
-
-	/**
-	 * Get all FAQ items for a post id.
-	 *
-	 * @param $request $request WP_REST_Request $request {@link WP_REST_Request instance}.
-	 *
-	 * @return array Result array, if post id is not given then error array is returned.
-	 */
-	public static function get_faq_items( $request ) {
-		$data    = $request->get_params();
-		$post_id = (int) $data['post_id'];
-
-		return get_post_meta( $post_id, self::FAQ_META_KEY );
-	}
-
-	/**
-	 * Update all FAQ items for a post id.
-	 *
-	 * @param $request $request WP_REST_Request $request {@link WP_REST_Request instance}.
-	 *
-	 * @return array Result array, if post id is not given then error array is returned.
-	 */
-	public static function update_faq_items( $request ) {
-		$data      = $request->get_params();
-		$post_id   = (int) $data['post_id'];
-		$faq_items = (array) $data['faq_items'];
-		foreach ( $faq_items as $faq_item ) {
-			$previous_value = array(
-				'question' => (string) $faq_item['previous_question_value'],
-				'answer'   => (string) $faq_item['previous_answer_value'],
-				'id'       => (int) $faq_item['id'],
-			);
-			$new_value      = array(
-				'question' => (string) $faq_item['question'],
-				'answer'   => (string) $faq_item['answer'],
-				'id'       => (int) $faq_item['id'],
-			);
-			update_post_meta( $post_id, self::FAQ_META_KEY, $new_value, $previous_value );
-		}
-
-		return array(
-			'status'  => 'success',
-			'message' => __( 'Faq Items updated successfully', 'wordlift' ),
-		);
-
-	}
-
-	/**
-	 * Delete Faq items.
-	 *
-	 * @param $request WP_REST_Request $request {@link WP_REST_Request instance}.
-	 *
-	 * @return array Associative array whether the faq item is deleted or not.
-	 */
-	public static function delete_faq_items( $request ) {
-		$post_data = $request->get_params();
-		$post_id   = $post_data['post_id'];
-		$faq_items = $post_data['faq_items'];
-		foreach ( $faq_items as $faq_item ) {
-			/**
-			 * Note: the order of keys is important in order to delete it properly
-			 * If the order change, delete operation will fail since it is converted
-			 * in to a string when it was stored.
-			 * we cant rely on client to post it in correct order, so we create an array
-			 * in correct order.
-			 */
-			$deleted_faq_item = array(
-				'question' => $faq_item['question'],
-				'answer'   => $faq_item['answer'],
-				'id'       => (int) $faq_item['id'],
-			);
-			/**
-			 * If the field to be deleted is answer, then don't delete the faq item,
-			 * if it is question then delete the faq item.
-			 */
-			if ( self::ANSWER === $faq_item['field_to_be_deleted'] ) {
-				$previous_value = $deleted_faq_item;
-				// then don't delete the faq item, set the answer as empty.
-				$deleted_faq_item['answer'] = '';
-				$new_value                  = $deleted_faq_item;
-				update_post_meta( $post_id, self::FAQ_META_KEY, $new_value, $previous_value );
-			} elseif ( self::QUESTION === $faq_item['field_to_be_deleted'] ) {
-				/**
-				 * If the question is deleted, then delete the faq item.
-				 */
-				delete_post_meta( $post_id, self::FAQ_META_KEY, $deleted_faq_item );
-			}
-		}
-
-		/**
-		 * We are returning only the first item id, since the user can select only one text at a time.
-		 */
-		return array(
-			'status'  => 'success',
-			'message' => __( 'Faq item successfully deleted.', 'wordlift' ),
-		);
-	}
-
-	/**
-	 * Insert or update FAQ items.
-	 *
-	 * @param $request WP_REST_Request $request {@link WP_REST_Request instance}.
-	 *
-	 * @return array Associative array whether the faq item is inserted or not
-	 */
-	public static function add_faq_items( $request ) {
-		$post_data = $request->get_params();
-		$post_id   = $post_data['post_id'];
-		$faq_items = $post_data['faq_items'];
-		foreach ( $faq_items as &$faq_item ) {
-			// Add an identifier id to the faq item, it helps to prevent duplication problem.
-			/**
-			 * We are using time() and a random integer to prevent
-			 * duplication problem.
-			 */
-			$faq_item['id'] = time() + wp_rand( 1, 100 ) + wp_rand( 1, 100 );
-			add_post_meta( (int) $post_id, self::FAQ_META_KEY, $faq_item );
-		}
-
-		/**
-		 * We are returning only the first item id, since the user can select only one text at a time.
-		 */
-		return array(
-			'status'  => 'success',
-			'message' => __( 'Question successfully added.', 'wordlift' ),
-			'id'      => (int) $faq_items[0]['id'],
-		);
-
-	}
+    const FAQ_META_KEY = '_wl_faq';
+    /**
+     * Enumerations to determine the field to be deleted on
+     * the DELETE request in API.
+     */
+    const QUESTION = 'question';
+    const ANSWER   = 'answer';
+
+    public static function register_routes() {
+        add_action( 'rest_api_init', 'Wordlift\FAQ\FAQ_Rest_Controller::register_route_callback' );
+    }
+
+    public static function register_route_callback() {
+        /**
+         * Specifies the list of arguments to be present for update request.
+         */
+        $post_id_validation_settings   = array(
+            'required'          => true,
+            // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+            'validate_callback' => function ( $param, $request, $key ) {
+                return is_numeric( $param );
+            },
+        );
+        $faq_items_validation_settings = array(
+            'required'          => true,
+            // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+            'validate_callback' => function ( $param, $request, $key ) {
+                return is_array( $param ) && count( $param ) > 0;
+            },
+        );
+        /**
+         * Array of args to be present in order to
+         * make create, update or delete request.
+         */
+        $create_or_update_or_delete_args = array(
+            'post_id'   => $post_id_validation_settings,
+            'faq_items' => $faq_items_validation_settings,
+        );
+
+        /**
+         * Rest route for creating new faq item.
+         */
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/faq',
+            array(
+                'methods'             => \WP_REST_Server::CREATABLE,
+                'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::add_faq_items',
+                'permission_callback' => function () {
+                    return current_user_can( 'publish_posts' );
+                },
+                'args'                => $create_or_update_or_delete_args,
+            )
+        );
+        /**
+         * Rest route for updating faq items.
+         */
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/faq',
+            array(
+                'methods'             => \WP_REST_Server::EDITABLE,
+                'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::update_faq_items',
+                'permission_callback' => function () {
+                    return current_user_can( 'publish_posts' );
+                },
+                'args'                => $create_or_update_or_delete_args,
+            )
+        );
+        /**
+         * Rest route for getting the faq items.
+         */
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/faq/(?P<post_id>\d+)',
+            array(
+                'methods'             => \WP_REST_Server::READABLE,
+                'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::get_faq_items',
+                'permission_callback' => function () {
+                    return current_user_can( 'publish_posts' );
+                },
+                'args'                => array( 'post_id' => $post_id_validation_settings ),
+            )
+        );
+        /**
+         * Rest route for deleting faq item.
+         */
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/faq',
+            array(
+                'methods'             => \WP_REST_Server::DELETABLE,
+                'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::delete_faq_items',
+                'permission_callback' => function () {
+                    return current_user_can( 'publish_posts' );
+                },
+                'args'                => $create_or_update_or_delete_args,
+            )
+        );
+    }
+
+    /**
+     * Get all FAQ items for a post id.
+     *
+     * @param $request $request WP_REST_Request $request {@link WP_REST_Request instance}.
+     *
+     * @return array Result array, if post id is not given then error array is returned.
+     */
+    public static function get_faq_items( $request ) {
+        $data    = $request->get_params();
+        $post_id = (int) $data['post_id'];
+
+        return get_post_meta( $post_id, self::FAQ_META_KEY );
+    }
+
+    /**
+     * Update all FAQ items for a post id.
+     *
+     * @param $request $request WP_REST_Request $request {@link WP_REST_Request instance}.
+     *
+     * @return array Result array, if post id is not given then error array is returned.
+     */
+    public static function update_faq_items( $request ) {
+        $data      = $request->get_params();
+        $post_id   = (int) $data['post_id'];
+        $faq_items = (array) $data['faq_items'];
+        foreach ( $faq_items as $faq_item ) {
+            $previous_value = array(
+                'question' => (string) $faq_item['previous_question_value'],
+                'answer'   => (string) $faq_item['previous_answer_value'],
+                'id'       => (int) $faq_item['id'],
+            );
+            $new_value      = array(
+                'question' => (string) $faq_item['question'],
+                'answer'   => (string) $faq_item['answer'],
+                'id'       => (int) $faq_item['id'],
+            );
+            update_post_meta( $post_id, self::FAQ_META_KEY, $new_value, $previous_value );
+        }
+
+        return array(
+            'status'  => 'success',
+            'message' => __( 'Faq Items updated successfully', 'wordlift' ),
+        );
+
+    }
+
+    /**
+     * Delete Faq items.
+     *
+     * @param $request WP_REST_Request $request {@link WP_REST_Request instance}.
+     *
+     * @return array Associative array whether the faq item is deleted or not.
+     */
+    public static function delete_faq_items( $request ) {
+        $post_data = $request->get_params();
+        $post_id   = $post_data['post_id'];
+        $faq_items = $post_data['faq_items'];
+        foreach ( $faq_items as $faq_item ) {
+            /**
+             * Note: the order of keys is important in order to delete it properly
+             * If the order change, delete operation will fail since it is converted
+             * in to a string when it was stored.
+             * we cant rely on client to post it in correct order, so we create an array
+             * in correct order.
+             */
+            $deleted_faq_item = array(
+                'question' => $faq_item['question'],
+                'answer'   => $faq_item['answer'],
+                'id'       => (int) $faq_item['id'],
+            );
+            /**
+             * If the field to be deleted is answer, then don't delete the faq item,
+             * if it is question then delete the faq item.
+             */
+            if ( self::ANSWER === $faq_item['field_to_be_deleted'] ) {
+                $previous_value = $deleted_faq_item;
+                // then don't delete the faq item, set the answer as empty.
+                $deleted_faq_item['answer'] = '';
+                $new_value                  = $deleted_faq_item;
+                update_post_meta( $post_id, self::FAQ_META_KEY, $new_value, $previous_value );
+            } elseif ( self::QUESTION === $faq_item['field_to_be_deleted'] ) {
+                /**
+                 * If the question is deleted, then delete the faq item.
+                 */
+                delete_post_meta( $post_id, self::FAQ_META_KEY, $deleted_faq_item );
+            }
+        }
+
+        /**
+         * We are returning only the first item id, since the user can select only one text at a time.
+         */
+        return array(
+            'status'  => 'success',
+            'message' => __( 'Faq item successfully deleted.', 'wordlift' ),
+        );
+    }
+
+    /**
+     * Insert or update FAQ items.
+     *
+     * @param $request WP_REST_Request $request {@link WP_REST_Request instance}.
+     *
+     * @return array Associative array whether the faq item is inserted or not
+     */
+    public static function add_faq_items( $request ) {
+        $post_data = $request->get_params();
+        $post_id   = $post_data['post_id'];
+        $faq_items = $post_data['faq_items'];
+        foreach ( $faq_items as &$faq_item ) {
+            // Add an identifier id to the faq item, it helps to prevent duplication problem.
+            /**
+             * We are using time() and a random integer to prevent
+             * duplication problem.
+             */
+            $faq_item['id'] = time() + wp_rand( 1, 100 ) + wp_rand( 1, 100 );
+            add_post_meta( (int) $post_id, self::FAQ_META_KEY, $faq_item );
+        }
+
+        /**
+         * We are returning only the first item id, since the user can select only one text at a time.
+         */
+        return array(
+            'status'  => 'success',
+            'message' => __( 'Question successfully added.', 'wordlift' ),
+            'id'      => (int) $faq_items[0]['id'],
+        );
+
+    }
 }

Spacing +34 added lines, -34 removed lines

@@ -26,25 +26,25 @@ 
 	const ANSWER   = 'answer';
 
 	public static function register_routes() {
-		add_action( 'rest_api_init', 'Wordlift\FAQ\FAQ_Rest_Controller::register_route_callback' );
+		add_action('rest_api_init', 'Wordlift\FAQ\FAQ_Rest_Controller::register_route_callback');
 	}
 
 	public static function register_route_callback() {
 		/**
 		 * Specifies the list of arguments to be present for update request.
 		 */
-		$post_id_validation_settings   = array(
+		$post_id_validation_settings = array(
 			'required'          => true,
 			// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-			'validate_callback' => function ( $param, $request, $key ) {
-				return is_numeric( $param );
+			'validate_callback' => function($param, $request, $key) {
+				return is_numeric($param);
 			},
 		);
 		$faq_items_validation_settings = array(
 			'required'          => true,
 			// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-			'validate_callback' => function ( $param, $request, $key ) {
-				return is_array( $param ) && count( $param ) > 0;
+			'validate_callback' => function($param, $request, $key) {
+				return is_array($param) && count($param) > 0;
 			},
 		);
 		/**
@@ -65,8 +65,8 @@ 
 			array(
 				'methods'             => \WP_REST_Server::CREATABLE,
 				'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::add_faq_items',
-				'permission_callback' => function () {
-					return current_user_can( 'publish_posts' );
+				'permission_callback' => function() {
+					return current_user_can('publish_posts');
 				},
 				'args'                => $create_or_update_or_delete_args,
 			)
@@ -80,8 +80,8 @@ 
 			array(
 				'methods'             => \WP_REST_Server::EDITABLE,
 				'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::update_faq_items',
-				'permission_callback' => function () {
-					return current_user_can( 'publish_posts' );
+				'permission_callback' => function() {
+					return current_user_can('publish_posts');
 				},
 				'args'                => $create_or_update_or_delete_args,
 			)
@@ -95,10 +95,10 @@ 
 			array(
 				'methods'             => \WP_REST_Server::READABLE,
 				'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::get_faq_items',
-				'permission_callback' => function () {
-					return current_user_can( 'publish_posts' );
+				'permission_callback' => function() {
+					return current_user_can('publish_posts');
 				},
-				'args'                => array( 'post_id' => $post_id_validation_settings ),
+				'args'                => array('post_id' => $post_id_validation_settings),
 			)
 		);
 		/**
@@ -110,8 +110,8 @@ 
 			array(
 				'methods'             => \WP_REST_Server::DELETABLE,
 				'callback'            => 'Wordlift\Faq\Faq_Rest_Controller::delete_faq_items',
-				'permission_callback' => function () {
-					return current_user_can( 'publish_posts' );
+				'permission_callback' => function() {
+					return current_user_can('publish_posts');
 				},
 				'args'                => $create_or_update_or_delete_args,
 			)
@@ -125,11 +125,11 @@ 
 	 *
 	 * @return array Result array, if post id is not given then error array is returned.
 	 */
-	public static function get_faq_items( $request ) {
+	public static function get_faq_items($request) {
 		$data    = $request->get_params();
 		$post_id = (int) $data['post_id'];
 
-		return get_post_meta( $post_id, self::FAQ_META_KEY );
+		return get_post_meta($post_id, self::FAQ_META_KEY);
 	}
 
 	/**
@@ -139,27 +139,27 @@ 
 	 *
 	 * @return array Result array, if post id is not given then error array is returned.
 	 */
-	public static function update_faq_items( $request ) {
+	public static function update_faq_items($request) {
 		$data      = $request->get_params();
 		$post_id   = (int) $data['post_id'];
 		$faq_items = (array) $data['faq_items'];
-		foreach ( $faq_items as $faq_item ) {
+		foreach ($faq_items as $faq_item) {
 			$previous_value = array(
 				'question' => (string) $faq_item['previous_question_value'],
 				'answer'   => (string) $faq_item['previous_answer_value'],
 				'id'       => (int) $faq_item['id'],
 			);
-			$new_value      = array(
+			$new_value = array(
 				'question' => (string) $faq_item['question'],
 				'answer'   => (string) $faq_item['answer'],
 				'id'       => (int) $faq_item['id'],
 			);
-			update_post_meta( $post_id, self::FAQ_META_KEY, $new_value, $previous_value );
+			update_post_meta($post_id, self::FAQ_META_KEY, $new_value, $previous_value);
 		}
 
 		return array(
 			'status'  => 'success',
-			'message' => __( 'Faq Items updated successfully', 'wordlift' ),
+			'message' => __('Faq Items updated successfully', 'wordlift'),
 		);
 
 	}
@@ -171,11 +171,11 @@ 
 	 *
 	 * @return array Associative array whether the faq item is deleted or not.
 	 */
-	public static function delete_faq_items( $request ) {
+	public static function delete_faq_items($request) {
 		$post_data = $request->get_params();
 		$post_id   = $post_data['post_id'];
 		$faq_items = $post_data['faq_items'];
-		foreach ( $faq_items as $faq_item ) {
+		foreach ($faq_items as $faq_item) {
 			/**
 			 * Note: the order of keys is important in order to delete it properly
 			 * If the order change, delete operation will fail since it is converted
@@ -192,17 +192,17 @@ 
 			 * If the field to be deleted is answer, then don't delete the faq item,
 			 * if it is question then delete the faq item.
 			 */
-			if ( self::ANSWER === $faq_item['field_to_be_deleted'] ) {
+			if (self::ANSWER === $faq_item['field_to_be_deleted']) {
 				$previous_value = $deleted_faq_item;
 				// then don't delete the faq item, set the answer as empty.
 				$deleted_faq_item['answer'] = '';
 				$new_value                  = $deleted_faq_item;
-				update_post_meta( $post_id, self::FAQ_META_KEY, $new_value, $previous_value );
-			} elseif ( self::QUESTION === $faq_item['field_to_be_deleted'] ) {
+				update_post_meta($post_id, self::FAQ_META_KEY, $new_value, $previous_value);
+			} elseif (self::QUESTION === $faq_item['field_to_be_deleted']) {
 				/**
 				 * If the question is deleted, then delete the faq item.
 				 */
-				delete_post_meta( $post_id, self::FAQ_META_KEY, $deleted_faq_item );
+				delete_post_meta($post_id, self::FAQ_META_KEY, $deleted_faq_item);
 			}
 		}
 
@@ -211,7 +211,7 @@ 
 		 */
 		return array(
 			'status'  => 'success',
-			'message' => __( 'Faq item successfully deleted.', 'wordlift' ),
+			'message' => __('Faq item successfully deleted.', 'wordlift'),
 		);
 	}
 
@@ -222,18 +222,18 @@ 
 	 *
 	 * @return array Associative array whether the faq item is inserted or not
 	 */
-	public static function add_faq_items( $request ) {
+	public static function add_faq_items($request) {
 		$post_data = $request->get_params();
 		$post_id   = $post_data['post_id'];
 		$faq_items = $post_data['faq_items'];
-		foreach ( $faq_items as &$faq_item ) {
+		foreach ($faq_items as &$faq_item) {
 			// Add an identifier id to the faq item, it helps to prevent duplication problem.
 			/**
 			 * We are using time() and a random integer to prevent
 			 * duplication problem.
 			 */
-			$faq_item['id'] = time() + wp_rand( 1, 100 ) + wp_rand( 1, 100 );
-			add_post_meta( (int) $post_id, self::FAQ_META_KEY, $faq_item );
+			$faq_item['id'] = time() + wp_rand(1, 100) + wp_rand(1, 100);
+			add_post_meta((int) $post_id, self::FAQ_META_KEY, $faq_item);
 		}
 
 		/**
@@ -241,7 +241,7 @@ 
 		 */
 		return array(
 			'status'  => 'success',
-			'message' => __( 'Question successfully added.', 'wordlift' ),
+			'message' => __('Question successfully added.', 'wordlift'),
 			'id'      => (int) $faq_items[0]['id'],
 		);
 

src/wordlift/faq/class-faq-tinymce-adapter.php

Indentation +36 added lines, -36 removed lines

@@ -13,41 +13,41 @@
 
 class Faq_Tinymce_Adapter {
 
-	const FAQ_TINYMCE_PLUGIN_NAME = 'wl_faq_tinymce';
-
-	const FAQ_TINYMCE_ADD_BUTTON_ID = 'wl-faq-toolbar-button';
-
-	/**
-	 * Add a list of custom tags which is to be used by our highlighting program.
-	 *
-	 * @param $init_array
-	 *
-	 * @return array
-	 */
-	public function register_custom_tags( $init_array ) {
-		$opts                                   = '~wl-faq-question,~wl-faq-answer';
-		$init_array['custom_elements']         .= ( empty( $init_array['custom_elements'] ) ? '' : ',' ) . $opts;
-		$init_array['extended_valid_elements'] .= ( empty( $init_array['extended_valid_elements'] ) ? '' : ',' ) . $opts;
-
-		return $init_array;
-	}
-
-	public function register_faq_tinymce_plugin( $plugins ) {
-		/**
-		 * Registering the tinymce plugin for FAQ here.
-		 *
-		 * @since 3.26.0
-		 */
-		$version                                  = Wordlift::get_instance()->get_version();
-		$plugins[ self::FAQ_TINYMCE_PLUGIN_NAME ] = plugin_dir_url( dirname( __DIR__ ) ) . 'js/dist/tinymce-faq-plugin.full.js?ver=' . $version;
-
-		return $plugins;
-	}
-
-	public function register_faq_toolbar_button( $buttons ) {
-		array_push( $buttons, self::FAQ_TINYMCE_ADD_BUTTON_ID );
-
-		return $buttons;
-	}
+    const FAQ_TINYMCE_PLUGIN_NAME = 'wl_faq_tinymce';
+
+    const FAQ_TINYMCE_ADD_BUTTON_ID = 'wl-faq-toolbar-button';
+
+    /**
+     * Add a list of custom tags which is to be used by our highlighting program.
+     *
+     * @param $init_array
+     *
+     * @return array
+     */
+    public function register_custom_tags( $init_array ) {
+        $opts                                   = '~wl-faq-question,~wl-faq-answer';
+        $init_array['custom_elements']         .= ( empty( $init_array['custom_elements'] ) ? '' : ',' ) . $opts;
+        $init_array['extended_valid_elements'] .= ( empty( $init_array['extended_valid_elements'] ) ? '' : ',' ) . $opts;
+
+        return $init_array;
+    }
+
+    public function register_faq_tinymce_plugin( $plugins ) {
+        /**
+         * Registering the tinymce plugin for FAQ here.
+         *
+         * @since 3.26.0
+         */
+        $version                                  = Wordlift::get_instance()->get_version();
+        $plugins[ self::FAQ_TINYMCE_PLUGIN_NAME ] = plugin_dir_url( dirname( __DIR__ ) ) . 'js/dist/tinymce-faq-plugin.full.js?ver=' . $version;
+
+        return $plugins;
+    }
+
+    public function register_faq_toolbar_button( $buttons ) {
+        array_push( $buttons, self::FAQ_TINYMCE_ADD_BUTTON_ID );
+
+        return $buttons;
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -24,28 +24,28 @@
 	 *
 	 * @return array
 	 */
-	public function register_custom_tags( $init_array ) {
+	public function register_custom_tags($init_array) {
 		$opts                                   = '~wl-faq-question,~wl-faq-answer';
-		$init_array['custom_elements']         .= ( empty( $init_array['custom_elements'] ) ? '' : ',' ) . $opts;
-		$init_array['extended_valid_elements'] .= ( empty( $init_array['extended_valid_elements'] ) ? '' : ',' ) . $opts;
+		$init_array['custom_elements']         .= (empty($init_array['custom_elements']) ? '' : ',').$opts;
+		$init_array['extended_valid_elements'] .= (empty($init_array['extended_valid_elements']) ? '' : ',').$opts;
 
 		return $init_array;
 	}
 
-	public function register_faq_tinymce_plugin( $plugins ) {
+	public function register_faq_tinymce_plugin($plugins) {
 		/**
 		 * Registering the tinymce plugin for FAQ here.
 		 *
 		 * @since 3.26.0
 		 */
 		$version                                  = Wordlift::get_instance()->get_version();
-		$plugins[ self::FAQ_TINYMCE_PLUGIN_NAME ] = plugin_dir_url( dirname( __DIR__ ) ) . 'js/dist/tinymce-faq-plugin.full.js?ver=' . $version;
+		$plugins[self::FAQ_TINYMCE_PLUGIN_NAME] = plugin_dir_url(dirname(__DIR__)).'js/dist/tinymce-faq-plugin.full.js?ver='.$version;
 
 		return $plugins;
 	}
 
-	public function register_faq_toolbar_button( $buttons ) {
-		array_push( $buttons, self::FAQ_TINYMCE_ADD_BUTTON_ID );
+	public function register_faq_toolbar_button($buttons) {
+		array_push($buttons, self::FAQ_TINYMCE_ADD_BUTTON_ID);
 
 		return $buttons;
 	}

src/wordlift/cleanup/class-post-handler.php

Indentation +65 added lines, -65 removed lines

@@ -14,70 +14,70 @@
 
 class Post_Handler {
 
-	/**
-	 * Process post.
-	 *
-	 * @param int $post_id
-	 */
-	public static function fix( $post_id ) {
-		// 3 cases:
-		// 1. item id w/o the base URI: itemid="/entity/marketing"
-		// 2. item id w/ domain outside the scope of dataset URI: itemid="https://data.wordlift.io/wl95583/entity/emailing"
-		// 3. item id w/ domain within the scope of the dataset URI, but non existent.
-		// 4. should we manipulate also the `wp:wordlift/classification` block?
-
-		$post             = get_post( $post_id );
-		$new_post_content = preg_replace_callback(
-			'@<(\w+)[^<]*class="([^"]*)"\sitemid="([^"]+)"[^>]*>(.*?)</\1>@i',
-			array( get_class(), 'fix_annotations' ),
-			$post->post_content,
-			- 1,
-			$count
-		);
-
-		if ( 0 === $count || $new_post_content === $post->post_content ) {
-			// Bail out if the regex doesn't match or if the post content didn't change.
-			return;
-		}
-
-		wp_update_post(
-			array(
-				'ID'           => $post_id,
-				'post_content' => $new_post_content,
-			)
-		);
-
-	}
-
-	public static function fix_annotations( $args ) {
-		// Make the item id relative.
-		if ( 0 === preg_match( '@(?:https?://[^/]+/[^/]+)?/?(.*)@', $args[3], $matches ) ) {
-			return $args[4];
-		}
-
-		$item_id = $matches[1];
-
-		// Bail out if the item id is empty.
-		if ( empty( $item_id ) ) {
-			return $args[4];
-		}
-
-		// Find a matching content.
-		$content_service = Wordpress_Content_Service::get_instance();
-		$content         = $content_service->get_by_entity_id_or_same_as( $item_id );
-
-		if ( ! isset( $content ) ) {
-			// No content found return only label.
-			return $args[4];
-		} else {
-			// Get the actual entity id.
-			$new_item_id = $content_service->get_entity_id(
-				new Wordpress_Content_Id( $content->get_id(), $content->get_object_type_enum() )
-			);
-
-			// Replace the incoming entity id with the actual entity id.
-			return str_replace( $args[3], $new_item_id, $args[0] );
-		}
-	}
+    /**
+     * Process post.
+     *
+     * @param int $post_id
+     */
+    public static function fix( $post_id ) {
+        // 3 cases:
+        // 1. item id w/o the base URI: itemid="/entity/marketing"
+        // 2. item id w/ domain outside the scope of dataset URI: itemid="https://data.wordlift.io/wl95583/entity/emailing"
+        // 3. item id w/ domain within the scope of the dataset URI, but non existent.
+        // 4. should we manipulate also the `wp:wordlift/classification` block?
+
+        $post             = get_post( $post_id );
+        $new_post_content = preg_replace_callback(
+            '@<(\w+)[^<]*class="([^"]*)"\sitemid="([^"]+)"[^>]*>(.*?)</\1>@i',
+            array( get_class(), 'fix_annotations' ),
+            $post->post_content,
+            - 1,
+            $count
+        );
+
+        if ( 0 === $count || $new_post_content === $post->post_content ) {
+            // Bail out if the regex doesn't match or if the post content didn't change.
+            return;
+        }
+
+        wp_update_post(
+            array(
+                'ID'           => $post_id,
+                'post_content' => $new_post_content,
+            )
+        );
+
+    }
+
+    public static function fix_annotations( $args ) {
+        // Make the item id relative.
+        if ( 0 === preg_match( '@(?:https?://[^/]+/[^/]+)?/?(.*)@', $args[3], $matches ) ) {
+            return $args[4];
+        }
+
+        $item_id = $matches[1];
+
+        // Bail out if the item id is empty.
+        if ( empty( $item_id ) ) {
+            return $args[4];
+        }
+
+        // Find a matching content.
+        $content_service = Wordpress_Content_Service::get_instance();
+        $content         = $content_service->get_by_entity_id_or_same_as( $item_id );
+
+        if ( ! isset( $content ) ) {
+            // No content found return only label.
+            return $args[4];
+        } else {
+            // Get the actual entity id.
+            $new_item_id = $content_service->get_entity_id(
+                new Wordpress_Content_Id( $content->get_id(), $content->get_object_type_enum() )
+            );
+
+            // Replace the incoming entity id with the actual entity id.
+            return str_replace( $args[3], $new_item_id, $args[0] );
+        }
+    }
 
 }

Spacing +11 added lines, -11 removed lines

@@ -19,23 +19,23 @@ 
 	 *
 	 * @param int $post_id
 	 */
-	public static function fix( $post_id ) {
+	public static function fix($post_id) {
 		// 3 cases:
 		// 1. item id w/o the base URI: itemid="/entity/marketing"
 		// 2. item id w/ domain outside the scope of dataset URI: itemid="https://data.wordlift.io/wl95583/entity/emailing"
 		// 3. item id w/ domain within the scope of the dataset URI, but non existent.
 		// 4. should we manipulate also the `wp:wordlift/classification` block?
 
-		$post             = get_post( $post_id );
+		$post             = get_post($post_id);
 		$new_post_content = preg_replace_callback(
 			'@<(\w+)[^<]*class="([^"]*)"\sitemid="([^"]+)"[^>]*>(.*?)</\1>@i',
-			array( get_class(), 'fix_annotations' ),
+			array(get_class(), 'fix_annotations'),
 			$post->post_content,
 			- 1,
 			$count
 		);
 
-		if ( 0 === $count || $new_post_content === $post->post_content ) {
+		if (0 === $count || $new_post_content === $post->post_content) {
 			// Bail out if the regex doesn't match or if the post content didn't change.
 			return;
 		}
@@ -49,34 +49,34 @@ 
 
 	}
 
-	public static function fix_annotations( $args ) {
+	public static function fix_annotations($args) {
 		// Make the item id relative.
-		if ( 0 === preg_match( '@(?:https?://[^/]+/[^/]+)?/?(.*)@', $args[3], $matches ) ) {
+		if (0 === preg_match('@(?:https?://[^/]+/[^/]+)?/?(.*)@', $args[3], $matches)) {
 			return $args[4];
 		}
 
 		$item_id = $matches[1];
 
 		// Bail out if the item id is empty.
-		if ( empty( $item_id ) ) {
+		if (empty($item_id)) {
 			return $args[4];
 		}
 
 		// Find a matching content.
 		$content_service = Wordpress_Content_Service::get_instance();
-		$content         = $content_service->get_by_entity_id_or_same_as( $item_id );
+		$content         = $content_service->get_by_entity_id_or_same_as($item_id);
 
-		if ( ! isset( $content ) ) {
+		if ( ! isset($content)) {
 			// No content found return only label.
 			return $args[4];
 		} else {
 			// Get the actual entity id.
 			$new_item_id = $content_service->get_entity_id(
-				new Wordpress_Content_Id( $content->get_id(), $content->get_object_type_enum() )
+				new Wordpress_Content_Id($content->get_id(), $content->get_object_type_enum())
 			);
 
 			// Replace the incoming entity id with the actual entity id.
-			return str_replace( $args[3], $new_item_id, $args[0] );
+			return str_replace($args[3], $new_item_id, $args[0]);
 		}
 	}