handle case where no profiles installed

keeleysam · Nov 22, 2016 · 753eb03 · 753eb03
1 parent c147b0e
commit 753eb03
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/lib/facter/profiles.rb b/lib/facter/profiles.rb
@@ -16,10 +16,8 @@
 
       plist = Puppet::Util::Plist.read_plist_file(path)
 
-      data = plist['_computerlevel']
-
-      unless data.empty?
-        for item in data
+      if plist.key?('_computerlevel')
+        for item in plist['_computerlevel']
           profiles[item['ProfileIdentifier']] = {
             'display_name' => item['ProfileDisplayName'],
             'description' => item['ProfileDescription'],

diff --git a/lib/puppet/provider/profile_manager/macos.rb b/lib/puppet/provider/profile_manager/macos.rb
@@ -68,10 +68,8 @@ def getinstalledstate
 
     plist = Puppet::Util::Plist.read_plist_file(path)
 
-    data = plist['_computerlevel']
-
-    unless data.empty?
-      for item in data
+    if plist.key?('_computerlevel')
+      for item in plist['_computerlevel']
         if item['ProfileIdentifier'] == resource[:name]
           return {
             'identifier' => item['ProfileIdentifier'],