From 877278dcedddcbb447b89075e28b4d3b6692ee41 Mon Sep 17 00:00:00 2001
From: Phil Dibowitz <phil@ipom.com>
Date: Fri, 19 Dec 2014 20:02:33 -0800
Subject: Several more cleanups

---
 lib/chef/provider/package.rb | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

(limited to 'lib/chef/provider/package.rb')

diff --git a/lib/chef/provider/package.rb b/lib/chef/provider/package.rb
index 1283538a0b..5c852017cb 100644
--- a/lib/chef/provider/package.rb
+++ b/lib/chef/provider/package.rb
@@ -48,6 +48,14 @@ class Chef
         [ candidate_version ].flatten
       end
 
+      def current_version_array
+        [ @current_resource.version ].flatten
+      end
+
+      def new_version_array
+        [ @new_resource.version ].flatten
+      end
+
       def define_resource_requirements
         requirements.assert(:install) do |a|
           a.assertion { ((@new_resource.version != nil) && !(target_version_already_installed?)) \
@@ -120,10 +128,10 @@ class Chef
       end
 
       def removing_package?
-        if ![ @current_resource.version ].flatten.any?
+        if !current_version_array.any?
           # ! any? means it's all nil's, which means nothing is installed
           false
-        elsif ![ @new_resource.version ].flatten.any?
+        elsif !new_version_array.any?
           true # remove any version of all packages
         elsif @new_resource.version == @current_resource.version
           true # remove the version we have
-- 
cgit v1.2.1