summaryrefslogtreecommitdiffstats
path: root/modules
diff options
context:
space:
mode:
authorUnknwon <u@gogs.io>2015-07-26 19:22:17 +0800
committerUnknwon <u@gogs.io>2015-07-26 19:22:17 +0800
commit6f8e388b5530e85f141ea3aa345b1c6842fbe1f5 (patch)
tree3ea3119fa2a1292c801aaddddc00308c434bf6ca /modules
parent436ef5b50a32c7ff2053bce7f5b3eaf6abe0c984 (diff)
downloadgitea-6f8e388b5530e85f141ea3aa345b1c6842fbe1f5.tar.gz
gitea-6f8e388b5530e85f141ea3aa345b1c6842fbe1f5.zip
fix #1169
- prevent create reop on existed path
Diffstat (limited to 'modules')
-rw-r--r--modules/git/repo_tag.go2
-rw-r--r--modules/git/utils.go8
-rw-r--r--modules/middleware/org.go6
3 files changed, 15 insertions, 1 deletions
diff --git a/modules/git/repo_tag.go b/modules/git/repo_tag.go
index ed994d48ae..45a1df7083 100644
--- a/modules/git/repo_tag.go
+++ b/modules/git/repo_tag.go
@@ -27,7 +27,7 @@ func (repo *Repository) GetTags() ([]string, error) {
}
stdout, stderr, err := com.ExecCmdDir(repo.Path, "git", "tag", "-l")
if err != nil {
- return nil, errors.New(stderr)
+ return nil, concatenateError(err, stderr)
}
tags := strings.Split(stdout, "\n")
return tags[:len(tags)-1], nil
diff --git a/modules/git/utils.go b/modules/git/utils.go
index 6abbca557b..78792aaf5e 100644
--- a/modules/git/utils.go
+++ b/modules/git/utils.go
@@ -7,6 +7,7 @@ package git
import (
"bytes"
"container/list"
+ "fmt"
"os"
"path/filepath"
"strings"
@@ -67,3 +68,10 @@ func isFile(filePath string) bool {
}
return !f.IsDir()
}
+
+func concatenateError(err error, stderr string) error {
+ if len(stderr) == 0 {
+ return err
+ }
+ return fmt.Errorf("%v: %s", err, stderr)
+}
diff --git a/modules/middleware/org.go b/modules/middleware/org.go
index 0e544fe4a2..1ac4bcf1ae 100644
--- a/modules/middleware/org.go
+++ b/modules/middleware/org.go
@@ -47,6 +47,12 @@ func OrgAssignment(redirect bool, args ...bool) macaron.Handler {
org := ctx.Org.Organization
ctx.Data["Org"] = org
+ // Force redirection when username is actually a user.
+ if !org.IsOrganization() {
+ ctx.Redirect("/" + org.Name)
+ return
+ }
+
if ctx.IsSigned {
ctx.Org.IsOwner = org.IsOwnedBy(ctx.User.Id)
if ctx.Org.IsOwner {