insideout10 / wordlift-plugin

src/libraries/action-scheduler/classes/migration/Config.php

Indentation +148 added lines, -148 removed lines

@@ -17,152 +17,152 @@
  * A config builder for the ActionScheduler\Migration\Runner class
  */
 class Config {
-	/** @var ActionScheduler_Store */
-	private $source_store;
-
-	/** @var ActionScheduler_Logger */
-	private $source_logger;
-
-	/** @var ActionScheduler_Store */
-	private $destination_store;
-
-	/** @var ActionScheduler_Logger */
-	private $destination_logger;
-
-	/** @var Progress bar */
-	private $progress_bar;
-
-	/** @var bool */
-	private $dry_run = false;
-
-	/**
-	 * Config constructor.
-	 */
-	public function __construct() {
-
-	}
-
-	/**
-	 * Get the configured source store.
-	 *
-	 * @return ActionScheduler_Store
-	 */
-	public function get_source_store() {
-		if ( empty( $this->source_store ) ) {
-			throw new \RuntimeException( __( 'Source store must be configured before running a migration', 'action-scheduler' ) );
-		}
-
-		return $this->source_store;
-	}
-
-	/**
-	 * Set the configured source store.
-	 *
-	 * @param ActionScheduler_Store $store Source store object.
-	 */
-	public function set_source_store( Store $store ) {
-		$this->source_store = $store;
-	}
-
-	/**
-	 * Get the configured source loger.
-	 *
-	 * @return ActionScheduler_Logger
-	 */
-	public function get_source_logger() {
-		if ( empty( $this->source_logger ) ) {
-			throw new \RuntimeException( __( 'Source logger must be configured before running a migration', 'action-scheduler' ) );
-		}
-
-		return $this->source_logger;
-	}
-
-	/**
-	 * Set the configured source logger.
-	 *
-	 * @param ActionScheduler_Logger $logger
-	 */
-	public function set_source_logger( Logger $logger ) {
-		$this->source_logger = $logger;
-	}
-
-	/**
-	 * Get the configured destination store.
-	 *
-	 * @return ActionScheduler_Store
-	 */
-	public function get_destination_store() {
-		if ( empty( $this->destination_store ) ) {
-			throw new \RuntimeException( __( 'Destination store must be configured before running a migration', 'action-scheduler' ) );
-		}
-
-		return $this->destination_store;
-	}
-
-	/**
-	 * Set the configured destination store.
-	 *
-	 * @param ActionScheduler_Store $store
-	 */
-	public function set_destination_store( Store $store ) {
-		$this->destination_store = $store;
-	}
-
-	/**
-	 * Get the configured destination logger.
-	 *
-	 * @return ActionScheduler_Logger
-	 */
-	public function get_destination_logger() {
-		if ( empty( $this->destination_logger ) ) {
-			throw new \RuntimeException( __( 'Destination logger must be configured before running a migration', 'action-scheduler' ) );
-		}
-
-		return $this->destination_logger;
-	}
-
-	/**
-	 * Set the configured destination logger.
-	 *
-	 * @param ActionScheduler_Logger $logger
-	 */
-	public function set_destination_logger( Logger $logger ) {
-		$this->destination_logger = $logger;
-	}
-
-	/**
-	 * Get flag indicating whether it's a dry run.
-	 *
-	 * @return bool
-	 */
-	public function get_dry_run() {
-		return $this->dry_run;
-	}
-
-	/**
-	 * Set flag indicating whether it's a dry run.
-	 *
-	 * @param bool $dry_run
-	 */
-	public function set_dry_run( $dry_run ) {
-		$this->dry_run = (bool) $dry_run;
-	}
-
-	/**
-	 * Get progress bar object.
-	 *
-	 * @return ActionScheduler\WPCLI\ProgressBar
-	 */
-	public function get_progress_bar() {
-		return $this->progress_bar;
-	}
-
-	/**
-	 * Set progress bar object.
-	 *
-	 * @param ActionScheduler\WPCLI\ProgressBar $progress_bar
-	 */
-	public function set_progress_bar( ProgressBar $progress_bar ) {
-		$this->progress_bar = $progress_bar;
-	}
+    /** @var ActionScheduler_Store */
+    private $source_store;
+
+    /** @var ActionScheduler_Logger */
+    private $source_logger;
+
+    /** @var ActionScheduler_Store */
+    private $destination_store;
+
+    /** @var ActionScheduler_Logger */
+    private $destination_logger;
+
+    /** @var Progress bar */
+    private $progress_bar;
+
+    /** @var bool */
+    private $dry_run = false;
+
+    /**
+     * Config constructor.
+     */
+    public function __construct() {
+
+    }
+
+    /**
+     * Get the configured source store.
+     *
+     * @return ActionScheduler_Store
+     */
+    public function get_source_store() {
+        if ( empty( $this->source_store ) ) {
+            throw new \RuntimeException( __( 'Source store must be configured before running a migration', 'action-scheduler' ) );
+        }
+
+        return $this->source_store;
+    }
+
+    /**
+     * Set the configured source store.
+     *
+     * @param ActionScheduler_Store $store Source store object.
+     */
+    public function set_source_store( Store $store ) {
+        $this->source_store = $store;
+    }
+
+    /**
+     * Get the configured source loger.
+     *
+     * @return ActionScheduler_Logger
+     */
+    public function get_source_logger() {
+        if ( empty( $this->source_logger ) ) {
+            throw new \RuntimeException( __( 'Source logger must be configured before running a migration', 'action-scheduler' ) );
+        }
+
+        return $this->source_logger;
+    }
+
+    /**
+     * Set the configured source logger.
+     *
+     * @param ActionScheduler_Logger $logger
+     */
+    public function set_source_logger( Logger $logger ) {
+        $this->source_logger = $logger;
+    }
+
+    /**
+     * Get the configured destination store.
+     *
+     * @return ActionScheduler_Store
+     */
+    public function get_destination_store() {
+        if ( empty( $this->destination_store ) ) {
+            throw new \RuntimeException( __( 'Destination store must be configured before running a migration', 'action-scheduler' ) );
+        }
+
+        return $this->destination_store;
+    }
+
+    /**
+     * Set the configured destination store.
+     *
+     * @param ActionScheduler_Store $store
+     */
+    public function set_destination_store( Store $store ) {
+        $this->destination_store = $store;
+    }
+
+    /**
+     * Get the configured destination logger.
+     *
+     * @return ActionScheduler_Logger
+     */
+    public function get_destination_logger() {
+        if ( empty( $this->destination_logger ) ) {
+            throw new \RuntimeException( __( 'Destination logger must be configured before running a migration', 'action-scheduler' ) );
+        }
+
+        return $this->destination_logger;
+    }
+
+    /**
+     * Set the configured destination logger.
+     *
+     * @param ActionScheduler_Logger $logger
+     */
+    public function set_destination_logger( Logger $logger ) {
+        $this->destination_logger = $logger;
+    }
+
+    /**
+     * Get flag indicating whether it's a dry run.
+     *
+     * @return bool
+     */
+    public function get_dry_run() {
+        return $this->dry_run;
+    }
+
+    /**
+     * Set flag indicating whether it's a dry run.
+     *
+     * @param bool $dry_run
+     */
+    public function set_dry_run( $dry_run ) {
+        $this->dry_run = (bool) $dry_run;
+    }
+
+    /**
+     * Get progress bar object.
+     *
+     * @return ActionScheduler\WPCLI\ProgressBar
+     */
+    public function get_progress_bar() {
+        return $this->progress_bar;
+    }
+
+    /**
+     * Set progress bar object.
+     *
+     * @param ActionScheduler\WPCLI\ProgressBar $progress_bar
+     */
+    public function set_progress_bar( ProgressBar $progress_bar ) {
+        $this->progress_bar = $progress_bar;
+    }
 }

Spacing +14 added lines, -14 removed lines

