summaryrefslogtreecommitdiffstats
path: root/routers
diff options
context:
space:
mode:
Diffstat (limited to 'routers')
-rw-r--r--routers/org/org.go45
-rw-r--r--routers/repo/setting.go18
-rw-r--r--routers/user/user.go2
3 files changed, 59 insertions, 6 deletions
diff --git a/routers/org/org.go b/routers/org/org.go
index c036a8e5dc..7b2c4d7320 100644
--- a/routers/org/org.go
+++ b/routers/org/org.go
@@ -30,7 +30,6 @@ func Members(ctx *middleware.Context, params martini.Params) {
ctx.HTML(200, "org/members")
}
-
func New(ctx *middleware.Context) {
ctx.Data["Title"] = "Create An Organization"
ctx.HTML(200, NEW)
@@ -160,3 +159,47 @@ func SettingsPost(ctx *middleware.Context, params martini.Params, form auth.OrgS
ctx.Flash.Success("Organization profile has been successfully updated.")
ctx.Redirect("/org/" + org.Name + "/settings")
}
+
+func DeletePost(ctx *middleware.Context, params martini.Params) {
+ ctx.Data["Title"] = "Settings"
+
+ org, err := models.GetUserByName(params["org"])
+ if err != nil {
+ if err == models.ErrUserNotExist {
+ ctx.Handle(404, "org.DeletePost(GetUserByName)", err)
+ } else {
+ ctx.Handle(500, "org.DeletePost(GetUserByName)", err)
+ }
+ return
+ }
+ ctx.Data["Org"] = org
+
+ if !models.IsOrganizationOwner(org.Id, ctx.User.Id) {
+ ctx.Error(403)
+ return
+ }
+
+ tmpUser := models.User{
+ Passwd: ctx.Query("password"),
+ Salt: ctx.User.Salt,
+ }
+ tmpUser.EncodePasswd()
+ if tmpUser.Passwd != ctx.User.Passwd {
+ ctx.Flash.Error("Password is not correct. Make sure you are owner of this account.")
+ } else {
+ if err := models.DeleteOrganization(org); err != nil {
+ switch err {
+ case models.ErrUserOwnRepos:
+ ctx.Flash.Error("This organization still have ownership of repository, you have to delete or transfer them first.")
+ default:
+ ctx.Handle(500, "org.DeletePost(DeleteOrganization)", err)
+ return
+ }
+ } else {
+ ctx.Redirect("/")
+ return
+ }
+ }
+
+ ctx.Redirect("/org/" + org.Name + "/settings")
+}
diff --git a/routers/repo/setting.go b/routers/repo/setting.go
index 3d48e79c3d..e97eca1239 100644
--- a/routers/repo/setting.go
+++ b/routers/repo/setting.go
@@ -122,13 +122,23 @@ func SettingPost(ctx *middleware.Context, form auth.RepoSettingForm) {
return
}
- if err := models.DeleteRepository(ctx.User.Id, ctx.Repo.Repository.Id, ctx.User.LowerName); err != nil {
- ctx.Handle(500, "setting.Delete", err)
+ if ctx.Repo.Owner.IsOrganization() &&
+ !models.IsOrganizationOwner(ctx.Repo.Owner.Id, ctx.User.Id) {
+ ctx.Error(403)
return
}
- log.Trace("%s Repository deleted: %s/%s", ctx.Req.RequestURI, ctx.User.LowerName, ctx.Repo.Repository.LowerName)
- ctx.Redirect("/")
+ if err := models.DeleteRepository(ctx.Repo.Owner.Id, ctx.Repo.Repository.Id, ctx.Repo.Owner.Name); err != nil {
+ ctx.Handle(500, "setting.Delete(DeleteRepository)", err)
+ return
+ }
+ log.Trace("%s Repository deleted: %s/%s", ctx.Req.RequestURI, ctx.Repo.Owner.LowerName, ctx.Repo.Repository.LowerName)
+
+ if ctx.Repo.Owner.IsOrganization() {
+ ctx.Redirect("/org/" + ctx.Repo.Owner.Name + "/dashboard")
+ } else {
+ ctx.Redirect("/")
+ }
}
}
diff --git a/routers/user/user.go b/routers/user/user.go
index a402744b86..561fe1c111 100644
--- a/routers/user/user.go
+++ b/routers/user/user.go
@@ -296,7 +296,7 @@ func DeletePost(ctx *middleware.Context) {
case models.ErrUserOwnRepos:
ctx.Flash.Error("Your account still have ownership of repository, you have to delete or transfer them first.")
default:
- ctx.Handle(500, "user.Delete(DeleteUser)", err)
+ ctx.Handle(500, "user.DeletePost(DeleteUser)", err)
return
}
} else {