Changeset 4974 for sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-hooks.php

Timestamp:

02/21/2017 05:23:13 PM (9 years ago)

Author:

SergeyBiryukov

Message:

Support Forums: Reduce the list of inline topic admin links to Edit, Close, Spam, Unapprove, Archive:

• This helps to avoid visual clutter and provide enough room for translations.
• Less commonly used actions (Merge, Stick) are still available via "Topic Admin" sidebar section.
• Reply link is removed completely, as it's redundant since [4720] introduced a sidebar link for the same purpose.

Fixes #2398.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-hooks.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-hooks.php

@@ -43 +43 @@
 
     /**
-     * Remove "Trash" from admin links. Trashing a topic or reply will eventually
-     * permanently delete it when the trash is emptied. Better to mark it as
-     * pending or spam.
+     * Remove some unneeded or redundant admin links for topics and replies,
+     * move less commonly used inline quick links to 'Topic Admin' sidebar section.
      *
      * @param array $r       Admin links array.
@@ -52 +51 @@
      */
     public function admin_links( $r, $post_id ) {
+        /*
+         * Remove 'Trash' from admin links. Trashing a topic or reply will eventually
+         * permanently delete it when the trash is emptied. Better to mark it as pending or spam.
+         */
         unset( $r['trash'] );
+
+        /*
+         * Remove 'Reply' link. The theme adds its own 'Reply to Topic' sidebar link
+         * for quick access to reply form, making the default inline link redundant.
+         */
+        unset( $r['reply'] );
+
+        /*
+         * The following actions are removed from inline quick links as less commonly used,
+         * but are still available via 'Topic Admin' sidebar section.
+         */
+        if ( ! did_action( 'wporg_compat_single_topic_sidebar_pre' ) ) {
+            // Remove 'Merge' link.
+            unset( $r['merge'] );
+
+            // Remove 'Stick' link for moderators, but keep it for plugin/theme authors and contributors.
+            if ( current_user_can( 'moderate', $post_id ) ) {
+                unset( $r['stick'] );
+            }
+        }
 
         return $r;