Merge pull request #78 from schubergphilis/refactor

Refactor style issues

Author: miguelaferreira

lib/vagrant-cloudstack/action/connect_cloudstack.rb

@@ -1,5 +1,5 @@
-require "fog"
-require "log4r"
+require 'fog'
+require 'log4r'
 
 module VagrantPlugins
   module Cloudstack
@@ -10,7 +10,7 @@ module Action
       class ConnectCloudstack
         def initialize(app, env)
           @app    = app
-          @logger = Log4r::Logger.new("vagrant_cloudstack::action::connect_cloudstack")
+          @logger = Log4r::Logger.new('vagrant_cloudstack::action::connect_cloudstack')
         end
 
         def call(env)
@@ -36,7 +36,7 @@ def call(env)
           fog_config[:cloudstack_port]   = domain_config.port if domain_config.port
           fog_config[:cloudstack_scheme] = domain_config.scheme if domain_config.scheme
 
-          @logger.info("Connecting to Cloudstack...")
+          @logger.info('Connecting to Cloudstack...')
           env[:cloudstack_compute] = Fog::Compute.new(fog_config)
 
           @app.call(env)

lib/vagrant-cloudstack/action/read_ssh_info.rb

@@ -1,4 +1,4 @@
-require "log4r"
+require 'log4r'
 
 module VagrantPlugins
   module Cloudstack
@@ -63,8 +63,7 @@ def read_ssh_info(cloudstack, machine)
             :password         => nil
           }) unless domain_config.ssh_key.nil?
           ssh_info = ssh_info.merge({ :username => domain_config.ssh_user }) unless domain_config.ssh_user.nil?
-
-          return ssh_info
+          ssh_info
         end
       end
     end

lib/vagrant-cloudstack/action/read_state.rb

@@ -30,7 +30,7 @@ def read_state(cloudstack, machine)
           end
 
           # Return the state
-          return server.state.downcase.to_sym
+          server.state.downcase.to_sym
         end
       end
     end