Appease Credo

mbta · Sep 18, 2023 · 77528d2 · 77528d2
1 parent 954fa5c
commit 77528d2
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 21 deletions.
diff --git a/lib/screens/log_screen_data.ex b/lib/screens/log_screen_data.ex
@@ -17,14 +17,17 @@ defmodule Screens.LogScreenData do
         screen_id,
         last_refresh,
         is_screen,
-        requestor,
-        screen_side \\ nil,
-        rotation_index \\ nil,
-        triptych_pane \\ nil,
-        triptych_player_name \\ nil,
-        ofm_app_package_version \\ nil
+        params
       ) do
+    requestor = params["requestor"]
+
     if is_screen or not is_nil(requestor) do
+      screen_side = params["screen_side"]
+      rotation_index = params["rotation_index"]
+      triptych_pane = params["pane"]
+      triptych_player_name = params["player_name"]
+      ofm_app_package_version = params["version"]
+
       data =
         %{
           screen_id: screen_id,

diff --git a/lib/screens_web/controllers/screen_api_controller.ex b/lib/screens_web/controllers/screen_api_controller.ex
@@ -34,7 +34,7 @@ defmodule ScreensWeb.ScreenApiController do
         screen_id,
         last_refresh,
         is_screen,
-        params["requestor"]
+        params
       )
 
     if nonexistent_screen?(screen_id) do
@@ -60,9 +60,7 @@ defmodule ScreensWeb.ScreenApiController do
         screen_id,
         nil,
         is_screen,
-        params["requestor"],
-        nil,
-        rotation_index
+        params
       )
 
     if nonexistent_screen?(screen_id) do

diff --git a/lib/screens_web/controllers/v2/screen_api_controller.ex b/lib/screens_web/controllers/v2/screen_api_controller.ex
@@ -22,21 +22,13 @@ defmodule ScreensWeb.V2.ScreenApiController do
   def show(conn, %{"id" => screen_id, "last_refresh" => last_refresh} = params) do
     is_screen = ScreensWeb.UserAgent.is_screen_conn?(conn, screen_id)
     screen_side = params["screen_side"]
-    rotation_index = params["rotation_index"]
     triptych_pane = params["pane"]
-    triptych_player_name = params["player_name"]
-    ofm_app_package_version = params["version"]
 
     LogScreenData.log_data_request(
       screen_id,
       last_refresh,
       is_screen,
-      params["requestor"],
-      screen_side,
-      rotation_index,
-      triptych_pane,
-      triptych_player_name,
-      ofm_app_package_version
+      params
     )
 
     cond do
@@ -117,8 +109,7 @@ defmodule ScreensWeb.V2.ScreenApiController do
       screen_id,
       last_refresh,
       false,
-      params["requestor"],
-      params["screen_side"]
+      params
     )
 
     cond do