handle tasks when snooze period is over

Author: ilicfilip

classes/class-page-types.php

@@ -383,14 +383,14 @@ public function update_option_page_on_front( $page_id ) {
 	 *
 	 * Runs on post_updated hook.
 	 *
-	 * @param int      $post_id The post ID.
-	 * @param \WP_Post $post    The post object.
+	 * @param int           $post_id The post ID.
+	 * @param \WP_Post|null $post    The post object.
 	 *
 	 * @return void
 	 */
 	public function post_updated( $post_id, $post ) {
 		// Check if the post is set as a homepage.
-		if ( 'page' !== $post->post_type || 'publish' !== $post->post_status ) {
+		if ( ! $post || 'page' !== $post->post_type || 'publish' !== $post->post_status ) {
 			return;
 		}
 

classes/suggested-tasks/class-tasks-manager.php

@@ -84,6 +84,9 @@ public function __construct() {
 
 		// Add the cleanup action.
 		\add_action( 'admin_init', [ $this, 'cleanup_pending_tasks' ] );
+
+		// Handle tasks when snoozed period is over.
+		\add_action( 'transition_post_status', [ $this, 'handle_task_unsnooze' ], 10, 3 );
 	}
 
 	/**
@@ -287,4 +290,38 @@ public function cleanup_pending_tasks() {
 
 		\progress_planner()->get_utils__cache()->set( 'cleanup_pending_tasks', true, DAY_IN_SECONDS );
 	}
+
+	/**
+	 * Handle task unsnooze.
+	 *
+	 * @param string   $new_status The new status.
+	 * @param string   $old_status The old status.
+	 * @param \WP_Post $post The post.
+	 *
+	 * @return void
+	 */
+	public function handle_task_unsnooze( $new_status, $old_status, $post ) {
+
+		// Early exit if it's not task for which snooze period is over.
+		if ( 'future' !== $old_status || 'publish' !== $new_status || 'prpl_recommendations' !== get_post_type( $post ) ) {
+			return;
+		}
+
+		$task = \progress_planner()->get_suggested_tasks_db()->get_post( $post->ID );
+		if ( ! $task ) {
+			return;
+		}
+
+		$task_provider = $this->get_task_provider( $task->get_provider_id() );
+
+		// Delete tasks which don't have a task provider or repetitive tasks which were created in the previous week.
+		if ( ! $task_provider || ( $task_provider->is_repetitive() && ( ! $task->date || \gmdate( 'YW' ) !== (string) $task->date ) ) ) {
+			\progress_planner()->get_suggested_tasks_db()->delete_recommendation( $task->ID );
+		}
+
+		// Delete the task if it is not relevant anymore.
+		if ( $task_provider instanceof \Progress_Planner\Suggested_Tasks\Providers\Tasks && ! $task_provider->should_add_task() ) {
+			\progress_planner()->get_suggested_tasks_db()->delete_recommendation( $task->ID );
+		}
+	}
 }

classes/suggested-tasks/data-collector/class-post-author.php

@@ -34,14 +34,14 @@ public function init() {
 	/**
 	 * Update the cache when the post author changes.
 	 *
-	 * @param int      $post_id The post ID.
-	 * @param \WP_Post $post_after The post object following the update.
-	 * @param \WP_Post $post_before The post object before the update.
+	 * @param int           $post_id The post ID.
+	 * @param \WP_Post|null $post_after The post object following the update.
+	 * @param \WP_Post      $post_before The post object before the update.
 	 *
 	 * @return void
 	 */
 	public function update_post_author_on_change( $post_id, $post_after, $post_before ) {
-		if ( $post_after->post_author !== $post_before->post_author ) {
+		if ( $post_after && $post_after->post_author !== $post_before->post_author ) {
 			$this->update_cache();
 		}
 	}

classes/suggested-tasks/providers/class-tasks.php

@@ -443,7 +443,7 @@ public function evaluate_task( $task_id ) {
 	 *
 	 * @return bool true means that the task condition is satisfied, meaning that we don't need to add the task or task was completed.
 	 */
-	abstract protected function should_add_task();
+	abstract public function should_add_task();
 
 	/**
 	 * Alias for should_add_task(), for better readability when using in the evaluate_task() method.