#980 enhancement
dcpedit

Sort query parameters in toQueryString()

Reported by dcpedit | January 14th, 2010 @ 11:33 PM

Sorting the query string will help with sites using a CDN since there's a better chance that the url used in the ajax request will hit a cached copy. Especially if the serverside generated urls follow the same convention.

Change from:

return this.inject([], function(results, pair) {
...
}).join('&');

To

return this.inject([], function(results, pair) {
...
}).sort().join('&');

Comments and changes to this ticket

  • dcpedit

    dcpedit January 15th, 2010 @ 03:30 AM

    • Tag changed from toquerystring sort parameters ajax request to ajax, parameters, request, sort, toquerystring
  • Tobie Langel

    Tobie Langel January 20th, 2010 @ 02:02 PM

    • Tag changed from ajax, parameters, request, sort, toquerystring to section:ajax
    • State changed from “new” to “enhancement”

    Interesting suggestion.

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 ยป

The Prototype JavaScript library.

Shared Ticket Bins

Pages