code refactor

src/webapi/router/router_login.go

@@ -324,9 +324,18 @@ func loginCallbackCas(c *gin.Context) {
 	ginx.Dangerous(err)
 	if user != nil {
 		if config.C.CAS.CoverAttributes {
-			user.Nickname = ret.Nickname
-			user.Email = ret.Email
-			user.Phone = ret.Phone
+			if ret.Nickname != "" {
+				user.Nickname = ret.Nickname
+			}
+
+			if ret.Email != "" {
+				user.Email = ret.Email
+			}
+
+			if ret.Phone != "" {
+				user.Phone = ret.Phone
+			}
+
 			user.UpdateAt = time.Now().Unix()
 			ginx.Dangerous(user.Update("email", "nickname", "phone", "update_at"))
 		}
@@ -417,11 +426,19 @@ func loginCallbackOAuth(c *gin.Context) {
 
 	if user != nil {
 		if config.C.OAuth.CoverAttributes {
-			user.Nickname = ret.Nickname
-			user.Email = ret.Email
-			user.Phone = ret.Phone
-			user.UpdateAt = time.Now().Unix()
+			if ret.Nickname != "" {
+				user.Nickname = ret.Nickname
+			}
+
+			if ret.Email != "" {
+				user.Email = ret.Email
+			}
+
+			if ret.Phone != "" {
+				user.Phone = ret.Phone
+			}
 
+			user.UpdateAt = time.Now().Unix()
 			user.Update("email", "nickname", "phone", "update_at")
 		}
 	} else {