scorm-again 2.4.1 → 2.6.0

package/src/cmi/common/score.ts

@@ -1,14 +1,10 @@
-import APIConstants from "../../constants/api_constants";
-import Regex from "../../constants/regex";
+import { scorm12_constants } from "../../constants/api_constants";
+import { scorm12_regex } from "../../constants/regex";
 import { BaseCMI } from "./base_cmi";
 import { checkValidFormat, checkValidRange } from "./validation";
-import ErrorCodes from "../../constants/error_codes";
+import { scorm12_errors } from "../../constants/error_codes";
 import { BaseScormValidationError } from "../../exceptions";
 
-const scorm12_constants = APIConstants.scorm12;
-const scorm12_regex = Regex.scorm12;
-export const scorm12_error_codes = ErrorCodes.scorm12;
-
 /**
  * Base class for cmi *.score objects
  */
@@ -20,9 +16,9 @@ export class CMIScore extends BaseCMI {
   private readonly __invalid_range_code: number;
   private readonly __decimal_regex: string;
   private readonly __error_class: typeof BaseScormValidationError;
-  private _raw = "";
-  private _min = "";
-  private _max: string;
+  protected _raw = "";
+  protected _min = "";
+  protected _max: string;
 
   /**
    * Constructor for *.score
@@ -55,15 +51,22 @@ 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_error_codes.INVALID_SET_VALUE;
+      params.invalidErrorCode || scorm12_errors.INVALID_SET_VALUE;
     this.__invalid_type_code =
-      params.invalidTypeCode || scorm12_error_codes.TYPE_MISMATCH;
+      params.invalidTypeCode || scorm12_errors.TYPE_MISMATCH;
     this.__invalid_range_code =
-      params.invalidRangeCode || scorm12_error_codes.VALUE_OUT_OF_RANGE;
+      params.invalidRangeCode || scorm12_errors.VALUE_OUT_OF_RANGE;
     this.__decimal_regex = params.decimalRegex || scorm12_regex.CMIDecimal;
     this.__error_class = params.errorClass;
   }
 
+  /**
+   * Called when the API has been reset
+   */
+  reset(): void {
+    this._initialized = false;
+  }
+
   /**
    * Getter for _children
    * @return {string}

package/src/cmi/scorm12/cmi.ts

@@ -1,7 +1,7 @@
-import APIConstants from "../../constants/api_constants";
-import ErrorCodes from "../../constants/error_codes";
-import Regex from "../../constants/regex";
-import { Scorm12ValidationError } from "../../exceptions";
+import { scorm12_constants } from "../../constants/api_constants";
+import { scorm12_errors } from "../../constants/error_codes";
+import { scorm12_regex } from "../../constants/regex";
+import { Scorm12ValidationError } from "../../exceptions/scorm12_exceptions";
 import { BaseRootCMI } from "../common/base_cmi";
 import { check12ValidFormat } from "./validation";
 import { CMICore } from "../aicc/core";
@@ -35,7 +35,7 @@ export class CMI extends BaseRootCMI {
     if (initialized) this.initialize();
     this.__children = cmi_children
       ? cmi_children
-      : APIConstants.scorm12.cmi_children;
+      : scorm12_constants.cmi_children;
     this.core = new CMICore();
     this.objectives = new CMIObjectives();
     this.student_data = student_data ? student_data : new CMIStudentData();
@@ -49,6 +49,21 @@ export class CMI extends BaseRootCMI {
   public student_preference: CMIStudentPreference;
   public interactions: CMIInteractions;
 
+  /**
+   * Called when the API has been reset
+   */
+  reset(): void {
+    this._initialized = false;
+
+    this._launch_data = "";
+    this._comments = "";
+    this.core?.reset();
+    this.objectives = new CMIObjectives();
+    this.interactions = new CMIInteractions();
+    this.student_data?.reset();
+    this.student_preference?.reset();
+  }
+
   /**
    * Called when the API has been initialized after the CMI has been created
    */
@@ -118,7 +133,7 @@ export class CMI extends BaseRootCMI {
    * @param {string} _version
    */
   set _version(_version: string) {
-    throw new Scorm12ValidationError(ErrorCodes.scorm12.INVALID_SET_VALUE);
+    throw new Scorm12ValidationError(scorm12_errors.INVALID_SET_VALUE);
   }
 
   /**
@@ -134,7 +149,7 @@ export class CMI extends BaseRootCMI {
    * @param {string} _children
    */
   set _children(_children: string) {
-    throw new Scorm12ValidationError(ErrorCodes.scorm12.INVALID_SET_VALUE);
+    throw new Scorm12ValidationError(scorm12_errors.INVALID_SET_VALUE);
   }
 
   /**
@@ -169,7 +184,7 @@ export class CMI extends BaseRootCMI {
    */
   set launch_data(launch_data: string) {
     if (this.initialized) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.READ_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.READ_ONLY_ELEMENT);
     } else {
       this._launch_data = launch_data;
     }
@@ -188,7 +203,7 @@ export class CMI extends BaseRootCMI {
    * @param {string} comments
    */
   set comments(comments: string) {
-    if (check12ValidFormat(comments, Regex.scorm12.CMIString4096, true)) {
+    if (check12ValidFormat(comments, scorm12_regex.CMIString4096, true)) {
       this._comments = comments;
     }
   }
@@ -207,7 +222,7 @@ export class CMI extends BaseRootCMI {
    */
   set comments_from_lms(comments_from_lms: string) {
     if (this.initialized) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.READ_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.READ_ONLY_ELEMENT);
     } else {
       this._comments_from_lms = comments_from_lms;
     }

package/src/cmi/scorm12/interactions.ts

@@ -1,10 +1,10 @@
 import { CMIArray } from "../common/array";
-import APIConstants from "../../constants/api_constants";
-import ErrorCodes from "../../constants/error_codes";
-import { Scorm12ValidationError } from "../../exceptions";
+import { scorm12_constants } from "../../constants/api_constants";
+import { scorm12_errors } from "../../constants/error_codes";
+import { Scorm12ValidationError } from "../../exceptions/scorm12_exceptions";
 import { BaseCMI } from "../common/base_cmi";
 import { check12ValidFormat, check12ValidRange } from "./validation";
-import Regex from "../../constants/regex";
+import { scorm12_regex } from "../../constants/regex";
 
 /**
  * Class representing the SCORM 1.2 `cmi.interactions`
@@ -16,8 +16,8 @@ export class CMIInteractions extends CMIArray {
    */
   constructor() {
     super({
-      children: APIConstants.scorm12.interactions_children,
-      errorCode: ErrorCodes.scorm12.INVALID_SET_VALUE,
+      children: scorm12_constants.interactions_children,
+      errorCode: scorm12_errors.INVALID_SET_VALUE,
       errorClass: Scorm12ValidationError,
     });
   }
@@ -34,14 +34,14 @@ export class CMIInteractionsObject extends BaseCMI {
   constructor() {
     super();
     this.objectives = new CMIArray({
-      errorCode: ErrorCodes.scorm12.INVALID_SET_VALUE,
+      errorCode: scorm12_errors.INVALID_SET_VALUE,
       errorClass: Scorm12ValidationError,
-      children: APIConstants.scorm12.objectives_children,
+      children: scorm12_constants.objectives_children,
     });
     this.correct_responses = new CMIArray({
-      errorCode: ErrorCodes.scorm12.INVALID_SET_VALUE,
+      errorCode: scorm12_errors.INVALID_SET_VALUE,
       errorClass: Scorm12ValidationError,
-      children: APIConstants.scorm12.correct_responses_children,
+      children: scorm12_constants.correct_responses_children,
     });
   }
 
@@ -65,13 +65,31 @@ export class CMIInteractionsObject extends BaseCMI {
   private _result = "";
   private _latency = "";
 
+  /**
+   * Called when the API has been reset
+   */
+  override reset(): void {
+    this._initialized = false;
+
+    this._id = "";
+    this._time = "";
+    this._type = "";
+    this._weighting = "";
+    this._student_response = "";
+    this._result = "";
+    this._latency = "";
+
+    this.objectives?.reset();
+    this.correct_responses?.reset();
+  }
+
   /**
    * Getter for _id. Should only be called during JSON export.
    * @return {string}
    */
   get id(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
     }
     return this._id;
   }
@@ -81,7 +99,7 @@ export class CMIInteractionsObject extends BaseCMI {
    * @param {string} id
    */
   set id(id: string) {
-    if (check12ValidFormat(id, Regex.scorm12.CMIIdentifier)) {
+    if (check12ValidFormat(id, scorm12_regex.CMIIdentifier)) {
       this._id = id;
     }
   }
@@ -92,7 +110,7 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get time(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
     }
     return this._time;
   }
@@ -102,7 +120,7 @@ export class CMIInteractionsObject extends BaseCMI {
    * @param {string} time
    */
   set time(time: string) {
-    if (check12ValidFormat(time, Regex.scorm12.CMITime)) {
+    if (check12ValidFormat(time, scorm12_regex.CMITime)) {
       this._time = time;
     }
   }
@@ -113,7 +131,7 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get type(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
     }
     return this._type;
   }
@@ -123,7 +141,7 @@ export class CMIInteractionsObject extends BaseCMI {
    * @param {string} type
    */
   set type(type: string) {
-    if (check12ValidFormat(type, Regex.scorm12.CMIType)) {
+    if (check12ValidFormat(type, scorm12_regex.CMIType)) {
       this._type = type;
     }
   }
@@ -134,7 +152,7 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get weighting(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
     }
     return this._weighting;
   }
@@ -145,8 +163,8 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   set weighting(weighting: string) {
     if (
-      check12ValidFormat(weighting, Regex.scorm12.CMIDecimal) &&
-      check12ValidRange(weighting, Regex.scorm12.weighting_range)
+      check12ValidFormat(weighting, scorm12_regex.CMIDecimal) &&
+      check12ValidRange(weighting, scorm12_regex.weighting_range)
     ) {
       this._weighting = weighting;
     }
@@ -158,7 +176,7 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get student_response(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
     }
     return this._student_response;
   }
@@ -168,7 +186,7 @@ export class CMIInteractionsObject extends BaseCMI {
    * @param {string} student_response
    */
   set student_response(student_response: string) {
-    if (check12ValidFormat(student_response, Regex.scorm12.CMIFeedback, true)) {
+    if (check12ValidFormat(student_response, scorm12_regex.CMIFeedback, true)) {
       this._student_response = student_response;
     }
   }
@@ -179,7 +197,7 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get result(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
     }
     return this._result;
   }
@@ -189,7 +207,7 @@ export class CMIInteractionsObject extends BaseCMI {
    * @param {string} result
    */
   set result(result: string) {
-    if (check12ValidFormat(result, Regex.scorm12.CMIResult)) {
+    if (check12ValidFormat(result, scorm12_regex.CMIResult)) {
       this._result = result;
     }
   }
@@ -200,7 +218,7 @@ export class CMIInteractionsObject extends BaseCMI {
    */
   get latency(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
     }
     return this._latency;
   }
@@ -210,7 +228,7 @@ export class CMIInteractionsObject extends BaseCMI {
    * @param {string} latency
    */
   set latency(latency: string) {
-    if (check12ValidFormat(latency, Regex.scorm12.CMITimespan)) {
+    if (check12ValidFormat(latency, scorm12_regex.CMITimespan)) {
       this._latency = latency;
     }
   }
@@ -274,6 +292,14 @@ export class CMIInteractionsObjectivesObject extends BaseCMI {
 
   private _id = "";
 
+  /**
+   * Called when the API has been reset
+   */
+  reset(): void {
+    this._initialized = false;
+    this._id = "";
+  }
+
   /**
    * Getter for _id
    * @return {string}
@@ -287,7 +313,7 @@ export class CMIInteractionsObjectivesObject extends BaseCMI {
    * @param {string} id
    */
   set id(id: string) {
-    if (check12ValidFormat(id, Regex.scorm12.CMIIdentifier)) {
+    if (check12ValidFormat(id, scorm12_regex.CMIIdentifier)) {
       this._id = id;
     }
   }
@@ -326,13 +352,21 @@ export class CMIInteractionsCorrectResponsesObject extends BaseCMI {
 
   private _pattern = "";
 
+  /**
+   * Called when the API has been reset
+   */
+  reset(): void {
+    this._initialized = false;
+    this._pattern = "";
+  }
+
   /**
    * Getter for _pattern
    * @return {string}
    */
   get pattern(): string {
     if (!this.jsonString) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.WRITE_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.WRITE_ONLY_ELEMENT);
     }
     return this._pattern;
   }
@@ -342,7 +376,7 @@ export class CMIInteractionsCorrectResponsesObject extends BaseCMI {
    * @param {string} pattern
    */
   set pattern(pattern: string) {
-    if (check12ValidFormat(pattern, Regex.scorm12.CMIFeedback, true)) {
+    if (check12ValidFormat(pattern, scorm12_regex.CMIFeedback, true)) {
       this._pattern = pattern;
     }
   }

package/src/cmi/scorm12/nav.ts

@@ -1,6 +1,6 @@
-import {BaseCMI} from "../common/base_cmi";
-import {check12ValidFormat} from "./validation";
-import Regex from "../../constants/regex";
+import { BaseCMI } from "../common/base_cmi";
+import { check12ValidFormat } from "./validation";
+import { scorm12_regex } from "../../constants/regex";
 
 /**
  * Class for AICC Navigation object
@@ -13,6 +13,14 @@ export class NAV extends BaseCMI {
     super();
   }
 
+  /**
+   * Called when the API has been reset
+   */
+  reset(): void {
+    this._event = "";
+    this._initialized = false;
+  }
+
   private _event = "";
 
   /**
@@ -28,7 +36,7 @@ export class NAV extends BaseCMI {
    * @param {string} event
    */
   set event(event: string) {
-    if (check12ValidFormat(event, Regex.scorm12.NAVEvent)) {
+    if (check12ValidFormat(event, scorm12_regex.NAVEvent)) {
       this._event = event;
     }
   }
@@ -51,4 +59,4 @@ export class NAV extends BaseCMI {
     delete this.jsonString;
     return result;
   }
-}
+}

package/src/cmi/scorm12/objectives.ts

@@ -1,11 +1,11 @@
-import {BaseCMI} from "../common/base_cmi";
-import {CMIScore} from "../common/score";
-import APIConstants from "../../constants/api_constants";
-import Regex from "../../constants/regex";
-import ErrorCodes from "../../constants/error_codes";
-import {Scorm12ValidationError} from "../../exceptions";
-import {check12ValidFormat} from "./validation";
-import {CMIArray} from "../common/array";
+import { BaseCMI } from "../common/base_cmi";
+import { CMIScore } from "../common/score";
+import { scorm12_constants } from "../../constants/api_constants";
+import { scorm12_regex } from "../../constants/regex";
+import { scorm12_errors } from "../../constants/error_codes";
+import { Scorm12ValidationError } from "../../exceptions/scorm12_exceptions";
+import { check12ValidFormat } from "./validation";
+import { CMIArray } from "../common/array";
 
 /**
  * Class representing SCORM 1.2's `cmi.objectives` object
@@ -17,8 +17,8 @@ export class CMIObjectives extends CMIArray {
    */
   constructor() {
     super({
-      children: APIConstants.scorm12.objectives_children,
-      errorCode: ErrorCodes.scorm12.INVALID_SET_VALUE,
+      children: scorm12_constants.objectives_children,
+      errorCode: scorm12_errors.INVALID_SET_VALUE,
       errorClass: Scorm12ValidationError,
     });
   }
@@ -35,11 +35,11 @@ export class CMIObjectivesObject extends BaseCMI {
   constructor() {
     super();
     this.score = new CMIScore({
-      score_children: APIConstants.scorm12.score_children,
-      score_range: Regex.scorm12.score_range,
-      invalidErrorCode: ErrorCodes.scorm12.INVALID_SET_VALUE,
-      invalidTypeCode: ErrorCodes.scorm12.TYPE_MISMATCH,
-      invalidRangeCode: ErrorCodes.scorm12.VALUE_OUT_OF_RANGE,
+      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,
       errorClass: Scorm12ValidationError,
     });
   }
@@ -49,6 +49,16 @@ export class CMIObjectivesObject extends BaseCMI {
   private _id = "";
   private _status = "";
 
+  /**
+   * Called when the API has been reset
+   */
+  reset(): void {
+    this._initialized = false;
+    this._id = "";
+    this._status = "";
+    this.score?.reset();
+  }
+
   /**
    * Getter for _id
    * @return {string}
@@ -62,7 +72,7 @@ export class CMIObjectivesObject extends BaseCMI {
    * @param {string} id
    */
   set id(id: string) {
-    if (check12ValidFormat(id, Regex.scorm12.CMIIdentifier)) {
+    if (check12ValidFormat(id, scorm12_regex.CMIIdentifier)) {
       this._id = id;
     }
   }
@@ -80,7 +90,7 @@ export class CMIObjectivesObject extends BaseCMI {
    * @param {string} status
    */
   set status(status: string) {
-    if (check12ValidFormat(status, Regex.scorm12.CMIStatus2)) {
+    if (check12ValidFormat(status, scorm12_regex.CMIStatus2)) {
       this._status = status;
     }
   }
@@ -109,4 +119,4 @@ export class CMIObjectivesObject extends BaseCMI {
     delete this.jsonString;
     return result;
   }
-}
+}

package/src/cmi/scorm12/student_data.ts

@@ -1,7 +1,7 @@
 import { BaseCMI } from "../common/base_cmi";
-import APIConstants from "../../constants/api_constants";
-import { Scorm12ValidationError } from "../../exceptions";
-import ErrorCodes from "../../constants/error_codes";
+import { scorm12_constants } from "../../constants/api_constants";
+import { Scorm12ValidationError } from "../../exceptions/scorm12_exceptions";
+import { scorm12_errors } from "../../constants/error_codes";
 
 /**
  * Class representing the SCORM 1.2 cmi.student_data object
@@ -21,7 +21,14 @@ export class CMIStudentData extends BaseCMI {
     super();
     this.__children = student_data_children
       ? student_data_children
-      : APIConstants.scorm12.student_data_children;
+      : scorm12_constants.student_data_children;
+  }
+
+  /**
+   * Called when the API has been reset
+   */
+  reset(): void {
+    this._initialized = false;
   }
 
   /**
@@ -39,7 +46,7 @@ export class CMIStudentData extends BaseCMI {
    * @private
    */
   set _children(_children: string) {
-    throw new Scorm12ValidationError(ErrorCodes.scorm12.INVALID_SET_VALUE);
+    throw new Scorm12ValidationError(scorm12_errors.INVALID_SET_VALUE);
   }
 
   /**
@@ -56,7 +63,7 @@ export class CMIStudentData extends BaseCMI {
    */
   set mastery_score(mastery_score: string) {
     if (this.initialized) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.READ_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.READ_ONLY_ELEMENT);
     } else {
       this._mastery_score = mastery_score;
     }
@@ -76,7 +83,7 @@ export class CMIStudentData extends BaseCMI {
    */
   set max_time_allowed(max_time_allowed: string) {
     if (this.initialized) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.READ_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.READ_ONLY_ELEMENT);
     } else {
       this._max_time_allowed = max_time_allowed;
     }
@@ -96,7 +103,7 @@ export class CMIStudentData extends BaseCMI {
    */
   set time_limit_action(time_limit_action: string) {
     if (this.initialized) {
-      throw new Scorm12ValidationError(ErrorCodes.scorm12.READ_ONLY_ELEMENT);
+      throw new Scorm12ValidationError(scorm12_errors.READ_ONLY_ELEMENT);
     } else {
       this._time_limit_action = time_limit_action;
     }

package/src/cmi/scorm12/student_preference.ts

@@ -1,9 +1,9 @@
 import { BaseCMI } from "../common/base_cmi";
-import APIConstants from "../../constants/api_constants";
-import { Scorm12ValidationError } from "../../exceptions";
-import ErrorCodes from "../../constants/error_codes";
+import { scorm12_constants } from "../../constants/api_constants";
+import { Scorm12ValidationError } from "../../exceptions/scorm12_exceptions";
 import { check12ValidFormat, check12ValidRange } from "./validation";
-import Regex from "../../constants/regex";
+import { scorm12_regex } from "../../constants/regex";
+import { scorm12_errors } from "../../constants/error_codes";
 
 /**
  * Class representing the SCORM 1.2 cmi.student_preference object
@@ -20,7 +20,7 @@ export class CMIStudentPreference extends BaseCMI {
     super();
     this.__children = student_preference_children
       ? student_preference_children
-      : APIConstants.scorm12.student_preference_children;
+      : scorm12_constants.student_preference_children;
   }
 
   private _audio = "";
@@ -28,6 +28,13 @@ export class CMIStudentPreference extends BaseCMI {
   private _speed = "";
   private _text = "";
 
+  /**
+   * Called when the API has been reset
+   */
+  reset(): void {
+    this._initialized = false;
+  }
+
   /**
    * Getter for __children
    * @return {string}
@@ -43,7 +50,7 @@ export class CMIStudentPreference extends BaseCMI {
    * @private
    */
   set _children(_children: string) {
-    throw new Scorm12ValidationError(ErrorCodes.scorm12.INVALID_SET_VALUE);
+    throw new Scorm12ValidationError(scorm12_errors.INVALID_SET_VALUE);
   }
 
   /**
@@ -60,8 +67,8 @@ export class CMIStudentPreference extends BaseCMI {
    */
   set audio(audio: string) {
     if (
-      check12ValidFormat(audio, Regex.scorm12.CMISInteger) &&
-      check12ValidRange(audio, Regex.scorm12.audio_range)
+      check12ValidFormat(audio, scorm12_regex.CMISInteger) &&
+      check12ValidRange(audio, scorm12_regex.audio_range)
     ) {
       this._audio = audio;
     }
@@ -80,7 +87,7 @@ export class CMIStudentPreference extends BaseCMI {
    * @param {string} language
    */
   set language(language: string) {
-    if (check12ValidFormat(language, Regex.scorm12.CMIString256)) {
+    if (check12ValidFormat(language, scorm12_regex.CMIString256)) {
       this._language = language;
     }
   }
@@ -99,8 +106,8 @@ export class CMIStudentPreference extends BaseCMI {
    */
   set speed(speed: string) {
     if (
-      check12ValidFormat(speed, Regex.scorm12.CMISInteger) &&
-      check12ValidRange(speed, Regex.scorm12.speed_range)
+      check12ValidFormat(speed, scorm12_regex.CMISInteger) &&
+      check12ValidRange(speed, scorm12_regex.speed_range)
     ) {
       this._speed = speed;
     }
@@ -120,8 +127,8 @@ export class CMIStudentPreference extends BaseCMI {
    */
   set text(text: string) {
     if (
-      check12ValidFormat(text, Regex.scorm12.CMISInteger) &&
-      check12ValidRange(text, Regex.scorm12.text_range)
+      check12ValidFormat(text, scorm12_regex.CMISInteger) &&
+      check12ValidRange(text, scorm12_regex.text_range)
     ) {
       this._text = text;
     }