code comments/indent 3
code comments/indent 3

--- a/css/vcaching.css
+++ b/css/vcaching.css
@@ -1,37 +1,37 @@
 /**
  * Icons
  */
- #page-title .path-admin-config-system-vcaching span.icon {
-  width:35px;
-  height:30px;
-  display:block;
-  background-color:transparent;
-  background-image:url(images/icon.png);
-  background-repeat:no-repeat;
+#page-title .path-admin-config-system-vcaching span.icon {
+  width: 35px;
+  height: 30px;
+  display: block;
+  background-color: transparent;
+  background-image: url(images/icon.png);
+  background-repeat: no-repeat;
   background-position: 0px 0px;
 }
 
 #block-system-main .path-admin-config-system-vcaching span.icon {
-  width:35px;
-  height:30px;
-  display:block;
-  background-color:transparent;
-  background-image:url(images/icon.png);
-  background-repeat:no-repeat;
+  width: 35px;
+  height: 30px;
+  display: block;
+  background-color: transparent;
+  background-image: url(images/icon.png);
+  background-repeat: no-repeat;
   background-position: 0px 0px;
 }
 
 #vcaching-settings-form span.vcaching-reload {
-  width:20px;
-  height:20px;
+  width: 20px;
+  height: 20px;
   font-size: 20px;
   line-height: 1;
   vertical-align: top;
   text-align: center;
-  display:inline-block;
-  background-color:transparent;
-  background-image:url(images/refresh.png);
-  background-repeat:no-repeat;
+  display: inline-block;
+  background-color: transparent;
+  background-image: url(images/refresh.png);
+  background-repeat: no-repeat;
   background-position: 0px 0px;
 }
 

--- a/vcaching.class.php
+++ b/vcaching.class.php
@@ -1,17 +1,22 @@
 <?php
