From eb9a1633944d84cbefa727b18a129feff27c9f56 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Fri, 1 Mar 2024 11:04:30 +0100
Subject: [PATCH] Making code more readable

---
 .../backend-core/src/cache/docWritethrough.ts     | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/packages/backend-core/src/cache/docWritethrough.ts b/packages/backend-core/src/cache/docWritethrough.ts
index 80063e4772..5148950c1d 100644
--- a/packages/backend-core/src/cache/docWritethrough.ts
+++ b/packages/backend-core/src/cache/docWritethrough.ts
@@ -15,7 +15,7 @@ async function getCache() {
 }
 
 interface CacheItem {
-  lastWrite: number
+  nextWrite: number
 }
 
 export class DocWritethrough {
@@ -40,8 +40,8 @@ export class DocWritethrough {
     return this._docId
   }
 
-  private makeCacheItem(): CacheItem {
-    return { lastWrite: Date.now() }
+  private makeNextWriteInfoItem(): CacheItem {
+    return { nextWrite: Date.now() + this.writeRateMs }
   }
 
   async patch(data: Record<string, any>) {
@@ -62,7 +62,10 @@ export class DocWritethrough {
         async () => {
           if (await this.shouldUpdateDb(cache)) {
             await this.persistToDb(cache)
-            await cache.store(this.docInfoCacheKey, this.makeCacheItem())
+            await cache.store(
+              this.docInfoCacheKey,
+              this.makeNextWriteInfoItem()
+            )
           }
         }
       )
@@ -75,9 +78,9 @@ export class DocWritethrough {
 
   private async shouldUpdateDb(cache: BaseCache) {
     const cacheItem = await cache.withCache(this.docInfoCacheKey, null, () =>
-      this.makeCacheItem()
+      this.makeNextWriteInfoItem()
     )
-    return cacheItem.lastWrite <= Date.now() - this.writeRateMs
+    return Date.now() >= cacheItem.nextWrite
   }
 
   private async storeToCache(cache: BaseCache, data: Record<string, any>) {