[REM] UUID: remove fast strategy

THe fast strategy for uuid generator was introduced to speed up the
startup of a model, specifically when generating cells which could be
numerous and the calls to crypto could become a hassle.
However, we realised later on that a uuid was useless for that specific
purpose. More specifically, a uuid is useful to generate unique
identifiers that will be shared accross users to avoid collisions in
multi-user context (i.e. concurrent updates).
In other cases, shortcutting the uuid generator to become a simple
incremented integer value is either wrong or shows the uselessness of
calling uuidGenerator in the first place.

Task: 4216816
Part-of: #5030
Signed-off-by: Vincent Schippefilt (vsc) <[email protected]>

src/helpers/uuid.ts

@@ -3,20 +3,9 @@
  * */
 
 export class UuidGenerator {
-  private isFastIdStrategy = false;
-
-  private fastIdStart = 0;
-
-  setIsFastStrategy(isFast: boolean) {
-    this.isFastIdStrategy = isFast;
-  }
-
   uuidv4(): string {
-    if (this.isFastIdStrategy) {
-      this.fastIdStart++;
-      return String(this.fastIdStart);
-      //@ts-ignore
-    } else if (window.crypto && window.crypto.getRandomValues) {
+    //@ts-ignore
+    if (window.crypto && window.crypto.getRandomValues) {
       //@ts-ignore
       return ([1e7] + -1e3 + -4e3 + -8e3 + -1e11).replace(/[018]/g, (c) =>
         (c ^ (crypto.getRandomValues(new Uint8Array(1))[0] & (15 >> (c / 4)))).toString(16)

src/model.ts

@@ -266,7 +266,6 @@ export class Model extends EventBus<any> implements CommandDispatcher {
       this.handlers.push(plugin);
       this.uiHandlers.push(plugin);
     }
-    this.uuidGenerator.setIsFastStrategy(false);
 
     // starting plugins
     this.dispatch("START");

tests/__mocks__/uuid.ts

@@ -1,8 +1,6 @@
 export class UuidGenerator {
   private nextId = 1;
 
-  setIsFastStrategy(isFast: boolean) {}
-
   uuidv4(): string {
     return String(this.nextId++);
   }