fix(inboxQuery): port _createRequest from Query to prevent URI too long

Author: leeyeh

src/query.js

@@ -250,7 +250,7 @@ module.exports = function(AV) {
       return new this.objectClass();
     },
     _createRequest(params = this.toJSON(), options) {
-      if (JSON.stringify(params).length > 2000) {
+      if (encodeURIComponent(JSON.stringify(params)).length > 2000) {
         const body = {
           requests: [{
             method: 'GET',

src/status.js

@@ -313,9 +313,27 @@ module.exports = function(AV) {
      _newObject: function(){
       return new AV.Status();
     },
-    _createRequest: function(params, options){
-      return AVRequest('subscribe/statuses', null, null, 'GET',
-                                   params || this.toJSON(), options);
+    _createRequest: function (params = this.toJSON(), options) {
+      if (encodeURIComponent(JSON.stringify(params)).length > 2000) {
+        const body = {
+          requests: [{
+            method: 'GET',
+            path: '/1.1/subscribe/statuses',
+            params,
+          }],
+        };
+        return AVRequest('batch', null, null, 'POST', body, options)
+          .then(response => {
+            const result = response[0];
+            if (result.success) {
+              return result.success;
+            }
+            const error = new Error(result.error.error || 'Unknown batch error');
+            error.code = result.error.code;
+            throw error;
+          });
+      }
+      return AVRequest('subscribe/statuses', null, null, 'GET', params, options);
     },
 
 

test/status.js

@@ -60,6 +60,11 @@ describe("AV.Status",function(){
           expect(response.unread).to.be.eql(0);
         });
     });
+    it('should not cause URI too long', () => { 
+      return AV.Status.inboxQuery(user) 
+        .containsAll('arr', new Array(50).fill(AV.Object.createWithoutData('Todo', '5850f138128fe1006978f766'))) 
+        .find(); 
+    }); 
   });
 
   describe("Status guide test.", function(){