Autocomplete: try to type a slash

Most likely this is the Apache %2F bug which is worked around in core. A similar fix such as drupal_urlencode() would be needed for autocomplete.js.

Cannot reproduce in CVS HEAD. Can somebody confirm this is fixed? If yes, this can be closed.

I still experience this bug with 4.7.2 and I consider it to be rather important for some of the data I'm using.

Found this problem using ecommerce 4.7 file.module in 4.7.2
http://urbits.com/_media/some_error.gif

Marked "won't fix" by killes in this thread.
http://drupal.org/node/7930

So I'll do the same here.

Sorry, a "won't fix" from 2004 doesn't address how critical this is to this years autocomplete.

Here's a patch (much simpler than the other proposed workaround) that just causes the AJAX requests to be made with ?q= notation rather than as a clean URL.
This is technically more correct than relying on mod_rewrite to do everything for us anyway.

Where my errors were:

"GET /drupal_4_7/filebrowser/autocomplete_directory/imported%2F HTTP/1.1" 404 

(404) I now can use:

"GET /drupal_4_7/?q=filebrowser/autocomplete_directory/imported%2F HTTP/1.1" 200

and things work again.

The change looks like this in the code of theme_textfield()

    $extra =  '<input class="autocomplete" type="hidden" id="'. $element['#id'] .'-autocomplete" value="'. check_url(url('','q='.$element['#autocomplete_path'], NULL, TRUE)) .'" disabled="disabled" />';

Attached patch is against 4.7.2. It appears problem persists in todays HEAD/CVS.

.dan.

moving to cvs to increase exposure. The solution seems to be ok to me, the patch needs some coding style love.

Fair call, I guess, but meaning what?
Drop the explanatory comments, and I can't see the stylistic difference between the old and new syntax & layout. It only juggles the parameters to a function. Yeh it's an ugly big line, would it be clearer in two?
I'd be happy to accede, but I'm not sure how...
.dan.

Using a clean URL for autocomplete is not a very clean fix. We should just mirror all the code in drupal_urlencode() to JS, work around all the Apache / mod_rewrite bugs (/, &, #) that way.

Better patch attached.

that is 'using a non-clean URL' of course...

I'm getting a
warning: Missing argument 1 for user_autocomplete() in /home/drumm/drupal/drupal/C-HEAD/modules/user/user.module on line 2365.

Well, slashes don't actually work because they are indistinguishable from menu path slashes, and user_autocomplete() fails. I fixed that bug and merged this into the jQuery patch, as it changes all that code around anyway.