Changing request to params in controllers

Author: A-Wheeto

app/controllers/class_marker/webhooks_controller.rb

@@ -4,7 +4,7 @@ class ClassMarker::WebhooksController < ApplicationController
   before_action :verify_hmac_signature
 
   def assessment
-    UpdateUserAssessmentAttemptFromClassMarkerJob.perform_later(request[:test][:test_id], request[:result][:cm_user_id], request[:result][:percentage])
+    UpdateUserAssessmentAttemptFromClassMarkerJob.perform_later(params[:test][:test_id], params[:result][:cm_user_id], params[:result][:percentage])
     head :ok
   end
 

app/controllers/concerns/curriculum/rateable.rb

@@ -33,9 +33,9 @@ def comment
       raise NoMethodError unless respond_to?(:client, true)
 
       response = client.update_rating(
-        id: request[:rating_id],
+        id: params[:rating_id],
         key: :comment,
-        value: request[:comment].gsub('"', '\"')
+        value: params[:comment].gsub('"', '\"')
       )
 
       render json: {
@@ -48,11 +48,11 @@ def choices
       raise NoMethodError unless respond_to?(:client, true)
 
       response = client.update_rating(
-        id: request[:rating_id],
+        id: params[:rating_id],
         key: :choices,
-        value: request[:rating_choices]
+        value: params[:rating_choices]
       )
-      response.rating_id = request[:rating_id]
+      response.rating_id = params[:rating_id]
       render json: {
         origin: __method__.to_s,
         data: response

app/controllers/dynamics/webhooks_controller.rb

@@ -4,7 +4,7 @@ class WebhooksController < ApplicationController
     before_action :verify_bearer_token
 
     def user
-      stem_achiever_contact_no = request[:stem_achiever_contact_no]&.downcase
+      stem_achiever_contact_no = params[:stem_achiever_contact_no]&.downcase
       user = User.find_by(stem_achiever_contact_no:)
 
       Sentry.capture_message("Dynamics webhook failed as no user found with contact_no: #{stem_achiever_contact_no || "nil"}") unless user