Fix setting ACL if they had been set before

voxpupuli · Sep 7, 2018 · ad2570c · ad2570c
1 parent ae8715a
commit ad2570c
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/lib/puppet/provider/openldap_access/olc.rb b/lib/puppet/provider/openldap_access/olc.rb
@@ -57,7 +57,7 @@ def self.prefetch(resources)
       if provider = accesses.find{ |access|
         if resources[name][:position]
           access.suffix == resources[name][:suffix] &&
-          access.position == resources[name][:position]
+          access.position == resources[name][:position].to_s
         else
           access.suffix == resources[name][:suffix] &&
           access.access.flatten == resources[name][:access].flatten &&
@@ -105,7 +105,17 @@ def self.getDn(suffix)
   def getDn(*args); self.class.getDn(*args); end
 
   def exists?
-    @property_hash[:ensure] == :present
+    acls = self.class.instances.map { | instance |
+      {
+        :position => instance.position,
+        :suffix   => instance.suffix
+      }
+    }
+    acl = {
+      :position => resource[:position].to_s,
+      :suffix  => resource[:suffix]
+    }
+    acls.include?(acl)
   end
 
   def create