From 5e6f283032449bc708453d220c6a5e8872d994ef Mon Sep 17 00:00:00 2001
From: fantasticit <fantasticit@163.com>
Date: Wed, 25 May 2022 10:00:59 +0800
Subject: [PATCH] server: fix view record time

---
 packages/server/src/services/view.service.ts | 28 ++++++++++++--------
 1 file changed, 17 insertions(+), 11 deletions(-)

diff --git a/packages/server/src/services/view.service.ts b/packages/server/src/services/view.service.ts
index efafc635..f9b8936d 100644
--- a/packages/server/src/services/view.service.ts
+++ b/packages/server/src/services/view.service.ts
@@ -19,16 +19,22 @@ export class ViewService {
    * @returns
    */
   async create({ userId = 'public', documentId, userAgent }) {
-    const data = {
-      userId,
-      documentId,
-      originUserAgent: userAgent,
-      parsedUserAgent: parseUserAgent(userAgent).text,
-    };
-
-    const res = await this.viewRepo.create(data);
-    const ret = await this.viewRepo.save(res);
+    const old = await this.viewRepo.findOne({ documentId, userId });
+    let data;
 
+    if (old) {
+      data = await this.viewRepo.merge(old, {
+        updatedAt: convertDateToMysqlTimestamp(Date.now()),
+      });
+    } else {
+      data = await this.viewRepo.create({
+        userId,
+        documentId,
+        originUserAgent: userAgent,
+        parsedUserAgent: parseUserAgent(userAgent).text,
+      });
+    }
+    const ret = await this.viewRepo.save(data);
     return ret;
   }
 
@@ -74,10 +80,10 @@ export class ViewService {
     const count = 20;
 
     const ret = await this.viewRepo.query(
-      `SELECT documentId, ANY_VALUE(created_at) as visitedAt
+      `SELECT documentId, ANY_VALUE(updated_at) as visitedAt
        FROM view 
        WHERE view.userId = '${userId}'
-       AND (view.created_at BETWEEN '${from}' AND '${end}')
+       AND (view.updated_at BETWEEN '${from}' AND '${end}')
        GROUP BY documentId
        LIMIT ${count}
       `