From 0c5ad79e568cbda6362ec5ab3b8d72f6642b1b64 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Jan=20Kr=C3=BCger?= Date: Mon, 15 Nov 2010 21:52:15 +0100 Subject: [PATCH] Girocco::Project: when editing user list, filter non-existent users MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit We used to accept usernames into the list even if no corresponding user account existed. For now, silently filter those, which is probably an improvement. Signed-off-by: Jan Krüger --- Girocco/Project.pm | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Girocco/Project.pm b/Girocco/Project.pm index 5f32320..e4bd91c 100644 --- a/Girocco/Project.pm +++ b/Girocco/Project.pm @@ -349,8 +349,7 @@ sub cgi_fill { or $gcgi->err("Invalid homepage URL. Note that only HTTP protocol is supported. If the URL contains funny characters, contact me."); } - # FIXME: Permit only existing users - $self->{users} = [grep { Girocco::User::valid_name($_) } $cgi->param('user')]; + $self->{users} = [grep { Girocco::User::valid_name($_) && Girocco::User::does_exist($_) } $cgi->param('user')]; $self->{notifymail} = $gcgi->wparam('notifymail'); if ($self->{notifymail}) { -- 2.11.4.GIT