Changeset 7235 for trunk/src/includes/common/functions.php

Timestamp:

02/17/2022 08:08:28 PM (3 years ago)

Author:

johnjamesjacoby

Message:

Moderation: use the correct post_status when untrashing a topic/reply.

This change fixes a bug/regression (since WordPress 5.6.0) that was causing untrashed topics & replies to use an unintended post_status value.

It fixes it by adding a new function ('bbp_fix_untrash_post_status()') and hooking it to the wp_untrash_post_status filter in WordPress, and overriding the results as needed.

Props r-a-y.

In trunk for 2.7.0. Fixes #3433.

File:

• trunk/src/includes/common/functions.php (modified) (1 diff)

trunk/src/includes/common/functions.php

@@ -186 +186 @@
 
     return $data;
+}
+
+/**
+ * Use the previous status when restoring a topic or reply.
+ *
+ * Fixes an issue since WordPress 5.6.0. See
+ * {@link https://bbpress.trac.wordpress.org/ticket/3433}.
+ *
+ * @since 2.6.10 bbPress (r7233)
+ *
+ * @param string $new_status      New status to use when untrashing. Default: 'draft'
+ * @param int    $post_id         Post ID
+ * @param string $previous_status Previous post status from '_wp_trash_meta_status' meta key. Default: 'pending'
+ */
+function bbp_fix_untrash_post_status( $new_status = 'draft', $post_id = 0, $previous_status = 'pending' ) {
+
+    // Bail if not Topic or Reply
+    if ( ! bbp_is_topic( $post_id ) && ! bbp_is_reply( $post_id ) ) {
+        return $new_status;
+    }
+
+    // Prefer the previous status, falling back to the new status
+    $retval = ! empty( $previous_status )
+        ? $previous_status
+        : $new_status;
+
+    return $retval;
 }