Resolves #2386, switches listeners out for 'configModel:dataLoaded' (#…

…2387) The tracking, logging and scrolling modules were only checking the Adapt.config object so once it's loaded they are ok to continue.
adaptlearning · Mar 19, 2019 · 14bde8b · 14bde8b
1 parent 1d2c3d0
commit 14bde8b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/core/js/logging.js b/src/core/js/logging.js
@@ -13,11 +13,11 @@ define([
 
         initialize: function() {
 
-            Adapt.once('configModel:loadCourseData', this.onLoadCourseData.bind(this));
+            Adapt.once('configModel:dataLoaded', this.onLoadConfigData.bind(this));
 
         },
 
-        onLoadCourseData: function() {
+        onLoadConfigData: function() {
 
             this.loadConfig();
 

diff --git a/src/core/js/scrolling.js b/src/core/js/scrolling.js
@@ -10,7 +10,7 @@ define([
 
         initialize: function() {
             this._checkApp();
-            Adapt.once('configModel:loadCourseData', this._loadConfig.bind(this));
+            Adapt.once('configModel:dataLoaded', this._loadConfig.bind(this));
         },
 
         _checkApp: function() {

diff --git a/src/core/js/tracking.js b/src/core/js/tracking.js
@@ -13,7 +13,7 @@ define([
         _assessmentState: null,
 
         initialize: function() {
-            Adapt.once('configModel:loadCourseData', this.loadConfig.bind(this));
+            Adapt.once('configModel:dataLoaded', this.loadConfig.bind(this));
             Adapt.on('app:dataReady', this.setupEventListeners.bind(this));
         },