opendev/gerrit
Code Issues Proposed changes

Merge "Remove age operator when drilling down from a dashboard to a query"

Browse Source
This commit is contained in:
Edwin Kempin 2014-07-16 10:47:12 +00:00 committed by Gerrit Code Review
commit c1c2b10adf
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/DashboardTable.java
View File

@ -70,7 +70,7 @@ public class DashboardTable extends ChangeTable2 {
int i = 0;
for (String title : titles) {
Section s = new Section();
String query = removeLimit(queries.get(i++));
String query = removeLimitAndAge(queries.get(i++));
s.setTitleWidget(new InlineHyperlink(title, PageLinks.toChangeQuery(query)));
addSection(s);
sections.add(s);
@ -84,11 +84,12 @@ public class DashboardTable extends ChangeTable2 {
});
}
private String removeLimit(String query) {
private String removeLimitAndAge(String query) {
StringBuilder unlimitedQuery = new StringBuilder();
String[] operators = query.split(" ");
for (String o : operators) {
if (!o.startsWith("limit:")) {
if (!o.startsWith("limit:")
&& !o.startsWith("age:") && !o.startsWith("-age:")) {
unlimitedQuery.append(o).append(" ");
}
}