opendev/gerrit
Code Issues Proposed changes

Changescreen: correct topic search query

Browse Source 
There are two problems with the current code:
1) the OR operator must be in capital letters
2) the OR operator is not a prefix operator, but an infix operator.

Change-Id: Icf91285096bd65753948c55578f86d9e80a4d13b
Signed-off-by: Stefan Beller <sbeller@google.com>
This commit is contained in:
Stefan Beller 2016-06-22 09:58:38 -07:00
parent 8fea317a86
commit e945e54c7b
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-common/src/main/java/com/google/gerrit/common/PageLinks.java
View File

@ -131,8 +131,8 @@ public class PageLinks {
case DRAFT:
case MERGED:
case NEW:
return toChangeQuery(op("topic", topic) + " or(" +
status(Status.NEW) + " " +
return toChangeQuery(op("topic", topic) + " (" +
status(Status.NEW) + " OR " +
status(Status.MERGED) + ")");
}
return toChangeQuery(status(status) + " " + op("topic", topic));