insideout10 / wordlift-plugin

src/includes/class-wordlift-deactivator-feedback.php

Indentation +141 added lines, -141 removed lines

@@ -21,145 +21,145 @@
  */
 class Wordlift_Deactivator_Feedback {
 
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.19.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * Wordlift_Deactivator_Feedback constructor.
-	 *
-	 * @since 3.19.0
-	 */
-	public function __construct() {
-
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Deactivator_Feedback' );
-
-	}
-
-	/**
-	 * Checks whether we have permissions to show the popup.
-	 *
-	 * @return  bool `true` if we have permissions, false otherwise.
-	 * @version 3.19.0
-	 */
-	private function has_permission_to_show_popup() {
-		// Get the current page.
-		global $pagenow;
-
-		// Bail if the user doesn't have permissions
-		// or if it's not the plugins page.
-		if ( 'plugins.php' !== $pagenow ) {
-			return false;
-		}
-
-		// Get the user preferences. We shouldn't show the feedback popup
-		// if we don't have permissions for that.
-		$user_preferences = Wordlift_Configuration_Service::get_instance()->get_diagnostic_preferences();
-
-		// Bail. We don't have preferences to show the popup.
-		if ( 'yes' !== $user_preferences ) {
-			return false;
-		}
-
-		return true;
-	}
-
-	/**
-	 * Render the feedback popup in the footer.
-	 *
-	 * @return  void
-	 * @version 3.19.0
-	 */
-	public function render_feedback_popup() {
-		// Bail if we don't have permissions to show the popup.
-		if ( ! $this->has_permission_to_show_popup() ) {
-			return;
-		}
-		// Include the partial.
-		include plugin_dir_path( __FILE__ ) . '../admin/partials/wordlift-admin-deactivation-feedback-popup.php';
-	}
-
-	/**
-	 * Enqueue required popup scripts and styles.
-	 *
-	 * @return  void
-	 * @version 3.19.0
-	 */
-	public function enqueue_popup_scripts() {
-		// Bail if we don't have permissions to show the popup.
-		if ( ! $this->has_permission_to_show_popup() ) {
-			return;
-		}
-
-		wp_enqueue_style( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/css/wordlift-admin-feedback-popup.css', array(), WORDLIFT_VERSION );
-		wp_enqueue_script( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/js/wordlift-admin-feedback-popup.js', array( 'jquery' ), WORDLIFT_VERSION, false );
-	}
-
-	/**
-	 * Handle the deactivation ajax call
-	 * and perform a request to external server.
-	 *
-	 * @return  void
-	 * @version 3.19.0
-	 */
-	public function wl_deactivation_feedback() {
-		// Bail if the nonce is not valid.
-		if (
-			empty( $_POST['wl_deactivation_feedback_nonce'] ) || // The nonce doens't exists.
-			! wp_verify_nonce( sanitize_text_field( wp_unslash( $_POST['wl_deactivation_feedback_nonce'] ) ), 'wl_deactivation_feedback_nonce' ) // The nonce is invalid.
-		) {
-			wp_send_json_error( __( 'Nonce Security Check Failed!', 'wordlift' ) );
-		}
-
-		// We allow user to deactivate without providing a reason
-		// so bail and send success response.
-		if ( empty( $_POST['code'] ) ) {
-			wp_send_json_success();
-		}
-
-		$plugin_data = get_plugin_data( plugin_dir_path( __DIR__ ) . 'wordlift.php', false, false );
-
-		// Prepare the options.
-		$options = array(
-			// The deactivation reason.
-			'code'             => sanitize_text_field( wp_unslash( (string) $_POST['code'] ) ),
-			// Additional information if provided.
-			'details'          => ( ! empty( $_POST['details'] ) ) ? sanitize_text_field( wp_unslash( (string) $_POST['details'] ) ) : '',
-			// The website url.
-			'url'              => get_bloginfo( 'url' ),
-			// WP version.
-			'wordpressVersion' => get_bloginfo( 'version' ),
-			// WL version.
-			'wordliftVersion'  => $plugin_data['Version'],
-			// The admin email.
-			'email'            => get_bloginfo( 'admin_email' ),
-		);
-
-		$response = wp_remote_post(
-			Wordlift_Configuration_Service::get_instance()->get_deactivation_feedback_url(),
-			array(
-				'method'  => 'POST',
-				'body'    => wp_json_encode( $options ),
-				'headers' => array( 'Content-Type' => 'application/json; charset=utf-8' ),
-			)
-		);
-
-		$code    = wp_remote_retrieve_response_code( $response );
-		$message = wp_remote_retrieve_response_message( $response );
-
-		// Add message to the error log if the response code is not 200.
-		if ( 201 !== $code ) {
-			// Write the error in the logs.
-			$this->log->error( 'An error occurred while requesting a feedback endpoint error_code: ' . $code . ' message: ' . $message );
-		}
-
-		// We should send success message even when the feedback is not
-		// send, because otherwise the plugin cannot be deactivated.
-		wp_send_json_success();
-	}
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.19.0
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * Wordlift_Deactivator_Feedback constructor.
+     *
+     * @since 3.19.0
+     */
+    public function __construct() {
+
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Deactivator_Feedback' );
+
+    }
+
+    /**
+     * Checks whether we have permissions to show the popup.
+     *
+     * @return  bool `true` if we have permissions, false otherwise.
+     * @version 3.19.0
+     */
+    private function has_permission_to_show_popup() {
+        // Get the current page.
+        global $pagenow;
+
+        // Bail if the user doesn't have permissions
+        // or if it's not the plugins page.
+        if ( 'plugins.php' !== $pagenow ) {
+            return false;
+        }
+
+        // Get the user preferences. We shouldn't show the feedback popup
+        // if we don't have permissions for that.
+        $user_preferences = Wordlift_Configuration_Service::get_instance()->get_diagnostic_preferences();
+
+        // Bail. We don't have preferences to show the popup.
+        if ( 'yes' !== $user_preferences ) {
+            return false;
+        }
+
+        return true;
+    }
+
+    /**
+     * Render the feedback popup in the footer.
+     *
+     * @return  void
+     * @version 3.19.0
+     */
+    public function render_feedback_popup() {
+        // Bail if we don't have permissions to show the popup.
+        if ( ! $this->has_permission_to_show_popup() ) {
+            return;
+        }
+        // Include the partial.
+        include plugin_dir_path( __FILE__ ) . '../admin/partials/wordlift-admin-deactivation-feedback-popup.php';
+    }
+
+    /**
+     * Enqueue required popup scripts and styles.
+     *
+     * @return  void
+     * @version 3.19.0
+     */
+    public function enqueue_popup_scripts() {
+        // Bail if we don't have permissions to show the popup.
+        if ( ! $this->has_permission_to_show_popup() ) {
+            return;
+        }
+
+        wp_enqueue_style( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/css/wordlift-admin-feedback-popup.css', array(), WORDLIFT_VERSION );
+        wp_enqueue_script( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/js/wordlift-admin-feedback-popup.js', array( 'jquery' ), WORDLIFT_VERSION, false );
+    }
+
+    /**
+     * Handle the deactivation ajax call
+     * and perform a request to external server.
+     *
+     * @return  void
+     * @version 3.19.0
+     */
+    public function wl_deactivation_feedback() {
+        // Bail if the nonce is not valid.
+        if (
+            empty( $_POST['wl_deactivation_feedback_nonce'] ) || // The nonce doens't exists.
+            ! wp_verify_nonce( sanitize_text_field( wp_unslash( $_POST['wl_deactivation_feedback_nonce'] ) ), 'wl_deactivation_feedback_nonce' ) // The nonce is invalid.
+        ) {
+            wp_send_json_error( __( 'Nonce Security Check Failed!', 'wordlift' ) );
+        }
+
+        // We allow user to deactivate without providing a reason
+        // so bail and send success response.
+        if ( empty( $_POST['code'] ) ) {
+            wp_send_json_success();
+        }
+
+        $plugin_data = get_plugin_data( plugin_dir_path( __DIR__ ) . 'wordlift.php', false, false );
+
+        // Prepare the options.
+        $options = array(
+            // The deactivation reason.
+            'code'             => sanitize_text_field( wp_unslash( (string) $_POST['code'] ) ),
+            // Additional information if provided.
+            'details'          => ( ! empty( $_POST['details'] ) ) ? sanitize_text_field( wp_unslash( (string) $_POST['details'] ) ) : '',
+            // The website url.
+            'url'              => get_bloginfo( 'url' ),
+            // WP version.
+            'wordpressVersion' => get_bloginfo( 'version' ),
+            // WL version.
+            'wordliftVersion'  => $plugin_data['Version'],
+            // The admin email.
+            'email'            => get_bloginfo( 'admin_email' ),
+        );
+
+        $response = wp_remote_post(
+            Wordlift_Configuration_Service::get_instance()->get_deactivation_feedback_url(),
+            array(
+                'method'  => 'POST',
+                'body'    => wp_json_encode( $options ),
+                'headers' => array( 'Content-Type' => 'application/json; charset=utf-8' ),
+            )
+        );
+
+        $code    = wp_remote_retrieve_response_code( $response );
+        $message = wp_remote_retrieve_response_message( $response );
+
+        // Add message to the error log if the response code is not 200.
+        if ( 201 !== $code ) {
+            // Write the error in the logs.
+            $this->log->error( 'An error occurred while requesting a feedback endpoint error_code: ' . $code . ' message: ' . $message );
+        }
+
+        // We should send success message even when the feedback is not
+        // send, because otherwise the plugin cannot be deactivated.
+        wp_send_json_success();
+    }
 }

Spacing +24 added lines, -24 removed lines

@@ -37,7 +37,7 @@ 
 	 */
 	public function __construct() {
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Deactivator_Feedback' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Deactivator_Feedback');
 
 	}
 
@@ -53,7 +53,7 @@ 
 
 		// Bail if the user doesn't have permissions
 		// or if it's not the plugins page.
-		if ( 'plugins.php' !== $pagenow ) {
+		if ('plugins.php' !== $pagenow) {
 			return false;
 		}
 
@@ -62,7 +62,7 @@ 
 		$user_preferences = Wordlift_Configuration_Service::get_instance()->get_diagnostic_preferences();
 
 		// Bail. We don't have preferences to show the popup.
-		if ( 'yes' !== $user_preferences ) {
+		if ('yes' !== $user_preferences) {
 			return false;
 		}
 
@@ -77,11 +77,11 @@ 
 	 */
 	public function render_feedback_popup() {
 		// Bail if we don't have permissions to show the popup.
-		if ( ! $this->has_permission_to_show_popup() ) {
+		if ( ! $this->has_permission_to_show_popup()) {
 			return;
 		}
 		// Include the partial.
-		include plugin_dir_path( __FILE__ ) . '../admin/partials/wordlift-admin-deactivation-feedback-popup.php';
+		include plugin_dir_path(__FILE__).'../admin/partials/wordlift-admin-deactivation-feedback-popup.php';
 	}
 
 	/**
@@ -92,12 +92,12 @@ 
 	 */
 	public function enqueue_popup_scripts() {
 		// Bail if we don't have permissions to show the popup.
-		if ( ! $this->has_permission_to_show_popup() ) {
+		if ( ! $this->has_permission_to_show_popup()) {
 			return;
 		}
 
-		wp_enqueue_style( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/css/wordlift-admin-feedback-popup.css', array(), WORDLIFT_VERSION );
-		wp_enqueue_script( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/js/wordlift-admin-feedback-popup.js', array( 'jquery' ), WORDLIFT_VERSION, false );
+		wp_enqueue_style('wordlift-admin-feedback-popup', plugin_dir_url(__DIR__).'admin/css/wordlift-admin-feedback-popup.css', array(), WORDLIFT_VERSION);
+		wp_enqueue_script('wordlift-admin-feedback-popup', plugin_dir_url(__DIR__).'admin/js/wordlift-admin-feedback-popup.js', array('jquery'), WORDLIFT_VERSION, false);
 	}
 
 	/**
@@ -110,52 +110,52 @@ 
 	public function wl_deactivation_feedback() {
 		// Bail if the nonce is not valid.
 		if (
-			empty( $_POST['wl_deactivation_feedback_nonce'] ) || // The nonce doens't exists.
-			! wp_verify_nonce( sanitize_text_field( wp_unslash( $_POST['wl_deactivation_feedback_nonce'] ) ), 'wl_deactivation_feedback_nonce' ) // The nonce is invalid.
+			empty($_POST['wl_deactivation_feedback_nonce']) || // The nonce doens't exists.
+			! wp_verify_nonce(sanitize_text_field(wp_unslash($_POST['wl_deactivation_feedback_nonce'])), 'wl_deactivation_feedback_nonce') // The nonce is invalid.
 		) {
-			wp_send_json_error( __( 'Nonce Security Check Failed!', 'wordlift' ) );
+			wp_send_json_error(__('Nonce Security Check Failed!', 'wordlift'));
 		}
 
 		// We allow user to deactivate without providing a reason
 		// so bail and send success response.
-		if ( empty( $_POST['code'] ) ) {
+		if (empty($_POST['code'])) {
 			wp_send_json_success();
 		}
 
-		$plugin_data = get_plugin_data( plugin_dir_path( __DIR__ ) . 'wordlift.php', false, false );
+		$plugin_data = get_plugin_data(plugin_dir_path(__DIR__).'wordlift.php', false, false);
 
 		// Prepare the options.
 		$options = array(
 			// The deactivation reason.
-			'code'             => sanitize_text_field( wp_unslash( (string) $_POST['code'] ) ),
+			'code'             => sanitize_text_field(wp_unslash((string) $_POST['code'])),
 			// Additional information if provided.
-			'details'          => ( ! empty( $_POST['details'] ) ) ? sanitize_text_field( wp_unslash( (string) $_POST['details'] ) ) : '',
+			'details'          => ( ! empty($_POST['details'])) ? sanitize_text_field(wp_unslash((string) $_POST['details'])) : '',
 			// The website url.
-			'url'              => get_bloginfo( 'url' ),
+			'url'              => get_bloginfo('url'),
 			// WP version.
-			'wordpressVersion' => get_bloginfo( 'version' ),
+			'wordpressVersion' => get_bloginfo('version'),
 			// WL version.
 			'wordliftVersion'  => $plugin_data['Version'],
 			// The admin email.
-			'email'            => get_bloginfo( 'admin_email' ),
+			'email'            => get_bloginfo('admin_email'),
 		);
 
 		$response = wp_remote_post(
 			Wordlift_Configuration_Service::get_instance()->get_deactivation_feedback_url(),
 			array(
 				'method'  => 'POST',
-				'body'    => wp_json_encode( $options ),
-				'headers' => array( 'Content-Type' => 'application/json; charset=utf-8' ),
+				'body'    => wp_json_encode($options),
+				'headers' => array('Content-Type' => 'application/json; charset=utf-8'),
 			)
 		);
 
-		$code    = wp_remote_retrieve_response_code( $response );
-		$message = wp_remote_retrieve_response_message( $response );
+		$code    = wp_remote_retrieve_response_code($response);
+		$message = wp_remote_retrieve_response_message($response);
 
 		// Add message to the error log if the response code is not 200.
-		if ( 201 !== $code ) {
+		if (201 !== $code) {
 			// Write the error in the logs.
-			$this->log->error( 'An error occurred while requesting a feedback endpoint error_code: ' . $code . ' message: ' . $message );
+			$this->log->error('An error occurred while requesting a feedback endpoint error_code: '.$code.' message: '.$message);
 		}
 
 		// We should send success message even when the feedback is not

src/includes/class-wordlift-loader.php

Indentation +102 added lines, -102 removed lines

@@ -23,107 +23,107 @@
  */
 class Wordlift_Loader {
 
-	/**
-	 * The array of actions registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @access   protected
-	 * @var      array    $actions    The actions registered with WordPress to fire when the plugin loads.
-	 */
-	protected $actions;
-
-	/**
-	 * The array of filters registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @access   protected
-	 * @var      array    $filters    The filters registered with WordPress to fire when the plugin loads.
-	 */
-	protected $filters;
-
-	/**
-	 * Initialize the collections used to maintain the actions and filters.
-	 *
-	 * @since    1.0.0
-	 */
-	public function __construct() {
-
-		$this->actions = array();
-		$this->filters = array();
-
-	}
-
-	/**
-	 * Add a new action to the collection to be registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @param    string $hook             The name of the WordPress action that is being registered.
-	 * @param    object $component        A reference to the instance of the object on which the action is defined.
-	 * @param    string $callback         The name of the function definition on the $component.
-	 * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
-	 * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1.
-	 */
-	public function add_action( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		add_action( $hook, array( $component, $callback ), $priority, $accepted_args );
-	}
-
-	/**
-	 * Add a new filter to the collection to be registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @param    string $hook             The name of the WordPress filter that is being registered.
-	 * @param    object $component        A reference to the instance of the object on which the filter is defined.
-	 * @param    string $callback         The name of the function definition on the $component.
-	 * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
-	 * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1
-	 */
-	public function add_filter( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		add_filter( $hook, array( $component, $callback ), $priority, $accepted_args );
-	}
-
-	/**
-	 * A utility function that is used to register the actions and hooks into a single
-	 * collection.
-	 *
-	 * @since    1.0.0
-	 * @access   private
-	 * @param    array  $hooks            The collection of hooks that is being registered (that is, actions or filters).
-	 * @param    string $hook             The name of the WordPress filter that is being registered.
-	 * @param    object $component        A reference to the instance of the object on which the filter is defined.
-	 * @param    string $callback         The name of the function definition on the $component.
-	 * @param    int    $priority         The priority at which the function should be fired.
-	 * @param    int    $accepted_args    The number of arguments that should be passed to the $callback.
-	 * @return   array                                  The collection of actions and filters registered with WordPress.
-	 */
-	private function add( $hooks, $hook, $component, $callback, $priority, $accepted_args ) {
-
-		$hooks[] = array(
-			'hook'          => $hook,
-			'component'     => $component,
-			'callback'      => $callback,
-			'priority'      => $priority,
-			'accepted_args' => $accepted_args,
-		);
-
-		return $hooks;
-
-	}
-
-	/**
-	 * Register the filters and actions with WordPress.
-	 *
-	 * @since    1.0.0
-	 */
-	public function run() {
-
-		foreach ( $this->filters as $hook ) {
-			add_filter( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
-		}
-
-		foreach ( $this->actions as $hook ) {
-			add_action( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
-		}
-
-	}
+    /**
+     * The array of actions registered with WordPress.
+     *
+     * @since    1.0.0
+     * @access   protected
+     * @var      array    $actions    The actions registered with WordPress to fire when the plugin loads.
+     */
+    protected $actions;
+
+    /**
+     * The array of filters registered with WordPress.
+     *
+     * @since    1.0.0
+     * @access   protected
+     * @var      array    $filters    The filters registered with WordPress to fire when the plugin loads.
+     */
+    protected $filters;
+
+    /**
+     * Initialize the collections used to maintain the actions and filters.
+     *
+     * @since    1.0.0
+     */
+    public function __construct() {
+
+        $this->actions = array();
+        $this->filters = array();
+
+    }
+
+    /**
+     * Add a new action to the collection to be registered with WordPress.
+     *
+     * @since    1.0.0
+     * @param    string $hook             The name of the WordPress action that is being registered.
+     * @param    object $component        A reference to the instance of the object on which the action is defined.
+     * @param    string $callback         The name of the function definition on the $component.
+     * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
+     * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1.
+     */
+    public function add_action( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
+        add_action( $hook, array( $component, $callback ), $priority, $accepted_args );
+    }
+
+    /**
+     * Add a new filter to the collection to be registered with WordPress.
+     *
+     * @since    1.0.0
+     * @param    string $hook             The name of the WordPress filter that is being registered.
+     * @param    object $component        A reference to the instance of the object on which the filter is defined.
+     * @param    string $callback         The name of the function definition on the $component.
+     * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
+     * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1
+     */
+    public function add_filter( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
+        add_filter( $hook, array( $component, $callback ), $priority, $accepted_args );
+    }
+
+    /**
+     * A utility function that is used to register the actions and hooks into a single
+     * collection.
+     *
+     * @since    1.0.0
+     * @access   private
+     * @param    array  $hooks            The collection of hooks that is being registered (that is, actions or filters).
+     * @param    string $hook             The name of the WordPress filter that is being registered.
+     * @param    object $component        A reference to the instance of the object on which the filter is defined.
+     * @param    string $callback         The name of the function definition on the $component.
+     * @param    int    $priority         The priority at which the function should be fired.
+     * @param    int    $accepted_args    The number of arguments that should be passed to the $callback.
+     * @return   array                                  The collection of actions and filters registered with WordPress.
+     */
+    private function add( $hooks, $hook, $component, $callback, $priority, $accepted_args ) {
+
+        $hooks[] = array(
+            'hook'          => $hook,
+            'component'     => $component,
+            'callback'      => $callback,
+            'priority'      => $priority,
+            'accepted_args' => $accepted_args,
+        );
+
+        return $hooks;
+
+    }
+
+    /**
+     * Register the filters and actions with WordPress.
+     *
+     * @since    1.0.0
+     */
+    public function run() {
+
+        foreach ( $this->filters as $hook ) {
+            add_filter( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+        }
+
+        foreach ( $this->actions as $hook ) {
+            add_action( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+        }
+
+    }
 
 }

Spacing +9 added lines, -9 removed lines

@@ -63,8 +63,8 @@ 
 	 * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
 	 * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1.
 	 */
-	public function add_action( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		add_action( $hook, array( $component, $callback ), $priority, $accepted_args );
+	public function add_action($hook, $component, $callback, $priority = 10, $accepted_args = 1) {
+		add_action($hook, array($component, $callback), $priority, $accepted_args);
 	}
 
 	/**
@@ -77,8 +77,8 @@ 
 	 * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
 	 * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1
 	 */
-	public function add_filter( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		add_filter( $hook, array( $component, $callback ), $priority, $accepted_args );
+	public function add_filter($hook, $component, $callback, $priority = 10, $accepted_args = 1) {
+		add_filter($hook, array($component, $callback), $priority, $accepted_args);
 	}
 
 	/**
@@ -95,7 +95,7 @@ 
 	 * @param    int    $accepted_args    The number of arguments that should be passed to the $callback.
 	 * @return   array                                  The collection of actions and filters registered with WordPress.
 	 */
-	private function add( $hooks, $hook, $component, $callback, $priority, $accepted_args ) {
+	private function add($hooks, $hook, $component, $callback, $priority, $accepted_args) {
 
 		$hooks[] = array(
 			'hook'          => $hook,
@@ -116,12 +116,12 @@ 
 	 */
 	public function run() {
 
-		foreach ( $this->filters as $hook ) {
-			add_filter( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+		foreach ($this->filters as $hook) {
+			add_filter($hook['hook'], array($hook['component'], $hook['callback']), $hook['priority'], $hook['accepted_args']);
 		}
 
-		foreach ( $this->actions as $hook ) {
-			add_action( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+		foreach ($this->actions as $hook) {
+			add_action($hook['hook'], array($hook['component'], $hook['callback']), $hook['priority'], $hook['accepted_args']);
 		}
 
 	}

src/includes/class-wordlift-api-service.php

Indentation +187 added lines, -187 removed lines

@@ -17,202 +17,202 @@
  */
 class Wordlift_Api_Service {
 
-	/**
-	 * The {@link Wordlift_Api_Service} singleton instance.
-	 *
-	 * @since 3.20.0
-	 * @access private
-	 * @var \Wordlift_Api_Service $instance The singleton instance.
-	 */
-	private static $instance;
-
-	/**
-	 * @var Api_Headers_Service|null
-	 */
-	private $headers_service;
-
-	/**
-	 * Create a {@link Wordlift_Api_Service} instance.
-	 *
-	 * @since 3.20.0
-	 */
-	public function __construct() {
-		self::$instance        = $this;
-		$this->headers_service = Api_Headers_Service::get_instance();
-	}
-
-	/**
-	 * Get the {@link Wordlift_Api_Service} singleton instance.
-	 *
-	 * @return \Wordlift_Api_Service The {@link Wordlift_Api_Service} singleton instance.
-	 * @since 3.20.0
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Perform a `GET` request towards the requested path.
-	 *
-	 * @param string $path The relative path.
-	 *
-	 * @return array|WP_Error
-	 * @since 3.20.0
-	 */
-	public function get( $path ) {
-
-		// Prepare the target URL.
-		$url = Wordlift_Configuration_Service::get_instance()->get_api_url() . $path;
-
-		// Get the response value.
-		$response = wp_remote_get(
-			$url,
-			array(
-				'user-agent' => User_Agent::get_user_agent(),
-				'headers'    => array(
-					'X-Authorization' => Wordlift_Configuration_Service::get_instance()->get_key(),
-				) + $this->headers_service->get_wp_headers(),
-				/*
+    /**
+     * The {@link Wordlift_Api_Service} singleton instance.
+     *
+     * @since 3.20.0
+     * @access private
+     * @var \Wordlift_Api_Service $instance The singleton instance.
+     */
+    private static $instance;
+
+    /**
+     * @var Api_Headers_Service|null
+     */
+    private $headers_service;
+
+    /**
+     * Create a {@link Wordlift_Api_Service} instance.
+     *
+     * @since 3.20.0
+     */
+    public function __construct() {
+        self::$instance        = $this;
+        $this->headers_service = Api_Headers_Service::get_instance();
+    }
+
+    /**
+     * Get the {@link Wordlift_Api_Service} singleton instance.
+     *
+     * @return \Wordlift_Api_Service The {@link Wordlift_Api_Service} singleton instance.
+     * @since 3.20.0
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Perform a `GET` request towards the requested path.
+     *
+     * @param string $path The relative path.
+     *
+     * @return array|WP_Error
+     * @since 3.20.0
+     */
+    public function get( $path ) {
+
+        // Prepare the target URL.
+        $url = Wordlift_Configuration_Service::get_instance()->get_api_url() . $path;
+
+        // Get the response value.
+        $response = wp_remote_get(
+            $url,
+            array(
+                'user-agent' => User_Agent::get_user_agent(),
+                'headers'    => array(
+                    'X-Authorization' => Wordlift_Configuration_Service::get_instance()->get_key(),
+                ) + $this->headers_service->get_wp_headers(),
+                /*
 				* Increase the timeout from the default of 5 to 30 secs.
 				*
 				* @see https://github.com/insideout10/wordlift-plugin/issues/906
 				*
 				* @since 3.20.1
 				*/
-				'timeout'    => 30,
-			)
-		);
-
-		return self::get_message_or_error( $response );
-	}
-
-	/**
-	 * Perform a `POST` request towards the requested path.
-	 *
-	 * @param string       $path The relative path.
-	 * @param array|object $body The request body (will be serialized to JSON).
-	 *
-	 * @return array|WP_Error
-	 * @since 3.20.0
-	 */
-	public function post( $path, $body ) {
-
-		return $this->post_custom_content_type(
-			$path,
-			wp_json_encode( $body ),
-			'application/json; ' . get_bloginfo( 'charset' )
-		);
-	}
-
-	public function post_custom_content_type( $path, $body, $content_type ) {
-
-		// Prepare the target URL.
-		$url = apply_filters( 'wl_api_service_api_url_path', Wordlift_Configuration_Service::get_instance()->get_api_url() . $path );
-
-		// Give some time for the operation to complete, more than the time we give to the HTTP operation to complete.
-		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@set_time_limit( 90 );
-
-		// Get the response value.
-		$key      = Wordlift_Configuration_Service::get_instance()->get_key();
-		$response = wp_remote_post(
-			$url,
-			array(
-				'timeout'    => 60,
-				'user-agent' => User_Agent::get_user_agent(),
-				'headers'    => array(
-					'Content-Type'    => $content_type,
-					'X-Authorization' => $key,
-					'Authorization'   => "Key $key",
-					/*
+                'timeout'    => 30,
+            )
+        );
+
+        return self::get_message_or_error( $response );
+    }
+
+    /**
+     * Perform a `POST` request towards the requested path.
+     *
+     * @param string       $path The relative path.
+     * @param array|object $body The request body (will be serialized to JSON).
+     *
+     * @return array|WP_Error
+     * @since 3.20.0
+     */
+    public function post( $path, $body ) {
+
+        return $this->post_custom_content_type(
+            $path,
+            wp_json_encode( $body ),
+            'application/json; ' . get_bloginfo( 'charset' )
+        );
+    }
+
+    public function post_custom_content_type( $path, $body, $content_type ) {
+
+        // Prepare the target URL.
+        $url = apply_filters( 'wl_api_service_api_url_path', Wordlift_Configuration_Service::get_instance()->get_api_url() . $path );
+
+        // Give some time for the operation to complete, more than the time we give to the HTTP operation to complete.
+        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
+        @set_time_limit( 90 );
+
+        // Get the response value.
+        $key      = Wordlift_Configuration_Service::get_instance()->get_key();
+        $response = wp_remote_post(
+            $url,
+            array(
+                'timeout'    => 60,
+                'user-agent' => User_Agent::get_user_agent(),
+                'headers'    => array(
+                    'Content-Type'    => $content_type,
+                    'X-Authorization' => $key,
+                    'Authorization'   => "Key $key",
+                    /*
 					 * This is required to avoid CURL receiving 502 Bad Gateway errors.
 					 *
 					 * @see https://stackoverflow.com/questions/30601075/curl-to-google-compute-load-balancer-gets-error-502
 					 */
-					'Expect'          => '',
-				) + $this->headers_service->get_wp_headers(),
-				'body'       => $body,
-			)
-		);
-
-		return self::get_message_or_error( $response );
-	}
-
-	public function delete( $path ) {
-
-		// Prepare the target URL.
-		$url = Wordlift_Configuration_Service::get_instance()->get_api_url() . $path;
-
-		// Get the response value.
-		$response = wp_remote_request(
-			$url,
-			array(
-				'method'     => 'DELETE',
-				'user-agent' => User_Agent::get_user_agent(),
-				'headers'    => array(
-					'X-Authorization' => Wordlift_Configuration_Service::get_instance()->get_key(),
-				) + $this->headers_service->get_wp_headers(),
-			)
-		);
-
-		return self::get_message_or_error( $response );
-	}
-
-	/**
-	 * Return the {@link WP_Error} in case of error or the actual reply if successful.
-	 *
-	 * @param array|WP_Error $response The response of an http call.
-	 *
-	 * @return string|object|WP_Error A {@link WP_Error} instance or the actual response content.
-	 * @since 3.20.0
-	 */
-	private static function get_message_or_error( $response ) {
-
-		// Result is WP_Error.
-		if ( is_wp_error( $response ) ) {
-			return $response;
-		}
-
-		// `code` not set or not numeric.
-		$code = wp_remote_retrieve_response_code( $response );
-		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		$message = @wp_remote_retrieve_response_message( $response );
-
-		if ( empty( $code ) || ! is_numeric( $code ) ) {
-			return new WP_Error( 0, $message );
-		}
-
-		// Code not 2xx.
-		if ( 2 !== intval( $code / 100 ) ) {
-			return new WP_Error( $code, $message );
-		}
-
-		// Everything's fine, return the message.
-		return self::try_json_decode( $response );
-	}
-
-	/**
-	 * Try to decode the json response
-	 *
-	 * @param array $response The response array.
-	 *
-	 * @return array|mixed|object The decoded response or the original response body.
-	 * @since 3.20.0
-	 */
-	private static function try_json_decode( $response ) {
-
-		// Get the headers.
-		$content_type = wp_remote_retrieve_header( $response, 'content-type' );
-		$body         = wp_remote_retrieve_body( $response );
-
-		// If it's not an `application/json` return the plain response body.
-		if ( 0 !== strpos( strtolower( $content_type ), 'application/json' ) ) {
-			return $body;
-		}
-
-		// Decode and return the structured result.
-		return json_decode( $body );
-	}
+                    'Expect'          => '',
+                ) + $this->headers_service->get_wp_headers(),
+                'body'       => $body,
+            )
+        );
+
+        return self::get_message_or_error( $response );
+    }
+
+    public function delete( $path ) {
+
+        // Prepare the target URL.
+        $url = Wordlift_Configuration_Service::get_instance()->get_api_url() . $path;
+
+        // Get the response value.
+        $response = wp_remote_request(
+            $url,
+            array(
+                'method'     => 'DELETE',
+                'user-agent' => User_Agent::get_user_agent(),
+                'headers'    => array(
+                    'X-Authorization' => Wordlift_Configuration_Service::get_instance()->get_key(),
+                ) + $this->headers_service->get_wp_headers(),
+            )
+        );
+
+        return self::get_message_or_error( $response );
+    }
+
+    /**
+     * Return the {@link WP_Error} in case of error or the actual reply if successful.
+     *
+     * @param array|WP_Error $response The response of an http call.
+     *
+     * @return string|object|WP_Error A {@link WP_Error} instance or the actual response content.
+     * @since 3.20.0
+     */
+    private static function get_message_or_error( $response ) {
+
+        // Result is WP_Error.
+        if ( is_wp_error( $response ) ) {
+            return $response;
+        }
+
+        // `code` not set or not numeric.
+        $code = wp_remote_retrieve_response_code( $response );
+        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
+        $message = @wp_remote_retrieve_response_message( $response );
+
+        if ( empty( $code ) || ! is_numeric( $code ) ) {
+            return new WP_Error( 0, $message );
+        }
+
+        // Code not 2xx.
+        if ( 2 !== intval( $code / 100 ) ) {
+            return new WP_Error( $code, $message );
+        }
+
+        // Everything's fine, return the message.
+        return self::try_json_decode( $response );
+    }
+
+    /**
+     * Try to decode the json response
+     *
+     * @param array $response The response array.
+     *
+     * @return array|mixed|object The decoded response or the original response body.
+     * @since 3.20.0
+     */
+    private static function try_json_decode( $response ) {
+
+        // Get the headers.
+        $content_type = wp_remote_retrieve_header( $response, 'content-type' );
+        $body         = wp_remote_retrieve_body( $response );
+
+        // If it's not an `application/json` return the plain response body.
+        if ( 0 !== strpos( strtolower( $content_type ), 'application/json' ) ) {
+            return $body;
+        }
+
+        // Decode and return the structured result.
+        return json_decode( $body );
+    }
 
 }

Spacing +27 added lines, -27 removed lines

@@ -60,10 +60,10 @@ 
 	 * @return array|WP_Error
 	 * @since 3.20.0
 	 */
-	public function get( $path ) {
+	public function get($path) {
 
 		// Prepare the target URL.
-		$url = Wordlift_Configuration_Service::get_instance()->get_api_url() . $path;
+		$url = Wordlift_Configuration_Service::get_instance()->get_api_url().$path;
 
 		// Get the response value.
 		$response = wp_remote_get(
@@ -84,7 +84,7 @@ 
 			)
 		);
 
-		return self::get_message_or_error( $response );
+		return self::get_message_or_error($response);
 	}
 
 	/**
@@ -96,23 +96,23 @@ 
 	 * @return array|WP_Error
 	 * @since 3.20.0
 	 */
-	public function post( $path, $body ) {
+	public function post($path, $body) {
 
 		return $this->post_custom_content_type(
 			$path,
-			wp_json_encode( $body ),
-			'application/json; ' . get_bloginfo( 'charset' )
+			wp_json_encode($body),
+			'application/json; '.get_bloginfo('charset')
 		);
 	}
 
-	public function post_custom_content_type( $path, $body, $content_type ) {
+	public function post_custom_content_type($path, $body, $content_type) {
 
 		// Prepare the target URL.
-		$url = apply_filters( 'wl_api_service_api_url_path', Wordlift_Configuration_Service::get_instance()->get_api_url() . $path );
+		$url = apply_filters('wl_api_service_api_url_path', Wordlift_Configuration_Service::get_instance()->get_api_url().$path);
 
 		// Give some time for the operation to complete, more than the time we give to the HTTP operation to complete.
 		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@set_time_limit( 90 );
+		@set_time_limit(90);
 
 		// Get the response value.
 		$key      = Wordlift_Configuration_Service::get_instance()->get_key();
@@ -136,13 +136,13 @@ 
 			)
 		);
 
-		return self::get_message_or_error( $response );
+		return self::get_message_or_error($response);
 	}
 
-	public function delete( $path ) {
+	public function delete($path) {
 
 		// Prepare the target URL.
-		$url = Wordlift_Configuration_Service::get_instance()->get_api_url() . $path;
+		$url = Wordlift_Configuration_Service::get_instance()->get_api_url().$path;
 
 		// Get the response value.
 		$response = wp_remote_request(
@@ -156,7 +156,7 @@ 
 			)
 		);
 
-		return self::get_message_or_error( $response );
+		return self::get_message_or_error($response);
 	}
 
 	/**
@@ -167,29 +167,29 @@ 
 	 * @return string|object|WP_Error A {@link WP_Error} instance or the actual response content.
 	 * @since 3.20.0
 	 */
-	private static function get_message_or_error( $response ) {
+	private static function get_message_or_error($response) {
 
 		// Result is WP_Error.
-		if ( is_wp_error( $response ) ) {
+		if (is_wp_error($response)) {
 			return $response;
 		}
 
 		// `code` not set or not numeric.
-		$code = wp_remote_retrieve_response_code( $response );
+		$code = wp_remote_retrieve_response_code($response);
 		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		$message = @wp_remote_retrieve_response_message( $response );
+		$message = @wp_remote_retrieve_response_message($response);
 
-		if ( empty( $code ) || ! is_numeric( $code ) ) {
-			return new WP_Error( 0, $message );
+		if (empty($code) || ! is_numeric($code)) {
+			return new WP_Error(0, $message);
 		}
 
 		// Code not 2xx.
-		if ( 2 !== intval( $code / 100 ) ) {
-			return new WP_Error( $code, $message );
+		if (2 !== intval($code / 100)) {
+			return new WP_Error($code, $message);
 		}
 
 		// Everything's fine, return the message.
-		return self::try_json_decode( $response );
+		return self::try_json_decode($response);
 	}
 
 	/**
@@ -200,19 +200,19 @@ 
 	 * @return array|mixed|object The decoded response or the original response body.
 	 * @since 3.20.0
 	 */
-	private static function try_json_decode( $response ) {
+	private static function try_json_decode($response) {
 
 		// Get the headers.
-		$content_type = wp_remote_retrieve_header( $response, 'content-type' );
-		$body         = wp_remote_retrieve_body( $response );
+		$content_type = wp_remote_retrieve_header($response, 'content-type');
+		$body         = wp_remote_retrieve_body($response);
 
 		// If it's not an `application/json` return the plain response body.
-		if ( 0 !== strpos( strtolower( $content_type ), 'application/json' ) ) {
+		if (0 !== strpos(strtolower($content_type), 'application/json')) {
 			return $body;
 		}
 
 		// Decode and return the structured result.
-		return json_decode( $body );
+		return json_decode($body);
 	}
 
 }

src/includes/class-wordlift-google-analytics-export-service.php

Indentation +135 added lines, -135 removed lines

@@ -18,103 +18,103 @@ 
  */
 class Wordlift_Google_Analytics_Export_Service {
 
-	/**
-	 * Export the site data that could be imported in Google Analytics.
-	 * It will works only when permalink structure is set to "Postname".
-	 *
-	 * @since 3.16.0
-	 *
-	 * @return void
-	 */
-	public function export() {
-		// Bail if the permalink structure is different from "Post name".
-		if ( ! $this->is_postname_permalink_structure() ) {
-			wp_die( 'The current permalink structure do not allow to export your data. Please change the permalink structure to "Post name".' );
-		}
-
-		// Output the file data. @codingStandardsIgnoreLine
-		@ob_end_clean();
-
-		// Generate unique filename using current timestamp.
-		$filename = 'wl-ga-export-' . gmdate( 'Y-m-d-H-i-s' ) . '.csv';
-
-		// Add proper file headers.
-		header( "Content-Disposition: attachment; filename=$filename" );
-		header( 'Content-Type: text/csv; charset=' . get_bloginfo( 'charset' ) );
-
-		// Do not cache the file.
-		header( 'Pragma: no-cache' );
-		header( 'Expires: 0' );
-
-		// Build the CSV file.
-		$this->create_csv();
-
-		wp_die();
-	}
-
-	/**
-	 * Return site path. Some installations are in subdirectories
-	 * and we need add them to expported permalinks.
-	 *
-	 * @since 3.16.0
-	 *
-	 * @return string The site path.
-	 */
-	public function get_site_path() {
-		// Get home url from database.
-		$home_url = home_url( '/' );
-
-		// Parse the url.
-		$parsed = wp_parse_url( $home_url );
-
-		// Return the path.
-		return $parsed['path'];
-	}
-
-	/**
-	 * Check if the current permalink structure is set to "Post name".
-	 *
-	 * @since 3.16.0
-	 *
-	 * @return bool whether the structure is "Post name" or not.
-	 */
-	public static function is_postname_permalink_structure() {
-		// Get current permalink structure.
-		$structure = get_option( 'permalink_structure' );
-
-		// The regular expression. It will check if the site structure contains postname.
-		$regex = '~^/\%postname\%/$~';
-
-		// Check if the site structure match the rquired one.
-		preg_match( $regex, $structure, $matches );
-
-		// Bail if the site have different structure.
-		if ( empty( $matches ) ) {
-			return false;
-		}
-
-		return true;
-	}
-
-	/**
-	 * Generate array data, that should be exported as csv.
-	 * The data contains the post/page title, entity name and type.
-	 *
-	 * @since 3.16.0
-	 *
-	 * @return array $items Content data.
-	 */
-	public function get_content_data() {
-		// Get the global $wpdb.
-		global $wpdb;
-
-		// Site path (optional).
-		$path = $this->get_site_path();
-
-		// Get the data.
-		$items = $wpdb->get_results(
-			$wpdb->prepare(
-				"SELECT
+    /**
+     * Export the site data that could be imported in Google Analytics.
+     * It will works only when permalink structure is set to "Postname".
+     *
+     * @since 3.16.0
+     *
+     * @return void
+     */
+    public function export() {
+        // Bail if the permalink structure is different from "Post name".
+        if ( ! $this->is_postname_permalink_structure() ) {
+            wp_die( 'The current permalink structure do not allow to export your data. Please change the permalink structure to "Post name".' );
+        }
+
+        // Output the file data. @codingStandardsIgnoreLine
+        @ob_end_clean();
+
+        // Generate unique filename using current timestamp.
+        $filename = 'wl-ga-export-' . gmdate( 'Y-m-d-H-i-s' ) . '.csv';
+
+        // Add proper file headers.
+        header( "Content-Disposition: attachment; filename=$filename" );
+        header( 'Content-Type: text/csv; charset=' . get_bloginfo( 'charset' ) );
+
+        // Do not cache the file.
+        header( 'Pragma: no-cache' );
+        header( 'Expires: 0' );
+
+        // Build the CSV file.
+        $this->create_csv();
+
+        wp_die();
+    }
+
+    /**
+     * Return site path. Some installations are in subdirectories
+     * and we need add them to expported permalinks.
+     *
+     * @since 3.16.0
+     *
+     * @return string The site path.
+     */
+    public function get_site_path() {
+        // Get home url from database.
+        $home_url = home_url( '/' );
+
+        // Parse the url.
+        $parsed = wp_parse_url( $home_url );
+
+        // Return the path.
+        return $parsed['path'];
+    }
+
+    /**
+     * Check if the current permalink structure is set to "Post name".
+     *
+     * @since 3.16.0
+     *
+     * @return bool whether the structure is "Post name" or not.
+     */
+    public static function is_postname_permalink_structure() {
+        // Get current permalink structure.
+        $structure = get_option( 'permalink_structure' );
+
+        // The regular expression. It will check if the site structure contains postname.
+        $regex = '~^/\%postname\%/$~';
+
+        // Check if the site structure match the rquired one.
+        preg_match( $regex, $structure, $matches );
+
+        // Bail if the site have different structure.
+        if ( empty( $matches ) ) {
+            return false;
+        }
+
+        return true;
+    }
+
+    /**
+     * Generate array data, that should be exported as csv.
+     * The data contains the post/page title, entity name and type.
+     *
+     * @since 3.16.0
+     *
+     * @return array $items Content data.
+     */
+    public function get_content_data() {
+        // Get the global $wpdb.
+        global $wpdb;
+
+        // Site path (optional).
+        $path = $this->get_site_path();
+
+        // Get the data.
+        $items = $wpdb->get_results(
+            $wpdb->prepare(
+                "SELECT
 					CONCAT( %s, p.post_name, '/' ) AS 'post_name',
 					p1.post_name AS 'entity_name',
 					t.slug AS 'entity_type'
@@ -131,43 +131,43 @@ 
 					INNER JOIN {$wpdb->prefix}terms t
 						ON t.term_id = tt.term_id
 					WHERE p.post_type IN ( 'page', 'post' );",
-				$path
-			)
-		); // db call ok; no-cache ok.
-
-		return $items;
-	}
-
-	/**
-	 * Create the CSV file that will be downloaded.
-	 *
-	 * @since 3.16.0
-	 *
-	 * @return void
-	 */
-	public function create_csv() {
-		// Create a file pointer connected to the output stream.
-		// Ignoring linter notices below that complain about file output which do not actually happen.
-		$file = fopen( 'php://output', 'w' );
-
-		// Add the column headers. @codingStandardsIgnoreLine
-		fputcsv(
-			$file,
-			array(
-				'ga:pagePath',
-				'ga:dimension1',
-				'ga:dimension2',
-			)
-		);
-
-		// Cycle through items and add each item data to the file.
-		foreach ( $this->get_content_data() as $row ) {
-			// Add new line in the file. @codingStandardsIgnoreLine
-			fputcsv(
-				$file,
-				(array) $row // convert the object to array.
-			);
-		}
-	}
+                $path
+            )
+        ); // db call ok; no-cache ok.
+
+        return $items;
+    }
+
+    /**
+     * Create the CSV file that will be downloaded.
+     *
+     * @since 3.16.0
+     *
+     * @return void
+     */
+    public function create_csv() {
+        // Create a file pointer connected to the output stream.
+        // Ignoring linter notices below that complain about file output which do not actually happen.
+        $file = fopen( 'php://output', 'w' );
+
+        // Add the column headers. @codingStandardsIgnoreLine
+        fputcsv(
+            $file,
+            array(
+                'ga:pagePath',
+                'ga:dimension1',
+                'ga:dimension2',
+            )
+        );
+
+        // Cycle through items and add each item data to the file.
+        foreach ( $this->get_content_data() as $row ) {
+            // Add new line in the file. @codingStandardsIgnoreLine
+            fputcsv(
+                $file,
+                (array) $row // convert the object to array.
+            );
+        }
+    }
 
 }

Spacing +14 added lines, -14 removed lines

@@ -28,23 +28,23 @@ 
 	 */
 	public function export() {
 		// Bail if the permalink structure is different from "Post name".
-		if ( ! $this->is_postname_permalink_structure() ) {
-			wp_die( 'The current permalink structure do not allow to export your data. Please change the permalink structure to "Post name".' );
+		if ( ! $this->is_postname_permalink_structure()) {
+			wp_die('The current permalink structure do not allow to export your data. Please change the permalink structure to "Post name".');
 		}
 
 		// Output the file data. @codingStandardsIgnoreLine
 		@ob_end_clean();
 
 		// Generate unique filename using current timestamp.
-		$filename = 'wl-ga-export-' . gmdate( 'Y-m-d-H-i-s' ) . '.csv';
+		$filename = 'wl-ga-export-'.gmdate('Y-m-d-H-i-s').'.csv';
 
 		// Add proper file headers.
-		header( "Content-Disposition: attachment; filename=$filename" );
-		header( 'Content-Type: text/csv; charset=' . get_bloginfo( 'charset' ) );
+		header("Content-Disposition: attachment; filename=$filename");
+		header('Content-Type: text/csv; charset='.get_bloginfo('charset'));
 
 		// Do not cache the file.
-		header( 'Pragma: no-cache' );
-		header( 'Expires: 0' );
+		header('Pragma: no-cache');
+		header('Expires: 0');
 
 		// Build the CSV file.
 		$this->create_csv();
@@ -62,10 +62,10 @@ 
 	 */
 	public function get_site_path() {
 		// Get home url from database.
-		$home_url = home_url( '/' );
+		$home_url = home_url('/');
 
 		// Parse the url.
-		$parsed = wp_parse_url( $home_url );
+		$parsed = wp_parse_url($home_url);
 
 		// Return the path.
 		return $parsed['path'];
@@ -80,16 +80,16 @@ 
 	 */
 	public static function is_postname_permalink_structure() {
 		// Get current permalink structure.
-		$structure = get_option( 'permalink_structure' );
+		$structure = get_option('permalink_structure');
 
 		// The regular expression. It will check if the site structure contains postname.
 		$regex = '~^/\%postname\%/$~';
 
 		// Check if the site structure match the rquired one.
-		preg_match( $regex, $structure, $matches );
+		preg_match($regex, $structure, $matches);
 
 		// Bail if the site have different structure.
-		if ( empty( $matches ) ) {
+		if (empty($matches)) {
 			return false;
 		}
 
@@ -148,7 +148,7 @@ 
 	public function create_csv() {
 		// Create a file pointer connected to the output stream.
 		// Ignoring linter notices below that complain about file output which do not actually happen.
-		$file = fopen( 'php://output', 'w' );
+		$file = fopen('php://output', 'w');
 
 		// Add the column headers. @codingStandardsIgnoreLine
 		fputcsv(
@@ -161,7 +161,7 @@ 
 		);
 
 		// Cycle through items and add each item data to the file.
-		foreach ( $this->get_content_data() as $row ) {
+		foreach ($this->get_content_data() as $row) {
 			// Add new line in the file. @codingStandardsIgnoreLine
 			fputcsv(
 				$file,

src/includes/properties/class-wordlift-duration-property-service.php

Indentation +19 added lines, -19 removed lines

@@ -19,31 +19,31 @@
  */
 class Wordlift_Duration_Property_Service extends Wordlift_Simple_Property_Service {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get( $id, $meta_key, $type ) {
+    /**
+     * {@inheritdoc}
+     */
+    public function get( $id, $meta_key, $type ) {
 
-		// Get the values and filter out the empty ones (or the ones with 00:00).
-		$values = array_filter(
-			parent::get( $id, $meta_key, $type ),
-			function ( $item ) {
-				return ! empty( $item ) && '00:00' !== $item;
-			}
-		);
+        // Get the values and filter out the empty ones (or the ones with 00:00).
+        $values = array_filter(
+            parent::get( $id, $meta_key, $type ),
+            function ( $item ) {
+                return ! empty( $item ) && '00:00' !== $item;
+            }
+        );
 
-		/*
+        /*
 		 * Map the value in the meta
 		 * The UI for the meta date enable two forms, a number of minutes
 		 * or an h:mm format.
 		 * Both needs to be adjusted to the iso format.
 		 */
-		return array_map(
-			function ( $value ) {
-				return 'PT' . str_replace( ':', 'H', $value ) . 'M';
-			},
-			$values
-		);
-	}
+        return array_map(
+            function ( $value ) {
+                return 'PT' . str_replace( ':', 'H', $value ) . 'M';
+            },
+            $values
+        );
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -22,13 +22,13 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function get( $id, $meta_key, $type ) {
+	public function get($id, $meta_key, $type) {
 
 		// Get the values and filter out the empty ones (or the ones with 00:00).
 		$values = array_filter(
-			parent::get( $id, $meta_key, $type ),
-			function ( $item ) {
-				return ! empty( $item ) && '00:00' !== $item;
+			parent::get($id, $meta_key, $type),
+			function($item) {
+				return ! empty($item) && '00:00' !== $item;
 			}
 		);
 
@@ -39,8 +39,8 @@ 
 		 * Both needs to be adjusted to the iso format.
 		 */
 		return array_map(
-			function ( $value ) {
-				return 'PT' . str_replace( ':', 'H', $value ) . 'M';
+			function($value) {
+				return 'PT'.str_replace(':', 'H', $value).'M';
 			},
 			$values
 		);

src/includes/properties/class-wordlift-property-getter.php

Indentation +60 added lines, -60 removed lines

@@ -9,74 +9,74 @@
  */
 class Wordlift_Property_Getter {
 
-	/**
-	 * An array of {@link Wordlift_Simple_Property_Service}s which can access a
-	 * property.
-	 *
-	 * @since 3.8.0
-	 * @access private
-	 * @var Wordlift_Simple_Property_Service[] $services An array of {@link Wordlift_Simple_Property_Service}s.
-	 */
-	private $services = array();
+    /**
+     * An array of {@link Wordlift_Simple_Property_Service}s which can access a
+     * property.
+     *
+     * @since 3.8.0
+     * @access private
+     * @var Wordlift_Simple_Property_Service[] $services An array of {@link Wordlift_Simple_Property_Service}s.
+     */
+    private $services = array();
 
-	/**
-	 * The default {@link Wordlift_Simple_Property_Service} which is used to access
-	 * a property when no specific {@link Wordlift_Simple_Property_Service} is found
-	 * in the {@see $services} array.
-	 *
-	 * @var Wordlift_Simple_Property_Service
-	 */
-	private $default;
+    /**
+     * The default {@link Wordlift_Simple_Property_Service} which is used to access
+     * a property when no specific {@link Wordlift_Simple_Property_Service} is found
+     * in the {@see $services} array.
+     *
+     * @var Wordlift_Simple_Property_Service
+     */
+    private $default;
 
-	/**
-	 * Create a property service with the provided {@link Wordlift_Simple_Property_Service}
-	 * as default.
-	 *
-	 * @param $default
-	 *
-	 * @since 3.8.0
-	 */
-	public function __construct( $default ) {
+    /**
+     * Create a property service with the provided {@link Wordlift_Simple_Property_Service}
+     * as default.
+     *
+     * @param $default
+     *
+     * @since 3.8.0
+     */
+    public function __construct( $default ) {
 
-		$this->default = $default;
+        $this->default = $default;
 
-	}
+    }
 
-	/**
-	 * Register a {@link Wordlift_Simple_Property_Service} for the specified meta keys.
-	 *
-	 * @param \Wordlift_Simple_Property_Service $property_service A {@link Wordlift_Simple_Property_Service} instance.
-	 * @param array                             $meta_keys An array of meta keys that the provided {@link Wordlift_Simple_Property_Service} will handle.
-	 *
-	 * @since 3.8.0
-	 */
-	public function register( $property_service, $meta_keys ) {
+    /**
+     * Register a {@link Wordlift_Simple_Property_Service} for the specified meta keys.
+     *
+     * @param \Wordlift_Simple_Property_Service $property_service A {@link Wordlift_Simple_Property_Service} instance.
+     * @param array                             $meta_keys An array of meta keys that the provided {@link Wordlift_Simple_Property_Service} will handle.
+     *
+     * @since 3.8.0
+     */
+    public function register( $property_service, $meta_keys ) {
 
-		// Register the specified property service for each meta key.
-		foreach ( $meta_keys as $meta_key ) {
-			$this->services[ $meta_key ] = $property_service;
-		}
+        // Register the specified property service for each meta key.
+        foreach ( $meta_keys as $meta_key ) {
+            $this->services[ $meta_key ] = $property_service;
+        }
 
-	}
+    }
 
-	/**
-	 * Get the value for the specified entity post id and WP's meta key.
-	 *
-	 * @param int    $post_id The post id.
-	 * @param string $meta_key The meta key.
-	 *
-	 * @param int    $type Term or Post, by default Post is used.
-	 *
-	 * @return mixed|null The property value or null.
-	 * @since 3.8.0
-	 */
-	public function get( $post_id, $meta_key, $type ) {
+    /**
+     * Get the value for the specified entity post id and WP's meta key.
+     *
+     * @param int    $post_id The post id.
+     * @param string $meta_key The meta key.
+     *
+     * @param int    $type Term or Post, by default Post is used.
+     *
+     * @return mixed|null The property value or null.
+     * @since 3.8.0
+     */
+    public function get( $post_id, $meta_key, $type ) {
 
-		return isset( $this->services[ $meta_key ] )
-			// Use a specific property service.
-			? $this->services[ $meta_key ]->get( $post_id, $meta_key, $type )
-			// Use the default property service.
-			: $this->default->get( $post_id, $meta_key, $type );
-	}
+        return isset( $this->services[ $meta_key ] )
+            // Use a specific property service.
+            ? $this->services[ $meta_key ]->get( $post_id, $meta_key, $type )
+            // Use the default property service.
+            : $this->default->get( $post_id, $meta_key, $type );
+    }
 
 }

Spacing +8 added lines, -8 removed lines

@@ -36,7 +36,7 @@ 
 	 *
 	 * @since 3.8.0
 	 */
-	public function __construct( $default ) {
+	public function __construct($default) {
 
 		$this->default = $default;
 
@@ -50,11 +50,11 @@ 
 	 *
 	 * @since 3.8.0
 	 */
-	public function register( $property_service, $meta_keys ) {
+	public function register($property_service, $meta_keys) {
 
 		// Register the specified property service for each meta key.
-		foreach ( $meta_keys as $meta_key ) {
-			$this->services[ $meta_key ] = $property_service;
+		foreach ($meta_keys as $meta_key) {
+			$this->services[$meta_key] = $property_service;
 		}
 
 	}
@@ -70,13 +70,13 @@ 
 	 * @return mixed|null The property value or null.
 	 * @since 3.8.0
 	 */
-	public function get( $post_id, $meta_key, $type ) {
+	public function get($post_id, $meta_key, $type) {
 
-		return isset( $this->services[ $meta_key ] )
+		return isset($this->services[$meta_key])
 			// Use a specific property service.
-			? $this->services[ $meta_key ]->get( $post_id, $meta_key, $type )
+			? $this->services[$meta_key]->get($post_id, $meta_key, $type)
 			// Use the default property service.
-			: $this->default->get( $post_id, $meta_key, $type );
+			: $this->default->get($post_id, $meta_key, $type);
 	}
 
 }

src/includes/properties/class-wordlift-property-getter-factory.php

Indentation +51 added lines, -51 removed lines

@@ -26,60 +26,60 @@
  */
 class Wordlift_Property_Getter_Factory {
 
-	/**
-	 * Create a {@link Wordlift_Property_Getter} instance.
-	 *
-	 * @return \Wordlift_Property_Getter A {@link Wordlift_Property_Getter} instance.
-	 * @since 3.8.0
-	 */
-	public static function create() {
+    /**
+     * Create a {@link Wordlift_Property_Getter} instance.
+     *
+     * @return \Wordlift_Property_Getter A {@link Wordlift_Property_Getter} instance.
+     * @since 3.8.0
+     */
+    public static function create() {
 
-		$property_getter = new Wordlift_Property_Getter( new Wordlift_Simple_Property_Service() );
-		$property_getter->register(
-			new Wordlift_Entity_Property_Service(),
-			array(
-				Wordlift_Schema_Service::FIELD_FOUNDER,
-				Wordlift_Schema_Service::FIELD_AUTHOR,
-				Wordlift_Schema_Service::FIELD_KNOWS,
-				Wordlift_Schema_Service::FIELD_BIRTH_PLACE,
-				Wordlift_Schema_Service::FIELD_AFFILIATION,
-				Wordlift_Schema_Service::FIELD_PERFORMER,
-				Wordlift_Schema_Service::FIELD_OFFERS,
-				Wordlift_Schema_Service::FIELD_ITEM_OFFERED,
-			)
-		);
-		$property_getter->register(
-			new Wordlift_Location_Property_Service(),
-			array(
-				Wordlift_Schema_Service::FIELD_LOCATION,
-			)
-		);
-		$property_getter->register( new Wordlift_Url_Property_Service(), array( Wordlift_Url_Property_Service::META_KEY ) );
-		$property_getter->register(
-			new Wordlift_Double_Property_Service(),
-			array(
-				Wordlift_Schema_Service::FIELD_GEO_LATITUDE,
-				Wordlift_Schema_Service::FIELD_GEO_LONGITUDE,
-			)
-		);
+        $property_getter = new Wordlift_Property_Getter( new Wordlift_Simple_Property_Service() );
+        $property_getter->register(
+            new Wordlift_Entity_Property_Service(),
+            array(
+                Wordlift_Schema_Service::FIELD_FOUNDER,
+                Wordlift_Schema_Service::FIELD_AUTHOR,
+                Wordlift_Schema_Service::FIELD_KNOWS,
+                Wordlift_Schema_Service::FIELD_BIRTH_PLACE,
+                Wordlift_Schema_Service::FIELD_AFFILIATION,
+                Wordlift_Schema_Service::FIELD_PERFORMER,
+                Wordlift_Schema_Service::FIELD_OFFERS,
+                Wordlift_Schema_Service::FIELD_ITEM_OFFERED,
+            )
+        );
+        $property_getter->register(
+            new Wordlift_Location_Property_Service(),
+            array(
+                Wordlift_Schema_Service::FIELD_LOCATION,
+            )
+        );
+        $property_getter->register( new Wordlift_Url_Property_Service(), array( Wordlift_Url_Property_Service::META_KEY ) );
+        $property_getter->register(
+            new Wordlift_Double_Property_Service(),
+            array(
+                Wordlift_Schema_Service::FIELD_GEO_LATITUDE,
+                Wordlift_Schema_Service::FIELD_GEO_LONGITUDE,
+            )
+        );
 
-		$property_getter->register(
-			new Wordlift_Duration_Property_Service(),
-			array(
-				Wordlift_Schema_Service::FIELD_PREP_TIME,
-				Wordlift_Schema_Service::FIELD_COOK_TIME,
-				Wordlift_Schema_Service::FIELD_TOTAL_TIME,
-			)
-		);
+        $property_getter->register(
+            new Wordlift_Duration_Property_Service(),
+            array(
+                Wordlift_Schema_Service::FIELD_PREP_TIME,
+                Wordlift_Schema_Service::FIELD_COOK_TIME,
+                Wordlift_Schema_Service::FIELD_TOTAL_TIME,
+            )
+        );
 
-		add_action(
-			'after_setup_theme',
-			function () use ( $property_getter ) {
-				$property_getter->register( new Wordlift_Required_Property_Service(), apply_filters( 'wl_required_property', array() ) );
-			}
-		);
+        add_action(
+            'after_setup_theme',
+            function () use ( $property_getter ) {
+                $property_getter->register( new Wordlift_Required_Property_Service(), apply_filters( 'wl_required_property', array() ) );
+            }
+        );
 
-		return $property_getter;
-	}
+        return $property_getter;
+    }
 
 }

Spacing +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
 	 */
 	public static function create() {
 
-		$property_getter = new Wordlift_Property_Getter( new Wordlift_Simple_Property_Service() );
+		$property_getter = new Wordlift_Property_Getter(new Wordlift_Simple_Property_Service());
 		$property_getter->register(
 			new Wordlift_Entity_Property_Service(),
 			array(
@@ -54,7 +54,7 @@ 
 				Wordlift_Schema_Service::FIELD_LOCATION,
 			)
 		);
-		$property_getter->register( new Wordlift_Url_Property_Service(), array( Wordlift_Url_Property_Service::META_KEY ) );
+		$property_getter->register(new Wordlift_Url_Property_Service(), array(Wordlift_Url_Property_Service::META_KEY));
 		$property_getter->register(
 			new Wordlift_Double_Property_Service(),
 			array(
@@ -74,8 +74,8 @@ 
 
 		add_action(
 			'after_setup_theme',
-			function () use ( $property_getter ) {
-				$property_getter->register( new Wordlift_Required_Property_Service(), apply_filters( 'wl_required_property', array() ) );
+			function() use ($property_getter) {
+				$property_getter->register(new Wordlift_Required_Property_Service(), apply_filters('wl_required_property', array()));
 			}
 		);
 

src/includes/properties/class-wordlift-simple-property-service.php

Indentation +26 added lines, -26 removed lines

@@ -12,33 +12,33 @@
  */
 class Wordlift_Simple_Property_Service {
 
-	/**
-	 * The meta key for this property service.
-	 *
-	 * @since 3.8.0
-	 */
-	const META_KEY = '*';
+    /**
+     * The meta key for this property service.
+     *
+     * @since 3.8.0
+     */
+    const META_KEY = '*';
 
-	/**
-	 * Get the property value for the specified post id and meta with the specified key.
-	 *
-	 * @param int                   $id The post id.
-	 * @param string                $meta_key The meta key.
-	 *
-	 * @param $type int Post or Term
-	 *
-	 * @return mixed|null The property value.
-	 * @since 3.8.0
-	 */
-	public function get( $id, $meta_key, $type ) {
+    /**
+     * Get the property value for the specified post id and meta with the specified key.
+     *
+     * @param int                   $id The post id.
+     * @param string                $meta_key The meta key.
+     *
+     * @param $type int Post or Term
+     *
+     * @return mixed|null The property value.
+     * @since 3.8.0
+     */
+    public function get( $id, $meta_key, $type ) {
 
-		if ( Object_Type_Enum::POST === $type ) {
-			// Get the value stored in WP.
-			return get_post_meta( $id, $meta_key );
-		} elseif ( Object_Type_Enum::TERM === $type ) {
-			return get_term_meta( $id, $meta_key );
-		}
-		return null;
-	}
+        if ( Object_Type_Enum::POST === $type ) {
+            // Get the value stored in WP.
+            return get_post_meta( $id, $meta_key );
+        } elseif ( Object_Type_Enum::TERM === $type ) {
+            return get_term_meta( $id, $meta_key );
+        }
+        return null;
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -30,13 +30,13 @@
 	 * @return mixed|null The property value.
 	 * @since 3.8.0
 	 */
-	public function get( $id, $meta_key, $type ) {
+	public function get($id, $meta_key, $type) {
 
-		if ( Object_Type_Enum::POST === $type ) {
+		if (Object_Type_Enum::POST === $type) {
 			// Get the value stored in WP.
-			return get_post_meta( $id, $meta_key );
-		} elseif ( Object_Type_Enum::TERM === $type ) {
-			return get_term_meta( $id, $meta_key );
+			return get_post_meta($id, $meta_key);
+		} elseif (Object_Type_Enum::TERM === $type) {
+			return get_term_meta($id, $meta_key);
 		}
 		return null;
 	}

src/includes/properties/class-wordlift-double-property-service.php

Indentation +12 added lines, -12 removed lines

@@ -16,18 +16,18 @@
  */
 class Wordlift_Double_Property_Service extends Wordlift_Simple_Property_Service {
 
-	/**
-	 * @inheritdoc
-	 */
-	public function get( $id, $meta_key, $type ) {
+    /**
+     * @inheritdoc
+     */
+    public function get( $id, $meta_key, $type ) {
 
-		// Map the result to a numeric value when possible.
-		return array_map(
-			function ( $value ) {
-				return is_numeric( $value ) ? (float) $value : $value;
-			},
-			parent::get( $id, $meta_key, $type )
-		);
-	}
+        // Map the result to a numeric value when possible.
+        return array_map(
+            function ( $value ) {
+                return is_numeric( $value ) ? (float) $value : $value;
+            },
+            parent::get( $id, $meta_key, $type )
+        );
+    }
 
 }

Spacing +4 added lines, -4 removed lines

@@ -19,14 +19,14 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function get( $id, $meta_key, $type ) {
+	public function get($id, $meta_key, $type) {
 
 		// Map the result to a numeric value when possible.
 		return array_map(
-			function ( $value ) {
-				return is_numeric( $value ) ? (float) $value : $value;
+			function($value) {
+				return is_numeric($value) ? (float) $value : $value;
 			},
-			parent::get( $id, $meta_key, $type )
+			parent::get($id, $meta_key, $type)
 		);
 	}