Changeset 11892

Timestamp:

05/26/2022 05:12:01 PM (2 years ago)

Author:

iandunn

Message:

Profiles: Don't create redundant activity for x-posts.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-profiles-wp-activity-notifier

Files:

• tests/e2e.php (modified) (2 diffs)
• wporg-profiles-wp-activity-notifier.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-profiles-wp-activity-notifier/tests/e2e.php

@@ -57 +57 @@
 
     } else {
+        // These should notify
         $notifier->maybe_notify_new_published_post( 'publish', 'draft', get_post( 1802 ) ); // post
         sleep( 1 ); // buddypress doesn't show activity that happens at the exact same time
@@ -62 +63 @@
         sleep( 1 );
         $notifier->maybe_notify_new_published_post( 'publish', 'draft', get_post( 1832 ) ); // course
+
+        // These should not notify
+        $notifier->maybe_notify_new_published_post( 'publish', 'draft', get_post( 722 ) ); // x-post (should not notify, check https://profiles.wordpress.org/jmdodd/)
     }
 }

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-profiles-wp-activity-notifier/wporg-profiles-wp-activity-notifier.php

@@ -113 +113 @@
         // Some Handbook posts are automatically created and don't have an author.
         elseif ( $post->wporg_markdown_source || ! $post->post_author ) {
+            $notifiable = false;
+        }
+
+        elseif ( $post->_xpost_original_permalink || str_starts_with( $post->post_name, 'xpost-' ) ) {
             $notifiable = false;
         }