opendev/gerrit
Code Issues Proposed changes

Merge "Show filter field on project/group list screen before populating the list" into stable-2.6

Browse Source
This commit is contained in:
Shawn Pearce 2013-04-17 14:31:29 +00:00 committed by Gerrit Code Review
commit dbcfd690f1
2 changed files with 8 additions and 6 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/admin/GroupListScreen.java
View File

@ -18,7 +18,7 @@ import static com.google.gerrit.common.PageLinks.ADMIN_GROUPS;
import com.google.gerrit.client.Gerrit;
import com.google.gerrit.client.groups.GroupMap;
import com.google.gerrit.client.rpc.ScreenLoadCallback;
import com.google.gerrit.client.rpc.GerritCallback;
import com.google.gerrit.client.ui.AccountScreen;
import com.google.gerrit.client.ui.FilteredUserInterface;
import com.google.gerrit.client.ui.IgnoreOutdatedFilterResultsCallbackWrapper;
@ -54,6 +54,7 @@ public class GroupListScreen extends AccountScreen implements FilteredUserInterf
@Override
protected void onLoad() {
super.onLoad();
display();
refresh();
}
@ -62,9 +63,9 @@ public class GroupListScreen extends AccountScreen implements FilteredUserInterf
: ADMIN_GROUPS + "?filter=" + URL.encodeQueryString(subname));
GroupMap.match(subname,
new IgnoreOutdatedFilterResultsCallbackWrapper<GroupMap>(this,
new ScreenLoadCallback<GroupMap>(this) {
new GerritCallback<GroupMap>() {
@Override
protected void preDisplay(final GroupMap result) {
public void onSuccess(GroupMap result) {
groups.display(result, subname);
groups.finishDisplay();
}

7
gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectListScreen.java
View File

@ -21,7 +21,7 @@ import com.google.gerrit.client.Gerrit;
import com.google.gerrit.client.GitwebLink;
import com.google.gerrit.client.projects.ProjectInfo;
import com.google.gerrit.client.projects.ProjectMap;
import com.google.gerrit.client.rpc.ScreenLoadCallback;
import com.google.gerrit.client.rpc.GerritCallback;
import com.google.gerrit.client.ui.FilteredUserInterface;
import com.google.gerrit.client.ui.HighlightingInlineHyperlink;
import com.google.gerrit.client.ui.IgnoreOutdatedFilterResultsCallbackWrapper;
@ -63,6 +63,7 @@ public class ProjectListScreen extends Screen implements FilteredUserInterface {
@Override
protected void onLoad() {
super.onLoad();
display();
refresh();
}
@ -71,9 +72,9 @@ public class ProjectListScreen extends Screen implements FilteredUserInterface {
: ADMIN_PROJECTS + "?filter=" + URL.encodeQueryString(subname));
ProjectMap.match(subname,
new IgnoreOutdatedFilterResultsCallbackWrapper<ProjectMap>(this,
new ScreenLoadCallback<ProjectMap>(this) {
new GerritCallback<ProjectMap>() {
@Override
protected void preDisplay(final ProjectMap result) {
public void onSuccess(ProjectMap result) {
projects.display(result);
}
}));