This project is archived and is in readonly mode.

#65 ✓invalid
Thomas Aylott

toQueryString types of data

Reported by Thomas Aylott | May 28th, 2008 @ 10:06 AM

Originally posted on Trac by moro

Original Trac Ticket

Description

allow send array, hash and object, but not only object

Hast.toQueryString

replace

case 'object': result = Hash.toQueryString(value, key); break;

with

case 'hash': case 'object': result = Hash.toQueryString(value, key); break;

Request.send

replace

case 'object': result = Hash.toQueryString(value, key); break;

with

case 'object':case 'hash':case 'array': result = Hash.toQueryString(value, key); break;

Trac Attachments

Trac Comments

Changed 1 month ago by moro


        
in Request.send must be: 

replace 

case 'object': case 'hash': data = Hash.toQueryString(data);

with 

case 'object': case 'hash': case 'array': data = Hash.toQueryString(data);

      

Comments and changes to this ticket

  • adamnfish

    adamnfish August 4th, 2008 @ 01:54 AM

    • Tag set to core, defect, swiff

    I think this one's non-trivial because it's causing problems for Swiff. I'm porting / extending the jQuery control script for the popular flash player 'flowplayer'. Flowplayer accepts configuration options as a JSON string, rather than the standard querystring but because Swiff calls Hash.toQueryString on 'vars' the config string gets ruined.

    Testing the variable in Swiff prior to calling toQueryString would do the job and indeed should be added as well (should I put a ticket in for this?) because it shouldn't be calling Hash methods on a string at all, but certainly it doesn't make sense as it is:

    Hash.toQueryString('test');
    // returns
    "0=t&1=e&2=s&3=t&4=%26"
    
  • fakedarren

    fakedarren February 8th, 2010 @ 02:58 PM

    • State changed from “new” to “invalid”
    • Tag changed from core, defect, swiff to defect, swiff
    • Assigned user cleared.
    • Milestone cleared.

    As part of the Lighthouse cleansing, given it's age and the last comment I am closing this.....reopen if thy doth protest

Create your profile

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

Shared Ticket Bins

Tags

Referenced by

Pages