From 0f0f987a310f673d1d0ac3cfa103aa2318104eec Mon Sep 17 00:00:00 2001
From: Michael Scherer <misc@mageia.org>
Date: Wed, 23 Feb 2011 16:47:15 +0000
Subject: fix previous try to have a array of directories writable by apache

---
 modules/phpbb/manifests/init.pp | 21 ++++++++++++---------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/modules/phpbb/manifests/init.pp b/modules/phpbb/manifests/init.pp
index 8373952e..9ac8f562 100644
--- a/modules/phpbb/manifests/init.pp
+++ b/modules/phpbb/manifests/init.pp
@@ -65,6 +65,16 @@ class phpbb {
         }
     }
 
+    define apache_writable_dir() {
+        file { "$forums_dir/$lang/phpBB/$name":
+            ensure => directory,
+            owner => apache,
+            group => root,
+            mode => 755,
+            require => Exec["git_clone $lang"],
+        }
+    }
+
     # TODO find a way to avoid all the phpbb::base prefix
     define instance() {
         include phpbb::base
@@ -95,17 +105,10 @@ class phpbb {
 
         # list found by reading ./install/install_install.php
         # end of check_server_requirements ( 2 loops )
-        $writable_dir = ['cache',
+        apache_writable_dir { ['cache',
                          'images/avatars/upload',
                          'files',
-                         'store',
-                        ]
-        file { "$forums_dir/$lang/phpBB/$writable_dir":
-            ensure => directory,
-            owner => apache,
-            group => root,
-            mode => 755,
-            require => Exec["git_clone $lang"],
+                         'store' ]: 
         }
 
         file { "$forums_dir/$lang/phpBB/config.php":
-- 
cgit v1.2.1