Fix: Adjust condition for sending 'attempted' statement (fixes #129) (#130)

nachocinalli · web-flow · commit 068fb35c13f0 · 2024-06-18T14:18:28.000+01:00
diff --git a/js/XAPI.js b/js/XAPI.js
@@ -144,7 +144,8 @@ class XAPI extends Backbone.Model {
       return this;
     }
 
-    if (this.get('state').length === 0) {
+    const state = this.get('state');
+    if (!state || Object.keys(state).length === 0) {
       // This is a new attempt, send 'attempted'.
       await this.sendStatement(this.getCourseStatement(window.ADL.verbs.attempted));
     } else {

Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,8 @@ class XAPI extends Backbone.Model {`
`144`	`144`	`return this;`
`145`	`145`	`}`
`146`	`146`
`147`		`- if (this.get('state').length === 0) {`
	`147`	`+ const state = this.get('state');`
	`148`	`+ if (!state \|\| Object.keys(state).length === 0) {`
`148`	`149`	`// This is a new attempt, send 'attempted'.`
`149`	`150`	`await this.sendStatement(this.getCourseStatement(window.ADL.verbs.attempted));`
`150`	`151`	`} else {`