From 85731f09a6374f82ee6b1efcffa236068b643a3c Mon Sep 17 00:00:00 2001
From: "lpsolit%gmail.com" <>
Date: Wed, 21 Dec 2005 05:46:22 +0000
Subject: Bug 320866: Watching the same address twice produces software error -
 Patch by Frédéric Buclin <LpSolit@gmail.com> r=kevin r=bkor a=justdave
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 userprefs.cgi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'userprefs.cgi')

diff --git a/userprefs.cgi b/userprefs.cgi
index a5590e4eb..df2255ce4 100755
--- a/userprefs.cgi
+++ b/userprefs.cgi
@@ -309,12 +309,12 @@ sub SaveEmail {
  
        # The new information given to us by the user.
         my @new_watch_names = split(/[,\s]+/, $cgi->param('watchedusers'));
-        my @new_watch_ids = ();
+        my %new_watch_ids;
         foreach my $username (@new_watch_names) {
             my $watched_userid = DBNameToIdAndCheck(trim($username));
-            push(@new_watch_ids, $watched_userid);
+            $new_watch_ids{$watched_userid} = 1;
         }
-        my ($removed, $added) = diff_arrays($old_watch_ids, \@new_watch_ids);
+        my ($removed, $added) = diff_arrays($old_watch_ids, [keys %new_watch_ids]);
 
         # Remove people who were removed.
         my $delete_sth = $dbh->prepare('DELETE FROM watch WHERE watched = ?'
-- 
cgit v1.2.3-24-g4f1b