Changeset 8117 for sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-bbp-user-moderation/inc/class-plugin.php

Timestamp:

01/21/2019 05:38:26 AM (6 years ago)

Author:

dd32

Message:

Support Forums: User Moderation: Allow modwatched users to edit posts, prior to the topic/reply being approved.

This is a follow up to #4035 where posting a new topic would result in a 404 page rather than the pending thread, simply allowing edits prior to approval seems the easiest solution.
Modwatched users are still unable to edit approved topics/replies as per #4035.

See #4035.
Fixes #4087.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-bbp-user-moderation/inc/class-plugin.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-bbp-user-moderation/inc/class-plugin.php

@@ -65 +65 @@
 
             // Prevent editing their topics/replies.
-            add_filter( 'bbp_map_meta_caps', array( $this, 'bbp_map_meta_caps' ), 100, 3 );
+            add_filter( 'bbp_map_meta_caps', array( $this, 'bbp_map_meta_caps' ), 100, 4 );
         }
 
@@ -315 +315 @@
     }
 
-    public function bbp_map_meta_caps( $caps, $cap, $user_id ) {
+    public function bbp_map_meta_caps( $caps, $cap, $user_id, $args ) {
         $limited_caps = array(
             'edit_topic',
@@ -322 +322 @@
 
         if ( in_array( $cap, $limited_caps, true ) ) {
-            $caps = array( 'do_not_allow' );
+            $do_not_allow = array( 'do_not_allow' );
+
+            if ( ! isset( $args[0] ) ) {
+                // No post was specified.. Do not allow editing of an unknown.
+                $caps = $do_not_allow;
+            } else {
+                // If we can't find the post OR the post is NOT pending.. do not allow
+                $_post = get_post( $args[0] );
+                if ( ! $_post || $_post->post_status !== bbp_get_pending_status_id() ) {
+                    $caps = $do_not_allow;
+                }
+                // else {
+                //  // We found the post, and it's status is pending.. fall-through and allow.
+                // }
+            }
         }