diff --git a/gogs.go b/gogs.go
index a5825b38f..97a278992 100644
--- a/gogs.go
+++ b/gogs.go
@@ -17,7 +17,7 @@ import (
"github.com/gogits/gogs/modules/setting"
)
-const APP_VER = "0.9.94.0829"
+const APP_VER = "0.9.94.0830"
func init() {
runtime.GOMAXPROCS(runtime.NumCPU())
diff --git a/modules/base/tool.go b/modules/base/tool.go
index cf8fece02..e321d2b86 100644
--- a/modules/base/tool.go
+++ b/modules/base/tool.go
@@ -518,26 +518,14 @@ func IsLetter(ch rune) bool {
return 'a' <= ch && ch <= 'z' || 'A' <= ch && ch <= 'Z' || ch == '_' || ch >= 0x80 && unicode.IsLetter(ch)
}
-func IsTextFile(data []byte) (string, bool) {
- contentType := http.DetectContentType(data)
- if strings.Index(contentType, "text/") != -1 {
- return contentType, true
- }
- return contentType, false
+func IsTextFile(data []byte) bool {
+ return strings.Index(http.DetectContentType(data), "text/") != -1
}
-func IsImageFile(data []byte) (string, bool) {
- contentType := http.DetectContentType(data)
- if strings.Index(contentType, "image/") != -1 {
- return contentType, true
- }
- return contentType, false
+func IsImageFile(data []byte) bool {
+ return strings.Index(http.DetectContentType(data), "image/") != -1
}
-func IsPDFFile(data []byte) (string, bool) {
- contentType := http.DetectContentType(data)
- if strings.Index(contentType, "application/pdf") != -1 {
- return contentType, true
- }
- return contentType, false
+func IsPDFFile(data []byte) bool {
+ return strings.Index(http.DetectContentType(data), "application/pdf") != -1
}
diff --git a/modules/context/context.go b/modules/context/context.go
index 971d075d0..ac46bb38f 100644
--- a/modules/context/context.go
+++ b/modules/context/context.go
@@ -105,13 +105,12 @@ func (ctx *Context) Handle(status int, title string, err error) {
ctx.HTML(status, base.TplName(fmt.Sprintf("status/%d", status)))
}
-// HandleError use error check function to determine if server should
-// response as client input error or server internal error.
-// It responses with given status code for client error,
-// or error context description for logging purpose of server error.
-func (ctx *Context) HandleError(title string, errck func(error) bool, err error, status int) {
+// NotFoundOrServerError use error check function to determine if the error
+// is about not found. It responses with 404 status code for not found error,
+// or error context description for logging purpose of 500 server error.
+func (ctx *Context) NotFoundOrServerError(title string, errck func(error) bool, err error) {
if errck(err) {
- ctx.Error(status, err.Error())
+ ctx.Handle(404, title, err)
return
}
diff --git a/modules/context/repo.go b/modules/context/repo.go
index f5f26b1e5..d72e64e42 100644
--- a/modules/context/repo.go
+++ b/modules/context/repo.go
@@ -48,7 +48,7 @@ type Repository struct {
CommitsCount int64
Mirror *models.Mirror
- PullRequest *PullRequest
+ PullRequest *PullRequest
}
// IsOwner returns true if current user is the owner of repository.
@@ -71,6 +71,11 @@ func (r *Repository) HasAccess() bool {
return r.AccessMode >= models.ACCESS_MODE_READ
}
+// CanEnableEditor returns true if repository is editable and user has proper access level.
+func (r *Repository) CanEnableEditor() bool {
+ return r.Repository.CanEnableEditor() && r.IsViewBranch && r.IsWriter()
+}
+
// GetEditorconfig returns the .editorconfig definition if found in the
// HEAD of the default repo branch.
func (r *Repository) GetEditorconfig() (*editorconfig.Editorconfig, error) {
@@ -167,6 +172,7 @@ func RepoAssignment(args ...bool) macaron.Handler {
}
}
ctx.Repo.Owner = owner
+ ctx.Data["Username"] = ctx.Repo.Owner.Name
// Get repository.
repo, err := models.GetRepositoryByName(owner.ID, repoName)
@@ -221,6 +227,7 @@ func RepoAssignment(args ...bool) macaron.Handler {
}
ctx.Repo.Repository = repo
+ ctx.Data["RepoName"] = ctx.Repo.Repository.Name
ctx.Data["IsBareRepo"] = ctx.Repo.Repository.IsBare
gitRepo, err := git.OpenRepository(models.RepoPath(userName, repoName))
@@ -348,12 +355,11 @@ func RepoRef() macaron.Handler {
// For API calls.
if ctx.Repo.GitRepo == nil {
repoPath := models.RepoPath(ctx.Repo.Owner.Name, ctx.Repo.Repository.Name)
- gitRepo, err := git.OpenRepository(repoPath)
+ ctx.Repo.GitRepo, err = git.OpenRepository(repoPath)
if err != nil {
ctx.Handle(500, "RepoRef Invalid repo "+repoPath, err)
return
}
- ctx.Repo.GitRepo = gitRepo
}
// Get default branch.
@@ -431,6 +437,7 @@ func RepoRef() macaron.Handler {
ctx.Repo.BranchName = refName
ctx.Data["BranchName"] = ctx.Repo.BranchName
ctx.Data["CommitID"] = ctx.Repo.CommitID
+ ctx.Data["TreePath"] = ctx.Repo.TreePath
ctx.Data["IsViewBranch"] = ctx.Repo.IsViewBranch
ctx.Data["IsViewTag"] = ctx.Repo.IsViewTag
ctx.Data["IsViewCommit"] = ctx.Repo.IsViewCommit
diff --git a/routers/repo/download.go b/routers/repo/download.go
index 85aa76472..faa30ac9d 100644
--- a/routers/repo/download.go
+++ b/routers/repo/download.go
@@ -21,10 +21,8 @@ func ServeData(ctx *context.Context, name string, reader io.Reader) error {
buf = buf[:n]
}
- _, isTextFile := base.IsTextFile(buf)
- if !isTextFile {
- _, isImageFile := base.IsImageFile(buf)
- if !isImageFile {
+ if !base.IsTextFile(buf) {
+ if !base.IsImageFile(buf) {
ctx.Resp.Header().Set("Content-Disposition", "attachment; filename=\""+path.Base(ctx.Repo.TreePath)+"\"")
ctx.Resp.Header().Set("Content-Transfer-Encoding", "binary")
}
diff --git a/routers/repo/editor.go b/routers/repo/editor.go
index 81f95956a..465de0021 100644
--- a/routers/repo/editor.go
+++ b/routers/repo/editor.go
@@ -72,8 +72,7 @@ func editFile(ctx *context.Context, isNewFile bool) {
}
// Only text file are editable online.
- _, isTextFile := base.IsTextFile(buf)
- if !isTextFile {
+ if !base.IsTextFile(buf) {
ctx.Handle(404, "", nil)
return
}
diff --git a/routers/repo/issue.go b/routers/repo/issue.go
index 24fecaf41..a5908c182 100644
--- a/routers/repo/issue.go
+++ b/routers/repo/issue.go
@@ -795,7 +795,7 @@ func UpdateIssueAssignee(ctx *context.Context) {
func NewComment(ctx *context.Context, form auth.CreateCommentForm) {
issue, err := models.GetIssueByIndex(ctx.Repo.Repository.ID, ctx.ParamsInt64(":index"))
if err != nil {
- ctx.HandleError("GetIssueByIndex", models.IsErrIssueNotExist, err, 404)
+ ctx.NotFoundOrServerError("GetIssueByIndex", models.IsErrIssueNotExist, err)
return
}
@@ -881,7 +881,7 @@ func NewComment(ctx *context.Context, form auth.CreateCommentForm) {
func UpdateCommentContent(ctx *context.Context) {
comment, err := models.GetCommentByID(ctx.ParamsInt64(":id"))
if err != nil {
- ctx.HandleError("GetCommentByID", models.IsErrCommentNotExist, err, 404)
+ ctx.NotFoundOrServerError("GetCommentByID", models.IsErrCommentNotExist, err)
return
}
@@ -913,7 +913,7 @@ func UpdateCommentContent(ctx *context.Context) {
func DeleteComment(ctx *context.Context) {
comment, err := models.GetCommentByID(ctx.ParamsInt64(":id"))
if err != nil {
- ctx.HandleError("GetCommentByID", models.IsErrCommentNotExist, err, 404)
+ ctx.NotFoundOrServerError("GetCommentByID", models.IsErrCommentNotExist, err)
return
}
diff --git a/routers/repo/view.go b/routers/repo/view.go
index 50b158467..42bf62c60 100644
--- a/routers/repo/view.go
+++ b/routers/repo/view.go
@@ -32,6 +32,192 @@ const (
FORKS base.TplName = "repo/forks"
)
+func renderDirectory(ctx *context.Context, treeLink string) {
+ tree, err := ctx.Repo.Commit.SubTree(ctx.Repo.TreePath)
+ if err != nil {
+ ctx.NotFoundOrServerError("Repo.Commit.SubTree", git.IsErrNotExist, err)
+ return
+ }
+
+ entries, err := tree.ListEntries()
+ if err != nil {
+ ctx.Handle(500, "ListEntries", err)
+ return
+ }
+ entries.Sort()
+
+ ctx.Data["Files"], err = entries.GetCommitsInfo(ctx.Repo.Commit, ctx.Repo.TreePath)
+ if err != nil {
+ ctx.Handle(500, "GetCommitsInfo", err)
+ return
+ }
+
+ var readmeFile *git.Blob
+ for _, entry := range entries {
+ if entry.IsDir() || !markdown.IsReadmeFile(entry.Name()) {
+ continue
+ }
+
+ // TODO: collect all possible README files and show with priority.
+ readmeFile = entry.Blob()
+ break
+ }
+
+ if readmeFile != nil {
+ ctx.Data["ReadmeInList"] = true
+ ctx.Data["ReadmeExist"] = true
+
+ dataRc, err := readmeFile.Data()
+ if err != nil {
+ ctx.Handle(500, "Data", err)
+ return
+ }
+
+ buf := make([]byte, 1024)
+ n, _ := dataRc.Read(buf)
+ if n > 0 {
+ buf = buf[:n]
+ }
+
+ isTextFile := base.IsTextFile(buf)
+ ctx.Data["FileIsText"] = isTextFile
+ ctx.Data["FileName"] = readmeFile.Name()
+ // FIXME: what happens when README file is an image?
+ if isTextFile {
+ d, _ := ioutil.ReadAll(dataRc)
+ buf = append(buf, d...)
+ switch {
+ case markdown.IsMarkdownFile(readmeFile.Name()):
+ ctx.Data["IsMarkdown"] = true
+ buf = markdown.Render(buf, treeLink, ctx.Repo.Repository.ComposeMetas())
+ default:
+ buf = bytes.Replace(buf, []byte("\n"), []byte(`
`), -1)
+ }
+ ctx.Data["FileContent"] = string(buf)
+ }
+ }
+
+ // Show latest commit info of repository in table header,
+ // or of directory if not in root directory.
+ latestCommit := ctx.Repo.Commit
+ if len(ctx.Repo.TreePath) > 0 {
+ latestCommit, err = ctx.Repo.Commit.GetCommitByPath(ctx.Repo.TreePath)
+ if err != nil {
+ ctx.Handle(500, "GetCommitByPath", err)
+ return
+ }
+ }
+ ctx.Data["LatestCommit"] = latestCommit
+ ctx.Data["LatestCommitUser"] = models.ValidateCommitWithEmail(latestCommit)
+
+ // Check permission to add or upload new file.
+ if ctx.Repo.IsWriter() && ctx.Repo.IsViewBranch {
+ ctx.Data["CanAddFile"] = true
+ // uploadFileLink := ctx.Repo.RepoLink + "/upload/" + branchName
+ // if setting.Repository.Upload.Enabled {
+ // ctx.Data["UploadFileLink"] = uploadFileLink + "/" + ctx.Repo.TreePath
+ // }
+ }
+}
+
+func renderFile(ctx *context.Context, entry *git.TreeEntry, treeLink, rawLink string) {
+ ctx.Data["IsViewFile"] = true
+
+ blob := entry.Blob()
+ dataRc, err := blob.Data()
+ if err != nil {
+ ctx.Handle(500, "Data", err)
+ return
+ }
+
+ ctx.Data["FileSize"] = blob.Size()
+ ctx.Data["FileName"] = blob.Name()
+ ctx.Data["HighlightClass"] = highlight.FileNameToHighlightClass(blob.Name())
+ ctx.Data["RawFileLink"] = rawLink + "/" + ctx.Repo.TreePath
+
+ buf := make([]byte, 1024)
+ n, _ := dataRc.Read(buf)
+ if n > 0 {
+ buf = buf[:n]
+ }
+
+ isTextFile := base.IsTextFile(buf)
+ ctx.Data["IsTextFile"] = isTextFile
+
+ // Assume file is not editable first.
+ if !isTextFile {
+ ctx.Data["EditFileTooltip"] = ctx.Tr("repo.editor.cannot_edit_non_text_files")
+ }
+
+ switch {
+ case isTextFile:
+ if blob.Size() >= setting.UI.MaxDisplayFileSize {
+ ctx.Data["IsFileTooLarge"] = true
+ break
+ }
+
+ d, _ := ioutil.ReadAll(dataRc)
+ buf = append(buf, d...)
+
+ isMarkdown := markdown.IsMarkdownFile(blob.Name())
+ ctx.Data["IsMarkdown"] = isMarkdown
+
+ readmeExist := isMarkdown || markdown.IsReadmeFile(blob.Name())
+ ctx.Data["ReadmeExist"] = readmeExist
+ if readmeExist {
+ // TODO: don't need to render if it's a README but not Markdown file.
+ ctx.Data["FileContent"] = string(markdown.Render(buf, path.Dir(treeLink), ctx.Repo.Repository.ComposeMetas()))
+ } else {
+ // Building code view blocks with line number on server side.
+ var fileContent string
+ if err, content := template.ToUTF8WithErr(buf); err != nil {
+ if err != nil {
+ log.Error(4, "ToUTF8WithErr: %s", err)
+ }
+ fileContent = string(buf)
+ } else {
+ fileContent = content
+ }
+
+ var output bytes.Buffer
+ lines := strings.Split(fileContent, "\n")
+ for index, line := range lines {
+ output.WriteString(fmt.Sprintf(`