I opened http://dev.jquery.com/ticket/4920

Matt

On Sat, Jul 18, 2009 at 3:46 PM, John Resig <jere...@gmail.com> wrote:

> Ah - good call, forgot about that. Would you care to file a ticket on this?
> http://dev.jquery.com/newticket
>
> It seems like a relatively painless tweak.
>
> --John
>
>
>
> On Sat, Jul 18, 2009 at 3:16 PM, Matthew M. Boedicker <
> matth...@boedicker.org> wrote:
>
>> It was putting an extra %3F back in the url because of $1 being used later
>> on. I tried making the new group jsre = /=(?:\?|%3F)(&|$)/g and it worked.
>>
>> Matt
>>
>>
>> On Sat, Jul 18, 2009 at 12:26 PM, John Resig <jere...@gmail.com> wrote:
>>
>>> I understand your point but I think it might work better if the jsre
>>> regexp was tweaked, instead:
>>>
>>>     jsre = /=(\?|%3F)(&|$)/g
>>>
>>> Does making the above change work for you?
>>>
>>> --John
>>>
>>>
>>>
>>> On Fri, Jul 17, 2009 at 8:41 PM, Matthew M. Boedicker <
>>> matth...@boedicker.org> wrote:
>>>
>>>>
>>>> I like passing query string params to $.getJSON as an array using the
>>>> optional data argument because it's more readable. However when using
>>>> a jsonp callback parameter it gets encoded to callback=%3F, causing it
>>>> to fail. Any way to accomplish this without modifying jquery?
>>>>
>>>> This patch to jquery made it work:
>>>>
>>>> --- jquery-1.3.2.js.orig        2009-02-19 18:02:00.000000000 -0500
>>>> +++ jquery-1.3.2.js     2009-07-17 20:21:30.000000000 -0400
>>>> @@ -3403,8 +3403,11 @@
>>>>                        type = s.type.toUpperCase();
>>>>
>>>>                // convert data if not already a string
>>>> -               if ( s.data && s.processData && typeof s.data !==
>>>> "string" )
>>>> +               if ( s.data && s.processData && typeof s.data !==
>>>> "string" ) {
>>>>                        s.data = jQuery.param(s.data);
>>>> +                       // reverse encoding of jsonp callback param
>>>> +                       s.data = s.data.replace(/callback=%3F$/,
>>>> "callback=?");
>>>> +               }
>>>>
>>>>                // Handle JSONP Parameter Callbacks
>>>>                if ( s.dataType == "jsonp" ) {
>>>>
>>>>
>>>>
>>>
>>>
>>>
>>
>>
>>
>
> >
>

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"jQuery Development" group.
To post to this group, send email to jquery-dev@googlegroups.com
To unsubscribe from this group, send email to 
jquery-dev+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/jquery-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to