Fixes issues with rest.each

Author: flovilmart

spec/rest.spec.js

@@ -626,6 +626,31 @@ describe('rest update', () => {
   });
 });
 
+describe('rest each', () => {
+  it('should iterate through all results', () => {
+    const className = 'Yolo';
+    const config = Config.get('test');
+    const master = auth.master(config);
+    let promise = Promise.resolve();
+    let done = 0;
+    while (done != 10) {
+      done++;
+      promise = promise.then(() => {
+        return rest.create(config, auth, className, {});
+      });
+    }
+    return promise.then(() => {
+      let seen = 0;
+      // use limit 1 to get them 1 by one
+      return rest.each(config, master, className, {}, {limit: 1}, (obj) => {
+        seen++;
+      }).then(() => {
+        expect(seen).toBe(10);
+      })
+    }).then(done, done.fail);
+  });
+})
+
 describe('read-only masterKey', () => {
   it('properly throws on rest.create, rest.update and rest.del', () => {
     const config = Config.get('test');

src/rest.js

@@ -37,6 +37,7 @@ function find(config, auth, className, restWhere, restOptions, clientSDK) {
 
 function each(config, auth, className, where, options, callback) {
   options = Object.assign({}, options);
+  options.order = 'objectId'; // force ordering on objectId
   where = Object.assign({}, where);
   let finished;
   return Parse.Promise._continueWhile(() => {
@@ -52,7 +53,7 @@ function each(config, auth, className, where, options, callback) {
       return callbacksDone.then(() => {
         if (results.length >= options.limit) {
           where['objectId'] = where['objectId'] || {};
-          where['objectId']['$gt'] = results[results.length - 1].id;
+          where['objectId']['$gt'] = results[results.length - 1].objectId;
         } else {
           finished = true;
         }