Delete old posts after processing items

akirk · Feb 21, 2025 · 583e0fa · 583e0fa
1 parent b09921d
commit 583e0fa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 10 deletions.
diff --git a/friends.php b/friends.php
@@ -60,7 +60,7 @@
 add_action( 'plugins_loaded', array( __NAMESPACE__ . '\Friends', 'init' ) );
 add_action( 'admin_init', array( __NAMESPACE__ . '\Plugin_Installer', 'register_hooks' ) );
 
-if ( is_admin() && FRIENDS_VERSION > get_option( 'friends_plugin_version' ) ) {
+if ( is_admin() && version_compare( get_option( 'friends_plugin_version' ), FRIENDS_VERSION, '<' ) ) {
 	add_action( 'admin_init', array( __NAMESPACE__ . '\Friends', 'upgrade_plugin' ) );
 }
 

diff --git a/includes/class-feed.php b/includes/class-feed.php
@@ -120,10 +120,6 @@ public function cron_friends_refresh_feeds() {
 			$feed->set_polling_now();
 			$this->retrieve_feed( $feed );
 			$feed->was_polled();
-			$friend_user = $feed->get_friend_user();
-			if ( $friend_user ) {
-				$friend_user->delete_outdated_posts();
-			}
 		}
 	}
 
@@ -144,7 +140,6 @@ public function friends_retrieve_user_feeds( $user_id ) {
 				$feed->set_polling_now();
 				$this->retrieve_feed( $feed );
 				$feed->was_polled();
-				$friend_user->delete_outdated_posts();
 			}
 		}
 	}
@@ -299,10 +294,6 @@ public function retrieve_friend_posts( $ignore_due_date = false ) {
 			$feed->set_polling_now();
 			$this->retrieve_feed( $feed );
 			$feed->was_polled();
-			$friend_user = $feed->get_friend_user();
-			if ( $friend_user ) {
-				$friend_user->delete_outdated_posts();
-			}
 		}
 	}
 
@@ -728,6 +719,8 @@ public function process_incoming_feed_items( array $items, User_Feed $user_feed
 
 		do_action( 'friends_retrieved_new_posts', $user_feed, $new_posts, $modified_posts, $friend_user );
 
+		$friend_user->delete_outdated_posts();
+
 		return $new_posts;
 	}