simplified new entity ID allocation

src/entities/api_token.js

@@ -8,9 +8,7 @@ class ApiToken extends Entity {
     static get primaryKeyName() { return "apiTokenId"; }
 
     beforeSaving() {
-        if (!this.apiTokenId) {
+        super.beforeSaving();
-            this.apiTokenId = utils.newApiTokenId();
-        }
 
         if (!this.isDeleted) {
             this.isDeleted = false;

src/entities/branch.js

@@ -13,9 +13,7 @@ class Branch extends Entity {
     }
 
     beforeSaving() {
-        if (!this.branchId) {
+        super.beforeSaving();
-            this.branchId = utils.newBranchId();
-        }
 
         if (!this.isDeleted) {
             this.isDeleted = false;

src/entities/entity.js

@@ -12,6 +12,12 @@ class Entity {
         }
     }
 
+    beforeSaving() {
+        if (!this[this.constructor.primaryKeyName]) {
+            this[this.constructor.primaryKeyName] = utils.newEntityId();
+        }
+    }
+
     async save() {
         await repository.updateEntity(this);
     }

src/entities/image.js

@@ -8,9 +8,7 @@ class Image extends Entity {
     static get primaryKeyName() { return "imageId"; }
 
     beforeSaving() {
-        if (!this.imageId) {
+        super.beforeSaving();
-            this.imageId = utils.newImageId();
-        }
 
         if (!this.isDeleted) {
             this.isDeleted = false;

src/entities/label.js

@@ -14,9 +14,7 @@ class Label extends Entity {
     }
 
     async beforeSaving() {
-        if (!this.labelId) {
+        super.beforeSaving();
-            this.labelId = utils.newLabelId();
-        }
 
         if (!this.value) {
             // null value isn't allowed

src/entities/note.js

@@ -131,9 +131,7 @@ class Note extends Entity {
     }
 
     beforeSaving() {
-        if (!this.noteId) {
+        super.beforeSaving();
-            this.noteId = utils.newNoteId();
-        }
 
         if (this.isJson()) {
             this.content = JSON.stringify(this.jsonContent, null, '\t');

src/entities/note_image.js

@@ -17,9 +17,7 @@ class NoteImage extends Entity {
     }
 
     beforeSaving() {
-        if (!this.noteImageId) {
+        super.beforeSaving();
-            this.noteImageId = utils.newNoteImageId();
-        }
 
         if (!this.isDeleted) {
             this.isDeleted = false;

src/entities/note_revision.js

@@ -22,9 +22,7 @@ class NoteRevision extends Entity {
     }
 
     beforeSaving() {
-        if (!this.noteRevisionId) {
+        super.beforeSaving();
-            this.noteRevisionId = utils.newNoteRevisionId();
-        }
 
         if (this.isProtected) {
             protected_session.encryptNoteRevision(this);

src/services/utils.js

@@ -4,31 +4,7 @@ const crypto = require('crypto');
 const randtoken = require('rand-token').generator({source: 'crypto'});
 const unescape = require('unescape');
 
-function newNoteId() {
+function newEntityId() {
-    return randomString(12);
-}
-
-function newBranchId() {
-    return randomString(12);
-}
-
-function newNoteRevisionId() {
-    return randomString(12);
-}
-
-function newImageId() {
-    return randomString(12);
-}
-
-function newNoteImageId() {
-    return randomString(12);
-}
-
-function newLabelId() {
-    return randomString(12);
-}
-
-function newApiTokenId() {
     return randomString(12);
 }
 
@@ -154,13 +130,7 @@ module.exports = {
     dateStr,
     parseDate,
     parseDateTime,
-    newNoteId,
+    newEntityId,
-    newBranchId,
-    newNoteRevisionId,
-    newImageId,
-    newNoteImageId,
-    newLabelId,
-    newApiTokenId,
     toBase64,
     fromBase64,
     hmac,