insideout10 / wordlift-plugin

src/wordlift/tasks/class-task-single-instance-task-runner.php

Indentation +191 added lines, -191 removed lines

@@ -24,196 +24,196 @@
  */
 class Task_Single_Instance_Task_Runner {
 
-	/**
-	 * Define the transient prefix.
-	 *
-	 * @since 1.0.0
-	 */
-	const IS_RUNNING_PREFIX = '_wf_task_runner__';
-
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since 1.0.0
-	 * @var Wordlift_Log_Service A {@link Wordlift_Log_Service} instance.
-	 * @access private
-	 */
-	private $log;
-
-	/**
-	 * The {@link Task} to execute.
-	 *
-	 * @since 1.0.0
-	 * @var Task $task The {@link Task} to execute.
-	 * @access private
-	 */
-	private $task;
-
-	/**
-	 * One or more callbacks to call to update about the task progress.
-	 *
-	 * @since 1.0.0
-	 * @var Task_Progress[] $callbacks An array of {@link Wordlift_For_Bungalowparkoverzicht_Progress}.
-	 * @access private
-	 */
-	private $callbacks;
-
-	/**
-	 * Whether to force starting a task even if another instance of the task is already running.
-	 *
-	 * @since 1.0.0
-	 * @var bool $force Whether to force starting a task even if another instance of the task is already running.
-	 * @access private
-	 */
-	private $force;
-
-	/**
-	 * Create a {@link Task_Single_Instance_Task_Runner} instance.
-	 *
-	 * @param Task  $task The {@link Task} instance.
-	 * @param bool  $force Whether to force starting a task even if another instance of the task is already running, default `false`.
-	 * @param array $callbacks An array of {@link Wordlift_For_Bungalowparkoverzicht_Progress}.
-	 *
-	 * @since 1.0.0
-	 */
-	public function __construct( $task, $force = false, $callbacks = array() ) {
-
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
-
-		$this->task      = $task;
-		$this->force     = $force;
-		$this->callbacks = $callbacks;
-
-	}
-
-	/**
-	 * Get the transient name for running flag.
-	 *
-	 * @return string The transient name.
-	 * @since 1.0.0
-	 */
-	private function get_running_transient() {
-
-		return self::IS_RUNNING_PREFIX . $this->task->get_id();
-	}
-
-	/**
-	 * Check whether a task is running.
-	 *
-	 * @return bool
-	 * @since 1.0.0
-	 */
-	public function is_running() {
-		return 'yes' === get_transient( $this->get_running_transient() );
-	}
-
-	/**
-	 * Set whether the task is running or not.
-	 *
-	 * @param bool $value Whether the task is running or not.
-	 *
-	 * @since 1.0.0
-	 */
-	public function set_running( $value ) {
-		set_transient( $this->get_running_transient(), $value ? 'yes' : 'no' );
-	}
-
-	/**
-	 * Start the task.
-	 *
-	 * @param int $limit The maximum number of items to process.
-	 * @param int $offset The starting offset.
-	 *
-	 * @throws Task_Another_Instance_Is_Running_Exception if the task is already running.
-	 * @since 1.0.0
-	 */
-	public function start( $limit = 0, $offset = 0 ) {
-
-		// Bail out if the task is already running.
-		if ( ! $this->force && $this->is_running() ) {
-			throw new Task_Another_Instance_Is_Running_Exception();
-		}
-
-		// Set the task as running.
-		$this->set_running( true );
-
-		// List the chunk of elements to process.
-		$items = $this->task->list_items( $limit, $offset );
-
-		$count = count( $items );
-		for ( $i = 0; $i < $count; $i ++ ) {
-			// Process the item.
-			$this->task->process_item( $items[ $i ] );
-
-			// Update the progress.
-			$this->set_progress( $offset + $i, $items[ $i ] );
-		}
-
-		// Set the total number of elements to process.
-		$this->set_count( $this->task->count_items() );
-
-		// Unset the running flag.
-		$this->set_running( false );
-
-		// Set the task to complete.
-		$this->finish();
-
-	}
-
-	/**
-	 * Set the total number of items to process.
-	 *
-	 * @param int $value The total number of items to process.
-	 *
-	 * @since 1.0.0
-	 */
-	private function set_count( $value ) {
-
-		if ( empty( $this->callbacks ) ) {
-			return;
-		}
-
-		foreach ( $this->callbacks as $callback ) {
-			call_user_func( array( $callback, 'set_count' ), $value );
-		}
-
-	}
-
-	/**
-	 * Set the task progress.
-	 *
-	 * @param int   $index The current item index.
-	 * @param mixed $item The current item.
-	 *
-	 * @since 1.0.0
-	 */
-	private function set_progress( $index, $item ) {
-
-		if ( empty( $this->callbacks ) ) {
-			return;
-		}
-
-		foreach ( $this->callbacks as $callback ) {
-			call_user_func( array( $callback, 'set_progress' ), $index, $item );
-		}
-
-	}
-
-	/**
-	 * Inform the callbacks that the task completed.
-	 *
-	 * @since 1.0.0
-	 */
-	private function finish() {
-
-		if ( empty( $this->callbacks ) ) {
-			return;
-		}
-
-		foreach ( $this->callbacks as $callback ) {
-			call_user_func( array( $callback, 'finish' ) );
-		}
-
-	}
+    /**
+     * Define the transient prefix.
+     *
+     * @since 1.0.0
+     */
+    const IS_RUNNING_PREFIX = '_wf_task_runner__';
+
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since 1.0.0
+     * @var Wordlift_Log_Service A {@link Wordlift_Log_Service} instance.
+     * @access private
+     */
+    private $log;
+
+    /**
+     * The {@link Task} to execute.
+     *
+     * @since 1.0.0
+     * @var Task $task The {@link Task} to execute.
+     * @access private
+     */
+    private $task;
+
+    /**
+     * One or more callbacks to call to update about the task progress.
+     *
+     * @since 1.0.0
+     * @var Task_Progress[] $callbacks An array of {@link Wordlift_For_Bungalowparkoverzicht_Progress}.
+     * @access private
+     */
+    private $callbacks;
+
+    /**
+     * Whether to force starting a task even if another instance of the task is already running.
+     *
+     * @since 1.0.0
+     * @var bool $force Whether to force starting a task even if another instance of the task is already running.
+     * @access private
+     */
+    private $force;
+
+    /**
+     * Create a {@link Task_Single_Instance_Task_Runner} instance.
+     *
+     * @param Task  $task The {@link Task} instance.
+     * @param bool  $force Whether to force starting a task even if another instance of the task is already running, default `false`.
+     * @param array $callbacks An array of {@link Wordlift_For_Bungalowparkoverzicht_Progress}.
+     *
+     * @since 1.0.0
+     */
+    public function __construct( $task, $force = false, $callbacks = array() ) {
+
+        $this->log = Wordlift_Log_Service::get_logger( get_class() );
+
+        $this->task      = $task;
+        $this->force     = $force;
+        $this->callbacks = $callbacks;
+
+    }
+
+    /**
+     * Get the transient name for running flag.
+     *
+     * @return string The transient name.
+     * @since 1.0.0
+     */
+    private function get_running_transient() {
+
+        return self::IS_RUNNING_PREFIX . $this->task->get_id();
+    }
+
+    /**
+     * Check whether a task is running.
+     *
+     * @return bool
+     * @since 1.0.0
+     */
+    public function is_running() {
+        return 'yes' === get_transient( $this->get_running_transient() );
+    }
+
+    /**
+     * Set whether the task is running or not.
+     *
+     * @param bool $value Whether the task is running or not.
+     *
+     * @since 1.0.0
+     */
+    public function set_running( $value ) {
+        set_transient( $this->get_running_transient(), $value ? 'yes' : 'no' );
+    }
+
+    /**
+     * Start the task.
+     *
+     * @param int $limit The maximum number of items to process.
+     * @param int $offset The starting offset.
+     *
+     * @throws Task_Another_Instance_Is_Running_Exception if the task is already running.
+     * @since 1.0.0
+     */
+    public function start( $limit = 0, $offset = 0 ) {
+
+        // Bail out if the task is already running.
+        if ( ! $this->force && $this->is_running() ) {
+            throw new Task_Another_Instance_Is_Running_Exception();
+        }
+
+        // Set the task as running.
+        $this->set_running( true );
+
+        // List the chunk of elements to process.
+        $items = $this->task->list_items( $limit, $offset );
+
+        $count = count( $items );
+        for ( $i = 0; $i < $count; $i ++ ) {
+            // Process the item.
+            $this->task->process_item( $items[ $i ] );
+
+            // Update the progress.
+            $this->set_progress( $offset + $i, $items[ $i ] );
+        }
+
+        // Set the total number of elements to process.
+        $this->set_count( $this->task->count_items() );
+
+        // Unset the running flag.
+        $this->set_running( false );
+
+        // Set the task to complete.
+        $this->finish();
+
+    }
+
+    /**
+     * Set the total number of items to process.
+     *
+     * @param int $value The total number of items to process.
+     *
+     * @since 1.0.0
+     */
+    private function set_count( $value ) {
+
+        if ( empty( $this->callbacks ) ) {
+            return;
+        }
+
+        foreach ( $this->callbacks as $callback ) {
+            call_user_func( array( $callback, 'set_count' ), $value );
+        }
+
+    }
+
+    /**
+     * Set the task progress.
+     *
+     * @param int   $index The current item index.
+     * @param mixed $item The current item.
+     *
+     * @since 1.0.0
+     */
+    private function set_progress( $index, $item ) {
+
+        if ( empty( $this->callbacks ) ) {
+            return;
+        }
+
+        foreach ( $this->callbacks as $callback ) {
+            call_user_func( array( $callback, 'set_progress' ), $index, $item );
+        }
+
+    }
+
+    /**
+     * Inform the callbacks that the task completed.
+     *
+     * @since 1.0.0
+     */
+    private function finish() {
+
+        if ( empty( $this->callbacks ) ) {
+            return;
+        }
+
+        foreach ( $this->callbacks as $callback ) {
+            call_user_func( array( $callback, 'finish' ) );
+        }
+
+    }
 
 }

