#23 new
Phil Mayers

Ajax.Autocompleter "parameters" must be a string

Reported by Phil Mayers | June 11th, 2008 @ 04:12 PM

The Ajax.Autocompleter docs state:

parameters... If you need to send any additional parameters through your search form, add them here, in the usual {field: 'value',another: 'value'} or 'field=value&another=value' manner.

...however the source code says (~line 363 of controls.js v1.8.1):

Ajax.Autocompleter = Class.create(Autocompleter.Base, {

...

this.options.defaultParams = this.options.parameters || null;

...

if(this.options.defaultParams)

this.options.parameters += '&' + this.options.defaultParams;

...and supplying a hash sets the params to:

?name=foo&%5Bobject%20Object%5D

i.e. the encoded string: "[object Object]"

Comments and changes to this ticket

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

script.aculo.us is an open-source JavaScript framework for visual effects and interface behaviours.
<br/><b>Source available from github</b>
The Git repository resides at:
<a href="http://github.com/madrobby/scriptaculous">http://github.com/madrobby/scriptaculous</a>
<br/>Check out the current development trunk with:
<code>git clone git://github.com/madrobby/scriptaculous.git</code>
<br/>As <b>script.aculo.us 1.xx is feature-frozen</b>, this development trunk is for <b>bugfixes only</b>.
<br/>New development should happen only for
<b>script.aculo.us 2</b>.
<br/><b>Creating a bug report</b>
When creating a bug report, be sure to include as much relevant information as possible. Post a an example that shows off the problem. Preferably, <b>alter the unit tests</b> and show through either changed or added tests how the expected behavior is not occuring.

People watching this ticket

Attachments

Pages