module VagrantPlugins
  module HostsUpdater
    module Action
      class RemoveHosts < BaseAction

        def run(env)
          machine_action = env[:machine_action]
          if machine_action != :destroy || !@machine.id
            if machine_action != :suspend || false != @machine.config.hostsupdater.remove_on_suspend
              if machine_action != :halt || false != @machine.config.hostsupdater.remove_on_suspend
                @ui.info "[vagrant-hostsupdater] Removing hosts"
                removeHostEntries
              else
                @ui.info "[vagrant-hostsupdater] Removing hosts on suspend disabled"
              end
            end
          end
        end

      end
    end
  end
end

module VagrantPlugins
  module HostsUpdater
    module Action
      class RemoveHosts < BaseAction

        def run(env)
          machine_action = env[:machine_action]
          if [:suspend, :halt].include? machine_action
            if @machine.config.hostsupdater.remove_on_suspend == false
              @ui.info "[vagrant-hostsupdater] Not removing hosts (remove_on_suspend false)"
            else
              @ui.info "[vagrant-hostsupdater] Removing hosts on suspend"
              removeHostEntries
            end
          else
            @ui.info "[vagrant-hostsupdater] Removing hosts"
            removeHostEntries
          end
        end

      end
    end
  end
end

Clean this up now that it will only be run once
Clean this up now that it will only be run once

This seemed to be coded in a format to accommodate it being called many times.. now we will only call it once by design so clean it up.