Reading pull attachments should depend on read UnitTypePullRequests (#10346)

* Make pull attachments depend on read UnitTypePullRequests

Fixes #10336

* Fix test
tokarchuk/v1.17
zeripath 5 years ago committed by GitHub
parent 9f1f2e6675
commit 41294b53b3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 6
      models/attachment.go
  2. 2
      models/attachment_test.go

@ -79,7 +79,11 @@ func (a *Attachment) LinkedRepository() (*Repository, UnitType, error) {
return nil, UnitTypeIssues, err
}
repo, err := GetRepositoryByID(iss.RepoID)
return repo, UnitTypeIssues, err
unitType := UnitTypeIssues
if iss.IsPull {
unitType = UnitTypePullRequests
}
return repo, unitType, err
} else if a.ReleaseID != 0 {
rel, err := GetReleaseByID(a.ReleaseID)
if err != nil {

@ -138,7 +138,7 @@ func TestLinkedRepository(t *testing.T) {
expectedUnitType UnitType
}{
{"LinkedIssue", 1, &Repository{ID: 1}, UnitTypeIssues},
{"LinkedComment", 3, &Repository{ID: 1}, UnitTypeIssues},
{"LinkedComment", 3, &Repository{ID: 1}, UnitTypePullRequests},
{"LinkedRelease", 9, &Repository{ID: 1}, UnitTypeReleases},
{"Notlinked", 10, nil, -1},
}

Loading…
Cancel
Save