From 89de356424ce5717e6b5e0dbbd65609b84df40a0 Mon Sep 17 00:00:00 2001
From: "gerv%gerv.net" <>
Date: Thu, 22 Jan 2004 16:10:53 +0000
Subject: Bug 227155 - make sure running collectstats.pl twice in a day, or
 migrating data from old charts to new charts where there are duplicate
 entries, doesn't cause an SQL error. Patch by gerv; r,a=justdave.

---
 collectstats.pl | 7 +++++++
 1 file changed, 7 insertions(+)

(limited to 'collectstats.pl')

diff --git a/collectstats.pl b/collectstats.pl
index 002d5fba2..bc7b24884 100755
--- a/collectstats.pl
+++ b/collectstats.pl
@@ -453,6 +453,12 @@ sub CollectSeriesData {
                             "(series_id, date, value) " .
                             "VALUES (?, " . $dbh->quote($today) . ", ?)");
 
+    # We delete from the table beforehand, to avoid SQL errors if people run
+    # collectstats.pl twice on the same day.
+    my $deletesth = $dbh->prepare("DELETE FROM series_data 
+                                   WHERE series_id = ? AND date = " .
+                                   $dbh->quote($today));
+                                     
     foreach my $series_id (keys %$serieses) {
         # We set up the user for Search.pm's permission checking - each series
         # runs with the permissions of its creator.
@@ -470,6 +476,7 @@ sub CollectSeriesData {
         
         my $count = scalar(@$data) || 0;
 
+        $deletesth->execute($series_id);
         $sth->execute($series_id, $count);
     }
 }
-- 
cgit v1.2.3-24-g4f1b