Merge pull request #285 from wangxiao/node

Support API server response status 410 and redirect location.

Author: leeyeh

.eslintrc.js

@@ -8,5 +8,6 @@ module.exports = {
     'max-len': 0,
     'no-console': [ 2, { allow: ['warn'] } ],
     'no-restricted-syntax': [ 0, 'ForInStatement' ],
+    'no-new': 0,
   }
 };

src/av.js

@@ -22,9 +22,7 @@ AV.Cache = require('./cache');
 AV._config = AV._config || {};
 
 require('./utils').init(AV);
-require('./request').init(AV);
 
-require('./error')(AV);
 require('./event')(AV);
 require('./geopoint')(AV);
 require('./acl')(AV);
@@ -40,3 +38,13 @@ require('./push')(AV);
 require('./status')(AV);
 require('./search')(AV);
 require('./insight')(AV);
+
+// TODO: deprecated AV.Error()
+const AVError = require('./error');
+/**
+ * @deprecated AV.Error() is deprecated, and will be removed in next release.
+ */
+AV.Error = (...args) => {
+  console.warn('AV.Error() is deprecated, and will be removed in next release.');
+  new AVError(...args);
+};

src/cache.js

@@ -1,11 +1,15 @@
-'use strict';
+/**
+ * 每位工程师都有保持代码优雅的义务
+ * Each engineer has a duty to keep the code elegant
+**/
+
 const storage = require('./localstorage');
 const AV = require('./av');
 
 const remove = exports.remove = storage.removeItemAsync.bind(storage);
 
-exports.get = (key) => {
-  return storage.getItemAsync(`${AV.applicationId}/${key}`)
+exports.get = (key) =>
+  storage.getItemAsync(`${AV.applicationId}/${key}`)
     .then(cache => {
       try {
         cache = JSON.parse(cache);
@@ -21,12 +25,9 @@ exports.get = (key) => {
       }
       return null;
     });
-};
 
 exports.set = (key, value, ttl) => {
-  const cache = {
-    value
-  };
+  const cache = { value };
   if (typeof ttl === 'number') {
     cache.expiredAt = Date.now() + ttl;
   }

src/cloudfunction.js

@@ -3,9 +3,8 @@
  * Each engineer has a duty to keep the code elegant
 **/
 
-'use strict';
-
-var _ = require('underscore');
+const _ = require('underscore');
+const AVRequest = require('./request').request;
 
 module.exports = function(AV) {
   /**
@@ -31,7 +30,7 @@ module.exports = function(AV) {
      * of the function.
      */
     run: function(name, data, options) {
-      var request = AV._request('functions', name, null, 'POST',
+      var request = AVRequest('functions', name, null, 'POST',
                                    AV._encode(data, null, true), options && options.sessionToken);
 
       return request.then(function(resp) {
@@ -53,7 +52,7 @@ module.exports = function(AV) {
           ._thenRunCallbacks(options);
       }
 
-      return AV._request('call', name, null, 'POST', AV._encodeObjectOrArray(data)).then(function(resp) {
+      return AVRequest('call', name, null, 'POST', AV._encodeObjectOrArray(data)).then(function(resp) {
         return AV._decode('', resp).result;
       })._thenRunCallbacks(options);
     },
@@ -70,7 +69,7 @@ module.exports = function(AV) {
      * @since 0.5.9
      */
     getServerDate: function(options) {
-      var request = AV._request("date", null, null, 'GET');
+      var request = AVRequest("date", null, null, 'GET');
 
       return request.then(function(resp) {
         return AV._decode(null, resp);
@@ -92,7 +91,7 @@ module.exports = function(AV) {
       if(!data.mobilePhoneNumber) {
         throw "Missing mobilePhoneNumber.";
       }
-      var request = AV._request("requestSmsCode", null, null, 'POST',
+      var request = AVRequest("requestSmsCode", null, null, 'POST',
                                     data);
       return request._thenRunCallbacks(options);
     },
@@ -116,7 +115,7 @@ module.exports = function(AV) {
          options = phone;
       }
 
-      var request = AV._request("verifySmsCode", code, null, 'POST',
+      var request = AVRequest("verifySmsCode", code, null, 'POST',
                                    params);
       return request._thenRunCallbacks(options);
     }