Add settings disableDefaultSort for find method

Author: HugoPoi

lib/mongodb.js

@@ -936,10 +936,22 @@ MongoDB.prototype.buildWhere = function(model, where) {
   return query;
 };
 
-MongoDB.prototype.buildSort = function(model, order) {
+MongoDB.prototype.buildSort = function(model, order, options) {
   var sort = {};
   var idName = this.idName(model);
-  if (!order) {
+
+  var modelClass = this._models[model];
+
+  var disableDefaultSort = this.settings.disableDefaultSort;
+  if (options && options.hasOwnProperty('disableDefaultSort')) {
+    disableDefaultSort = options.disableDefaultSort === true;
+  } else if (disableDefaultSort !== false && modelClass.settings.hasOwnProperty('disableDefaultSort')) {
+    disableDefaultSort = modelClass.settings.disableDefaultSort === true;
+  } else if (disableDefaultSort === true) {
+    disableDefaultSort = true;
+  }
+
+  if (!order && !disableDefaultSort) {
     var idNames = this.idNames(model);
     if (idNames && idNames.length) {
       order = idNames;
@@ -966,7 +978,7 @@ MongoDB.prototype.buildSort = function(model, order) {
         sort[key] = 1;
       }
     }
-  } else {
+  } else if (!disableDefaultSort) {
     // order by _id by default
     sort = {_id: 1};
   }
@@ -1224,7 +1236,7 @@ MongoDB.prototype.all = function all(model, filter, options, callback) {
 
     // don't apply sorting if dealing with a geo query
     if (!hasNearFilter(filter.where)) {
-      var order = self.buildSort(model, filter.order);
+      var order = self.buildSort(model, filter.order, options);
       cursor.sort(order);
     }
 

test/mongodb.test.js

@@ -2490,6 +2490,34 @@ describe('mongodb connector', function() {
     });
   });
 
+  it('find should not order by id if the order is not set for the query filter and settings.disableDefaultSort is true',
+    function(done) {
+      PostWithStringId.settings.disableDefaultSort = true;
+
+      PostWithStringId.create({ id: '2', title: 'c', content: 'CCC' }, function(err, post) {
+        PostWithStringId.create({ id: '1', title: 'd', content: 'DDD' }, function(err, post) {
+          PostWithStringId.find({}, function(err, posts) {
+            should.not.exist(err);
+            posts.length.should.be.equal(2);
+            posts[0].id.should.be.equal('2');
+
+            PostWithStringId.find({ limit: 1, offset: 0 }, function(err, posts) {
+              should.not.exist(err);
+              posts.length.should.be.equal(1);
+              posts[0].id.should.be.equal('2');
+
+              PostWithStringId.find({ limit: 1, offset: 1 }, function(err, posts) {
+                should.not.exist(err);
+                posts.length.should.be.equal(1);
+                posts[0].id.should.be.equal('1');
+                done();
+              });
+            });
+          });
+        });
+      });
+    });
+
   it('should report error on duplicate keys', function(done) {
     Post.create({title: 'd', content: 'DDD'}, function(err, post) {
       Post.create({id: post.id, title: 'd', content: 'DDD'}, function(