Merge pull request #69 from oat-sa/hotfix/NCSUP-268/external-logs

Hotfix/ncsup 268/external logs
oat-sa · Apr 24, 2020 · a16f1a2 · a16f1a2
2 parents ad28b5e + 814c35d
commit a16f1a2
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "@oat-sa/tao-item-runner-qti",
-    "version": "0.4.18",
+    "version": "0.4.18-1",
     "displayName": "TAO Item Runner QTI",
     "description": "TAO QTI Item Runner modules",
     "files": [

diff --git a/src/qtiItem/core/Loader.js b/src/qtiItem/core/Loader.js
@@ -23,6 +23,9 @@ import qtiClasses from 'taoQtiItem/qtiItem/core/qtiClasses';
 import Element from 'taoQtiItem/qtiItem/core/Element';
 import xmlNsHandler from 'taoQtiItem/qtiItem/helper/xmlNsHandler';
 import moduleLoader from 'core/moduleLoader';
+import loggerFactory from 'core/logger';
+
+var logger = loggerFactory('taoQtiItem/qtiItem/core/Loader');
 
 var Loader = Class.extend({
     init: function(item, classesLocation) {
@@ -54,7 +57,8 @@ var Loader = Class.extend({
         var i;
         var requiredClass,
             requiredClasses = this.getRequiredClasses(data, reload),
-            required = [];
+            required = [],
+            callStack;
 
         for (i in requiredClasses) {
             requiredClass = requiredClasses[i];
@@ -64,7 +68,8 @@ var Loader = Class.extend({
                     category: 'qti'
                 });
             } else {
-                throw new Error('missing qti class location declaration : ' + requiredClass);
+                callStack = new Error().stack;
+                logger.error({requiredClasses, data, callStack}, 'Missing qti class location declaration:');
             }
         }