Add reactiondeck limit variable and improve sql

app/controllers/api/v1/reaction_deck_controller.rb

@@ -20,9 +20,16 @@ class Api::V1::ReactionDeckController < Api::BaseController
   def create
     deck = []
 
+    shortcodes = []
     (deck_params['emojis'] || []).each do |shortcode|
-      shortcode = shortcode.delete(':')
+      shortcodes << shortcode.delete(':')
-      custom_emoji = CustomEmoji.find_by(shortcode: shortcode, domain: nil)
+      break if shortcodes.length >= User::REACTION_DECK_MAX
+    end
+
+    custom_emojis = CustomEmoji.where(shortcode: shortcodes, domain: nil)
+
+    shortcodes.each do |shortcode|
+      custom_emoji = custom_emojis.find { |em| em.shortcode == shortcode }
 
       emoji_data = {}
 

app/models/user.rb

@@ -65,6 +65,8 @@ class User < ApplicationRecord
   # to check their feed
   ACTIVE_DURATION = ENV.fetch('USER_ACTIVE_DAYS', 7).to_i.days.freeze
 
+  REACTION_DECK_MAX = 256
+
   devise :two_factor_authenticatable,
          otp_secret_encryption_key: Rails.configuration.x.otp_secret
 

app/serializers/rest/instance_serializer.rb

@@ -86,7 +86,7 @@ class REST::InstanceSerializer < ActiveModel::Serializer
       },
 
       reaction_deck: {
-        max_emojis: 32_767,
+        max_emojis: User::REACTION_DECK_MAX,
       },
 
       reactions: {

app/serializers/rest/v1/instance_serializer.rb

@@ -92,7 +92,7 @@ class REST::V1::InstanceSerializer < ActiveModel::Serializer
       },
 
       reaction_deck: {
-        max_emojis: 32_767,
+        max_emojis: User::REACTION_DECK_MAX,
       },
 
       reactions: {