tweaked some of the cache logic

Author: fatfingers23

server/api/social/like.delete.ts

@@ -33,7 +33,9 @@ export default eventHandlerWithOAuthSession(async (event, oAuthSession) => {
     return result
   }
 
-  console.error(
+  console.warn(
     `User ${loggedInUsersDid} tried to unlike a package ${body.packageName} but it was not liked by them.`,
   )
+
+  return await likesUtil.getLikes(body.packageName, loggedInUsersDid)
 })

server/api/social/like.post.ts

@@ -20,12 +20,10 @@ export default eventHandlerWithOAuthSession(async (event, oAuthSession) => {
 
   const likesUtil = new PackageLikesUtils()
 
-  const hasLiked = await likesUtil.hasTheUserLikedThePackage(body.packageName, loggedInUsersDid)
-  if (hasLiked) {
-    throw createError({
-      status: 400,
-      message: 'User has already liked the package',
-    })
+  // Checks to see if the user has liked the package already
+  const likesResult = await likesUtil.getLikes(body.packageName, loggedInUsersDid)
+  if (likesResult.userHasLiked) {
+    return likesResult
   }
 
   const subjectRef = PACKAGE_SUBJECT_REF(body.packageName)

server/utils/atproto/utils/likes.ts

@@ -230,11 +230,11 @@ export class PackageLikesUtils {
     if (!totalLikes) {
       totalLikes = await this.constellationLikes(subjectRef)
     }
-    totalLikes = totalLikes - 1
+    totalLikes = Math.max(totalLikes - 1, 0)
     await this.cache.set(totalLikesKey, totalLikes, CACHE_MAX_AGE)
 
     //Clean up
-    await this.cache.delete(CACHE_USER_LIKES_KEY(packageName, usersDid))
+    await this.cache.set(CACHE_USER_LIKES_KEY(packageName, usersDid), false, CACHE_MAX_AGE)
     await this.cache.delete(CACHE_USERS_BACK_LINK(packageName, usersDid))
 
     return {