From 00e7b04518aba95aac918c0419c8fb31fd745b29 Mon Sep 17 00:00:00 2001
From: Richard Hansen <rhansen@rhansen.org>
Date: Sat, 30 Oct 2021 01:16:17 -0400
Subject: [PATCH] PadMessageHandler: Improve readability of changeset loading

---
 src/node/handler/PadMessageHandler.js | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/node/handler/PadMessageHandler.js b/src/node/handler/PadMessageHandler.js
index c69403670..a005eb922 100644
--- a/src/node/handler/PadMessageHandler.js
+++ b/src/node/handler/PadMessageHandler.js
@@ -905,15 +905,14 @@ const handleClientReady = async (socket, message) => {
       changesets[r] = {};
     }
 
-    // get changesets, author and timestamp needed for pending revisions (in parallel)
-    const promises = [];
-    for (const revNum of revisionsNeeded) {
+    await Promise.all(revisionsNeeded.map(async (revNum) => {
       const cs = changesets[revNum];
-      promises.push(pad.getRevisionChangeset(revNum).then((result) => cs.changeset = result));
-      promises.push(pad.getRevisionAuthor(revNum).then((result) => cs.author = result));
-      promises.push(pad.getRevisionDate(revNum).then((result) => cs.timestamp = result));
-    }
-    await Promise.all(promises);
+      [cs.changeset, cs.author, cs.timestamp] = await Promise.all([
+        pad.getRevisionChangeset(revNum),
+        pad.getRevisionAuthor(revNum),
+        pad.getRevisionDate(revNum),
+      ]);
+    }));
 
     // return pending changesets
     for (const r of revisionsNeeded) {