Changeset 29060

Timestamp:

Jul 20, 2014, 11:32:56 PM (10 years ago)

Author:

rvelices

Message:

bug 3100 display IP address of comment author on admin page and also save the entire ip address in the database, not only the first three ip components

Location:

trunk

Files:

• admin/comments.php (modified) (2 diffs)
• admin/themes/default/template/comments.tpl (modified) (1 diff)
• include/functions_comment.inc.php (modified) (5 diffs)

trunk/admin/comments.php

@@ -168 +168 @@
     i.path,
     i.representative_ext,
-    validated
+    validated,
+    c.anonymous_id
   FROM '.COMMENTS_TABLE.' AS c
     INNER JOIN '.IMAGES_TABLE.' AS i
@@ -205 +206 @@
       'CONTENT' => trigger_change('render_comment_content',$row['content']),
       'IS_PENDING' => ('false' == $row['validated']),
+      'IP' => $row['anonymous_id'],
       )
     );

trunk/admin/themes/default/template/comments.tpl

@@ -66 +66 @@
   <div class="comment">
     <a class="illustration" href="{$comment.U_PICTURE}"><img src="{$comment.TN_SRC}"></a>
-    <p class="commentHeader">{if $comment.IS_PENDING}<span class="pendingFlag">{'Waiting'|@translate}</span> - {/if}<strong>{$comment.AUTHOR}</strong> - <em>{$comment.DATE}</em></p>
+    <p class="commentHeader">{if $comment.IS_PENDING}<span class="pendingFlag">{'Waiting'|@translate}</span> - {/if}{if !empty($comment.IP)}{$comment.IP} - {/if}<strong>{$comment.AUTHOR}</strong> - <em>{$comment.DATE}</em></p>
     <blockquote>{$comment.CONTENT}</blockquote>
   </div>

trunk/include/functions_comment.inc.php

@@ -174 +174 @@
       $comm['email'] = $user['email'];
     }
-    else if ($conf['comments_email_mandatory'])
+    elseif ($conf['comments_email_mandatory'])
     {
       $infos[] = l10n('Email address is missing. Please specify an email address.');
@@ -180 +180 @@
     }
   }
-  else if (!email_check_format($comm['email']))
+  elseif (!email_check_format($comm['email']))
   {
     $infos[] = l10n('mail address must be like xxx@yyy.eee (example : jack@altern.org)');
@@ -192 +192 @@
     array_pop($ip_components);
   }
-  $comm['anonymous_id'] = implode('.', $ip_components);
+  $anonymous_id = implode('.', $ip_components);
 
   if ($comment_action!='reject' and $conf['anti-flood_time']>0 and !is_admin())
@@ -205 +205 @@
     {
       $query.= '
-      AND anonymous_id = "'.$comm['anonymous_id'].'"';
+      AND anonymous_id LIKE "'.$anonymous_id.'.%"';
     }
     $query.= '
@@ -232 +232 @@
     \''.$comm['author'].'\',
     '.$comm['author_id'].',
-    \''.$comm['anonymous_id'].'\',
+    \''.$comm['ip'].'\',
     \''.$comm['content'].'\',
     NOW(),