diff options
| author | Zettat123 <zettat123@gmail.com> | 2023-02-16 01:29:13 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-02-15 12:29:13 -0500 |
| commit | 0ab22a1a020865c9906487cf7735796ff01c4fd2 (patch) | |
| tree | c591b41ca81d6b8d5211eec36844f1e1977f9dce /models/issues/issue.go | |
| parent | 2c04595762b5c6ecfef8a32b05de7a5ef5f7066c (diff) | |
| download | gitea-0ab22a1a020865c9906487cf7735796ff01c4fd2.tar.gz gitea-0ab22a1a020865c9906487cf7735796ff01c4fd2.zip | |
fix incorrect role labels for migrated issues and comments (#22914)
Fix #22797.
## Reason
If a comment was migrated from other platforms, this comment may have an
original author and its poster is always not the original author. When
the `roleDescriptor` func get the poster's role descriptor for a
comment, it does not check if the comment has an original author. So the
migrated comments' original authors might be marked as incorrect roles.
---------
Co-authored-by: zeripath <art27@cantab.net>
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Diffstat (limited to 'models/issues/issue.go')
| -rw-r--r-- | models/issues/issue.go | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/models/issues/issue.go b/models/issues/issue.go index 62b936331b..e0dcf3d269 100644 --- a/models/issues/issue.go +++ b/models/issues/issue.go @@ -2403,3 +2403,8 @@ func DeleteOrphanedIssues(ctx context.Context) error { } return nil } + +// HasOriginalAuthor returns if an issue was migrated and has an original author. +func (issue *Issue) HasOriginalAuthor() bool { + return issue.OriginalAuthor != "" && issue.OriginalAuthorID != 0 +} |