From 76ecb18fcac37ce40f5f4a00bc8215e8b0480ecb Mon Sep 17 00:00:00 2001 From: Dylan Hardison Date: Tue, 15 Dec 2015 20:27:38 -0500 Subject: Bug 1232180 - Incorrect regexp used to filter bug IDs in Bugzilla::WebService::BugUserLastVisit r=dkl,a=dkl --- Bugzilla/API/1_0/Resource/BugUserLastVisit.pm | 4 ++-- Bugzilla/WebService/BugUserLastVisit.pm | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'Bugzilla') diff --git a/Bugzilla/API/1_0/Resource/BugUserLastVisit.pm b/Bugzilla/API/1_0/Resource/BugUserLastVisit.pm index 1cb12a05a..aeb52bbb6 100644 --- a/Bugzilla/API/1_0/Resource/BugUserLastVisit.pm +++ b/Bugzilla/API/1_0/Resource/BugUserLastVisit.pm @@ -71,7 +71,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; @@ -107,7 +107,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 { _bug_user_last_visit_to_hash($_, $last_visit{$_}, $params) } @$ids; diff --git a/Bugzilla/WebService/BugUserLastVisit.pm b/Bugzilla/WebService/BugUserLastVisit.pm index 498dd9044..151327368 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($_, $last_visit{$_}, $params) } @$ids; -- cgit v1.2.3-24-g4f1b