Changeset 7193

Timestamp:

04/26/2021 01:30:21 PM (7 months ago)

Author:

johnjamesjacoby

Message:

Akismet: Remove "publish" status restriction in check_post().

This commit ensures that posts of any status are sent to Akismet, even if they had previously been published and are now being edited. The intent of this change is to remove a restriction without changing the behavior from a user's perspective.

Also fixes a typo and an incorrectly typed return value.

Props procifer, douglsmith.

In trunk for 2.7.0. See #3410.

File:

• trunk/src/includes/extend/akismet.php (modified) (3 diffs)

trunk/src/includes/extend/akismet.php

@@ -89 +89 @@
         $user_data = array();
         $post_permalink = '';
-
-        // Post is not published
-        if ( bbp_get_public_status_id() !== $post_data['post_status'] ) {
-            return $post_data;
-        }
 
         // Cast the post_author to 0 if it's empty
@@ -200 +195 @@
      * necessary. For example, switch the status to `spam` if spammy.
      *
-     * Note: this method also skiis responsible for allowing users who can moderate, to
+     * Note: this method also is responsible for allowing users who can moderate to
      * never have their posts marked as spam. This is because they are "trusted"
      * users. However, their posts are still sent to Akismet to be checked.
@@ -775 +770 @@
         // Bail if the response is an error
         if ( is_wp_error( $response ) ) {
-            return '';
+            return array( '', '' );
         }