Use local variables

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}")