Merge pull request #531 from ProgressPlanner/ari/new-task-publish-draft-content

New task provider: Check unpublished (draft/pending) content

Author: aristath

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

@@ -31,6 +31,7 @@
 use Progress_Planner\Suggested_Tasks\Providers\Fewer_Tags;
 use Progress_Planner\Suggested_Tasks\Providers\Remove_Terms_Without_Posts;
 use Progress_Planner\Suggested_Tasks\Providers\Update_Term_Description;
+use Progress_Planner\Suggested_Tasks\Providers\Unpublished_Content;
 use Progress_Planner\Suggested_Tasks\Providers\Collaborator;
 
 /**
@@ -73,6 +74,7 @@ public function __construct() {
 			new Remove_Terms_Without_Posts(),
 			new Fewer_Tags(),
 			new Update_Term_Description(),
+			new Unpublished_Content(),
 			new Collaborator(),
 		];
 

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

@@ -19,6 +19,7 @@
 use Progress_Planner\Suggested_Tasks\Data_Collector\Post_Tag_Count;
 use Progress_Planner\Suggested_Tasks\Data_Collector\Published_Post_Count;
 use Progress_Planner\Suggested_Tasks\Data_Collector\Yoast_Orphaned_Content;
+use Progress_Planner\Suggested_Tasks\Data_Collector\Unpublished_Content;
 
 /**
  * Base data collector.
@@ -50,6 +51,7 @@ public function __construct() {
 			new Terms_Without_Description(),
 			new Post_Tag_Count(),
 			new Published_Post_Count(),
+			new Unpublished_Content(),
 		];
 
 		// Add the plugin integration.

classes/suggested-tasks/data-collector/class-unpublished-content.php

@@ -0,0 +1,110 @@
+<?php
+/**
+ * Terms without description data collector.
+ *
+ * @package Progress_Planner
+ */
+
+namespace Progress_Planner\Suggested_Tasks\Data_Collector;
+
+use Progress_Planner\Suggested_Tasks\Data_Collector\Base_Data_Collector;
+
+/**
+ * Terms without posts data collector class.
+ */
+class Unpublished_Content extends Base_Data_Collector {
+
+	/**
+	 * The data key.
+	 *
+	 * @var string
+	 */
+	protected const DATA_KEY = 'unpublished_content';
+
+	/**
+	 * The minimum number of posts.
+	 *
+	 * @var int
+	 */
+	protected const POSTS_TO_COLLECT = 1;
+
+	/**
+	 * Initialize the data collector.
+	 *
+	 * @return void
+	 */
+	public function init() {
+		\add_action( 'delete_post', [ $this, 'update_unpublished_content_cache' ], 10 );
+
+		// We need to update the cache when a post status changes.
+		\add_action( 'transition_post_status', [ $this, 'on_post_status_changed' ], 10, 3 );
+	}
+
+	/**
+	 * Update the cache when post status changes.
+	 *
+	 * @param string   $new_status The new status.
+	 * @param string   $old_status The old status.
+	 * @param \WP_Post $post The post.
+	 *
+	 * @return void
+	 */
+	public function on_post_status_changed( $new_status, $old_status, $post ) {
+		// If the status is the same or we don't include the post type, do nothing.
+		if ( $old_status === $new_status || ! \in_array( \get_post_type( $post ), \progress_planner()->get_settings()->get_post_types_names(), true ) ) {
+			return;
+		}
+
+		if ( $new_status === 'publish' ) {
+			$this->update_cache();
+		}
+	}
+
+	/**
+	 * Update the cache when term is edited or deleted.
+	 *
+	 * @return void
+	 */
+	public function update_unpublished_content_cache() {
+		$this->update_cache();
+	}
+
+	/**
+	 * Query the terms without posts.
+	 *
+	 * @return array|null
+	 */
+	protected function calculate_data() {
+		$args = [
+			'post_type'      => \progress_planner()->get_settings()->get_post_types_names(),
+			'posts_per_page' => static::POSTS_TO_COLLECT,
+			'post_status'    => [ 'draft', 'pending' ],
+			'orderby'        => 'modified',
+			'order'          => 'ASC',
+			'fields'         => 'ids',
+			'date_query'     => [
+				[
+					'column' => 'post_modified',
+					'before' => '-1 week',
+				],
+			],
+		];
+
+		/**
+		 * Filter the post IDs to exclude from the query.
+		 *
+		 * @param array $post__not_in The post IDs to exclude.
+		 *
+		 * @return array
+		 */
+		$post__not_in = \apply_filters( 'progress_planner_unpublished_content_exclude_post_ids', [] );
+
+		if ( ! empty( $post__not_in ) ) {
+			$args['post__not_in'] = $post__not_in;
+		}
+
+		$posts = \get_posts( $args );
+
+		return $posts ? [ 'post_id' => $posts[0] ] : null;
+	}
+}

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

@@ -344,25 +344,6 @@ public function get_tasks_to_inject() {
 		return $added_tasks;
 	}
 
-	/**
-	 * Get the post ID from the task ID.
-	 *
-	 * @param string $task_id The task ID.
-	 *
-	 * @return \WP_Post|null
-	 */
-	public function get_post_from_task_id( $task_id ) {
-		$tasks = \progress_planner()->get_suggested_tasks_db()->get_tasks_by( [ 'task_id' => $task_id ] );
-
-		if ( empty( $tasks ) ) {
-			return null;
-		}
-
-		return isset( $tasks[0]->target_post_id ) && $tasks[0]->target_post_id
-			? \get_post( $tasks[0]->target_post_id )
-			: null;
-	}
-
 	/**
 	 * This method is added just to override the parent method.
 	 * For this task provider we can't check if it is snoozed like for other as we snooze the task for specific post.

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

@@ -341,7 +341,7 @@ public function is_onboarding_task() {
 	}
 
 	/**
-	 * Check if a task category is snoozed.
+	 * Check if task provider is snoozed.
 	 *
 	 * @return bool
 	 */