Do not build whitelist in every request

openpoke · Oct 6, 2020 · 008a044 · 008a044
1 parent 588ab31
commit 008a044
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/app/permissions/decidim/action_delegator/permissions.rb b/app/permissions/decidim/action_delegator/permissions.rb
@@ -3,6 +3,8 @@
 module Decidim
   module ActionDelegator
     class Permissions < Decidim::DefaultPermissions
+      SUBJECTS_WHITELIST = [:delegation, :setting, :consultation].freeze
+
       def permissions
         return permission_action unless user.admin?
         return permission_action unless permission_action.scope == :admin
@@ -20,7 +22,7 @@ def consultation_results_exports_action?
       end
 
       def action_delegator_subject?
-        [:delegation, :setting, :consultation].include?(permission_action.subject)
+        SUBJECTS_WHITELIST.include?(permission_action.subject)
       end
 
       def can_perform_action?