Set last login when activating account (#21731) (#21754)

wolfogre · lunny · web-flow · commit 43bddc1405fc · 2022-11-10T11:15:28.000+08:00
Backport #21731. 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
@@ -775,6 +775,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
`@@ -775,6 +775,13 @@ func handleAccountActivation(ctx context.Context, user user_model.User) {`
`775`	`775`	`return`
`776`	`776`	`}`
`777`	`777`
	`778`	`+ // Register last login`
	`779`	`+ user.SetLastLogin()`
	`780`	`+ if err := user_model.UpdateUserCols(ctx, user, "last_login_unix"); err != nil {`
	`781`	`+ ctx.ServerError("UpdateUserCols", err)`
	`782`	`+ return`
	`783`	`+ }`
	`784`	`+`
`778`	`785`	`ctx.Flash.Success(ctx.Tr("auth.account_activated"))`
`779`	`786`	`ctx.Redirect(setting.AppSubURL + "/")`
`780`	`787`	`}`