Bug Tracker

Ticket #2726 (closed enhancement: wontfix)

Opened 8 months ago

Last modified 7 months ago

AJAX functions don't pass JSON with arrays properly

Reported by: lukebrookhart Assigned to: flesler
Type: enhancement Priority: minor
Milestone: 1.2.4 Component: ajax
Version: 1.2.3 Keywords: JSON arrays
Cc: grabanski@gmail.com; jeresig@gmail.com Needs: Patch

Description

With all the AJAX functions (ie. .load, .post .get, etc) the JSON data that is sent to the server does not pass the array data properly. Due to the way arrays are passed to web servers, the names of the array variables need to be concatenated with "[]" to signal to the server that there is more than one value.

Example:

$('#property_search_results').load('search.php', {towns:['Apalachin','Owego','Binghamton']});

Web servers will first define the "town" variable with "Apalachin". It will then immediately overwrite "Apalachin" with "Owego", and then overwrite "Owego" with "Binghamton".

If we change the variable name to towns[] within the AJAX functions, web servers will know that this is an array of values and all will be well. This is same way that web browsers pass an array of values for one variable in forms and query strings.

Attachments

param.diff (1.7 kB) - added by flesler 7 months ago.
This is the change, but I think I won't be able to apply it as many people/servers don't expect this to happen.

Change History

Changed 7 months ago by flesler

  • owner set to flesler
  • priority changed from major to minor
  • status changed from new to assigned
  • component changed from core to ajax
  • type changed from bug to enhancement

Will look into it.

Changed 7 months ago by flesler

This is the change, but I think I won't be able to apply it as many people/servers don't expect this to happen.

Changed 7 months ago by flesler

  • status changed from assigned to closed
  • resolution set to wontfix

According to this, the index/key shouldn't be specified.

So I'll have to close this, you can apply the diff to your file if you want, or you can just use:

{
 'towns[]':['Apalachin','Owego','Binghamton']
}

That should work as expected.

Note: See TracTickets for help on using tickets.