Set last login when activating account (#21731)

wolfogre · lunny · web-flow · commit 5a6cba4cf43f · 2022-11-09T11:42:06.000-05:00
Fix #21698. Set the last login time to the current time when activating the user successfully. Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
diff --git a/routers/web/auth/auth.go b/routers/web/auth/auth.go
@@ -783,6 +783,13 @@ func handleAccountActivation(ctx *context.Context, user *user_model.User) {
 		return
 	}
 
+	// Register last login
+	user.SetLastLogin()
+	if err := user_model.UpdateUserCols(ctx, user, "last_login_unix"); err != nil {
+		ctx.ServerError("UpdateUserCols", err)
+		return
+	}
+
 	ctx.Flash.Success(ctx.Tr("auth.account_activated"))
 	ctx.Redirect(setting.AppSubURL + "/")
 }

Original file line number	Diff line number	Diff line change
`@@ -783,6 +783,13 @@ func handleAccountActivation(ctx context.Context, user user_model.User) {`
`783`	`783`	`return`
`784`	`784`	`}`
`785`	`785`
	`786`	`+ // Register last login`
	`787`	`+ user.SetLastLogin()`
	`788`	`+ if err := user_model.UpdateUserCols(ctx, user, "last_login_unix"); err != nil {`
	`789`	`+ ctx.ServerError("UpdateUserCols", err)`
	`790`	`+ return`
	`791`	`+ }`
	`792`	`+`
`786`	`793`	`ctx.Flash.Success(ctx.Tr("auth.account_activated"))`
`787`	`794`	`ctx.Redirect(setting.AppSubURL + "/")`
`788`	`795`	`}`