Permalink
Browse files

Use local variables

  • Loading branch information...
1 parent 7dbfdeb commit 5a4fb459291c7133d81ac731fca3fd45f42a5c07 Nuo Yan committed Jun 3, 2011
Showing with 9 additions and 9 deletions.
  1. +9 −9 chef/lib/chef/knife/client_bulk_delete.rb
@@ -6,9 +6,9 @@
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
-#
+#
# http://www.apache.org/licenses/LICENSE-2.0
-#
+#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -29,31 +29,31 @@ class ClientBulkDelete < Knife
banner "knife client bulk delete REGEX (options)"
- def run
- if @name_args.length < 1
+ def run
+ if name_args.length < 1
ui.fatal("You must supply a regular expression to match the results against")
exit 42
end
all_clients = Chef::ApiClient.list(true)
- matcher = /#{@name_args[0]}/
+ matcher = /#{name_args[0]}/
clients_to_delete = {}
all_clients.each do |name, client|
next unless name =~ matcher
clients_to_delete[client.name] = client
end
-
+
if clients_to_delete.empty?
- ui.info "No clients match the expression /#{@name_args[0]}/"
+ ui.info "No clients match the expression /#{name_args[0]}/"
exit 0
end
-
+
ui.msg("The following clients will be deleted:")
ui.msg("")
ui.msg(ui.list(clients_to_delete.keys.sort, :columns_down))
ui.msg("")
ui.confirm("Are you sure you want to delete these clients")
-
+
clients_to_delete.sort.each do |name, client|
client.destroy
ui.msg("Deleted client #{name}")

0 comments on commit 5a4fb45

Please sign in to comment.