Changeset 8214

Timestamp:

06/30/2008 12:04:22 AM (18 years ago)

Author:

markjaquith

Message:

Enable tag auto-suggest for multiple tags at once (without having to hit enter and send each one "down below"). fixes #5580

Location:

trunk

Files:

• wp-admin/admin-ajax.php (modified) (1 diff)
• wp-admin/js/post.js (modified) (1 diff)
• wp-includes/js/jquery/suggest.js (modified) (4 diffs)
• wp-includes/script-loader.php (modified) (2 diffs)

trunk/wp-admin/admin-ajax.php

@@ -14 +14 @@
         $s = $_GET['q']; // is this slashed already?
 
-        if ( strstr( $s, ',' ) )
-                die; // it's a multiple tag insert, we won't find anything
-        $results = $wpdb->get_col( $wpdb->prepare("SELECT name FROM $wpdb->terms WHERE name LIKE (%s)", '%' . $s . '%') );
+        if ( strstr( $s, ',' ) ) { 
+                $s = explode( ',', $s ); 
+                $s = $s[count( $s ) - 1]; 
+        }
+        $s = trim( $s );
+        if ( strlen( $s ) < 2 )
+         die; // require 2 chars for matching
+        $results = $wpdb->get_col( "SELECT name FROM $wpdb->terms WHERE name LIKE ('%". $s . "%')" );
         echo join( $results, "\n" );
         die;

trunk/wp-admin/js/post.js

@@ -99 +99 @@
 
         // auto-suggest stuff
-        jQuery('#newtag').suggest( 'admin-ajax.php?action=ajax-tag-search', { delay: 500, minchars: 2 } );
+        jQuery('#newtag').suggest( 'admin-ajax.php?action=ajax-tag-search', { delay: 500, minchars: 2, multiple: true, multipleSep: ", " } );
         jQuery('#newtag').keypress( tag_press_key );
 

trunk/wp-includes/js/jquery/suggest.js

@@ -1 +1 @@
 /*
- *      jquery.suggest 1.1 - 2007-08-06
+ *      jquery.suggest 1.1b - 2007-08-06
+ * Patched by Mark Jaquith with Alexander Dick's "multiple items" patch to allow for auto-suggesting of more than one tag before submitting
+ * See: http://www.vulgarisoip.com/2007/06/29/jquerysuggest-an-alternative-jquery-based-autocomplete-library/#comment-7228
  *      
  *      Uses code and techniques from following libraries:
@@ -114 +116 @@
                         var q = $.trim($input.val());
 
+                        if ( options.multiple ) {
+                                var multipleSepPos = q.lastIndexOf(options.multipleSep);
+                                if ( multipleSepPos != -1 ) {
+                                        q = q.substr(multipleSepPos + options.multipleSep.length);
+                                }
+                        }
                         if (q.length >= options.minchars) {
                                 
@@ -246 +254 @@
                 
                         if ($currentResult) {
-                                $input.val($currentResult.text());
+                                if ( options.multiple ) {
+                                        if ( $input.val().indexOf(options.multipleSep) != -1 ) {
+                                                $currentVal = $input.val().substr( 0, ( $input.val().lastIndexOf(options.multipleSep) + options.multipleSep.length ) );
+                                        } else {
+                                                $currentVal = "";
+                                        }
+                                        $input.val( $currentVal + $currentResult.text() + options.multipleSep);
+                                        $input.focus();
+                                } else {
+                                        $input.val($currentResult.text());
+                                }
                                 $results.hide();
                                 
@@ -292 +310 @@
         
                 options = options || {};
+                options.multiple = options.multiple || false;
+                options.multipleSep = options.multipleSep || ", ";
                 options.source = source;
                 options.delay = options.delay || 100;

trunk/wp-includes/script-loader.php

@@ -71 +71 @@
         $scripts->add( 'jquery-color', '/wp-includes/js/jquery/jquery.color.js', array('jquery'), '2.0-4561');
         $scripts->add( 'interface', '/wp-includes/js/jquery/interface.js', array('jquery'), '1.2' );
-        $scripts->add( 'suggest', '/wp-includes/js/jquery/suggest.js', array('jquery'), '1.1');
+        $scripts->add( 'suggest', '/wp-includes/js/jquery/suggest.js', array('jquery'), '1.1b');
         $scripts->add( 'schedule', '/wp-includes/js/jquery/jquery.schedule.js', array('jquery'), '20');
         $scripts->add( 'thickbox', '/wp-includes/js/thickbox/thickbox.js', array('jquery'), '3.1-20080430');
@@ -139 +139 @@
                         'cancel' => __('Cancel'),
                 ) );
-                $scripts->add( 'post', '/wp-admin/js/post.js', array('suggest', 'jquery-ui-tabs', 'wp-lists', 'postbox', 'slug'), '20080623' );
+                $scripts->add( 'post', '/wp-admin/js/post.js', array('suggest', 'jquery-ui-tabs', 'wp-lists', 'postbox', 'slug'), '20080629' );
                 $scripts->localize( 'post', 'postL10n', array(
                         'tagsUsed' =>  __('Tags used on this post:'),