From 06a17395530bdda809060442b5bb230edfa216f3 Mon Sep 17 00:00:00 2001 From: Lunny Xiao Date: Thu, 7 Feb 2019 14:51:23 +0800 Subject: [PATCH] fix bug when deleting a linked account will removed all (#5989) --- routers/user/setting/security.go | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/routers/user/setting/security.go b/routers/user/setting/security.go index 862e4413c..f8f5cc0cd 100644 --- a/routers/user/setting/security.go +++ b/routers/user/setting/security.go @@ -34,10 +34,15 @@ func Security(ctx *context.Context) { // DeleteAccountLink delete a single account link func DeleteAccountLink(ctx *context.Context) { - if _, err := models.RemoveAccountLink(ctx.User, ctx.QueryInt64("loginSourceID")); err != nil { - ctx.Flash.Error("RemoveAccountLink: " + err.Error()) + id := ctx.QueryInt64("id") + if id <= 0 { + ctx.Flash.Error("Account link id is not given") } else { - ctx.Flash.Success(ctx.Tr("settings.remove_account_link_success")) + if _, err := models.RemoveAccountLink(ctx.User, id); err != nil { + ctx.Flash.Error("RemoveAccountLink: " + err.Error()) + } else { + ctx.Flash.Success(ctx.Tr("settings.remove_account_link_success")) + } } ctx.JSON(200, map[string]interface{}{