Pad: drop one Promise.all call as saveToDatabase and addPad already use await

This commit is contained in:
webzwo0i 2021-06-18 12:23:32 +02:00
parent c93a6194a2
commit 32806c31fd

View file

@ -92,15 +92,11 @@ Pad.prototype.appendRevision = async function (aChangeset, author) {
newRevData.meta.atext = this.atext;
}
const p = [
db.set(`pad:${this.id}:revs:${newRev}`, newRevData),
this.saveToDatabase(),
];
this.saveToDatabase();
const p_newRevData = db.set(`pad:${this.id}:revs:${newRev}`, newRevData);
// set the author to pad
if (author) {
p.push(authorManager.addPad(author, this.id));
}
if (author) authorManager.addPad(author, this.id);
if (this.head === 0) {
hooks.callAll('padCreate', {pad: this, author});
@ -108,7 +104,7 @@ Pad.prototype.appendRevision = async function (aChangeset, author) {
hooks.callAll('padUpdate', {pad: this, author, revs: newRev, changeset: aChangeset});
}
await Promise.all(p);
await p_newRevData;
};
// save all attributes to the database