Use the initializer to avoid NoMethodError on nil.include?

Rename the argument so as to not collide with the tags method.

1 files changed, 4 insertions, 4 deletions

diff --git a/lib/chef/node.rb b/lib/chef/node.rb
index 1e4a850277..e04ac23c54 100644
--- a/lib/chef/node.rb
+++ b/lib/chef/node.rb
@@ -376,12 +376,12 @@ class Chef
       normal[:tags]
     end
 
-    def tag(*tags)
-      tags.each do |tag|
-        self.normal[:tags].push(tag.to_s) unless self[:tags].include? tag.to_s
+    def tag(*args)
+      args.each do |tag|
+        tags.push(tag.to_s) unless tags.include? tag.to_s
       end
 
-      self[:tags]
+      tags
     end
 
     # Extracts the run list from +attrs+ and applies it. Returns the remaining attributes