insideout10 / wordlift-plugin

src/wordlift/tasks/class-task.php

Indentation +38 added lines, -38 removed lines

@@ -15,43 +15,43 @@
  */
 interface Task {
 
-	/**
-	 * Define the task ID.
-	 *
-	 * @return string The task id.
-	 * @since 1.0.0
-	 */
-	function get_id();
-
-	function get_label();
-
-	/**
-	 * List the items to process.
-	 *
-	 * @param int $limit
-	 * @param int $offset
-	 *
-	 * @return array An array of items.
-	 * @since 1.0.0
-	 */
-	function list_items( $limit = 10, $offset = 0 );
-
-	/**
-	 * Count the total number of items to process.
-	 *
-	 * @return int Total number of items to process.
-	 * @since 1.0.0
-	 */
-	function count_items();
-
-	/**
-	 * Process the provided item.
-	 *
-	 * @param mixed $item Process the provided item.
-	 *
-	 * @since 1.0.0
-	 *
-	 */
-	function process_item( $item );
+    /**
+     * Define the task ID.
+     *
+     * @return string The task id.
+     * @since 1.0.0
+     */
+    function get_id();
+
+    function get_label();
+
+    /**
+     * List the items to process.
+     *
+     * @param int $limit
+     * @param int $offset
+     *
+     * @return array An array of items.
+     * @since 1.0.0
+     */
+    function list_items( $limit = 10, $offset = 0 );
+
+    /**
+     * Count the total number of items to process.
+     *
+     * @return int Total number of items to process.
+     * @since 1.0.0
+     */
+    function count_items();
+
+    /**
+     * Process the provided item.
+     *
+     * @param mixed $item Process the provided item.
+     *
+     * @since 1.0.0
+     *
+     */
+    function process_item( $item );
 
 }

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
 	 * @return array An array of items.
 	 * @since 1.0.0
 	 */
-	function list_items( $limit = 10, $offset = 0 );
+	function list_items($limit = 10, $offset = 0);
 
 	/**
 	 * Count the total number of items to process.
@@ -52,6 +52,6 @@ 
 	 * @since 1.0.0
 	 *
 	 */
-	function process_item( $item );
+	function process_item($item);
 
 }

src/wordlift/wordpress/class-submenu-page-base.php

Indentation +67 added lines, -67 removed lines

