From 3d61e0e7146e9e9414401ebe85493e16014ae9d2 Mon Sep 17 00:00:00 2001
From: "Kyle J. McKay" <mackyle@gmail.com>
Date: Wed, 28 Jan 2015 05:20:05 -0800
Subject: [PATCH] Project.pm: remove duplicate users from the project
 membership list

There's no reason to allow duplicates of a user id.
---
 Girocco/Project.pm | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/Girocco/Project.pm b/Girocco/Project.pm
index 30455ac..a386e40 100644
--- a/Girocco/Project.pm
+++ b/Girocco/Project.pm
@@ -454,7 +454,15 @@ sub cgi_fill {
 			or $gcgi->err("Invalid homepage URL. Note that only HTTP protocol is supported. If the URL contains funny characters, contact me.");
 	}
 
-	$self->{users} = [grep { Girocco::User::valid_name($_) && Girocco::User::does_exist($_) } $cgi->param('user')];
+	my %users = ();
+	my @users = ();
+	foreach my $user ($cgi->param('user')) {
+		if (!exists($users{$user})) {
+			$users{$user} = 1;
+			push(@users, $user) if Girocco::User::valid_name($user) && Girocco::User::does_exist($user)
+		}
+	}
+	$self->{users} = \@users;
 
 	$self->{HEAD} = $cgi->param('HEAD') if $cgi->param('HEAD');
 
-- 
2.11.4.GIT