Fix: #72 #81 make sure the alphabet is max 20 characters long

bartificer · Apr 6, 2024 · cca4e5d · cca4e5d
1 parent 5f15b07
commit cca4e5d
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/lib/presets.mjs b/src/lib/presets.mjs
@@ -398,6 +398,7 @@ class Presets {
   #current;
   #presetName;
   #presets = Object.keys(thePresets);
+  #max_alphabet = 20;
 
   /**
    * Constructor: set either the default preset
@@ -576,7 +577,11 @@ class Presets {
 
     alphabet = ((is.undefined(alphabet) || (alphabet.length === 0)) ?
       thePresets.DEFAULT.config.symbol_alphabet : alphabet);
-    return is.array(alphabet) ? alphabet.join('') : alphabet;
+
+    // make sure the alphabet is not longer than the max length
+    return is.array(alphabet) ?
+      alphabet.join('').substring(0, this.#max_alphabet) :
+      alphabet.substring(0, this.#max_alphabet);
   }
 
   /**
@@ -599,7 +604,12 @@ class Presets {
             tmpConfig.symbol_alphabet);
     alphabet = ((is.undefined(alphabet) || (alphabet.length === 0)) ?
       thePresets.DEFAULT.config.symbol_alphabet : alphabet);
-    return is.array(alphabet) ? alphabet.join('') : alphabet;
+
+    // make sure the alphabet is not longer than the max length
+    return is.array(alphabet) ?
+      alphabet.join('').substring(0, this.#max_alphabet) :
+      alphabet.substring(0, this.#max_alphabet);
+
   }
 
   /**