Fix bug

Author: holzmaster

src/commands/penis.ts

@@ -138,9 +138,14 @@ export default class PenisCommand implements MessageCommand {
     }
 
     async #getOrCreateMeasurement(userToMeasure: User): Promise<Penis> {
-        const recentMeasurement = await penis.fetchRecentMeasurement(userToMeasure);
-        if (recentMeasurement !== undefined) {
-            return recentMeasurement;
+        const lastMeasurement = await penis.fetchLastMeasurement(userToMeasure);
+        if (lastMeasurement !== undefined) {
+            const now = new Date();
+            const measurement = new Date(`${lastMeasurement.measuredAt}Z`);
+            // TODO: Make use of temporal lol
+            if (measurement.toISOString().split("T")[0] === now.toISOString().split("T")[0]) {
+                return lastMeasurement;
+            }
         }
 
         log.debug(`No recent measuring of ${userToMeasure.id} found. Creating Measurement.`);

src/storage/penis.ts

@@ -26,15 +26,12 @@ export function insertMeasurement(
         .executeTakeFirstOrThrow();
 }
 
-export function fetchRecentMeasurement(user: User, ctx = db()): Promise<Penis | undefined> {
-    const now = Temporal.Now.instant();
-    const { startOfToday, startOfTomorrow } = getStartAndEndDay(now);
-
+export function fetchLastMeasurement(user: User, ctx = db()): Promise<Penis | undefined> {
     return ctx
         .selectFrom("penis")
         .where("userId", "=", user.id)
-        .where("measuredAt", ">=", startOfToday.toString())
-        .where("measuredAt", "<", startOfTomorrow.toString())
+        .orderBy("id", "desc")
+        .limit(1)
         .selectAll()
         .executeTakeFirst();
 }