opendev/gerrit
Code Issues Proposed changes

Require the user to be signed in to view a group's detail page

Browse Source 
Signed-off-by: Shawn O. Pearce <sop@google.com>
This commit is contained in:
Shawn O. Pearce
2008-12-30 08:39:30 -08:00
parent 60e9a606f3
commit 9a7a375f79
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
appjar/src/main/java/com/google/gerrit/client/admin/AccountGroupScreen.java
View File

@@ -20,9 +20,9 @@ import com.google.gerrit.client.reviewdb.AccountGroup;
import com.google.gerrit.client.reviewdb.AccountGroupMember; import com.google.gerrit.client.reviewdb.AccountGroupMember;
import com.google.gerrit.client.rpc.GerritCallback; import com.google.gerrit.client.rpc.GerritCallback;
import com.google.gerrit.client.ui.AccountDashboardLink; import com.google.gerrit.client.ui.AccountDashboardLink;
import com.google.gerrit.client.ui.AccountScreen;
import com.google.gerrit.client.ui.AccountSuggestOracle; import com.google.gerrit.client.ui.AccountSuggestOracle;
import com.google.gerrit.client.ui.FancyFlexTable; import com.google.gerrit.client.ui.FancyFlexTable;
import com.google.gerrit.client.ui.Screen;
import com.google.gerrit.client.ui.TextSaveButtonListener; import com.google.gerrit.client.ui.TextSaveButtonListener;
import com.google.gwt.user.client.ui.Button; import com.google.gwt.user.client.ui.Button;
import com.google.gwt.user.client.ui.CheckBox; import com.google.gwt.user.client.ui.CheckBox;
@@ -43,7 +43,7 @@ import com.google.gwtjsonrpc.client.VoidResult;
import java.util.HashSet; import java.util.HashSet;
import java.util.List; import java.util.List;
public class AccountGroupScreen extends Screen { public class AccountGroupScreen extends AccountScreen {
private AccountGroup.Id groupId; private AccountGroup.Id groupId;
private AccountInfoCache accounts = AccountInfoCache.empty(); private AccountInfoCache accounts = AccountInfoCache.empty();
private MemberTable members; private MemberTable members;

4
appjar/src/main/java/com/google/gerrit/client/ui/AccountScreen.java
View File

@@ -20,4 +20,8 @@ public class AccountScreen extends Screen {
super(heading); super(heading);
setRequiresSignIn(true); setRequiresSignIn(true);
} }
public AccountScreen() {
this("");
}
} }