+
+/**
+ * @file
+ * VCaching class.
+ */
 
 /**
  * VCaching Class
  */
 class VCaching {
   protected $prefix = '';
-  protected $purgeUrls = array();
-  protected $varnishIp = null;
-  protected $varnishHost = null;
-  protected $dynamicHost = null;
-  protected $ipsToHosts = array();
-  protected $statsJsons = array();
-  protected $purgeKey = null;
+  protected $purge_urls = array();
+  protected $varnish_ip = NULL;
+  protected $varnish_host = NULL;
+  protected $dynamic_host = NULL;
+  protected $ips_to_hosts = array();
+  protected $stats_jsons = array();
+  protected $purgeKey = NULL;
   protected $noticeMessage = '';
   protected $debug = 0;
 
@@ -35,26 +40,27 @@
   protected function setupIpsToHosts()
   {
     $this->debug = variable_get($this->prefix . 'debug');
-    $this->varnishIp = variable_get($this->prefix . 'ips');
-    $this->varnishHost = variable_get($this->prefix . 'hosts');
-    $this->dynamicHost = variable_get($this->prefix . 'dynamic_host');
-    $this->statsJsons = variable_get($this->prefix . 'stats_json_file');
-    $varnishIp = explode(',', $this->varnishIp);
-    $varnishHost = explode(',', $this->varnishHost);
-    $statsJsons = explode(',', $this->statsJsons);
-    foreach ($varnishIp as $key => $ip) {
+    $this->varnish_ip = variable_get($this->prefix . 'ips');
+    $this->varnish_host = variable_get($this->prefix . 'hosts');
+    $this->dynamic_host = variable_get($this->prefix . 'dynamic_host');
+    $this->stats_jsons = variable_get($this->prefix . 'stats_json_file');
+    $varnish_ip = explode(',', $this->varnish_ip);
+    $varnish_host = explode(',', $this->varnish_host);
+    $stats_jsons = explode(',', $this->stats_jsons);
+    foreach ($varnish_ip as $key => $ip) {
       if (strpos($ip, ':')) {
         $_ip = explode(':', $ip);
         $ip = $_ip[0];
         $port = $_ip[1];
-      } else {
+      }
+      else {
         $port = 80;
       }
-      $this->ipsToHosts[] = array(
+      $this->ips_to_hosts[] = array(
         'ip' => $ip,
         'port' => $port,
-        'host' => $this->dynamicHost ? $_SERVER['HTTP_HOST'] : $varnishHost[$key],
-        'statsJson' => isset($statsJsons[$key]) ? $statsJsons[$key] : null
+        'host' => $this->dynamic_host ? $_SERVER['HTTP_HOST'] : $varnish_host[$key],
+        'statsJson' => isset($stats_jsons[$key]) ? $stats_jsons[$key] : null
         );
     }
 
@@ -68,10 +74,10 @@
    */
   public function purgeCache()
   {
-    $purgeUrls = array_unique($this->purgeUrls);
-
-    if (!empty($purgeUrls)) {
-      foreach($purgeUrls as $url) {
+    $purge_urls = array_unique($this->purge_urls);
+
+    if (!empty($purge_urls)) {
+      foreach($purge_urls as $url) {
         $this->purgeUrl($url);
       }
     }
@@ -80,10 +86,10 @@
   /**
    * Fetches the notice message
    *
-   * @param  boolean $console
+   * @param boolean $console
    * @return string
    */
-  public function getNoticeMessage($console = false)
+  public function getNoticeMessage($console = FALSE)
   {
     return ($console ? str_replace("<br />", "\n", $this->noticeMessage): $this->noticeMessage);
   }
@@ -91,7 +97,7 @@
   /**
    * Purges the given URL
    *
-   * @param  string $url relative url
+   * @param string $url relative url
    * @return void
    */
   public function purgeUrl($url)
@@ -100,24 +106,26 @@
 
     if (isset($p['path'])) {
       $path = $p['path'];
-    } else {
+    }
+    else {
       $path = '';
     }
 
     if (isset($p['query']) && ($p['query'] == 'vc-regex')) {
       $purgemethod = 'regex';
       $purgeurl = base_path() . '.*';
-    } else {
+    }
+    else {
       $purgemethod = 'default';
       $purgeurl = $path;
     }
-    foreach ($this->ipsToHosts as $ipToHost) {
-      $headers = array('host' => $ipToHost['host'], 'X-VC-Purge-Method' => $purgemethod, 'X-VC-Purge-Host' => $ipToHost['host']);
+    foreach ($this->ips_to_hosts as $ip_to_host) {
+      $headers = array('host' => $ip_to_host['host'], 'X-VC-Purge-Method' => $purgemethod, 'X-VC-Purge-Host' => $ip_to_host['host']);
       if (!is_null($this->purgeKey)) {
         $headers['X-VC-Purge-Key'] = $this->purgeKey;
       }
-      $response = $this->_vcaching_cache_purge($ipToHost['ip'], $ipToHost['port'], $purgeurl, $headers);
-      if ($response['error'] == true) {
+      $response = $this->_vcachingCachePurge($ip_to_host['ip'], $ip_to_host['port'], $purgeurl, $headers);
+      if ($response['error'] == TRUE) {
         $this->noticeMessage .= 'Error ' . $response['message'];
       } else {
         $this->noticeMessage .= '<br />' . t('Trying to purge URL :') . $purgeurl;
@@ -125,7 +133,7 @@
         $this->noticeMessage .= ' => <br /> ' . isset($matches[1]) ? ' => ' . $matches[1] : $response['message'];
         $this->noticeMessage .= '<br />';
         if ($this->debug) {
-          $this->noticeMessage .= nl2br($response['message'], true) . '<br />';
+          $this->noticeMessage .= nl2br($response['message'], TRUE) . '<br />';
         }
       }
     }
@@ -176,26 +184,26 @@
       $html .= '</div>' . "\n";
       $html .= '</div>' . "\n";
     }
-    if(trim($this->statsJsons)){
+    if(trim($this->stats_jsons)){
       $html .= '<div class="block clearfix block-system">' . "\n";
       $html .= '<div class="block-content clearfix"><p class="clearfix"><strong>Select server</strong></p>' . "\n";
       $html .= '<ul class="secondary-tabs links clearfix">' . "\n";
-      foreach ($this->ipsToHosts as $server => $ipToHost) {
-        $html .= '<li class="links ' . (($server == 0) ? 'active' : '') . '"><a class="server nav-tab" href="#" server="' . $server . '">'. sprintf(t('Server %1$s'), $ipToHost['ip']).'</a></li>' . "\n";
+      foreach ($this->ips_to_hosts as $server => $ip_to_host) {
+        $html .= '<li class="links ' . (($server == 0) ? 'active' : '') . '"><a class="server nav-tab" href="#" server="' . $server . '">'. sprintf(t('Server %1$s'), $ip_to_host['ip']).'</a></li>' . "\n";
       }
       $html .= '</ul>';
       $html .= '</div>' . "\n";
       $html .= '</div>' . "\n";
 
-      foreach ($this->ipsToHosts as $server => $ipToHost) {
+      foreach ($this->ips_to_hosts as $server => $ip_to_host) {
         $html .= '<div style="display:' . (($server == 0) ? 'block' : 'none') . '" class="servers server_' . $server . '">';
-        $html .= sprintf(t('Fetching stats for server %1$s'), $ipToHost['ip']);
+        $html .= sprintf(t('Fetching stats for server %1$s'), $ip_to_host['ip']);
         $html .= '</div>' . "\n";
         $html .= '<script type="text/javascript">' . "\n";
-        $html .= 'jQuery.getJSON("' . $ipToHost['statsJson'] . '", function(data) {' . "\n";
+        $html .= 'jQuery.getJSON("' . $ip_to_host['statsJson'] . '", function(data) {' . "\n";
         $html .= 'var server = \'.server_' . $server .'\'' . "\n";
         $html .= 'jQuery(server).html(\'\');' . "\n";
-        $html .= 'jQuery(server).append(\'<div id="block-system-vcaching-help" class="block block-system"><div class="block-content clearfix"><p>' . sprintf(t('Stats for server %1$s generated on '), $ipToHost['ip']) . '\' + data.timestamp);' . " + '</p></div></div>'\n";
+        $html .= 'jQuery(server).append(\'<div id="block-system-vcaching-help" class="block block-system"><div class="block-content clearfix"><p>' . sprintf(t('Stats for server %1$s generated on '), $ip_to_host['ip']) . '\' + data.timestamp);' . " + '</p></div></div>'\n";
         $html .= 'jQuery(server).append(\'<table><thead><tr><th><strong>' . t('Description') . '</strong></th><th><strong>' . t('Value') . '</strong></th><th><strong>' . t('Key') .'</strong></th></tr></thead><tbody class="varnishstats_' . $server . '"></tbody></table>\')' . "\n";
         $html .= 'delete data.timestamp;' . "\n";
         $html .= 'jQuery.each(data, function(key, val) {' . "\n";
@@ -214,26 +222,27 @@
       $html .= '});' . "\n";
       $html .= '</script>' . "\n";
     }
-  $html .= '</div>' . "\n";
-  $html .= '</fieldset>' . "\n";
-  return $html;
-}
+    $html .= '</div>' . "\n";
+    $html .= '</fieldset>' . "\n";
+    return $html;
+  }
 
   /**
    * The socket connection to Varnish
    *
-   * @param  string $server_ip
-   * @param  string $server_port
-   * @param  string $path
-   * @param  array $headers
+   * @param string $server_ip
+   * @param string $server_port
+   * @param string $path
+   * @param array $headers
    * @return array
    */
-  protected function _vcaching_cache_purge($server_ip, $server_port, $path = '/.*', $headers)
+  protected function _vcachingCachePurge($server_ip, $server_port, $path = '/.*', $headers)
   {
     $fp = fsockopen($server_ip, $server_port, $errno, $errstr, 2);
     if (!$fp) {
-      return array('error' => true, 'message' => $errstr .'(' . $errno . ')');
-    } else {
+      return array('error' => TRUE, 'message' => $errstr .'(' . $errno . ')');
+    }
+    else {
       $out = "PURGE " . $path . " HTTP/1.0\n";
       foreach ($headers as $key => $value) {
         $out .= $key . ': '. $value . "\n";
@@ -245,14 +254,14 @@
         $ret .= fgets($fp, 128);
       }
       fclose($fp);
-      return array('error' => false, 'message' => $ret);
+      return array('error' => FALSE, 'message' => $ret);
     }
   }
 
   /**
    * Creates the configuration zip archive
    *
-   * @param  int $version
+   * @param int $version
    * @return void
    */
   public function downloadConf($version)
@@ -261,25 +270,26 @@
     $zip = new ZipArchive();
     $zip->open($tmpfile, ZipArchive::OVERWRITE);
     $files = array(
-      'default.vcl' => true,
-      'LICENSE' => false,
-      'README.rst' => false,
-      'conf/acl.vcl' => true,
-      'conf/backend.vcl' => true,
-      'lib/bigfiles.vcl' => false,
-      'lib/bigfiles_pipe.vcl' => false,
-      'lib/cloudflare.vcl' => false,
-      'lib/mobile_cache.vcl' => false,
-      'lib/mobile_pass.vcl' => false,
-      'lib/purge.vcl' => true,
-      'lib/static.vcl' => false,
-      'lib/xforward.vcl' => false,
+      'default.vcl' => TRUE,
+      'LICENSE' => FALSE,
+      'README.rst' => FALSE,
+      'conf/acl.vcl' => TRUE,
+      'conf/backend.vcl' => TRUE,
+      'lib/bigfiles.vcl' => FALSE,
+      'lib/bigfiles_pipe.vcl' => FALSE,
+      'lib/cloudflare.vcl' => FALSE,
+      'lib/mobile_cache.vcl' => FALSE,
+      'lib/mobile_pass.vcl' => FALSE,
+      'lib/purge.vcl' => TRUE,
+      'lib/static.vcl' => FALSE,
+      'lib/xforward.vcl' => FALSE,
     );
     foreach ($files as $file => $parse) {
       $filepath = __DIR__ . '/varnish-conf/v' . $version . '/' . $file;
       if ($parse) {
-        $content = $this->_parse_conf_file($version, $file, file_get_contents($filepath));
-      } else {
+        $content = $this->_parseConfFile($version, $file, file_get_contents($filepath));
+      }
+      else {
         $content = file_get_contents($filepath);
       }
       $zip->addFromString($file, $content);
@@ -296,20 +306,20 @@
   /**
    * Parses the config files
    *
-   * @param  int $version
-   * @param  string $file
-   * @param  string $content
+   * @param int $version
+   * @param string $file
+   * @param string $content
    * @return string
    */
-  protected function _parse_conf_file($version, $file, $content)
+  protected function _parseConfFile($version, $file, $content)
   {
     if ($file == 'default.vcl') {
       $logged_in_cookie = variable_get($this->prefix . 'cookie');
       $content = str_replace('c005492c65', $logged_in_cookie, $content);
-    } else if ($file == 'conf/backend.vcl') {
+    } elseif ($file == 'conf/backend.vcl') {
       if ($version == 3) {
         $content = "";
-      } else if ($version == 4) {
+      } elseif ($version == 4) {
         $content = "import directors;\n\n";
       }
       $backend = array();
@@ -350,7 +360,7 @@
         $content .= "\tset req.backend_hint = backends.backend();\n";
         $content .= "}\n";
       }
-    } else if ($file == 'conf/acl.vcl') {
+    } elseif ($file == 'conf/acl.vcl') {
       $acls = variable_get($this->prefix . 'conf_acls');
       $acls = explode(',', $acls);
       $content = "acl cloudflare {\n";
@@ -364,7 +374,7 @@
         $content .= "\t\"" . $acl . "\";\n";
       }
       $content .= "}\n";
-    } else if ($file == 'lib/purge.vcl') {
+    } elseif ($file == 'lib/purge.vcl') {
       $purge_key = variable_get($this->prefix . 'purge_key');
       $content = str_replace('ff93c3cb929cee86901c7eefc8088e9511c005492c6502a930360c02221cf8f4', $purge_key, $content);
     }

--- a/vcaching.drush.inc
+++ b/vcaching.drush.inc
@@ -15,35 +15,31 @@
     'drupal dependencies' => array('vcaching'),
     'aliases' => array('vpa'),
     'callback' => 'vcaching_purge_all',
-    );
+  );
   $items['vcaching-purge-urls'] = array(
     'description' => "Purge a list of URLs in varnish.",
     'arguments' => array(
       'paths' => 'A list of paths to purge separated by ;. Regular expressions allowed.',
-      ),
+    ),
     'aliases' => array('vpu'),
     'drupal dependencies' => array('vcaching'),
     'callback' => 'vcaching_purge_urls',
-    );
+  );
   return $items;
 }
 
 /**
  * Callback for vcaching-purge-all drush command.
- *
- * @return void
  */
 function vcaching_purge_all() {
   $vcaching = new VCaching('vcaching_');
   drush_log(t('Purging all Varnish cache'), 'ok');
   $vcaching->purgeUrl(base_path() . '?vc-regex');
-  drush_log($vcaching->getNoticeMessage(true), 'ok');
+  drush_log($vcaching->getNoticeMessage(TRUE), 'ok');
 }
 
 /**
  * Callback for vcaching-purge-urls drush command.
- *
- * @param string $paths A space separated list of paths.
  */
 function vcaching_purge_urls($paths) {
   $vcaching = new VCaching('vcaching_');
@@ -52,7 +48,7 @@
     drush_log("Purging Varnish cache path " . $path);
     $vcaching->purgeUrl($path);
   }
-  drush_log($vcaching->getNoticeMessage(true), 'ok');
+  drush_log($vcaching->getNoticeMessage(TRUE), 'ok');
 }
 
 /**

--- a/vcaching.module
+++ b/vcaching.module
@@ -1,5 +1,4 @@
 <?php
-
 /**
  * @file
  * Functions used for the module.
@@ -34,10 +33,12 @@
     if (user_is_logged_in()) {
       drupal_add_http_header('X-VC-Cacheable', 'NO:User is logged in');
       $ttl = 0;
-    } else {
+    }
+    else {
       if (drupal_is_front_page()) {
         $ttl = variable_get('vcaching_homepage_ttl');
-      } else {
+      }
+      else {
         $ttl = variable_get('vcaching_ttl');
       }
     }
@@ -68,7 +69,7 @@
 /**
  * Implements hook_node_delete().
  */
-function varnish_node_delete($node) {
+function vcaching_node_delete($node) {
   if (variable_get('vcaching_enable')) {
     $vcaching = new VCaching('vcaching_');
     $vcaching->purgeUrl(base_path() . $node->path['alias']);
@@ -91,7 +92,7 @@
     'type' => MENU_NORMAL_ITEM,
     'weight' => 50,
     'file' => 'system.admin.inc',
-    'file path' => drupal_get_path('module', 'system')
+    'file path' => drupal_get_path('module', 'system'),
     );
 
   $items['admin/config/system/vcaching/settings'] = array(
@@ -101,7 +102,7 @@
     'page arguments' => array('vcaching_settings_form'),
     'access arguments' => array('varnish caching settings access'),
     'type' => MENU_LOCAL_TASK,
-    'weight' => 10
+    'weight' => 10,
     );
 
   $items['admin/config/system/vcaching/console'] = array(
@@ -111,7 +112,7 @@
     'page arguments' => array('vcaching_console_form'),
     'access arguments' => array('varnish caching console access'),
     'type' => MENU_DEFAULT_LOCAL_TASK,
-    'weight' => 20
+    'weight' => 20,
     );
 
   $items['admin/config/system/vcaching/stats'] = array(
@@ -122,7 +123,7 @@
     'access callback' => TRUE,
     'access arguments' => array('varnish caching stats access'),
     'type' => MENU_LOCAL_TASK,
-    'weight' => 30
+    'weight' => 30,
     );
 
   $items['admin/config/system/vcaching/conf'] = array(
@@ -132,7 +133,7 @@
     'page arguments' => array('vcaching_conf_form'),
     'access arguments' => array('varnish caching settings access'),
     'type' => MENU_LOCAL_TASK,
-    'weight' => 40
+    'weight' => 40,
     );
 
   $items['vcaching_purge_all_cache'] = array(
@@ -142,7 +143,7 @@
     'access arguments' => array('varnish caching purge all'),
     'menu_name' => 'shortcut-set-1',
     'type' => MENU_NORMAL_ITEM,
-    'weight' => 10
+    'weight' => 10,
     );
 
   return $items;
@@ -155,17 +156,17 @@
   return array(
     'varnish caching purge all' => array(
       'title' => t('Purge all Varnish cache from shortcut'),
-      ),
+    ),
     'varnish caching settings access' => array(
       'title' => t('Change settings'),
-      ),
+    ),
     'varnish caching console access' => array(
       'title' => t('Use the console'),
-      ),
+    ),
     'varnish caching stats access' => array(
       'title' => t('View stats'),
-      )
-    );
+    ),
+  );
 }
 
 /**
@@ -180,15 +181,15 @@
     '#title' => t('Settings'),
     '#weight' => 5,
     '#collapsible' => FALSE,
-    '#collapsed' => FALSE
-    );
+    '#collapsed' => FALSE,
+  );
   $form['general']['vcaching_enable'] = array(
     '#type' => 'checkbox',
     '#title' => t('Enable'),
     '#description' => t('Enable Varnish Caching'),
     '#default_value' => variable_get('vcaching_enable', '0'),
-    '#title_display' => 'before'
-    );
+    '#title_display' => 'before',
+  );
   $form['general']['vcaching_homepage_ttl'] = array(
     '#type' => 'textfield',
     '#title' => t('Homepage cache TTL'),
@@ -197,7 +198,7 @@
     '#size' => 10,
     '#maxlength' => 10,
     '#required' => TRUE,
-    );
+  );
   $form['general']['vcaching_ttl'] = array(
     '#type' => 'textfield',
     '#title' => t('Cache TTL'),
@@ -206,7 +207,7 @@
     '#size' => 10,
     '#maxlength' => 10,
     '#required' => TRUE,
-    );
+  );
   $form['general']['vcaching_ips'] = array(
     '#type' => 'textfield',
     '#title' => t('IPs'),
@@ -215,13 +216,13 @@
     '#size' => 60,
     '#maxlength' => 500,
     '#required' => TRUE,
-    );
+  );
   $form['general']['vcaching_dynamic_host'] = array(
     '#type' => 'checkbox',
     '#title' => t('Dynamic host'),
     '#description' => t('Uses the $_SERVER[\'HTTP_HOST\'] as hash for Varnish. This means the purge cache action will work on the domain you\'re on. Use this option if you use only one domain.'),
-    '#default_value' => variable_get('vcaching_dynamic_host', 0)
-    );
+    '#default_value' => variable_get('vcaching_dynamic_host', 0),
+  );
   if (variable_get('vcaching_dynamic_host') == 0) {
     $form['general']['vcaching_hosts'] = array(
       '#type' => 'textfield',
@@ -259,21 +260,21 @@
     '#suffix' => '<div class="description">' . t('Comma separated relative URLs. One for each IP. <a href="vcaching/stats?info=1">Click here</a> for more info on how to set this up.') . '</div>',
     '#default_value' => variable_get('vcaching_stats_json_file', '/varnishstats.json'),
     '#size' => 100,
-    '#maxlength' => 128
+    '#maxlength' => 128,
     );
   $form['general']['vcaching_clear_cache_node_save'] = array(
     '#type' => 'checkbox',
     '#title' => t('Clear all cache on node save'),
     '#description' => t('If this option is on, each node save will trigger a full cache purge (/.*). If it\'s off, only the pages associated with the node will be purged.'),
     '#default_value' => variable_get('vcaching_clear_cache_node_save', 0),
-    '#title_display' => 'before'
+    '#title_display' => 'before',
     );
   $form['general']['vcaching_debug'] = array(
     '#type' => 'checkbox',
     '#title' => t('Enable debug'),
     '#description' => t('Send all debugging headers to the client. Also shows complete response from Varnish on purge all.'),
     '#default_value' => variable_get('vcaching_debug', 0),
-    '#title_display' => 'before'
+    '#title_display' => 'before',
     );
 
   drupal_add_js('
@@ -303,8 +304,8 @@
     '#title' => t('Console'),
     '#weight' => 5,
     '#collapsible' => FALSE,
-    '#collapsed' => FALSE
-    );
+    '#collapsed' => FALSE,
+  );
   $form['action']['vcaching_console_url'] = array(
     '#type' => 'textfield',
     '#title' => t('Relative URL'),
@@ -313,15 +314,13 @@
     '#size' => 60,
     '#maxlength' => 500,
     '#required' => TRUE,
-    );
+  );
   $form['action']['submit'] = array('#type' => 'submit', '#value' => t('Purge cache'), '#submit' => array('vcaching_console_form_submit'));
   return system_settings_form($form);
 }
 
 /**
  * Console form submit.
- *
- * @return void
  */
 function vcaching_console_form_submit($form, &$form_state) {
   if (variable_get('vcaching_enable')) {
@@ -355,40 +354,40 @@
     '#title' => t('Varnish Cache configuration'),
     '#weight' => 5,
     '#collapsible' => FALSE,
-    '#collapsed' => FALSE
-    );
+    '#collapsed' => FALSE,
+  );
   $form['general']['vcaching_conf_backends'] = array(
     '#type' => 'textfield',
     '#title' => t('Backends'),
     '#description' => t('Comma separated ip/ip:port. Example : 192.168.0.2,192.168.0.3:8080'),
     '#default_value' => variable_get('vcaching_conf_backends'),
     '#size' => 60,
-    '#required' => TRUE
-    );
+    '#required' => TRUE,
+  );
   $form['general']['vcaching_conf_acls'] = array(
     '#type' => 'textfield',
     '#title' => t('ACLs'),
     '#description' => t('Comma separated ip/ip range. Example : 192.168.0.2,192.168.1.1/24'),
     '#default_value' => variable_get('vcaching_conf_acls'),
     '#size' => 60,
-    '#required' => TRUE
-    );
+    '#required' => TRUE,
+  );
   $form['general']['submit'] = array('#type' => 'submit', '#value' => t('Save configuration'));
   $form['action'] = array(
     '#type' => 'fieldset',
     '#title' => t('Get configuration files'),
     '#weight' => 5,
     '#collapsible' => FALSE,
-    '#collapsed' => FALSE
-    );
+    '#collapsed' => FALSE,
+  );
   $form['action']['vcaching_conf_version'] = array(
     '#type' => 'select',
     '#title' => t('Version'),
     '#description' => t('Varnish Cache version'),
     '#options' => array(3 => 3,4 => 4),
     '#default_value' => 3,
-    '#required' => TRUE
-    );
+    '#required' => TRUE,
+  );
   $form['action']['submit'] = array('#type' => 'submit', '#value' => t('Download'), '#submit' => array('vcaching_conf_form_submit'));
   return system_settings_form($form);
 }
@@ -405,8 +404,6 @@
 
 /**
  * Conf form submit.
- *
- * @return void
  */
 function vcaching_conf_form_submit($form, &$form_state)
 {
@@ -414,15 +411,14 @@
   if (variable_get('vcaching_enable')) {
     $vcaching = new VCaching('vcaching_');
     $vcaching->downloadConf($version);
-  } else {
+  }
+  else {
     drupal_goto($_SERVER['HTTP_REFERER']);
   }
 }
 
 /**
  * Shortcut callback.
- *
- * @return void
  */
 function vcaching_purge_all_cache_callback()
 {

comments