JsonRest should nor use a plus sign when sorting ascending.

[alaska]

I am using a DataGrid linked to a JsonRest store pointing at a Sinatra backend. I set "sortParam" to "sort" when I initialize the store, which should be sending urls like the following when sorting:

/baseURL/?sort=+field
/baseURL/?sort=-field

The problem is that Sinatra (like a lot of other frameworks, I imagine) is helpfully replacing the "+" with a space, given that that is what it is supposed to signify in URLs, so in the backend I'm getting params like

{"sort" =>" field"} and {"sort" =>"-field"}

This behavior should be changed or at least user-customizable. I recommend something like "ASC:field" "DSC:field".

[alaska]

'DESC:' would probably be better than 'DSC:', actually, as you can just parse it out and drop it directly into the SQL (provided you're smart enough to use a regex like the following that won't allow just anything in):

/(ASC|DESC):(\w+)/

Patch:

--- dojo/store/JsonRest.js.uncompressed.js      2012-02-16 15:01:29.000000000 -0
500
+++ new/store/JsonRest.js.uncompressed.js       2012-03-20 21:40:00.985753363 -0
400
@@ -128,13 +128,13 @@
                if(query && typeof query == "object"){
                        query = xhr.objectToQuery(query);
                        query = query ? "?" + query: "";
-               }
-               if(options && options.sort){
+                }
+                if(options && options.sort){
                        var sortParam = this.sortParam;
                        query += (query ? "&" : "?") + (sortParam ? sortParam + 
'=' : "sort(");
                        for(var i = 0; i<options.sort.length; i++){
                                var sort = options.sort[i];
-                               query += (i > 0 ? "," : "") + (sort.descending ?
 '-' : '+') + encodeURIComponent(sort.attribute);
+                               query += (i > 0 ? "," : "") + (sort.descending ?
 'ASC:' : 'DESC:') + encodeURIComponent(sort.attribute);
                        }
                        if(!sortParam){
                                query += ")";

I suppose you could also do something like

query += (i > 0 ? "," : "") + encodeURIComponent((sort.descending ? '-' : '+') + sort.attribute);

}}}

[b0dh1]

I believe it's the same problem as in #14537 which is closed now. Actually according to RFC ​http://www.ietf.org/rfc/rfc2396.txt (see 3.4. Query Component):

Within a query component, the characters ";", "/", "?", ":", "@", "&", "=", "+", ",", and "$" are reserved.

Which means that using an unencoded plus in a query is a violation of this specification (as well as a suggestion to treat space as a plus on a server-side). Therefore if you are using "+" in a query it should be encoded. Perhaps there is a way in Dojo to make it configurable which strings JsonRest uses as DESC and ASC query attributes?

[Kris Zyp]

In [30873]:

Allow for alternate ascending and descending prefixes, fixes #15047 !strict

[cmketchu]

Allowing for alternate prefixes is nice, but leaving the default sort indicators as + and - will never work for the same reasons the bug was opened. I feel like the correct fix here is to encode the prefixes so that we are not in violation the RFC spec and can continue to use the defaults.

[bill]

Yeah, sounds like you need the encodeURIComponent() call to process the prefixes too, like:

query += (i > 0 ? "," : "") +
encodeURIComponent((sort.descending ? '-' : '+') + sort.attribute)

or if you still want the customization from [30873] then:

query += (i > 0 ? "," : "") + encodeURIComponent(
(sort.descending ? this.descendingPrefix : this.ascendingPrefix) +
sort.attribute); 

Reopening so this gets considered.

[freddefisk]

Related ticket: #16876

[bill]

Bumping this ticket since we are past the deadline for the 1.9RC. The fix can be put into 1.9.1 too, if desired.

[Kris Zyp]

But this isn't a backwards compatible fix is it? Also, I am probably reading this wrong, but I am curious how this violates the RFC spec. According to the grammar in RFC 2986, it looks like the query string is supposed to consist of pchar's which include sub-delims, which include '+'.