fix undeleting note when its relation points to yet-undeleted note

src/becca/entities/abstract_becca_entity.js

@@ -80,14 +80,14 @@ class AbstractBeccaEntity {
      *
      * @returns {this}
      */
-    save() {
+    save(opts = {}) {
         const entityName = this.constructor.entityName;
         const primaryKeyName = this.constructor.primaryKeyName;
 
         const isNewEntity = !this[primaryKeyName];
 
         if (this.beforeSaving) {
-            this.beforeSaving();
+            this.beforeSaving(opts);
         }
 
         const pojo = this.getPojoToSave();

src/becca/entities/battribute.js

@@ -176,8 +176,10 @@ class BAttribute extends AbstractBeccaEntity {
         return !(this.attributeId in this.becca.attributes);
     }
 
-    beforeSaving() {
+    beforeSaving(opts = {}) {
-        this.validate();
+        if (!opts.skipValidation) {
+            this.validate();
+        }
 
         this.name = sanitizeAttributeName(this.name);
 

src/services/notes.js

@@ -670,7 +670,8 @@ function undeleteBranch(branchId, deleteId, taskContext) {
                            OR (type = 'relation' AND value = ?))`, [deleteId, note.noteId, note.noteId]);
 
         for (const attribute of attributes) {
-            new BAttribute(attribute).save();
+            // relation might point to a note which hasn't been undeleted yet and would thus throw up
+            new BAttribute(attribute).save({skipValidation: true});
         }
 
         const noteAttachments = sql.getRows(`