[jQuery] Re: Bug on line 184

2009-03-24 Thread ddorothy 

I forgot to mention that this is the autocomplete found here:
http://bassistance.de/jquery-plugins/jquery-plugin-autocomplete/

On Mar 23, 4:18 pm, ddorothy didoro...@gmail.com wrote:
 We discovered that when the auto complete values contain commas and
 you are not using the multiple input mode, the result would not show
 up in the input box (actually it would but it would be removed before
 you could tell if you were not stepping through the code).

 We discovered that on line 184 there is the following

 $.each(trimWords($input.val()), function(i, value) {
   request(value, findValueCallback, findValueCallback);

 });

 However, trimWords in any other case is preceded by an if statement
 checking options.multiple.

 We made the following change to correct the issue:

 $.each(options.multiple ? trimWords($input.val()) : [$input.val()],
 function(i, value) {
   request(value, findValueCallback, findValueCallback);

 });

 Thanks.

[jQuery] [autocomplete] Bug on line 184

2009-03-23 Thread ddorothy 

We discovered that when the auto complete values contain commas and
you are not using the multiple input mode, the result would not show
up in the input box (actually it would but it would be removed before
you could tell if you were not stepping through the code).

We discovered that on line 184 there is the following

$.each(trimWords($input.val()), function(i, value) {
  request(value, findValueCallback, findValueCallback);
});

However, trimWords in any other case is preceded by an if statement
checking options.multiple.

We made the following change to correct the issue:

$.each(options.multiple ? trimWords($input.val()) : [$input.val()],
function(i, value) {
  request(value, findValueCallback, findValueCallback);
});

Thanks.