@@ -48,8 +48,8 @@ 
 	 * @return ActionScheduler_Store
 	 */
 	public function get_source_store() {
-		if ( empty( $this->source_store ) ) {
-			throw new \RuntimeException( __( 'Source store must be configured before running a migration', 'action-scheduler' ) );
+		if (empty($this->source_store)) {
+			throw new \RuntimeException(__('Source store must be configured before running a migration', 'action-scheduler'));
 		}
 
 		return $this->source_store;
@@ -60,7 +60,7 @@ 
 	 *
 	 * @param ActionScheduler_Store $store Source store object.
 	 */
-	public function set_source_store( Store $store ) {
+	public function set_source_store(Store $store) {
 		$this->source_store = $store;
 	}
 
@@ -70,8 +70,8 @@ 
 	 * @return ActionScheduler_Logger
 	 */
 	public function get_source_logger() {
-		if ( empty( $this->source_logger ) ) {
-			throw new \RuntimeException( __( 'Source logger must be configured before running a migration', 'action-scheduler' ) );
+		if (empty($this->source_logger)) {
+			throw new \RuntimeException(__('Source logger must be configured before running a migration', 'action-scheduler'));
 		}
 
 		return $this->source_logger;
@@ -82,7 +82,7 @@ 
 	 *
 	 * @param ActionScheduler_Logger $logger
 	 */
-	public function set_source_logger( Logger $logger ) {
+	public function set_source_logger(Logger $logger) {
 		$this->source_logger = $logger;
 	}
 
@@ -92,8 +92,8 @@ 
 	 * @return ActionScheduler_Store
 	 */
 	public function get_destination_store() {
-		if ( empty( $this->destination_store ) ) {
-			throw new \RuntimeException( __( 'Destination store must be configured before running a migration', 'action-scheduler' ) );
+		if (empty($this->destination_store)) {
+			throw new \RuntimeException(__('Destination store must be configured before running a migration', 'action-scheduler'));
 		}
 
 		return $this->destination_store;
@@ -104,7 +104,7 @@ 
 	 *
 	 * @param ActionScheduler_Store $store
 	 */
-	public function set_destination_store( Store $store ) {
+	public function set_destination_store(Store $store) {
 		$this->destination_store = $store;
 	}
 
@@ -114,8 +114,8 @@ 
 	 * @return ActionScheduler_Logger
 	 */
 	public function get_destination_logger() {
-		if ( empty( $this->destination_logger ) ) {
-			throw new \RuntimeException( __( 'Destination logger must be configured before running a migration', 'action-scheduler' ) );
+		if (empty($this->destination_logger)) {
+			throw new \RuntimeException(__('Destination logger must be configured before running a migration', 'action-scheduler'));
 		}
 
 		return $this->destination_logger;
@@ -126,7 +126,7 @@ 
 	 *
 	 * @param ActionScheduler_Logger $logger
 	 */
-	public function set_destination_logger( Logger $logger ) {
+	public function set_destination_logger(Logger $logger) {
 		$this->destination_logger = $logger;
 	}
 
@@ -144,7 +144,7 @@ 
 	 *
 	 * @param bool $dry_run
 	 */
-	public function set_dry_run( $dry_run ) {
+	public function set_dry_run($dry_run) {
 		$this->dry_run = (bool) $dry_run;
 	}
 
@@ -162,7 +162,7 @@ 
 	 *
 	 * @param ActionScheduler\WPCLI\ProgressBar $progress_bar
 	 */
-	public function set_progress_bar( ProgressBar $progress_bar ) {
+	public function set_progress_bar(ProgressBar $progress_bar) {
 		$this->progress_bar = $progress_bar;
 	}
 }

src/libraries/action-scheduler/classes/migration/DryRun_ActionMigrator.php

Indentation +11 added lines, -11 removed lines

@@ -13,16 +13,16 @@
  * @codeCoverageIgnore
  */
 class DryRun_ActionMigrator extends ActionMigrator {
-	/**
-	 * Simulate migrating an action.
-	 *
-	 * @param int $source_action_id Action ID.
-	 *
-	 * @return int
-	 */
-	public function migrate( $source_action_id ) {
-		do_action( 'action_scheduler/migrate_action_dry_run', $source_action_id );
+    /**
+     * Simulate migrating an action.
+     *
+     * @param int $source_action_id Action ID.
+     *
+     * @return int
+     */
+    public function migrate( $source_action_id ) {
+        do_action( 'action_scheduler/migrate_action_dry_run', $source_action_id );
 
-		return 0;
-	}
+        return 0;
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -20,8 +20,8 @@
 	 *
 	 * @return int
 	 */
-	public function migrate( $source_action_id ) {
-		do_action( 'action_scheduler/migrate_action_dry_run', $source_action_id );
+	public function migrate($source_action_id) {
+		do_action('action_scheduler/migrate_action_dry_run', $source_action_id);
 
 		return 0;
 	}

src/libraries/action-scheduler/classes/migration/Controller.php

Indentation +195 added lines, -195 removed lines

@@ -19,85 +19,85 @@ 
  * @codeCoverageIgnore
  */
 class Controller {
-	private static $instance;
-
-	/** @var Action_Scheduler\Migration\Scheduler */
-	private $migration_scheduler;
-
-	/** @var string */
-	private $store_classname;
-
-	/** @var string */
-	private $logger_classname;
-
-	/** @var bool */
-	private $migrate_custom_store;
-
-	/**
-	 * Controller constructor.
-	 *
-	 * @param Scheduler $migration_scheduler Migration scheduler object.
-	 */
-	protected function __construct( Scheduler $migration_scheduler ) {
-		$this->migration_scheduler = $migration_scheduler;
-		$this->store_classname     = '';
-	}
-
-	/**
-	 * Set the action store class name.
-	 *
-	 * @param string $class Classname of the store class.
-	 *
-	 * @return string
-	 */
-	public function get_store_class( $class ) {
-		if ( \ActionScheduler_DataController::is_migration_complete() ) {
-			return \ActionScheduler_DataController::DATASTORE_CLASS;
-		} elseif ( \ActionScheduler_Store::DEFAULT_CLASS !== $class ) {
-			$this->store_classname = $class;
-			return $class;
-		} else {
-			return 'ActionScheduler_HybridStore';
-		}
-	}
-
-	/**
-	 * Set the action logger class name.
-	 *
-	 * @param string $class Classname of the logger class.
-	 *
-	 * @return string
-	 */
-	public function get_logger_class( $class ) {
-		\ActionScheduler_Store::instance();
-
-		if ( $this->has_custom_datastore() ) {
-			$this->logger_classname = $class;
-			return $class;
-		} else {
-			return \ActionScheduler_DataController::LOGGER_CLASS;
-		}
-	}
-
-	/**
-	 * Get flag indicating whether a custom datastore is in use.
-	 *
-	 * @return bool
-	 */
-	public function has_custom_datastore() {
-		return (bool) $this->store_classname;
-	}
-
-	/**
-	 * Set up the background migration process.
-	 *
-	 * @return void
-	 */
-	public function schedule_migration() {
-		$logging_tables = new ActionScheduler_LoggerSchema();
-		$store_tables   = new ActionScheduler_StoreSchema();
-
-		/*
+    private static $instance;
+
+    /** @var Action_Scheduler\Migration\Scheduler */
+    private $migration_scheduler;
+
+    /** @var string */
+    private $store_classname;
+
+    /** @var string */
+    private $logger_classname;
+
+    /** @var bool */
+    private $migrate_custom_store;
+
+    /**
+     * Controller constructor.
+     *
+     * @param Scheduler $migration_scheduler Migration scheduler object.
+     */
+    protected function __construct( Scheduler $migration_scheduler ) {
+        $this->migration_scheduler = $migration_scheduler;
+        $this->store_classname     = '';
+    }
+
+    /**
+     * Set the action store class name.
+     *
+     * @param string $class Classname of the store class.
+     *
+     * @return string
+     */
+    public function get_store_class( $class ) {
+        if ( \ActionScheduler_DataController::is_migration_complete() ) {
+            return \ActionScheduler_DataController::DATASTORE_CLASS;
+        } elseif ( \ActionScheduler_Store::DEFAULT_CLASS !== $class ) {
+            $this->store_classname = $class;
+            return $class;
+        } else {
+            return 'ActionScheduler_HybridStore';
+        }
+    }
+
+    /**
+     * Set the action logger class name.
+     *
+     * @param string $class Classname of the logger class.
+     *
+     * @return string
+     */
+    public function get_logger_class( $class ) {
+        \ActionScheduler_Store::instance();
+
+        if ( $this->has_custom_datastore() ) {
+            $this->logger_classname = $class;
+            return $class;
+        } else {
+            return \ActionScheduler_DataController::LOGGER_CLASS;
+        }
+    }
+
+    /**
+     * Get flag indicating whether a custom datastore is in use.
+     *
+     * @return bool
+     */
+    public function has_custom_datastore() {
+        return (bool) $this->store_classname;
+    }
+
+    /**
+     * Set up the background migration process.
+     *
+     * @return void
+     */
+    public function schedule_migration() {
+        $logging_tables = new ActionScheduler_LoggerSchema();
+        $store_tables   = new ActionScheduler_StoreSchema();
+
+        /*
 		 * In some unusual cases, the expected tables may not have been created. In such cases
 		 * we do not schedule a migration as doing so will lead to fatal error conditions.
 		 *
@@ -107,120 +107,120 @@ 
 		 *
 		 * @see https://github.com/woocommerce/action-scheduler/issues/653
 		 */
-		if (
-			ActionScheduler_DataController::is_migration_complete()
-			|| $this->migration_scheduler->is_migration_scheduled()
-			|| ! $store_tables->tables_exist()
-			|| ! $logging_tables->tables_exist()
-		) {
-			return;
-		}
-
-		$this->migration_scheduler->schedule_migration();
-	}
-
-	/**
-	 * Get the default migration config object
-	 *
-	 * @return ActionScheduler\Migration\Config
-	 */
-	public function get_migration_config_object() {
-		static $config = null;
-
-		if ( ! $config ) {
-			$source_store  = $this->store_classname ? new $this->store_classname() : new \ActionScheduler_wpPostStore();
-			$source_logger = $this->logger_classname ? new $this->logger_classname() : new \ActionScheduler_wpCommentLogger();
-
-			$config = new Config();
-			$config->set_source_store( $source_store );
-			$config->set_source_logger( $source_logger );
-			$config->set_destination_store( new \ActionScheduler_DBStoreMigrator() );
-			$config->set_destination_logger( new \ActionScheduler_DBLogger() );
-
-			if ( defined( 'WP_CLI' ) && WP_CLI ) {
-				$config->set_progress_bar( new ProgressBar( '', 0 ) );
-			}
-		}
-
-		return apply_filters( 'action_scheduler/migration_config', $config );
-	}
-
-	/**
-	 * Hook dashboard migration notice.
-	 */
-	public function hook_admin_notices() {
-		if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) {
-			return;
-		}
-		add_action( 'admin_notices', array( $this, 'display_migration_notice' ), 10, 0 );
-	}
-
-	/**
-	 * Show a dashboard notice that migration is in progress.
-	 */
-	public function display_migration_notice() {
-		printf( '<div class="notice notice-warning"><p>%s</p></div>', esc_html__( 'Action Scheduler migration in progress. The list of scheduled actions may be incomplete.', 'action-scheduler' ) );
-	}
-
-	/**
-	 * Add store classes. Hook migration.
-	 */
-	private function hook() {
-		add_filter( 'action_scheduler_store_class', array( $this, 'get_store_class' ), 100, 1 );
-		add_filter( 'action_scheduler_logger_class', array( $this, 'get_logger_class' ), 100, 1 );
-		add_action( 'init', array( $this, 'maybe_hook_migration' ) );
-		add_action( 'wp_loaded', array( $this, 'schedule_migration' ) );
-
-		// Action Scheduler may be displayed as a Tools screen or WooCommerce > Status administration screen
-		add_action( 'load-tools_page_action-scheduler', array( $this, 'hook_admin_notices' ), 10, 0 );
-		add_action( 'load-woocommerce_page_wc-status', array( $this, 'hook_admin_notices' ), 10, 0 );
-	}
-
-	/**
-	 * Possibly hook the migration scheduler action.
-	 *
-	 * @author Jeremy Pry
-	 */
-	public function maybe_hook_migration() {
-		if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) {
-			return;
-		}
-
-		$this->migration_scheduler->hook();
-	}
-
-	/**
-	 * Allow datastores to enable migration to AS tables.
-	 */
-	public function allow_migration() {
-		if ( ! \ActionScheduler_DataController::dependencies_met() ) {
-			return false;
-		}
-
-		if ( null === $this->migrate_custom_store ) {
-			$this->migrate_custom_store = apply_filters( 'action_scheduler_migrate_data_store', false );
-		}
-
-		return ( ! $this->has_custom_datastore() ) || $this->migrate_custom_store;
-	}
-
-	/**
-	 * Proceed with the migration if the dependencies have been met.
-	 */
-	public static function init() {
-		if ( \ActionScheduler_DataController::dependencies_met() ) {
-			self::instance()->hook();
-		}
-	}
-
-	/**
-	 * Singleton factory.
-	 */
-	public static function instance() {
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new static( new Scheduler() );
-		}
-
-		return self::$instance;
-	}
+        if (
+            ActionScheduler_DataController::is_migration_complete()
+            || $this->migration_scheduler->is_migration_scheduled()
+            || ! $store_tables->tables_exist()
+            || ! $logging_tables->tables_exist()
+        ) {
+            return;
+        }
+
+        $this->migration_scheduler->schedule_migration();
+    }
+
+    /**
+     * Get the default migration config object
+     *
+     * @return ActionScheduler\Migration\Config
+     */
+    public function get_migration_config_object() {
+        static $config = null;
+
+        if ( ! $config ) {
+            $source_store  = $this->store_classname ? new $this->store_classname() : new \ActionScheduler_wpPostStore();
+            $source_logger = $this->logger_classname ? new $this->logger_classname() : new \ActionScheduler_wpCommentLogger();
+
+            $config = new Config();
+            $config->set_source_store( $source_store );
+            $config->set_source_logger( $source_logger );
+            $config->set_destination_store( new \ActionScheduler_DBStoreMigrator() );
+            $config->set_destination_logger( new \ActionScheduler_DBLogger() );
+
+            if ( defined( 'WP_CLI' ) && WP_CLI ) {
+                $config->set_progress_bar( new ProgressBar( '', 0 ) );
+            }
+        }
+
+        return apply_filters( 'action_scheduler/migration_config', $config );
+    }
+
+    /**
+     * Hook dashboard migration notice.
+     */
+    public function hook_admin_notices() {
+        if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) {
+            return;
+        }
+        add_action( 'admin_notices', array( $this, 'display_migration_notice' ), 10, 0 );
+    }
+
+    /**
+     * Show a dashboard notice that migration is in progress.
+     */
+    public function display_migration_notice() {
+        printf( '<div class="notice notice-warning"><p>%s</p></div>', esc_html__( 'Action Scheduler migration in progress. The list of scheduled actions may be incomplete.', 'action-scheduler' ) );
+    }
+
+    /**
+     * Add store classes. Hook migration.
+     */
+    private function hook() {
+        add_filter( 'action_scheduler_store_class', array( $this, 'get_store_class' ), 100, 1 );
+        add_filter( 'action_scheduler_logger_class', array( $this, 'get_logger_class' ), 100, 1 );
+        add_action( 'init', array( $this, 'maybe_hook_migration' ) );
+        add_action( 'wp_loaded', array( $this, 'schedule_migration' ) );
+
+        // Action Scheduler may be displayed as a Tools screen or WooCommerce > Status administration screen
+        add_action( 'load-tools_page_action-scheduler', array( $this, 'hook_admin_notices' ), 10, 0 );
+        add_action( 'load-woocommerce_page_wc-status', array( $this, 'hook_admin_notices' ), 10, 0 );
+    }
+
+    /**
+     * Possibly hook the migration scheduler action.
+     *
+     * @author Jeremy Pry
+     */
+    public function maybe_hook_migration() {
+        if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) {
+            return;
+        }
+
+        $this->migration_scheduler->hook();
+    }
+
+    /**
+     * Allow datastores to enable migration to AS tables.
+     */
+    public function allow_migration() {
+        if ( ! \ActionScheduler_DataController::dependencies_met() ) {
+            return false;
+        }
+
+        if ( null === $this->migrate_custom_store ) {
+            $this->migrate_custom_store = apply_filters( 'action_scheduler_migrate_data_store', false );
+        }
+
+        return ( ! $this->has_custom_datastore() ) || $this->migrate_custom_store;
+    }
+
+    /**
+     * Proceed with the migration if the dependencies have been met.
+     */
+    public static function init() {
+        if ( \ActionScheduler_DataController::dependencies_met() ) {
+            self::instance()->hook();
+        }
+    }
+
+    /**
+     * Singleton factory.
+     */
+    public static function instance() {
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new static( new Scheduler() );
+        }
+
+        return self::$instance;
+    }
 }

Spacing +31 added lines, -31 removed lines

@@ -38,7 +38,7 @@ 
 	 *
 	 * @param Scheduler $migration_scheduler Migration scheduler object.
 	 */
-	protected function __construct( Scheduler $migration_scheduler ) {
+	protected function __construct(Scheduler $migration_scheduler) {
 		$this->migration_scheduler = $migration_scheduler;
 		$this->store_classname     = '';
 	}
@@ -50,10 +50,10 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_store_class( $class ) {
-		if ( \ActionScheduler_DataController::is_migration_complete() ) {
+	public function get_store_class($class) {
+		if (\ActionScheduler_DataController::is_migration_complete()) {
 			return \ActionScheduler_DataController::DATASTORE_CLASS;
-		} elseif ( \ActionScheduler_Store::DEFAULT_CLASS !== $class ) {
+		} elseif (\ActionScheduler_Store::DEFAULT_CLASS !== $class) {
 			$this->store_classname = $class;
 			return $class;
 		} else {
@@ -68,10 +68,10 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_logger_class( $class ) {
+	public function get_logger_class($class) {
 		\ActionScheduler_Store::instance();
 
-		if ( $this->has_custom_datastore() ) {
+		if ($this->has_custom_datastore()) {
 			$this->logger_classname = $class;
 			return $class;
 		} else {
@@ -127,53 +127,53 @@ 
 	public function get_migration_config_object() {
 		static $config = null;
 
-		if ( ! $config ) {
+		if ( ! $config) {
 			$source_store  = $this->store_classname ? new $this->store_classname() : new \ActionScheduler_wpPostStore();
 			$source_logger = $this->logger_classname ? new $this->logger_classname() : new \ActionScheduler_wpCommentLogger();
 
 			$config = new Config();
-			$config->set_source_store( $source_store );
-			$config->set_source_logger( $source_logger );
-			$config->set_destination_store( new \ActionScheduler_DBStoreMigrator() );
-			$config->set_destination_logger( new \ActionScheduler_DBLogger() );
+			$config->set_source_store($source_store);
+			$config->set_source_logger($source_logger);
+			$config->set_destination_store(new \ActionScheduler_DBStoreMigrator());
+			$config->set_destination_logger(new \ActionScheduler_DBLogger());
 
-			if ( defined( 'WP_CLI' ) && WP_CLI ) {
-				$config->set_progress_bar( new ProgressBar( '', 0 ) );
+			if (defined('WP_CLI') && WP_CLI) {
+				$config->set_progress_bar(new ProgressBar('', 0));
 			}
 		}
 
-		return apply_filters( 'action_scheduler/migration_config', $config );
+		return apply_filters('action_scheduler/migration_config', $config);
 	}
 
 	/**
 	 * Hook dashboard migration notice.
 	 */
 	public function hook_admin_notices() {
-		if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) {
+		if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete()) {
 			return;
 		}
-		add_action( 'admin_notices', array( $this, 'display_migration_notice' ), 10, 0 );
+		add_action('admin_notices', array($this, 'display_migration_notice'), 10, 0);
 	}
 
 	/**
 	 * Show a dashboard notice that migration is in progress.
 	 */
 	public function display_migration_notice() {
-		printf( '<div class="notice notice-warning"><p>%s</p></div>', esc_html__( 'Action Scheduler migration in progress. The list of scheduled actions may be incomplete.', 'action-scheduler' ) );
+		printf('<div class="notice notice-warning"><p>%s</p></div>', esc_html__('Action Scheduler migration in progress. The list of scheduled actions may be incomplete.', 'action-scheduler'));
 	}
 
 	/**
 	 * Add store classes. Hook migration.
 	 */
 	private function hook() {
-		add_filter( 'action_scheduler_store_class', array( $this, 'get_store_class' ), 100, 1 );
-		add_filter( 'action_scheduler_logger_class', array( $this, 'get_logger_class' ), 100, 1 );
-		add_action( 'init', array( $this, 'maybe_hook_migration' ) );
-		add_action( 'wp_loaded', array( $this, 'schedule_migration' ) );
+		add_filter('action_scheduler_store_class', array($this, 'get_store_class'), 100, 1);
+		add_filter('action_scheduler_logger_class', array($this, 'get_logger_class'), 100, 1);
+		add_action('init', array($this, 'maybe_hook_migration'));
+		add_action('wp_loaded', array($this, 'schedule_migration'));
 
 		// Action Scheduler may be displayed as a Tools screen or WooCommerce > Status administration screen
-		add_action( 'load-tools_page_action-scheduler', array( $this, 'hook_admin_notices' ), 10, 0 );
-		add_action( 'load-woocommerce_page_wc-status', array( $this, 'hook_admin_notices' ), 10, 0 );
+		add_action('load-tools_page_action-scheduler', array($this, 'hook_admin_notices'), 10, 0);
+		add_action('load-woocommerce_page_wc-status', array($this, 'hook_admin_notices'), 10, 0);
 	}
 
 	/**
@@ -182,7 +182,7 @@ 
 	 * @author Jeremy Pry
 	 */
 	public function maybe_hook_migration() {
-		if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete() ) {
+		if ( ! $this->allow_migration() || \ActionScheduler_DataController::is_migration_complete()) {
 			return;
 		}
 
@@ -193,22 +193,22 @@ 
 	 * Allow datastores to enable migration to AS tables.
 	 */
 	public function allow_migration() {
-		if ( ! \ActionScheduler_DataController::dependencies_met() ) {
+		if ( ! \ActionScheduler_DataController::dependencies_met()) {
 			return false;
 		}
 
-		if ( null === $this->migrate_custom_store ) {
-			$this->migrate_custom_store = apply_filters( 'action_scheduler_migrate_data_store', false );
+		if (null === $this->migrate_custom_store) {
+			$this->migrate_custom_store = apply_filters('action_scheduler_migrate_data_store', false);
 		}
 
-		return ( ! $this->has_custom_datastore() ) || $this->migrate_custom_store;
+		return ( ! $this->has_custom_datastore()) || $this->migrate_custom_store;
 	}
 
 	/**
 	 * Proceed with the migration if the dependencies have been met.
 	 */
 	public static function init() {
-		if ( \ActionScheduler_DataController::dependencies_met() ) {
+		if (\ActionScheduler_DataController::dependencies_met()) {
 			self::instance()->hook();
 		}
 	}
@@ -217,8 +217,8 @@ 
 	 * Singleton factory.
 	 */
 	public static function instance() {
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new static( new Scheduler() );
+		if ( ! isset(self::$instance)) {
+			self::$instance = new static(new Scheduler());
 		}
 
 		return self::$instance;

src/libraries/action-scheduler/classes/migration/Scheduler.php

Spacing +17 added lines, -17 removed lines

@@ -23,14 +23,14 @@ 
 	 * Set up the callback for the scheduled job.
 	 */
 	public function hook() {
-		add_action( self::HOOK, array( $this, 'run_migration' ), 10, 0 );
+		add_action(self::HOOK, array($this, 'run_migration'), 10, 0);
 	}
 
 	/**
 	 * Remove the callback for the scheduled job.
 	 */
 	public function unhook() {
-		remove_action( self::HOOK, array( $this, 'run_migration' ), 10 );
+		remove_action(self::HOOK, array($this, 'run_migration'), 10);
 	}
 
 	/**
@@ -38,12 +38,12 @@ 
 	 */
 	public function run_migration() {
 		$migration_runner = $this->get_migration_runner();
-		$count            = $migration_runner->run( $this->get_batch_size() );
+		$count            = $migration_runner->run($this->get_batch_size());
 
-		if ( $count === 0 ) {
+		if ($count === 0) {
 			$this->mark_complete();
 		} else {
-			$this->schedule_migration( time() + $this->get_schedule_interval() );
+			$this->schedule_migration(time() + $this->get_schedule_interval());
 		}
 	}
 
@@ -54,7 +54,7 @@ 
 		$this->unschedule_migration();
 
 		\ActionScheduler_DataController::mark_migration_complete();
-		do_action( 'action_scheduler/migration_complete' );
+		do_action('action_scheduler/migration_complete');
 	}
 
 	/**
@@ -63,9 +63,9 @@ 
 	 * @return bool Whether there is a pending action in the store to handle the migration
 	 */
 	public function is_migration_scheduled() {
-		$next = as_next_scheduled_action( self::HOOK );
+		$next = as_next_scheduled_action(self::HOOK);
 
-		return ! empty( $next );
+		return ! empty($next);
 	}
 
 	/**
@@ -75,25 +75,25 @@ 
 	 *
 	 * @return string The action ID
 	 */
-	public function schedule_migration( $when = 0 ) {
-		$next = as_next_scheduled_action( self::HOOK );
+	public function schedule_migration($when = 0) {
+		$next = as_next_scheduled_action(self::HOOK);
 
-		if ( ! empty( $next ) ) {
+		if ( ! empty($next)) {
 			return $next;
 		}
 
-		if ( empty( $when ) ) {
+		if (empty($when)) {
 			$when = time() + MINUTE_IN_SECONDS;
 		}
 
-		return as_schedule_single_action( $when, self::HOOK, array(), self::GROUP );
+		return as_schedule_single_action($when, self::HOOK, array(), self::GROUP);
 	}
 
 	/**
 	 * Remove the scheduled migration action.
 	 */
 	public function unschedule_migration() {
-		as_unschedule_action( self::HOOK, null, self::GROUP );
+		as_unschedule_action(self::HOOK, null, self::GROUP);
 	}
 
 	/**
@@ -102,7 +102,7 @@ 
 	 * @return int Seconds between migration runs. Defaults to 0 seconds to allow chaining migration via Async Runners.
 	 */
 	private function get_schedule_interval() {
-		return (int) apply_filters( 'action_scheduler/migration_interval', 0 );
+		return (int) apply_filters('action_scheduler/migration_interval', 0);
 	}
 
 	/**
@@ -111,7 +111,7 @@ 
 	 * @return int Number of actions to migrate in each batch. Defaults to 250.
 	 */
 	private function get_batch_size() {
-		return (int) apply_filters( 'action_scheduler/migration_batch_size', 250 );
+		return (int) apply_filters('action_scheduler/migration_batch_size', 250);
 	}
 
 	/**
@@ -122,7 +122,7 @@ 
 	private function get_migration_runner() {
 		$config = Controller::instance()->get_migration_config_object();
 
-		return new Runner( $config );
+		return new Runner($config);
 	}
 
 }

Indentation +111 added lines, -111 removed lines

@@ -12,116 +12,116 @@
  * @codeCoverageIgnore
  */
 class Scheduler {
-	/** Migration action hook. */
-	const HOOK = 'action_scheduler/migration_hook';
-
-	/** Migration action group. */
-	const GROUP = 'action-scheduler-migration';
-
-	/**
-	 * Set up the callback for the scheduled job.
-	 */
-	public function hook() {
-		add_action( self::HOOK, array( $this, 'run_migration' ), 10, 0 );
-	}
-
-	/**
-	 * Remove the callback for the scheduled job.
-	 */
-	public function unhook() {
-		remove_action( self::HOOK, array( $this, 'run_migration' ), 10 );
-	}
-
-	/**
-	 * The migration callback.
-	 */
-	public function run_migration() {
-		$migration_runner = $this->get_migration_runner();
-		$count            = $migration_runner->run( $this->get_batch_size() );
-
-		if ( $count === 0 ) {
-			$this->mark_complete();
-		} else {
-			$this->schedule_migration( time() + $this->get_schedule_interval() );
-		}
-	}
-
-	/**
-	 * Mark the migration complete.
-	 */
-	public function mark_complete() {
-		$this->unschedule_migration();
-
-		\ActionScheduler_DataController::mark_migration_complete();
-		do_action( 'action_scheduler/migration_complete' );
-	}
-
-	/**
-	 * Get a flag indicating whether the migration is scheduled.
-	 *
-	 * @return bool Whether there is a pending action in the store to handle the migration
-	 */
-	public function is_migration_scheduled() {
-		$next = as_next_scheduled_action( self::HOOK );
-
-		return ! empty( $next );
-	}
-
-	/**
-	 * Schedule the migration.
-	 *
-	 * @param int $when Optional timestamp to run the next migration batch. Defaults to now.
-	 *
-	 * @return string The action ID
-	 */
-	public function schedule_migration( $when = 0 ) {
-		$next = as_next_scheduled_action( self::HOOK );
-
-		if ( ! empty( $next ) ) {
-			return $next;
-		}
-
-		if ( empty( $when ) ) {
-			$when = time() + MINUTE_IN_SECONDS;
-		}
-
-		return as_schedule_single_action( $when, self::HOOK, array(), self::GROUP );
-	}
-
-	/**
-	 * Remove the scheduled migration action.
-	 */
-	public function unschedule_migration() {
-		as_unschedule_action( self::HOOK, null, self::GROUP );
-	}
-
-	/**
-	 * Get migration batch schedule interval.
-	 *
-	 * @return int Seconds between migration runs. Defaults to 0 seconds to allow chaining migration via Async Runners.
-	 */
-	private function get_schedule_interval() {
-		return (int) apply_filters( 'action_scheduler/migration_interval', 0 );
-	}
-
-	/**
-	 * Get migration batch size.
-	 *
-	 * @return int Number of actions to migrate in each batch. Defaults to 250.
-	 */
-	private function get_batch_size() {
-		return (int) apply_filters( 'action_scheduler/migration_batch_size', 250 );
-	}
-
-	/**
-	 * Get migration runner object.
-	 *
-	 * @return Runner
-	 */
-	private function get_migration_runner() {
-		$config = Controller::instance()->get_migration_config_object();
-
-		return new Runner( $config );
-	}
+    /** Migration action hook. */
+    const HOOK = 'action_scheduler/migration_hook';
+
+    /** Migration action group. */
+    const GROUP = 'action-scheduler-migration';
+
+    /**
+     * Set up the callback for the scheduled job.
+     */
+    public function hook() {
+        add_action( self::HOOK, array( $this, 'run_migration' ), 10, 0 );
+    }
+
+    /**
+     * Remove the callback for the scheduled job.
+     */
+    public function unhook() {
+        remove_action( self::HOOK, array( $this, 'run_migration' ), 10 );
+    }
+
+    /**
+     * The migration callback.
+     */
+    public function run_migration() {
+        $migration_runner = $this->get_migration_runner();
+        $count            = $migration_runner->run( $this->get_batch_size() );
+
+        if ( $count === 0 ) {
+            $this->mark_complete();
+        } else {
+            $this->schedule_migration( time() + $this->get_schedule_interval() );
+        }
+    }
+
+    /**
+     * Mark the migration complete.
+     */
+    public function mark_complete() {
+        $this->unschedule_migration();
+
+        \ActionScheduler_DataController::mark_migration_complete();
+        do_action( 'action_scheduler/migration_complete' );
+    }
+
+    /**
+     * Get a flag indicating whether the migration is scheduled.
+     *
+     * @return bool Whether there is a pending action in the store to handle the migration
+     */
+    public function is_migration_scheduled() {
+        $next = as_next_scheduled_action( self::HOOK );
+
+        return ! empty( $next );
+    }
+
+    /**
+     * Schedule the migration.
+     *
+     * @param int $when Optional timestamp to run the next migration batch. Defaults to now.
+     *
+     * @return string The action ID
+     */
+    public function schedule_migration( $when = 0 ) {
+        $next = as_next_scheduled_action( self::HOOK );
+
+        if ( ! empty( $next ) ) {
+            return $next;
+        }
+
+        if ( empty( $when ) ) {
+            $when = time() + MINUTE_IN_SECONDS;
+        }
+
+        return as_schedule_single_action( $when, self::HOOK, array(), self::GROUP );
+    }
+
+    /**
+     * Remove the scheduled migration action.
+     */
+    public function unschedule_migration() {
+        as_unschedule_action( self::HOOK, null, self::GROUP );
+    }
+
+    /**
+     * Get migration batch schedule interval.
+     *
+     * @return int Seconds between migration runs. Defaults to 0 seconds to allow chaining migration via Async Runners.
+     */
+    private function get_schedule_interval() {
+        return (int) apply_filters( 'action_scheduler/migration_interval', 0 );
+    }
+
+    /**
+     * Get migration batch size.
+     *
+     * @return int Number of actions to migrate in each batch. Defaults to 250.
+     */
+    private function get_batch_size() {
+        return (int) apply_filters( 'action_scheduler/migration_batch_size', 250 );
+    }
+
+    /**
+     * Get migration runner object.
+     *
+     * @return Runner
+     */
+    private function get_migration_runner() {
+        $config = Controller::instance()->get_migration_config_object();
+
+        return new Runner( $config );
+    }
 
 }

src/libraries/action-scheduler/classes/data-stores/ActionScheduler_wpPostStore_PostStatusRegistrar.php

Indentation +44 added lines, -44 removed lines

@@ -5,54 +5,54 @@
  * @codeCoverageIgnore
  */
 class ActionScheduler_wpPostStore_PostStatusRegistrar {
-	public function register() {
-		register_post_status( ActionScheduler_Store::STATUS_RUNNING, array_merge( $this->post_status_args(), $this->post_status_running_labels() ) );
-		register_post_status( ActionScheduler_Store::STATUS_FAILED, array_merge( $this->post_status_args(), $this->post_status_failed_labels() ) );
-	}
+    public function register() {
+        register_post_status( ActionScheduler_Store::STATUS_RUNNING, array_merge( $this->post_status_args(), $this->post_status_running_labels() ) );
+        register_post_status( ActionScheduler_Store::STATUS_FAILED, array_merge( $this->post_status_args(), $this->post_status_failed_labels() ) );
+    }
 
-	/**
-	 * Build the args array for the post type definition
-	 *
-	 * @return array
-	 */
-	protected function post_status_args() {
-		$args = array(
-			'public'                    => false,
-			'exclude_from_search'       => false,
-			'show_in_admin_all_list'    => true,
-			'show_in_admin_status_list' => true,
-		);
+    /**
+     * Build the args array for the post type definition
+     *
+     * @return array
+     */
+    protected function post_status_args() {
+        $args = array(
+            'public'                    => false,
+            'exclude_from_search'       => false,
+            'show_in_admin_all_list'    => true,
+            'show_in_admin_status_list' => true,
+        );
 
-		return apply_filters( 'action_scheduler_post_status_args', $args );
-	}
+        return apply_filters( 'action_scheduler_post_status_args', $args );
+    }
 
-	/**
-	 * Build the args array for the post type definition
-	 *
-	 * @return array
-	 */
-	protected function post_status_failed_labels() {
-		$labels = array(
-			'label'       => _x( 'Failed', 'post', 'action-scheduler' ),
-			/* translators: %s: count */
-			'label_count' => _n_noop( 'Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'action-scheduler' ),
-		);
+    /**
+     * Build the args array for the post type definition
+     *
+     * @return array
+     */
+    protected function post_status_failed_labels() {
+        $labels = array(
+            'label'       => _x( 'Failed', 'post', 'action-scheduler' ),
+            /* translators: %s: count */
+            'label_count' => _n_noop( 'Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'action-scheduler' ),
+        );
 
-		return apply_filters( 'action_scheduler_post_status_failed_labels', $labels );
-	}
+        return apply_filters( 'action_scheduler_post_status_failed_labels', $labels );
+    }
 
-	/**
-	 * Build the args array for the post type definition
-	 *
-	 * @return array
-	 */
-	protected function post_status_running_labels() {
-		$labels = array(
-			'label'       => _x( 'In-Progress', 'post', 'action-scheduler' ),
-			/* translators: %s: count */
-			'label_count' => _n_noop( 'In-Progress <span class="count">(%s)</span>', 'In-Progress <span class="count">(%s)</span>', 'action-scheduler' ),
-		);
+    /**
+     * Build the args array for the post type definition
+     *
+     * @return array
+     */
+    protected function post_status_running_labels() {
+        $labels = array(
+            'label'       => _x( 'In-Progress', 'post', 'action-scheduler' ),
+            /* translators: %s: count */
+            'label_count' => _n_noop( 'In-Progress <span class="count">(%s)</span>', 'In-Progress <span class="count">(%s)</span>', 'action-scheduler' ),
+        );
 
-		return apply_filters( 'action_scheduler_post_status_running_labels', $labels );
-	}
+        return apply_filters( 'action_scheduler_post_status_running_labels', $labels );
+    }
 }

Spacing +9 added lines, -9 removed lines

@@ -6,8 +6,8 @@ 
  */
 class ActionScheduler_wpPostStore_PostStatusRegistrar {
 	public function register() {
-		register_post_status( ActionScheduler_Store::STATUS_RUNNING, array_merge( $this->post_status_args(), $this->post_status_running_labels() ) );
-		register_post_status( ActionScheduler_Store::STATUS_FAILED, array_merge( $this->post_status_args(), $this->post_status_failed_labels() ) );
+		register_post_status(ActionScheduler_Store::STATUS_RUNNING, array_merge($this->post_status_args(), $this->post_status_running_labels()));
+		register_post_status(ActionScheduler_Store::STATUS_FAILED, array_merge($this->post_status_args(), $this->post_status_failed_labels()));
 	}
 
 	/**
@@ -23,7 +23,7 @@ 
 			'show_in_admin_status_list' => true,
 		);
 
-		return apply_filters( 'action_scheduler_post_status_args', $args );
+		return apply_filters('action_scheduler_post_status_args', $args);
 	}
 
 	/**
@@ -33,12 +33,12 @@ 
 	 */
 	protected function post_status_failed_labels() {
 		$labels = array(
-			'label'       => _x( 'Failed', 'post', 'action-scheduler' ),
+			'label'       => _x('Failed', 'post', 'action-scheduler'),
 			/* translators: %s: count */
-			'label_count' => _n_noop( 'Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'action-scheduler' ),
+			'label_count' => _n_noop('Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'action-scheduler'),
 		);
 
-		return apply_filters( 'action_scheduler_post_status_failed_labels', $labels );
+		return apply_filters('action_scheduler_post_status_failed_labels', $labels);
 	}
 
 	/**
@@ -48,11 +48,11 @@ 
 	 */
 	protected function post_status_running_labels() {
 		$labels = array(
-			'label'       => _x( 'In-Progress', 'post', 'action-scheduler' ),
+			'label'       => _x('In-Progress', 'post', 'action-scheduler'),
 			/* translators: %s: count */
-			'label_count' => _n_noop( 'In-Progress <span class="count">(%s)</span>', 'In-Progress <span class="count">(%s)</span>', 'action-scheduler' ),
+			'label_count' => _n_noop('In-Progress <span class="count">(%s)</span>', 'In-Progress <span class="count">(%s)</span>', 'action-scheduler'),
 		);
 
-		return apply_filters( 'action_scheduler_post_status_running_labels', $labels );
+		return apply_filters('action_scheduler_post_status_running_labels', $labels);
 	}
 }

src/libraries/action-scheduler/classes/ActionScheduler_DataController.php

Spacing +25 added lines, -25 removed lines

@@ -44,8 +44,8 @@ 
 	 * @return bool
 	 */
 	public static function dependencies_met() {
-		$php_support = version_compare( PHP_VERSION, self::MIN_PHP_VERSION, '>=' );
-		return $php_support && apply_filters( 'action_scheduler_migration_dependencies_met', true );
+		$php_support = version_compare(PHP_VERSION, self::MIN_PHP_VERSION, '>=');
+		return $php_support && apply_filters('action_scheduler_migration_dependencies_met', true);
 	}
 
 	/**
@@ -54,14 +54,14 @@ 
 	 * @return bool Whether the flag has been set marking the migration as complete
 	 */
 	public static function is_migration_complete() {
-		return get_option( self::STATUS_FLAG ) === self::STATUS_COMPLETE;
+		return get_option(self::STATUS_FLAG) === self::STATUS_COMPLETE;
 	}
 
 	/**
 	 * Mark the migration as complete.
 	 */
 	public static function mark_migration_complete() {
-		update_option( self::STATUS_FLAG, self::STATUS_COMPLETE );
+		update_option(self::STATUS_FLAG, self::STATUS_COMPLETE);
 	}
 
 	/**
@@ -70,7 +70,7 @@ 
 	 * deactivated and the site was running on AS 2.x again.
 	 */
 	public static function mark_migration_incomplete() {
-		delete_option( self::STATUS_FLAG );
+		delete_option(self::STATUS_FLAG);
 	}
 
 	/**
@@ -80,7 +80,7 @@ 
 	 *
 	 * @return string
 	 */
-	public static function set_store_class( $class ) {
+	public static function set_store_class($class) {
 		return self::DATASTORE_CLASS;
 	}
 
@@ -91,7 +91,7 @@ 
 	 *
 	 * @return string
 	 */
-	public static function set_logger_class( $class ) {
+	public static function set_logger_class($class) {
 		return self::LOGGER_CLASS;
 	}
 
@@ -100,7 +100,7 @@ 
 	 *
 	 * @param integer $sleep_time The number of seconds to pause before resuming operation.
 	 */
-	public static function set_sleep_time( $sleep_time ) {
+	public static function set_sleep_time($sleep_time) {
 		self::$sleep_time = (int) $sleep_time;
 	}
 
@@ -109,7 +109,7 @@ 
 	 *
 	 * @param integer $free_ticks The number of ticks to free memory on.
 	 */
-	public static function set_free_ticks( $free_ticks ) {
+	public static function set_free_ticks($free_ticks) {
 		self::$free_ticks = (int) $free_ticks;
 	}
 
@@ -118,8 +118,8 @@ 
 	 *
 	 * @param int $ticks Current tick count.
 	 */
-	public static function maybe_free_memory( $ticks ) {
-		if ( self::$free_ticks && 0 === $ticks % self::$free_ticks ) {
+	public static function maybe_free_memory($ticks) {
+		if (self::$free_ticks && 0 === $ticks % self::$free_ticks) {
 			self::free_memory();
 		}
 	}
@@ -128,13 +128,13 @@ 
 	 * Reduce memory footprint by clearing the database query and object caches.
 	 */
 	public static function free_memory() {
-		if ( 0 < self::$sleep_time ) {
+		if (0 < self::$sleep_time) {
 			/* translators: %d: amount of time */
-			\WP_CLI::warning( sprintf( _n( 'Stopped the insanity for %d second', 'Stopped the insanity for %d seconds', self::$sleep_time, 'action-scheduler' ), self::$sleep_time ) );
-			sleep( self::$sleep_time );
+			\WP_CLI::warning(sprintf(_n('Stopped the insanity for %d second', 'Stopped the insanity for %d seconds', self::$sleep_time, 'action-scheduler'), self::$sleep_time));
+			sleep(self::$sleep_time);
 		}
 
-		\WP_CLI::warning( __( 'Attempting to reduce used memory...', 'action-scheduler' ) );
+		\WP_CLI::warning(__('Attempting to reduce used memory...', 'action-scheduler'));
 
 		/**
 		 * @var $wpdb            \wpdb
@@ -144,7 +144,7 @@ 
 
 		$wpdb->queries = array();
 
-		if ( ! is_a( $wp_object_cache, 'WP_Object_Cache' ) ) {
+		if ( ! is_a($wp_object_cache, 'WP_Object_Cache')) {
 			return;
 		}
 
@@ -153,8 +153,8 @@ 
 		$wp_object_cache->memcache_debug = array();
 		$wp_object_cache->cache          = array();
 
-		if ( is_callable( array( $wp_object_cache, '__remoteset' ) ) ) {
-			call_user_func( array( $wp_object_cache, '__remoteset' ) ); // important
+		if (is_callable(array($wp_object_cache, '__remoteset'))) {
+			call_user_func(array($wp_object_cache, '__remoteset')); // important
 		}
 	}
 
@@ -163,22 +163,22 @@ 
 	 * Otherwise, proceed with the migration if the dependencies have been met.
 	 */
 	public static function init() {
-		if ( self::is_migration_complete() ) {
-			add_filter( 'action_scheduler_store_class', array( 'ActionScheduler_DataController', 'set_store_class' ), 100 );
-			add_filter( 'action_scheduler_logger_class', array( 'ActionScheduler_DataController', 'set_logger_class' ), 100 );
-			add_action( 'deactivate_plugin', array( 'ActionScheduler_DataController', 'mark_migration_incomplete' ) );
-		} elseif ( self::dependencies_met() ) {
+		if (self::is_migration_complete()) {
+			add_filter('action_scheduler_store_class', array('ActionScheduler_DataController', 'set_store_class'), 100);
+			add_filter('action_scheduler_logger_class', array('ActionScheduler_DataController', 'set_logger_class'), 100);
+			add_action('deactivate_plugin', array('ActionScheduler_DataController', 'mark_migration_incomplete'));
+		} elseif (self::dependencies_met()) {
 			Controller::init();
 		}
 
-		add_action( 'action_scheduler/progress_tick', array( 'ActionScheduler_DataController', 'maybe_free_memory' ) );
+		add_action('action_scheduler/progress_tick', array('ActionScheduler_DataController', 'maybe_free_memory'));
 	}
 
 	/**
 	 * Singleton factory.
 	 */
 	public static function instance() {
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			self::$instance = new static();
 		}
 

Indentation +170 added lines, -170 removed lines

@@ -14,174 +14,174 @@
  * @since 3.0.0
  */
 class ActionScheduler_DataController {
-	/** Action data store class name. */
-	const DATASTORE_CLASS = 'ActionScheduler_DBStore';
-
-	/** Logger data store class name. */
-	const LOGGER_CLASS = 'ActionScheduler_DBLogger';
-
-	/** Migration status option name. */
-	const STATUS_FLAG = 'action_scheduler_migration_status';
-
-	/** Migration status option value. */
-	const STATUS_COMPLETE = 'complete';
-
-	/** Migration minimum required PHP version. */
-	const MIN_PHP_VERSION = '5.5';
-
-	/** @var ActionScheduler_DataController */
-	private static $instance;
-
-	/** @var int */
-	private static $sleep_time = 0;
-
-	/** @var int */
-	private static $free_ticks = 50;
-
-	/**
-	 * Get a flag indicating whether the migration environment dependencies are met.
-	 *
-	 * @return bool
-	 */
-	public static function dependencies_met() {
-		$php_support = version_compare( PHP_VERSION, self::MIN_PHP_VERSION, '>=' );
-		return $php_support && apply_filters( 'action_scheduler_migration_dependencies_met', true );
-	}
-
-	/**
-	 * Get a flag indicating whether the migration is complete.
-	 *
-	 * @return bool Whether the flag has been set marking the migration as complete
-	 */
-	public static function is_migration_complete() {
-		return get_option( self::STATUS_FLAG ) === self::STATUS_COMPLETE;
-	}
-
-	/**
-	 * Mark the migration as complete.
-	 */
-	public static function mark_migration_complete() {
-		update_option( self::STATUS_FLAG, self::STATUS_COMPLETE );
-	}
-
-	/**
-	 * Unmark migration when a plugin is de-activated. Will not work in case of silent activation, for example in an update.
-	 * We do this to mitigate the bug of lost actions which happens if there was an AS 2.x to AS 3.x migration in the past, but that plugin is now
-	 * deactivated and the site was running on AS 2.x again.
-	 */
-	public static function mark_migration_incomplete() {
-		delete_option( self::STATUS_FLAG );
-	}
-
-	/**
-	 * Set the action store class name.
-	 *
-	 * @param string $class Classname of the store class.
-	 *
-	 * @return string
-	 */
-	public static function set_store_class( $class ) {
-		return self::DATASTORE_CLASS;
-	}
-
-	/**
-	 * Set the action logger class name.
-	 *
-	 * @param string $class Classname of the logger class.
-	 *
-	 * @return string
-	 */
-	public static function set_logger_class( $class ) {
-		return self::LOGGER_CLASS;
-	}
-
-	/**
-	 * Set the sleep time in seconds.
-	 *
-	 * @param integer $sleep_time The number of seconds to pause before resuming operation.
-	 */
-	public static function set_sleep_time( $sleep_time ) {
-		self::$sleep_time = (int) $sleep_time;
-	}
-
-	/**
-	 * Set the tick count required for freeing memory.
-	 *
-	 * @param integer $free_ticks The number of ticks to free memory on.
-	 */
-	public static function set_free_ticks( $free_ticks ) {
-		self::$free_ticks = (int) $free_ticks;
-	}
-
-	/**
-	 * Free memory if conditions are met.
-	 *
-	 * @param int $ticks Current tick count.
-	 */
-	public static function maybe_free_memory( $ticks ) {
-		if ( self::$free_ticks && 0 === $ticks % self::$free_ticks ) {
-			self::free_memory();
-		}
-	}
-
-	/**
-	 * Reduce memory footprint by clearing the database query and object caches.
-	 */
-	public static function free_memory() {
-		if ( 0 < self::$sleep_time ) {
-			/* translators: %d: amount of time */
-			\WP_CLI::warning( sprintf( _n( 'Stopped the insanity for %d second', 'Stopped the insanity for %d seconds', self::$sleep_time, 'action-scheduler' ), self::$sleep_time ) );
-			sleep( self::$sleep_time );
-		}
-
-		\WP_CLI::warning( __( 'Attempting to reduce used memory...', 'action-scheduler' ) );
-
-		/**
-		 * @var $wpdb            \wpdb
-		 * @var $wp_object_cache \WP_Object_Cache
-		 */
-		global $wpdb, $wp_object_cache;
-
-		$wpdb->queries = array();
-
-		if ( ! is_a( $wp_object_cache, 'WP_Object_Cache' ) ) {
-			return;
-		}
-
-		$wp_object_cache->group_ops      = array();
-		$wp_object_cache->stats          = array();
-		$wp_object_cache->memcache_debug = array();
-		$wp_object_cache->cache          = array();
-
-		if ( is_callable( array( $wp_object_cache, '__remoteset' ) ) ) {
-			call_user_func( array( $wp_object_cache, '__remoteset' ) ); // important
-		}
-	}
-
-	/**
-	 * Connect to table datastores if migration is complete.
-	 * Otherwise, proceed with the migration if the dependencies have been met.
-	 */
-	public static function init() {
-		if ( self::is_migration_complete() ) {
-			add_filter( 'action_scheduler_store_class', array( 'ActionScheduler_DataController', 'set_store_class' ), 100 );
-			add_filter( 'action_scheduler_logger_class', array( 'ActionScheduler_DataController', 'set_logger_class' ), 100 );
-			add_action( 'deactivate_plugin', array( 'ActionScheduler_DataController', 'mark_migration_incomplete' ) );
-		} elseif ( self::dependencies_met() ) {
-			Controller::init();
-		}
-
-		add_action( 'action_scheduler/progress_tick', array( 'ActionScheduler_DataController', 'maybe_free_memory' ) );
-	}
-
-	/**
-	 * Singleton factory.
-	 */
-	public static function instance() {
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new static();
-		}
-
-		return self::$instance;
-	}
+    /** Action data store class name. */
+    const DATASTORE_CLASS = 'ActionScheduler_DBStore';
+
+    /** Logger data store class name. */
+    const LOGGER_CLASS = 'ActionScheduler_DBLogger';
+
+    /** Migration status option name. */
+    const STATUS_FLAG = 'action_scheduler_migration_status';
+
+    /** Migration status option value. */
+    const STATUS_COMPLETE = 'complete';
+
+    /** Migration minimum required PHP version. */
+    const MIN_PHP_VERSION = '5.5';
+
+    /** @var ActionScheduler_DataController */
+    private static $instance;
+
+    /** @var int */
+    private static $sleep_time = 0;
+
+    /** @var int */
+    private static $free_ticks = 50;
+
+    /**
+     * Get a flag indicating whether the migration environment dependencies are met.
+     *
+     * @return bool
+     */
+    public static function dependencies_met() {
+        $php_support = version_compare( PHP_VERSION, self::MIN_PHP_VERSION, '>=' );
+        return $php_support && apply_filters( 'action_scheduler_migration_dependencies_met', true );
+    }
+
+    /**
+     * Get a flag indicating whether the migration is complete.
+     *
+     * @return bool Whether the flag has been set marking the migration as complete
+     */
+    public static function is_migration_complete() {
+        return get_option( self::STATUS_FLAG ) === self::STATUS_COMPLETE;
+    }
+
+    /**
+     * Mark the migration as complete.
+     */
+    public static function mark_migration_complete() {
+        update_option( self::STATUS_FLAG, self::STATUS_COMPLETE );
+    }
+
+    /**
+     * Unmark migration when a plugin is de-activated. Will not work in case of silent activation, for example in an update.
+     * We do this to mitigate the bug of lost actions which happens if there was an AS 2.x to AS 3.x migration in the past, but that plugin is now
+     * deactivated and the site was running on AS 2.x again.
+     */
+    public static function mark_migration_incomplete() {
+        delete_option( self::STATUS_FLAG );
+    }
+
+    /**
+     * Set the action store class name.
+     *
+     * @param string $class Classname of the store class.
+     *
+     * @return string
+     */
+    public static function set_store_class( $class ) {
+        return self::DATASTORE_CLASS;
+    }
+
+    /**
+     * Set the action logger class name.
+     *
+     * @param string $class Classname of the logger class.
+     *
+     * @return string
+     */
+    public static function set_logger_class( $class ) {
+        return self::LOGGER_CLASS;
+    }
+
+    /**
+     * Set the sleep time in seconds.
+     *
+     * @param integer $sleep_time The number of seconds to pause before resuming operation.
+     */
+    public static function set_sleep_time( $sleep_time ) {
+        self::$sleep_time = (int) $sleep_time;
+    }
+
+    /**
+     * Set the tick count required for freeing memory.
+     *
+     * @param integer $free_ticks The number of ticks to free memory on.
+     */
+    public static function set_free_ticks( $free_ticks ) {
+        self::$free_ticks = (int) $free_ticks;
+    }
+
+    /**
+     * Free memory if conditions are met.
+     *
+     * @param int $ticks Current tick count.
+     */
+    public static function maybe_free_memory( $ticks ) {
+        if ( self::$free_ticks && 0 === $ticks % self::$free_ticks ) {
+            self::free_memory();
+        }
+    }
+
+    /**
+     * Reduce memory footprint by clearing the database query and object caches.
+     */
+    public static function free_memory() {
+        if ( 0 < self::$sleep_time ) {
+            /* translators: %d: amount of time */
+            \WP_CLI::warning( sprintf( _n( 'Stopped the insanity for %d second', 'Stopped the insanity for %d seconds', self::$sleep_time, 'action-scheduler' ), self::$sleep_time ) );
+            sleep( self::$sleep_time );
+        }
+
+        \WP_CLI::warning( __( 'Attempting to reduce used memory...', 'action-scheduler' ) );
+
+        /**
+         * @var $wpdb            \wpdb
+         * @var $wp_object_cache \WP_Object_Cache
+         */
+        global $wpdb, $wp_object_cache;
+
+        $wpdb->queries = array();
+
+        if ( ! is_a( $wp_object_cache, 'WP_Object_Cache' ) ) {
+            return;
+        }
+
+        $wp_object_cache->group_ops      = array();
+        $wp_object_cache->stats          = array();
+        $wp_object_cache->memcache_debug = array();
+        $wp_object_cache->cache          = array();
+
+        if ( is_callable( array( $wp_object_cache, '__remoteset' ) ) ) {
+            call_user_func( array( $wp_object_cache, '__remoteset' ) ); // important
+        }
+    }
+
+    /**
+     * Connect to table datastores if migration is complete.
+     * Otherwise, proceed with the migration if the dependencies have been met.
+     */
+    public static function init() {
+        if ( self::is_migration_complete() ) {
+            add_filter( 'action_scheduler_store_class', array( 'ActionScheduler_DataController', 'set_store_class' ), 100 );
+            add_filter( 'action_scheduler_logger_class', array( 'ActionScheduler_DataController', 'set_logger_class' ), 100 );
+            add_action( 'deactivate_plugin', array( 'ActionScheduler_DataController', 'mark_migration_incomplete' ) );
+        } elseif ( self::dependencies_met() ) {
+            Controller::init();
+        }
+
+        add_action( 'action_scheduler/progress_tick', array( 'ActionScheduler_DataController', 'maybe_free_memory' ) );
+    }
+
+    /**
+     * Singleton factory.
+     */
+    public static function instance() {
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new static();
+        }
+
+        return self::$instance;
+    }
 }

src/libraries/action-scheduler/lib/WP_Async_Request.php

Indentation +174 added lines, -174 removed lines

@@ -14,178 +14,178 @@
 
 if ( ! class_exists( 'WP_Async_Request' ) ) {
 
-	/**
-	 * Abstract WP_Async_Request class.
-	 *
-	 * @abstract
-	 */
-	abstract class WP_Async_Request {
-
-		/**
-		 * Prefix
-		 *
-		 * (default value: 'wp')
-		 *
-		 * @var string
-		 * @access protected
-		 */
-		protected $prefix = 'wp';
-
-		/**
-		 * Action
-		 *
-		 * (default value: 'async_request')
-		 *
-		 * @var string
-		 * @access protected
-		 */
-		protected $action = 'async_request';
-
-		/**
-		 * Identifier
-		 *
-		 * @var mixed
-		 * @access protected
-		 */
-		protected $identifier;
-
-		/**
-		 * Data
-		 *
-		 * (default value: array())
-		 *
-		 * @var array
-		 * @access protected
-		 */
-		protected $data = array();
-
-		/**
-		 * Initiate new async request
-		 */
-		public function __construct() {
-			$this->identifier = $this->prefix . '_' . $this->action;
-
-			add_action( 'wp_ajax_' . $this->identifier, array( $this, 'maybe_handle' ) );
-			add_action( 'wp_ajax_nopriv_' . $this->identifier, array( $this, 'maybe_handle' ) );
-		}
-
-		/**
-		 * Set data used during the request
-		 *
-		 * @param array $data Data.
-		 *
-		 * @return $this
-		 */
-		public function data( $data ) {
-			$this->data = $data;
-
-			return $this;
-		}
-
-		/**
-		 * Dispatch the async request
-		 *
-		 * @return array|WP_Error
-		 */
-		public function dispatch() {
-			$url  = add_query_arg( $this->get_query_args(), $this->get_query_url() );
-			$args = $this->get_post_args();
-
-			return wp_remote_post( esc_url_raw( $url ), $args );
-		}
-
-		/**
-		 * Get query args
-		 *
-		 * @return array
-		 */
-		protected function get_query_args() {
-			if ( property_exists( $this, 'query_args' ) ) {
-				return $this->query_args;
-			}
-
-			$args = array(
-				'action' => $this->identifier,
-				'nonce'  => wp_create_nonce( $this->identifier ),
-			);
-
-			/**
-			 * Filters the post arguments used during an async request.
-			 *
-			 * @param array $url
-			 */
-			return apply_filters( $this->identifier . '_query_args', $args );
-		}
-
-		/**
-		 * Get query URL
-		 *
-		 * @return string
-		 */
-		protected function get_query_url() {
-			if ( property_exists( $this, 'query_url' ) ) {
-				return $this->query_url;
-			}
-
-			$url = admin_url( 'admin-ajax.php' );
-
-			/**
-			 * Filters the post arguments used during an async request.
-			 *
-			 * @param string $url
-			 */
-			return apply_filters( $this->identifier . '_query_url', $url );
-		}
-
-		/**
-		 * Get post args
-		 *
-		 * @return array
-		 */
-		protected function get_post_args() {
-			if ( property_exists( $this, 'post_args' ) ) {
-				return $this->post_args;
-			}
-
-			$args = array(
-				'timeout'   => 0.01,
-				'blocking'  => false,
-				'body'      => $this->data,
-				'cookies'   => $_COOKIE,
-				'sslverify' => apply_filters( 'https_local_ssl_verify', false ),
-			);
-
-			/**
-			 * Filters the post arguments used during an async request.
-			 *
-			 * @param array $args
-			 */
-			return apply_filters( $this->identifier . '_post_args', $args );
-		}
-
-		/**
-		 * Maybe handle
-		 *
-		 * Check for correct nonce and pass to handler.
-		 */
-		public function maybe_handle() {
-			// Don't lock up other requests while processing
-			session_write_close();
-
-			check_ajax_referer( $this->identifier, 'nonce' );
-
-			$this->handle();
-
-			wp_die();
-		}
-
-		/**
-		 * Handle
-		 *
-		 * Override this method to perform any actions required
-		 * during the async request.
-		 */
-		abstract protected function handle();
-
-	}
+    /**
+     * Abstract WP_Async_Request class.
+     *
+     * @abstract
+     */
+    abstract class WP_Async_Request {
+
+        /**
+         * Prefix
+         *
+         * (default value: 'wp')
+         *
+         * @var string
+         * @access protected
+         */
+        protected $prefix = 'wp';
+
+        /**
+         * Action
+         *
+         * (default value: 'async_request')
+         *
+         * @var string
+         * @access protected
+         */
+        protected $action = 'async_request';
+
+        /**
+         * Identifier
+         *
+         * @var mixed
+         * @access protected
+         */
+        protected $identifier;
+
+        /**
+         * Data
+         *
+         * (default value: array())
+         *
+         * @var array
+         * @access protected
+         */
+        protected $data = array();
+
+        /**
+         * Initiate new async request
+         */
+        public function __construct() {
+            $this->identifier = $this->prefix . '_' . $this->action;
+
+            add_action( 'wp_ajax_' . $this->identifier, array( $this, 'maybe_handle' ) );
+            add_action( 'wp_ajax_nopriv_' . $this->identifier, array( $this, 'maybe_handle' ) );
+        }
+
+        /**
+         * Set data used during the request
+         *
+         * @param array $data Data.
+         *
+         * @return $this
+         */
+        public function data( $data ) {
+            $this->data = $data;
+
+            return $this;
+        }
+
+        /**
+         * Dispatch the async request
+         *
+         * @return array|WP_Error
+         */
+        public function dispatch() {
+            $url  = add_query_arg( $this->get_query_args(), $this->get_query_url() );
+            $args = $this->get_post_args();
+
+            return wp_remote_post( esc_url_raw( $url ), $args );
+        }
+
+        /**
+         * Get query args
+         *
+         * @return array
+         */
+        protected function get_query_args() {
+            if ( property_exists( $this, 'query_args' ) ) {
+                return $this->query_args;
+            }
+
+            $args = array(
+                'action' => $this->identifier,
+                'nonce'  => wp_create_nonce( $this->identifier ),
+            );
+
+            /**
+             * Filters the post arguments used during an async request.
+             *
+             * @param array $url
+             */
+            return apply_filters( $this->identifier . '_query_args', $args );
+        }
+
+        /**
+         * Get query URL
+         *
+         * @return string
+         */
+        protected function get_query_url() {
+            if ( property_exists( $this, 'query_url' ) ) {
+                return $this->query_url;
+            }
+
+            $url = admin_url( 'admin-ajax.php' );
+
+            /**
+             * Filters the post arguments used during an async request.
+             *
+             * @param string $url
+             */
+            return apply_filters( $this->identifier . '_query_url', $url );
+        }
+
+        /**
+         * Get post args
+         *
+         * @return array
+         */
+        protected function get_post_args() {
+            if ( property_exists( $this, 'post_args' ) ) {
+                return $this->post_args;
+            }
+
+            $args = array(
+                'timeout'   => 0.01,
+                'blocking'  => false,
+                'body'      => $this->data,
+                'cookies'   => $_COOKIE,
+                'sslverify' => apply_filters( 'https_local_ssl_verify', false ),
+            );
+
+            /**
+             * Filters the post arguments used during an async request.
+             *
+             * @param array $args
+             */
+            return apply_filters( $this->identifier . '_post_args', $args );
+        }
+
+        /**
+         * Maybe handle
+         *
+         * Check for correct nonce and pass to handler.
+         */
+        public function maybe_handle() {
+            // Don't lock up other requests while processing
+            session_write_close();
+
+            check_ajax_referer( $this->identifier, 'nonce' );
+
+            $this->handle();
+
+            wp_die();
+        }
+
+        /**
+         * Handle
+         *
+         * Override this method to perform any actions required
+         * during the async request.
+         */
+        abstract protected function handle();
+
+    }
 }

Spacing +17 added lines, -17 removed lines

@@ -12,7 +12,7 @@ 
 License URI: https://github.com/deliciousbrains/wp-background-processing/commit/126d7945dd3d39f39cb6488ca08fe1fb66cb351a
 */
 
-if ( ! class_exists( 'WP_Async_Request' ) ) {
+if ( ! class_exists('WP_Async_Request')) {
 
 	/**
 	 * Abstract WP_Async_Request class.
@@ -63,10 +63,10 @@ 
 		 * Initiate new async request
 		 */
 		public function __construct() {
-			$this->identifier = $this->prefix . '_' . $this->action;
+			$this->identifier = $this->prefix.'_'.$this->action;
 
-			add_action( 'wp_ajax_' . $this->identifier, array( $this, 'maybe_handle' ) );
-			add_action( 'wp_ajax_nopriv_' . $this->identifier, array( $this, 'maybe_handle' ) );
+			add_action('wp_ajax_'.$this->identifier, array($this, 'maybe_handle'));
+			add_action('wp_ajax_nopriv_'.$this->identifier, array($this, 'maybe_handle'));
 		}
 
 		/**
@@ -76,7 +76,7 @@ 
 		 *
 		 * @return $this
 		 */
-		public function data( $data ) {
+		public function data($data) {
 			$this->data = $data;
 
 			return $this;
@@ -88,10 +88,10 @@ 
 		 * @return array|WP_Error
 		 */
 		public function dispatch() {
-			$url  = add_query_arg( $this->get_query_args(), $this->get_query_url() );
+			$url  = add_query_arg($this->get_query_args(), $this->get_query_url());
 			$args = $this->get_post_args();
 
-			return wp_remote_post( esc_url_raw( $url ), $args );
+			return wp_remote_post(esc_url_raw($url), $args);
 		}
 
 		/**
@@ -100,13 +100,13 @@ 
 		 * @return array
 		 */
 		protected function get_query_args() {
-			if ( property_exists( $this, 'query_args' ) ) {
+			if (property_exists($this, 'query_args')) {
 				return $this->query_args;
 			}
 
 			$args = array(
 				'action' => $this->identifier,
-				'nonce'  => wp_create_nonce( $this->identifier ),
+				'nonce'  => wp_create_nonce($this->identifier),
 			);
 
 			/**
@@ -114,7 +114,7 @@ 
 			 *
 			 * @param array $url
 			 */
-			return apply_filters( $this->identifier . '_query_args', $args );
+			return apply_filters($this->identifier.'_query_args', $args);
 		}
 
 		/**
@@ -123,18 +123,18 @@ 
 		 * @return string
 		 */
 		protected function get_query_url() {
-			if ( property_exists( $this, 'query_url' ) ) {
+			if (property_exists($this, 'query_url')) {
 				return $this->query_url;
 			}
 
-			$url = admin_url( 'admin-ajax.php' );
+			$url = admin_url('admin-ajax.php');
 
 			/**
 			 * Filters the post arguments used during an async request.
 			 *
 			 * @param string $url
 			 */
-			return apply_filters( $this->identifier . '_query_url', $url );
+			return apply_filters($this->identifier.'_query_url', $url);
 		}
 
 		/**
@@ -143,7 +143,7 @@ 
 		 * @return array
 		 */
 		protected function get_post_args() {
-			if ( property_exists( $this, 'post_args' ) ) {
+			if (property_exists($this, 'post_args')) {
 				return $this->post_args;
 			}
 
@@ -152,7 +152,7 @@ 
 				'blocking'  => false,
 				'body'      => $this->data,
 				'cookies'   => $_COOKIE,
-				'sslverify' => apply_filters( 'https_local_ssl_verify', false ),
+				'sslverify' => apply_filters('https_local_ssl_verify', false),
 			);
 
 			/**
@@ -160,7 +160,7 @@ 
 			 *
 			 * @param array $args
 			 */
-			return apply_filters( $this->identifier . '_post_args', $args );
+			return apply_filters($this->identifier.'_post_args', $args);
 		}
 
 		/**
@@ -172,7 +172,7 @@ 
 			// Don't lock up other requests while processing
 			session_write_close();
 
-			check_ajax_referer( $this->identifier, 'nonce' );
+			check_ajax_referer($this->identifier, 'nonce');
 
 			$this->handle();
 

src/libraries/action-scheduler/deprecated/functions.php

Indentation +12 added lines, -12 removed lines

@@ -19,8 +19,8 @@ 
  * @return string The job ID
  */
 function wc_schedule_single_action( $timestamp, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_single_action()' );
-	return as_schedule_single_action( $timestamp, $hook, $args, $group );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_single_action()' );
+    return as_schedule_single_action( $timestamp, $hook, $args, $group );
 }
 
 /**
@@ -37,8 +37,8 @@ 
  * @return string The job ID
  */
 function wc_schedule_recurring_action( $timestamp, $interval_in_seconds, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_recurring_action()' );
-	return as_schedule_recurring_action( $timestamp, $interval_in_seconds, $hook, $args, $group );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_recurring_action()' );
+    return as_schedule_recurring_action( $timestamp, $interval_in_seconds, $hook, $args, $group );
 }
 
 /**
@@ -65,8 +65,8 @@ 
  * @return string The job ID
  */
 function wc_schedule_cron_action( $timestamp, $schedule, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_cron_action()' );
-	return as_schedule_cron_action( $timestamp, $schedule, $hook, $args, $group );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_cron_action()' );
+    return as_schedule_cron_action( $timestamp, $schedule, $hook, $args, $group );
 }
 
 /**
@@ -79,8 +79,8 @@ 
  * @deprecated 2.1.0
  */
 function wc_unschedule_action( $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_unschedule_action()' );
-	as_unschedule_action( $hook, $args, $group );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_unschedule_action()' );
+    as_unschedule_action( $hook, $args, $group );
 }
 
 /**
@@ -93,8 +93,8 @@ 
  * @return int|bool The timestamp for the next occurrence, or false if nothing was found
  */
 function wc_next_scheduled_action( $hook, $args = null, $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_next_scheduled_action()' );
-	return as_next_scheduled_action( $hook, $args, $group );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_next_scheduled_action()' );
+    return as_next_scheduled_action( $hook, $args, $group );
 }
 
 /**
@@ -121,6 +121,6 @@ 
  * @return array
  */
 function wc_get_scheduled_actions( $args = array(), $return_format = OBJECT ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_get_scheduled_actions()' );
-	return as_get_scheduled_actions( $args, $return_format );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_get_scheduled_actions()' );
+    return as_get_scheduled_actions( $args, $return_format );
 }

Spacing +18 added lines, -18 removed lines

@@ -18,9 +18,9 @@ 
  *
  * @return string The job ID
  */
-function wc_schedule_single_action( $timestamp, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_single_action()' );
-	return as_schedule_single_action( $timestamp, $hook, $args, $group );
+function wc_schedule_single_action($timestamp, $hook, $args = array(), $group = '') {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_schedule_single_action()');
+	return as_schedule_single_action($timestamp, $hook, $args, $group);
 }
 
 /**
@@ -36,9 +36,9 @@ 
  *
  * @return string The job ID
  */
-function wc_schedule_recurring_action( $timestamp, $interval_in_seconds, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_recurring_action()' );
-	return as_schedule_recurring_action( $timestamp, $interval_in_seconds, $hook, $args, $group );
+function wc_schedule_recurring_action($timestamp, $interval_in_seconds, $hook, $args = array(), $group = '') {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_schedule_recurring_action()');
+	return as_schedule_recurring_action($timestamp, $interval_in_seconds, $hook, $args, $group);
 }
 
 /**
@@ -64,9 +64,9 @@ 
  *
  * @return string The job ID
  */
-function wc_schedule_cron_action( $timestamp, $schedule, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_cron_action()' );
-	return as_schedule_cron_action( $timestamp, $schedule, $hook, $args, $group );
+function wc_schedule_cron_action($timestamp, $schedule, $hook, $args = array(), $group = '') {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_schedule_cron_action()');
+	return as_schedule_cron_action($timestamp, $schedule, $hook, $args, $group);
 }
 
 /**
@@ -78,9 +78,9 @@ 
  *
  * @deprecated 2.1.0
  */
-function wc_unschedule_action( $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_unschedule_action()' );
-	as_unschedule_action( $hook, $args, $group );
+function wc_unschedule_action($hook, $args = array(), $group = '') {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_unschedule_action()');
+	as_unschedule_action($hook, $args, $group);
 }
 
 /**
@@ -92,9 +92,9 @@ 
  *
  * @return int|bool The timestamp for the next occurrence, or false if nothing was found
  */
-function wc_next_scheduled_action( $hook, $args = null, $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_next_scheduled_action()' );
-	return as_next_scheduled_action( $hook, $args, $group );
+function wc_next_scheduled_action($hook, $args = null, $group = '') {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_next_scheduled_action()');
+	return as_next_scheduled_action($hook, $args, $group);
 }
 
 /**
@@ -120,7 +120,7 @@ 
  *
  * @return array
  */
-function wc_get_scheduled_actions( $args = array(), $return_format = OBJECT ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_get_scheduled_actions()' );
-	return as_get_scheduled_actions( $args, $return_format );
+function wc_get_scheduled_actions($args = array(), $return_format = OBJECT) {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_get_scheduled_actions()');
+	return as_get_scheduled_actions($args, $return_format);
 }

src/libraries/action-scheduler/deprecated/ActionScheduler_AdminView_Deprecated.php

Indentation +123 added lines, -123 removed lines

@@ -10,138 +10,138 @@
  */
 class ActionScheduler_AdminView_Deprecated {
 
-	public function action_scheduler_post_type_args( $args ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
-		return $args;
-	}
+    public function action_scheduler_post_type_args( $args ) {
+        _deprecated_function( __METHOD__, '2.0.0' );
+        return $args;
+    }
 
-	/**
-	 * Customise the post status related views displayed on the Scheduled Actions administration screen.
-	 *
-	 * @param array $views An associative array of views and view labels which can be used to filter the 'scheduled-action' posts displayed on the Scheduled Actions administration screen.
-	 * @return array $views An associative array of views and view labels which can be used to filter the 'scheduled-action' posts displayed on the Scheduled Actions administration screen.
-	 */
-	public function list_table_views( $views ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
-		return $views;
-	}
+    /**
+     * Customise the post status related views displayed on the Scheduled Actions administration screen.
+     *
+     * @param array $views An associative array of views and view labels which can be used to filter the 'scheduled-action' posts displayed on the Scheduled Actions administration screen.
+     * @return array $views An associative array of views and view labels which can be used to filter the 'scheduled-action' posts displayed on the Scheduled Actions administration screen.
+     */
+    public function list_table_views( $views ) {
+        _deprecated_function( __METHOD__, '2.0.0' );
+        return $views;
+    }
 
-	/**
-	 * Do not include the "Edit" action for the Scheduled Actions administration screen.
-	 *
-	 * Hooked to the 'bulk_actions-edit-action-scheduler' filter.
-	 *
-	 * @param array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
-	 * @return array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
-	 */
-	public function bulk_actions( $actions ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
-		return $actions;
-	}
+    /**
+     * Do not include the "Edit" action for the Scheduled Actions administration screen.
+     *
+     * Hooked to the 'bulk_actions-edit-action-scheduler' filter.
+     *
+     * @param array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
+     * @return array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
+     */
+    public function bulk_actions( $actions ) {
+        _deprecated_function( __METHOD__, '2.0.0' );
+        return $actions;
+    }
 
-	/**
-	 * Completely customer the columns displayed on the Scheduled Actions administration screen.
-	 *
-	 * Because we can't filter the content of the default title and date columns, we need to recreate our own
-	 * custom columns for displaying those post fields. For the column content, @see self::list_table_column_content().
-	 *
-	 * @param array $columns An associative array of columns that are use for the table on the Scheduled Actions administration screen.
-	 * @return array $columns An associative array of columns that are use for the table on the Scheduled Actions administration screen.
-	 */
-	public function list_table_columns( $columns ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
-		return $columns;
-	}
+    /**
+     * Completely customer the columns displayed on the Scheduled Actions administration screen.
+     *
+     * Because we can't filter the content of the default title and date columns, we need to recreate our own
+     * custom columns for displaying those post fields. For the column content, @see self::list_table_column_content().
+     *
+     * @param array $columns An associative array of columns that are use for the table on the Scheduled Actions administration screen.
+     * @return array $columns An associative array of columns that are use for the table on the Scheduled Actions administration screen.
+     */
+    public function list_table_columns( $columns ) {
+        _deprecated_function( __METHOD__, '2.0.0' );
+        return $columns;
+    }
 
-	/**
-	 * Make our custom title & date columns use defaulting title & date sorting.
-	 *
-	 * @param array $columns An associative array of columns that can be used to sort the table on the Scheduled Actions administration screen.
-	 * @return array $columns An associative array of columns that can be used to sort the table on the Scheduled Actions administration screen.
-	 */
-	public static function list_table_sortable_columns( $columns ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
-		return $columns;
-	}
+    /**
+     * Make our custom title & date columns use defaulting title & date sorting.
+     *
+     * @param array $columns An associative array of columns that can be used to sort the table on the Scheduled Actions administration screen.
+     * @return array $columns An associative array of columns that can be used to sort the table on the Scheduled Actions administration screen.
+     */
+    public static function list_table_sortable_columns( $columns ) {
+        _deprecated_function( __METHOD__, '2.0.0' );
+        return $columns;
+    }
 
-	/**
-	 * Print the content for our custom columns.
-	 *
-	 * @param string $column_name The key for the column for which we should output our content.
-	 * @param int    $post_id The ID of the 'scheduled-action' post for which this row relates.
-	 */
-	public static function list_table_column_content( $column_name, $post_id ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
-	}
+    /**
+     * Print the content for our custom columns.
+     *
+     * @param string $column_name The key for the column for which we should output our content.
+     * @param int    $post_id The ID of the 'scheduled-action' post for which this row relates.
+     */
+    public static function list_table_column_content( $column_name, $post_id ) {
+        _deprecated_function( __METHOD__, '2.0.0' );
+    }
 
-	/**
-	 * Hide the inline "Edit" action for all 'scheduled-action' posts.
-	 *
-	 * Hooked to the 'post_row_actions' filter.
-	 *
-	 * @param array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
-	 * @return array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
-	 */
-	public static function row_actions( $actions, $post ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
-		return $actions;
-	}
+    /**
+     * Hide the inline "Edit" action for all 'scheduled-action' posts.
+     *
+     * Hooked to the 'post_row_actions' filter.
+     *
+     * @param array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
+     * @return array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
+     */
+    public static function row_actions( $actions, $post ) {
+        _deprecated_function( __METHOD__, '2.0.0' );
+        return $actions;
+    }
 
-	/**
-	 * Run an action when triggered from the Action Scheduler administration screen.
-	 *
-	 * @codeCoverageIgnore
-	 */
-	public static function maybe_execute_action() {
-		_deprecated_function( __METHOD__, '2.0.0' );
-	}
+    /**
+     * Run an action when triggered from the Action Scheduler administration screen.
+     *
+     * @codeCoverageIgnore
+     */
+    public static function maybe_execute_action() {
+        _deprecated_function( __METHOD__, '2.0.0' );
+    }
 
-	/**
-	 * Convert an interval of seconds into a two part human friendly string.
-	 *
-	 * The WordPress human_time_diff() function only calculates the time difference to one degree, meaning
-	 * even if an action is 1 day and 11 hours away, it will display "1 day". This funciton goes one step
-	 * further to display two degrees of accuracy.
-	 *
-	 * Based on Crontrol::interval() function by Edward Dale: https://wordpress.org/plugins/wp-crontrol/
-	 *
-	 * @param int $interval A interval in seconds.
-	 * @return string A human friendly string representation of the interval.
-	 */
-	public static function admin_notices() {
-		_deprecated_function( __METHOD__, '2.0.0' );
-	}
+    /**
+     * Convert an interval of seconds into a two part human friendly string.
+     *
+     * The WordPress human_time_diff() function only calculates the time difference to one degree, meaning
+     * even if an action is 1 day and 11 hours away, it will display "1 day". This funciton goes one step
+     * further to display two degrees of accuracy.
+     *
+     * Based on Crontrol::interval() function by Edward Dale: https://wordpress.org/plugins/wp-crontrol/
+     *
+     * @param int $interval A interval in seconds.
+     * @return string A human friendly string representation of the interval.
+     */
+    public static function admin_notices() {
+        _deprecated_function( __METHOD__, '2.0.0' );
+    }
 
-	/**
-	 * Filter search queries to allow searching by Claim ID (i.e. post_password).
-	 *
-	 * @param string   $orderby MySQL orderby string.
-	 * @param WP_Query $query Instance of a WP_Query object
-	 * @return string MySQL orderby string.
-	 */
-	public function custom_orderby( $orderby, $query ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
-	}
+    /**
+     * Filter search queries to allow searching by Claim ID (i.e. post_password).
+     *
+     * @param string   $orderby MySQL orderby string.
+     * @param WP_Query $query Instance of a WP_Query object
+     * @return string MySQL orderby string.
+     */
+    public function custom_orderby( $orderby, $query ) {
+        _deprecated_function( __METHOD__, '2.0.0' );
+    }
 
-	/**
-	 * Filter search queries to allow searching by Claim ID (i.e. post_password).
-	 *
-	 * @param string   $search MySQL search string.
-	 * @param WP_Query $query Instance of a WP_Query object
-	 * @return string MySQL search string.
-	 */
-	public function search_post_password( $search, $query ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
-	}
+    /**
+     * Filter search queries to allow searching by Claim ID (i.e. post_password).
+     *
+     * @param string   $search MySQL search string.
+     * @param WP_Query $query Instance of a WP_Query object
+     * @return string MySQL search string.
+     */
+    public function search_post_password( $search, $query ) {
+        _deprecated_function( __METHOD__, '2.0.0' );
+    }
 
-	/**
-	 * Change messages when a scheduled action is updated.
-	 *
-	 * @param  array $messages
-	 * @return array
-	 */
-	public function post_updated_messages( $messages ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
-		return $messages;
-	}
+    /**
+     * Change messages when a scheduled action is updated.
+     *
+     * @param  array $messages
+     * @return array
+     */
+    public function post_updated_messages( $messages ) {
+        _deprecated_function( __METHOD__, '2.0.0' );
+        return $messages;
+    }
 }

Spacing +22 added lines, -22 removed lines

@@ -10,8 +10,8 @@ 
  */
 class ActionScheduler_AdminView_Deprecated {
 
-	public function action_scheduler_post_type_args( $args ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
+	public function action_scheduler_post_type_args($args) {
+		_deprecated_function(__METHOD__, '2.0.0');
 		return $args;
 	}
 
@@ -21,8 +21,8 @@ 
 	 * @param array $views An associative array of views and view labels which can be used to filter the 'scheduled-action' posts displayed on the Scheduled Actions administration screen.
 	 * @return array $views An associative array of views and view labels which can be used to filter the 'scheduled-action' posts displayed on the Scheduled Actions administration screen.
 	 */
-	public function list_table_views( $views ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
+	public function list_table_views($views) {
+		_deprecated_function(__METHOD__, '2.0.0');
 		return $views;
 	}
 
@@ -34,8 +34,8 @@ 
 	 * @param array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
 	 * @return array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
 	 */
-	public function bulk_actions( $actions ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
+	public function bulk_actions($actions) {
+		_deprecated_function(__METHOD__, '2.0.0');
 		return $actions;
 	}
 
@@ -48,8 +48,8 @@ 
 	 * @param array $columns An associative array of columns that are use for the table on the Scheduled Actions administration screen.
 	 * @return array $columns An associative array of columns that are use for the table on the Scheduled Actions administration screen.
 	 */
-	public function list_table_columns( $columns ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
+	public function list_table_columns($columns) {
+		_deprecated_function(__METHOD__, '2.0.0');
 		return $columns;
 	}
 
@@ -59,8 +59,8 @@ 
 	 * @param array $columns An associative array of columns that can be used to sort the table on the Scheduled Actions administration screen.
 	 * @return array $columns An associative array of columns that can be used to sort the table on the Scheduled Actions administration screen.
 	 */
-	public static function list_table_sortable_columns( $columns ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
+	public static function list_table_sortable_columns($columns) {
+		_deprecated_function(__METHOD__, '2.0.0');
 		return $columns;
 	}
 
@@ -70,8 +70,8 @@ 
 	 * @param string $column_name The key for the column for which we should output our content.
 	 * @param int    $post_id The ID of the 'scheduled-action' post for which this row relates.
 	 */
-	public static function list_table_column_content( $column_name, $post_id ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
+	public static function list_table_column_content($column_name, $post_id) {
+		_deprecated_function(__METHOD__, '2.0.0');
 	}
 
 	/**
@@ -82,8 +82,8 @@ 
 	 * @param array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
 	 * @return array $actions An associative array of actions which can be performed on the 'scheduled-action' post type.
 	 */
-	public static function row_actions( $actions, $post ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
+	public static function row_actions($actions, $post) {
+		_deprecated_function(__METHOD__, '2.0.0');
 		return $actions;
 	}
 
@@ -93,7 +93,7 @@ 
 	 * @codeCoverageIgnore
 	 */
 	public static function maybe_execute_action() {
-		_deprecated_function( __METHOD__, '2.0.0' );
+		_deprecated_function(__METHOD__, '2.0.0');
 	}
 
 	/**
@@ -109,7 +109,7 @@ 
 	 * @return string A human friendly string representation of the interval.
 	 */
 	public static function admin_notices() {
-		_deprecated_function( __METHOD__, '2.0.0' );
+		_deprecated_function(__METHOD__, '2.0.0');
 	}
 
 	/**
@@ -119,8 +119,8 @@ 
 	 * @param WP_Query $query Instance of a WP_Query object
 	 * @return string MySQL orderby string.
 	 */
-	public function custom_orderby( $orderby, $query ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
+	public function custom_orderby($orderby, $query) {
+		_deprecated_function(__METHOD__, '2.0.0');
 	}
 
 	/**
@@ -130,8 +130,8 @@ 
 	 * @param WP_Query $query Instance of a WP_Query object
 	 * @return string MySQL search string.
 	 */
-	public function search_post_password( $search, $query ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
+	public function search_post_password($search, $query) {
+		_deprecated_function(__METHOD__, '2.0.0');
 	}
 
 	/**
@@ -140,8 +140,8 @@ 
 	 * @param  array $messages
 	 * @return array
 	 */
-	public function post_updated_messages( $messages ) {
-		_deprecated_function( __METHOD__, '2.0.0' );
+	public function post_updated_messages($messages) {
+		_deprecated_function(__METHOD__, '2.0.0');
 		return $messages;
 	}
 }