insideout10 / wordlift-plugin

src/libraries/action-scheduler/tests/phpunit/lock/ActionScheduler_OptionLock_Test.php

Indentation +28 added lines, -28 removed lines

@@ -5,41 +5,41 @@
  * @package test_cases\lock
  */
 class ActionScheduler_OptionLock_Test extends ActionScheduler_UnitTestCase {
-	public function test_instance() {
-		$lock = ActionScheduler::lock();
-		$this->assertInstanceOf( 'ActionScheduler_Lock', $lock );
-		$this->assertInstanceOf( 'ActionScheduler_OptionLock', $lock );
-	}
+    public function test_instance() {
+        $lock = ActionScheduler::lock();
+        $this->assertInstanceOf( 'ActionScheduler_Lock', $lock );
+        $this->assertInstanceOf( 'ActionScheduler_OptionLock', $lock );
+    }
 
-	public function test_is_locked() {
-		$lock      = ActionScheduler::lock();
-		$lock_type = md5( rand() );
+    public function test_is_locked() {
+        $lock      = ActionScheduler::lock();
+        $lock_type = md5( rand() );
 
-		$this->assertFalse( $lock->is_locked( $lock_type ) );
+        $this->assertFalse( $lock->is_locked( $lock_type ) );
 
-		$lock->set( $lock_type );
-		$this->assertTrue( $lock->is_locked( $lock_type ) );
-	}
+        $lock->set( $lock_type );
+        $this->assertTrue( $lock->is_locked( $lock_type ) );
+    }
 
-	public function test_set() {
-		$lock      = ActionScheduler::lock();
-		$lock_type = md5( rand() );
+    public function test_set() {
+        $lock      = ActionScheduler::lock();
+        $lock_type = md5( rand() );
 
-		$lock->set( $lock_type );
-		$this->assertTrue( $lock->is_locked( $lock_type ) );
-	}
+        $lock->set( $lock_type );
+        $this->assertTrue( $lock->is_locked( $lock_type ) );
+    }
 
-	public function test_get_expiration() {
-		$lock      = ActionScheduler::lock();
-		$lock_type = md5( rand() );
+    public function test_get_expiration() {
+        $lock      = ActionScheduler::lock();
+        $lock_type = md5( rand() );
 
-		$lock->set( $lock_type );
+        $lock->set( $lock_type );
 
-		$expiration   = $lock->get_expiration( $lock_type );
-		$current_time = time();
+        $expiration   = $lock->get_expiration( $lock_type );
+        $current_time = time();
 
-		$this->assertGreaterThanOrEqual( 0, $expiration );
-		$this->assertGreaterThan( $current_time, $expiration );
-		$this->assertLessThan( $current_time + MINUTE_IN_SECONDS + 1, $expiration );
-	}
+        $this->assertGreaterThanOrEqual( 0, $expiration );
+        $this->assertGreaterThan( $current_time, $expiration );
+        $this->assertLessThan( $current_time + MINUTE_IN_SECONDS + 1, $expiration );
+    }
 }

Spacing +15 added lines, -15 removed lines

