From 656d58281eedfbad680de2f2f168a5b534c4724b Mon Sep 17 00:00:00 2001
From: Thomas Schuiki <thomas.schuiki@learnchamp.com>
Date: Mon, 10 May 2021 13:33:45 +0200
Subject: [PATCH] resolve #283

---
 src/Scorm12API.js   | 13 +++++--------
 src/Scorm2004API.js | 13 +++++--------
 2 files changed, 10 insertions(+), 16 deletions(-)

diff --git a/src/Scorm12API.js b/src/Scorm12API.js
index 6aff9a3..0afb476 100644
--- a/src/Scorm12API.js
+++ b/src/Scorm12API.js
@@ -297,18 +297,15 @@ export default class Scorm12API extends BaseAPI {
     const commitObject = this.renderCommitCMI(terminateCommit ||
         this.settings.alwaysSendTotalTime);
 
-    if (this.settings.lmsCommitUrl) {
-      if (this.apiLogLevel === global_constants.LOG_LEVEL_DEBUG) {
-        console.debug('Commit (terminated: ' +
+    if (this.apiLogLevel === global_constants.LOG_LEVEL_DEBUG) {
+      console.debug('Commit (terminated: ' +
             (terminateCommit ? 'yes' : 'no') + '): ');
-        console.debug(commitObject);
-      }
+      console.debug(commitObject);
+    }
+    if (this.settings.lmsCommitUrl) {
       return this.processHttpRequest(this.settings.lmsCommitUrl, commitObject,
           terminateCommit);
     } else {
-      console.log('Commit (terminated: ' +
-          (terminateCommit ? 'yes' : 'no') + '): ');
-      console.log(commitObject);
       return global_constants.SCORM_TRUE;
     }
   }
diff --git a/src/Scorm2004API.js b/src/Scorm2004API.js
index 72b6003..85c35aa 100644
--- a/src/Scorm2004API.js
+++ b/src/Scorm2004API.js
@@ -548,12 +548,12 @@ export default class Scorm2004API extends BaseAPI {
     const commitObject = this.renderCommitCMI(terminateCommit ||
         this.settings.alwaysSendTotalTime);
 
-    if (this.settings.lmsCommitUrl) {
-      if (this.apiLogLevel === global_constants.LOG_LEVEL_DEBUG) {
-        console.debug('Commit (terminated: ' +
+    if (this.apiLogLevel === global_constants.LOG_LEVEL_DEBUG) {
+      console.debug('Commit (terminated: ' +
             (terminateCommit ? 'yes' : 'no') + '): ');
-        console.debug(commitObject);
-      }
+      console.debug(commitObject);
+    }
+    if (this.settings.lmsCommitUrl) {
       const result = this.processHttpRequest(this.settings.lmsCommitUrl,
           commitObject, terminateCommit);
 
@@ -566,9 +566,6 @@ export default class Scorm2004API extends BaseAPI {
       }
       return result;
     } else {
-      console.log('Commit (terminated: ' +
-          (terminateCommit ? 'yes' : 'no') + '): ');
-      console.log(commitObject);
       return global_constants.SCORM_TRUE;
     }
   }