Do not use action as "local" variable

buildresult is a controller action, re-using it as variable is a desaster bound to happen...
openSUSE · Sep 21, 2023 · 5101acf · 5101acf
1 parent e41741a
commit 5101acf
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/api/app/controllers/webui/project_controller.rb b/src/api/app/controllers/webui/project_controller.rb
@@ -310,12 +310,12 @@ def move_path
   end
 
   def monitor
-    unless (buildresult = monitor_buildresult)
+    unless (build_results = monitor_buildresult)
       @buildresult_unavailable = true
       return
     end
 
-    monitor_parse_buildresult(buildresult)
+    monitor_parse_buildresult(build_results)
 
     # extract repos
     repohash = {}