Rubocop cleanup

theforeman · Oct 14, 2024 · bffa054 · bffa054
1 parent 05b6ff9
commit bffa054
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/app/helpers/proxmox_vm_attrs_helper.rb b/app/helpers/proxmox_vm_attrs_helper.rb
@@ -47,7 +47,7 @@ def object_to_attributes_hash(vms, from_profile, start_checked)
 
   def cpu_flags_attrs(param_scope, config)
     flag_attrs = ActiveSupport::HashWithIndifferentAccess.new
-    Fog::Proxmox::CpuHelper.flags.each do |key, _val|
+    Fog::Proxmox::CpuHelper.flags.each_key do |key|
       flag_attrs.merge!({ key => { :name => "#{param_scope}[config_attributes][#{key}]", :value => config.public_send(key) } })
     end
     flag_attrs

diff --git a/app/helpers/proxmox_vm_volumes_helper.rb b/app/helpers/proxmox_vm_volumes_helper.rb
@@ -56,8 +56,11 @@ def parse_hard_disk_volume(args)
     disk[:storage] = args['storage'].to_s if args.key?('storage') && !args['storage'].empty?
     disk[:size] = args['size'].to_i if args.key?('size') && !args['size'].empty?
     args['backup'] = '1' if args['backup'].nil?
-    add_disk_options(disk, args) unless args.key?('options')
-    disk[:options] = args['options'] if args.key?('options')
+    if args.key?('options')
+      disk[:options] = args['options']
+    else
+      add_disk_options(disk, args)
+    end
     disk.key?(:storage) ? disk : {}
   end