]> git.ipfire.org Git - thirdparty/bugzilla.git/commitdiff
Bug 1232180 - Incorrect regexp used to filter bug IDs in Bugzilla::WebService::BugUse...
authorDylan Hardison <dylan@mozilla.com>
Wed, 16 Dec 2015 01:27:38 +0000 (20:27 -0500)
committerDylan Hardison <dylan@mozilla.com>
Wed, 16 Dec 2015 01:27:38 +0000 (20:27 -0500)
r=dkl,a=dkl

Bugzilla/API/1_0/Resource/BugUserLastVisit.pm
Bugzilla/WebService/BugUserLastVisit.pm

index 1cb12a05a6094e5eca6a9b4c09c4e8aa92bcd892..aeb52bbb6a045955062270c51583a0d2e0252406 100644 (file)
@@ -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;
index 498dd9044bacfeade60ba58b23c3607ec2ac17fe..15132736860395ff1f8f6068bd5e1c3b3a9413f4 100644 (file)
@@ -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;