scorm-again 2.6.3 → 2.6.5

package/src/cmi/aicc/student_demographics.ts

@@ -59,7 +59,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set city(city: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._city = city;
     }
@@ -80,7 +80,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set class(clazz: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._class = clazz;
     }
@@ -101,7 +101,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set company(company: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._company = company;
     }
@@ -122,7 +122,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set country(country: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._country = country;
     }
@@ -143,7 +143,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set experience(experience: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._experience = experience;
     }
@@ -164,7 +164,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set familiar_name(familiar_name: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._familiar_name = familiar_name;
     }
@@ -185,7 +185,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set instructor_name(instructor_name: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._instructor_name = instructor_name;
     }
@@ -206,7 +206,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set title(title: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._title = title;
     }
@@ -227,7 +227,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set native_language(native_language: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._native_language = native_language;
     }
@@ -248,7 +248,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set state(state: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._state = state;
     }
@@ -269,7 +269,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set street_address(street_address: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._street_address = street_address;
     }
@@ -290,7 +290,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set telephone(telephone: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._telephone = telephone;
     }
@@ -311,7 +311,7 @@ export class CMIStudentDemographics extends BaseCMI {
    */
   set years_experience(years_experience: string) {
     if (this.initialized) {
-      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new AICCValidationError(scorm12_errors.READ_ONLY_ELEMENT as number);
     } else {
       this._years_experience = years_experience;
     }

package/src/cmi/aicc/student_preferences.ts

@@ -16,7 +16,7 @@ export class AICCStudentPreferences extends CMIStudentPreference {
   constructor() {
     super(aicc_constants.student_preference_children);
     this.windows = new CMIArray({
-      errorCode: scorm12_errors.INVALID_SET_VALUE,
+      errorCode: scorm12_errors.INVALID_SET_VALUE as number,
       errorClass: AICCValidationError,
       children: "",
     });

package/src/cmi/aicc/tries.ts

@@ -36,9 +36,9 @@ export class CMITriesObject extends BaseCMI {
     this.score = new CMIScore({
       score_children: aicc_constants.score_children,
       score_range: aicc_regex.score_range,
-      invalidErrorCode: scorm12_errors.INVALID_SET_VALUE,
-      invalidTypeCode: scorm12_errors.TYPE_MISMATCH,
-      invalidRangeCode: scorm12_errors.VALUE_OUT_OF_RANGE,
+      invalidErrorCode: scorm12_errors.INVALID_SET_VALUE as number,
+      invalidTypeCode: scorm12_errors.TYPE_MISMATCH as number,
+      invalidRangeCode: scorm12_errors.VALUE_OUT_OF_RANGE as number,
       errorClass: AICCValidationError,
     });
   }

package/src/cmi/aicc/validation.ts

@@ -17,7 +17,7 @@ export function checkAICCValidFormat(
   return checkValidFormat(
     value,
     regexPattern,
-    scorm12_errors.TYPE_MISMATCH,
+    scorm12_errors.TYPE_MISMATCH as number,
     AICCValidationError,
     allowEmptyString,
   );

package/src/cmi/common/array.ts

@@ -22,7 +22,7 @@ export class CMIArray extends BaseCMI {
   }) {
     super();
     this.__children = params.children;
-    this._errorCode = params.errorCode || scorm12_errors.GENERAL;
+    this._errorCode = params.errorCode || (scorm12_errors.GENERAL as number);
     this._errorClass = params.errorClass || BaseScormValidationError;
     this.childArray = [];
   }

package/src/cmi/common/score.ts

@@ -51,11 +51,11 @@ export class CMIScore extends BaseCMI {
       : scorm12_regex.score_range;
     this._max = params.max || params.max === "" ? params.max : "100";
     this.__invalid_error_code =
-      params.invalidErrorCode || scorm12_errors.INVALID_SET_VALUE;
+      params.invalidErrorCode || (scorm12_errors.INVALID_SET_VALUE as number);
     this.__invalid_type_code =
-      params.invalidTypeCode || scorm12_errors.TYPE_MISMATCH;
+      params.invalidTypeCode || (scorm12_errors.TYPE_MISMATCH as number);
     this.__invalid_range_code =
-      params.invalidRangeCode || scorm12_errors.VALUE_OUT_OF_RANGE;
+      params.invalidRangeCode || (scorm12_errors.VALUE_OUT_OF_RANGE as number);
     this.__decimal_regex = params.decimalRegex || scorm12_regex.CMIDecimal;
     this.__error_class = params.errorClass;
   }

package/src/cmi/common/validation.ts

@@ -48,8 +48,8 @@ export function checkValidRange(
 ): boolean {
   const ranges = rangePattern.split("#");
   value = value * 1.0;
-  if (value >= ranges[0]) {
-    if (ranges[1] === "*" || value <= ranges[1]) {
+  if (ranges[0] && value >= ranges[0]) {
+    if (ranges[1] && (ranges[1] === "*" || value <= ranges[1])) {
       return true;
     } else {
       throw new errorClass(errorCode);

package/src/cmi/scorm12/cmi.ts

@@ -133,7 +133,9 @@ export class CMI extends BaseRootCMI {
    * @param {string} _version
    */
   set _version(_version: string) {
-    throw new Scorm12ValidationError(scorm12_errors.INVALID_SET_VALUE);
+    throw new Scorm12ValidationError(
+      scorm12_errors.INVALID_SET_VALUE as number,
+    );
   }
 
   /**
@@ -149,7 +151,9 @@ export class CMI extends BaseRootCMI {
    * @param {string} _children
    */
   set _children(_children: string) {
-    throw new Scorm12ValidationError(scorm12_errors.INVALID_SET_VALUE);
+    throw new Scorm12ValidationError(
+      scorm12_errors.INVALID_SET_VALUE as number,
+    );
   }
 
   /**
@@ -184,7 +188,9 @@ export class CMI extends BaseRootCMI {
    */
   set launch_data(launch_data: string) {
     if (this.initialized) {
-      throw new Scorm12ValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.READ_ONLY_ELEMENT as number,
+      );
     } else {
       this._launch_data = launch_data;
     }
@@ -222,7 +228,9 @@ export class CMI extends BaseRootCMI {
    */
   set comments_from_lms(comments_from_lms: string) {
     if (this.initialized) {
-      throw new Scorm12ValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.READ_ONLY_ELEMENT as number,
+      );
     } else {
       this._comments_from_lms = comments_from_lms;
     }

package/src/cmi/scorm12/interactions.ts

@@ -17,7 +17,7 @@ export class CMIInteractions extends CMIArray {
   constructor() {
     super({
       children: scorm12_constants.interactions_children,
-      errorCode: scorm12_errors.INVALID_SET_VALUE,
+      errorCode: scorm12_errors.INVALID_SET_VALUE as number,
       errorClass: Scorm12ValidationError,
     });
   }
@@ -34,12 +34,12 @@ export class CMIInteractionsObject extends BaseCMI {
   constructor() {
     super();
     this.objectives = new CMIArray({
-      errorCode: scorm12_errors.INVALID_SET_VALUE,
+      errorCode: scorm12_errors.INVALID_SET_VALUE as number,
       errorClass: Scorm12ValidationError,
       children: scorm12_constants.objectives_children,
     });
     this.correct_responses = new CMIArray({
-      errorCode: scorm12_errors.INVALID_SET_VALUE,
+      errorCode: scorm12_errors.INVALID_SET_VALUE as number,
       errorClass: Scorm12ValidationError,
       children: scorm12_constants.correct_responses_children,
     });
@@ -89,7 +89,9 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get id(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.WRITE_ONLY_ELEMENT as number,
+      );
     }
     return this._id;
   }
@@ -110,7 +112,9 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get time(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.WRITE_ONLY_ELEMENT as number,
+      );
     }
     return this._time;
   }
@@ -131,7 +135,9 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get type(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.WRITE_ONLY_ELEMENT as number,
+      );
     }
     return this._type;
   }
@@ -152,7 +158,9 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get weighting(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.WRITE_ONLY_ELEMENT as number,
+      );
     }
     return this._weighting;
   }
@@ -176,7 +184,9 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get student_response(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.WRITE_ONLY_ELEMENT as number,
+      );
     }
     return this._student_response;
   }
@@ -197,7 +207,9 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get result(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.WRITE_ONLY_ELEMENT as number,
+      );
     }
     return this._result;
   }
@@ -218,7 +230,9 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get latency(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.WRITE_ONLY_ELEMENT as number,
+      );
     }
     return this._latency;
   }
