Code cleanup

Related: #18466 (review)
Author-Change-Id: IB#1105051

Author: pboguslawski

models/user/user.go

@@ -830,7 +830,7 @@ func validateUser(u *User) error {
 // updateUserAllowed is used to block updating selected user fields when local user managemement is disabled.
 func updateUserAllowed(u *User) error {
 	// Don't allow changes of selected user fields if local user management is disabled.
-	if setting.Service.DisableLocalUserManagement && (u.Type == UserTypeIndividual) {
+	if setting.Service.DisableLocalUserManagement && u.Type == UserTypeIndividual {
 		if currUser, err := GetUserByID(u.ID); err == nil {
 			if currUser.Name != u.Name {
 				return fmt.Errorf("cannot change user %s username; local user management disabled", u.Name)

routers/web/admin/emails.go

@@ -118,7 +118,6 @@ func isKeywordValid(keyword string) bool {
 
 // ActivateEmail serves a POST request for activating/deactivating a user's email
 func ActivateEmail(ctx *context.Context) {
-
 	// Don't allow to activate/deactivate emails if local user management is disabled.
 	if setting.Service.DisableLocalUserManagement {
 		ctx.ServerError("ActivateEmail", fmt.Errorf("cannot activate email; local user management disabled"))