Fix ReferenceError for module.exports in browser environment (sugarlabs#4229)

BeNikk · web-flow · commit b020cb8b2af4 · 2025-01-05T13:54:54.000-05:00
diff --git a/js/js-export/API/DictBlocksAPI.js b/js/js-export/API/DictBlocksAPI.js
@@ -51,5 +51,6 @@ class DictBlocksAPI {
         return this.runCommand("getValue", [args[1], args[0], this.turIndex]);
     }
 }
-
-module.exports = DictBlocksAPI;
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = DictBlocksAPI;
+}
diff --git a/js/js-export/API/DrumBlocksAPI.js b/js/js-export/API/DrumBlocksAPI.js
@@ -51,4 +51,6 @@ class DrumBlocksAPI {
         return this.runCommand("playNoise", [args[0], this.turIndex]);
     }
 }
-module.exports=DrumBlocksAPI;
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports=DrumBlocksAPI;
+}
diff --git a/js/js-export/API/GraphicsBlocksAPI.js b/js/js-export/API/GraphicsBlocksAPI.js
@@ -86,4 +86,6 @@ class GraphicsBlocksAPI {
         return this.runCommand("doScrollXY", [args[0], args[1]]);
     }
 }
-module.exports = GraphicsBlocksAPI;
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = GraphicsBlocksAPI;
+}
diff --git a/js/js-export/API/IntervalsBlocksAPI.js b/js/js-export/API/IntervalsBlocksAPI.js
@@ -58,4 +58,6 @@ class IntervalsBlocksAPI {
         return this.runCommand("setTemperament", [args[0], args[1], args[2]]);
     }
 }
-module.exports = IntervalsBlocksAPI;
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = IntervalsBlocksAPI;
+}
diff --git a/js/js-export/API/MeterBlocksAPI.js b/js/js-export/API/MeterBlocksAPI.js
@@ -74,6 +74,7 @@ class MeterBlocksAPI {
         return Singer.MeterActions.getNotesPlayed(args[0], this.turIndex);
     }
 }
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = MeterBlocksAPI;
+}
 
-
-module.exports = MeterBlocksAPI;
diff --git a/js/js-export/API/OrnamentBlocksAPI.js b/js/js-export/API/OrnamentBlocksAPI.js
@@ -48,4 +48,6 @@ class OrnamentBlocksAPI {
         return this.ENDFLOWCOMMAND;
     }
 }
-module.exports = OrnamentBlocksAPI;
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = OrnamentBlocksAPI;
+}
diff --git a/js/js-export/API/PenBlocksAPI.js b/js/js-export/API/PenBlocksAPI.js
@@ -99,4 +99,6 @@ class PenBlocksAPI {
         return this.runCommand("doSetFont", [args[0]]);
     }
 }
-module.exports = PenBlocksAPI;
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = PenBlocksAPI;
+}
diff --git a/js/js-export/API/PitchBlocksAPI.js b/js/js-export/API/PitchBlocksAPI.js
@@ -179,4 +179,6 @@ class PitchBlocksAPI {
         return Singer.PitchActions.numToPitch(args[0], "octave", this.turIndex);
     }
 }
-module.exports = PitchBlocksAPI;
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = PitchBlocksAPI;
+}
diff --git a/js/js-export/API/RhythmBlocksAPI.js b/js/js-export/API/RhythmBlocksAPI.js
@@ -113,4 +113,6 @@ class RhythmBlocksAPI {
         return this.ENDFLOWCOMMAND;
     }
 }
-module.exports=RhythmBlocksAPI;
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports=RhythmBlocksAPI;
+}
diff --git a/js/js-export/API/ToneBlocksAPI.js b/js/js-export/API/ToneBlocksAPI.js
@@ -129,4 +129,6 @@ class ToneBlocksAPI {
         return this.ENDFLOWCOMMAND;
     }
 }
-module.exports = ToneBlocksAPI;
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = ToneBlocksAPI;
+}
diff --git a/js/js-export/API/VolumeBlocksAPI.js b/js/js-export/API/VolumeBlocksAPI.js
@@ -58,4 +58,6 @@ class VolumeBlocksAPI {
         return Singer.VolumeActions.getSynthVolume(args[0], this.turIndex);
     }
 }
-module.exports = VolumeBlocksAPI;
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = VolumeBlocksAPI;
+}
diff --git a/js/utils/mathutils.js b/js/utils/mathutils.js
@@ -326,4 +326,6 @@ class MathUtility {
     }
 }
 // Ensure mathutils.js exports the MathUtility class
-module.exports = MathUtility;
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = MathUtility;
+}
diff --git a/js/utils/munsell.js b/js/utils/munsell.js
@@ -6858,7 +6858,9 @@ let searchColors = (r, g, b) => {
 
     return nearestColor;
 };
