Changeset 811 for trunk/Modules/Referrer/Referrer.php

Timestamp:

May 28, 2014, 11:00:26 PM (10 years ago)

Author:

chronos

Message:

• Added: Better handling of repeater false referrer links. Now use manual BlackList table for checking which IP source is blocked.

File:

• trunk/Modules/Referrer/Referrer.php (modified) (3 diffs)

trunk/Modules/Referrer/Referrer.php

@@ -41 +41 @@
       {
         $IP = GetRemoteAddress();
-        
-        // Check if client IP is not blocked as spam source
-        $DbResult = $this->System->Database->query('SELECT COUNT(*) FROM `Referrer` WHERE `LastIP` = "'.$IP.'" AND (`Visible` = 0)');
+
+        // Check if client IP is not blaclisted as spam source. If yes then add new referrer as invisible
+        $DbResult = $this->System->Database->query('SELECT COUNT(*) FROM `BlackList` WHERE `IP` = "'.$IP.'"');
         $DbRow = $DbResult->fetch_row();
-        if($DbRow[0] == 0) $Visible = '1';
-          else $Visible = '0';
+        if($DbRow[0] == 0)
+        {
+          $Visible = '1';
+          $Description = '';
+        } else
+        {
+          $Visible = '0';
+          $Description = 'Spam';
+        }
 
         // Check if host name is already stored and need just to update hit counter
@@ -55 +62 @@
           $this->System->Database->query('UPDATE `Referrer` SET `Hits` = `Hits` + 1, `DateLast` = NOW(), `LastURL` = "'.
            addslashes($Referrer).'", `LastIP` = "'.$IP.'" WHERE `Id` = '.$DbRow['Id']);
-        } else $this->System->Database->query('INSERT INTO `Referrer` (`Web`, `DateFirst`, `DateLast`, `LastURL`, `Hits`, `LastIP`, `Visible`) '.
-                  'VALUES ("'.$HostName.'", NOW(), NOW( ), "'.addslashes($Referrer).'", 1, "'.$IP.'", '.$Visible.')');
+        } else
+        {
+          $this->System->Database->query('INSERT INTO `Referrer` (`Web`, `DateFirst`, `DateLast`, `LastURL`, `Hits`, `LastIP`, `Visible`, `Description`) '.
+                  'VALUES ("'.$HostName.'", NOW(), NOW( ), "'.addslashes($Referrer).'", 1, "'.$IP.'", '.$Visible.', "'.$Description.'")');
+        }
       }
     }
@@ -177 +187 @@
           $Item = $DbResult->fetch_assoc();
           $this->Database->update('Referrer', 'Id='.$_GET['id'], array('Visible' => 0, 'Description' => 'Spam'));
+          $DbResult2 = $this->Database->select('BlackList', '*', 'IP="'.$Item['LastIP'].'"');
+          if($DbResult2->num_rows == 0)
+          {
+            $this->Database->insert('BlackList', array('Time' => 'NOW()', 'IP' => $Item['LastIP']));
+          }
           $Output = ShowMessage(T('Set as spam'), MESSAGE_CRITICAL);
           $Output .= $this->ShowList();