@@ -7,39 +7,39 @@
 class ActionScheduler_OptionLock_Test extends ActionScheduler_UnitTestCase {
 	public function test_instance() {
 		$lock = ActionScheduler::lock();
-		$this->assertInstanceOf( 'ActionScheduler_Lock', $lock );
-		$this->assertInstanceOf( 'ActionScheduler_OptionLock', $lock );
+		$this->assertInstanceOf('ActionScheduler_Lock', $lock);
+		$this->assertInstanceOf('ActionScheduler_OptionLock', $lock);
 	}
 
 	public function test_is_locked() {
 		$lock      = ActionScheduler::lock();
-		$lock_type = md5( rand() );
+		$lock_type = md5(rand());
 
-		$this->assertFalse( $lock->is_locked( $lock_type ) );
+		$this->assertFalse($lock->is_locked($lock_type));
 
-		$lock->set( $lock_type );
-		$this->assertTrue( $lock->is_locked( $lock_type ) );
+		$lock->set($lock_type);
+		$this->assertTrue($lock->is_locked($lock_type));
 	}
 
 	public function test_set() {
 		$lock      = ActionScheduler::lock();
-		$lock_type = md5( rand() );
+		$lock_type = md5(rand());
 
-		$lock->set( $lock_type );
-		$this->assertTrue( $lock->is_locked( $lock_type ) );
+		$lock->set($lock_type);
+		$this->assertTrue($lock->is_locked($lock_type));
 	}
 
 	public function test_get_expiration() {
 		$lock      = ActionScheduler::lock();
-		$lock_type = md5( rand() );
+		$lock_type = md5(rand());
 
-		$lock->set( $lock_type );
+		$lock->set($lock_type);
 
-		$expiration   = $lock->get_expiration( $lock_type );
+		$expiration   = $lock->get_expiration($lock_type);
 		$current_time = time();
 
-		$this->assertGreaterThanOrEqual( 0, $expiration );
-		$this->assertGreaterThan( $current_time, $expiration );
-		$this->assertLessThan( $current_time + MINUTE_IN_SECONDS + 1, $expiration );
+		$this->assertGreaterThanOrEqual(0, $expiration);
+		$this->assertGreaterThan($current_time, $expiration);
+		$this->assertLessThan($current_time + MINUTE_IN_SECONDS + 1, $expiration);
 	}
 }

src/libraries/action-scheduler/tests/phpunit/migration/Controller_Test.php

Indentation +49 added lines, -49 removed lines

@@ -15,62 +15,62 @@
  * @group migration
  */
 class Controller_Test extends ActionScheduler_UnitTestCase {
-	/**
-	 * Test to ensure the Migration Controller will schedule the migration.
-	 */
-	public function test_schedules_migration() {
-		as_unschedule_action( Scheduler::HOOK );
-		Controller::instance()->schedule_migration();
+    /**
+     * Test to ensure the Migration Controller will schedule the migration.
+     */
+    public function test_schedules_migration() {
+        as_unschedule_action( Scheduler::HOOK );
+        Controller::instance()->schedule_migration();
 
-		$this->assertTrue(
-			as_next_scheduled_action( Scheduler::HOOK ) > 0,
-			'Confirm that the Migration Controller scheduled the migration.'
-		);
+        $this->assertTrue(
+            as_next_scheduled_action( Scheduler::HOOK ) > 0,
+            'Confirm that the Migration Controller scheduled the migration.'
+        );
 
-		as_unschedule_action( Scheduler::HOOK );
-	}
+        as_unschedule_action( Scheduler::HOOK );
+    }
 
-	/**
-	 * Test to ensure that if an essential table is missing, the Migration
-	 * Controller will not schedule a migration.
-	 *
-	 * @see https://github.com/woocommerce/action-scheduler/issues/653
-	 */
-	public function test_migration_not_scheduled_if_tables_are_missing() {
-		as_unschedule_action( Scheduler::HOOK );
-		$this->rename_claims_table();
-		Controller::instance()->schedule_migration();
+    /**
+     * Test to ensure that if an essential table is missing, the Migration
+     * Controller will not schedule a migration.
+     *
+     * @see https://github.com/woocommerce/action-scheduler/issues/653
+     */
+    public function test_migration_not_scheduled_if_tables_are_missing() {
+        as_unschedule_action( Scheduler::HOOK );
+        $this->rename_claims_table();
+        Controller::instance()->schedule_migration();
 
-		$this->assertFalse(
-			as_next_scheduled_action( Scheduler::HOOK ),
-			'When required tables are missing, the migration will not be scheduled.'
-		);
+        $this->assertFalse(
+            as_next_scheduled_action( Scheduler::HOOK ),
+            'When required tables are missing, the migration will not be scheduled.'
+        );
 
-		$this->restore_claims_table_name();
-	}
+        $this->restore_claims_table_name();
+    }
 
-	/**
-	 * Rename the claims table, so that it cannot be used by the library.
-	 */
-	private function rename_claims_table() {
-		global $wpdb;
-		$normal_table_name   = $wpdb->prefix . Schema::CLAIMS_TABLE;
-		$modified_table_name = $normal_table_name . 'x';
+    /**
+     * Rename the claims table, so that it cannot be used by the library.
+     */
+    private function rename_claims_table() {
+        global $wpdb;
+        $normal_table_name   = $wpdb->prefix . Schema::CLAIMS_TABLE;
+        $modified_table_name = $normal_table_name . 'x';
 
-		// phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
-		$wpdb->query( "RENAME TABLE {$normal_table_name} TO {$modified_table_name}" );
-	}
+        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
+        $wpdb->query( "RENAME TABLE {$normal_table_name} TO {$modified_table_name}" );
+    }
 
-	/**
-	 * Restore the expected name of the claims table, so that it can be used by the library
-	 * and any further tests.
-	 */
-	private function restore_claims_table_name() {
-		global $wpdb;
-		$normal_table_name   = $wpdb->prefix . Schema::CLAIMS_TABLE;
-		$modified_table_name = $normal_table_name . 'x';
+    /**
+     * Restore the expected name of the claims table, so that it can be used by the library
+     * and any further tests.
+     */
+    private function restore_claims_table_name() {
+        global $wpdb;
+        $normal_table_name   = $wpdb->prefix . Schema::CLAIMS_TABLE;
+        $modified_table_name = $normal_table_name . 'x';
 
-		// phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
-		$wpdb->query( "RENAME TABLE {$modified_table_name} TO {$normal_table_name}" );
-	}
+        // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
+        $wpdb->query( "RENAME TABLE {$modified_table_name} TO {$normal_table_name}" );
+    }
 }

