Fix #173

public/js/app.js

@@ -650,3 +650,7 @@ function initRepoSetting() {
         }
     });
 })(jQuery);
+
+String.prototype.endsWith = function(suffix) {
+    return this.indexOf(suffix, this.length - suffix.length) !== -1;
+};

routers/admin/auths.go

@@ -9,6 +9,7 @@ import (
 
 	"github.com/go-martini/martini"
 	"github.com/go-xorm/core"
+
 	"github.com/gogits/gogs/models"
 	"github.com/gogits/gogs/modules/auth"
 	"github.com/gogits/gogs/modules/auth/ldap"
@@ -114,7 +115,8 @@ func EditAuthSourcePost(ctx *middleware.Context, form auth.AuthenticationForm) {
 	}
 
 	var config core.Conversion
-	if form.Type == models.LT_LDAP {
+	switch form.Type {
+	case models.LT_LDAP:
 		config = &models.LDAPConfig{
 			Ldapsource: ldap.Ldapsource{
 				Host:         form.Host,
@@ -127,13 +129,16 @@ func EditAuthSourcePost(ctx *middleware.Context, form auth.AuthenticationForm) {
 				Name:         form.AuthName,
 			},
 		}
-	} else if form.Type == models.LT_SMTP {
+	case models.LT_SMTP:
 		config = &models.SMTPConfig{
 			Auth: form.SmtpAuth,
 			Host: form.Host,
 			Port: form.Port,
 			TLS:  form.Tls,
 		}
+	default:
+		ctx.Error(400)
+		return
 	}
 
 	u := models.LoginSource{
@@ -145,10 +150,7 @@ func EditAuthSourcePost(ctx *middleware.Context, form auth.AuthenticationForm) {
 	}
 
 	if err := models.UpdateSource(&u); err != nil {
-		switch err {
-		default:
 		ctx.Handle(500, "admin.auths.EditAuth", err)
-		}
 		return
 	}
 

templates/admin/auths/edit.tmpl

@@ -32,7 +32,6 @@
                     </div>
 
                     {{if eq $type 2}}
-
                     <div class="form-group {{if .Err_Domain}}has-error has-feedback{{end}}">
                         <label class="col-md-3 control-label">Domain: </label>
                         <div class="col-md-7">
@@ -81,9 +80,7 @@
                             <input name="ms_ad_sa" class="form-control" placeholder="Type Ms Ad SA" value="{{.Source.LDAP.MsAdSAFormat}}" required="required">
                         </div>
                     </div>
-                    {{else}}
-                    {{if eq $type 3}}
-
+                    {{else if eq $type 3}}
                     <div class="form-group {{if .Err_TLS}}has-error has-feedback{{end}}">
                         <label class="col-md-3 control-label">SMTP Auth: </label>
                         <div class="col-md-7">
@@ -119,7 +116,6 @@
                         </div>
                     </div>
                     {{end}}
-                    {{end}}
                     
                     <div class="form-group {{if .Err_TLS}}has-error has-feedback{{end}}">
                             <label class="col-md-3 control-label">Auto Register: </label>