Initialize logLevel the right way

blazzy · blazzy · commit 8bbc3eca1f6c · 2017-10-24T10:20:09.000-07:00
diff --git a/amplitude.js b/amplitude.js
@@ -5646,6 +5646,7 @@ var DEFAULT_OPTIONS = {
   includeReferrer: false,
   includeUtm: false,
   language: language.language,
+  logLevel: 'WARN',
   optOut: false,
   platform: 'Web',
   savedMaxCount: 1000,
@@ -5753,10 +5754,6 @@ AmplitudeClient.prototype.init = function init(apiKey, opt_userId, opt_config, o
       }
     }
 
-    if (type(this.options.logLevel) === 'string') {
-      utils.setLogLevel(this.options.logLevel);
-    }
-
     var now = new Date().getTime();
     if (!this._sessionId || !this._lastEventTime || now - this._lastEventTime > this.options.sessionTimeout) {
       this._newSession = true;
diff --git a/amplitude.min.js b/amplitude.min.js
diff --git a/src/amplitude-client.js b/src/amplitude-client.js
@@ -108,10 +108,6 @@ AmplitudeClient.prototype.init = function init(apiKey, opt_userId, opt_config, o
       }
     }
 
-    if (type(this.options.logLevel) === 'string') {
-      utils.setLogLevel(this.options.logLevel);
-    }
-
     var now = new Date().getTime();
     if (!this._sessionId || !this._lastEventTime || now - this._lastEventTime > this.options.sessionTimeout) {
       this._newSession = true;
diff --git a/src/options.js b/src/options.js
@@ -9,6 +9,7 @@ export default {
   includeReferrer: false,
   includeUtm: false,
   language: language.language,
+  logLevel: 'WARN',
   optOut: false,
   platform: 'Web',
   savedMaxCount: 1000,

Original file line number	Diff line number	Diff line change
`@@ -108,10 +108,6 @@ AmplitudeClient.prototype.init = function init(apiKey, opt_userId, opt_config, o`
`108`	`108`	`}`
`109`	`109`	`}`
`110`	`110`
`111`		`- if (type(this.options.logLevel) === 'string') {`
`112`		`- utils.setLogLevel(this.options.logLevel);`
`113`		`- }`
`114`		`-`
`115`	`111`	`var now = new Date().getTime();`
`116`	`112`	`if (!this._sessionId \|\| !this._lastEventTime \|\| now - this._lastEventTime > this.options.sessionTimeout) {`
`117`	`113`	`this._newSession = true;`