From cb28484fe3f92ca80d3b7903d125b8ef1ffcc5c9 Mon Sep 17 00:00:00 2001
From: Don Armstrong <don@donarmstrong.com>
Date: Sat, 9 Jun 2007 16:05:19 -0700
Subject: [PATCH]   - Don't doubly select users

---
 cgi/pkgreport.cgi | 6 +++++-
 debian/changelog  | 1 +
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/cgi/pkgreport.cgi b/cgi/pkgreport.cgi
index 5a1d86b..7f16855 100755
--- a/cgi/pkgreport.cgi
+++ b/cgi/pkgreport.cgi
@@ -161,7 +161,7 @@ quitcgi("You have to choose something to select by") if (!$found);
 my %bugusertags;
 my %ut;
 for my $user (split /[\s*,]+/, $users) {
-    next unless ($user =~ m/..../);
+    next unless length($user) >= 4;
     add_user($user);
 }
 
@@ -206,10 +206,14 @@ set_option("use-bug-idx", defined($param{'use-bug-idx'}) ? $param{'use-bug-idx'}
 set_option("show_list_header", $show_list_header);
 set_option("show_list_footer", $show_list_footer);
 
+our %seen_users;
 sub add_user {
     my $ut = \%ut;
     my $u = shift;
 
+    return if $seen_users{$u};
+    $seen_users{$u} = 1;
+
     my $user = Debbugs::User::get_user($u);
 
     my %vis = map { $_, 1 } @{$user->{"visible_cats"}};
diff --git a/debian/changelog b/debian/changelog
index 10e060d..44b9a2c 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -179,6 +179,7 @@ debbugs (2.4.2) UNRELEASED; urgency=low
     - Add PR-Package header for control messages (closes: #414023)
     - Fix double leading spaces of format=flowed messages
       (closes: #428056)
+    - Don't doubly select users
 
   
  -- Colin Watson <cjwatson@debian.org>  Fri, 20 Jun 2003 18:57:25 +0100
-- 
2.39.2