X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=modules%2Fapache2%2Fmanifests%2Fsite.pp;h=4588a60242e707202e15557c8fd5d1aef55a89bf;hb=b8ad6aee2a6d1f39aecdaacbf234941edcc1d4f0;hp=8fa5fdf526097df06ce8a681d4acb282848c0c5c;hpb=175c9bc26de888ba32a7677aa393621425d595f5;p=mirror%2Fdsa-puppet.git

diff --git a/modules/apache2/manifests/site.pp b/modules/apache2/manifests/site.pp
index 8fa5fdf52..4588a6024 100644
--- a/modules/apache2/manifests/site.pp
+++ b/modules/apache2/manifests/site.pp
@@ -1,16 +1,20 @@
 define apache2::site (
-	$config = undef,
-	$template = false,
-	$ensure = present,
-	$site = undef
+	$source=undef,
+	$content=undef,
+	$ensure=present,
+	$site=undef
 ) {
 
 	include apache2
 
-	if $ensure == present {
-		if ! ($config or $template) {
-			fail ( "No configuration found for ${name}" )
+	case $ensure {
+		present: {
+			if ! ($source or $content) {
+				fail ( "No configuration found for ${name}" )
+			}
 		}
+		absent:  {}
+		default: { fail ( "Unknown ensure value: '$ensure'" ) }
 	}
 
 	if $site {
@@ -23,26 +27,22 @@ define apache2::site (
 
 	$link_target = $ensure ? {
 		present => $target,
-		absent  => absent,
-		default => fail ( "Unknown ensure value: '$ensure'" ),
+		absent  => absent
 	}
 
-	case $template {
-		false: {
-			file { $target:
-				ensure  => $ensure,
-				source  => $config,
-				require => Package['apache2'],
-				notify  => Service['apache2'],
-			}
+	if $content {
+		file { $target:
+			ensure  => $ensure,
+			content => $content,
+			require => Package['apache2'],
+			notify  => Service['apache2'],
 		}
-		default: {
-			file { $target:
-				ensure  => $ensure,
-				content => template($template),
-				require => Package['apache2'],
-				notify  => Service['apache2'],
-			}
+	} else {
+		file { $target:
+			ensure  => $ensure,
+			source  => $source,
+			require => Package['apache2'],
+			notify  => Service['apache2'],
 		}
 	}