JsonRest add sortOptions member

[Herman Vierendeels]

Would it be possible to add sortOptions: to JsonRest?.

e.g. for store-queries that should always have sort-parameter

//sortOptions: Array e.g
// [{attribute:"tmstmp",descending:true}]

query: function(query, options){
options = options || {};
....

if(! options.sort){
 if(this.sortOptions){
  options.sort=this.sortOptions;
 }
}
                
if(options && options.sort){
....

}

by the way, why is there if(options && options.sort)

options is ensured to exist by initial statement:

options = options || {};

Or am i totally wrong?

[ben hockey]

this sounds like something you could do with a subclass of JsonRest?

declare(JsonRest, {
  query: function (query, options) {
    options = options || {};
    options.sort = options.sort || this.sortOptions;
    return this.inherited(arguments, [query, options]);
  }
});

given that, maybe it's not right to put it into JsonRest? itself?

[Herman Vierendeels]

yes indeed this could be done by subclassing.

but i think that , being able to set sortOptions for all queries, is a rather general requirement.

[dylan]

Will consider for dstore ( ​http://dstorejs.io/ ). A pull request is welcome for dojo/store.