@@ -3,83 +3,83 @@
 namespace Wordlift\Wordpress;
 
 abstract class Submenu_Page_Base implements Page {
-	/**
-	 * @var string
-	 */
-	private $menu_slug;
-	/**
-	 * @var string
-	 */
-	private $page_title;
-	/**
-	 * @var string
-	 */
-	private $capability;
-	/**
-	 * @var string|null
-	 */
-	private $parent_slug;
-	/**
-	 * @var string|null
-	 */
-	private $menu_title;
+    /**
+     * @var string
+     */
+    private $menu_slug;
+    /**
+     * @var string
+     */
+    private $page_title;
+    /**
+     * @var string
+     */
+    private $capability;
+    /**
+     * @var string|null
+     */
+    private $parent_slug;
+    /**
+     * @var string|null
+     */
+    private $menu_title;
 
-	/**
-	 * Abstract_Submenu_Page constructor.
-	 *
-	 * @param string $menu_slug
-	 * @param string $page_title
-	 * @param string $capability
-	 * @param string|null $parent_slug
-	 * @param string|null $menu_title
-	 */
-	public function __construct( $menu_slug, $page_title, $capability = 'manage_options', $parent_slug = null, $menu_title = null ) {
+    /**
+     * Abstract_Submenu_Page constructor.
+     *
+     * @param string $menu_slug
+     * @param string $page_title
+     * @param string $capability
+     * @param string|null $parent_slug
+     * @param string|null $menu_title
+     */
+    public function __construct( $menu_slug, $page_title, $capability = 'manage_options', $parent_slug = null, $menu_title = null ) {
 
-		add_action( 'admin_menu', array( $this, 'admin_menu', ) );
+        add_action( 'admin_menu', array( $this, 'admin_menu', ) );
 
-		$this->menu_slug   = $menu_slug;
-		$this->page_title  = $page_title;
-		$this->capability  = $capability;
-		$this->parent_slug = $parent_slug;
-		$this->menu_title  = isset( $menu_title ) ? $menu_title : $page_title;
-	}
+        $this->menu_slug   = $menu_slug;
+        $this->page_title  = $page_title;
+        $this->capability  = $capability;
+        $this->parent_slug = $parent_slug;
+        $this->menu_title  = isset( $menu_title ) ? $menu_title : $page_title;
+    }
 
-	public function get_menu_slug() {
+    public function get_menu_slug() {
 
-		return $this->menu_slug;
-	}
+        return $this->menu_slug;
+    }
 
-	/**
-	 * The `admin_menu` callback. Will call {@link add_submenu_page} to add the
-	 * page to the admin menu.
-	 *
-	 * @return false|string The resulting page's hook_suffix, or false if the user does not have the capability required.
-	 * @since 1.0.0
-	 *
-	 */
-	public function admin_menu() {
+    /**
+     * The `admin_menu` callback. Will call {@link add_submenu_page} to add the
+     * page to the admin menu.
+     *
+     * @return false|string The resulting page's hook_suffix, or false if the user does not have the capability required.
+     * @since 1.0.0
+     *
+     */
+    public function admin_menu() {
 
-		// Add the sub-menu page.
-		//
-		// See http://codex.wordpress.org/Function_Reference/add_submenu_page
-		$page = add_submenu_page(
-			$this->parent_slug,
-			$this->page_title,
-			$this->menu_title,
-			$this->capability,
-			$this->menu_slug,
-			array( $this, 'render' )
-		);
+        // Add the sub-menu page.
+        //
+        // See http://codex.wordpress.org/Function_Reference/add_submenu_page
+        $page = add_submenu_page(
+            $this->parent_slug,
+            $this->page_title,
+            $this->menu_title,
+            $this->capability,
+            $this->menu_slug,
+            array( $this, 'render' )
+        );
 
-		// Set a hook to enqueue scripts only when the settings page is displayed.
-		add_action( 'admin_print_scripts-' . $page, array( $this, 'enqueue_scripts', ) );
+        // Set a hook to enqueue scripts only when the settings page is displayed.
+        add_action( 'admin_print_scripts-' . $page, array( $this, 'enqueue_scripts', ) );
 
-		// Finally return the page hook_suffix.
-		return $page;
-	}
+        // Finally return the page hook_suffix.
+        return $page;
+    }
 
-	abstract function enqueue_scripts();
+    abstract function enqueue_scripts();
 
-	abstract function render();
+    abstract function render();
 
 }

Spacing +5 added lines, -5 removed lines

@@ -33,15 +33,15 @@ 
 	 * @param string|null $parent_slug
 	 * @param string|null $menu_title
 	 */
