Merge pull request #6711 from vpereira/service_for_architectures_Cont…

…roller

Move code to service

src/api/app/controllers/webui/architectures_controller.rb

@@ -7,17 +7,12 @@ def index
   end
 
   def bulk_update_availability
-    all_valid = true
-    params[:archs].each do |name, value|
-      arch = Architecture.find_by(name: name)
-      arch.available = value
-      all_valid &&= arch.save
-    end
+    result = ::ArchitecturesControllerService::ArchitectureUpdater.new(params[:archs]).call
 
     ::Configuration.write_to_backend
 
     respond_to do |format|
-      if all_valid
+      if result.valid?
         format.html { redirect_to architectures_path, notice: 'Architectures successfully updated.' }
       else
         format.html { redirect_back(fallback_location: root_path, error: 'Not all architectures could be saved') }

src/api/app/services/architectures_controller_service/architecture_updater.rb

@@ -0,0 +1,21 @@
+module ArchitecturesControllerService
+  class ArchitectureUpdater
+    def initialize(archs)
+      @archs = archs
+      @all_valid = true
+    end
+
+    def call
+      @archs.each do |name, value|
+        arch = Architecture.find_by(name: name)
+        arch.available = value
+        @all_valid &&= arch.save
+      end
+      self
+    end
+
+    def valid?
+      @all_valid
+    end
+  end
+end