AV.applicationProduction -> AV._config.applicationProduction and deprcated AV._isNullOrUndefined (#288)

* AV.applicationProduction -> AV._config.applicationProduction

* Deprecated AV._isNullOrUndefined()

* Only when choosing a test environment, set header X-LC-Prod.

* Change AV._config.applicationProduction default value.

Author: wangxiao

src/object.js

@@ -6,6 +6,7 @@
 const _ = require('underscore');
 const AVError = require('./error');
 const AVRequest = require('./request').request;
+const utils = require('./utils');
 
 // AV.Object is analogous to the Java AVObject.
 // It also implements the same interface as a Backbone model.
@@ -312,7 +313,7 @@ module.exports = function(AV) {
       }
       var val = this.attributes[attr];
       var escaped;
-      if (AV._isNullOrUndefined(val)) {
+      if (utils.isNullOrUndefined(val)) {
         escaped = '';
       } else {
         escaped = _.escape(val.toString());
@@ -328,7 +329,7 @@ module.exports = function(AV) {
      * @return {Boolean}
      */
     has: function(attr) {
-      return !AV._isNullOrUndefined(this.attributes[attr]);
+      return !utils.isNullOrUndefined(this.attributes[attr]);
     },
 
     /**
@@ -579,7 +580,7 @@ module.exports = function(AV) {
      */
     set: function(key, value, options) {
       var attrs, attr;
-      if (_.isObject(key) || AV._isNullOrUndefined(key)) {
+      if (_.isObject(key) || utils.isNullOrUndefined(key)) {
         attrs = key;
         AV._objectEach(attrs, function(v, k) {
           attrs[k] = AV._decode(k, v);
@@ -859,7 +860,7 @@ module.exports = function(AV) {
      */
     save: function(arg1, arg2, arg3) {
       var i, attrs, current, options, saved;
-      if (_.isObject(arg1) || AV._isNullOrUndefined(arg1)) {
+      if (_.isObject(arg1) || utils.isNullOrUndefined(arg1)) {
         attrs = arg1;
         options = arg2;
       } else {

src/request.js

@@ -99,8 +99,8 @@ const setHeaders = (sessionToken) => {
   } else {
     headers['X-LC-Sign'] = sign(AV.applicationKey);
   }
-  if (!AV._isNullOrUndefined(AV.applicationProduction)) {
-    headers['X-LC-Prod'] = AV.applicationProduction;
+  if (!AV._config.applicationProduction === null) {
+    headers['X-LC-Prod'] = AV._config.applicationProduction;
   }
   if (!AV._config.isNode) {
     headers['X-LC-UA'] = `AV/${AV.version}`;

src/utils.js

@@ -6,6 +6,9 @@
 const _ = require('underscore');
 const request = require('./request');
 
+// Helper function to check null or undefined.
+const isNullOrUndefined = (x) => _.isNull(x) || _.isUndefined(x);
+
 const init = (AV) => {
   // 挂载一些配置
   const AVConfig = AV._config;
@@ -26,6 +29,10 @@ const init = (AV) => {
 
     // Internal config can modifie the UserAgent
     userAgent: null,
+
+    // set production environment or test environment
+    // 1: production environment, 0: test environment, null: default environment
+    applicationProduction: null,
   });
 
   /**
@@ -47,7 +54,6 @@ const init = (AV) => {
   // Shared empty constructor function to aid in prototype-chain creation.
   var EmptyConstructor = function() {};
 
-
   // Helper function to correctly set up the prototype chain, for subclasses.
   // Similar to `goog.inherits`, but uses a hash of prototype properties and
   // class properties to be extended.
@@ -177,13 +183,13 @@ const init = (AV) => {
    * @param {Boolean} production True is production environment,and
    *  it's true by default.
    */
-  AV.setProduction = function(production){
-    if(!AV._isNullOrUndefined(production)) {
-      //make sure it's a number
-      production = production ? 1 : 0;
+  AV.setProduction = (production) => {
+    if (!isNullOrUndefined(production)) {
+      AVConfig.applicationProduction = production ? 1 : 0;
+    } else {
+      // change to default value
+      AVConfig.applicationProduction = null;
     }
-    //default is 1
-    AV.applicationProduction = AV._isNullOrUndefined(production) ? 1: production;
   };
 
   /**
@@ -540,13 +546,9 @@ const init = (AV) => {
       _.each(obj, callback);
     }
   };
-
-  // Helper function to check null or undefined.
-  AV._isNullOrUndefined = function(x) {
-    return _.isNull(x) || _.isUndefined(x);
-  };
 };
 
 module.exports = {
   init,
+  isNullOrUndefined,
 };