@@ -366,7 +380,9 @@ export class CMIInteractionsCorrectResponsesObject extends BaseCMI {
    */
   get pattern(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.WRITE_ONLY_ELEMENT as number,
+      );
     }
     return this._pattern;
   }

package/src/cmi/scorm12/objectives.ts

@@ -18,7 +18,7 @@ export class CMIObjectives extends CMIArray {
   constructor() {
     super({
       children: scorm12_constants.objectives_children,
-      errorCode: scorm12_errors.INVALID_SET_VALUE,
+      errorCode: scorm12_errors.INVALID_SET_VALUE as number,
       errorClass: Scorm12ValidationError,
     });
   }
@@ -37,9 +37,9 @@ export class CMIObjectivesObject extends BaseCMI {
     this.score = new CMIScore({
       score_children: scorm12_constants.score_children,
       score_range: scorm12_regex.score_range,
-      invalidErrorCode: scorm12_errors.INVALID_SET_VALUE,
-      invalidTypeCode: scorm12_errors.TYPE_MISMATCH,
-      invalidRangeCode: scorm12_errors.VALUE_OUT_OF_RANGE,
+      invalidErrorCode: scorm12_errors.INVALID_SET_VALUE as number,
+      invalidTypeCode: scorm12_errors.TYPE_MISMATCH as number,
+      invalidRangeCode: scorm12_errors.VALUE_OUT_OF_RANGE as number,
       errorClass: Scorm12ValidationError,
     });
   }

package/src/cmi/scorm12/student_data.ts

@@ -46,7 +46,9 @@ export class CMIStudentData extends BaseCMI {
    * @private
    */
   set _children(_children: string) {
-    throw new Scorm12ValidationError(scorm12_errors.INVALID_SET_VALUE);
+    throw new Scorm12ValidationError(
+      scorm12_errors.INVALID_SET_VALUE as number,
+    );
   }
 
   /**
@@ -63,7 +65,9 @@ export class CMIStudentData extends BaseCMI {
    */
   set mastery_score(mastery_score: string) {
     if (this.initialized) {
-      throw new Scorm12ValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.READ_ONLY_ELEMENT as number,
+      );
     } else {
       this._mastery_score = mastery_score;
     }
@@ -83,7 +87,9 @@ export class CMIStudentData extends BaseCMI {
    */
   set max_time_allowed(max_time_allowed: string) {
     if (this.initialized) {
-      throw new Scorm12ValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.READ_ONLY_ELEMENT as number,
+      );
     } else {
       this._max_time_allowed = max_time_allowed;
     }
@@ -103,7 +109,9 @@ export class CMIStudentData extends BaseCMI {
    */
   set time_limit_action(time_limit_action: string) {
     if (this.initialized) {
-      throw new Scorm12ValidationError(scorm12_errors.READ_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(
+        scorm12_errors.READ_ONLY_ELEMENT as number,
+      );
     } else {
       this._time_limit_action = time_limit_action;
     }

package/src/cmi/scorm12/student_preference.ts

@@ -50,7 +50,9 @@ export class CMIStudentPreference extends BaseCMI {
    * @private
    */
   set _children(_children: string) {
-    throw new Scorm12ValidationError(scorm12_errors.INVALID_SET_VALUE);
+    throw new Scorm12ValidationError(
+      scorm12_errors.INVALID_SET_VALUE as number,
+    );
   }
 
   /**

package/src/cmi/scorm12/validation.ts

@@ -17,7 +17,7 @@ export function check12ValidFormat(
   return checkValidFormat(
     value,
     regexPattern,
-    scorm12_errors.TYPE_MISMATCH,
+    scorm12_errors.TYPE_MISMATCH as number,
     Scorm12ValidationError,
     allowEmptyString,
   );
@@ -36,13 +36,15 @@ export function check12ValidRange(
   allowEmptyString?: boolean,
 ): boolean {
   if (!allowEmptyString && value === "") {
-    throw new Scorm12ValidationError(scorm12_errors.VALUE_OUT_OF_RANGE);
+    throw new Scorm12ValidationError(
+      scorm12_errors.VALUE_OUT_OF_RANGE as number,
+    );
   }
 
   return checkValidRange(
     value,
     rangePattern,
-    scorm12_errors.VALUE_OUT_OF_RANGE,
+    scorm12_errors.VALUE_OUT_OF_RANGE as number,
     Scorm12ValidationError,
   );
 }

package/src/cmi/scorm2004/adl.ts

@@ -142,7 +142,7 @@ export class ADLData extends CMIArray {
   constructor() {
     super({
       children: scorm2004_constants.adl_data_children,
-      errorCode: scorm2004_errors.READ_ONLY_ELEMENT,
+      errorCode: scorm2004_errors.READ_ONLY_ELEMENT as number,
       errorClass: Scorm2004ValidationError,
     });
   }
@@ -270,7 +270,9 @@ export class ADLNavRequestValid extends BaseCMI {
    */
   set continue(_continue: string) {
     if (this.initialized) {
-      throw new Scorm2004ValidationError(scorm2004_errors.READ_ONLY_ELEMENT);
+      throw new Scorm2004ValidationError(
+        scorm2004_errors.READ_ONLY_ELEMENT as number,
+      );
     }
     if (check2004ValidFormat(_continue, scorm2004_regex.NAVBoolean)) {
       this._continue = _continue;
@@ -291,7 +293,9 @@ export class ADLNavRequestValid extends BaseCMI {
    */
   set previous(_previous: string) {
     if (this.initialized) {
-      throw new Scorm2004ValidationError(scorm2004_errors.READ_ONLY_ELEMENT);
+      throw new Scorm2004ValidationError(
+        scorm2004_errors.READ_ONLY_ELEMENT as number,
+      );
     }
     if (check2004ValidFormat(_previous, scorm2004_regex.NAVBoolean)) {
       this._previous = _previous;
@@ -308,23 +312,28 @@ export class ADLNavRequestValid extends BaseCMI {
 
   /**
    * Setter for _choice
-   * @param {{ [key: string]: string }} choice
+   * @param {{ [key: string]: string | NAVBoolean }} choice
    */
-  set choice(choice: { [key: string]: string }) {
+  set choice(choice: { [key: string]: string | NAVBoolean }) {
     if (this.initialized) {
-      throw new Scorm2004ValidationError(scorm2004_errors.READ_ONLY_ELEMENT);
+      throw new Scorm2004ValidationError(
+        scorm2004_errors.READ_ONLY_ELEMENT as number,
+      );
     }
     if (typeof choice !== "object") {
-      throw new Scorm2004ValidationError(scorm2004_errors.TYPE_MISMATCH);
+      throw new Scorm2004ValidationError(
+        scorm2004_errors.TYPE_MISMATCH as number,
+      );
     }
     for (const key in choice) {
       if ({}.hasOwnProperty.call(choice, key)) {
-        if (
-          check2004ValidFormat(choice[key], scorm2004_regex.NAVBoolean) &&
-          check2004ValidFormat(key, scorm2004_regex.NAVTarget)
-        ) {
-          this._choice[key] =
-            NAVBoolean[choice[key] as keyof typeof NAVBoolean];
+        if (choice[key] !== undefined && check2004ValidFormat(key, scorm2004_regex.NAVTarget)) {
+          const value = choice[key];
+          if (typeof value === 'string' && check2004ValidFormat(value, scorm2004_regex.NAVBoolean)) {
+            this._choice[key] = NAVBoolean[value as keyof typeof NAVBoolean];
+          } else if (Object.values(NAVBoolean).includes(value as NAVBoolean)) {
+            this._choice[key] = value as NAVBoolean;
+          }
         }
       }
     }
@@ -340,22 +349,28 @@ export class ADLNavRequestValid extends BaseCMI {
 
   /**
    * Setter for _jump
-   * @param {{ [key: string]: string }} jump
+   * @param {{ [key: string]: string | NAVBoolean }} jump
    */
-  set jump(jump: { [key: string]: string }) {
+  set jump(jump: { [key: string]: string | NAVBoolean }) {
     if (this.initialized) {
-      throw new Scorm2004ValidationError(scorm2004_errors.READ_ONLY_ELEMENT);
+      throw new Scorm2004ValidationError(
+        scorm2004_errors.READ_ONLY_ELEMENT as number,
+      );
     }
     if (typeof jump !== "object") {
-      throw new Scorm2004ValidationError(scorm2004_errors.TYPE_MISMATCH);
+      throw new Scorm2004ValidationError(
+        scorm2004_errors.TYPE_MISMATCH as number,
+      );
     }
     for (const key in jump) {
       if ({}.hasOwnProperty.call(jump, key)) {
-        if (
-          check2004ValidFormat(jump[key], scorm2004_regex.NAVBoolean) &&
-          check2004ValidFormat(key, scorm2004_regex.NAVTarget)
-        ) {
-          this._jump[key] = NAVBoolean[jump[key] as keyof typeof NAVBoolean];
+        if (jump[key] !== undefined && check2004ValidFormat(key, scorm2004_regex.NAVTarget)) {
+          const value = jump[key];
+          if (typeof value === 'string' && check2004ValidFormat(value, scorm2004_regex.NAVBoolean)) {
+            this._jump[key] = NAVBoolean[value as keyof typeof NAVBoolean];
+          } else if (Object.values(NAVBoolean).includes(value as NAVBoolean)) {
+            this._jump[key] = value as NAVBoolean;
+          }
         }
       }
     }
@@ -374,6 +389,8 @@ export class ADLNavRequestValid extends BaseCMI {
   toJSON(): {
     previous: string;
     continue: string;
+    choice: { [key: string]: NAVBoolean };
+    jump: { [key: string]: NAVBoolean };
   } {
     this.jsonString = true;
     const result = {