Bug 451449: "Can't use undefined value as ARRAY reference" when logging into a new...

Patch By Max Kanat-Alexander <mkanat@bugzilla.org> r=LpSolit, a=LpSolit

diff --git a/Bugzilla/User.pm b/Bugzilla/User.pm
index 760723279765ab03797e593e0b0e1a42e9fabc75..ccee85755b50a2899e3644df4fdf4804f654d791 100644 (file)
--- a/Bugzilla/User.pm
+++ b/Bugzilla/User.pm
@@ -445,15 +445,15 @@ sub bless_groups {
                        AND groups.id=user_group_map.group_id)
                      OR (groups.id = ggm.grantor_id
                          AND ggm.grant_type = ' . GROUP_BLESS . '
-                         AND ggm.member_id ' . $dbh->sql_in(\@group_ids)
+                         AND ' . $dbh->sql_in('ggm.member_id', \@group_ids)
                      . ') )';
 
     # If visibilitygroups are used, restrict the set of groups.
     if (Bugzilla->params->{'usevisibilitygroups'}) {
         return [] if !$self->visible_groups_as_string;
         # Users need to see a group in order to bless it.
-        $query .= " AND groups.id "
-                  . $dbh->sql_in($self->visible_groups_inherited);
+        $query .= " AND "
+            . $dbh->sql_in('groups.id', $self->visible_groups_inherited);
     }
 
     my $ids = $dbh->selectcol_arrayref($query, undef, $self->id);