From 093862a340288468e8d27d1fe0eedc567f468e56 Mon Sep 17 00:00:00 2001
From: Adam Jahn <ajjahn@gmail.com>
Date: Fri, 24 Aug 2012 13:22:36 -0400
Subject: resource params should look like samba config options

---
 manifests/share.pp | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

(limited to 'manifests/share.pp')

diff --git a/manifests/share.pp b/manifests/share.pp
index a5ed122..9b708a8 100644
--- a/manifests/share.pp
+++ b/manifests/share.pp
@@ -2,8 +2,8 @@ define samba::share($ensure = present,
                     $description = '',
                     $path = '',
                     $browsable = '',
-                    $guestok = '',
-                    $readonly = '') {
+                    $guest_ok = '',
+                    $read_only = '') {
   
   $context = "/files/etc/samba/smb.conf"
   $target = "target[. = '${name}']"
@@ -45,20 +45,20 @@ define samba::share($ensure = present,
       require => Augeas["${name}-section"],
     }
 
-    augeas { "${name}-guestok":
+    augeas { "${name}-guest_ok":
       context => $context,
-      changes => $guestok ? {
-        default => "set ${target}/guestok ${guestok}",
-        '' => "rm ${target}/guestok",
+      changes => $guest_ok ? {
+        default => "set \"${target}/guest ok\" ${guest_ok}",
+        '' => "rm \"${target}/guest ok\"",
       },
       require => Augeas["${name}-section"],
     }
     
-    augeas { "${name}-readonly":
+    augeas { "${name}-read_only":
       context => $context,
-      changes => $readonly ? {
-        default => "set ${target}/readonly ${readonly}",
-        '' => "rm ${target}/readonly",
+      changes => $read_only ? {
+        default => "set \"${target}/read only\" ${read_only}",
+        '' => "rm \"${target}/read_only\"",
       },
       require => Augeas["${name}-section"],
     }
-- 
cgit v1.2.3