commit b2ccfb276adafbdb26461dfb3660cf7dcffeea0f
author David Pursehouse <dpursehouse@collab.net> Wed Mar 01 17:48:33 2017 +0900
committer David Pursehouse <dpursehouse@collab.net> Wed Mar 01 17:51:28 2017 +0900
tree f8db24e29265b892d81e411d624999cdcdfd8a0f
parent cdc381ea72bcaf335356abc48e01dd88c0be4420

Remove references to isGitBasicAuth in GWT UI

Change-Id: I45c60597e636918eaecfa57bc4c9be11bf2110bf

gerrit-gwtui-common/src/main/java/com/google/gerrit/client/info/AuthInfo.java

diff --git a/gerrit-gwtui-common/src/main/java/com/google/gerrit/client/info/AuthInfo.java b/gerrit-gwtui-common/src/main/java/com/google/gerrit/client/info/AuthInfo.java
index 9ae74ac..2f6ef79 100644
--- a/gerrit-gwtui-common/src/main/java/com/google/gerrit/client/info/AuthInfo.java
+++ b/gerrit-gwtui-common/src/main/java/com/google/gerrit/client/info/AuthInfo.java
@@ -81,10 +81,7 @@
   }
 
   public final boolean isHttpPasswordSettingsEnabled() {
-    if (isGitBasicAuth() && gitBasicAuthPolicy() == GitBasicAuthPolicy.LDAP) {
-      return false;
-    }
-    return true;
+    return gitBasicAuthPolicy() != GitBasicAuthPolicy.LDAP;
   }
 
   public final GitBasicAuthPolicy gitBasicAuthPolicy() {
@@ -108,8 +105,6 @@
 
   public final native String httpPasswordUrl() /*-{ return this.http_password_url; }-*/;
 
-  public final native boolean isGitBasicAuth() /*-{ return this.is_git_basic_auth || false; }-*/;
-
   private native String gitBasicAuthPolicyRaw() /*-{ return this.git_basic_auth_policy; }-*/;
 
   private native String authTypeRaw() /*-{ return this.auth_type; }-*/;

gerrit-gwtui/src/main/java/com/google/gerrit/client/Dispatcher.java

diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/Dispatcher.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/Dispatcher.java
index f5f2bde..e4dc40d 100644
--- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/Dispatcher.java
+++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/Dispatcher.java
@@ -592,9 +592,7 @@
               return new MyPasswordScreen();
             }
 
-            if (matchExact(SETTINGS_OAUTH_TOKEN, token)
-                && Gerrit.info().auth().isOAuth()
-                && Gerrit.info().auth().isGitBasicAuth()) {
+            if (matchExact(SETTINGS_OAUTH_TOKEN, token) && Gerrit.info().auth().isOAuth()) {
               return new MyOAuthTokenScreen();
             }
 

gerrit-gwtui/src/main/java/com/google/gerrit/client/account/SettingsScreen.java

diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/SettingsScreen.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/SettingsScreen.java
index 49ce13e..bbc192d 100644
--- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/SettingsScreen.java
+++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/SettingsScreen.java
@@ -46,7 +46,7 @@
     if (Gerrit.info().auth().isHttpPasswordSettingsEnabled()) {
       linkByGerrit(Util.C.tabHttpAccess(), PageLinks.SETTINGS_HTTP_PASSWORD);
     }
-    if (Gerrit.info().auth().isOAuth() && Gerrit.info().auth().isGitBasicAuth()) {
+    if (Gerrit.info().auth().isOAuth()) {
       linkByGerrit(Util.C.tabOAuthToken(), PageLinks.SETTINGS_OAUTH_TOKEN);
     }
     if (Gerrit.info().gerrit().editGpgKeys()) {