Ticket #1498 (closed defect: fixed)

Opened 1 year ago

Last modified 3 months ago

Multiple input fields with the same name do not yield multiple values in ajax.js RemoteRequest

Reported by: jype Assigned to: chrisz
Priority: normal Milestone: 1.1
Component: TurboGears Version: 1.0.3
Severity: normal Keywords: multipleselectfield,checkboxlist,ajax,discarded values
Cc:

Description

When using MultipleSelectField? or CheckBoxList? in a RemoteForm, ajax.js invalidly transforms form contents from two lists into a dictionary. Thus, if multiple fields have the same name, only the last value is stored in the dictionary, others are clobbered in turn.

The solution is not to create a dictionary - MochiKits? queryString function already properly handles a list of two lists.

I have created a patch for ajax.js that fixes the problem for me for MochiKit? 1.3.1 and 1.4(svn) simultaneously.

Attachments

ajax.js.diff (1.2 kB) - added by jype on 09/12/07 21:30:24.
Patch against 1.0.4b2 ajax.js

Change History

09/12/07 21:30:24 changed by jype

  • attachment ajax.js.diff added.

Patch against 1.0.4b2 ajax.js

09/12/07 21:34:40 changed by jype

I also checked the trunk version - it doesn't use MochiKit? anymore, but still fails to address this problem. While lack of time doesn't permit me to fix the trunk right now, it should be fairly straightforward, just check for checkboxes and select multiples and handle them appropriately.

Before the ticket is fixed I'd really like some peer review - while it works for me, I haven't tested it extensively.

08/27/08 11:41:32 changed by Chris Arndt

  • owner changed from anonymous to Chris Arndt.
  • milestone set to 1.1.

08/28/08 13:26:57 changed by droggisch

  • owner changed from Chris Arndt to chriz.

I fixed this in r5290. However, as ajax.js relies on MochiKit?, and formContents below MK1.4 has a bug that also exposes that behavior, one needs to use MK1.4. Which shouldn't be to much of a problem, however it should be mentioned.

08/28/08 14:04:52 changed by Chris Arndt

  • owner changed from chriz to chrisz.

08/28/08 14:06:41 changed by Chris Arndt

We should set tg_mochikit_version = "1.4" in app.cfg in the quickstart templates then.

09/01/08 03:39:27 changed by Chris Arndt

  • status changed from new to closed.
  • resolution set to fixed.

Apps quickstarted with TG >= 1.1 now specify MochiKit 1.4 by default (r5329 & r5330).