Fixing #310 -- users no longer autorequire their homedirs, and files now autorequire their owner and group.

git-svn-id: https://reductivelabs.com/svn/puppet/trunk@1747 980ebf18-57e1-0310-9a29-db15c13687c0

2 files changed, 16 insertions, 9 deletions

diff --git a/lib/puppet/type/pfile.rb b/lib/puppet/type/pfile.rb
index 5d7a3e881..d89fe17e3 100644
--- a/lib/puppet/type/pfile.rb
+++ b/lib/puppet/type/pfile.rb
@@ -172,6 +172,7 @@ module Puppet
             newvalues(:true, :false)
         end
 
+        # Autorequire any parent directories.
         autorequire(:file) do
             cur = []
             pary = self[:path].split(File::SEPARATOR)
@@ -187,6 +188,21 @@ module Puppet
             cur
         end
 
+        # Autorequire the owner and group of the file.
+        {:user => :owner, :group => :group}.each do |type, state|
+            autorequire(type) do
+                if @states.include?(state)
+                    # The user/group states automatically converts to IDs
+                    val = @states[state].shouldorig[0]
+                    if val.is_a?(Integer) or val =~ /^\d+$/
+                        nil
+                    else
+                        val
+                    end
+                end
+            end
+        end
+
         validate do
             if self[:content] and self[:source]
                 self.fail "You cannot specify both content and a source"
diff --git a/lib/puppet/type/user.rb b/lib/puppet/type/user.rb
index ac6ce461d..2fffc6f48 100755
--- a/lib/puppet/type/user.rb
+++ b/lib/puppet/type/user.rb
@@ -342,15 +342,6 @@ module Puppet
             autos
         end
 
-        autorequire(:file) do
-            dir = self.should(:home) or self.is(:home)
-            if dir =~ /^#{File::SEPARATOR}/
-                dir
-            else
-                nil
-            end
-        end
-
         def self.list_by_name
             users = []
             defaultprovider.listbyname do |user|