diff --git a/gogs.go b/gogs.go index a5825b38..97a27899 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 cf8fece0..e321d2b8 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 971d075d..ac46bb38 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 f5f26b1e..d72e64e4 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 85aa7647..faa30ac9 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 81f95956..465de002 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 24fecaf4..a5908c18 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 50b15846..42bf62c6 100644 --- a/routers/repo/view.go +++ b/routers/repo/view.go @@ -32,222 +32,224 @@ const ( FORKS base.TplName = "repo/forks" ) -func Home(ctx *context.Context) { - title := ctx.Repo.Repository.Owner.Name + "/" + ctx.Repo.Repository.Name - if len(ctx.Repo.Repository.Description) > 0 { - title += ": " + ctx.Repo.Repository.Description +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 } - ctx.Data["Title"] = title - ctx.Data["PageIsViewCode"] = true - ctx.Data["RequireHighlightJS"] = true - - branchName := ctx.Repo.BranchName - userName := ctx.Repo.Owner.Name - repoName := ctx.Repo.Repository.Name - - branchLink := ctx.Repo.RepoLink + "/src/" + branchName - treeLink := branchLink - rawLink := ctx.Repo.RepoLink + "/raw/" + branchName - // uploadFileLink := ctx.Repo.RepoLink + "/upload/" + branchName - treePath := ctx.Repo.TreePath - if len(treePath) > 0 { - treeLink += "/" + treePath + entries, err := tree.ListEntries() + if err != nil { + ctx.Handle(500, "ListEntries", err) + return } + entries.Sort() - entry, err := ctx.Repo.Commit.GetTreeEntryByPath(treePath) + ctx.Data["Files"], err = entries.GetCommitsInfo(ctx.Repo.Commit, ctx.Repo.TreePath) if err != nil { - if git.IsErrNotExist(err) { - ctx.Handle(404, "GetTreeEntryByPath", err) - } else { - ctx.Handle(500, "GetTreeEntryByPath", err) - } + ctx.Handle(500, "GetCommitsInfo", err) return } - if !entry.IsDir() { - blob := entry.Blob() - dataRc, err := blob.Data() + 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(404, "blob.Data", err) + ctx.Handle(500, "Data", err) return } - ctx.Data["FileSize"] = blob.Size() - ctx.Data["IsFile"] = true - ctx.Data["FileName"] = blob.Name() - ctx.Data["HighlightClass"] = highlight.FileNameToHighlightClass(blob.Name()) - ctx.Data["FileLink"] = rawLink + "/" + treePath - buf := make([]byte, 1024) n, _ := dataRc.Read(buf) if n > 0 { buf = buf[:n] } - _, isTextFile := base.IsTextFile(buf) - _, isImageFile := base.IsImageFile(buf) - _, isPDFFile := base.IsPDFFile(buf) - ctx.Data["IsFileText"] = isTextFile - - // Assume file is not editable first. - if !isTextFile { - ctx.Data["EditFileTooltip"] = ctx.Tr("repo.editor.cannot_edit_non_text_files") - } - - switch { - case isPDFFile: - ctx.Data["IsPDFFile"] = true - case isImageFile: - ctx.Data["IsImageFile"] = true - case isTextFile: - if blob.Size() >= setting.UI.MaxDisplayFileSize { - ctx.Data["IsFileTooLarge"] = true - } else { - 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(`