A little more work on kandsten's fix for IPv6 support in

10785b1e820c5e10d982c6b49125903886f7b889:

- some style cleanup
- bump the module version in module.info
- rebuild the installer.sql

3 files changed, 13 insertions, 13 deletions

diff --git a/installer/install.sql b/installer/install.sql
index 865cb2a4..77cda72b 100644
--- a/installer/install.sql
+++ b/installer/install.sql
@@ -67,8 +67,8 @@ CREATE TABLE {comments} (
   `server_http_referer` varchar(255) DEFAULT NULL,
   `server_http_user_agent` varchar(128) DEFAULT NULL,
   `server_query_string` varchar(64) DEFAULT NULL,
-  `server_remote_addr` varchar(32) DEFAULT NULL,
-  `server_remote_host` varchar(64) DEFAULT NULL,
+  `server_remote_addr` varchar(40) DEFAULT NULL,
+  `server_remote_host` varchar(255) DEFAULT NULL,
   `server_remote_port` varchar(16) DEFAULT NULL,
   `state` varchar(15) DEFAULT 'unpublished',
   `text` text,
@@ -246,7 +246,7 @@ CREATE TABLE {modules} (
 /*!40101 SET character_set_client = @saved_cs_client */;
 INSERT INTO {modules} VALUES (1,1,'gallery',46,1);
 INSERT INTO {modules} VALUES (2,1,'user',3,2);
-INSERT INTO {modules} VALUES (3,1,'comment',3,3);
+INSERT INTO {modules} VALUES (3,1,'comment',4,3);
 INSERT INTO {modules} VALUES (4,1,'organize',4,4);
 INSERT INTO {modules} VALUES (5,1,'info',2,5);
 INSERT INTO {modules} VALUES (6,1,'rss',1,6);
diff --git a/modules/comment/helpers/comment_installer.php b/modules/comment/helpers/comment_installer.php
index cd20ef72..5c6bd586 100644
--- a/modules/comment/helpers/comment_installer.php
+++ b/modules/comment/helpers/comment_installer.php
@@ -64,15 +64,15 @@ class comment_installer {
     }
 
     if ($version == 3) {
-      /*
-        40 bytes for server_remote_addr is enough to swallow the longest
-        representation of an IPv6 addy.
-        
-        255 bytes for server_remote_host is enough to swallow the longest
-        legit DNS entry, with a few bytes to spare.
-      */ 
-      $db->query("ALTER TABLE {comments} CHANGE `server_remote_addr` `server_remote_addr` varchar(40)");
-      $db->query("ALTER TABLE {comments} CHANGE `server_remote_host` `server_remote_host` varchar(255)");
+      // 40 bytes for server_remote_addr is enough to swallow the longest
+      // representation of an IPv6 addy.
+      //
+      // 255 bytes for server_remote_host is enough to swallow the longest
+      // legit DNS entry, with a few bytes to spare.
+      $db->query(
+        "ALTER TABLE {comments} CHANGE `server_remote_addr` `server_remote_addr` varchar(40)");
+      $db->query(
+        "ALTER TABLE {comments} CHANGE `server_remote_host` `server_remote_host` varchar(255)");
       module::set_version("comment", $version = 4);
     }
   }
diff --git a/modules/comment/module.info b/modules/comment/module.info
index cd34f140..e5aa454d 100644
--- a/modules/comment/module.info
+++ b/modules/comment/module.info
@@ -1,3 +1,3 @@
 name = "Comments"
 description = "Allows users and guests to leave comments on photos and albums."
-version = 3
+version = 4