insideout10 / wordlift-plugin

src/ext/action-scheduler/classes/schedules/ActionScheduler_CanceledSchedule.php

Indentation +45 added lines, -45 removed lines

@@ -5,53 +5,53 @@
  */
 class ActionScheduler_CanceledSchedule extends ActionScheduler_SimpleSchedule {
 
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $timestamp = null;
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $timestamp = null;
 
-	/**
-	 * @param DateTime $after
-	 *
-	 * @return DateTime|null
-	 */
-	public function calculate_next( DateTime $after ) {
-		return null;
-	}
+    /**
+     * @param DateTime $after
+     *
+     * @return DateTime|null
+     */
+    public function calculate_next( DateTime $after ) {
+        return null;
+    }
 
-	/**
-	 * Cancelled actions should never have a next schedule, even if get_next()
-	 * is called with $after < $this->scheduled_date.
-	 *
-	 * @param DateTime $after
-	 * @return DateTime|null
-	 */
-	public function get_next( DateTime $after ) {
-		return null;
-	}
+    /**
+     * Cancelled actions should never have a next schedule, even if get_next()
+     * is called with $after < $this->scheduled_date.
+     *
+     * @param DateTime $after
+     * @return DateTime|null
+     */
+    public function get_next( DateTime $after ) {
+        return null;
+    }
 
-	/**
-	 * @return bool
-	 */
-	public function is_recurring() {
-		return false;
-	}
+    /**
+     * @return bool
+     */
+    public function is_recurring() {
+        return false;
+    }
 
-	/**
-	 * Unserialize recurring schedules serialized/stored prior to AS 3.0.0
-	 *
-	 * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
-	 * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
-	 * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
-	 * aligned properties and property names for better inheritance. To maintain backward
-	 * compatibility with schedules serialized and stored prior to 3.0, we need to correctly
-	 * map the old property names with matching visibility.
-	 */
-	public function __wakeup() {
-		if ( $this->timestamp !== null ) {
-			$this->scheduled_timestamp = $this->timestamp;
-			unset( $this->timestamp );
-		}
-		parent::__wakeup();
-	}
+    /**
+     * Unserialize recurring schedules serialized/stored prior to AS 3.0.0
+     *
+     * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
+     * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
+     * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
+     * aligned properties and property names for better inheritance. To maintain backward
+     * compatibility with schedules serialized and stored prior to 3.0, we need to correctly
+     * map the old property names with matching visibility.
+     */
+    public function __wakeup() {
+        if ( $this->timestamp !== null ) {
+            $this->scheduled_timestamp = $this->timestamp;
+            unset( $this->timestamp );
+        }
+        parent::__wakeup();
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -15,7 +15,7 @@ 
 	 *
 	 * @return DateTime|null
 	 */
-	public function calculate_next( DateTime $after ) {
+	public function calculate_next(DateTime $after) {
 		return null;
 	}
 
@@ -26,7 +26,7 @@ 
 	 * @param DateTime $after
 	 * @return DateTime|null
 	 */
-	public function get_next( DateTime $after ) {
+	public function get_next(DateTime $after) {
 		return null;
 	}
 
@@ -48,9 +48,9 @@ 
 	 * map the old property names with matching visibility.
 	 */
 	public function __wakeup() {
-		if ( $this->timestamp !== null ) {
+		if ($this->timestamp !== null) {
 			$this->scheduled_timestamp = $this->timestamp;
-			unset( $this->timestamp );
+			unset($this->timestamp);
 		}
 		parent::__wakeup();
 	}

src/ext/action-scheduler/classes/schedules/ActionScheduler_Schedule.php

Indentation +9 added lines, -9 removed lines

@@ -4,15 +4,15 @@
  * Class ActionScheduler_Schedule
  */
 interface ActionScheduler_Schedule {
-	/**
-	 * @param DateTime $after
-	 * @return DateTime|null
-	 */
-	public function next( DateTime $after = null );
+    /**
+     * @param DateTime $after
+     * @return DateTime|null
+     */
+    public function next( DateTime $after = null );
 
-	/**
-	 * @return bool
-	 */
-	public function is_recurring();
+    /**
+     * @return bool
+     */
+    public function is_recurring();
 }
 

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 	 * @param DateTime $after
 	 * @return DateTime|null
 	 */
-	public function next( DateTime $after = null );
+	public function next(DateTime $after = null);
 
 	/**
 	 * @return bool

src/ext/action-scheduler/classes/schedules/ActionScheduler_IntervalSchedule.php

Indentation +67 added lines, -67 removed lines

@@ -5,80 +5,80 @@
  */
 class ActionScheduler_IntervalSchedule extends ActionScheduler_Abstract_RecurringSchedule implements ActionScheduler_Schedule {
 
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $start_timestamp = null;
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $start_timestamp = null;
 
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $interval_in_seconds = null;
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $interval_in_seconds = null;
 
-	/**
-	 * Calculate when this schedule should start after a given date & time using
-	 * the number of seconds between recurrences.
-	 *
-	 * @param DateTime $after
-	 * @return DateTime
-	 */
-	protected function calculate_next( DateTime $after ) {
-		$after->modify( '+' . (int) $this->get_recurrence() . ' seconds' );
-		return $after;
-	}
+    /**
+     * Calculate when this schedule should start after a given date & time using
+     * the number of seconds between recurrences.
+     *
+     * @param DateTime $after
+     * @return DateTime
+     */
+    protected function calculate_next( DateTime $after ) {
+        $after->modify( '+' . (int) $this->get_recurrence() . ' seconds' );
+        return $after;
+    }
 
-	/**
-	 * @return int
-	 */
-	public function interval_in_seconds() {
-		_deprecated_function( __METHOD__, '3.0.0', '(int)ActionScheduler_Abstract_RecurringSchedule::get_recurrence()' );
-		return (int) $this->get_recurrence();
-	}
+    /**
+     * @return int
+     */
+    public function interval_in_seconds() {
+        _deprecated_function( __METHOD__, '3.0.0', '(int)ActionScheduler_Abstract_RecurringSchedule::get_recurrence()' );
+        return (int) $this->get_recurrence();
+    }
 
-	/**
-	 * Serialize interval schedules with data required prior to AS 3.0.0
-	 *
-	 * Prior to Action Scheduler 3.0.0, reccuring schedules used different property names to
-	 * refer to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
-	 * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
-	 * aligned properties and property names for better inheritance. To guard against the
-	 * possibility of infinite loops if downgrading to Action Scheduler < 3.0.0, we need to
-	 * also store the data with the old property names so if it's unserialized in AS < 3.0,
-	 * the schedule doesn't end up with a null/false/0 recurrence.
-	 *
-	 * @return array
-	 */
-	public function __sleep() {
+    /**
+     * Serialize interval schedules with data required prior to AS 3.0.0
+     *
+     * Prior to Action Scheduler 3.0.0, reccuring schedules used different property names to
+     * refer to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
+     * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
+     * aligned properties and property names for better inheritance. To guard against the
+     * possibility of infinite loops if downgrading to Action Scheduler < 3.0.0, we need to
+     * also store the data with the old property names so if it's unserialized in AS < 3.0,
+     * the schedule doesn't end up with a null/false/0 recurrence.
+     *
+     * @return array
+     */
+    public function __sleep() {
 
-		$sleep_params = parent::__sleep();
+        $sleep_params = parent::__sleep();
 
-		$this->start_timestamp     = $this->scheduled_timestamp;
-		$this->interval_in_seconds = $this->recurrence;
+        $this->start_timestamp     = $this->scheduled_timestamp;
+        $this->interval_in_seconds = $this->recurrence;
 
-		return array_merge(
-			$sleep_params,
-			array(
-				'start_timestamp',
-				'interval_in_seconds',
-			)
-		);
-	}
+        return array_merge(
+            $sleep_params,
+            array(
+                'start_timestamp',
+                'interval_in_seconds',
+            )
+        );
+    }
 
-	/**
-	 * Unserialize interval schedules serialized/stored prior to AS 3.0.0
-	 *
-	 * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
-	 */
-	public function __wakeup() {
-		if ( $this->scheduled_timestamp === null && $this->start_timestamp !== null ) {
-			$this->scheduled_timestamp = $this->start_timestamp;
-			unset( $this->start_timestamp );
-		}
+    /**
+     * Unserialize interval schedules serialized/stored prior to AS 3.0.0
+     *
+     * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
+     */
+    public function __wakeup() {
+        if ( $this->scheduled_timestamp === null && $this->start_timestamp !== null ) {
+            $this->scheduled_timestamp = $this->start_timestamp;
+            unset( $this->start_timestamp );
+        }
 
-		if ( $this->recurrence === null && $this->interval_in_seconds !== null ) {
-			$this->recurrence = $this->interval_in_seconds;
-			unset( $this->interval_in_seconds );
-		}
-		parent::__wakeup();
-	}
+        if ( $this->recurrence === null && $this->interval_in_seconds !== null ) {
+            $this->recurrence = $this->interval_in_seconds;
+            unset( $this->interval_in_seconds );
+        }
+        parent::__wakeup();
+    }
 }

Spacing +7 added lines, -7 removed lines

@@ -22,8 +22,8 @@ 
 	 * @param DateTime $after
 	 * @return DateTime
 	 */
-	protected function calculate_next( DateTime $after ) {
-		$after->modify( '+' . (int) $this->get_recurrence() . ' seconds' );
+	protected function calculate_next(DateTime $after) {
+		$after->modify('+'.(int) $this->get_recurrence().' seconds');
 		return $after;
 	}
 
@@ -31,7 +31,7 @@ 
 	 * @return int
 	 */
 	public function interval_in_seconds() {
-		_deprecated_function( __METHOD__, '3.0.0', '(int)ActionScheduler_Abstract_RecurringSchedule::get_recurrence()' );
+		_deprecated_function(__METHOD__, '3.0.0', '(int)ActionScheduler_Abstract_RecurringSchedule::get_recurrence()');
 		return (int) $this->get_recurrence();
 	}
 
@@ -70,14 +70,14 @@ 
 	 * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
 	 */
 	public function __wakeup() {
-		if ( $this->scheduled_timestamp === null && $this->start_timestamp !== null ) {
+		if ($this->scheduled_timestamp === null && $this->start_timestamp !== null) {
 			$this->scheduled_timestamp = $this->start_timestamp;
-			unset( $this->start_timestamp );
+			unset($this->start_timestamp);
 		}
 
-		if ( $this->recurrence === null && $this->interval_in_seconds !== null ) {
+		if ($this->recurrence === null && $this->interval_in_seconds !== null) {
 			$this->recurrence = $this->interval_in_seconds;
-			unset( $this->interval_in_seconds );
+			unset($this->interval_in_seconds);
 		}
 		parent::__wakeup();
 	}

src/ext/action-scheduler/classes/schedules/ActionScheduler_CronSchedule.php

Indentation +96 added lines, -96 removed lines

@@ -5,101 +5,101 @@
  */
 class ActionScheduler_CronSchedule extends ActionScheduler_Abstract_RecurringSchedule implements ActionScheduler_Schedule {
 
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $start_timestamp = null;
-
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $cron = null;
-
-	/**
-	 * Wrapper for parent constructor to accept a cron expression string and map it to a CronExpression for this
-	 * objects $recurrence property.
-	 *
-	 * @param DateTime              $start The date & time to run the action at or after. If $start aligns with the CronSchedule passed via $recurrence, it will be used. If it does not align, the first matching date after it will be used.
-	 * @param CronExpression|string $recurrence The CronExpression used to calculate the schedule's next instance.
-	 * @param DateTime|null         $first (Optional) The date & time the first instance of this interval schedule ran. Default null, meaning this is the first instance.
-	 */
-	public function __construct( DateTime $start, $recurrence, DateTime $first = null ) {
-		if ( ! is_a( $recurrence, 'CronExpression' ) ) {
-			$recurrence = CronExpression::factory( $recurrence );
-		}
-
-		// For backward compatibility, we need to make sure the date is set to the first matching cron date, not whatever date is passed in. Importantly, by passing true as the 3rd param, if $start matches the cron expression, then it will be used. This was previously handled in the now deprecated next() method.
-		$date = $recurrence->getNextRunDate( $start, 0, true );
-
-		// parent::__construct() will set this to $date by default, but that may be different to $start now.
-		$first = empty( $first ) ? $start : $first;
-
-		parent::__construct( $date, $recurrence, $first );
-	}
-
-	/**
-	 * Calculate when an instance of this schedule would start based on a given
-	 * date & time using its the CronExpression.
-	 *
-	 * @param DateTime $after
-	 * @return DateTime
-	 */
-	protected function calculate_next( DateTime $after ) {
-		return $this->recurrence->getNextRunDate( $after, 0, false );
-	}
-
-	/**
-	 * @return string
-	 */
-	public function get_recurrence() {
-		return strval( $this->recurrence );
-	}
-
-	/**
-	 * Serialize cron schedules with data required prior to AS 3.0.0
-	 *
-	 * Prior to Action Scheduler 3.0.0, reccuring schedules used different property names to
-	 * refer to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
-	 * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
-	 * aligned properties and property names for better inheritance. To guard against the
-	 * possibility of infinite loops if downgrading to Action Scheduler < 3.0.0, we need to
-	 * also store the data with the old property names so if it's unserialized in AS < 3.0,
-	 * the schedule doesn't end up with a null recurrence.
-	 *
-	 * @return array
-	 */
-	public function __sleep() {
-
-		$sleep_params = parent::__sleep();
-
-		$this->start_timestamp = $this->scheduled_timestamp;
-		$this->cron            = $this->recurrence;
-
-		return array_merge(
-			$sleep_params,
-			array(
-				'start_timestamp',
-				'cron',
-			)
-		);
-	}
-
-	/**
-	 * Unserialize cron schedules serialized/stored prior to AS 3.0.0
-	 *
-	 * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
-	 */
-	public function __wakeup() {
-		if ( $this->scheduled_timestamp === null && $this->start_timestamp !== null ) {
-			$this->scheduled_timestamp = $this->start_timestamp;
-			unset( $this->start_timestamp );
-		}
-
-		if ( $this->recurrence === null && $this->cron !== null ) {
-			$this->recurrence = $this->cron;
-			unset( $this->cron );
-		}
-		parent::__wakeup();
-	}
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $start_timestamp = null;
+
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $cron = null;
+
+    /**
+     * Wrapper for parent constructor to accept a cron expression string and map it to a CronExpression for this
+     * objects $recurrence property.
+     *
+     * @param DateTime              $start The date & time to run the action at or after. If $start aligns with the CronSchedule passed via $recurrence, it will be used. If it does not align, the first matching date after it will be used.
+     * @param CronExpression|string $recurrence The CronExpression used to calculate the schedule's next instance.
+     * @param DateTime|null         $first (Optional) The date & time the first instance of this interval schedule ran. Default null, meaning this is the first instance.
+     */
+    public function __construct( DateTime $start, $recurrence, DateTime $first = null ) {
+        if ( ! is_a( $recurrence, 'CronExpression' ) ) {
+            $recurrence = CronExpression::factory( $recurrence );
+        }
+
+        // For backward compatibility, we need to make sure the date is set to the first matching cron date, not whatever date is passed in. Importantly, by passing true as the 3rd param, if $start matches the cron expression, then it will be used. This was previously handled in the now deprecated next() method.
+        $date = $recurrence->getNextRunDate( $start, 0, true );
+
+        // parent::__construct() will set this to $date by default, but that may be different to $start now.
+        $first = empty( $first ) ? $start : $first;
+
+        parent::__construct( $date, $recurrence, $first );
+    }
+
+    /**
+     * Calculate when an instance of this schedule would start based on a given
+     * date & time using its the CronExpression.
+     *
+     * @param DateTime $after
+     * @return DateTime
+     */
+    protected function calculate_next( DateTime $after ) {
+        return $this->recurrence->getNextRunDate( $after, 0, false );
+    }
+
+    /**
+     * @return string
+     */
+    public function get_recurrence() {
+        return strval( $this->recurrence );
+    }
+
+    /**
+     * Serialize cron schedules with data required prior to AS 3.0.0
+     *
+     * Prior to Action Scheduler 3.0.0, reccuring schedules used different property names to
+     * refer to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
+     * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
+     * aligned properties and property names for better inheritance. To guard against the
+     * possibility of infinite loops if downgrading to Action Scheduler < 3.0.0, we need to
+     * also store the data with the old property names so if it's unserialized in AS < 3.0,
+     * the schedule doesn't end up with a null recurrence.
+     *
+     * @return array
+     */
+    public function __sleep() {
+
+        $sleep_params = parent::__sleep();
+
+        $this->start_timestamp = $this->scheduled_timestamp;
+        $this->cron            = $this->recurrence;
+
+        return array_merge(
+            $sleep_params,
+            array(
+                'start_timestamp',
+                'cron',
+            )
+        );
+    }
+
+    /**
+     * Unserialize cron schedules serialized/stored prior to AS 3.0.0
+     *
+     * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
+     */
+    public function __wakeup() {
+        if ( $this->scheduled_timestamp === null && $this->start_timestamp !== null ) {
+            $this->scheduled_timestamp = $this->start_timestamp;
+            unset( $this->start_timestamp );
+        }
+
+        if ( $this->recurrence === null && $this->cron !== null ) {
+            $this->recurrence = $this->cron;
+            unset( $this->cron );
+        }
+        parent::__wakeup();
+    }
 }
 

Spacing +13 added lines, -13 removed lines

@@ -23,18 +23,18 @@ 
 	 * @param CronExpression|string $recurrence The CronExpression used to calculate the schedule's next instance.
 	 * @param DateTime|null         $first (Optional) The date & time the first instance of this interval schedule ran. Default null, meaning this is the first instance.
 	 */
-	public function __construct( DateTime $start, $recurrence, DateTime $first = null ) {
-		if ( ! is_a( $recurrence, 'CronExpression' ) ) {
-			$recurrence = CronExpression::factory( $recurrence );
+	public function __construct(DateTime $start, $recurrence, DateTime $first = null) {
+		if ( ! is_a($recurrence, 'CronExpression')) {
+			$recurrence = CronExpression::factory($recurrence);
 		}
 
 		// For backward compatibility, we need to make sure the date is set to the first matching cron date, not whatever date is passed in. Importantly, by passing true as the 3rd param, if $start matches the cron expression, then it will be used. This was previously handled in the now deprecated next() method.
-		$date = $recurrence->getNextRunDate( $start, 0, true );
+		$date = $recurrence->getNextRunDate($start, 0, true);
 
 		// parent::__construct() will set this to $date by default, but that may be different to $start now.
-		$first = empty( $first ) ? $start : $first;
+		$first = empty($first) ? $start : $first;
 
-		parent::__construct( $date, $recurrence, $first );
+		parent::__construct($date, $recurrence, $first);
 	}
 
 	/**
@@ -44,15 +44,15 @@ 
 	 * @param DateTime $after
 	 * @return DateTime
 	 */
-	protected function calculate_next( DateTime $after ) {
-		return $this->recurrence->getNextRunDate( $after, 0, false );
+	protected function calculate_next(DateTime $after) {
+		return $this->recurrence->getNextRunDate($after, 0, false);
 	}
 
 	/**
 	 * @return string
 	 */
 	public function get_recurrence() {
-		return strval( $this->recurrence );
+		return strval($this->recurrence);
 	}
 
 	/**
@@ -90,14 +90,14 @@ 
 	 * For more background, @see ActionScheduler_Abstract_RecurringSchedule::__wakeup().
 	 */
 	public function __wakeup() {
-		if ( $this->scheduled_timestamp === null && $this->start_timestamp !== null ) {
+		if ($this->scheduled_timestamp === null && $this->start_timestamp !== null) {
 			$this->scheduled_timestamp = $this->start_timestamp;
-			unset( $this->start_timestamp );
+			unset($this->start_timestamp);
 		}
 
-		if ( $this->recurrence === null && $this->cron !== null ) {
+		if ($this->recurrence === null && $this->cron !== null) {
 			$this->recurrence = $this->cron;
-			unset( $this->cron );
+			unset($this->cron);
 		}
 		parent::__wakeup();
 	}

src/ext/action-scheduler/classes/schedules/ActionScheduler_SimpleSchedule.php

Indentation +58 added lines, -58 removed lines

@@ -5,70 +5,70 @@
  */
 class ActionScheduler_SimpleSchedule extends ActionScheduler_Abstract_Schedule {
 
-	/**
-	 * Deprecated property @see $this->__wakeup() for details.
-	 **/
-	private $timestamp = null;
+    /**
+     * Deprecated property @see $this->__wakeup() for details.
+     **/
+    private $timestamp = null;
 
-	/**
-	 * @param DateTime $after
-	 *
-	 * @return DateTime|null
-	 */
-	public function calculate_next( DateTime $after ) {
-		return null;
-	}
+    /**
+     * @param DateTime $after
+     *
+     * @return DateTime|null
+     */
+    public function calculate_next( DateTime $after ) {
+        return null;
+    }
 
-	/**
-	 * @return bool
-	 */
-	public function is_recurring() {
-		return false;
-	}
+    /**
+     * @return bool
+     */
+    public function is_recurring() {
+        return false;
+    }
 
-	/**
-	 * Serialize schedule with data required prior to AS 3.0.0
-	 *
-	 * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
-	 * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
-	 * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
-	 * aligned properties and property names for better inheritance. To guard against the
-	 * scheduled date for single actions always being seen as "now" if downgrading to
-	 * Action Scheduler < 3.0.0, we need to also store the data with the old property names
-	 * so if it's unserialized in AS < 3.0, the schedule doesn't end up with a null recurrence.
-	 *
-	 * @return array
-	 */
-	public function __sleep() {
+    /**
+     * Serialize schedule with data required prior to AS 3.0.0
+     *
+     * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
+     * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
+     * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
+     * aligned properties and property names for better inheritance. To guard against the
+     * scheduled date for single actions always being seen as "now" if downgrading to
+     * Action Scheduler < 3.0.0, we need to also store the data with the old property names
+     * so if it's unserialized in AS < 3.0, the schedule doesn't end up with a null recurrence.
+     *
+     * @return array
+     */
+    public function __sleep() {
 
-		$sleep_params = parent::__sleep();
+        $sleep_params = parent::__sleep();
 
-		$this->timestamp = $this->scheduled_timestamp;
+        $this->timestamp = $this->scheduled_timestamp;
 
-		return array_merge(
-			$sleep_params,
-			array(
-				'timestamp',
-			)
-		);
-	}
+        return array_merge(
+            $sleep_params,
+            array(
+                'timestamp',
+            )
+        );
+    }
 
-	/**
-	 * Unserialize recurring schedules serialized/stored prior to AS 3.0.0
-	 *
-	 * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
-	 * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
-	 * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
-	 * aligned properties and property names for better inheritance. To maintain backward
-	 * compatibility with schedules serialized and stored prior to 3.0, we need to correctly
-	 * map the old property names with matching visibility.
-	 */
-	public function __wakeup() {
+    /**
+     * Unserialize recurring schedules serialized/stored prior to AS 3.0.0
+     *
+     * Prior to Action Scheduler 3.0.0, schedules used different property names to refer
+     * to equivalent data. For example, ActionScheduler_IntervalSchedule::start_timestamp
+     * was the same as ActionScheduler_SimpleSchedule::timestamp. Action Scheduler 3.0.0
+     * aligned properties and property names for better inheritance. To maintain backward
+     * compatibility with schedules serialized and stored prior to 3.0, we need to correctly
+     * map the old property names with matching visibility.
+     */
+    public function __wakeup() {
 
-		if ( $this->scheduled_timestamp === null && $this->timestamp !== null ) {
-			$this->scheduled_timestamp = $this->timestamp;
-			unset( $this->timestamp );
-		}
-		parent::__wakeup();
-	}
+        if ( $this->scheduled_timestamp === null && $this->timestamp !== null ) {
+            $this->scheduled_timestamp = $this->timestamp;
+            unset( $this->timestamp );
+        }
+        parent::__wakeup();
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
 	 *
 	 * @return DateTime|null
 	 */
-	public function calculate_next( DateTime $after ) {
+	public function calculate_next(DateTime $after) {
 		return null;
 	}
 
@@ -65,9 +65,9 @@ 
 	 */
 	public function __wakeup() {
 
-		if ( $this->scheduled_timestamp === null && $this->timestamp !== null ) {
+		if ($this->scheduled_timestamp === null && $this->timestamp !== null) {
 			$this->scheduled_timestamp = $this->timestamp;
-			unset( $this->timestamp );
+			unset($this->timestamp);
 		}
 		parent::__wakeup();
 	}

src/ext/action-scheduler/classes/ActionScheduler_LogEntry.php

Indentation +44 added lines, -44 removed lines

@@ -5,63 +5,63 @@
  */
 class ActionScheduler_LogEntry {
 
-	/**
-	 * @var int $action_id
-	 */
-	protected $action_id = '';
+    /**
+     * @var int $action_id
+     */
+    protected $action_id = '';
 
-	/**
-	 * @var string $message
-	 */
-	protected $message = '';
+    /**
+     * @var string $message
+     */
+    protected $message = '';
 
-	/**
-	 * @var Datetime $date
-	 */
-	protected $date;
+    /**
+     * @var Datetime $date
+     */
+    protected $date;
 
-	/**
-	 * Constructor
-	 *
-	 * @param mixed    $action_id Action ID
-	 * @param string   $message   Message
-	 * @param Datetime $date    Datetime object with the time when this log entry was created. If this parameter is
-	 *                          not provided a new Datetime object (with current time) will be created.
-	 */
-	public function __construct( $action_id, $message, $date = null ) {
+    /**
+     * Constructor
+     *
+     * @param mixed    $action_id Action ID
+     * @param string   $message   Message
+     * @param Datetime $date    Datetime object with the time when this log entry was created. If this parameter is
+     *                          not provided a new Datetime object (with current time) will be created.
+     */
+    public function __construct( $action_id, $message, $date = null ) {
 
-		/*
+        /*
 		 * ActionScheduler_wpCommentLogger::get_entry() previously passed a 3rd param of $comment->comment_type
 		 * to ActionScheduler_LogEntry::__construct(), goodness knows why, and the Follow-up Emails plugin
 		 * hard-codes loading its own version of ActionScheduler_wpCommentLogger with that out-dated method,
 		 * goodness knows why, so we need to guard against that here instead of using a DateTime type declaration
 		 * for the constructor's 3rd param of $date and causing a fatal error with older versions of FUE.
 		 */
-		if ( null !== $date && ! is_a( $date, 'DateTime' ) ) {
-			_doing_it_wrong( __METHOD__, 'The third parameter must be a valid DateTime instance, or null.', '2.0.0' );
-			$date = null;
-		}
+        if ( null !== $date && ! is_a( $date, 'DateTime' ) ) {
+            _doing_it_wrong( __METHOD__, 'The third parameter must be a valid DateTime instance, or null.', '2.0.0' );
+            $date = null;
+        }
 
-		$this->action_id = $action_id;
-		$this->message   = $message;
-		$this->date      = $date ? $date : new Datetime();
-	}
+        $this->action_id = $action_id;
+        $this->message   = $message;
+        $this->date      = $date ? $date : new Datetime();
+    }
 
-	/**
-	 * Returns the date when this log entry was created
-	 *
-	 * @return Datetime
-	 */
-	public function get_date() {
-		return $this->date;
-	}
+    /**
+     * Returns the date when this log entry was created
+     *
+     * @return Datetime
+     */
+    public function get_date() {
+        return $this->date;
+    }
 
-	public function get_action_id() {
-		return $this->action_id;
-	}
+    public function get_action_id() {
+        return $this->action_id;
+    }
 
-	public function get_message() {
-		return $this->message;
-	}
+    public function get_message() {
+        return $this->message;
+    }
 }
 

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
 	 * @param Datetime $date    Datetime object with the time when this log entry was created. If this parameter is
 	 *                          not provided a new Datetime object (with current time) will be created.
 	 */
-	public function __construct( $action_id, $message, $date = null ) {
+	public function __construct($action_id, $message, $date = null) {
 
 		/*
 		 * ActionScheduler_wpCommentLogger::get_entry() previously passed a 3rd param of $comment->comment_type
@@ -37,8 +37,8 @@ 
 		 * goodness knows why, so we need to guard against that here instead of using a DateTime type declaration
 		 * for the constructor's 3rd param of $date and causing a fatal error with older versions of FUE.
 		 */
-		if ( null !== $date && ! is_a( $date, 'DateTime' ) ) {
-			_doing_it_wrong( __METHOD__, 'The third parameter must be a valid DateTime instance, or null.', '2.0.0' );
+		if (null !== $date && ! is_a($date, 'DateTime')) {
+			_doing_it_wrong(__METHOD__, 'The third parameter must be a valid DateTime instance, or null.', '2.0.0');
 			$date = null;
 		}
 

src/ext/action-scheduler/classes/actions/ActionScheduler_Action.php

Indentation +76 added lines, -76 removed lines

@@ -4,93 +4,93 @@
  * Class ActionScheduler_Action
  */
 class ActionScheduler_Action {
-	protected $hook = '';
-	protected $args = array();
-	/** @var ActionScheduler_Schedule */
-	protected $schedule = null;
-	protected $group    = '';
+    protected $hook = '';
+    protected $args = array();
+    /** @var ActionScheduler_Schedule */
+    protected $schedule = null;
+    protected $group    = '';
 
-	public function __construct( $hook, array $args = array(), ActionScheduler_Schedule $schedule = null, $group = '' ) {
-		$schedule = empty( $schedule ) ? new ActionScheduler_NullSchedule() : $schedule;
-		$this->set_hook( $hook );
-		$this->set_schedule( $schedule );
-		$this->set_args( $args );
-		$this->set_group( $group );
-	}
+    public function __construct( $hook, array $args = array(), ActionScheduler_Schedule $schedule = null, $group = '' ) {
+        $schedule = empty( $schedule ) ? new ActionScheduler_NullSchedule() : $schedule;
+        $this->set_hook( $hook );
+        $this->set_schedule( $schedule );
+        $this->set_args( $args );
+        $this->set_group( $group );
+    }
 
-	/**
-	 * Executes the action.
-	 *
-	 * If no callbacks are registered, an exception will be thrown and the action will not be
-	 * fired. This is useful to help detect cases where the code responsible for setting up
-	 * a scheduled action no longer exists.
-	 *
-	 * @throws Exception If no callbacks are registered for this action.
-	 */
-	public function execute() {
-		$hook = $this->get_hook();
+    /**
+     * Executes the action.
+     *
+     * If no callbacks are registered, an exception will be thrown and the action will not be
+     * fired. This is useful to help detect cases where the code responsible for setting up
+     * a scheduled action no longer exists.
+     *
+     * @throws Exception If no callbacks are registered for this action.
+     */
+    public function execute() {
+        $hook = $this->get_hook();
 
-		if ( ! has_action( $hook ) ) {
-			throw new Exception(
-				sprintf(
-					/* translators: 1: action hook. */
-					__( 'Scheduled action for %1$s will not be executed as no callbacks are registered.', 'action-scheduler' ),
-					$hook
-				)
-			);
-		}
+        if ( ! has_action( $hook ) ) {
+            throw new Exception(
+                sprintf(
+                    /* translators: 1: action hook. */
+                    __( 'Scheduled action for %1$s will not be executed as no callbacks are registered.', 'action-scheduler' ),
+                    $hook
+                )
+            );
+        }
 
-		do_action_ref_array( $hook, array_values( $this->get_args() ) );
-	}
+        do_action_ref_array( $hook, array_values( $this->get_args() ) );
+    }
 
-	/**
-	 * @param string $hook
-	 */
-	protected function set_hook( $hook ) {
-		$this->hook = $hook;
-	}
+    /**
+     * @param string $hook
+     */
+    protected function set_hook( $hook ) {
+        $this->hook = $hook;
+    }
 
-	public function get_hook() {
-		return $this->hook;
-	}
+    public function get_hook() {
+        return $this->hook;
+    }
 
-	protected function set_schedule( ActionScheduler_Schedule $schedule ) {
-		$this->schedule = $schedule;
-	}
+    protected function set_schedule( ActionScheduler_Schedule $schedule ) {
+        $this->schedule = $schedule;
+    }
 
-	/**
-	 * @return ActionScheduler_Schedule
-	 */
-	public function get_schedule() {
-		return $this->schedule;
-	}
+    /**
+     * @return ActionScheduler_Schedule
+     */
+    public function get_schedule() {
+        return $this->schedule;
+    }
 
-	protected function set_args( array $args ) {
-		$this->args = $args;
-	}
+    protected function set_args( array $args ) {
+        $this->args = $args;
+    }
 
-	public function get_args() {
-		return $this->args;
-	}
+    public function get_args() {
+        return $this->args;
+    }
 
-	/**
-	 * @param string $group
-	 */
-	protected function set_group( $group ) {
-		$this->group = $group;
-	}
+    /**
+     * @param string $group
+     */
+    protected function set_group( $group ) {
+        $this->group = $group;
+    }
 
-	/**
-	 * @return string
-	 */
-	public function get_group() {
-		return $this->group;
-	}
+    /**
+     * @return string
+     */
+    public function get_group() {
+        return $this->group;
+    }
 
-	/**
-	 * @return bool If the action has been finished
-	 */
-	public function is_finished() {
-		return false;
-	}
+    /**
+     * @return bool If the action has been finished
+     */
+    public function is_finished() {
+        return false;
+    }
 }

Spacing +13 added lines, -13 removed lines

@@ -10,12 +10,12 @@ 
 	protected $schedule = null;
 	protected $group    = '';
 
-	public function __construct( $hook, array $args = array(), ActionScheduler_Schedule $schedule = null, $group = '' ) {
-		$schedule = empty( $schedule ) ? new ActionScheduler_NullSchedule() : $schedule;
-		$this->set_hook( $hook );
-		$this->set_schedule( $schedule );
-		$this->set_args( $args );
-		$this->set_group( $group );
+	public function __construct($hook, array $args = array(), ActionScheduler_Schedule $schedule = null, $group = '') {
+		$schedule = empty($schedule) ? new ActionScheduler_NullSchedule() : $schedule;
+		$this->set_hook($hook);
+		$this->set_schedule($schedule);
+		$this->set_args($args);
+		$this->set_group($group);
 	}
 
 	/**
@@ -30,23 +30,23 @@ 
 	public function execute() {
 		$hook = $this->get_hook();
 
-		if ( ! has_action( $hook ) ) {
+		if ( ! has_action($hook)) {
 			throw new Exception(
 				sprintf(
 					/* translators: 1: action hook. */
-					__( 'Scheduled action for %1$s will not be executed as no callbacks are registered.', 'action-scheduler' ),
+					__('Scheduled action for %1$s will not be executed as no callbacks are registered.', 'action-scheduler'),
 					$hook
 				)
 			);
 		}
 
-		do_action_ref_array( $hook, array_values( $this->get_args() ) );
+		do_action_ref_array($hook, array_values($this->get_args()));
 	}
 
 	/**
 	 * @param string $hook
 	 */
-	protected function set_hook( $hook ) {
+	protected function set_hook($hook) {
 		$this->hook = $hook;
 	}
 
@@ -54,7 +54,7 @@ 
 		return $this->hook;
 	}
 
-	protected function set_schedule( ActionScheduler_Schedule $schedule ) {
+	protected function set_schedule(ActionScheduler_Schedule $schedule) {
 		$this->schedule = $schedule;
 	}
 
@@ -65,7 +65,7 @@ 
 		return $this->schedule;
 	}
 
-	protected function set_args( array $args ) {
+	protected function set_args(array $args) {
 		$this->args = $args;
 	}
 
@@ -76,7 +76,7 @@ 
 	/**
 	 * @param string $group
 	 */
-	protected function set_group( $group ) {
+	protected function set_group($group) {
 		$this->group = $group;
 	}
 

src/ext/action-scheduler/classes/actions/ActionScheduler_FinishedAction.php

Indentation +6 added lines, -6 removed lines

@@ -5,12 +5,12 @@
  */
 class ActionScheduler_FinishedAction extends ActionScheduler_Action {
 
-	public function execute() {
-		// don't execute
-	}
+    public function execute() {
+        // don't execute
+    }
 
-	public function is_finished() {
-		return true;
-	}
+    public function is_finished() {
+        return true;
+    }
 }
 

src/ext/action-scheduler/classes/actions/ActionScheduler_NullAction.php

Indentation +6 added lines, -6 removed lines

@@ -5,12 +5,12 @@
  */
 class ActionScheduler_NullAction extends ActionScheduler_Action {
 
-	public function __construct( $hook = '', array $args = array(), ActionScheduler_Schedule $schedule = null ) {
-		$this->set_schedule( new ActionScheduler_NullSchedule() );
-	}
+    public function __construct( $hook = '', array $args = array(), ActionScheduler_Schedule $schedule = null ) {
+        $this->set_schedule( new ActionScheduler_NullSchedule() );
+    }
 
-	public function execute() {
-		// don't execute
-	}
+    public function execute() {
+        // don't execute
+    }
 }
 

Spacing +2 added lines, -2 removed lines

@@ -5,8 +5,8 @@
  */
 class ActionScheduler_NullAction extends ActionScheduler_Action {
 
-	public function __construct( $hook = '', array $args = array(), ActionScheduler_Schedule $schedule = null ) {
-		$this->set_schedule( new ActionScheduler_NullSchedule() );
+	public function __construct($hook = '', array $args = array(), ActionScheduler_Schedule $schedule = null) {
+		$this->set_schedule(new ActionScheduler_NullSchedule());
 	}
 
 	public function execute() {