Merge pull request redhat-openstack#295 from sodabrew/patch-1

Idiomatic syntax for should_policy and should_vhost
strider · Jan 15, 2015 · 52d3557 · 52d3557
2 parents 384892a + d37c9ab
commit 52d3557
Showing 1 changed file with 2 additions and 10 deletions.
diff --git a/lib/puppet/provider/rabbitmq_policy/rabbitmqctl.rb b/lib/puppet/provider/rabbitmq_policy/rabbitmqctl.rb
@@ -37,19 +37,11 @@ def policies(name, vhost)
   end
 
   def should_policy
-    if @should_policy
-      @should_policy
-    else
-      @should_policy = resource[:name].rpartition('@').first
-    end
+    @should_policy ||= resource[:name].rpartition('@').first
   end
 
   def should_vhost
-    if @should_vhost
-      @should_vhost
-    else
-      @should_vhost = resource[:name].rpartition('@').last
-    end
+    @should_vhost ||= resource[:name].rpartition('@').last
   end
 
   def create