Merge pull request #30 from WhatsARanjit/issue_26

Issue 26

Author: WhatsARanjit

lib/puppet/provider/node_group/https.rb

@@ -135,6 +135,10 @@ def classes
     PuppetX::Node_manager::Common.sort_hash(@property_hash[:classes])
   end
 
+  def rule
+    @property_hash[:rule].nil? ? [''] : @property_hash[:rule]
+  end
+
   friendly_name.each do |property,friendly|
     define_method "#{friendly}=" do |value|
       if property == :parent

lib/puppet/util/nc_https.rb

@@ -70,6 +70,10 @@ def delete_group(id)
   end
 
   def update_group(data)
+    # ISSUE 26
+    # Add nil for empty rules
+    data = Hash[data.map { |k,v| v == [''] ? [k,nil] : [k,v] }]
+
     res = do_https("v1/groups/#{data['id']}", 'POST', data)
     if res.code.to_i != 200
       Puppet.debug("Response code: #{res.code}")