From: Dylan Hardison Date: Wed, 16 Dec 2015 01:21:08 +0000 (-0500) Subject: Bug 1232180 - Incorrect regexp used to filter bug IDs in Bugzilla::WebService::BugUse... X-Git-Tag: release-5.0.2~13 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c7affd0d2497640de0c7967543a2f2732979ea51;p=thirdparty%2Fbugzilla.git Bug 1232180 - Incorrect regexp used to filter bug IDs in Bugzilla::WebService::BugUserLastVisit r=dkl,a=dkl --- diff --git a/Bugzilla/WebService/BugUserLastVisit.pm b/Bugzilla/WebService/BugUserLastVisit.pm index c028125948..56e91ec31b 100644 --- a/Bugzilla/WebService/BugUserLastVisit.pm +++ b/Bugzilla/WebService/BugUserLastVisit.pm @@ -36,7 +36,7 @@ sub update { # Cache permissions for bugs. This highly reduces the number of calls to the # DB. visible_bugs() is only able to handle bug IDs, so we have to skip # aliases. - $user->visible_bugs([grep /^[0-9]$/, @$ids]); + $user->visible_bugs([grep /^[0-9]+$/, @$ids]); $dbh->bz_start_transaction(); my @results; @@ -72,7 +72,7 @@ sub get { # Cache permissions for bugs. This highly reduces the number of calls to # the DB. visible_bugs() is only able to handle bug IDs, so we have to # skip aliases. - $user->visible_bugs([grep /^[0-9]$/, @$ids]); + $user->visible_bugs([grep /^[0-9]+$/, @$ids]); my %last_visit = map { $_->bug_id => $_->last_visit_ts } @{ $user->last_visited($ids) }; @last_visits = map { $self->_bug_user_last_visit_to_hash($_->id, $last_visit{$_}, $params) } @$ids;