insideout10 / wordlift-plugin

src/wordlift/task/background/class-background-task-page.php

Indentation +72 added lines, -72 removed lines

@@ -8,73 +8,73 @@ 
 
 class Background_Task_Page {
 
-	private $title;
-
-	private $menu_slug;
-
-	/**
-	 * @var Background_Task_Route $background_task_route
-	 */
-	private $background_task_route;
-
-	/**
-	 * @throws Exception if one or more parameters are invalid.
-	 */
-	public function __construct( $title, $menu_slug, $background_task_route ) {
-		Assertions::not_empty( $title, '`$title` cannot be empty.' );
-		Assertions::not_empty( $menu_slug, '`$menu_slug` cannot be empty.' );
-		Assertions::is_a( $background_task_route, 'Wordlift\Task\Background\Background_Task_Route', '`Background_Task_Route` must be of type `Wordlift\Task\Background\Background_Route`.' );
-
-		add_action( 'admin_menu', array( $this, 'admin_menu' ) );
-
-		$this->title                 = $title;
-		$this->menu_slug             = $menu_slug;
-		$this->background_task_route = $background_task_route;
-	}
-
-	/**
-	 * @throws Exception if one or more parameters are invalid.
-	 */
-	public static function create( $title, $menu_slug, $background_route ) {
-		return new self( $title, $menu_slug, $background_route );
-	}
-
-	public function admin_menu() {
-
-		add_submenu_page(
-			'wl_admin_menu',
-			$this->title,
-			$this->title,
-			'manage_options',
-			$this->menu_slug,
-			array(
-				$this,
-				'render',
-			)
-		);
-
-	}
-
-	public function render() {
-
-		wp_enqueue_style(
-			'wl-task-page',
-			plugin_dir_url( __FILE__ ) . 'assets/task-page.css',
-			array(),
-			Wordlift::get_instance()->get_version(),
-			'all'
-		);
-
-		wp_enqueue_script(
-			'wl-task-page',
-			plugin_dir_url( __FILE__ ) . 'assets/task-page.js',
-			array( 'wp-api' ),
-			WORDLIFT_VERSION,
-			false
-		);
-
-		wp_localize_script( 'wl-task-page', '_wlTaskPageSettings', array( 'rest_path' => $this->background_task_route->get_rest_path() ) );
-		?>
+    private $title;
+
+    private $menu_slug;
+
+    /**
+     * @var Background_Task_Route $background_task_route
+     */
+    private $background_task_route;
+
+    /**
+     * @throws Exception if one or more parameters are invalid.
+     */
+    public function __construct( $title, $menu_slug, $background_task_route ) {
+        Assertions::not_empty( $title, '`$title` cannot be empty.' );
+        Assertions::not_empty( $menu_slug, '`$menu_slug` cannot be empty.' );
+        Assertions::is_a( $background_task_route, 'Wordlift\Task\Background\Background_Task_Route', '`Background_Task_Route` must be of type `Wordlift\Task\Background\Background_Route`.' );
+
+        add_action( 'admin_menu', array( $this, 'admin_menu' ) );
+
+        $this->title                 = $title;
+        $this->menu_slug             = $menu_slug;
+        $this->background_task_route = $background_task_route;
+    }
+
+    /**
+     * @throws Exception if one or more parameters are invalid.
+     */
+    public static function create( $title, $menu_slug, $background_route ) {
+        return new self( $title, $menu_slug, $background_route );
+    }
+
+    public function admin_menu() {
+
+        add_submenu_page(
+            'wl_admin_menu',
+            $this->title,
+            $this->title,
+            'manage_options',
+            $this->menu_slug,
+            array(
+                $this,
+                'render',
+            )
+        );
+
+    }
+
+    public function render() {
+
+        wp_enqueue_style(
+            'wl-task-page',
+            plugin_dir_url( __FILE__ ) . 'assets/task-page.css',
+            array(),
+            Wordlift::get_instance()->get_version(),
+            'all'
+        );
+
+        wp_enqueue_script(
+            'wl-task-page',
+            plugin_dir_url( __FILE__ ) . 'assets/task-page.js',
+            array( 'wp-api' ),
+            WORDLIFT_VERSION,
+            false
+        );
+
+        wp_localize_script( 'wl-task-page', '_wlTaskPageSettings', array( 'rest_path' => $this->background_task_route->get_rest_path() ) );
+        ?>
 		<div class="wrap">
 			<h2><?php echo esc_html( $this->title ); ?></h2>
 
@@ -85,17 +85,17 @@ 
 
 			<button id="wl-start-btn" type="button" class="button button-large button-primary">
 			<?php
-				esc_html_e( 'Start', 'wordlift' );
-			?>
+                esc_html_e( 'Start', 'wordlift' );
+            ?>
 				</button>
 			<button id="wl-stop-btn" type="button" class="button button-large button-primary hidden">
 			<?php
-				esc_html_e( 'Stop', 'wordlift' );
-			?>
+                esc_html_e( 'Stop', 'wordlift' );
+            ?>
 				</button>
 
 		</div>
 		<?php
-	}
+    }
 
 }

Spacing +14 added lines, -14 removed lines

@@ -20,12 +20,12 @@ 
 	/**
 	 * @throws Exception if one or more parameters are invalid.
 	 */
