Changeset 5015

Timestamp:

07/10/2013 06:54:54 AM (13 years ago)

Author:

johnjamesjacoby

Message:

Add user Suggest support for post author overrides for topics and replies. Props alex-ye. Fixes #2224.

Location:

trunk/includes/admin

Files:

• admin.php (modified) (3 diffs)
• metaboxes.php (modified) (1 diff)

trunk/includes/admin/admin.php

@@ -138 +138 @@
         add_action( 'wp_ajax_bbp_suggest_topic',        array( $this, 'suggest_topic' ) );
         add_action( 'wp_ajax_nopriv_bbp_suggest_topic', array( $this, 'suggest_topic' ) );
+
+        add_action( 'wp_ajax_bbp_suggest_user',         array( $this, 'suggest_user'  ) );
+        add_action( 'wp_ajax_nopriv_bbp_suggest_user',  array( $this, 'suggest_user'  ) );
 
         /** Filters ***********************************************************/
@@ -558 +561 @@
         $reply_class = sanitize_html_class( bbp_get_reply_post_type() );
 
-        if ( ( 'post' === get_current_screen()->base ) && ( bbp_get_reply_post_type() === get_current_screen()->post_type ) ) : ?>
-
-        <script type="text/javascript">
-            jQuery(document).ready(function() {
-
-                var bbp_topic_id = jQuery( '#bbp_topic_id' );
-
-                bbp_topic_id.suggest( ajaxurl + '?action=bbp_suggest_topic', {
-                    onSelect: function() {
-                        var value = this.value;
-                        bbp_topic_id.val( value.substr( 0, value.indexOf( ' ' ) ) );
-                    }
-                } );
-            });
-        </script>
-
-        <?php endif; ?>
+        // Post type checker (only topics and replies)
+        if ( 'post' === get_current_screen()->base ) :
+            switch( get_current_screen()->post_type ) :
+                case bbp_get_reply_post_type() :
+                case bbp_get_topic_post_type() : ?>
+                    <script type="text/javascript">
+                        jQuery(document).ready(function() {
+                            var bbp_author_id = jQuery( '#bbp_author_id' );
+                            bbp_author_id.suggest( ajaxurl + '?action=bbp_suggest_user', {
+                                onSelect: function() {
+                                    var value = this.value;
+                                    bbp_author_id.val( value.substr( 0, value.indexOf( ' ' ) ) );
+                                }
+                            } );
+                        });
+
+                    <?php if ( bbp_get_reply_post_type() == get_current_screen()->post_type ) : ?>
+                        jQuery(document).ready(function() {
+                            var bbp_topic_id = jQuery( '#bbp_topic_id' );
+                            bbp_topic_id.suggest( ajaxurl + '?action=bbp_suggest_topic', {
+                                onSelect: function() {
+                                    var value = this.value;
+                                    bbp_topic_id.val( value.substr( 0, value.indexOf( ' ' ) ) );
+                                }
+                            } );
+                        });
+                    <?php endif; ?>
+                    </script><?php
+
+                    break;
+            endswitch;
+        endif; ?>
 
         <style type="text/css" media="screen">
@@ -1321 +1339 @@
         if ( ! empty( $topics ) ) {
             foreach ( (array) $topics as $post ) {
-                printf( __( '%s - %s', 'bbpress' ), bbp_get_topic_id( $post->ID ), bbp_get_topic_title( $post->ID ) ) . "\n";
+                printf( esc_html__( '%s - %s', 'bbpress' ), bbp_get_topic_id( $post->ID ), bbp_get_topic_title( $post->ID ) . "\n" );
+            }
+        }
+        die();
+    }
+
+    /**
+     * Ajax action for facilitating the topic and reply author auto-suggest
+     *
+     * @since bbPress (r5014)
+     */
+    public function suggest_user() {
+
+        // Try to get some users
+        $users_query = new WP_User_Query( array(
+            'search'         => '*' . like_escape( $_REQUEST['q'] ) . '*',
+            'fields'         => array( 'ID', 'user_nicename' ),
+            'search_columns' => array( 'ID', 'user_nicename', 'user_email' ),
+            'orderby'        => 'ID'
+        ) );
+
+        // If we found some users, loop through and display them
+        if ( ! empty( $users_query->results ) ) {
+            foreach ( (array) $users_query->results as $user ) {
+                printf( esc_html__( '%s - %s', 'bbpress' ), bbp_get_user_id( $user->ID ), bbp_get_user_nicename( $user->ID, array( 'force' => $user->user_nicename ) ) . "\n" );
             }
         }

trunk/includes/admin/metaboxes.php

@@ -489 +489 @@
 
         <p>
-            <strong class="label"><?php esc_html_e( 'Name:', 'bbpress' ); ?></strong>
-            <label class="screen-reader-text" for="bbp_author_name"><?php esc_html_e( 'Name', 'bbpress' ); ?></label>
-            <input type="text" id="bbp_author_name" name="bbp_author_name" value="<?php echo esc_attr( get_the_author_meta( 'nicename', bbp_get_global_post_field( 'post_author' ) ) ); ?>" disabled="disabled" />
-        </p>
-
-        <p>
-            <strong class="label"><?php esc_html_e( 'Email:', 'bbpress' ); ?></strong>
-            <label class="screen-reader-text" for="bbp_author_email"><?php esc_html_e( 'Email', 'bbpress' ); ?></label>
-            <input type="text" id="bbp_author_email" name="bbp_author_email" value="<?php echo esc_attr( get_the_author_meta( 'email', bbp_get_global_post_field( 'post_author' ) ) ); ?>" disabled="disabled" />
+            <strong class="label"><?php esc_html_e( 'ID:', 'bbpress' ); ?></strong>
+            <label class="screen-reader-text" for="bbp_author_id"><?php esc_html_e( 'ID', 'bbpress' ); ?></label>
+            <input type="text" id="bbp_author_id" name="post_author_override" value="<?php echo esc_attr( bbp_get_global_post_field( 'post_author' ) ); ?>" />
         </p>