Make sure that the kind matches the type

openSUSE · Apr 19, 2024 · f848427 · f848427
1 parent 990ef08
commit f848427
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/api/app/controllers/webui/decisions_controller.rb b/src/api/app/controllers/webui/decisions_controller.rb
@@ -19,6 +19,8 @@ def create
   private
 
   def decision_params
-    params.require(:decision).permit(:reason, :type, report_ids: [])
+    # TODO: remove merge and replace decision_kind with decision_type
+    kind = params[:decision][:type] == 'DecisionFavored' ? 'favor' : 'cleared'
+    params.require(:decision).permit(:reason, :type, report_ids: []).merge(kind: kind)
   end
 end