-module.exports = {interpColor, getMunsellColor, getcolor, searchColors};
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = {interpColor, getMunsellColor, getcolor, searchColors};
+}
 // /**
 //  * @deprecated
 //  * @param {number} r - intensity of red
diff --git a/js/utils/musicutils.js b/js/utils/musicutils.js
@@ -5887,87 +5887,90 @@ const getPitchInfo = (activity, type, currentNote, tur) => {
         console.debug("Waiting for note to play");
     }
 };
-module.exports = {
-    updateTemperaments,
-    scaleDegreeToPitchMapping,
-    buildScale,
-    getNote,
-    getModeLength,
-    nthDegreeToPitch
-    ,getInterval,
-    _calculate_pitch_number,
-    _getStepSize,
-    reducedFraction,
-    toFraction,
-    durationToNoteValue,
-    calcNoteValueToDisplay,
-    noteToPitchOctave,
-    pitchToFrequency,
-    noteIsSolfege,
-    getSolfege,
-    splitSolfege,
-    i18nSolfege,
-    splitScaleDegree,
-    getNumNote,
-    calcOctave,
-    calcOctaveInterval,
-    isInt,
-    convertFromSolfege,
-    convertFactor,
-    getPitchInfo,
-    noteToFrequency,
-    TEMPERAMENT,
-    setOctaveRatio,
-    getOctaveRatio,
-    TEMPERAMENT,
-    TEMPERAMENTS,
-    INITIALTEMPERAMENTS,
-    PreDefinedTemperaments,
-    getTemperamentsList,
-    getTemperament,
-    getTemperamentKeys,
-    addTemperamentToList,
-    deleteTemperamentFromList,
-    addTemperamentToDictionary,
-    updateTemperaments,
-
-    DEFAULTINVERT,
-    DEFAULTMODE,
-    customMode,
-    getInvertMode,
-    getIntervalNumber,
-    getIntervalDirection,
-    getIntervalRatio,
-
-    getModeNumbers,
-    getDrumIndex,
-    getDrumName,
-    getDrumSymbol,
-    getFilterTypes,
-    getOscillatorTypes,
-    getDrumIcon,
-    getDrumSynthName,
-    getNoiseName,
-    getNoiseIcon,
-    getNoiseSynthName,
-    getVoiceName,
-    getVoiceIcon,
-    getVoiceSynthName,
-    isCustomTemperament,
-    getTemperamentName,
-    noteToObj,
-    frequencyToPitch,
-    getArticulation,
-    keySignatureToMode,
-    getScaleAndHalfSteps,
-    modeMapper,
-    getSharpFlatPreference,
-    getCustomNote,
-    pitchToNumber,
-    numberToPitchSharp,
-    getNumber,
-    getNoteFromInterval,
-    numberToPitch,
-    GetNotesForInterval,
-    base64Encode
-};
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = {
+        updateTemperaments,
+        scaleDegreeToPitchMapping,
+        buildScale,
+        getNote,
+        getModeLength,
+        nthDegreeToPitch
+        ,getInterval,
+        _calculate_pitch_number,
+        _getStepSize,
+        reducedFraction,
+        toFraction,
+        durationToNoteValue,
+        calcNoteValueToDisplay,
+        noteToPitchOctave,
+        pitchToFrequency,
+        noteIsSolfege,
+        getSolfege,
+        splitSolfege,
+        i18nSolfege,
+        splitScaleDegree,
+        getNumNote,
+        calcOctave,
+        calcOctaveInterval,
+        isInt,
+        convertFromSolfege,
+        convertFactor,
+        getPitchInfo,
+        noteToFrequency,
+        TEMPERAMENT,
+        setOctaveRatio,
+        getOctaveRatio,
+        TEMPERAMENT,
+        TEMPERAMENTS,
+        INITIALTEMPERAMENTS,
+        PreDefinedTemperaments,
+        getTemperamentsList,
+        getTemperament,
+        getTemperamentKeys,
+        addTemperamentToList,
+        deleteTemperamentFromList,
+        addTemperamentToDictionary,
+        updateTemperaments,
+    
+        DEFAULTINVERT,
+        DEFAULTMODE,
+        customMode,
+        getInvertMode,
+        getIntervalNumber,
+        getIntervalDirection,
+        getIntervalRatio,
+    
+        getModeNumbers,
+        getDrumIndex,
+        getDrumName,
+        getDrumSymbol,
+        getFilterTypes,
+        getOscillatorTypes,
+        getDrumIcon,
+        getDrumSynthName,
+        getNoiseName,
+        getNoiseIcon,
+        getNoiseSynthName,
+        getVoiceName,
+        getVoiceIcon,
+        getVoiceSynthName,
+        isCustomTemperament,
+        getTemperamentName,
+        noteToObj,
+        frequencyToPitch,
+        getArticulation,
+        keySignatureToMode,
+        getScaleAndHalfSteps,
+        modeMapper,
+        getSharpFlatPreference,
+        getCustomNote,
+        pitchToNumber,
+        numberToPitchSharp,
+        getNumber,
+        getNoteFromInterval,
+        numberToPitch,
+        GetNotesForInterval,
+        base64Encode
+    };
+}
+

