opendev/gerrit
Code Issues Proposed changes

Not displaying the "Sign Out" for CLIENT_SSL_CERT_LDAP.

Browse Source 
When CLIENT_SSL_CERT_LDAP authentication type is used the "Sign Out"
link has no effect as user gets immediately signed in again. This change
removes the "Sign Out" link when this authentication type is used.

Change-Id: I46923206b0c18be0660d078ee37a923ab736d27f
Signed-off-by: Sasa Zivkov <sasa.zivkov@sap.com>
This commit is contained in:
Sasa Zivkov
2011-02-07 15:43:59 +01:00
parent 84739ac801
commit 54b0eea379
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gerrit-gwtui/src/main/java/com/google/gerrit/client/Gerrit.java
View File

@@ -30,6 +30,7 @@ import com.google.gerrit.common.data.SystemInfoService;
import com.google.gerrit.reviewdb.Account;
import com.google.gerrit.reviewdb.AccountDiffPreference;
import com.google.gerrit.reviewdb.AccountGeneralPreferences;
import com.google.gerrit.reviewdb.AuthType;
import com.google.gwt.core.client.EntryPoint;
import com.google.gwt.core.client.GWT;
import com.google.gwt.core.client.JavaScriptObject;
@@ -476,7 +477,9 @@ public class Gerrit implements EntryPoint {
if (signedIn) {
whoAmI();
addLink(menuRight, C.menuSettings(), PageLinks.SETTINGS);
menuRight.add(anchor(C.menuSignOut(), "logout"));
if (cfg.getAuthType() != AuthType.CLIENT_SSL_CERT_LDAP) {
menuRight.add(anchor(C.menuSignOut(), "logout"));
}
} else {
switch (cfg.getAuthType()) {
case HTTP: