Changeset 5399

Timestamp:

06/14/2014 10:14:34 PM (11 years ago)

Author:

johnjamesjacoby

Message:

Normalize metabox UI across Forums, Topics, & Replies. Hat-tip netweb. Fixes #2463.

Location:

trunk/src/includes

Files:

• admin/admin.php (modified) (1 diff)
• admin/js/replies.js (modified) (1 diff)
• admin/metaboxes.php (modified) (4 diffs)
• replies/functions.php (modified) (1 diff)
• replies/template.php (modified) (1 diff)

trunk/src/includes/admin/admin.php

@@ -568 +568 @@
                     // Topics admin
                     if ( bbp_get_topic_post_type() === get_current_screen()->post_type ) {
-                        wp_enqueue_script( 'bbp-admin-common-js', $this->js_url . 'topics' . $suffix . '.js', array( 'jquery' ), $version );
+                        wp_enqueue_script( 'bbp-admin-topics-js', $this->js_url . 'topics' . $suffix . '.js', array( 'jquery' ), $version );
 
                     // Replies admin
                     } elseif ( bbp_get_reply_post_type() === get_current_screen()->post_type ) {
-                        wp_enqueue_script( 'bbp-admin-common-js', $this->js_url . 'replies' . $suffix . '.js', array( 'jquery' ), $version );
+                        wp_enqueue_script( 'bbp-admin-replies-js', $this->js_url . 'replies' . $suffix . '.js', array( 'jquery' ), $version );
                     }
 

trunk/src/includes/admin/js/replies.js

@@ -1 +1 @@
 jQuery( document ).ready(function() {
+
+    jQuery( '#misc-publishing-actions' ).find( '.misc-pub-section' ).first().remove();
+    jQuery( '#save-action' ).remove();
 
     var bbp_topic_id = jQuery( '#bbp_topic_id' );

trunk/src/includes/admin/metaboxes.php

@@ -422 +422 @@
     ?>
 
+    <hr />
+
     <p>
         <strong class="label"><?php esc_html_e( 'Forum:', 'bbpress' ); ?></strong>
@@ -471 +473 @@
     $reply_topic_id = bbp_get_reply_topic_id( $post_id );
     $reply_forum_id = bbp_get_reply_forum_id( $post_id );
+
+
+    /** Status ****************************************************************/
+
+    ?>
+
+    <p>
+        <strong class="label"><?php esc_html_e( 'Status:', 'bbpress' ); ?></strong>
+        <label class="screen-reader-text" for="post_status"><?php esc_html_e( 'Select what status to give the reply.', 'bbpress' ); ?></label>
+        <?php bbp_form_reply_status_dropdown( array( 'select_id' => 'post_status', 'reply_id' => $post_id ) ); ?>
+    </p>
+
+    <hr />
+
+    <?php
+
+    /** Forum *****************************************************************/
 
     // Allow individual manipulation of reply forum
@@ -497 +516 @@
         </p>
 
-    <?php endif; ?>
+    <?php endif;
+
+    /** Topic *****************************************************************/
+
+    ?>
 
     <p>
@@ -504 +527 @@
         <input name="parent_id" id="bbp_topic_id" type="text" value="<?php echo esc_attr( $reply_topic_id ); ?>" />
     </p>
+
+    <?php
+
+    /** Reply To **************************************************************/
+
+    ?>
 
     <p>

trunk/src/includes/replies/functions.php

@@ -1665 +1665 @@
 }
 
+/** Helpers *******************************************************************/
+
+/**
+ * Return an associative array of available reply statuses
+ *
+ * @since bbPress (r5399)
+ *
+ * @return array
+ */
+function bbp_get_reply_statuses() {
+    return apply_filters( 'bbp_get_reply_statuses', array(
+        bbp_get_public_status_id()  => _x( 'Publish', 'Publish the reply',     'bbpress' ),
+        bbp_get_spam_status_id()    => _x( 'Spam',    'Spam the reply',        'bbpress' ),
+        bbp_get_trash_status_id()   => _x( 'Trash',   'Trash the reply',       'bbpress' ),
+        bbp_get_pending_status_id() => _x( 'Pending', 'Mark reply as pending', 'bbpress' ),
+    ) );
+}
+
 /** Reply Actions *************************************************************/
 

trunk/src/includes/replies/template.php

@@ -2623 +2623 @@
         return apply_filters( 'bbp_get_form_reply_edit_reason', esc_attr( $reply_edit_reason ) );
     }
+
+/**
+ * Output value topic status dropdown
+ *
+ * @since bbPress (r5399)
+ *
+ * @param $args This function supports these arguments:
+ *  - select_id: Select id. Defaults to bbp_reply_status
+ *  - tab: Tabindex
+ *  - reply_id: Reply id
+ *  - selected: Override the selected option
+ */
+function bbp_form_reply_status_dropdown( $args = '' ) {
+    echo bbp_get_form_reply_status_dropdown( $args );
+}
+    /**
+     * Returns topic status downdown
+     *
+     * This dropdown is only intended to be seen by users with the 'moderate'
+     * capability. Because of this, no additional capablitiy checks are performed
+     * within this function to check available topic statuses.
+     *
+     * @since bbPress (r5399)
+     *
+     * @param $args This function supports these arguments:
+     *  - select_id: Select id. Defaults to bbp_reply_status
+     *  - tab: Tabindex
+     *  - topic_id: Reply id
+     *  - selected: Override the selected option
+     */
+    function bbp_get_form_reply_status_dropdown( $args = '' ) {
+
+        // Parse arguments against default values
+        $r = bbp_parse_args( $args, array(
+            'select_id' => 'bbp_reply_status',
+            'tab'       => bbp_get_tab_index(),
+            'reply_id'  => 0,
+            'selected'  => false
+        ), 'reply_status_dropdown' );
+
+        // No specific selected value passed
+        if ( empty( $r['selected'] ) ) {
+
+            // Post value is passed
+            if ( bbp_is_post_request() && isset( $_POST[ $r['select_id'] ] ) ) {
+                $r['selected'] = $_POST[ $r['select_id'] ];
+
+            // No Post value was passed
+            } else {
+
+                // Edit topic
+                if ( bbp_is_reply_edit() ) {
+                    $r['reply_id'] = bbp_get_reply_id( $r['reply_id'] );
+                    $r['selected'] = bbp_get_reply_status( $r['reply_id'] );
+
+                // New topic
+                } else {
+                    $r['selected'] = bbp_get_public_status_id();
+                }
+            }
+        }
+
+        // Used variables
+        $tab = ! empty( $r['tab'] ) ? ' tabindex="' . (int) $r['tab'] . '"' : '';
+
+        // Start an output buffer, we'll finish it after the select loop
+        ob_start(); ?>
+
+        <select name="<?php echo esc_attr( $r['select_id'] ) ?>" id="<?php echo esc_attr( $r['select_id'] ); ?>_select"<?php echo $tab; ?>>
+
+            <?php foreach ( bbp_get_reply_statuses( $r['topic_id'] ) as $key => $label ) : ?>
+
+                <option value="<?php echo esc_attr( $key ); ?>"<?php selected( $key, $r['selected'] ); ?>><?php echo esc_html( $label ); ?></option>
+
+            <?php endforeach; ?>
+
+        </select>
+
+        <?php
+
+        // Return the results
+        return apply_filters( 'bbp_get_form_reply_status_dropdown', ob_get_clean(), $r );
+    }