Original file line number	Diff line number	Diff line change
`@@ -51,5 +51,6 @@ class DictBlocksAPI {`
`51`	`51`	`return this.runCommand("getValue", [args[1], args[0], this.turIndex]);`
`52`	`52`	`}`
`53`	`53`	`}`
`54`		`-`
`55`		`-module.exports = DictBlocksAPI;`
	`54`	`+if (typeof module !== 'undefined' && module.exports) {`
	`55`	`+ module.exports = DictBlocksAPI;`
	`56`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -51,4 +51,6 @@ class DrumBlocksAPI {`
`51`	`51`	`return this.runCommand("playNoise", [args[0], this.turIndex]);`
`52`	`52`	`}`
`53`	`53`	`}`
`54`		`-module.exports=DrumBlocksAPI;`
	`54`	`+if (typeof module !== 'undefined' && module.exports) {`
	`55`	`+ module.exports=DrumBlocksAPI;`
	`56`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -86,4 +86,6 @@ class GraphicsBlocksAPI {`
`86`	`86`	`return this.runCommand("doScrollXY", [args[0], args[1]]);`
`87`	`87`	`}`
`88`	`88`	`}`
`89`		`-module.exports = GraphicsBlocksAPI;`
	`89`	`+if (typeof module !== 'undefined' && module.exports) {`
	`90`	`+ module.exports = GraphicsBlocksAPI;`
	`91`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -58,4 +58,6 @@ class IntervalsBlocksAPI {`
`58`	`58`	`return this.runCommand("setTemperament", [args[0], args[1], args[2]]);`
`59`	`59`	`}`
`60`	`60`	`}`
`61`		`-module.exports = IntervalsBlocksAPI;`
	`61`	`+if (typeof module !== 'undefined' && module.exports) {`
	`62`	`+ module.exports = IntervalsBlocksAPI;`
	`63`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -74,6 +74,7 @@ class MeterBlocksAPI {`
`74`	`74`	`return Singer.MeterActions.getNotesPlayed(args[0], this.turIndex);`
`75`	`75`	`}`
`76`	`76`	`}`
	`77`	`+if (typeof module !== 'undefined' && module.exports) {`
	`78`	`+ module.exports = MeterBlocksAPI;`
	`79`	`+}`
`77`	`80`
`78`		`-`
`79`		`-module.exports = MeterBlocksAPI;`
Original file line number	Diff line number	Diff line change
`@@ -48,4 +48,6 @@ class OrnamentBlocksAPI {`
`48`	`48`	`return this.ENDFLOWCOMMAND;`
`49`	`49`	`}`
`50`	`50`	`}`
`51`		`-module.exports = OrnamentBlocksAPI;`
	`51`	`+if (typeof module !== 'undefined' && module.exports) {`
	`52`	`+ module.exports = OrnamentBlocksAPI;`
	`53`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -99,4 +99,6 @@ class PenBlocksAPI {`
`99`	`99`	`return this.runCommand("doSetFont", [args[0]]);`
`100`	`100`	`}`
`101`	`101`	`}`
`102`		`-module.exports = PenBlocksAPI;`
	`102`	`+if (typeof module !== 'undefined' && module.exports) {`
	`103`	`+ module.exports = PenBlocksAPI;`
	`104`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -179,4 +179,6 @@ class PitchBlocksAPI {`
`179`	`179`	`return Singer.PitchActions.numToPitch(args[0], "octave", this.turIndex);`
`180`	`180`	`}`
`181`	`181`	`}`
`182`		`-module.exports = PitchBlocksAPI;`
	`182`	`+if (typeof module !== 'undefined' && module.exports) {`
	`183`	`+ module.exports = PitchBlocksAPI;`
	`184`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -113,4 +113,6 @@ class RhythmBlocksAPI {`
`113`	`113`	`return this.ENDFLOWCOMMAND;`
`114`	`114`	`}`
`115`	`115`	`}`
`116`		`-module.exports=RhythmBlocksAPI;`
	`116`	`+if (typeof module !== 'undefined' && module.exports) {`
	`117`	`+ module.exports=RhythmBlocksAPI;`
	`118`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -129,4 +129,6 @@ class ToneBlocksAPI {`
`129`	`129`	`return this.ENDFLOWCOMMAND;`
`130`	`130`	`}`
`131`	`131`	`}`
`132`		`-module.exports = ToneBlocksAPI;`
	`132`	`+if (typeof module !== 'undefined' && module.exports) {`
	`133`	`+ module.exports = ToneBlocksAPI;`
	`134`	`+}`