Fix: exclude non-trackable models from checkCompletionStatusFor (fixes

#526).
adaptlearning · May 3, 2024 · 8342e94 · 8342e94
1 parent f6fe365
commit 8342e94
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/js/models/adaptModel.js b/js/models/adaptModel.js
@@ -313,7 +313,7 @@ export default class AdaptModel extends LockingModel {
    */
   checkCompletionStatusFor(completionAttribute = '_isComplete') {
     let completed = false;
-    const children = this.getAvailableChildModels();
+    const children = this.getAvailableTrackableChildModels();
     const requireCompletionOf = this.get('_requireCompletionOf');
     const isOptional = this.get('_isOptional');
     const isEveryChildOptional = children.every(child => child.get('_isOptional'));
@@ -651,6 +651,10 @@ export default class AdaptModel extends LockingModel {
     });
   }
 
+  getAvailableTrackableChildModels() {
+    return this.getAvailableChildModels().filter(child => child.get('_isTrackable') !== false);
+  }
+
   getParent() {
     if (this._parentModel) {
       return this._parentModel;