-	public function __construct( $menu_slug, $page_title, $capability = 'manage_options', $parent_slug = null, $menu_title = null ) {
+	public function __construct($menu_slug, $page_title, $capability = 'manage_options', $parent_slug = null, $menu_title = null) {
 
-		add_action( 'admin_menu', array( $this, 'admin_menu', ) );
+		add_action('admin_menu', array($this, 'admin_menu',));
 
 		$this->menu_slug   = $menu_slug;
 		$this->page_title  = $page_title;
 		$this->capability  = $capability;
 		$this->parent_slug = $parent_slug;
-		$this->menu_title  = isset( $menu_title ) ? $menu_title : $page_title;
+		$this->menu_title  = isset($menu_title) ? $menu_title : $page_title;
 	}
 
 	public function get_menu_slug() {
@@ -68,11 +68,11 @@ 
 			$this->menu_title,
 			$this->capability,
 			$this->menu_slug,
-			array( $this, 'render' )
+			array($this, 'render')
 		);
 
 		// Set a hook to enqueue scripts only when the settings page is displayed.
-		add_action( 'admin_print_scripts-' . $page, array( $this, 'enqueue_scripts', ) );
+		add_action('admin_print_scripts-'.$page, array($this, 'enqueue_scripts',));
 
 		// Finally return the page hook_suffix.
 		return $page;

src/wordlift/wordpress/class-page.php

Indentation +1 added lines, -1 removed lines

@@ -4,6 +4,6 @@
 
 interface Page {
 
-	public function get_menu_slug();
+    public function get_menu_slug();
 
 }

src/wordlift/api/class-user-agent.php

Indentation +16 added lines, -16 removed lines

@@ -6,28 +6,28 @@
 
 class User_Agent {
 
-	public static function get_user_agent() {
+    public static function get_user_agent() {
 
-		// Get WL version.
-		$wl_version = Wordlift::get_instance()->get_version();
+        // Get WL version.
+        $wl_version = Wordlift::get_instance()->get_version();
 
-		// Get the WP version.
-		$wp_version = get_bloginfo( 'version' );
+        // Get the WP version.
+        $wp_version = get_bloginfo( 'version' );
 
-		// Get the home url.
-		$home_url = home_url( '/' );
+        // Get the home url.
+        $home_url = home_url( '/' );
 
-		// Get the locale flag.
-		$locale = apply_filters( 'core_version_check_locale', get_locale() );
+        // Get the locale flag.
+        $locale = apply_filters( 'core_version_check_locale', get_locale() );
 
-		// Get the multisite flag.
-		$multisite = is_multisite() ? 'yes' : 'no';
+        // Get the multisite flag.
+        $multisite = is_multisite() ? 'yes' : 'no';
 
-		// Get the PHP version.
-		$php_version = phpversion();
+        // Get the PHP version.
+        $php_version = phpversion();
 
-		/** @var string $wp_version The variable is defined in `version.php`. */
-		return "WordLift/$wl_version WordPress/$wp_version (multisite:$multisite, url:$home_url, locale:$locale) PHP/$php_version";
-	}
+        /** @var string $wp_version The variable is defined in `version.php`. */
+        return "WordLift/$wl_version WordPress/$wp_version (multisite:$multisite, url:$home_url, locale:$locale) PHP/$php_version";
+    }
 
 }
\ No newline at end of file

Spacing +3 added lines, -3 removed lines

@@ -12,13 +12,13 @@
 		$wl_version = Wordlift::get_instance()->get_version();
 
 		// Get the WP version.
-		$wp_version = get_bloginfo( 'version' );
+		$wp_version = get_bloginfo('version');
 
 		// Get the home url.
-		$home_url = home_url( '/' );
+		$home_url = home_url('/');
 
 		// Get the locale flag.
-		$locale = apply_filters( 'core_version_check_locale', get_locale() );
+		$locale = apply_filters('core_version_check_locale', get_locale());
 
 		// Get the multisite flag.
 		$multisite = is_multisite() ? 'yes' : 'no';

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

Indentation +28 added lines, -28 removed lines

@@ -10,34 +10,34 @@
 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_opening_tag = '<' . self::FAQ_QUESTION_TAG_NAME . '>';
-		$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_opening_tag = '<' . self::FAQ_ANSWER_TAG_NAME . '>';
-		$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_opening_tag = '<' . self::FAQ_QUESTION_TAG_NAME . '>';
+        $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_opening_tag = '<' . self::FAQ_ANSWER_TAG_NAME . '>';
+        $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;
+    }
 
 }
\ No newline at end of file

Spacing +7 added lines, -7 removed lines

@@ -20,20 +20,20 @@
 	 * @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_opening_tag = '<' . self::FAQ_QUESTION_TAG_NAME . '>';
-		$faq_question_closing_tag = '</' . self::FAQ_QUESTION_TAG_NAME . '>';
-		$content = preg_replace('/<wl-faq-question class=".+?">/m', '', $content );
+		$faq_question_opening_tag = '<'.self::FAQ_QUESTION_TAG_NAME.'>';
+		$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_opening_tag = '<' . self::FAQ_ANSWER_TAG_NAME . '>';
-		$faq_answer_closing_tag = '</' . self::FAQ_ANSWER_TAG_NAME . '>';
-		$content = preg_replace('/<wl-faq-answer class=".+?">/m', '', $content );
+		$faq_answer_opening_tag = '<'.self::FAQ_ANSWER_TAG_NAME.'>';
+		$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 */

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

