diff options
| author | Dylan Hardison <dylan@mozilla.com> | 2015-12-16 02:27:38 +0100 |
|---|---|---|
| committer | Dylan Hardison <dylan@mozilla.com> | 2015-12-16 02:27:38 +0100 |
| commit | 76ecb18fcac37ce40f5f4a00bc8215e8b0480ecb (patch) | |
| tree | 9c4ea185aa35675c10865a51bbc7d565b9a0150a | |
| parent | dee37d0636b3fa9572e516e61de0ee2ab74b6fc6 (diff) | |
| download | bugzilla-76ecb18fcac37ce40f5f4a00bc8215e8b0480ecb.tar.gz bugzilla-76ecb18fcac37ce40f5f4a00bc8215e8b0480ecb.tar.xz | |
Bug 1232180 - Incorrect regexp used to filter bug IDs in Bugzilla::WebService::BugUserLastVisit
r=dkl,a=dkl
| -rw-r--r-- | Bugzilla/API/1_0/Resource/BugUserLastVisit.pm | 4 | ||||
| -rw-r--r-- | Bugzilla/WebService/BugUserLastVisit.pm | 4 |
2 files changed, 4 insertions, 4 deletions
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; |