Spacing +11 added lines, -11 removed lines

@@ -19,15 +19,15 @@ 
 	 * Test to ensure the Migration Controller will schedule the migration.
 	 */
 	public function test_schedules_migration() {
-		as_unschedule_action( Scheduler::HOOK );
+		as_unschedule_action(Scheduler::HOOK);
 		Controller::instance()->schedule_migration();
 
 		$this->assertTrue(
-			as_next_scheduled_action( Scheduler::HOOK ) > 0,
+			as_next_scheduled_action(Scheduler::HOOK) > 0,
 			'Confirm that the Migration Controller scheduled the migration.'
 		);
 
-		as_unschedule_action( Scheduler::HOOK );
+		as_unschedule_action(Scheduler::HOOK);
 	}
 
 	/**
@@ -37,12 +37,12 @@ 
 	 * @see https://github.com/woocommerce/action-scheduler/issues/653
 	 */
 	public function test_migration_not_scheduled_if_tables_are_missing() {
-		as_unschedule_action( Scheduler::HOOK );
+		as_unschedule_action(Scheduler::HOOK);
 		$this->rename_claims_table();
 		Controller::instance()->schedule_migration();
 
 		$this->assertFalse(
-			as_next_scheduled_action( Scheduler::HOOK ),
+			as_next_scheduled_action(Scheduler::HOOK),
 			'When required tables are missing, the migration will not be scheduled.'
 		);
 
@@ -54,11 +54,11 @@ 
 	 */
 	private function rename_claims_table() {
 		global $wpdb;
-		$normal_table_name   = $wpdb->prefix . Schema::CLAIMS_TABLE;
-		$modified_table_name = $normal_table_name . 'x';
+		$normal_table_name   = $wpdb->prefix.Schema::CLAIMS_TABLE;
+		$modified_table_name = $normal_table_name.'x';
 
 		// phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
-		$wpdb->query( "RENAME TABLE {$normal_table_name} TO {$modified_table_name}" );
+		$wpdb->query("RENAME TABLE {$normal_table_name} TO {$modified_table_name}");
 	}
 
 	/**
@@ -67,10 +67,10 @@ 
 	 */
 	private function restore_claims_table_name() {
 		global $wpdb;
-		$normal_table_name   = $wpdb->prefix . Schema::CLAIMS_TABLE;
-		$modified_table_name = $normal_table_name . 'x';
+		$normal_table_name   = $wpdb->prefix.Schema::CLAIMS_TABLE;
+		$modified_table_name = $normal_table_name.'x';
 
 		// phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
-		$wpdb->query( "RENAME TABLE {$modified_table_name} TO {$normal_table_name}" );
+		$wpdb->query("RENAME TABLE {$modified_table_name} TO {$normal_table_name}");
 	}
 }

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