Braces +1 added lines, -2 removed lines

@@ -195,8 +195,7 @@
 				$deleted_faq_item['answer'] = '';
 				$new_value = $deleted_faq_item;
 				update_post_meta($post_id, self::FAQ_META_KEY, $new_value, $previous_value);
-			}
-			else if ( $faq_item['field_to_be_deleted'] === self::QUESTION) {
+			} else if ( $faq_item['field_to_be_deleted'] === self::QUESTION) {
 				/**
 				 * If the question is deleted, then delete the faq item.
 				 */

Indentation +228 added lines, -228 removed lines

@@ -17,232 +17,232 @@
  *
  */
 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,
-			'validate_callback' => function ( $param, $request, $key ) {
-				return is_numeric( $param );
-			}
-		);
-		$faq_items_validation_settings   = array(
-			'required'          => TRUE,
-			'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' )
-		);
-
-	}
-
-	/**
-	 * 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 dont delete the faq item,
-			 * if it is question then delete the faq item.
-			 */
-			if ( $faq_item['field_to_be_deleted'] === self::ANSWER) {
-				$previous_value = $deleted_faq_item;
-				// then dont 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);
-			}
-			else if ( $faq_item['field_to_be_deleted'] === self::QUESTION) {
-				/**
-				 * 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.' ),
-		);
-	}
-
-	/**
-	 * 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.' ),
-			'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,
+            'validate_callback' => function ( $param, $request, $key ) {
+                return is_numeric( $param );
+            }
+        );
+        $faq_items_validation_settings   = array(
+            'required'          => TRUE,
+            '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' )
+        );
+
+    }
+
+    /**
+     * 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 dont delete the faq item,
+             * if it is question then delete the faq item.
+             */
+            if ( $faq_item['field_to_be_deleted'] === self::ANSWER) {
+                $previous_value = $deleted_faq_item;
+                // then dont 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);
+            }
+            else if ( $faq_item['field_to_be_deleted'] === self::QUESTION) {
+                /**
+                 * 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.' ),
+        );
+    }
+
+    /**
+     * 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.' ),
+            'id'      => (int) $faq_items[0]['id'],
+        );
+
+    }
 }

Spacing +34 added lines, -34 removed lines

@@ -26,23 +26,23 @@ 
 	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,
-			'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(
+		$faq_items_validation_settings = array(
 			'required'          => TRUE,
-			'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;
 			}
 		);
 		/**
@@ -63,8 +63,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
 			) );
@@ -77,8 +77,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
 			)
@@ -92,10 +92,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)
 			)
 		);
 		/**
@@ -107,8 +107,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
 			)
@@ -122,11 +122,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);
 	}
 
 	/**
@@ -136,27 +136,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' )
+			'message' => __('Faq Items updated successfully')
 		);
 
 	}
@@ -168,11 +168,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
@@ -189,18 +189,18 @@ 
 			 * If the field to be deleted is answer, then dont delete the faq item,
 			 * if it is question then delete the faq item.
 			 */
-			if ( $faq_item['field_to_be_deleted'] === self::ANSWER) {
+			if ($faq_item['field_to_be_deleted'] === self::ANSWER) {
 				$previous_value = $deleted_faq_item;
 				// then dont 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);
 			}
-			else if ( $faq_item['field_to_be_deleted'] === self::QUESTION) {
+			else if ($faq_item['field_to_be_deleted'] === self::QUESTION) {
 				/**
 				 * 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);
 			}
 
 		}
@@ -210,7 +210,7 @@ 
 		 */
 		return array(
 			'status'  => 'success',
-			'message' => __( 'Faq item successfully deleted.' ),
+			'message' => __('Faq item successfully deleted.'),
 		);
 	}
 
@@ -221,18 +221,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);
 		}
 
 		/**
@@ -240,7 +240,7 @@ 
 		 */
 		return array(
 			'status'  => 'success',
-			'message' => __( 'Question successfully added.' ),
+			'message' => __('Question successfully added.'),
 			'id'      => (int) $faq_items[0]['id'],
 		);
 

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

Indentation +35 added lines, -35 removed lines

@@ -13,40 +13,40 @@
 
 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 +8 added lines, -8 removed lines

@@ -24,27 +24,27 @@
 	 *
 	 * @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;
+	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 ) {
+	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/includes/linked-data/storage/class-wordlift-post-image-storage.php

Indentation +40 added lines, -40 removed lines

@@ -18,45 +18,45 @@
  */
 class Wordlift_Post_Image_Storage extends Wordlift_Storage {
 
-	/**
-	 * Get the property value.
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array|string|null A single string, or an array of values or null
-	 *                           if the property isn't recognized.
-	 * @since 3.15.0
-	 *
-	 */
-	public function get( $post_id ) {
-
-		// Prepare the return array.
-		$image_urls = array();
-
-		// If there is a featured image it has the priority.
-		$featured_image_id = get_post_thumbnail_id( $post_id );
-		if ( is_numeric( $featured_image_id ) && 0 < $featured_image_id ) {
-			$image_url = wp_get_attachment_url( $featured_image_id );
-
-			$image_urls[] = $image_url;
-		}
-
-		$images = get_children( array(
-			'post_parent'    => $post_id,
-			'post_type'      => 'attachment',
-			'post_mime_type' => 'image',
-		) );
-
-		// Collect the URLs.
-		foreach ( $images as $attachment_id => $attachment ) {
-			$image_url = wp_get_attachment_url( $attachment_id );
-			// Ensure the URL isn't collected already.
-			if ( ! in_array( $image_url, $image_urls ) ) {
-				array_push( $image_urls, $image_url );
-			}
-		}
-
-		return $image_urls;
-	}
+    /**
+     * Get the property value.
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array|string|null A single string, or an array of values or null
+     *                           if the property isn't recognized.
+     * @since 3.15.0
+     *
+     */
+    public function get( $post_id ) {
+
+        // Prepare the return array.
+        $image_urls = array();
+
+        // If there is a featured image it has the priority.
+        $featured_image_id = get_post_thumbnail_id( $post_id );
+        if ( is_numeric( $featured_image_id ) && 0 < $featured_image_id ) {
+            $image_url = wp_get_attachment_url( $featured_image_id );
+
+            $image_urls[] = $image_url;
+        }
+
+        $images = get_children( array(
+            'post_parent'    => $post_id,
+            'post_type'      => 'attachment',
+            'post_mime_type' => 'image',
+        ) );
+
+        // Collect the URLs.
+        foreach ( $images as $attachment_id => $attachment ) {
+            $image_url = wp_get_attachment_url( $attachment_id );
+            // Ensure the URL isn't collected already.
+            if ( ! in_array( $image_url, $image_urls ) ) {
+                array_push( $image_urls, $image_url );
+            }
+        }
+
+        return $image_urls;
+    }
 
 }

Spacing +10 added lines, -10 removed lines

@@ -28,31 +28,31 @@
 	 * @since 3.15.0
 	 *
 	 */
-	public function get( $post_id ) {
+	public function get($post_id) {
 
 		// Prepare the return array.
 		$image_urls = array();
 
 		// If there is a featured image it has the priority.
-		$featured_image_id = get_post_thumbnail_id( $post_id );
-		if ( is_numeric( $featured_image_id ) && 0 < $featured_image_id ) {
-			$image_url = wp_get_attachment_url( $featured_image_id );
+		$featured_image_id = get_post_thumbnail_id($post_id);
+		if (is_numeric($featured_image_id) && 0 < $featured_image_id) {
+			$image_url = wp_get_attachment_url($featured_image_id);
 
 			$image_urls[] = $image_url;
 		}
 
-		$images = get_children( array(
+		$images = get_children(array(
 			'post_parent'    => $post_id,
 			'post_type'      => 'attachment',
 			'post_mime_type' => 'image',
-		) );
+		));
 
 		// Collect the URLs.
-		foreach ( $images as $attachment_id => $attachment ) {
-			$image_url = wp_get_attachment_url( $attachment_id );
+		foreach ($images as $attachment_id => $attachment) {
+			$image_url = wp_get_attachment_url($attachment_id);
 			// Ensure the URL isn't collected already.
-			if ( ! in_array( $image_url, $image_urls ) ) {
-				array_push( $image_urls, $image_url );
+			if ( ! in_array($image_url, $image_urls)) {
+				array_push($image_urls, $image_url);
 			}
 		}
 

src/wordlift/jsonld/class-jsonld-response-helper.php

Indentation +26 added lines, -26 removed lines

@@ -17,31 +17,31 @@
 
 class Jsonld_Response_Helper {
 
-	/**
-	 * Converts the provided data into a {@link WP_REST_Response} by adding the required caching and content-type
-	 * headers.
-	 *
-	 * @param mixed $data The output data.
-	 *
-	 * @return WP_REST_Response The {@link WP_REST_Response}.
-	 */
-	public static function to_response( $data ) {
-
-		$response = new WP_REST_Response( $data );
-
-		$cache_in_seconds = 86400;
-		$date_timezone    = new DateTimeZone( 'GMT' );
-		$date_now         = new DateTime( 'now', $date_timezone );
-		$date_interval    = new DateInterval( "PT{$cache_in_seconds}S" );
-		$expires          = $date_now->add( $date_interval )->format( 'D, j M Y H:i:s T' );
-
-		$response->set_headers( array(
-			'Content-Type'  => 'application/ld+json; charset=' . get_option( 'blog_charset' ),
-			'Cache-Control' => "max-age=$cache_in_seconds",
-			'Expires'       => $expires
-		) );
-
-		return $response;
-	}
+    /**
+     * Converts the provided data into a {@link WP_REST_Response} by adding the required caching and content-type
+     * headers.
+     *
+     * @param mixed $data The output data.
+     *
+     * @return WP_REST_Response The {@link WP_REST_Response}.
+     */
+    public static function to_response( $data ) {
+
+        $response = new WP_REST_Response( $data );
+
+        $cache_in_seconds = 86400;
+        $date_timezone    = new DateTimeZone( 'GMT' );
+        $date_now         = new DateTime( 'now', $date_timezone );
+        $date_interval    = new DateInterval( "PT{$cache_in_seconds}S" );
+        $expires          = $date_now->add( $date_interval )->format( 'D, j M Y H:i:s T' );
+
+        $response->set_headers( array(
+            'Content-Type'  => 'application/ld+json; charset=' . get_option( 'blog_charset' ),
+            'Cache-Control' => "max-age=$cache_in_seconds",
+            'Expires'       => $expires
+        ) );
+
+        return $response;
+    }
 
 }

Spacing +9 added lines, -9 removed lines

@@ -25,21 +25,21 @@
 	 *
 	 * @return WP_REST_Response The {@link WP_REST_Response}.
 	 */
-	public static function to_response( $data ) {
+	public static function to_response($data) {
 
-		$response = new WP_REST_Response( $data );
+		$response = new WP_REST_Response($data);
 
 		$cache_in_seconds = 86400;
-		$date_timezone    = new DateTimeZone( 'GMT' );
-		$date_now         = new DateTime( 'now', $date_timezone );
-		$date_interval    = new DateInterval( "PT{$cache_in_seconds}S" );
-		$expires          = $date_now->add( $date_interval )->format( 'D, j M Y H:i:s T' );
+		$date_timezone    = new DateTimeZone('GMT');
+		$date_now         = new DateTime('now', $date_timezone);
+		$date_interval    = new DateInterval("PT{$cache_in_seconds}S");
+		$expires          = $date_now->add($date_interval)->format('D, j M Y H:i:s T');
 
-		$response->set_headers( array(
-			'Content-Type'  => 'application/ld+json; charset=' . get_option( 'blog_charset' ),
+		$response->set_headers(array(
+			'Content-Type'  => 'application/ld+json; charset='.get_option('blog_charset'),
 			'Cache-Control' => "max-age=$cache_in_seconds",
 			'Expires'       => $expires
-		) );
+		));
 
 		return $response;
 	}