-	public function __construct( $title, $menu_slug, $background_task_route ) {
-		Assertions::not_empty( $title, '`$title` cannot be empty.' );
-		Assertions::not_empty( $menu_slug, '`$menu_slug` cannot be empty.' );
-		Assertions::is_a( $background_task_route, 'Wordlift\Task\Background\Background_Task_Route', '`Background_Task_Route` must be of type `Wordlift\Task\Background\Background_Route`.' );
+	public function __construct($title, $menu_slug, $background_task_route) {
+		Assertions::not_empty($title, '`$title` cannot be empty.');
+		Assertions::not_empty($menu_slug, '`$menu_slug` cannot be empty.');
+		Assertions::is_a($background_task_route, 'Wordlift\Task\Background\Background_Task_Route', '`Background_Task_Route` must be of type `Wordlift\Task\Background\Background_Route`.');
 
-		add_action( 'admin_menu', array( $this, 'admin_menu' ) );
+		add_action('admin_menu', array($this, 'admin_menu'));
 
 		$this->title                 = $title;
 		$this->menu_slug             = $menu_slug;
@@ -35,8 +35,8 @@ 
 	/**
 	 * @throws Exception if one or more parameters are invalid.
 	 */
-	public static function create( $title, $menu_slug, $background_route ) {
-		return new self( $title, $menu_slug, $background_route );
+	public static function create($title, $menu_slug, $background_route) {
+		return new self($title, $menu_slug, $background_route);
 	}
 
 	public function admin_menu() {
@@ -59,7 +59,7 @@ 
 
 		wp_enqueue_style(
 			'wl-task-page',
-			plugin_dir_url( __FILE__ ) . 'assets/task-page.css',
+			plugin_dir_url(__FILE__).'assets/task-page.css',
 			array(),
 			Wordlift::get_instance()->get_version(),
 			'all'
@@ -67,16 +67,16 @@ 
 
 		wp_enqueue_script(
 			'wl-task-page',
-			plugin_dir_url( __FILE__ ) . 'assets/task-page.js',
-			array( 'wp-api' ),
+			plugin_dir_url(__FILE__).'assets/task-page.js',
+			array('wp-api'),
 			WORDLIFT_VERSION,
 			false
 		);
 
-		wp_localize_script( 'wl-task-page', '_wlTaskPageSettings', array( 'rest_path' => $this->background_task_route->get_rest_path() ) );
+		wp_localize_script('wl-task-page', '_wlTaskPageSettings', array('rest_path' => $this->background_task_route->get_rest_path()));
 		?>
 		<div class="wrap">
-			<h2><?php echo esc_html( $this->title ); ?></h2>
+			<h2><?php echo esc_html($this->title); ?></h2>
 
 			<div class="wl-task__progress" style="border: 1px solid #23282D; height: 20px; margin: 8px 0;">
 				<div class="wl-task__progress__bar"
@@ -85,12 +85,12 @@ 
 
 			<button id="wl-start-btn" type="button" class="button button-large button-primary">
 			<?php
-				esc_html_e( 'Start', 'wordlift' );
+				esc_html_e('Start', 'wordlift');
 			?>
 				</button>
 			<button id="wl-stop-btn" type="button" class="button button-large button-primary hidden">
 			<?php
-				esc_html_e( 'Stop', 'wordlift' );
+				esc_html_e('Stop', 'wordlift');
 			?>
 				</button>
 

src/wordlift/task/background/class-background-task-route.php

Indentation +82 added lines, -82 removed lines

@@ -8,87 +8,87 @@
 
 class Background_Task_Route {
 
-	const VERSION_STRING = 'wordlift/v1';
-
-	/**
-	 * @var Background_Task
-	 */
-	private $background_task;
-
-	/**
-	 * @var string
-	 */
-	private $route_name;
-
-	public function __construct( $background_task, $route_name ) {
-		$this->background_task = $background_task;
-		$this->route_name      = $route_name;
-	}
-
-	public static function create( $task, $route_name ) {
-		$route = new self( $task, $route_name );
-
-		add_action( 'rest_api_init', array( $route, 'register' ) );
-
-		return $route;
-	}
-
-	/**
-	 * @throws Exception if the input value is invalid.
-	 */
-	public function register() {
-		Assertions::starts_with( $this->route_name, '/', 'The route name must start with a slash.' );
-
-		register_rest_route(
-			self::VERSION_STRING,
-			$this->route_name,
-			array(
-				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => array( $this->background_task, 'start' ),
-				'permission_callback' => array( $this, 'permission_callback' ),
-			)
-		);
-
-		register_rest_route(
-			self::VERSION_STRING,
-			$this->route_name,
-			array(
-				'methods'             => WP_REST_Server::READABLE,
-				'callback'            => array( $this->background_task, 'get_info' ),
-				'permission_callback' => array( $this, 'permission_callback' ),
-			)
-		);
-
-		register_rest_route(
-			self::VERSION_STRING,
-			$this->route_name,
-			array(
-				'methods'             => WP_REST_Server::DELETABLE,
-				'callback'            => array( $this->background_task, 'stop' ),
-				'permission_callback' => array( $this, 'permission_callback' ),
-			)
-		);
-
-		register_rest_route(
-			self::VERSION_STRING,
-			$this->route_name,
-			array(
-				'methods'             => 'PUT',
-				'callback'            => array( $this->background_task, 'resume' ),
-				'permission_callback' => array( $this, 'permission_callback' ),
-			)
-		);
-
-	}
-
-	public function permission_callback() {
-		$user = wp_get_current_user();
-
-		return is_super_admin( $user->ID ) || in_array( 'administrator', (array) $user->roles, true );
-	}
-
-	public function get_rest_path() {
-		return self::VERSION_STRING . $this->route_name;
-	}
+    const VERSION_STRING = 'wordlift/v1';
+
+    /**
+     * @var Background_Task
+     */
+    private $background_task;
+
+    /**
+     * @var string
+     */
+    private $route_name;
+
+    public function __construct( $background_task, $route_name ) {
+        $this->background_task = $background_task;
+        $this->route_name      = $route_name;
+    }
+
+    public static function create( $task, $route_name ) {
+        $route = new self( $task, $route_name );
+
+        add_action( 'rest_api_init', array( $route, 'register' ) );
+
+        return $route;
+    }
+
+    /**
+     * @throws Exception if the input value is invalid.
+     */
+    public function register() {
+        Assertions::starts_with( $this->route_name, '/', 'The route name must start with a slash.' );
+
+        register_rest_route(
+            self::VERSION_STRING,
+            $this->route_name,
+            array(
+                'methods'             => WP_REST_Server::CREATABLE,
+                'callback'            => array( $this->background_task, 'start' ),
+                'permission_callback' => array( $this, 'permission_callback' ),
+            )
+        );
+
+        register_rest_route(
+            self::VERSION_STRING,
+            $this->route_name,
+            array(
+                'methods'             => WP_REST_Server::READABLE,
+                'callback'            => array( $this->background_task, 'get_info' ),
+                'permission_callback' => array( $this, 'permission_callback' ),
+            )
+        );
+
+        register_rest_route(
+            self::VERSION_STRING,
+            $this->route_name,
+            array(
+                'methods'             => WP_REST_Server::DELETABLE,
+                'callback'            => array( $this->background_task, 'stop' ),
+                'permission_callback' => array( $this, 'permission_callback' ),
+            )
+        );
+
+        register_rest_route(
+            self::VERSION_STRING,
+            $this->route_name,
+            array(
+                'methods'             => 'PUT',
+                'callback'            => array( $this->background_task, 'resume' ),
+                'permission_callback' => array( $this, 'permission_callback' ),
+            )
+        );
+
+    }
+
+    public function permission_callback() {
+        $user = wp_get_current_user();
+
+        return is_super_admin( $user->ID ) || in_array( 'administrator', (array) $user->roles, true );
+    }
+
+    public function get_rest_path() {
+        return self::VERSION_STRING . $this->route_name;
+    }
 
 }

Spacing +15 added lines, -15 removed lines

@@ -20,15 +20,15 @@ 
 	 */
 	private $route_name;
 
-	public function __construct( $background_task, $route_name ) {
+	public function __construct($background_task, $route_name) {
 		$this->background_task = $background_task;
 		$this->route_name      = $route_name;
 	}
 
-	public static function create( $task, $route_name ) {
-		$route = new self( $task, $route_name );
+	public static function create($task, $route_name) {
+		$route = new self($task, $route_name);
 
-		add_action( 'rest_api_init', array( $route, 'register' ) );
+		add_action('rest_api_init', array($route, 'register'));
 
 		return $route;
 	}
@@ -37,15 +37,15 @@ 
 	 * @throws Exception if the input value is invalid.
 	 */
 	public function register() {
-		Assertions::starts_with( $this->route_name, '/', 'The route name must start with a slash.' );
+		Assertions::starts_with($this->route_name, '/', 'The route name must start with a slash.');
 
 		register_rest_route(
 			self::VERSION_STRING,
 			$this->route_name,
 			array(
 				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => array( $this->background_task, 'start' ),
-				'permission_callback' => array( $this, 'permission_callback' ),
+				'callback'            => array($this->background_task, 'start'),
+				'permission_callback' => array($this, 'permission_callback'),
 			)
 		);
 
@@ -54,8 +54,8 @@ 
 			$this->route_name,
 			array(
 				'methods'             => WP_REST_Server::READABLE,
-				'callback'            => array( $this->background_task, 'get_info' ),
-				'permission_callback' => array( $this, 'permission_callback' ),
+				'callback'            => array($this->background_task, 'get_info'),
+				'permission_callback' => array($this, 'permission_callback'),
 			)
 		);
 
@@ -64,8 +64,8 @@ 
 			$this->route_name,
 			array(
 				'methods'             => WP_REST_Server::DELETABLE,
-				'callback'            => array( $this->background_task, 'stop' ),
-				'permission_callback' => array( $this, 'permission_callback' ),
+				'callback'            => array($this->background_task, 'stop'),
+				'permission_callback' => array($this, 'permission_callback'),
 			)
 		);
 
@@ -74,8 +74,8 @@ 
 			$this->route_name,
 			array(
 				'methods'             => 'PUT',
-				'callback'            => array( $this->background_task, 'resume' ),
-				'permission_callback' => array( $this, 'permission_callback' ),
+				'callback'            => array($this->background_task, 'resume'),
+				'permission_callback' => array($this, 'permission_callback'),
 			)
 		);
 
@@ -84,11 +84,11 @@ 
 	public function permission_callback() {
 		$user = wp_get_current_user();
 
-		return is_super_admin( $user->ID ) || in_array( 'administrator', (array) $user->roles, true );
+		return is_super_admin($user->ID) || in_array('administrator', (array) $user->roles, true);
 	}
 
 	public function get_rest_path() {
-		return self::VERSION_STRING . $this->route_name;
+		return self::VERSION_STRING.$this->route_name;
 	}
 
 }

src/wordlift/task/background/class-background-task-started-state.php

Indentation +56 added lines, -56 removed lines

@@ -4,79 +4,79 @@
 
 class Background_Task_Started_State extends Abstract_Background_Task_State {
 
-	/**
-	 * @var Background_Task
-	 */
-	private $context;
+    /**
+     * @var Background_Task
+     */
+    private $context;
 
-	private $batch_size = 5;
+    private $batch_size = 5;
 
-	/**
-	 * @var Task
-	 */
-	private $task;
+    /**
+     * @var Task
+     */
+    private $task;
 
-	/**
-	 * Back_Started_State constructor.
-	 *
-	 * @param Background_Task $context
-	 */
-	public function __construct( $context, $task ) {
-		parent::__construct( $context, Background_Task::STATE_STARTED );
+    /**
+     * Back_Started_State constructor.
+     *
+     * @param Background_Task $context
+     */
+    public function __construct( $context, $task ) {
+        parent::__construct( $context, Background_Task::STATE_STARTED );
 
-		$this->context = $context;
-		$this->task    = $task;
-	}
+        $this->context = $context;
+        $this->task    = $task;
+    }
 
-	public function enter() {
+    public function enter() {
 
-		$count = $this->task->starting();
+        $count = $this->task->starting();
 
-		update_option( $this->context->get_option_prefix() . 'count', $count, true );
-		update_option( $this->context->get_option_prefix() . 'offset', 0, true );
-		update_option( $this->context->get_option_prefix() . 'started', time(), true );
-		update_option( $this->context->get_option_prefix() . 'updated', time(), true );
+        update_option( $this->context->get_option_prefix() . 'count', $count, true );
+        update_option( $this->context->get_option_prefix() . 'offset', 0, true );
+        update_option( $this->context->get_option_prefix() . 'started', time(), true );
+        update_option( $this->context->get_option_prefix() . 'updated', time(), true );
 
-		$this->context->set_state( Background_Task::STATE_STARTED );
+        $this->context->set_state( Background_Task::STATE_STARTED );
 
-		$this->resume();
-	}
+        $this->resume();
+    }
 
-	public function resume() {
-		$this->context->push_to_queue( true );
-		$this->context->save()->dispatch();
-	}
+    public function resume() {
+        $this->context->push_to_queue( true );
+        $this->context->save()->dispatch();
+    }
 
-	public function leave() {
-		$this->context->set_state( null );
-	}
+    public function leave() {
+        $this->context->set_state( null );
+    }
 
-	public function task( $value ) {
-		$offset = get_option( $this->context->get_option_prefix() . 'offset' );
-		$count  = get_option( $this->context->get_option_prefix() . 'count' );
+    public function task( $value ) {
+        $offset = get_option( $this->context->get_option_prefix() . 'offset' );
+        $count  = get_option( $this->context->get_option_prefix() . 'count' );
 
-		$this->task->tick(
-			$value,
-			array(
-				'offset'     => $offset,
-				'count'      => $count,
-				'batch_size' => $this->batch_size,
-			)
-		);
+        $this->task->tick(
+            $value,
+            array(
+                'offset'     => $offset,
+                'count'      => $count,
+                'batch_size' => $this->batch_size,
+            )
+        );
 
-		update_option( $this->context->get_option_prefix() . 'updated', time(), true );
+        update_option( $this->context->get_option_prefix() . 'updated', time(), true );
 
-		// Increase the offset.
-		if ( ( $offset + $this->batch_size ) < $count ) {
-			update_option( $this->context->get_option_prefix() . 'offset', $offset + $this->batch_size, true );
+        // Increase the offset.
+        if ( ( $offset + $this->batch_size ) < $count ) {
+            update_option( $this->context->get_option_prefix() . 'offset', $offset + $this->batch_size, true );
 
-			return true;
-		}
+            return true;
+        }
 
-		// Stop processing.
-		$this->context->stop();
+        // Stop processing.
+        $this->context->stop();
 
-		return false;
-	}
+        return false;
+    }
 
 }

Spacing +15 added lines, -15 removed lines

@@ -21,8 +21,8 @@ 
 	 *
 	 * @param Background_Task $context
 	 */
-	public function __construct( $context, $task ) {
-		parent::__construct( $context, Background_Task::STATE_STARTED );
+	public function __construct($context, $task) {
+		parent::__construct($context, Background_Task::STATE_STARTED);
 
 		$this->context = $context;
 		$this->task    = $task;
@@ -32,28 +32,28 @@ 
 
 		$count = $this->task->starting();
 
-		update_option( $this->context->get_option_prefix() . 'count', $count, true );
-		update_option( $this->context->get_option_prefix() . 'offset', 0, true );
-		update_option( $this->context->get_option_prefix() . 'started', time(), true );
-		update_option( $this->context->get_option_prefix() . 'updated', time(), true );
+		update_option($this->context->get_option_prefix().'count', $count, true);
+		update_option($this->context->get_option_prefix().'offset', 0, true);
+		update_option($this->context->get_option_prefix().'started', time(), true);
+		update_option($this->context->get_option_prefix().'updated', time(), true);
 
-		$this->context->set_state( Background_Task::STATE_STARTED );
+		$this->context->set_state(Background_Task::STATE_STARTED);
 
 		$this->resume();
 	}
 
 	public function resume() {
-		$this->context->push_to_queue( true );
+		$this->context->push_to_queue(true);
 		$this->context->save()->dispatch();
 	}
 
 	public function leave() {
-		$this->context->set_state( null );
+		$this->context->set_state(null);
 	}
 
-	public function task( $value ) {
-		$offset = get_option( $this->context->get_option_prefix() . 'offset' );
-		$count  = get_option( $this->context->get_option_prefix() . 'count' );
+	public function task($value) {
+		$offset = get_option($this->context->get_option_prefix().'offset');
+		$count  = get_option($this->context->get_option_prefix().'count');
 
 		$this->task->tick(
 			$value,
@@ -64,11 +64,11 @@ 
 			)
 		);
 
-		update_option( $this->context->get_option_prefix() . 'updated', time(), true );
+		update_option($this->context->get_option_prefix().'updated', time(), true);
 
 		// Increase the offset.
-		if ( ( $offset + $this->batch_size ) < $count ) {
-			update_option( $this->context->get_option_prefix() . 'offset', $offset + $this->batch_size, true );
+		if (($offset + $this->batch_size) < $count) {
+			update_option($this->context->get_option_prefix().'offset', $offset + $this->batch_size, true);
 
 			return true;
 		}

src/wordlift/task/background/class-abstract-background-task-state.php

Indentation +27 added lines, -27 removed lines

@@ -4,32 +4,32 @@
 
 abstract class Abstract_Background_Task_State implements Background_Task_State {
 
-	/**
-	 * @var Background_Task_State
-	 */
-	private $state;
-
-	/**
-	 * @var Background_Task
-	 */
-	private $context;
-
-	public function __construct( $context, $state ) {
-		$this->state   = $state;
-		$this->context = $context;
-	}
-
-	public function get_info() {
-		$started     = get_option( $this->context->get_option_prefix() . 'started' );
-		$offset      = get_option( $this->context->get_option_prefix() . 'offset' );
-		$count       = get_option( $this->context->get_option_prefix() . 'count', array( 0 ) );
-		$last_update = get_option( $this->context->get_option_prefix() . 'updated' );
-
-		return new Background_Task_Info( $this->state, $started, $offset, $count, $last_update );
-	}
-
-	public function resume() {
-		// do nothing.
-	}
+    /**
+     * @var Background_Task_State
+     */
+    private $state;
+
+    /**
+     * @var Background_Task
+     */
+    private $context;
+
+    public function __construct( $context, $state ) {
+        $this->state   = $state;
+        $this->context = $context;
+    }
+
+    public function get_info() {
+        $started     = get_option( $this->context->get_option_prefix() . 'started' );
+        $offset      = get_option( $this->context->get_option_prefix() . 'offset' );
+        $count       = get_option( $this->context->get_option_prefix() . 'count', array( 0 ) );
+        $last_update = get_option( $this->context->get_option_prefix() . 'updated' );
+
+        return new Background_Task_Info( $this->state, $started, $offset, $count, $last_update );
+    }
+
+    public function resume() {
+        // do nothing.
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -14,18 +14,18 @@
 	 */
 	private $context;
 
-	public function __construct( $context, $state ) {
+	public function __construct($context, $state) {
 		$this->state   = $state;
 		$this->context = $context;
 	}
 
 	public function get_info() {
-		$started     = get_option( $this->context->get_option_prefix() . 'started' );
-		$offset      = get_option( $this->context->get_option_prefix() . 'offset' );
-		$count       = get_option( $this->context->get_option_prefix() . 'count', array( 0 ) );
-		$last_update = get_option( $this->context->get_option_prefix() . 'updated' );
+		$started     = get_option($this->context->get_option_prefix().'started');
+		$offset      = get_option($this->context->get_option_prefix().'offset');
+		$count       = get_option($this->context->get_option_prefix().'count', array(0));
+		$last_update = get_option($this->context->get_option_prefix().'updated');
 
-		return new Background_Task_Info( $this->state, $started, $offset, $count, $last_update );
+		return new Background_Task_Info($this->state, $started, $offset, $count, $last_update);
 	}
 
 	public function resume() {

src/wordlift/task/background/class-background-task-state.php

Indentation +24 added lines, -24 removed lines

@@ -4,29 +4,29 @@
 
 interface Background_Task_State {
 
-	public function enter();
-
-	public function leave();
-
-	/**
-	 * Task
-	 *
-	 * Override this method to perform any actions required on each
-	 * queue item. Return the modified item for further processing
-	 * in the next pass through. Or, return false to remove the
-	 * item from the queue.
-	 *
-	 * @param mixed $item Queue item to iterate over.
-	 *
-	 * @return mixed
-	 */
-	public function task( $item );
-
-	public function get_info();
-
-	/**
-	 * Try to resume an interrupted task.
-	 */
-	public function resume();
+    public function enter();
+
+    public function leave();
+
+    /**
+     * Task
+     *
+     * Override this method to perform any actions required on each
+     * queue item. Return the modified item for further processing
+     * in the next pass through. Or, return false to remove the
+     * item from the queue.
+     *
+     * @param mixed $item Queue item to iterate over.
+     *
+     * @return mixed
+     */
+    public function task( $item );
+
+    public function get_info();
+
+    /**
+     * Try to resume an interrupted task.
+     */
+    public function resume();
 
 }

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 	 *
 	 * @return mixed
 	 */
-	public function task( $item );
+	public function task($item);
 
 	public function get_info();
 

src/wordlift/scripts/class-scripts-helper.php

Indentation +59 added lines, -59 removed lines

@@ -19,70 +19,70 @@
 
 class Scripts_Helper {
 
-	/**
-	 * This function loads the javascript file according to the WordPress version.
-	 *
-	 * For WordPress < 5.0 it'll load the javascript file using the `.full` suffix i.e. the file that embeds all the
-	 * dependencies.
-	 *
-	 * For WordPress >= 5.0 it'll load the stripped down js.
-	 *
-	 * @param string $handle The handle name.
-	 * @param string $script_name The full script URL without the `.js` extension.
-	 * @param array  $dependencies An array of dependencies to be added only in WordPress > 5.0.
-	 */
-	public static function enqueue_based_on_wordpress_version( $handle, $script_name, $dependencies, $in_footer = false ) {
-		global $wp_version;
+    /**
+     * This function loads the javascript file according to the WordPress version.
+     *
+     * For WordPress < 5.0 it'll load the javascript file using the `.full` suffix i.e. the file that embeds all the
+     * dependencies.
+     *
+     * For WordPress >= 5.0 it'll load the stripped down js.
+     *
+     * @param string $handle The handle name.
+     * @param string $script_name The full script URL without the `.js` extension.
+     * @param array  $dependencies An array of dependencies to be added only in WordPress > 5.0.
+     */
+    public static function enqueue_based_on_wordpress_version( $handle, $script_name, $dependencies, $in_footer = false ) {
+        global $wp_version;
 
-		if ( version_compare( $wp_version, '5.0', '<' ) ) {
-			$actual_script_name  = "$script_name.full.js";
-			$actual_dependencies = array();
-		} else {
-			$actual_script_name  = "$script_name.js";
-			$actual_dependencies = $dependencies;
-		}
+        if ( version_compare( $wp_version, '5.0', '<' ) ) {
+            $actual_script_name  = "$script_name.full.js";
+            $actual_dependencies = array();
+        } else {
+            $actual_script_name  = "$script_name.js";
+            $actual_dependencies = $dependencies;
+        }
 
-		$wordlift = \Wordlift::get_instance();
-		wp_enqueue_script( $handle, $actual_script_name, $actual_dependencies, $wordlift->get_version(), $in_footer );
+        $wordlift = \Wordlift::get_instance();
+        wp_enqueue_script( $handle, $actual_script_name, $actual_dependencies, $wordlift->get_version(), $in_footer );
 
-	}
-	/**
-	 * This function registers the javascript file according to the WordPress version.
-	 *
-	 * For WordPress < 5.0 it'll register the javascript file using the `.full` suffix i.e. the file that embeds all the
-	 * dependencies.
-	 *
-	 * For WordPress >= 5.0 it'll register the stripped down js.
-	 *
-	 * @param string $handle The handle name.
-	 * @param string $script_name The full script URL without the `.js` extension.
-	 * @param array  $dependencies An array of dependencies to be added only in WordPress > 5.0.
-	 */
-	public static function register_based_on_wordpress_version(
-			$handle,
-			$script_name,
-			$dependencies,
-			$action = 'wp_enqueue_scripts',
-			$in_footer = false
-		) {
-		global $wp_version;
+    }
+    /**
+     * This function registers the javascript file according to the WordPress version.
+     *
+     * For WordPress < 5.0 it'll register the javascript file using the `.full` suffix i.e. the file that embeds all the
+     * dependencies.
+     *
+     * For WordPress >= 5.0 it'll register the stripped down js.
+     *
+     * @param string $handle The handle name.
+     * @param string $script_name The full script URL without the `.js` extension.
+     * @param array  $dependencies An array of dependencies to be added only in WordPress > 5.0.
+     */
+    public static function register_based_on_wordpress_version(
+            $handle,
+            $script_name,
+            $dependencies,
+            $action = 'wp_enqueue_scripts',
+            $in_footer = false
+        ) {
+        global $wp_version;
 
-		if ( version_compare( $wp_version, '5.0', '<' ) ) {
-			$actual_script_name  = "$script_name.full.js";
-			$actual_dependencies = array();
-		} else {
-			$actual_script_name  = "$script_name.js";
-			$actual_dependencies = $dependencies;
-		}
+        if ( version_compare( $wp_version, '5.0', '<' ) ) {
+            $actual_script_name  = "$script_name.full.js";
+            $actual_dependencies = array();
+        } else {
+            $actual_script_name  = "$script_name.js";
+            $actual_dependencies = $dependencies;
+        }
 
-		$wordlift = \Wordlift::get_instance();
-		add_action(
-			$action,
-			function () use ( $handle, $actual_script_name, $actual_dependencies, $wordlift, $in_footer ) {
-				wp_register_script( $handle, $actual_script_name, $actual_dependencies, $wordlift->get_version(), $in_footer );
-			}
-		);
+        $wordlift = \Wordlift::get_instance();
+        add_action(
+            $action,
+            function () use ( $handle, $actual_script_name, $actual_dependencies, $wordlift, $in_footer ) {
+                wp_register_script( $handle, $actual_script_name, $actual_dependencies, $wordlift->get_version(), $in_footer );
+            }
+        );
 
-	}
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -31,10 +31,10 @@ 
 	 * @param string $script_name The full script URL without the `.js` extension.
 	 * @param array  $dependencies An array of dependencies to be added only in WordPress > 5.0.
 	 */
-	public static function enqueue_based_on_wordpress_version( $handle, $script_name, $dependencies, $in_footer = false ) {
+	public static function enqueue_based_on_wordpress_version($handle, $script_name, $dependencies, $in_footer = false) {
 		global $wp_version;
 
-		if ( version_compare( $wp_version, '5.0', '<' ) ) {
+		if (version_compare($wp_version, '5.0', '<')) {
 			$actual_script_name  = "$script_name.full.js";
 			$actual_dependencies = array();
 		} else {
@@ -43,7 +43,7 @@ 
 		}
 
 		$wordlift = \Wordlift::get_instance();
-		wp_enqueue_script( $handle, $actual_script_name, $actual_dependencies, $wordlift->get_version(), $in_footer );
+		wp_enqueue_script($handle, $actual_script_name, $actual_dependencies, $wordlift->get_version(), $in_footer);
 
 	}
 	/**
@@ -67,7 +67,7 @@ 
 		) {
 		global $wp_version;
 
-		if ( version_compare( $wp_version, '5.0', '<' ) ) {
+		if (version_compare($wp_version, '5.0', '<')) {
 			$actual_script_name  = "$script_name.full.js";
 			$actual_dependencies = array();
 		} else {
@@ -78,8 +78,8 @@ 
 		$wordlift = \Wordlift::get_instance();
 		add_action(
 			$action,
-			function () use ( $handle, $actual_script_name, $actual_dependencies, $wordlift, $in_footer ) {
-				wp_register_script( $handle, $actual_script_name, $actual_dependencies, $wordlift->get_version(), $in_footer );
+			function() use ($handle, $actual_script_name, $actual_dependencies, $wordlift, $in_footer) {
+				wp_register_script($handle, $actual_script_name, $actual_dependencies, $wordlift->get_version(), $in_footer);
 			}
 		);
 

src/wordlift/templates/class-templates-ajax-endpoint.php

Indentation +21 added lines, -21 removed lines

@@ -17,33 +17,33 @@
  */
 class Templates_Ajax_Endpoint {
 
-	/**
-	 * Templates_Ajax_Endpoint constructor.
-	 *
-	 * Hook to `wl_templates` Ajax action.
-	 */
-	public function __construct() {
+    /**
+     * Templates_Ajax_Endpoint constructor.
+     *
+     * Hook to `wl_templates` Ajax action.
+     */
+    public function __construct() {
 
-		add_action( 'wp_ajax_wl_templates', array( $this, 'template' ) );
+        add_action( 'wp_ajax_wl_templates', array( $this, 'template' ) );
 
-	}
+    }
 
-	/**
-	 * Display the requested template. The template is searched in wp-content/wordlift/templates/wordlift-widget-be/
-	 *
-	 * Non alphanumeric names (including `-`) are considered invalid.
-	 */
-	public function template() {
+    /**
+     * Display the requested template. The template is searched in wp-content/wordlift/templates/wordlift-widget-be/
+     *
+     * Non alphanumeric names (including `-`) are considered invalid.
+     */
+    public function template() {
 
-		$name = filter_input( INPUT_GET, 'name' );
+        $name = filter_input( INPUT_GET, 'name' );
 
-		if ( 1 !== preg_match( '|^[a-z0-9\-]+$|', $name ) ) {
-			return wp_send_json_error( 'Invalid name.' );
-		}
+        if ( 1 !== preg_match( '|^[a-z0-9\-]+$|', $name ) ) {
+            return wp_send_json_error( 'Invalid name.' );
+        }
 
-		require dirname( dirname( __DIR__ ) ) . "/templates/wordlift-widget-be/$name.html";
+        require dirname( dirname( __DIR__ ) ) . "/templates/wordlift-widget-be/$name.html";
 
-		die();
-	}
+        die();
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -24,7 +24,7 @@ 
 	 */
 	public function __construct() {
 
-		add_action( 'wp_ajax_wl_templates', array( $this, 'template' ) );
+		add_action('wp_ajax_wl_templates', array($this, 'template'));
 
 	}
 
@@ -35,13 +35,13 @@ 
 	 */
 	public function template() {
 
-		$name = filter_input( INPUT_GET, 'name' );
+		$name = filter_input(INPUT_GET, 'name');
 
-		if ( 1 !== preg_match( '|^[a-z0-9\-]+$|', $name ) ) {
-			return wp_send_json_error( 'Invalid name.' );
+		if (1 !== preg_match('|^[a-z0-9\-]+$|', $name)) {
+			return wp_send_json_error('Invalid name.');
 		}
 
-		require dirname( dirname( __DIR__ ) ) . "/templates/wordlift-widget-be/$name.html";
+		require dirname(dirname(__DIR__))."/templates/wordlift-widget-be/$name.html";
 
 		die();
 	}

src/wordlift/admin/class-key-validation-notice.php

Indentation +94 added lines, -94 removed lines

@@ -11,138 +11,138 @@
  */
 class Key_Validation_Notice {
 
-	const CACHE_KEY = 'is_key_valid';
+    const CACHE_KEY = 'is_key_valid';
 
-	const KEY_VALIDATION_NONCE_ACTION = 'wl_key_validation_notice_nonce';
+    const KEY_VALIDATION_NONCE_ACTION = 'wl_key_validation_notice_nonce';
 
-	const KEY_VALIDATION_NONCE_PARAM = '_wl_key_validation_notice_nonce';
+    const KEY_VALIDATION_NONCE_PARAM = '_wl_key_validation_notice_nonce';
 
-	const KEY_VALIDATION_NOTICE_PARAM = 'wl_key_validation_notice';
+    const KEY_VALIDATION_NOTICE_PARAM = 'wl_key_validation_notice';
 
-	const NOTIFICATION_OPTION_KEY = 'wordlift_key_validation_notification_shown';
+    const NOTIFICATION_OPTION_KEY = 'wordlift_key_validation_notification_shown';
 
-	/**
-	 * @var \Wordlift_Key_Validation_Service
-	 */
-	private $key_validation_service;
+    /**
+     * @var \Wordlift_Key_Validation_Service
+     */
+    private $key_validation_service;
 
-	/**
-	 * @var Ttl_Cache
-	 */
-	private $ttl_cache_service;
+    /**
+     * @var Ttl_Cache
+     */
+    private $ttl_cache_service;
 
-	/**
-	 * @var Wordlift_Configuration_Service
-	 */
-	private $configuration_service;
+    /**
+     * @var Wordlift_Configuration_Service
+     */
+    private $configuration_service;
 
-	/**
-	 * Key_Validation_Notice constructor.
-	 *
-	 * @param \Wordlift_Key_Validation_Service $key_validation_service
-	 * @param Wordlift_Configuration_Service   $configuration_service
-	 */
-	public function __construct( $key_validation_service, $configuration_service ) {
+    /**
+     * Key_Validation_Notice constructor.
+     *
+     * @param \Wordlift_Key_Validation_Service $key_validation_service
+     * @param Wordlift_Configuration_Service   $configuration_service
+     */
+    public function __construct( $key_validation_service, $configuration_service ) {
 
-		$this->key_validation_service = $key_validation_service;
-		$this->configuration_service  = $configuration_service;
+        $this->key_validation_service = $key_validation_service;
+        $this->configuration_service  = $configuration_service;
 
-		$this->ttl_cache_service = new Ttl_Cache( 'key-validation-notification', 60 * 60 * 8 );
+        $this->ttl_cache_service = new Ttl_Cache( 'key-validation-notification', 60 * 60 * 8 );
 
-		if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
-			$this->display_key_validation_notice();
-		}
+        if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
+            $this->display_key_validation_notice();
+        }
 
-		$that = $this;
-		add_action(
-			'plugins_loaded',
-			function () use ( $that ) {
-				$that->notification_close_handler();
-			}
-		);
-	}
+        $that = $this;
+        add_action(
+            'plugins_loaded',
+            function () use ( $that ) {
+                $that->notification_close_handler();
+            }
+        );
+    }
 
-	public function show_notification() {
-		$settings_url = admin_url( 'admin.php?page=wl_configuration_admin_menu' );
-		?>
+    public function show_notification() {
+        $settings_url = admin_url( 'admin.php?page=wl_configuration_admin_menu' );
+        ?>
 		<div class="wl-notice notice is-dismissible error">
 			<p>
 				<?php
-				/* translators: %s: the link to the WordLift configuration screen. */
-				echo wp_kses( sprintf( __( "Your WordLift key is not valid, please update the key in <a href='%s'>WordLift Settings</a> or contact our support at hello@wordlift.io.", 'wordlift' ), esc_attr( $settings_url ) ), array( 'a' => array( 'href' ) ) );
-				?>
+                /* translators: %s: the link to the WordLift configuration screen. */
+                echo wp_kses( sprintf( __( "Your WordLift key is not valid, please update the key in <a href='%s'>WordLift Settings</a> or contact our support at hello@wordlift.io.", 'wordlift' ), esc_attr( $settings_url ) ), array( 'a' => array( 'href' ) ) );
+                ?>
 			</p>
 			<button type="button" class="notice-dismiss"><span class="screen-reader-text">Dismiss this notice.</span>
 			</button>
 		</div>
 		<?php
-	}
+    }
 
-	private function is_key_valid() {
+    private function is_key_valid() {
 
-		$key = $this->configuration_service->get_key();
+        $key = $this->configuration_service->get_key();
 
-		// Check cache if the result is present, if not get the results
-		// save it and return the data.
-		if ( $this->ttl_cache_service->get( self::CACHE_KEY ) !== null ) {
-			return $this->ttl_cache_service->get( self::CACHE_KEY );
-		}
+        // Check cache if the result is present, if not get the results
+        // save it and return the data.
+        if ( $this->ttl_cache_service->get( self::CACHE_KEY ) !== null ) {
+            return $this->ttl_cache_service->get( self::CACHE_KEY );
+        }
 
-		$is_valid = $this->key_validation_service->is_key_valid( $key );
+        $is_valid = $this->key_validation_service->is_key_valid( $key );
 
-		// when the cache is set, clear the notification flag.
-		delete_option( self::NOTIFICATION_OPTION_KEY );
+        // when the cache is set, clear the notification flag.
+        delete_option( self::NOTIFICATION_OPTION_KEY );
 
-		$this->ttl_cache_service->put( self::CACHE_KEY, $is_valid );
+        $this->ttl_cache_service->put( self::CACHE_KEY, $is_valid );
 
-		return $is_valid;
-	}
+        return $is_valid;
+    }
 
-	private function display_key_validation_notice() {
-		$that = $this;
-		add_action(
-			'admin_notices',
-			function () use ( $that ) {
+    private function display_key_validation_notice() {
+        $that = $this;
+        add_action(
+            'admin_notices',
+            function () use ( $that ) {
 
-				$is_notification_shown = get_option( Key_Validation_Notice::NOTIFICATION_OPTION_KEY, false );
+                $is_notification_shown = get_option( Key_Validation_Notice::NOTIFICATION_OPTION_KEY, false );
 
-				$key = $that->configuration_service->get_key();
+                $key = $that->configuration_service->get_key();
 
-				if ( ! $key ) {
-					// Dont show warning or make API call, return early.
-					return;
-				}
+                if ( ! $key ) {
+                    // Dont show warning or make API call, return early.
+                    return;
+                }
 
-				if ( $that->is_key_valid() ) {
-					return;
-				}
+                if ( $that->is_key_valid() ) {
+                    return;
+                }
 
-				if ( $is_notification_shown ) {
-					return;
-				}
+                if ( $is_notification_shown ) {
+                    return;
+                }
 
-				$that->show_notification();
+                $that->show_notification();
 
-			}
-		);
-	}
+            }
+        );
+    }
 
-	public function notification_close_handler() {
-		if ( ! isset( $_GET['wl_key_validation_notice'] )
-			 || ! isset( $_GET['_wl_key_validation_notice_nonce'] ) ) {
-			return false;
-		}
-		$nonce = sanitize_text_field( wp_unslash( (string) $_GET['_wl_key_validation_notice_nonce'] ) );
-		if ( wp_verify_nonce( $nonce, self::KEY_VALIDATION_NONCE_ACTION )
-			 && current_user_can( 'manage_options' ) ) {
-			// close the notification.
-			update_option( self::NOTIFICATION_OPTION_KEY, true );
-		}
-	}
+    public function notification_close_handler() {
+        if ( ! isset( $_GET['wl_key_validation_notice'] )
+             || ! isset( $_GET['_wl_key_validation_notice_nonce'] ) ) {
+            return false;
+        }
+        $nonce = sanitize_text_field( wp_unslash( (string) $_GET['_wl_key_validation_notice_nonce'] ) );
+        if ( wp_verify_nonce( $nonce, self::KEY_VALIDATION_NONCE_ACTION )
+             && current_user_can( 'manage_options' ) ) {
+            // close the notification.
+            update_option( self::NOTIFICATION_OPTION_KEY, true );
+        }
+    }
 
-	public static function remove_notification_flag() {
-		delete_option( self::NOTIFICATION_OPTION_KEY );
-	}
+    public static function remove_notification_flag() {
+        delete_option( self::NOTIFICATION_OPTION_KEY );
+    }
 
 }
 

Spacing +23 added lines, -23 removed lines

@@ -42,34 +42,34 @@ 
 	 * @param \Wordlift_Key_Validation_Service $key_validation_service
 	 * @param Wordlift_Configuration_Service   $configuration_service
 	 */
-	public function __construct( $key_validation_service, $configuration_service ) {
+	public function __construct($key_validation_service, $configuration_service) {
 
 		$this->key_validation_service = $key_validation_service;
 		$this->configuration_service  = $configuration_service;
 
-		$this->ttl_cache_service = new Ttl_Cache( 'key-validation-notification', 60 * 60 * 8 );
+		$this->ttl_cache_service = new Ttl_Cache('key-validation-notification', 60 * 60 * 8);
 
-		if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
+		if (apply_filters('wl_feature__enable__notices', true)) {
 			$this->display_key_validation_notice();
 		}
 
 		$that = $this;
 		add_action(
 			'plugins_loaded',
-			function () use ( $that ) {
+			function() use ($that) {
 				$that->notification_close_handler();
 			}
 		);
 	}
 
 	public function show_notification() {
-		$settings_url = admin_url( 'admin.php?page=wl_configuration_admin_menu' );
+		$settings_url = admin_url('admin.php?page=wl_configuration_admin_menu');
 		?>
 		<div class="wl-notice notice is-dismissible error">
 			<p>
 				<?php
 				/* translators: %s: the link to the WordLift configuration screen. */
-				echo wp_kses( sprintf( __( "Your WordLift key is not valid, please update the key in <a href='%s'>WordLift Settings</a> or contact our support at hello@wordlift.io.", 'wordlift' ), esc_attr( $settings_url ) ), array( 'a' => array( 'href' ) ) );
+				echo wp_kses(sprintf(__("Your WordLift key is not valid, please update the key in <a href='%s'>WordLift Settings</a> or contact our support at hello@wordlift.io.", 'wordlift'), esc_attr($settings_url)), array('a' => array('href')));
 				?>
 			</p>
 			<button type="button" class="notice-dismiss"><span class="screen-reader-text">Dismiss this notice.</span>
@@ -84,16 +84,16 @@ 
 
 		// Check cache if the result is present, if not get the results
 		// save it and return the data.
-		if ( $this->ttl_cache_service->get( self::CACHE_KEY ) !== null ) {
-			return $this->ttl_cache_service->get( self::CACHE_KEY );
+		if ($this->ttl_cache_service->get(self::CACHE_KEY) !== null) {
+			return $this->ttl_cache_service->get(self::CACHE_KEY);
 		}
 
-		$is_valid = $this->key_validation_service->is_key_valid( $key );
+		$is_valid = $this->key_validation_service->is_key_valid($key);
 
 		// when the cache is set, clear the notification flag.
-		delete_option( self::NOTIFICATION_OPTION_KEY );
+		delete_option(self::NOTIFICATION_OPTION_KEY);
 
-		$this->ttl_cache_service->put( self::CACHE_KEY, $is_valid );
+		$this->ttl_cache_service->put(self::CACHE_KEY, $is_valid);
 
 		return $is_valid;
 	}
@@ -102,22 +102,22 @@ 
 		$that = $this;
 		add_action(
 			'admin_notices',
-			function () use ( $that ) {
+			function() use ($that) {
 
-				$is_notification_shown = get_option( Key_Validation_Notice::NOTIFICATION_OPTION_KEY, false );
+				$is_notification_shown = get_option(Key_Validation_Notice::NOTIFICATION_OPTION_KEY, false);
 
 				$key = $that->configuration_service->get_key();
 
-				if ( ! $key ) {
+				if ( ! $key) {
 					// Dont show warning or make API call, return early.
 					return;
 				}
 
-				if ( $that->is_key_valid() ) {
+				if ($that->is_key_valid()) {
 					return;
 				}
 
-				if ( $is_notification_shown ) {
+				if ($is_notification_shown) {
 					return;
 				}
 
@@ -128,20 +128,20 @@ 
 	}
 
 	public function notification_close_handler() {
-		if ( ! isset( $_GET['wl_key_validation_notice'] )
-			 || ! isset( $_GET['_wl_key_validation_notice_nonce'] ) ) {
+		if ( ! isset($_GET['wl_key_validation_notice'])
+			 || ! isset($_GET['_wl_key_validation_notice_nonce'])) {
 			return false;
 		}
-		$nonce = sanitize_text_field( wp_unslash( (string) $_GET['_wl_key_validation_notice_nonce'] ) );
-		if ( wp_verify_nonce( $nonce, self::KEY_VALIDATION_NONCE_ACTION )
-			 && current_user_can( 'manage_options' ) ) {
+		$nonce = sanitize_text_field(wp_unslash((string) $_GET['_wl_key_validation_notice_nonce']));
+		if (wp_verify_nonce($nonce, self::KEY_VALIDATION_NONCE_ACTION)
+			 && current_user_can('manage_options')) {
 			// close the notification.
-			update_option( self::NOTIFICATION_OPTION_KEY, true );
+			update_option(self::NOTIFICATION_OPTION_KEY, true);
 		}
 	}
 
 	public static function remove_notification_flag() {
-		delete_option( self::NOTIFICATION_OPTION_KEY );
+		delete_option(self::NOTIFICATION_OPTION_KEY);
 	}
 
 }

src/wordlift/admin/class-top-entities.php

Indentation +24 added lines, -24 removed lines

@@ -8,23 +8,23 @@ 
  */
 class Top_Entities {
 
-	const CRON_ACTION = 'wl_admin_dashboard_top_entities';
+    const CRON_ACTION = 'wl_admin_dashboard_top_entities';
 
-	/**
-	 * Option key where the top entities data is stored.
-	 */
-	const OPTION_KEY = 'wl_admin_dashboard_top_entities_option';
+    /**
+     * Option key where the top entities data is stored.
+     */
+    const OPTION_KEY = 'wl_admin_dashboard_top_entities_option';
 
-	public function __construct() {
-		add_action( self::CRON_ACTION, array( $this, 'save_top_entities' ) );
-	}
+    public function __construct() {
+        add_action( self::CRON_ACTION, array( $this, 'save_top_entities' ) );
+    }
 
-	public function save_top_entities() {
+    public function save_top_entities() {
 
-		global $wpdb;
+        global $wpdb;
 
-		$results = $wpdb->get_results(
-			"
+        $results = $wpdb->get_results(
+            "
 			SELECT p_as_object.ID
 			    , p_as_object.post_title
 			    , COALESCE(SUM(CASE WHEN t_as_subject.slug IS NULL THEN 1 END), 0)     entities
@@ -49,20 +49,20 @@ 
 			ORDER BY total DESC
 			LIMIT 20
 		"
-		);
+        );
 
-		update_option( self::OPTION_KEY, $results );
-	}
+        update_option( self::OPTION_KEY, $results );
+    }
 
-	public static function activate() {
-		if ( ! wp_next_scheduled( self::CRON_ACTION ) ) {
-			wp_schedule_event( time(), 'daily', self::CRON_ACTION );
-		}
-	}
+    public static function activate() {
+        if ( ! wp_next_scheduled( self::CRON_ACTION ) ) {
+            wp_schedule_event( time(), 'daily', self::CRON_ACTION );
+        }
+    }
 
-	public static function deactivate() {
-		$timestamp = wp_next_scheduled( self::CRON_ACTION );
-		wp_unschedule_event( $timestamp, self::CRON_ACTION );
-	}
+    public static function deactivate() {
+        $timestamp = wp_next_scheduled( self::CRON_ACTION );
+        wp_unschedule_event( $timestamp, self::CRON_ACTION );
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -16,7 +16,7 @@ 
 	const OPTION_KEY = 'wl_admin_dashboard_top_entities_option';
 
 	public function __construct() {
-		add_action( self::CRON_ACTION, array( $this, 'save_top_entities' ) );
+		add_action(self::CRON_ACTION, array($this, 'save_top_entities'));
 	}
 
 	public function save_top_entities() {
@@ -51,18 +51,18 @@ 
 		"
 		);
 
-		update_option( self::OPTION_KEY, $results );
+		update_option(self::OPTION_KEY, $results);
 	}
 
 	public static function activate() {
-		if ( ! wp_next_scheduled( self::CRON_ACTION ) ) {
-			wp_schedule_event( time(), 'daily', self::CRON_ACTION );
+		if ( ! wp_next_scheduled(self::CRON_ACTION)) {
+			wp_schedule_event(time(), 'daily', self::CRON_ACTION);
 		}
 	}
 
 	public static function deactivate() {
-		$timestamp = wp_next_scheduled( self::CRON_ACTION );
-		wp_unschedule_event( $timestamp, self::CRON_ACTION );
+		$timestamp = wp_next_scheduled(self::CRON_ACTION);
+		wp_unschedule_event($timestamp, self::CRON_ACTION);
 	}
 
 }