Indentation +37 added lines, -37 removed lines

@@ -6,44 +6,44 @@
  */
 abstract class ActionScheduler_Store_Deprecated {
 
-	/**
-	 * Mark an action that failed to fetch correctly as failed.
-	 *
-	 * @since 2.2.6
-	 *
-	 * @param int $action_id The ID of the action.
-	 */
-	public function mark_failed_fetch_action( $action_id ) {
-		_deprecated_function( __METHOD__, '3.0.0', 'ActionScheduler_Store::mark_failure()' );
-		self::$store->mark_failure( $action_id );
-	}
+    /**
+     * Mark an action that failed to fetch correctly as failed.
+     *
+     * @since 2.2.6
+     *
+     * @param int $action_id The ID of the action.
+     */
+    public function mark_failed_fetch_action( $action_id ) {
+        _deprecated_function( __METHOD__, '3.0.0', 'ActionScheduler_Store::mark_failure()' );
+        self::$store->mark_failure( $action_id );
+    }
 
-	/**
-	 * Add base hooks
-	 *
-	 * @since 2.2.6
-	 */
-	protected static function hook() {
-		_deprecated_function( __METHOD__, '3.0.0' );
-	}
+    /**
+     * Add base hooks
+     *
+     * @since 2.2.6
+     */
+    protected static function hook() {
+        _deprecated_function( __METHOD__, '3.0.0' );
+    }
 
-	/**
-	 * Remove base hooks
-	 *
-	 * @since 2.2.6
-	 */
-	protected static function unhook() {
-		_deprecated_function( __METHOD__, '3.0.0' );
-	}
+    /**
+     * Remove base hooks
+     *
+     * @since 2.2.6
+     */
+    protected static function unhook() {
+        _deprecated_function( __METHOD__, '3.0.0' );
+    }
 
-	/**
-	 * Get the site's local time.
-	 *
-	 * @deprecated 2.1.0
-	 * @return DateTimeZone
-	 */
-	protected function get_local_timezone() {
-		_deprecated_function( __FUNCTION__, '2.1.0', 'ActionScheduler_TimezoneHelper::set_local_timezone()' );
-		return ActionScheduler_TimezoneHelper::get_local_timezone();
-	}
+    /**
+     * Get the site's local time.
+     *
+     * @deprecated 2.1.0
+     * @return DateTimeZone
+     */
+    protected function get_local_timezone() {
+        _deprecated_function( __FUNCTION__, '2.1.0', 'ActionScheduler_TimezoneHelper::set_local_timezone()' );
+        return ActionScheduler_TimezoneHelper::get_local_timezone();
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -13,9 +13,9 @@ 
 	 *
 	 * @param int $action_id The ID of the action.
 	 */
-	public function mark_failed_fetch_action( $action_id ) {
-		_deprecated_function( __METHOD__, '3.0.0', 'ActionScheduler_Store::mark_failure()' );
-		self::$store->mark_failure( $action_id );
+	public function mark_failed_fetch_action($action_id) {
+		_deprecated_function(__METHOD__, '3.0.0', 'ActionScheduler_Store::mark_failure()');
+		self::$store->mark_failure($action_id);
 	}
 
 	/**
@@ -24,7 +24,7 @@ 
 	 * @since 2.2.6
 	 */
 	protected static function hook() {
-		_deprecated_function( __METHOD__, '3.0.0' );
+		_deprecated_function(__METHOD__, '3.0.0');
 	}
 
 	/**
@@ -33,7 +33,7 @@ 
 	 * @since 2.2.6
 	 */
 	protected static function unhook() {
-		_deprecated_function( __METHOD__, '3.0.0' );
+		_deprecated_function(__METHOD__, '3.0.0');
 	}
 
 	/**
@@ -43,7 +43,7 @@ 
 	 * @return DateTimeZone
 	 */
 	protected function get_local_timezone() {
-		_deprecated_function( __FUNCTION__, '2.1.0', 'ActionScheduler_TimezoneHelper::set_local_timezone()' );
+		_deprecated_function(__FUNCTION__, '2.1.0', 'ActionScheduler_TimezoneHelper::set_local_timezone()');
 		return ActionScheduler_TimezoneHelper::get_local_timezone();
 	}
 }

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

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
  * Class ActionScheduler_NullLogEntry
  */
 class ActionScheduler_NullLogEntry extends ActionScheduler_LogEntry {
-	public function __construct( $action_id = '', $message = '' ) {
+	public function __construct($action_id = '', $message = '') {
 		// nothing to see here
 	}
 }

Indentation +3 added lines, -3 removed lines

@@ -4,8 +4,8 @@
  * Class ActionScheduler_NullLogEntry
  */
 class ActionScheduler_NullLogEntry extends ActionScheduler_LogEntry {
-	public function __construct( $action_id = '', $message = '' ) {
-		// nothing to see here
-	}
+    public function __construct( $action_id = '', $message = '' ) {
+        // nothing to see here
+    }
 }
 

src/libraries/action-scheduler/classes/schedules/ActionScheduler_NullSchedule.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 	 *
 	 * @param null $date The date & time to run the action.
 	 */
-	public function __construct( DateTime $date = null ) {
+	public function __construct(DateTime $date = null) {
 		$this->scheduled_date = null;
 	}
 

Indentation +19 added lines, -19 removed lines

@@ -5,25 +5,25 @@
  */
 class ActionScheduler_NullSchedule extends ActionScheduler_SimpleSchedule {
 
-	/**
-	 * Make the $date param optional and default to null.
-	 *
-	 * @param null $date The date & time to run the action.
-	 */
-	public function __construct( DateTime $date = null ) {
-		$this->scheduled_date = null;
-	}
+    /**
+     * Make the $date param optional and default to null.
+     *
+     * @param null $date The date & time to run the action.
+     */
+    public function __construct( DateTime $date = null ) {
+        $this->scheduled_date = null;
+    }
 
-	/**
-	 * This schedule has no scheduled DateTime, so we need to override the parent __sleep()
-	 *
-	 * @return array
-	 */
-	public function __sleep() {
-		return array();
-	}
+    /**
+     * This schedule has no scheduled DateTime, so we need to override the parent __sleep()
+     *
+     * @return array
+     */
+    public function __sleep() {
+        return array();
+    }
 
-	public function __wakeup() {
-		$this->scheduled_date = null;
-	}
+    public function __wakeup() {
+        $this->scheduled_date = null;
+    }
 }

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

Indentation +35 added lines, -35 removed lines

@@ -9,41 +9,41 @@
  */
 class ActionScheduler_OptionLock extends ActionScheduler_Lock {
 
-	/**
-	 * Set a lock using options for a given amount of time (60 seconds by default).
-	 *
-	 * Using an autoloaded option avoids running database queries or other resource intensive tasks
-	 * on frequently triggered hooks, like 'init' or 'shutdown'.
-	 *
-	 * For example, ActionScheduler_QueueRunner->maybe_dispatch_async_request() uses a lock to avoid
-	 * calling ActionScheduler_QueueRunner->has_maximum_concurrent_batches() every time the 'shutdown',
-	 * hook is triggered, because that method calls ActionScheduler_QueueRunner->store->get_claim_count()
-	 * to find the current number of claims in the database.
-	 *
-	 * @param string $lock_type A string to identify different lock types.
-	 * @bool True if lock value has changed, false if not or if set failed.
-	 */
-	public function set( $lock_type ) {
-		return update_option( $this->get_key( $lock_type ), time() + $this->get_duration( $lock_type ) );
-	}
+    /**
+     * Set a lock using options for a given amount of time (60 seconds by default).
+     *
+     * Using an autoloaded option avoids running database queries or other resource intensive tasks
+     * on frequently triggered hooks, like 'init' or 'shutdown'.
+     *
+     * For example, ActionScheduler_QueueRunner->maybe_dispatch_async_request() uses a lock to avoid
+     * calling ActionScheduler_QueueRunner->has_maximum_concurrent_batches() every time the 'shutdown',
+     * hook is triggered, because that method calls ActionScheduler_QueueRunner->store->get_claim_count()
+     * to find the current number of claims in the database.
+     *
+     * @param string $lock_type A string to identify different lock types.
+     * @bool True if lock value has changed, false if not or if set failed.
+     */
+    public function set( $lock_type ) {
+        return update_option( $this->get_key( $lock_type ), time() + $this->get_duration( $lock_type ) );
+    }
 
-	/**
-	 * If a lock is set, return the timestamp it was set to expiry.
-	 *
-	 * @param string $lock_type A string to identify different lock types.
-	 * @return bool|int False if no lock is set, otherwise the timestamp for when the lock is set to expire.
-	 */
-	public function get_expiration( $lock_type ) {
-		return get_option( $this->get_key( $lock_type ) );
-	}
+    /**
+     * If a lock is set, return the timestamp it was set to expiry.
+     *
+     * @param string $lock_type A string to identify different lock types.
+     * @return bool|int False if no lock is set, otherwise the timestamp for when the lock is set to expire.
+     */
+    public function get_expiration( $lock_type ) {
+        return get_option( $this->get_key( $lock_type ) );
+    }
 
-	/**
-	 * Get the key to use for storing the lock in the transient
-	 *
-	 * @param string $lock_type A string to identify different lock types.
-	 * @return string
-	 */
-	protected function get_key( $lock_type ) {
-		return sprintf( 'action_scheduler_lock_%s', $lock_type );
-	}
+    /**
+     * Get the key to use for storing the lock in the transient
+     *
+     * @param string $lock_type A string to identify different lock types.
+     * @return string
+     */
+    protected function get_key( $lock_type ) {
+        return sprintf( 'action_scheduler_lock_%s', $lock_type );
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -23,8 +23,8 @@ 
 	 * @param string $lock_type A string to identify different lock types.
 	 * @bool True if lock value has changed, false if not or if set failed.
 	 */
-	public function set( $lock_type ) {
-		return update_option( $this->get_key( $lock_type ), time() + $this->get_duration( $lock_type ) );
+	public function set($lock_type) {
+		return update_option($this->get_key($lock_type), time() + $this->get_duration($lock_type));
 	}
 
 	/**
@@ -33,8 +33,8 @@ 
 	 * @param string $lock_type A string to identify different lock types.
 	 * @return bool|int False if no lock is set, otherwise the timestamp for when the lock is set to expire.
 	 */
-	public function get_expiration( $lock_type ) {
-		return get_option( $this->get_key( $lock_type ) );
+	public function get_expiration($lock_type) {
+		return get_option($this->get_key($lock_type));
 	}
 
 	/**
@@ -43,7 +43,7 @@ 
 	 * @param string $lock_type A string to identify different lock types.
 	 * @return string
 	 */
-	protected function get_key( $lock_type ) {
-		return sprintf( 'action_scheduler_lock_%s', $lock_type );
+	protected function get_key($lock_type) {
+		return sprintf('action_scheduler_lock_%s', $lock_type);
 	}
 }

src/libraries/action-scheduler/classes/abstracts/ActionScheduler_Abstract_Schedule.php

Spacing +7 added lines, -7 removed lines

@@ -22,7 +22,7 @@ 
 	/**
 	 * @param DateTime $date The date & time to run the action.
 	 */
-	public function __construct( DateTime $date ) {
+	public function __construct(DateTime $date) {
 		$this->scheduled_date = $date;
 	}
 
@@ -39,7 +39,7 @@ 
 	 * @param DateTime $after
 	 * @return DateTime
 	 */
-	abstract protected function calculate_next( DateTime $after );
+	abstract protected function calculate_next(DateTime $after);
 
 	/**
 	 * Get the next date & time when this schedule should run after a given date & time.
@@ -47,10 +47,10 @@ 
 	 * @param DateTime $after
 	 * @return DateTime|null
 	 */
-	public function get_next( DateTime $after ) {
+	public function get_next(DateTime $after) {
 		$after = clone $after;
-		if ( $after > $this->scheduled_date ) {
-			$after = $this->calculate_next( $after );
+		if ($after > $this->scheduled_date) {
+			$after = $this->calculate_next($after);
 			return $after;
 		}
 		return clone $this->scheduled_date;
@@ -77,7 +77,7 @@ 
 	}
 
 	public function __wakeup() {
-		$this->scheduled_date = as_get_datetime_object( $this->scheduled_timestamp );
-		unset( $this->scheduled_timestamp );
+		$this->scheduled_date = as_get_datetime_object($this->scheduled_timestamp);
+		unset($this->scheduled_timestamp);
 	}
 }

Indentation +68 added lines, -68 removed lines

@@ -5,80 +5,80 @@
  */
 abstract class ActionScheduler_Abstract_Schedule extends ActionScheduler_Schedule_Deprecated {
 
-	/**
-	 * The date & time the schedule is set to run.
-	 *
-	 * @var DateTime
-	 */
-	private $scheduled_date = null;
+    /**
+     * The date & time the schedule is set to run.
+     *
+     * @var DateTime
+     */
+    private $scheduled_date = null;
 
-	/**
-	 * Timestamp equivalent of @see $this->scheduled_date
-	 *
-	 * @var int
-	 */
-	protected $scheduled_timestamp = null;
+    /**
+     * Timestamp equivalent of @see $this->scheduled_date
+     *
+     * @var int
+     */
+    protected $scheduled_timestamp = null;
 
-	/**
-	 * @param DateTime $date The date & time to run the action.
-	 */
-	public function __construct( DateTime $date ) {
-		$this->scheduled_date = $date;
-	}
+    /**
+     * @param DateTime $date The date & time to run the action.
+     */
+    public function __construct( DateTime $date ) {
+        $this->scheduled_date = $date;
+    }
 
-	/**
-	 * Check if a schedule should recur.
-	 *
-	 * @return bool
-	 */
-	abstract public function is_recurring();
+    /**
+     * Check if a schedule should recur.
+     *
+     * @return bool
+     */
+    abstract public function is_recurring();
 
-	/**
-	 * Calculate when the next instance of this schedule would run based on a given date & time.
-	 *
-	 * @param DateTime $after
-	 * @return DateTime
-	 */
-	abstract protected function calculate_next( DateTime $after );
+    /**
+     * Calculate when the next instance of this schedule would run based on a given date & time.
+     *
+     * @param DateTime $after
+     * @return DateTime
+     */
+    abstract protected function calculate_next( DateTime $after );
 
-	/**
-	 * Get the next date & time when this schedule should run after a given date & time.
-	 *
-	 * @param DateTime $after
-	 * @return DateTime|null
-	 */
-	public function get_next( DateTime $after ) {
-		$after = clone $after;
-		if ( $after > $this->scheduled_date ) {
-			$after = $this->calculate_next( $after );
-			return $after;
-		}
-		return clone $this->scheduled_date;
-	}
+    /**
+     * Get the next date & time when this schedule should run after a given date & time.
+     *
+     * @param DateTime $after
+     * @return DateTime|null
+     */
+    public function get_next( DateTime $after ) {
+        $after = clone $after;
+        if ( $after > $this->scheduled_date ) {
+            $after = $this->calculate_next( $after );
+            return $after;
+        }
+        return clone $this->scheduled_date;
+    }
 
-	/**
-	 * Get the date & time the schedule is set to run.
-	 *
-	 * @return DateTime|null
-	 */
-	public function get_date() {
-		return $this->scheduled_date;
-	}
+    /**
+     * Get the date & time the schedule is set to run.
+     *
+     * @return DateTime|null
+     */
+    public function get_date() {
+        return $this->scheduled_date;
+    }
 
-	/**
-	 * For PHP 5.2 compat, since DateTime objects can't be serialized
-	 *
-	 * @return array
-	 */
-	public function __sleep() {
-		$this->scheduled_timestamp = $this->scheduled_date->getTimestamp();
-		return array(
-			'scheduled_timestamp',
-		);
-	}
+    /**
+     * For PHP 5.2 compat, since DateTime objects can't be serialized
+     *
+     * @return array
+     */
+    public function __sleep() {
+        $this->scheduled_timestamp = $this->scheduled_date->getTimestamp();
+        return array(
+            'scheduled_timestamp',
+        );
+    }
 
-	public function __wakeup() {
-		$this->scheduled_date = as_get_datetime_object( $this->scheduled_timestamp );
-		unset( $this->scheduled_timestamp );
-	}
+    public function __wakeup() {
+        $this->scheduled_date = as_get_datetime_object( $this->scheduled_timestamp );
+        unset( $this->scheduled_timestamp );
+    }
 }

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

Indentation +28 added lines, -28 removed lines

@@ -15,35 +15,35 @@
  * @codeCoverageIgnore
  */
 class LogMigrator {
-	/** @var ActionScheduler_Logger */
-	private $source;
+    /** @var ActionScheduler_Logger */
+    private $source;
 
-	/** @var ActionScheduler_Logger */
-	private $destination;
+    /** @var ActionScheduler_Logger */
+    private $destination;
 
-	/**
-	 * ActionMigrator constructor.
-	 *
-	 * @param ActionScheduler_Logger $source_logger Source logger object.
-	 * @param ActionScheduler_Logger $destination_Logger Destination logger object.
-	 */
-	public function __construct( ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger ) {
-		$this->source      = $source_logger;
-		$this->destination = $destination_Logger;
-	}
+    /**
+     * ActionMigrator constructor.
+     *
+     * @param ActionScheduler_Logger $source_logger Source logger object.
+     * @param ActionScheduler_Logger $destination_Logger Destination logger object.
+     */
+    public function __construct( ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger ) {
+        $this->source      = $source_logger;
+        $this->destination = $destination_Logger;
+    }
 
-	/**
-	 * Migrate an action log.
-	 *
-	 * @param int $source_action_id Source logger object.
-	 * @param int $destination_action_id Destination logger object.
-	 */
-	public function migrate( $source_action_id, $destination_action_id ) {
-		$logs = $this->source->get_logs( $source_action_id );
-		foreach ( $logs as $log ) {
-			if ( $log->get_action_id() == $source_action_id ) {
-				$this->destination->log( $destination_action_id, $log->get_message(), $log->get_date() );
-			}
-		}
-	}
+    /**
+     * Migrate an action log.
+     *
+     * @param int $source_action_id Source logger object.
+     * @param int $destination_action_id Destination logger object.
+     */
+    public function migrate( $source_action_id, $destination_action_id ) {
+        $logs = $this->source->get_logs( $source_action_id );
+        foreach ( $logs as $log ) {
+            if ( $log->get_action_id() == $source_action_id ) {
+                $this->destination->log( $destination_action_id, $log->get_message(), $log->get_date() );
+            }
+        }
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -27,7 +27,7 @@ 
 	 * @param ActionScheduler_Logger $source_logger Source logger object.
 	 * @param ActionScheduler_Logger $destination_Logger Destination logger object.
 	 */
-	public function __construct( ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger ) {
+	public function __construct(ActionScheduler_Logger $source_logger, ActionScheduler_Logger $destination_Logger) {
 		$this->source      = $source_logger;
 		$this->destination = $destination_Logger;
 	}
@@ -38,11 +38,11 @@ 
 	 * @param int $source_action_id Source logger object.
 	 * @param int $destination_action_id Destination logger object.
 	 */
-	public function migrate( $source_action_id, $destination_action_id ) {
-		$logs = $this->source->get_logs( $source_action_id );
-		foreach ( $logs as $log ) {
-			if ( $log->get_action_id() == $source_action_id ) {
-				$this->destination->log( $destination_action_id, $log->get_message(), $log->get_date() );
+	public function migrate($source_action_id, $destination_action_id) {
+		$logs = $this->source->get_logs($source_action_id);
+		foreach ($logs as $log) {
+			if ($log->get_action_id() == $source_action_id) {
+				$this->destination->log($destination_action_id, $log->get_message(), $log->get_date());
 			}
 		}
 	}

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;
 	}
 }