Browse Source

user/settings: fix bad error handle of leave organization

Unknwon 8 years ago
parent
commit
78f94986e3
1 changed files with 7 additions and 5 deletions
  1. 7 5
      routers/user/setting.go

+ 7 - 5
routers/user/setting.go

@@ -442,11 +442,13 @@ func SettingsOrganizations(ctx *context.Context) {
 
 func SettingsLeaveOrganization(ctx *context.Context) {
 	err := models.RemoveOrgUser(ctx.QueryInt64("id"), ctx.User.ID)
-	if models.IsErrLastOrgOwner(err) {
-		ctx.Flash.Error(ctx.Tr("form.last_org_owner"))
-	} else {
-		ctx.Handle(500, "RemoveOrgUser", err)
-		return
+	if err != nil {
+		if models.IsErrLastOrgOwner(err) {
+			ctx.Flash.Error(ctx.Tr("form.last_org_owner"))
+		} else {
+			ctx.Handle(500, "RemoveOrgUser", err)
+			return
+		}
 	}
 
 	ctx.JSON(200, map[string]interface{}{