insideout10 / wordlift-plugin

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

src/wordlift/http/class-cacheable-http-client.php

Indentation +107 added lines, -107 removed lines

@@ -15,112 +15,112 @@
 // @@todo: add a hook to clear the cached files now and then.
 class Cacheable_Http_Client extends Simple_Http_Client {
 
-	/**
-	 * The TTL of cached responses in seconds.
-	 *
-	 * @var int $ttl The TTL in seconds.
-	 * @access private
-	 * @since 1.0.0
-	 */
-	private $ttl;
-
-	/**
-	 * The cache dir where the cached responses are written.
-	 *
-	 * @since 1.0.0
-	 * @access private
-	 * @var string $cache_dir The cache dir where the cached responses are written.
-	 */
-	private $cache_dir;
-
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 * @access private
-	 * @since 1.0.0
-	 */
-	private $log;
-
-	/**
-	 * Create a {@link Cacheable_Http_Client} with the specified TTL, default 900 secs.
-	 *
-	 * @param int $ttl The cache TTL, default 900 secs.
-	 *
-	 * @since 1.0.0
-	 */
-	public function __construct( $ttl = 900 ) {
-
-		$this->log = \Wordlift_Log_Service::get_logger( get_class() );
-
-		$this->ttl = $ttl;
-
-		// Get the temp dir and add the directory separator if missing.
-		$temp_dir = get_temp_dir();
-		if ( DIRECTORY_SEPARATOR !== substr( $temp_dir, - strlen( DIRECTORY_SEPARATOR ) ) ) {
-			$temp_dir .= DIRECTORY_SEPARATOR;
-		}
-		$this->cache_dir = $temp_dir . 'wlfb-http-cache';
-
-		$this->log->trace( "Creating the cache folder {$this->cache_dir}..." );
-		wp_mkdir_p( $this->cache_dir );
-
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function request( $url, $options = array() ) {
-
-		// Create a hash and a path to the cache file.
-		// phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_serialize
-		$hash     = md5( $url ) . '-' . md5( serialize( $options ) );
-		$filename = $this->get_path( $hash );
-
-		// If the cache file exists and it's not too old, then return it.
-		if ( file_exists( $filename ) && $this->ttl >= time() - filemtime( $filename ) ) {
-			$this->log->trace( "Cache HIT.\n" );
-
-			// phpcs:ignore WordPress.WP.AlternativeFunctions.file_get_contents_file_get_contents
-			return json_decode( file_get_contents( $filename ), true );
-		}
-
-		$this->log->trace( "Cache MISS for URL $url, hash $hash.\n" );
-
-		// Get a fresh response and return it.
-		$response = parent::request( $url, $options );
-
-		// Return immediately, do not cache.
-		if ( is_wp_error( $response ) ) {
-			return $response;
-		}
-
-		// Do not cache response with invalid status codes or status code different from 2xx.
-		$code = wp_remote_retrieve_response_code( $response );
-		if ( ! is_numeric( $code ) || 2 !== intval( $code ) / 100 ) {
-			return $response;
-		}
-
-		// Cache.
-		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@unlink( $filename );
-		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged,WordPress.WP.AlternativeFunctions.json_encode_json_encode,WordPress.WP.AlternativeFunctions.file_system_read_file_put_contents
-		@file_put_contents( $filename, json_encode( $response ) );
-
-		return $response;
-	}
-
-	/**
-	 * Get the full path for the given `$hash`. The file is not checked for its existence.
-	 *
-	 * @param string $hash A file hash.
-	 *
-	 * @return string The full path to the file.
-	 * @since 1.0.0
-	 */
-	private function get_path( $hash ) {
-
-		return $this->cache_dir . DIRECTORY_SEPARATOR . $hash;
-	}
+    /**
+     * The TTL of cached responses in seconds.
+     *
+     * @var int $ttl The TTL in seconds.
+     * @access private
+     * @since 1.0.0
+     */
+    private $ttl;
+
+    /**
+     * The cache dir where the cached responses are written.
+     *
+     * @since 1.0.0
+     * @access private
+     * @var string $cache_dir The cache dir where the cached responses are written.
+     */
+    private $cache_dir;
+
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     * @access private
+     * @since 1.0.0
+     */
+    private $log;
+
+    /**
+     * Create a {@link Cacheable_Http_Client} with the specified TTL, default 900 secs.
+     *
+     * @param int $ttl The cache TTL, default 900 secs.
+     *
+     * @since 1.0.0
+     */
+    public function __construct( $ttl = 900 ) {
+
+        $this->log = \Wordlift_Log_Service::get_logger( get_class() );
+
+        $this->ttl = $ttl;
+
+        // Get the temp dir and add the directory separator if missing.
+        $temp_dir = get_temp_dir();
+        if ( DIRECTORY_SEPARATOR !== substr( $temp_dir, - strlen( DIRECTORY_SEPARATOR ) ) ) {
+            $temp_dir .= DIRECTORY_SEPARATOR;
+        }
+        $this->cache_dir = $temp_dir . 'wlfb-http-cache';
+
+        $this->log->trace( "Creating the cache folder {$this->cache_dir}..." );
+        wp_mkdir_p( $this->cache_dir );
+
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function request( $url, $options = array() ) {
+
+        // Create a hash and a path to the cache file.
+        // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_serialize
+        $hash     = md5( $url ) . '-' . md5( serialize( $options ) );
+        $filename = $this->get_path( $hash );
+
+        // If the cache file exists and it's not too old, then return it.
+        if ( file_exists( $filename ) && $this->ttl >= time() - filemtime( $filename ) ) {
+            $this->log->trace( "Cache HIT.\n" );
+
+            // phpcs:ignore WordPress.WP.AlternativeFunctions.file_get_contents_file_get_contents
+            return json_decode( file_get_contents( $filename ), true );
+        }
+
+        $this->log->trace( "Cache MISS for URL $url, hash $hash.\n" );
+
+        // Get a fresh response and return it.
+        $response = parent::request( $url, $options );
+
+        // Return immediately, do not cache.
+        if ( is_wp_error( $response ) ) {
+            return $response;
+        }
+
+        // Do not cache response with invalid status codes or status code different from 2xx.
+        $code = wp_remote_retrieve_response_code( $response );
+        if ( ! is_numeric( $code ) || 2 !== intval( $code ) / 100 ) {
+            return $response;
+        }
+
+        // Cache.
+        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
+        @unlink( $filename );
+        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged,WordPress.WP.AlternativeFunctions.json_encode_json_encode,WordPress.WP.AlternativeFunctions.file_system_read_file_put_contents
+        @file_put_contents( $filename, json_encode( $response ) );
+
+        return $response;
+    }
+
+    /**
+     * Get the full path for the given `$hash`. The file is not checked for its existence.
+     *
+     * @param string $hash A file hash.
+     *
+     * @return string The full path to the file.
+     * @since 1.0.0
+     */
+    private function get_path( $hash ) {
+
+        return $this->cache_dir . DIRECTORY_SEPARATOR . $hash;
+    }
 
 }

Spacing +21 added lines, -21 removed lines

@@ -49,63 +49,63 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	public function __construct( $ttl = 900 ) {
+	public function __construct($ttl = 900) {
 
-		$this->log = \Wordlift_Log_Service::get_logger( get_class() );
+		$this->log = \Wordlift_Log_Service::get_logger(get_class());
 
 		$this->ttl = $ttl;
 
 		// Get the temp dir and add the directory separator if missing.
 		$temp_dir = get_temp_dir();
-		if ( DIRECTORY_SEPARATOR !== substr( $temp_dir, - strlen( DIRECTORY_SEPARATOR ) ) ) {
+		if (DIRECTORY_SEPARATOR !== substr($temp_dir, - strlen(DIRECTORY_SEPARATOR))) {
 			$temp_dir .= DIRECTORY_SEPARATOR;
 		}
-		$this->cache_dir = $temp_dir . 'wlfb-http-cache';
+		$this->cache_dir = $temp_dir.'wlfb-http-cache';
 
-		$this->log->trace( "Creating the cache folder {$this->cache_dir}..." );
-		wp_mkdir_p( $this->cache_dir );
+		$this->log->trace("Creating the cache folder {$this->cache_dir}...");
+		wp_mkdir_p($this->cache_dir);
 
 	}
 
 	/**
 	 * @inheritDoc
 	 */
-	public function request( $url, $options = array() ) {
+	public function request($url, $options = array()) {
 
 		// Create a hash and a path to the cache file.
 		// phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_serialize
-		$hash     = md5( $url ) . '-' . md5( serialize( $options ) );
-		$filename = $this->get_path( $hash );
+		$hash     = md5($url).'-'.md5(serialize($options));
+		$filename = $this->get_path($hash);
 
 		// If the cache file exists and it's not too old, then return it.
-		if ( file_exists( $filename ) && $this->ttl >= time() - filemtime( $filename ) ) {
-			$this->log->trace( "Cache HIT.\n" );
+		if (file_exists($filename) && $this->ttl >= time() - filemtime($filename)) {
+			$this->log->trace("Cache HIT.\n");
 
 			// phpcs:ignore WordPress.WP.AlternativeFunctions.file_get_contents_file_get_contents
-			return json_decode( file_get_contents( $filename ), true );
+			return json_decode(file_get_contents($filename), true);
 		}
 
-		$this->log->trace( "Cache MISS for URL $url, hash $hash.\n" );
+		$this->log->trace("Cache MISS for URL $url, hash $hash.\n");
 
 		// Get a fresh response and return it.
-		$response = parent::request( $url, $options );
+		$response = parent::request($url, $options);
 
 		// Return immediately, do not cache.
-		if ( is_wp_error( $response ) ) {
+		if (is_wp_error($response)) {
 			return $response;
 		}
 
 		// Do not cache response with invalid status codes or status code different from 2xx.
-		$code = wp_remote_retrieve_response_code( $response );
-		if ( ! is_numeric( $code ) || 2 !== intval( $code ) / 100 ) {
+		$code = wp_remote_retrieve_response_code($response);
+		if ( ! is_numeric($code) || 2 !== intval($code) / 100) {
 			return $response;
 		}
 
 		// Cache.
 		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@unlink( $filename );
+		@unlink($filename);
 		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged,WordPress.WP.AlternativeFunctions.json_encode_json_encode,WordPress.WP.AlternativeFunctions.file_system_read_file_put_contents
-		@file_put_contents( $filename, json_encode( $response ) );
+		@file_put_contents($filename, json_encode($response));
 
 		return $response;
 	}
@@ -118,9 +118,9 @@ 
 	 * @return string The full path to the file.
 	 * @since 1.0.0
 	 */
-	private function get_path( $hash ) {
+	private function get_path($hash) {
 
-		return $this->cache_dir . DIRECTORY_SEPARATOR . $hash;
+		return $this->cache_dir.DIRECTORY_SEPARATOR.$hash;
 	}
 
 }

src/wordlift/http/class-http-client.php

Indentation +20 added lines, -20 removed lines

@@ -15,26 +15,26 @@
  */
 interface Http_Client {
 
-	/**
-	 * Perform a `GET` operation to the specified `$url`.
-	 *
-	 * @param string $url The URL.
-	 * @param array  $options An array of options to pass to WordPress {@link wp_remote_request} function, default: array().
-	 *
-	 * @return \WP_Error|array The response or WP_Error on failure.
-	 * @since 1.0.0
-	 */
-	public function get( $url, $options = array() );
+    /**
+     * Perform a `GET` operation to the specified `$url`.
+     *
+     * @param string $url The URL.
+     * @param array  $options An array of options to pass to WordPress {@link wp_remote_request} function, default: array().
+     *
+     * @return \WP_Error|array The response or WP_Error on failure.
+     * @since 1.0.0
+     */
+    public function get( $url, $options = array() );
 
-	/**
-	 * Perform a request to the specified `$url`.
-	 *
-	 * @param string $url The URL.
-	 * @param array  $options An array of options to pass to WordPress {@link wp_remote_request} function, default: array().
-	 *
-	 * @return \WP_Error|array The response or WP_Error on failure.
-	 * @since 1.0.0
-	 */
-	public function request( $url, $options = array() );
+    /**
+     * Perform a request to the specified `$url`.
+     *
+     * @param string $url The URL.
+     * @param array  $options An array of options to pass to WordPress {@link wp_remote_request} function, default: array().
+     *
+     * @return \WP_Error|array The response or WP_Error on failure.
+     * @since 1.0.0
+     */
+    public function request( $url, $options = array() );
 
 }

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
 	 * @return \WP_Error|array The response or WP_Error on failure.
 	 * @since 1.0.0
 	 */
-	public function get( $url, $options = array() );
+	public function get($url, $options = array());
 
 	/**
 	 * Perform a request to the specified `$url`.
@@ -35,6 +35,6 @@ 
 	 * @return \WP_Error|array The response or WP_Error on failure.
 	 * @since 1.0.0
 	 */
-	public function request( $url, $options = array() );
+	public function request($url, $options = array());
 
 }

src/wordlift/entity-type/class-entity-type-change-handler.php

Indentation +54 added lines, -54 removed lines

@@ -14,59 +14,59 @@
 use Wordlift_Entity_Type_Taxonomy_Service;
 
 class Entity_Type_Change_Handler {
-	/**
-	 * @var Wordlift_Entity_Service
-	 */
-	private $entity_service;
-	/**
-	 * @var Wordlift_Entity_Type_Service
-	 */
-	private $entity_type_service;
-
-	/**
-	 * Entity_Type_Change_Handler constructor.
-	 *
-	 * @param $entity_service Wordlift_Entity_Service
-	 * @param $entity_type_service Wordlift_Entity_Type_Service
-	 */
-	public function __construct( $entity_service, $entity_type_service ) {
-
-		$this->entity_service = $entity_service;
-
-		$this->entity_type_service = $entity_type_service;
-
-		// Takes a performance toll, do we really need it?
-		// add_action( 'set_object_terms', array( $this, 'set_object_terms' ), 10, 4 );
-	}
-
-	public function set_object_terms( $object_id, $terms, $tt_ids, $taxonomy ) {
-
-		if ( Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME !== $taxonomy ) {
-			return;
-		}
-
-		if ( count( $terms ) !== 1 ) {
-			// Unable to determine which entity type or multiple entity types.
-			return;
-		}
-
-		// This taxonomy is registered only for post, so the object id would
-		// be the post id.
-		$types = $this->entity_type_service->get_names( $object_id );
-
-		if ( count( $types ) !== 1 ) {
-			// Unable to determine which entity type or multiple entity types.
-			return;
-		}
-
-		// Check if set to Article or one of its descendants
-		if ( ! in_array( $types[0], Jsonld_Article_Wrapper::$article_types, true ) ) {
-			return;
-		}
-
-		// clear the labels.
-		$this->entity_service->set_alternative_labels( $object_id, array() );
-
-	}
+    /**
+     * @var Wordlift_Entity_Service
+     */
+    private $entity_service;
+    /**
+     * @var Wordlift_Entity_Type_Service
+     */
+    private $entity_type_service;
+
+    /**
+     * Entity_Type_Change_Handler constructor.
+     *
+     * @param $entity_service Wordlift_Entity_Service
+     * @param $entity_type_service Wordlift_Entity_Type_Service
+     */
+    public function __construct( $entity_service, $entity_type_service ) {
+
+        $this->entity_service = $entity_service;
+
+        $this->entity_type_service = $entity_type_service;
+
+        // Takes a performance toll, do we really need it?
+        // add_action( 'set_object_terms', array( $this, 'set_object_terms' ), 10, 4 );
+    }
+
+    public function set_object_terms( $object_id, $terms, $tt_ids, $taxonomy ) {
+
+        if ( Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME !== $taxonomy ) {
+            return;
+        }
+
+        if ( count( $terms ) !== 1 ) {
+            // Unable to determine which entity type or multiple entity types.
+            return;
+        }
+
+        // This taxonomy is registered only for post, so the object id would
+        // be the post id.
+        $types = $this->entity_type_service->get_names( $object_id );
+
+        if ( count( $types ) !== 1 ) {
+            // Unable to determine which entity type or multiple entity types.
+            return;
+        }
+
+        // Check if set to Article or one of its descendants
+        if ( ! in_array( $types[0], Jsonld_Article_Wrapper::$article_types, true ) ) {
+            return;
+        }
+
+        // clear the labels.
+        $this->entity_service->set_alternative_labels( $object_id, array() );
+
+    }
 
 }

Spacing +8 added lines, -8 removed lines

@@ -29,7 +29,7 @@ 
 	 * @param $entity_service Wordlift_Entity_Service
 	 * @param $entity_type_service Wordlift_Entity_Type_Service
 	 */
-	public function __construct( $entity_service, $entity_type_service ) {
+	public function __construct($entity_service, $entity_type_service) {
 
 		$this->entity_service = $entity_service;
 
@@ -39,33 +39,33 @@ 
 		// add_action( 'set_object_terms', array( $this, 'set_object_terms' ), 10, 4 );
 	}
 
-	public function set_object_terms( $object_id, $terms, $tt_ids, $taxonomy ) {
+	public function set_object_terms($object_id, $terms, $tt_ids, $taxonomy) {
 
-		if ( Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME !== $taxonomy ) {
+		if (Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME !== $taxonomy) {
 			return;
 		}
 
-		if ( count( $terms ) !== 1 ) {
+		if (count($terms) !== 1) {
 			// Unable to determine which entity type or multiple entity types.
 			return;
 		}
 
 		// This taxonomy is registered only for post, so the object id would
 		// be the post id.
-		$types = $this->entity_type_service->get_names( $object_id );
+		$types = $this->entity_type_service->get_names($object_id);
 
-		if ( count( $types ) !== 1 ) {
+		if (count($types) !== 1) {
 			// Unable to determine which entity type or multiple entity types.
 			return;
 		}
 
 		// Check if set to Article or one of its descendants
-		if ( ! in_array( $types[0], Jsonld_Article_Wrapper::$article_types, true ) ) {
+		if ( ! in_array($types[0], Jsonld_Article_Wrapper::$article_types, true)) {
 			return;
 		}
 
 		// clear the labels.
-		$this->entity_service->set_alternative_labels( $object_id, array() );
+		$this->entity_service->set_alternative_labels($object_id, array());
 
 	}