misoproject/dataset

ds.sort syntax is ugly

Closed this issue · 1 comments

currently to set and run a sort comparator you need to do

ds.sort( { comparator: function(a,b) {...} });

The signature could be changed to ds.sort( comparator, options ); to be a big nicer and remove the need for the extra object most of the time.

iros commented

Since you can set comparator on instantiation AND set ds.comparator = function(a,b), in this signature comparator is optional. As such, it goes into the optional arguments which can have silent:true. If we make it a param then it might require us passing null as first arg to set a silent:true.