Fix error log

Author: lunny

models/login/source.go

@@ -247,7 +247,7 @@ func CreateSource(source *Source) error {
 	if err != nil {
 		// remove the LoginSource in case of errors while registering configuration
 		if _, err := db.GetEngine(db.DefaultContext).Delete(source); err != nil {
-			log.Error("CreateLoginSource: Error while wrapOpenIDConnectInitializeError: %v", err)
+			log.Error("CreateSource: Error while wrapOpenIDConnectInitializeError: %v", err)
 		}
 	}
 	return err
@@ -294,7 +294,7 @@ func IsSSPIEnabled() bool {
 	}
 	sources, err := ActiveSources(SSPI)
 	if err != nil {
-		log.Error("ActiveLoginSources: %v", err)
+		log.Error("ActiveSources: %v", err)
 		return false
 	}
 	return len(sources) > 0

routers/api/v1/admin/user.go

@@ -33,7 +33,7 @@ func parseLoginSource(ctx *context.APIContext, u *models.User, sourceID int64, l
 		if login.IsErrSourceNotExist(err) {
 			ctx.Error(http.StatusUnprocessableEntity, "", err)
 		} else {
-			ctx.Error(http.StatusInternalServerError, "GetLoginSourceByID", err)
+			ctx.Error(http.StatusInternalServerError, "login.GetSourceByID", err)
 		}
 		return
 	}

routers/web/admin/auths.go

@@ -401,15 +401,15 @@ func EditAuthSourcePost(ctx *context.Context) {
 func DeleteAuthSource(ctx *context.Context) {
 	source, err := login.GetSourceByID(ctx.ParamsInt64(":authid"))
 	if err != nil {
-		ctx.ServerError("GetLoginSourceByID", err)
+		ctx.ServerError("login.GetSourceByID", err)
 		return
 	}
 
 	if err = auth_service.DeleteLoginSource(source); err != nil {
 		if login.IsErrSourceInUse(err) {
 			ctx.Flash.Error(ctx.Tr("admin.auths.still_in_used"))
 		} else {
-			ctx.Flash.Error(fmt.Sprintf("DeleteSource: %v", err))
+			ctx.Flash.Error(fmt.Sprintf("DeleteLoginSource: %v", err))
 		}
 		ctx.JSON(http.StatusOK, map[string]interface{}{
 			"redirect": setting.AppSubURL + "/admin/auths/" + ctx.Params(":authid"),

routers/web/admin/users.go

@@ -60,7 +60,7 @@ func NewUser(ctx *context.Context) {
 
 	sources, err := login.Sources()
 	if err != nil {
-		ctx.ServerError("LoginSources", err)
+		ctx.ServerError("login.Sources", err)
 		return
 	}
 	ctx.Data["Sources"] = sources
@@ -79,7 +79,7 @@ func NewUserPost(ctx *context.Context) {
 
 	sources, err := login.Sources()
 	if err != nil {
-		ctx.ServerError("LoginSources", err)
+		ctx.ServerError("login.Sources", err)
 		return
 	}
 	ctx.Data["Sources"] = sources
@@ -180,7 +180,7 @@ func prepareUserInfo(ctx *context.Context) *models.User {
 	if u.LoginSource > 0 {
 		ctx.Data["LoginSource"], err = login.GetSourceByID(u.LoginSource)
 		if err != nil {
-			ctx.ServerError("GetLoginSourceByID", err)
+			ctx.ServerError("login.GetSourceByID", err)
 			return nil
 		}
 	} else {
@@ -189,7 +189,7 @@ func prepareUserInfo(ctx *context.Context) *models.User {
 
 	sources, err := login.Sources()
 	if err != nil {
-		ctx.ServerError("LoginSources", err)
+		ctx.ServerError("login.Sources", err)
 		return nil
 	}
 	ctx.Data["Sources"] = sources

routers/web/user/oauth.go

@@ -316,7 +316,7 @@ func AuthorizeOAuth(ctx *context.Context) {
 
 	user, err := models.GetUserByID(app.UID)
 	if err != nil {
-		ctx.ServerError("LoadUser", err)
+		ctx.ServerError("GetUserByID", err)
 		return
 	}