insideout10 / wordlift-plugin

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>

src/wordlift/metabox/field/class-wl-metabox-field-uri.php

Indentation +136 added lines, -136 removed lines

@@ -21,140 +21,140 @@ 
 // phpcs:ignore PEAR.NamingConventions.ValidClassName.Invalid
 class Wl_Metabox_Field_uri extends Wl_Metabox_Field {
 
-	/**
-	 * Only accept URIs or local entity IDs.
-	 * Build new entity if the user inputted a name that is not present in DB.
-	 *
-	 * @param mixed $value The value to sanitize.
-	 *
-	 * @return int|mixed|WP_Error
-	 */
-	public function sanitize_data_filter( $value ) {
-
-		if ( empty( $value ) ) {
-			return null;
-		}
-
-		// Check that the inserted URI, ID or name does not point to a saved
-		// entity or when the editor types a string in the input box, we try to
-		// find an entity with that title and, if not found, we create that entity.
-		$absent_from_db = is_numeric( $value )
-			? get_post( $value ) === null
-			: ! $this->exists( $value );
-
-		// Is it an URI?
-		$name_is_uri = strpos( $value, 'http' ) === 0;
-
-		// We create a new entity only if the entity is not present in the DB.
-		// In the case of an external uri, we just save the uri.
-		if ( $absent_from_db && ! $name_is_uri ) {
-
-			// ...we create a new entity!
-			$new_entity_id = wp_insert_post(
-				array(
-					'post_status' => 'publish',
-					'post_type'   => Wordlift_Entity_Service::TYPE_NAME,
-					'post_title'  => $value,
-				)
-			);
-
-			$type = 'http://schema.org/' . ( isset( $this->expected_uri_type ) ? $this->expected_uri_type[0] : 'Thing' );
-
-			wl_set_entity_main_type( $new_entity_id, $type );
-
-			// Update the value that will be saved as meta.
-			$value = $new_entity_id;
-		}
-
-		return $value;
-	}
-
-	/**
-	 * Check whether an entity exists given a value.
-	 *
-	 * @param string $value An entity URI or a title string..
-	 *
-	 * @return bool True if the entity exists otherwise false.
-	 * @since 3.15.0
-	 */
-	private function exists( $value ) {
-
-		// When the editor types a string in the input box, we try to find
-		// an entity with that title and, if not found, we create that entity.
-		$entity_service = Wordlift_Entity_Service::get_instance();
-
-		// Try looking for an entity by URI.
-		$found_by_uri = null !== $entity_service->get_entity_post_by_uri( $value );
-
-		// Return true if found.
-		if ( $found_by_uri ) {
-			$this->log->debug( "Found entity for $value." );
-
-			return true;
-		}
-
-		// Try looking for an entity by title, get any potential candidate.
-		$candidate = get_page_by_title( $value, OBJECT, Wordlift_Entity_Service::valid_entity_post_types() );
-
-		// If a candidate has been found and it's an entity.
-		return null !== $candidate && $entity_service->is_entity( $candidate->ID );
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function html_wrapper_open() {
-
-		// The containing <div> contains info on cardinality and expected types.
-		$html = "<div class='wl-field' data-cardinality='$this->cardinality'";
-
-		if ( isset( $this->expected_uri_type ) && $this->expected_uri_type !== null ) {
-
-			if ( is_array( $this->expected_uri_type ) ) {
-				$html .= " data-expected-types='" . implode( ',', $this->expected_uri_type ) . "'";
-			} else {
-				$html .= " data-expected-types='$this->expected_uri_type'";
-			}
-		}
-
-		$html .= '>';
-
-		return $html;
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function html_input( $default_entity_identifier ) {
-		if ( empty( $default_entity_identifier ) ) {
-			$entity = null;
-		} elseif ( is_numeric( $default_entity_identifier ) ) {
-			$entity = get_post( $default_entity_identifier );
-		} else {
-			// @todo: we cannot be so sure this is a URI.
-			// It is an URI
-			$entity = Wordlift_Entity_Service::get_instance()
-				->get_entity_post_by_uri( $default_entity_identifier );
-		}
-
-		// Bail out if an entity id has been provided by the entity is not found.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/818
-		if ( ! empty( $default_entity_identifier ) && $entity === null ) {
-			return '';
-		}
-
-		$label = $entity === null ? '' : $entity->post_title;
-		$value = $entity === null ? '' : $entity->ID;
-
-		// Write saved value in page
-		// The <input> tags host the meta value.
-		// The visible <input> has the human readable value (i.e. entity name or uri)
-		// and is accompained by an hidden <input> tag, passed to the server,
-		// that contains the raw value (i.e. the uri or entity id).
+    /**
+     * Only accept URIs or local entity IDs.
+     * Build new entity if the user inputted a name that is not present in DB.
+     *
+     * @param mixed $value The value to sanitize.
+     *
+     * @return int|mixed|WP_Error
+     */
+    public function sanitize_data_filter( $value ) {
+
+        if ( empty( $value ) ) {
+            return null;
+        }
+
+        // Check that the inserted URI, ID or name does not point to a saved
+        // entity or when the editor types a string in the input box, we try to
+        // find an entity with that title and, if not found, we create that entity.
+        $absent_from_db = is_numeric( $value )
+            ? get_post( $value ) === null
+            : ! $this->exists( $value );
+
+        // Is it an URI?
+        $name_is_uri = strpos( $value, 'http' ) === 0;
+
+        // We create a new entity only if the entity is not present in the DB.
+        // In the case of an external uri, we just save the uri.
+        if ( $absent_from_db && ! $name_is_uri ) {
+
+            // ...we create a new entity!
+            $new_entity_id = wp_insert_post(
+                array(
+                    'post_status' => 'publish',
+                    'post_type'   => Wordlift_Entity_Service::TYPE_NAME,
+                    'post_title'  => $value,
+                )
+            );
+
+            $type = 'http://schema.org/' . ( isset( $this->expected_uri_type ) ? $this->expected_uri_type[0] : 'Thing' );
+
+            wl_set_entity_main_type( $new_entity_id, $type );
+
+            // Update the value that will be saved as meta.
+            $value = $new_entity_id;
+        }
+
+        return $value;
+    }
+
+    /**
+     * Check whether an entity exists given a value.
+     *
+     * @param string $value An entity URI or a title string..
+     *
+     * @return bool True if the entity exists otherwise false.
+     * @since 3.15.0
+     */
+    private function exists( $value ) {
+
+        // When the editor types a string in the input box, we try to find
+        // an entity with that title and, if not found, we create that entity.
+        $entity_service = Wordlift_Entity_Service::get_instance();
+
+        // Try looking for an entity by URI.
+        $found_by_uri = null !== $entity_service->get_entity_post_by_uri( $value );
+
+        // Return true if found.
+        if ( $found_by_uri ) {
+            $this->log->debug( "Found entity for $value." );
+
+            return true;
+        }
+
+        // Try looking for an entity by title, get any potential candidate.
+        $candidate = get_page_by_title( $value, OBJECT, Wordlift_Entity_Service::valid_entity_post_types() );
+
+        // If a candidate has been found and it's an entity.
+        return null !== $candidate && $entity_service->is_entity( $candidate->ID );
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function html_wrapper_open() {
+
+        // The containing <div> contains info on cardinality and expected types.
+        $html = "<div class='wl-field' data-cardinality='$this->cardinality'";
+
+        if ( isset( $this->expected_uri_type ) && $this->expected_uri_type !== null ) {
+
+            if ( is_array( $this->expected_uri_type ) ) {
+                $html .= " data-expected-types='" . implode( ',', $this->expected_uri_type ) . "'";
+            } else {
+                $html .= " data-expected-types='$this->expected_uri_type'";
+            }
+        }
+
+        $html .= '>';
+
+        return $html;
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function html_input( $default_entity_identifier ) {
+        if ( empty( $default_entity_identifier ) ) {
+            $entity = null;
+        } elseif ( is_numeric( $default_entity_identifier ) ) {
+            $entity = get_post( $default_entity_identifier );
+        } else {
+            // @todo: we cannot be so sure this is a URI.
+            // It is an URI
+            $entity = Wordlift_Entity_Service::get_instance()
+                ->get_entity_post_by_uri( $default_entity_identifier );
+        }
+
+        // Bail out if an entity id has been provided by the entity is not found.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/818
+        if ( ! empty( $default_entity_identifier ) && $entity === null ) {
+            return '';
+        }
+
+        $label = $entity === null ? '' : $entity->post_title;
+        $value = $entity === null ? '' : $entity->ID;
+
+        // Write saved value in page
+        // The <input> tags host the meta value.
+        // The visible <input> has the human readable value (i.e. entity name or uri)
+        // and is accompained by an hidden <input> tag, passed to the server,
+        // that contains the raw value (i.e. the uri or entity id).
         // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@ob_start();
-		?>
+        @ob_start();
+        ?>
 		<div class="wl-input-wrapper wl-autocomplete-wrapper">
 			<input
 					type="text"
@@ -176,9 +176,9 @@ 
 			<div class="wl-input-notice"></div>
 		</div>
 		<?php
-		$html = ob_get_clean();
+        $html = ob_get_clean();
 
-		return $html;
-	}
+        return $html;
+    }
 
 }

Spacing +30 added lines, -30 removed lines

@@ -29,25 +29,25 @@ 
 	 *
 	 * @return int|mixed|WP_Error
 	 */
-	public function sanitize_data_filter( $value ) {
+	public function sanitize_data_filter($value) {
 
-		if ( empty( $value ) ) {
+		if (empty($value)) {
 			return null;
 		}
 
 		// Check that the inserted URI, ID or name does not point to a saved
 		// entity or when the editor types a string in the input box, we try to
 		// find an entity with that title and, if not found, we create that entity.
-		$absent_from_db = is_numeric( $value )
-			? get_post( $value ) === null
-			: ! $this->exists( $value );
+		$absent_from_db = is_numeric($value)
+			? get_post($value) === null
+			: ! $this->exists($value);
 
 		// Is it an URI?
-		$name_is_uri = strpos( $value, 'http' ) === 0;
+		$name_is_uri = strpos($value, 'http') === 0;
 
 		// We create a new entity only if the entity is not present in the DB.
 		// In the case of an external uri, we just save the uri.
-		if ( $absent_from_db && ! $name_is_uri ) {
+		if ($absent_from_db && ! $name_is_uri) {
 
 			// ...we create a new entity!
 			$new_entity_id = wp_insert_post(
@@ -58,9 +58,9 @@ 
 				)
 			);
 
-			$type = 'http://schema.org/' . ( isset( $this->expected_uri_type ) ? $this->expected_uri_type[0] : 'Thing' );
+			$type = 'http://schema.org/'.(isset($this->expected_uri_type) ? $this->expected_uri_type[0] : 'Thing');
 
-			wl_set_entity_main_type( $new_entity_id, $type );
+			wl_set_entity_main_type($new_entity_id, $type);
 
 			// Update the value that will be saved as meta.
 			$value = $new_entity_id;
@@ -77,27 +77,27 @@ 
 	 * @return bool True if the entity exists otherwise false.
 	 * @since 3.15.0
 	 */
-	private function exists( $value ) {
+	private function exists($value) {
 
 		// When the editor types a string in the input box, we try to find
 		// an entity with that title and, if not found, we create that entity.
 		$entity_service = Wordlift_Entity_Service::get_instance();
 
 		// Try looking for an entity by URI.
-		$found_by_uri = null !== $entity_service->get_entity_post_by_uri( $value );
+		$found_by_uri = null !== $entity_service->get_entity_post_by_uri($value);
 
 		// Return true if found.
-		if ( $found_by_uri ) {
-			$this->log->debug( "Found entity for $value." );
+		if ($found_by_uri) {
+			$this->log->debug("Found entity for $value.");
 
 			return true;
 		}
 
 		// Try looking for an entity by title, get any potential candidate.
-		$candidate = get_page_by_title( $value, OBJECT, Wordlift_Entity_Service::valid_entity_post_types() );
+		$candidate = get_page_by_title($value, OBJECT, Wordlift_Entity_Service::valid_entity_post_types());
 
 		// If a candidate has been found and it's an entity.
-		return null !== $candidate && $entity_service->is_entity( $candidate->ID );
+		return null !== $candidate && $entity_service->is_entity($candidate->ID);
 	}
 
 	/**
@@ -108,10 +108,10 @@ 
 		// The containing <div> contains info on cardinality and expected types.
 		$html = "<div class='wl-field' data-cardinality='$this->cardinality'";
 
-		if ( isset( $this->expected_uri_type ) && $this->expected_uri_type !== null ) {
+		if (isset($this->expected_uri_type) && $this->expected_uri_type !== null) {
 
-			if ( is_array( $this->expected_uri_type ) ) {
-				$html .= " data-expected-types='" . implode( ',', $this->expected_uri_type ) . "'";
+			if (is_array($this->expected_uri_type)) {
+				$html .= " data-expected-types='".implode(',', $this->expected_uri_type)."'";
 			} else {
 				$html .= " data-expected-types='$this->expected_uri_type'";
 			}
@@ -125,22 +125,22 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function html_input( $default_entity_identifier ) {
-		if ( empty( $default_entity_identifier ) ) {
+	public function html_input($default_entity_identifier) {
+		if (empty($default_entity_identifier)) {
 			$entity = null;
-		} elseif ( is_numeric( $default_entity_identifier ) ) {
-			$entity = get_post( $default_entity_identifier );
+		} elseif (is_numeric($default_entity_identifier)) {
+			$entity = get_post($default_entity_identifier);
 		} else {
 			// @todo: we cannot be so sure this is a URI.
 			// It is an URI
 			$entity = Wordlift_Entity_Service::get_instance()
-				->get_entity_post_by_uri( $default_entity_identifier );
+				->get_entity_post_by_uri($default_entity_identifier);
 		}
 
 		// Bail out if an entity id has been provided by the entity is not found.
 		//
 		// See https://github.com/insideout10/wordlift-plugin/issues/818
-		if ( ! empty( $default_entity_identifier ) && $entity === null ) {
+		if ( ! empty($default_entity_identifier) && $entity === null) {
 			return '';
 		}
 
@@ -158,19 +158,19 @@ 
 		<div class="wl-input-wrapper wl-autocomplete-wrapper">
 			<input
 					type="text"
-					class="<?php echo esc_attr( $this->meta_name ); ?> wl-autocomplete"
-					value="<?php echo esc_attr( $label ); ?>"
+					class="<?php echo esc_attr($this->meta_name); ?> wl-autocomplete"
+					value="<?php echo esc_attr($label); ?>"
 					style="width:88%"
 			/>
 			<input
 					type="hidden"
-					class="<?php echo esc_attr( $this->meta_name ); ?>"
-					name="wl_metaboxes[<?php echo esc_attr( $this->meta_name ); ?>][]"
-					value="<?php echo esc_attr( $value ); ?>"
+					class="<?php echo esc_attr($this->meta_name); ?>"
+					name="wl_metaboxes[<?php echo esc_attr($this->meta_name); ?>][]"
+					value="<?php echo esc_attr($value); ?>"
 			/>
 
 			<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>

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

Indentation +37 added lines, -37 removed lines

@@ -20,42 +20,42 @@
  */
 class Wordlift_Metabox_Field_Duration extends Wl_Metabox_Field_date {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function __construct( $args, $id, $type ) {
-		parent::__construct( $args, $id, $type );
-
-		$this->date_format = 'H:i';
-		$this->timepicker  = true;
-		$this->no_calendar = true;
-
-	}
-
-	/**
-	 * Sanitize a single value. Called from $this->sanitize_data. Default sanitization excludes empty values.
-	 * make sure the value is either empty, an integer representing valid number of minutes
-	 * or an HH:MM time format.
-	 *
-	 * @param mixed $value The value being sanitized.
-	 *
-	 * @return mixed Returns sanitized value, or null.
-	 */
-	public function sanitize_data_filter( $value ) {
-
-		if ( $value !== null && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
-			preg_match(
-				'#((([01]?[0-9]{1}|2[0-3]{1}):)?[0-5]{1})?[0-9]{1}#',
-				trim( $value ),
-				$matches
-			);
-
-			if ( count( $matches ) > 0 ) {
-				return $matches[0];
-			}
-		}
-
-		return null;
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function __construct( $args, $id, $type ) {
+        parent::__construct( $args, $id, $type );
+
+        $this->date_format = 'H:i';
+        $this->timepicker  = true;
+        $this->no_calendar = true;
+
+    }
+
+    /**
+     * Sanitize a single value. Called from $this->sanitize_data. Default sanitization excludes empty values.
+     * make sure the value is either empty, an integer representing valid number of minutes
+     * or an HH:MM time format.
+     *
+     * @param mixed $value The value being sanitized.
+     *
+     * @return mixed Returns sanitized value, or null.
+     */
+    public function sanitize_data_filter( $value ) {
+
+        if ( $value !== null && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
+            preg_match(
+                '#((([01]?[0-9]{1}|2[0-3]{1}):)?[0-5]{1})?[0-9]{1}#',
+                trim( $value ),
+                $matches
+            );
+
+            if ( count( $matches ) > 0 ) {
+                return $matches[0];
+            }
+        }
+
+        return null;
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -23,8 +23,8 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function __construct( $args, $id, $type ) {
-		parent::__construct( $args, $id, $type );
+	public function __construct($args, $id, $type) {
+		parent::__construct($args, $id, $type);
 
 		$this->date_format = 'H:i';
 		$this->timepicker  = true;
@@ -41,16 +41,16 @@ 
 	 *
 	 * @return mixed Returns sanitized value, or null.
 	 */
-	public function sanitize_data_filter( $value ) {
+	public function sanitize_data_filter($value) {
 
-		if ( $value !== null && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
+		if ($value !== null && '' !== $value) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
 			preg_match(
 				'#((([01]?[0-9]{1}|2[0-3]{1}):)?[0-5]{1})?[0-9]{1}#',
-				trim( $value ),
+				trim($value),
 				$matches
 			);
 
-			if ( count( $matches ) > 0 ) {
+			if (count($matches) > 0) {
 				return $matches[0];
 			}
 		}