previous version using only_if and not_if didn't seem to work correctly

rabbitmq · Jun 5, 2012 · f6f0faf · f6f0faf
1 parent bc93a9d
commit f6f0faf
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/providers/plugin.rb b/providers/plugin.rb
@@ -18,18 +18,20 @@
 #
 
 action :enable do
-  execute "rabbitmq-plugins enable #{new_resource.plugin}" do
-    not_if "rabbitmq-plugins list #{new_resource.plugin} | grep '\\[[Ee]\\] #{new_resource.plugin}'"
-    Chef::Log.info "Enabling RabbitMQ plugin '#{new_resource.plugin}'."
-    new_resource.updated_by_last_action(true)
+  unless system("rabbitmq-plugins list #{new_resource.plugin} | grep '\\[[Ee]\\] #{new_resource.plugin}'")
+    execute "rabbitmq-plugins enable #{new_resource.plugin}" do
+      Chef::Log.info "Enabling RabbitMQ plugin '#{new_resource.plugin}'."
+      new_resource.updated_by_last_action(true)
+    end
   end
 end
 
 action :disable do
-  execute "rabbitmq-plugins disable #{new_resource.plugin}" do
-    only_if "rabbitmq-plugins list #{new_resource.plugin} | grep '\\[[Ee]\\] #{new_resource.plugin}'"
-    Chef::Log.info "Disabling RabbitMQ plugin '#{new_resource.plugin}'."
-    new_resource.updated_by_last_action(true)
+  if system("rabbitmq-plugins list #{new_resource.plugin} | grep '\\[[Ee]\\] #{new_resource.plugin}'")
+    execute "rabbitmq-plugins disable #{new_resource.plugin}" do
+      Chef::Log.info "Disabling RabbitMQ plugin '#{new_resource.plugin}'."
+      new_resource.updated_by_last_action(true)
+    end
   end
 end