Spacing +27 added lines, -27 removed lines

@@ -76,9 +76,9 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	public function __construct( $task, $force = false, $callbacks = array() ) {
+	public function __construct($task, $force = false, $callbacks = array()) {
 
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
+		$this->log = Wordlift_Log_Service::get_logger(get_class());
 
 		$this->task      = $task;
 		$this->force     = $force;
@@ -94,7 +94,7 @@ 
 	 */
 	private function get_running_transient() {
 
-		return self::IS_RUNNING_PREFIX . $this->task->get_id();
+		return self::IS_RUNNING_PREFIX.$this->task->get_id();
 	}
 
 	/**
@@ -104,7 +104,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function is_running() {
-		return 'yes' === get_transient( $this->get_running_transient() );
+		return 'yes' === get_transient($this->get_running_transient());
 	}
 
 	/**
@@ -114,8 +114,8 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	public function set_running( $value ) {
-		set_transient( $this->get_running_transient(), $value ? 'yes' : 'no' );
+	public function set_running($value) {
+		set_transient($this->get_running_transient(), $value ? 'yes' : 'no');
 	}
 
 	/**
@@ -127,33 +127,33 @@ 
 	 * @throws Task_Another_Instance_Is_Running_Exception if the task is already running.
 	 * @since 1.0.0
 	 */
-	public function start( $limit = 0, $offset = 0 ) {
+	public function start($limit = 0, $offset = 0) {
 
 		// Bail out if the task is already running.
-		if ( ! $this->force && $this->is_running() ) {
+		if ( ! $this->force && $this->is_running()) {
 			throw new Task_Another_Instance_Is_Running_Exception();
 		}
 
 		// Set the task as running.
-		$this->set_running( true );
+		$this->set_running(true);
 
 		// List the chunk of elements to process.
-		$items = $this->task->list_items( $limit, $offset );
+		$items = $this->task->list_items($limit, $offset);
 
-		$count = count( $items );
-		for ( $i = 0; $i < $count; $i ++ ) {
+		$count = count($items);
+		for ($i = 0; $i < $count; $i++) {
 			// Process the item.
-			$this->task->process_item( $items[ $i ] );
+			$this->task->process_item($items[$i]);
 
 			// Update the progress.
-			$this->set_progress( $offset + $i, $items[ $i ] );
+			$this->set_progress($offset + $i, $items[$i]);
 		}
 
 		// Set the total number of elements to process.
-		$this->set_count( $this->task->count_items() );
+		$this->set_count($this->task->count_items());
 
 		// Unset the running flag.
-		$this->set_running( false );
+		$this->set_running(false);
 
 		// Set the task to complete.
 		$this->finish();
@@ -167,14 +167,14 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	private function set_count( $value ) {
+	private function set_count($value) {
 
-		if ( empty( $this->callbacks ) ) {
+		if (empty($this->callbacks)) {
 			return;
 		}
 
-		foreach ( $this->callbacks as $callback ) {
-			call_user_func( array( $callback, 'set_count' ), $value );
+		foreach ($this->callbacks as $callback) {
+			call_user_func(array($callback, 'set_count'), $value);
 		}
 
 	}
@@ -187,14 +187,14 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	private function set_progress( $index, $item ) {
+	private function set_progress($index, $item) {
 
-		if ( empty( $this->callbacks ) ) {
+		if (empty($this->callbacks)) {
 			return;
 		}
 
-		foreach ( $this->callbacks as $callback ) {
-			call_user_func( array( $callback, 'set_progress' ), $index, $item );
+		foreach ($this->callbacks as $callback) {
+			call_user_func(array($callback, 'set_progress'), $index, $item);
 		}
 
 	}
@@ -206,12 +206,12 @@ 
 	 */
 	private function finish() {
 
-		if ( empty( $this->callbacks ) ) {
+		if (empty($this->callbacks)) {
 			return;
 		}
 
-		foreach ( $this->callbacks as $callback ) {
-			call_user_func( array( $callback, 'finish' ) );
+		foreach ($this->callbacks as $callback) {
+			call_user_func(array($callback, 'finish'));
 		}
 
 	}

src/wordlift/tasks/admin/class-tasks-page-base.php

Indentation +63 added lines, -63 removed lines

@@ -17,77 +17,77 @@
 
 abstract class Tasks_Page_Base extends Submenu_Page_Base {
 
-	/**
-	 * The ID of this admin page.
-	 *
-	 * @since    1.0.0
-	 * @access   private
-	 * @var      string $menu_slug The ID of this page.
-	 */
-	private $menu_slug;
+    /**
+     * The ID of this admin page.
+     *
+     * @since    1.0.0
+     * @access   private
+     * @var      string $menu_slug The ID of this page.
+     */
+    private $menu_slug;
 
-	/**
-	 * @var Task_Ajax_Adapters_Registry
-	 */
-	private $task_ajax_adapters_registry;
+    /**
+     * @var Task_Ajax_Adapters_Registry
+     */
+    private $task_ajax_adapters_registry;
 
-	/**
-	 * @var string
-	 */
-	private $version;
+    /**
+     * @var string
+     */
+    private $version;
 
-	/**
-	 * Define the {@link Wordlift_Admin_Page} constructor.
-	 *
-	 * @param Task_Ajax_Adapters_Registry $task_ajax_adapters_registry
-	 *
-	 * @param string                      $version
-	 * @param string                      $menu_slug
-	 * @param string                      $page_title
-	 * @param string                      $capability
-	 * @param string|null                 $parent_slug
-	 * @param string|null                 $menu_title
-	 *
-	 * @since 1.0.0
-	 */
-	public function __construct( $task_ajax_adapters_registry, $version, $menu_slug, $page_title, $capability = 'manage_options', $parent_slug = null, $menu_title = null ) {
-		parent::__construct( $menu_slug, $page_title, $capability, $parent_slug, $menu_title );
+    /**
+     * Define the {@link Wordlift_Admin_Page} constructor.
+     *
+     * @param Task_Ajax_Adapters_Registry $task_ajax_adapters_registry
+     *
+     * @param string                      $version
+     * @param string                      $menu_slug
+     * @param string                      $page_title
+     * @param string                      $capability
+     * @param string|null                 $parent_slug
+     * @param string|null                 $menu_title
+     *
+     * @since 1.0.0
+     */
+    public function __construct( $task_ajax_adapters_registry, $version, $menu_slug, $page_title, $capability = 'manage_options', $parent_slug = null, $menu_title = null ) {
+        parent::__construct( $menu_slug, $page_title, $capability, $parent_slug, $menu_title );
 
-		$this->task_ajax_adapters_registry = $task_ajax_adapters_registry;
-		$this->version                     = $version;
-		$this->menu_slug                   = $menu_slug;
+        $this->task_ajax_adapters_registry = $task_ajax_adapters_registry;
+        $this->version                     = $version;
+        $this->menu_slug                   = $menu_slug;
 
-	}
+    }
 
-	/**
-	 * Register the stylesheets and scripts for the admin area.
-	 *
-	 * @since    1.0.0
-	 */
-	public function enqueue_scripts() {
-		wp_enqueue_style( $this->menu_slug, plugin_dir_url( __FILE__ ) . 'assets/tasks-page.css', array(), $this->version, 'all' );
-		wp_enqueue_script(
-			$this->menu_slug,
-			plugin_dir_url( __FILE__ ) . 'assets/tasks-page.js',
-			array(
-				'jquery',
-				'wp-util',
-			),
-			$this->version,
-			true
-		);
-	}
+    /**
+     * Register the stylesheets and scripts for the admin area.
+     *
+     * @since    1.0.0
+     */
+    public function enqueue_scripts() {
+        wp_enqueue_style( $this->menu_slug, plugin_dir_url( __FILE__ ) . 'assets/tasks-page.css', array(), $this->version, 'all' );
+        wp_enqueue_script(
+            $this->menu_slug,
+            plugin_dir_url( __FILE__ ) . 'assets/tasks-page.js',
+            array(
+                'jquery',
+                'wp-util',
+            ),
+            $this->version,
+            true
+        );
+    }
 
-	/**
-	 * Render the page.
-	 *
-	 * @since 1.0.0
-	 */
-	public function render() {
+    /**
+     * Render the page.
+     *
+     * @since 1.0.0
+     */
+    public function render() {
 
-		// Include the partial.
-		include plugin_dir_path( __FILE__ ) . 'assets/tasks-page.php';
+        // Include the partial.
+        include plugin_dir_path( __FILE__ ) . 'assets/tasks-page.php';
 
-	}
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -50,8 +50,8 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	public function __construct( $task_ajax_adapters_registry, $version, $menu_slug, $page_title, $capability = 'manage_options', $parent_slug = null, $menu_title = null ) {
-		parent::__construct( $menu_slug, $page_title, $capability, $parent_slug, $menu_title );
+	public function __construct($task_ajax_adapters_registry, $version, $menu_slug, $page_title, $capability = 'manage_options', $parent_slug = null, $menu_title = null) {
+		parent::__construct($menu_slug, $page_title, $capability, $parent_slug, $menu_title);
 
 		$this->task_ajax_adapters_registry = $task_ajax_adapters_registry;
 		$this->version                     = $version;
@@ -65,10 +65,10 @@ 
 	 * @since    1.0.0
 	 */
 	public function enqueue_scripts() {
-		wp_enqueue_style( $this->menu_slug, plugin_dir_url( __FILE__ ) . 'assets/tasks-page.css', array(), $this->version, 'all' );
+		wp_enqueue_style($this->menu_slug, plugin_dir_url(__FILE__).'assets/tasks-page.css', array(), $this->version, 'all');
 		wp_enqueue_script(
 			$this->menu_slug,
-			plugin_dir_url( __FILE__ ) . 'assets/tasks-page.js',
+			plugin_dir_url(__FILE__).'assets/tasks-page.js',
 			array(
 				'jquery',
 				'wp-util',
@@ -86,7 +86,7 @@ 
 	public function render() {
 
 		// Include the partial.
-		include plugin_dir_path( __FILE__ ) . 'assets/tasks-page.php';
+		include plugin_dir_path(__FILE__).'assets/tasks-page.php';
 
 	}
 

src/wordlift/tasks/admin/assets/tasks-page.php

Indentation +5 added lines, -5 removed lines

@@ -14,9 +14,9 @@ 
 
 	<?php
     // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
-	foreach ( $this->task_ajax_adapters_registry->get_task_ajax_adapters() as $task_ajax_adapter ) {
-		$task = $task_ajax_adapter->get_task();
-		?>
+    foreach ( $this->task_ajax_adapters_registry->get_task_ajax_adapters() as $task_ajax_adapter ) {
+        $task = $task_ajax_adapter->get_task();
+        ?>
 		<div class="wl-task">
 			<h2><?php esc_html( $task->get_label() ); ?></h2>
 			<div class="wl-task__progress" style="border: 1px solid #23282D; height: 20px; margin: 8px 0;">
@@ -33,7 +33,7 @@ 
 
 		</div>
 		<?php
-	}
-	?>
+    }
+    ?>
 
 </div>

Spacing +6 added lines, -6 removed lines

@@ -10,15 +10,15 @@ 
 
 ?>
 <div class="wrap">
-	<h1><?php esc_html_e( 'Tasks', 'wordlift' ); ?></h1>
+	<h1><?php esc_html_e('Tasks', 'wordlift'); ?></h1>
 
 	<?php
     // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
-	foreach ( $this->task_ajax_adapters_registry->get_task_ajax_adapters() as $task_ajax_adapter ) {
+	foreach ($this->task_ajax_adapters_registry->get_task_ajax_adapters() as $task_ajax_adapter) {
 		$task = $task_ajax_adapter->get_task();
 		?>
 		<div class="wl-task">
-			<h2><?php esc_html( $task->get_label() ); ?></h2>
+			<h2><?php esc_html($task->get_label()); ?></h2>
 			<div class="wl-task__progress" style="border: 1px solid #23282D; height: 20px; margin: 8px 0;">
 				<div class="wl-task__progress__bar"
 					 style="width:0;background: #0073AA; text-align: center; height: 100%; color: #fff;"></div>
@@ -27,9 +27,9 @@ 
 			<button
 				type="button"
 				class="button button-large button-primary"
-				data-action="<?php echo esc_attr( $task->get_id() ); ?>"
-				data-nonce="<?php echo esc_attr( wp_create_nonce( $task->get_id() ) ); ?>"
-			><?php esc_html_e( 'Start', 'wordlift' ); ?></button>
+				data-action="<?php echo esc_attr($task->get_id()); ?>"
+				data-nonce="<?php echo esc_attr(wp_create_nonce($task->get_id())); ?>"
+			><?php esc_html_e('Start', 'wordlift'); ?></button>
 
 		</div>
 		<?php

src/wordlift/tasks/admin/class-tasks-page.php

Indentation +58 added lines, -58 removed lines

@@ -17,72 +17,72 @@
 
 class Tasks_Page extends Submenu_Page_Base {
 
-	/**
-	 * The ID of this admin page.
-	 *
-	 * @since    1.0.0
-	 * @access   private
-	 * @var      string $menu_slug The ID of this page.
-	 */
-	private $menu_slug = 'wl_tasks_page';
+    /**
+     * The ID of this admin page.
+     *
+     * @since    1.0.0
+     * @access   private
+     * @var      string $menu_slug The ID of this page.
+     */
+    private $menu_slug = 'wl_tasks_page';
 
-	/**
-	 * Used when enqueueing styles or scripts as the version string.
-	 *
-	 * @since  1.0.0
-	 * @access private
-	 * @var    string
-	 */
-	private $asset_version = '1.0.0';
+    /**
+     * Used when enqueueing styles or scripts as the version string.
+     *
+     * @since  1.0.0
+     * @access private
+     * @var    string
+     */
+    private $asset_version = '1.0.0';
 
-	/**
-	 * @var Task_Ajax_Adapters_Registry
-	 */
-	private $task_ajax_adapters_registry;
+    /**
+     * @var Task_Ajax_Adapters_Registry
+     */
+    private $task_ajax_adapters_registry;
 
-	/**
-	 * Define the {@link Wordlift_Admin_Page} constructor.
-	 *
-	 * @param Task_Ajax_Adapters_Registry $task_ajax_adapters_registry
-	 *
-	 * @since 1.0.0
-	 */
-	public function __construct( $task_ajax_adapters_registry ) {
-		parent::__construct( $this->menu_slug, __( 'Tasks', 'wordlift' ), 'manage_options', 'wl_admin_menu', __( 'Tasks', 'wordlift' ) );
+    /**
+     * Define the {@link Wordlift_Admin_Page} constructor.
+     *
+     * @param Task_Ajax_Adapters_Registry $task_ajax_adapters_registry
+     *
+     * @since 1.0.0
+     */
+    public function __construct( $task_ajax_adapters_registry ) {
+        parent::__construct( $this->menu_slug, __( 'Tasks', 'wordlift' ), 'manage_options', 'wl_admin_menu', __( 'Tasks', 'wordlift' ) );
 
-		$this->task_ajax_adapters_registry = $task_ajax_adapters_registry;
+        $this->task_ajax_adapters_registry = $task_ajax_adapters_registry;
 
-	}
+    }
 
-	/**
-	 * Register the stylesheets and scripts for the admin area.
-	 *
-	 * @since    1.0.0
-	 */
-	public function enqueue_scripts() {
-		wp_enqueue_style( $this->menu_slug, plugin_dir_url( __FILE__ ) . 'assets/tasks-page.css', array(), $this->asset_version, 'all' );
-		wp_enqueue_script(
-			$this->menu_slug,
-			plugin_dir_url( __FILE__ ) . 'assets/tasks-page.js',
-			array(
-				'jquery',
-				'wp-util',
-			),
-			$this->asset_version,
-			true
-		);
-	}
+    /**
+     * Register the stylesheets and scripts for the admin area.
+     *
+     * @since    1.0.0
+     */
+    public function enqueue_scripts() {
+        wp_enqueue_style( $this->menu_slug, plugin_dir_url( __FILE__ ) . 'assets/tasks-page.css', array(), $this->asset_version, 'all' );
+        wp_enqueue_script(
+            $this->menu_slug,
+            plugin_dir_url( __FILE__ ) . 'assets/tasks-page.js',
+            array(
+                'jquery',
+                'wp-util',
+            ),
+            $this->asset_version,
+            true
+        );
+    }
 
-	/**
-	 * Render the page.
-	 *
-	 * @since 1.0.0
-	 */
-	public function render() {
+    /**
+     * Render the page.
+     *
+     * @since 1.0.0
+     */
+    public function render() {
 
-		// Include the partial.
-		include plugin_dir_path( __FILE__ ) . 'assets/tasks-page.php';
+        // Include the partial.
+        include plugin_dir_path( __FILE__ ) . 'assets/tasks-page.php';
 
-	}
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -47,8 +47,8 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	public function __construct( $task_ajax_adapters_registry ) {
-		parent::__construct( $this->menu_slug, __( 'Tasks', 'wordlift' ), 'manage_options', 'wl_admin_menu', __( 'Tasks', 'wordlift' ) );
+	public function __construct($task_ajax_adapters_registry) {
+		parent::__construct($this->menu_slug, __('Tasks', 'wordlift'), 'manage_options', 'wl_admin_menu', __('Tasks', 'wordlift'));
 
 		$this->task_ajax_adapters_registry = $task_ajax_adapters_registry;
 
@@ -60,10 +60,10 @@ 
 	 * @since    1.0.0
 	 */
 	public function enqueue_scripts() {
-		wp_enqueue_style( $this->menu_slug, plugin_dir_url( __FILE__ ) . 'assets/tasks-page.css', array(), $this->asset_version, 'all' );
+		wp_enqueue_style($this->menu_slug, plugin_dir_url(__FILE__).'assets/tasks-page.css', array(), $this->asset_version, 'all');
 		wp_enqueue_script(
 			$this->menu_slug,
-			plugin_dir_url( __FILE__ ) . 'assets/tasks-page.js',
+			plugin_dir_url(__FILE__).'assets/tasks-page.js',
 			array(
 				'jquery',
 				'wp-util',
@@ -81,7 +81,7 @@ 
 	public function render() {
 
 		// Include the partial.
-		include plugin_dir_path( __FILE__ ) . 'assets/tasks-page.php';
+		include plugin_dir_path(__FILE__).'assets/tasks-page.php';
 
 	}
 

src/wordlift/tasks/class-task-runner.php

Indentation +10 added lines, -10 removed lines

@@ -15,15 +15,15 @@
  */
 interface Task_Runner {
 
-	/**
-	 * Start the task.
-	 *
-	 * @param int $limit The maximum number of items to process.
-	 * @param int $offset The starting offset (zero-based).
-	 *
-	 * @return mixed
-	 * @since 1.0.0
-	 */
-	public function start( $limit = 0, $offset = 0 );
+    /**
+     * Start the task.
+     *
+     * @param int $limit The maximum number of items to process.
+     * @param int $offset The starting offset (zero-based).
+     *
+     * @return mixed
+     * @since 1.0.0
+     */
+    public function start( $limit = 0, $offset = 0 );
 
 }

Spacing +1 added lines, -1 removed lines

@@ -24,6 +24,6 @@
 	 * @return mixed
 	 * @since 1.0.0
 	 */
-	public function start( $limit = 0, $offset = 0 );
+	public function start($limit = 0, $offset = 0);
 
 }

src/wordlift/tasks/class-task-progress.php

Indentation +23 added lines, -23 removed lines

@@ -16,30 +16,30 @@
  */
 interface Task_Progress {
 
-	/**
-	 * The total number of elements to process.
-	 *
-	 * @param int $value The total number of elements to process.
-	 *
-	 * @since 1.0.0
-	 */
-	public function set_count( $value );
+    /**
+     * The total number of elements to process.
+     *
+     * @param int $value The total number of elements to process.
+     *
+     * @since 1.0.0
+     */
+    public function set_count( $value );
 
-	/**
-	 * Set the current processed item.
-	 *
-	 * @param int   $counter The current item.
-	 * @param mixed $item The current item.
-	 *
-	 * @since 1.0.0
-	 */
-	public function set_progress( $counter, $item );
+    /**
+     * Set the current processed item.
+     *
+     * @param int   $counter The current item.
+     * @param mixed $item The current item.
+     *
+     * @since 1.0.0
+     */
+    public function set_progress( $counter, $item );
 
-	/**
-	 * Set the operation as complete.
-	 *
-	 * @since 1.0.0
-	 */
-	public function finish();
+    /**
+     * Set the operation as complete.
+     *
+     * @since 1.0.0
+     */
+    public function finish();
 
 }

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	public function set_count( $value );
+	public function set_count($value);
 
 	/**
 	 * Set the current processed item.
@@ -33,7 +33,7 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	public function set_progress( $counter, $item );
+	public function set_progress($counter, $item);
 
 	/**
 	 * Set the operation as complete.

src/wordlift/tasks/class-task-ajax-adapter.php

Indentation +44 added lines, -44 removed lines

@@ -16,64 +16,64 @@
  */
 class Task_Ajax_Adapter {
 
-	/**
-	 * @var Task
-	 */
-	private $task;
+    /**
+     * @var Task
+     */
+    private $task;
 
-	/**
-	 * @var string $action_name The action name.
-	 */
-	private $action_name;
+    /**
+     * @var string $action_name The action name.
+     */
+    private $action_name;
 
-	/**
-	 * Task_Ajax_Adapter constructor.
-	 *
-	 * @param Task $task
-	 */
-	public function __construct( $task ) {
+    /**
+     * Task_Ajax_Adapter constructor.
+     *
+     * @param Task $task
+     */
+    public function __construct( $task ) {
 
-		$this->task = $task;
+        $this->task = $task;
 
-		$this->action_name = $task->get_id();
-		add_action( 'wp_ajax_' . $this->action_name, array( $this, 'start' ) );
+        $this->action_name = $task->get_id();
+        add_action( 'wp_ajax_' . $this->action_name, array( $this, 'start' ) );
 
-	}
+    }
 
-	public function start() {
+    public function start() {
 
-		// First check if there is a valid nonce.
-		check_ajax_referer( $this->action_name );
+        // First check if there is a valid nonce.
+        check_ajax_referer( $this->action_name );
 
-		// Get the offset.
-		$offset = filter_input( INPUT_POST, 'offset', FILTER_SANITIZE_NUMBER_INT );
-		$offset = $offset ? $offset : 0;
+        // Get the offset.
+        $offset = filter_input( INPUT_POST, 'offset', FILTER_SANITIZE_NUMBER_INT );
+        $offset = $offset ? $offset : 0;
 
-		// Compatibility fix for FacetWP, which somewhere in some filter checks for the $_POST array.
-		unset( $_POST['offset'] );
+        // Compatibility fix for FacetWP, which somewhere in some filter checks for the $_POST array.
+        unset( $_POST['offset'] );
 
-		// Create an AJAX progress. The AJAX progress returns the progress data to the AJAX client, which
-		// in turn calls the next batch.
-		$ajax_progress = new Task_Ajax_Progress( $this->action_name );
+        // Create an AJAX progress. The AJAX progress returns the progress data to the AJAX client, which
+        // in turn calls the next batch.
+        $ajax_progress = new Task_Ajax_Progress( $this->action_name );
 
-		// Finally create the task runner and start it.
-		$task_runner = new Task_Single_Instance_Task_Runner( $this->task, true, array( $ajax_progress ) );
+        // Finally create the task runner and start it.
+        $task_runner = new Task_Single_Instance_Task_Runner( $this->task, true, array( $ajax_progress ) );
 
-		try {
-			// Start the task runner, 1 item at a time.
-			$task_runner->start( 1, $offset );
-		} catch ( Task_Another_Instance_Is_Running_Exception $e ) {
-			wp_send_json_error( 'A task is already running.' );
-		}
+        try {
+            // Start the task runner, 1 item at a time.
+            $task_runner->start( 1, $offset );
+        } catch ( Task_Another_Instance_Is_Running_Exception $e ) {
+            wp_send_json_error( 'A task is already running.' );
+        }
 
-	}
+    }
 
-	/**
-	 * @return Task
-	 */
-	public function get_task() {
+    /**
+     * @return Task
+     */
+    public function get_task() {
 
-		return $this->task;
-	}
+        return $this->task;
+    }
 
 }

Spacing +10 added lines, -10 removed lines

@@ -31,39 +31,39 @@
 	 *
 	 * @param Task $task
 	 */
-	public function __construct( $task ) {
+	public function __construct($task) {
 
 		$this->task = $task;
 
 		$this->action_name = $task->get_id();
-		add_action( 'wp_ajax_' . $this->action_name, array( $this, 'start' ) );
+		add_action('wp_ajax_'.$this->action_name, array($this, 'start'));
 
 	}
 
 	public function start() {
 
 		// First check if there is a valid nonce.
-		check_ajax_referer( $this->action_name );
+		check_ajax_referer($this->action_name);
 
 		// Get the offset.
-		$offset = filter_input( INPUT_POST, 'offset', FILTER_SANITIZE_NUMBER_INT );
+		$offset = filter_input(INPUT_POST, 'offset', FILTER_SANITIZE_NUMBER_INT);
 		$offset = $offset ? $offset : 0;
 
 		// Compatibility fix for FacetWP, which somewhere in some filter checks for the $_POST array.
-		unset( $_POST['offset'] );
+		unset($_POST['offset']);
 
 		// Create an AJAX progress. The AJAX progress returns the progress data to the AJAX client, which
 		// in turn calls the next batch.
-		$ajax_progress = new Task_Ajax_Progress( $this->action_name );
+		$ajax_progress = new Task_Ajax_Progress($this->action_name);
 
 		// Finally create the task runner and start it.
-		$task_runner = new Task_Single_Instance_Task_Runner( $this->task, true, array( $ajax_progress ) );
+		$task_runner = new Task_Single_Instance_Task_Runner($this->task, true, array($ajax_progress));
 
 		try {
 			// Start the task runner, 1 item at a time.
-			$task_runner->start( 1, $offset );
-		} catch ( Task_Another_Instance_Is_Running_Exception $e ) {
-			wp_send_json_error( 'A task is already running.' );
+			$task_runner->start(1, $offset);
+		} catch (Task_Another_Instance_Is_Running_Exception $e) {
+			wp_send_json_error('A task is already running.');
 		}
 
 	}

src/wordlift/tasks/class-task-ajax-progress.php

Indentation +81 added lines, -81 removed lines

@@ -18,86 +18,86 @@
  */
 class Task_Ajax_Progress implements Task_Progress {
 
-	/**
-	 * The AJAX action, used to generate new nonces.
-	 *
-	 * @since 1.0.0
-	 * @access private
-	 * @var string $action The AJAX action.
-	 */
-	private $action;
-
-	/**
-	 * The total number of items to process.
-	 *
-	 * @since 1.0.0
-	 * @access private
-	 * @var int The total number of items to process.
-	 */
-	private $count;
-
-	/**
-	 * The current item index.
-	 *
-	 * @since 1.0.0
-	 * @access private
-	 * @var int $index The current item index.
-	 */
-	private $index;
-
-	/**
-	 * @var Wordlift_Log_Service
-	 */
-	private $log;
-
-	/**
-	 * Create a Task_Ajax_Progress instance with the specified
-	 * AJAX action.
-	 *
-	 * @param string $action The AJAX action.
-	 *
-	 * @since 1.0.0
-	 */
-	public function __construct( $action ) {
-
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
-
-		$this->action = $action;
-
-	}
-
-	/**
-	 * {@inheritDoc}
-	 */
-	public function set_count( $value ) {
-
-		$this->log->debug( "New count $value for action $this->action..." );
-
-		$this->count = $value;
-
-	}
-
-	/**
-	 * {@inheritDoc}
-	 */
-	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function set_progress( $index, $item ) {
-		$this->index = $index;
-	}
-
-	/**
-	 * {@inheritDoc}
-	 */
-	public function finish() {
-		wp_send_json_success(
-			array(
-				'count'    => $this->count,
-				'index'    => $this->index,
-				// $this->index is zero based.
-				'complete' => $this->index >= $this->count - 1,
-				'nonce'    => wp_create_nonce( $this->action ),
-			)
-		);
-	}
+    /**
+     * The AJAX action, used to generate new nonces.
+     *
+     * @since 1.0.0
+     * @access private
+     * @var string $action The AJAX action.
+     */
+    private $action;
+
+    /**
+     * The total number of items to process.
+     *
+     * @since 1.0.0
+     * @access private
+     * @var int The total number of items to process.
+     */
+    private $count;
+
+    /**
+     * The current item index.
+     *
+     * @since 1.0.0
+     * @access private
+     * @var int $index The current item index.
+     */
+    private $index;
+
+    /**
+     * @var Wordlift_Log_Service
+     */
+    private $log;
+
+    /**
+     * Create a Task_Ajax_Progress instance with the specified
+     * AJAX action.
+     *
+     * @param string $action The AJAX action.
+     *
+     * @since 1.0.0
+     */
+    public function __construct( $action ) {
+
+        $this->log = Wordlift_Log_Service::get_logger( get_class() );
+
+        $this->action = $action;
+
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function set_count( $value ) {
+
+        $this->log->debug( "New count $value for action $this->action..." );
+
+        $this->count = $value;
+
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+    public function set_progress( $index, $item ) {
+        $this->index = $index;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function finish() {
+        wp_send_json_success(
+            array(
+                'count'    => $this->count,
+                'index'    => $this->index,
+                // $this->index is zero based.
+                'complete' => $this->index >= $this->count - 1,
+                'nonce'    => wp_create_nonce( $this->action ),
+            )
+        );
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -58,9 +58,9 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	public function __construct( $action ) {
+	public function __construct($action) {
 
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
+		$this->log = Wordlift_Log_Service::get_logger(get_class());
 
 		$this->action = $action;
 
@@ -69,9 +69,9 @@ 
 	/**
 	 * {@inheritDoc}
 	 */
-	public function set_count( $value ) {
+	public function set_count($value) {
 
-		$this->log->debug( "New count $value for action $this->action..." );
+		$this->log->debug("New count $value for action $this->action...");
 
 		$this->count = $value;
 
@@ -81,7 +81,7 @@ 
 	 * {@inheritDoc}
 	 */
 	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function set_progress( $index, $item ) {
+	public function set_progress($index, $item) {
 		$this->index = $index;
 	}
 
@@ -95,7 +95,7 @@ 
 				'index'    => $this->index,
 				// $this->index is zero based.
 				'complete' => $this->index >= $this->count - 1,
-				'nonce'    => wp_create_nonce( $this->action ),
+				'nonce'    => wp_create_nonce($this->action),
 			)
 		);
 	}

src/wordlift/metabox/field/class-wordlift-metabox-field-integer.php

Indentation +9 added lines, -9 removed lines

@@ -19,13 +19,13 @@ 
  * @package Wordlift
  */
 class Wordlift_Metabox_Field_Integer extends Wl_Metabox_Field {
-	/**
-	 * @inheritdoc
-	 */
-	public function html_input( $text ) {
+    /**
+     * @inheritdoc
+     */
+    public function html_input( $text ) {
         // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@ob_start();
-		?>
+        @ob_start();
+        ?>
 			<div class="wl-input-wrapper">
 				<input
 					type="number"
@@ -45,8 +45,8 @@ 
 			</div>
 
 		<?php
-		$html = ob_get_clean();
+        $html = ob_get_clean();
 
-		return $html;
-	}
+        return $html;
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -22,23 +22,23 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function html_input( $text ) {
+	public function html_input($text) {
         // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
 		@ob_start();
 		?>
 			<div class="wl-input-wrapper">
 				<input
 					type="number"
-					id="<?php echo esc_attr( $this->meta_name ); ?>"
-					class="<?php echo esc_attr( $this->meta_name ); ?>"
-					value="<?php echo esc_attr( $text ); ?>"
-					name="wl_metaboxes[<?php echo esc_attr( $this->meta_name ); ?>][]"
+					id="<?php echo esc_attr($this->meta_name); ?>"
+					class="<?php echo esc_attr($this->meta_name); ?>"
+					value="<?php echo esc_attr($text); ?>"
+					name="wl_metaboxes[<?php echo esc_attr($this->meta_name); ?>][]"
 					style="width:88%"
 					min="0"
 				/>
 
 				<button class="button wl-remove-input wl-button" type="button">
-					<?php esc_html_e( 'Remove', 'wordlift' ); ?>
+					<?php esc_html_e('Remove', 'wordlift'); ?>
 				</button>
 
 				<div class="wl-input-notice"></div>