lint fixes for apache

5 files changed, 11 insertions, 12 deletions

diff --git a/modules/apache/manifests/vhost/base.pp b/modules/apache/manifests/vhost/base.pp
index a7e5720d..1ddc00cc 100644
--- a/modules/apache/manifests/vhost/base.pp
+++ b/modules/apache/manifests/vhost/base.pp
@@ -10,7 +10,7 @@ define apache::vhost::base ($content = '',
                             $enable_public_html = false) {
     include apache::base
     $httpd_logdir = '/var/log/httpd'
-    $filename = "$name.conf"
+    $filename = "${name}.conf"
 
     if ! $vhost {
         $real_vhost = $name
@@ -19,12 +19,12 @@ define apache::vhost::base ($content = '',
     }
 
     if ! $access_logfile {
-        $real_access_logfile = "$httpd_logdir/${real_vhost}-access_log"
+        $real_access_logfile = "${httpd_logdir}/${real_vhost}-access_log"
     } else {
         $real_access_logfile = $access_logfile
     }
     if ! $error_logfile {
-        $real_error_logfile = "$httpd_logdir/${real_vhost}-error_log"
+        $real_error_logfile = "${httpd_logdir}/${real_vhost}-error_log"
     } else {
         $real_error_logfile = $error_logfile
     }
@@ -34,7 +34,7 @@ define apache::vhost::base ($content = '',
         if $wildcard_sslcert != true {
             openssl::self_signed_cert{ $real_vhost:
                 directory => '/etc/ssl/apache/',
-                before    => Apache::Config["/etc/httpd/conf/vhosts.d/$filename"],
+                before    => Apache::Config["/etc/httpd/conf/vhosts.d/${filename}"],
             }
         }
     }
@@ -43,8 +43,7 @@ define apache::vhost::base ($content = '',
         include apache::mod::public_html
     }
 
-    apache::config { "/etc/httpd/conf/vhosts.d/$filename":
+    apache::config { "/etc/httpd/conf/vhosts.d/${filename}":
         content => template('apache/vhost_base.conf')
     }
 }
-
diff --git a/modules/apache/manifests/vhost/catalyst_app.pp b/modules/apache/manifests/vhost/catalyst_app.pp
index 4aec9a5e..11c77a8a 100644
--- a/modules/apache/manifests/vhost/catalyst_app.pp
+++ b/modules/apache/manifests/vhost/catalyst_app.pp
@@ -6,10 +6,10 @@ define apache::vhost::catalyst_app( $script,
                                     $vhost = false) {
     include apache::mod::fastcgi
     if ($location) {
-      $aliases['/static'] = "$location/root/static"
+      $aliases['/static'] = "${location}/root/static"
     }
 
-    $aliases['/'] = "$script/"
+    $aliases['/'] = "${script}/"
 
     apache::vhost::base { $name:
         vhost   => $vhost,
diff --git a/modules/apache/manifests/vhost/django_app.pp b/modules/apache/manifests/vhost/django_app.pp
index 5613e384..91974acd 100644
--- a/modules/apache/manifests/vhost/django_app.pp
+++ b/modules/apache/manifests/vhost/django_app.pp
@@ -11,8 +11,8 @@ define apache::vhost::django_app ($module = false,
 
     # module is a ruby reserved keyword, cannot be used in templates
     $django_module = $module
-    file { "$name.wsgi":
-        path    => "/usr/local/lib/wsgi/$name.wsgi",
+    file { "${name}.wsgi":
+        path    => "/usr/local/lib/wsgi/${name}.wsgi",
         mode    => '0755',
         notify  => Service['apache'],
         content => template('apache/django.wsgi'),
diff --git a/modules/apache/manifests/vhost/other_app.pp b/modules/apache/manifests/vhost/other_app.pp
index 69b3ec56..f5a71574 100644
--- a/modules/apache/manifests/vhost/other_app.pp
+++ b/modules/apache/manifests/vhost/other_app.pp
@@ -1,6 +1,6 @@
 define apache::vhost::other_app($vhost_file) {
     include apache::base
-    apache::config { "/etc/httpd/conf/vhosts.d/$name.conf":
+    apache::config { "/etc/httpd/conf/vhosts.d/${name}.conf":
         content => template($vhost_file),
     }
 }
diff --git a/modules/apache/manifests/vhost/redirect_ssl.pp b/modules/apache/manifests/vhost/redirect_ssl.pp
index 3458d59d..22a4d4f6 100644
--- a/modules/apache/manifests/vhost/redirect_ssl.pp
+++ b/modules/apache/manifests/vhost/redirect_ssl.pp
@@ -1,5 +1,5 @@
 define apache::vhost::redirect_ssl() {
-    apache::vhost::base { "redirect_ssl_$name":
+    apache::vhost::base { "redirect_ssl_${name}":
         vhost   => $name,
         content => template('apache/vhost_ssl_redirect.conf')
     }