opendev/puppet-httpd
Code Issues Proposed changes

style guide fixes: 'true' <> true, $operatingsystem needs to be $::operatingsystem, etc.

Browse Source
This commit is contained in:
Walter Heck 2012-04-16 15:20:00 +08:00
parent 273f94d321
commit 073964152d
3 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/params.pp
View File

@ -20,7 +20,7 @@ class apache::params {
$user = 'www-data' $user = 'www-data'
$group = 'www-data' $group = 'www-data'
$ssl = 'true' $ssl = true
$template = 'apache/vhost-default.conf.erb' $template = 'apache/vhost-default.conf.erb'
$priority = '25' $priority = '25'
$servername = '' $servername = ''
@ -30,7 +30,7 @@ class apache::params {
$options = 'Indexes FollowSymLinks MultiViews' $options = 'Indexes FollowSymLinks MultiViews'
$vhost_name = '*' $vhost_name = '*'
case $operatingsystem { case $::operatingsystem {
'centos', 'redhat', 'fedora', 'scientific': { 'centos', 'redhat', 'fedora', 'scientific': {
$apache_name = 'httpd' $apache_name = 'httpd'
$php_package = 'php' $php_package = 'php'

2
manifests/ssl.pp
View File

@ -16,7 +16,7 @@ class apache::ssl {
include apache include apache
case $operatingsystem { case $::operatingsystem {
'centos', 'fedora', 'redhat', 'scientific': { 'centos', 'fedora', 'redhat', 'scientific': {
package { "apache_ssl_package": package { "apache_ssl_package":
name => "$apache::params::ssl_package", name => "$apache::params::ssl_package",

2
manifests/vhost.pp
View File

@ -57,7 +57,7 @@ define apache::vhost(
# Since the template will use auth, redirect to https requires mod_rewrite # Since the template will use auth, redirect to https requires mod_rewrite
if $redirect_ssl == true { if $redirect_ssl == true {
case $operatingsystem { case $::operatingsystem {
'debian','ubuntu': { 'debian','ubuntu': {
A2mod <| title == 'rewrite' |> A2mod <| title == 'rewrite' |>
} }