Skip to content

Commit e680759

Browse files
committed
simplify logic
fix logic fix a small bug about original author
1 parent 9b145dc commit e680759

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

routers/repo/issue.go

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -980,6 +980,10 @@ func commentTag(repo *models.Repository, poster *models.User, issue *models.Issu
980980
return models.CommentTagNone, err
981981
}
982982
if perm.IsOwner() {
983+
if !poster.IsAdmin {
984+
return models.CommentTagOwner, nil
985+
}
986+
983987
ok, err := models.IsUserRealRepoAdmin(repo, poster)
984988
if err != nil {
985989
return models.CommentTagNone, err
@@ -993,7 +997,11 @@ func commentTag(repo *models.Repository, poster *models.User, issue *models.Issu
993997
return models.CommentTagSysAdmin, nil
994998
}
995999

996-
return models.CommentTagNone, nil
1000+
if ok, err = repo.IsCollaborator(poster.ID); ok && err == nil {
1001+
return models.CommentTagWriter, nil
1002+
}
1003+
1004+
return models.CommentTagNone, err
9971005
}
9981006

9991007
if perm.CanWrite(models.UnitTypeCode) {

templates/repo/issue/view_content/comments.tmpl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@
4343
</div>
4444
<div class="comment-header-right actions df ac">
4545
{{if not $.Repository.IsArchived}}
46-
{{if eq .PosterID .Issue.PosterID }}
46+
{{if or (and (eq .PosterID .Issue.PosterID) (eq .Issue.OriginalAuthorID 0)) (eq .Issue.OriginalAuthorID .OriginalAuthorID) }}
4747
<div class="ui basic label">
4848
{{$.i18n.Tr "repo.issues.poster"}}
4949
</div>

0 commit comments

Comments
 (0)