PERF: static id for like post action type (#112)

Before flags were in the database, PostActionType.types were stored in memory and getting PostActionType.types[:like] was a very cheap operation.

Now, PostActionType is cached in Redis and we should try to avoid sending too many requests, especially that id for like is always 2.

plugin.rb

@@ -865,7 +865,12 @@
     post = activity.object.stored.model
 
     if user && post
-      PostActionCreator.new(user, post, PostActionType.types[:like], reason: :activity_pub).perform
+      PostActionCreator.new(
+        user,
+        post,
+        PostActionType::LIKE_POST_ACTION_ID,
+        reason: :activity_pub,
+      ).perform
     end
   end
 

spec/lib/discourse_activity_pub/ap/activity/like_spec.rb

@@ -46,7 +46,7 @@
             PostAction.exists?(
               post_id: post.id,
               user_id: @user.id,
-              post_action_type_id: PostActionType.types[:like],
+              post_action_type_id: PostActionType::LIKE_POST_ACTION_ID,
             ),
           ).to be(true)
         end
@@ -84,7 +84,7 @@
             PostAction.exists?(
               post_id: post.id,
               user_id: @user.id,
-              post_action_type_id: PostActionType.types[:like],
+              post_action_type_id: PostActionType::LIKE_POST_ACTION_ID,
             ),
           ).to be(true)
         end

spec/lib/discourse_activity_pub/ap/activity/undo_spec.rb

@@ -47,7 +47,7 @@
             :post_action,
             user: user,
             post: post,
-            post_action_type_id: PostActionType.types[:like],
+            post_action_type_id: PostActionType::LIKE_POST_ACTION_ID,
           )
         end
         let!(:person) { Fabricate(:discourse_activity_pub_actor_person, model: user) }
@@ -64,7 +64,7 @@
             PostAction.exists?(
               post_id: post.id,
               user_id: user.id,
-              post_action_type_id: PostActionType.types[:like],
+              post_action_type_id: PostActionType::LIKE_POST_ACTION_ID,
             ),
           ).to be(false)
         end

spec/lib/post_action_destroyer_spec.rb

@@ -16,15 +16,15 @@
       :post_action,
       user: user2,
       post: post,
-      post_action_type_id: PostActionType.types[:like],
+      post_action_type_id: PostActionType::LIKE_POST_ACTION_ID,
     )
   end
   let!(:post_action2) do
     Fabricate(
       :post_action,
       user: user3,
       post: post,
-      post_action_type_id: PostActionType.types[:like],
+      post_action_type_id: PostActionType::LIKE_POST_ACTION_ID,
     )
   end
   let!(:like1) { Fabricate(:discourse_activity_pub_activity_like, actor: actor2, object: note) }

spec/models/post_action_spec.rb

@@ -25,7 +25,7 @@
       :post_action,
       user: user2,
       post: post,
-      post_action_type_id: PostActionType.types[:like],
+      post_action_type_id: PostActionType::LIKE_POST_ACTION_ID,
     )
   end