Преглед изворни кода

Do not generate update activity if user id is undefined

arvid-e пре 1 недеља
родитељ
комит
a8382b5140
1 измењених фајлова са 6 додато и 4 уклоњено
  1. 6 4
      apps/app/src/server/service/activity/update-activity-logic.ts

+ 6 - 4
apps/app/src/server/service/activity/update-activity-logic.ts

@@ -4,7 +4,7 @@ import mongoose from 'mongoose';
 import { SupportedAction } from '~/interfaces/activity';
 import { SupportedAction } from '~/interfaces/activity';
 
 
 type GenerateUpdatePayload = {
 type GenerateUpdatePayload = {
-  currentUserId: string;
+  currentUserId: string | undefined;
   targetPageId: string;
   targetPageId: string;
   currentActivityId: string;
   currentActivityId: string;
 };
 };
@@ -14,9 +14,13 @@ const SUPPRESION_UPDATE_WINDOW_MS = 5 * 60 * 1000; // 5 min
 
 
 export const shouldGenerateUpdate = async (payload: GenerateUpdatePayload) => {
 export const shouldGenerateUpdate = async (payload: GenerateUpdatePayload) => {
   const { targetPageId, currentActivityId, currentUserId } = payload;
   const { targetPageId, currentActivityId, currentUserId } = payload;
-  const Activity = mongoose.model('Activity');
+
+  if (currentUserId == null) {
+    return false;
+  }
 
 
   // Get most recent update or create activity on the page
   // Get most recent update or create activity on the page
+  const Activity = mongoose.model('Activity');
   const lastContentActivity = await Activity.findOne({
   const lastContentActivity = await Activity.findOne({
     target: targetPageId,
     target: targetPageId,
     action: {
     action: {
@@ -29,9 +33,7 @@ export const shouldGenerateUpdate = async (payload: GenerateUpdatePayload) => {
   }).sort({ createdAt: -1 });
   }).sort({ createdAt: -1 });
 
 
   const isLastActivityByMe =
   const isLastActivityByMe =
-    !!currentUserId &&
     lastContentActivity?.user?._id?.toString() === currentUserId;
     lastContentActivity?.user?._id?.toString() === currentUserId;
-
   const lastActivityTime = lastContentActivity?.createdAt?.getTime?.() ?? 0;
   const lastActivityTime = lastContentActivity?.createdAt?.getTime?.() ?? 0;
   const timeSinceLastActivityMs = Date.now() - lastActivityTime;
   const timeSinceLastActivityMs = Date.now() - lastActivityTime;