foreman_datacenter 0.1.54 → 0.1.55

data/app/controllers/foreman_datacenter/interface_templates_controller.rb

@@ -1,6 +1,8 @@
 module ForemanDatacenter
   class InterfaceTemplatesController < ApplicationController
-    before_action :set_interface_template, only: [:destroy]
+    include ForemanDatacenter::Controller::Parameters::InterfaceTemplate
+
+    before_action :find_resource, only: [:destroy]
 
     def new
       @interface_template = InterfaceTemplate.new(
@@ -19,11 +21,10 @@ module ForemanDatacenter
     end
 
     def create
-      @interface_template = InterfaceTemplate.new(interface_template_params)
+      @interface_template = InterfaceTemplate.new(interface_template_params.merge(device_type_id: params[:device_type_id]))
 
       if @interface_template.save
-        redirect_to device_type_url(@interface_template.device_type),
-                    notice: 'New interface template was successfully created'
+        process_success success_redirect: device_type_url(@interface_template.device_type_id)
       else
         process_error object: @interface_template
       end
@@ -31,22 +32,10 @@ module ForemanDatacenter
 
     def destroy
       if @interface_template.destroy
-        redirect_to device_type_url(@interface_template.device_type),
-                    notice: 'Interface template was successfully destroyed'
+        process_success success_redirect: device_type_url(params[:device_type_id])
       else
         process_error object: @interface_template
       end
     end
-
-    private
-
-    def set_interface_template
-      @interface_template = InterfaceTemplate.find(params[:id])
-    end
-
-    def interface_template_params
-      params[:interface_template].
-        permit(:device_type_id, :name, :form_factor, :mgmt_only)
-    end
   end
 end

data/app/controllers/foreman_datacenter/management_devices_controller.rb

@@ -1,6 +1,8 @@
 module ForemanDatacenter
-  class ManagementDevicesController < ApplicationController
-    before_action :set_management_device, only: [:edit, :update, :destroy]
+  class ManagementDevicesController < ForemanDatacenter::ApplicationController
+    include ForemanDatacenter::Controller::Parameters::ManagementDevice
+
+    before_action :find_resource, only: [:edit, :update, :destroy]
 
     def new
       device = Device.find(params[:device_id])
@@ -11,7 +13,7 @@ module ForemanDatacenter
     end
 
     def create
-      @management_device = ManagementDevice.new(management_device_params)
+      @management_device = ManagementDevice.new(management_device_params.merge(device_id: params[:device_id]))
 
       if @management_device.save
         redirect_to device_url(@management_device.device),
@@ -38,16 +40,5 @@ module ForemanDatacenter
         process_error object: @management_device
       end
     end
-
-    private
-
-    def set_management_device
-      @management_device = ManagementDevice.find(params[:id])
-    end
-
-    def management_device_params
-      params[:management_device].
-        permit(:device_id, :console_url, :login, :password)
-    end
   end
 end

data/app/controllers/foreman_datacenter/manufacturers_controller.rb

@@ -1,9 +1,12 @@
 module ForemanDatacenter
-  class ManufacturersController < ApplicationController
-    before_action :set_manufacturer, only: [:show, :edit, :update, :destroy]
+  class ManufacturersController < ForemanDatacenter::ApplicationController
+    include Foreman::Controller::AutoCompleteSearch
+    include ForemanDatacenter::Controller::Parameters::Manufacturer
+
+    before_action :find_resource, only: [:show, :edit, :update, :destroy]
 
     def index
-      @manufacturers = Manufacturer.all
+      @manufacturers = resource_base_search_and_page.includes(:device_types)
     end
 
     def show
@@ -35,21 +38,17 @@ module ForemanDatacenter
     end
 
     def destroy
-      if @manufacturer.destroy
-        process_success object: @manufacturer
+      unless params['object_only']
+        @manufacturer.device_types.each { |dt| dt.devices.each { |d| d.destroy }; dt.destroy }
       else
-        process_error object: @manufacturer
+        @manufacturer.device_types.delete_all(:nullify)
       end
-    end
-
-    private
 
-    def set_manufacturer
-      @manufacturer = Manufacturer.find(params[:id])
-    end
-
-    def manufacturer_params
-      params[:manufacturer].permit(:name)
+      if @manufacturer.destroy
+        process_success success_redirect: manufacturers_path
+      else
+        process_error
+      end
     end
   end
 end

data/app/controllers/foreman_datacenter/platforms_controller.rb

@@ -1,9 +1,12 @@
 module ForemanDatacenter
-  class PlatformsController < ApplicationController
-    before_action :set_platform, only: [:show, :edit, :update, :destroy]
+  class PlatformsController < ForemanDatacenter::ApplicationController
+    include Foreman::Controller::AutoCompleteSearch
+    include ForemanDatacenter::Controller::Parameters::Platform
+
+    before_action :find_resource, only: [:show, :edit, :update, :destroy]
 
     def index
-      @platforms = Platform.includes(:devices).all
+      @platforms = resource_base_search_and_page
     end
 
     def show
@@ -35,21 +38,16 @@ module ForemanDatacenter
     end
 
     def destroy
+      unless params['object_only']
+        @platform.devices.each { |d| d.destroy }
+      else
+        @platform.devices.delete_all(:nullify)
+      end
       if @platform.destroy
-        process_success object: @platform
+        process_success success_redirect: platforms_path
       else
         process_error object: @platform
       end
     end
-
-    private
-
-    def set_platform
-      @platform = Platform.find(params[:id])
-    end
-
-    def platform_params
-      params[:platform].permit(:name, :rpc_client)
-    end
   end
 end

data/app/controllers/foreman_datacenter/power_outlet_templates_controller.rb

@@ -1,6 +1,8 @@
 module ForemanDatacenter
-  class PowerOutletTemplatesController < ApplicationController
-    before_action :set_power_outlet_template, only: [:destroy]
+  class PowerOutletTemplatesController < ForemanDatacenter::ApplicationController
+    include ForemanDatacenter::Controller::Parameters::PowerOutletTemplate
+
+    before_action :find_resource, only: [:destroy]
 
     def new
       @power_outlet_template = PowerOutletTemplate.new(
@@ -9,11 +11,10 @@ module ForemanDatacenter
     end
 
     def create
-      @power_outlet_template = PowerOutletTemplate.new(power_outlet_template_params)
+      @power_outlet_template = PowerOutletTemplate.new(power_outlet_template_params.merge(device_type_id: params[:device_type_id]))
 
       if @power_outlet_template.save
-        redirect_to device_type_url(@power_outlet_template.device_type),
-                    notice: 'New power outlet template was successfully created'
+        process_success success_redirect: device_type_url(@power_outlet_template.device_type_id)
       else
         process_error object: @power_outlet_template
       end
@@ -21,21 +22,10 @@ module ForemanDatacenter
 
     def destroy
       if @power_outlet_template.destroy
-        redirect_to device_type_url(@power_outlet_template.device_type),
-                    notice: 'Power outlet template was successfully destroyed'
+        process_success success_redirect: device_type_url(params[:device_type_id])
       else
         process_error object: @power_outlet_template
       end
     end
-
-    private
-
-    def set_power_outlet_template
-      @power_outlet_template = PowerOutletTemplate.find(params[:id])
-    end
-
-    def power_outlet_template_params
-      params[:power_outlet_template].permit(:device_type_id, :name)
-    end
   end
 end

data/app/controllers/foreman_datacenter/power_outlets_controller.rb

@@ -1,7 +1,10 @@
 module ForemanDatacenter
-  class PowerOutletsController < ApplicationController
-    before_action :set_power_outlet, only: [:edit, :update, :destroy, :new_connection,
-                                            :connect, :disconnect]
+  class PowerOutletsController < ForemanDatacenter::ApplicationController
+    include ForemanDatacenter::Controller::Parameters::PowerOutlet
+
+    before_action :find_resource, only: [:edit, :update, :destroy,
+                                         :new_connection,
+                                         :connect, :disconnect]
 
     def new
       device = Device.find(params[:device_id])
@@ -12,7 +15,7 @@ module ForemanDatacenter
     end
 
     def create
-      @power_outlet = PowerOutlet.new(power_outlet_params)
+      @power_outlet = PowerOutlet.new(power_outlet_params.merge(device_id: params[:device_id]))
 
       if @power_outlet.save
         redirect_to device_url(id: @power_outlet.device_id),
@@ -63,15 +66,5 @@ module ForemanDatacenter
       @device = Device.find(params[:device_id])
       render partial: 'for_device'
     end
-
-    private
-
-    def set_power_outlet
-      @power_outlet = PowerOutlet.find(params[:id])
-    end
-
-    def power_outlet_params
-      params[:power_outlet].permit(:device_id, :name)
-    end
   end
 end

data/app/controllers/foreman_datacenter/power_port_templates_controller.rb

@@ -1,19 +1,20 @@
 module ForemanDatacenter
-  class PowerPortTemplatesController < ApplicationController
-    before_action :set_power_port_template, only: [:destroy]
+  class PowerPortTemplatesController < ForemanDatacenter::ApplicationController
+    include ForemanDatacenter::Controller::Parameters::PowerPortTemplate
+
+    before_action :find_resource, only: [:destroy]
 
     def new
-      @power_port_template = PowerPortTemplate.new(
-        device_type: DeviceType.find(params[:device_type_id]),
+      @power_port_template = ForemanDatacenter::PowerPortTemplate.new(
+        device_type: ForemanDatacenter::DeviceType.find(params[:device_type_id]),
       )
     end
 
     def create
-      @power_port_template = PowerPortTemplate.new(power_port_template_params)
+      @power_port_template = ForemanDatacenter::PowerPortTemplate.new(power_port_template_params.merge(device_type_id: params[:device_type_id]))
 
       if @power_port_template.save
-        redirect_to device_type_url(@power_port_template.device_type),
-                    notice: 'New power port template was successfully created'
+        process_success success_redirect: device_type_url(@power_port_template.device_type_id)
       else
         process_error object: @power_port_template
       end
@@ -21,21 +22,10 @@ module ForemanDatacenter
 
     def destroy
       if @power_port_template.destroy
-        redirect_to device_type_url(@power_port_template.device_type),
-                    notice: 'Power port template was successfully destroyed'
+        process_success success_redirect: device_type_url(params[:device_type_id])
       else
         process_error object: @power_port_template
       end
     end
-
-    private
-
-    def set_power_port_template
-      @power_port_template = PowerPortTemplate.find(params[:id])
-    end
-
-    def power_port_template_params
-      params[:power_port_template].permit(:device_type_id, :name)
-    end
   end
 end

data/app/controllers/foreman_datacenter/power_ports_controller.rb

@@ -1,16 +1,18 @@
 module ForemanDatacenter
-  class PowerPortsController < ApplicationController
-    before_action :set_power_port, only: [:edit, :update, :destroy, :connected,
+  class PowerPortsController < ForemanDatacenter::ApplicationController
+    include Foreman::Controller::AutoCompleteSearch
+    include ForemanDatacenter::Controller::Parameters::PowerPort
+
+    before_action :find_resource, only: [:edit, :update, :destroy, :connected,
                                           :planned, :new_connection, :connect,
                                           :disconnect]
 
     def index
-      @power_ports = PowerPort.where.not(power_outlet_id: nil).
-        includes(:device, power_outlet: [:device])
+      @power_ports = resource_base_search_and_page.where.not(power_outlet_id: nil).includes(:device, power_outlet: [:device])
     end
 
     def new
-      @power_port = PowerPort.new(
+      @power_port = ForemanDatacenter::PowerPort.new(
         device: Device.find(params[:device_id])
       )
     end
@@ -19,7 +21,7 @@ module ForemanDatacenter
     end
 
     def create
-      @power_port = PowerPort.new(power_port_params)
+      @power_port = ForemanDatacenter::PowerPort.new(power_port_params.merge(device_id: params[:device_id]))
 
       if @power_port.save
         redirect_to device_url(id: @power_port.device_id),
@@ -80,15 +82,5 @@ module ForemanDatacenter
       redirect_to device_url(id: @power_port.device_id),
                   notice: 'Power port was successfully disconnected.'
     end
-
-    private
-
-    def set_power_port
-      @power_port = PowerPort.find(params[:id])
-    end
-
-    def power_port_params
-      params[:power_port].permit(:device_id, :name)
-    end
   end
 end

data/app/controllers/foreman_datacenter/rack_groups_controller.rb

@@ -1,23 +1,26 @@
 module ForemanDatacenter
-  class RackGroupsController < ApplicationController
-    before_action :set_rack_group, only: [:show, :edit, :update, :destroy]
+  class RackGroupsController < ForemanDatacenter::ApplicationController
+    include Foreman::Controller::AutoCompleteSearch
+    include ForemanDatacenter::Controller::Parameters::RackGroup
+
+    before_action :find_resource, only: [:show, :edit, :update, :destroy, :move]
 
     def index
-      @rack_groups = RackGroup.includes(:site, :racks).all
+      @rack_groups = resource_base_search_and_page.includes(:site, :racks)
     end
 
     def show
     end
 
     def new
-      @rack_group = RackGroup.new
+      @rack_group = ForemanDatacenter::RackGroup.new
     end
 
     def edit
     end
 
     def create
-      @rack_group = RackGroup.new(rack_group_params)
+      @rack_group = ForemanDatacenter::RackGroup.new(rack_group_params)
 
       if @rack_group.save
         process_success object: @rack_group
@@ -35,21 +38,35 @@ module ForemanDatacenter
     end
 
     def destroy
+      unless params['object_only']
+        @rack_group.racks.each { |r| r.devices.each { |d| d.destroy } }
+        @rack_group.racks.destroy_all
+      else
+        @rack_group.racks.delete_all(:nullify)
+      end
+
       if @rack_group.destroy
-        process_success object: @rack_group
+        process_success success_redirect: "/datacenter/rack_groups"
       else
         process_error object: @rack_group
       end
     end
 
-    private
-
-    def set_rack_group
-      @rack_group = RackGroup.find(params[:id])
+    def move
+      @rack_groups = resource_base_search_and_page
+      @racks = @rack_group.racks
+      process_error object: @rack_group, error_msg: 'Current Rack Group haven\'t any Racks.' if @racks.empty?
     end
 
-    def rack_group_params
-      params[:rack_group].permit(:name, :site_id)
+    def update_associated_objects
+      begin
+        @rack_group = ForemanDatacenter::RackGroup.find(request.env['HTTP_REFERER'].split('/')[-2])
+        @racks = @rack_group.racks
+        @racks.update_all(rack_group_id: params[:rack_group_id])
+        process_success success_redirect: rack_groups_path, success_msg: 'Associated objects successfully moved.'
+      rescue => e
+        process_error object: @rack_group, error_msg: "#{e}"
+      end
     end
   end
 end

data/app/controllers/foreman_datacenter/racks_controller.rb

@@ -1,20 +1,22 @@
 module ForemanDatacenter
-  class RacksController < ApplicationController
-    before_action :set_rack, only: [:show, :edit, :update, :destroy, :devices]
+  class RacksController < ForemanDatacenter::ApplicationController
+    include Foreman::Controller::AutoCompleteSearch
+    include ForemanDatacenter::Controller::Parameters::Rack
+
+    before_action :find_resource, only: [:show, :edit, :update, :destroy, :devices, :move]
 
     def index
-      @racks = ForemanDatacenter::Rack.
-        includes(:site, :rack_group, :devices).
-        order(:name).
-        all
+      @racks = resource_base_search_and_page.includes(:site, :rack_group)
     end
 
+    # action for async selecting rack_group in rack _form
     def rack_groups
-      @rack_groups = RackGroup.where(site_id: params[:site_id]).all
+      @rack_groups = ForemanDatacenter::RackGroup.where(site_id: params[:site_id]).all
       render partial: 'rack_groups'
     end
 
     def show
+      @rack = resource_base.includes(devices: [:device_role]).find(params[:id])
     end
 
     def new
@@ -43,22 +45,34 @@ module ForemanDatacenter
     end
 
     def destroy
+      unless params['object_only']
+        @rack.devices.each { |d| d.destroy }
+      else
+        @rack.devices.delete_all(:nullify)
+      end
+
       if @rack.destroy
-        process_success object: @rack
+        process_success success_redirect: racks_path
       else
         process_error object: @rack
       end
     end
 
-    private
-
-    def set_rack
-      @rack = ForemanDatacenter::Rack.find(params[:id])
+    def move
+      @racks = resource_base_search_and_page
+      @devices = @rack.devices
+      process_error object: @rack, error_msg: 'Current Rack haven\'t any Devices.' if @devices.empty?
     end
 
-    def rack_params
-      params[:rack].
-        permit(:site_id, :rack_group_id, :name, :facility_id, :height, :comments)
+    def update_associated_objects
+      begin
+        @rack = ForemanDatacenter::Rack.find(request.env['HTTP_REFERER'].split('/')[-2])
+        @devices = @rack.devices
+        @devices.update_all(rack_id: params[:rack_id])
+        process_success success_redirect: racks_path, success_msg: 'Associated objects successfully moved.'
+      rescue => e
+        process_error object: @rack, error_msg: "#{e}"
+      end
     end
   end
 end