appview/models: rename Pull.PullAt() to Pull.AtUri() #752

merged
opened by oppi.li targeting master from push-toykwqpnvqrs

in sync with Issue.AtUri.

Signed-off-by: oppiliappan me@oppi.li

Changed files
+15 -15
appview
db
models
notify
db
pages
templates
repo
pulls
pulls
+4 -4
appview/db/pulls.go
···
_, err = tx.Exec(`
insert into pull_submissions (pull_at, round_number, patch, combined, source_rev)
values (?, ?, ?, ?, ?)
-
`, pull.PullAt(), 0, pull.Submissions[0].Patch, pull.Submissions[0].Combined, pull.Submissions[0].SourceRev)
+
`, pull.AtUri(), 0, pull.Submissions[0].Patch, pull.Submissions[0].Combined, pull.Submissions[0].SourceRev)
return err
}
···
if err != nil {
return "", err
}
-
return pull.PullAt(), err
+
return pull.AtUri(), err
}
func NextPullId(e Execer, repoAt syntax.ATURI) (int, error) {
···
pull.ParentChangeId = parentChangeId.String
}
-
pulls[pull.PullAt()] = &pull
+
pulls[pull.AtUri()] = &pull
}
var pullAts []syntax.ATURI
for _, p := range pulls {
-
pullAts = append(pullAts, p.PullAt())
+
pullAts = append(pullAts, p.AtUri())
}
submissionsMap, err := GetPullSubmissions(e, FilterIn("pull_at", pullAts))
if err != nil {
+1 -1
appview/models/pull.go
···
return p.LatestSubmission().SourceRev
}
-
func (p *Pull) PullAt() syntax.ATURI {
+
func (p *Pull) AtUri() syntax.ATURI {
return syntax.ATURI(fmt.Sprintf("at://%s/%s/%s", p.OwnerDid, tangled.RepoPullNSID, p.Rkey))
}
+3 -3
appview/notify/db/db.go
···
actorDid := syntax.DID(pull.OwnerDid)
eventType := models.NotificationTypePullCreated
entityType := "pull"
-
entityId := pull.PullAt().String()
+
entityId := pull.AtUri().String()
repoId := &repo.Id
var issueId *int64
p := int64(pull.ID)
···
actorDid := syntax.DID(comment.OwnerDid)
eventType := models.NotificationTypePullCommented
entityType := "pull"
-
entityId := pull.PullAt().String()
+
entityId := pull.AtUri().String()
repoId := &repo.Id
var issueId *int64
p := int64(pull.ID)
···
}
entityType := "pull"
-
entityId := pull.PullAt().String()
+
entityId := pull.AtUri().String()
repoId := &repo.Id
var issueId *int64
var eventType models.NotificationType
+1 -1
appview/pages/templates/repo/pulls/fragments/pullHeader.html
···
"Kind" $kind
"Count" $reactionData.Count
"IsReacted" (index $.UserReacted $kind)
-
"ThreadAt" $.Pull.PullAt
+
"ThreadAt" $.Pull.AtUri
"Users" $reactionData.Users)
}}
{{ end }}
+1 -1
appview/pages/templates/repo/pulls/pull.html
···
{{ template "repo/fragments/labelPanel"
(dict "RepoInfo" $.RepoInfo
"Defs" $.LabelDefs
-
"Subject" $.Pull.PullAt
+
"Subject" $.Pull.AtUri
"State" $.Pull.Labels) }}
{{ template "repo/fragments/participants" $.Pull.Participants }}
</div>
+5 -5
appview/pulls/pulls.go
···
m[p.Sha] = p
}
-
reactionMap, err := db.GetReactionMap(s.db, 20, pull.PullAt())
+
reactionMap, err := db.GetReactionMap(s.db, 20, pull.AtUri())
if err != nil {
log.Println("failed to get pull reactions")
s.pages.Notice(w, "pulls", "Failed to load pull. Try again later.")
···
userReactions := map[models.ReactionKind]bool{}
if user != nil {
-
userReactions = db.GetReactionStatusMap(s.db, user.Did, pull.PullAt())
+
userReactions = db.GetReactionStatusMap(s.db, user.Did, pull.AtUri())
}
labelDefs, err := db.GetLabelDefinitions(
···
Rkey: tid.TID(),
Record: &lexutil.LexiconTypeDecoder{
Val: &tangled.RepoPullComment{
-
Pull: pull.PullAt().String(),
+
Pull: pull.AtUri().String(),
Body: body,
CreatedAt: createdAt,
},
···
defer tx.Rollback()
-
pullAt := pull.PullAt()
+
pullAt := pull.AtUri()
newRoundNumber := len(pull.Submissions)
newPatch := patch
newSourceRev := sourceRev
···
// resubmit the new pull
-
pullAt := op.PullAt()
+
pullAt := op.AtUri()
newRoundNumber := len(op.Submissions)
newPatch := np.LatestPatch()
combinedPatch := np.LatestSubmission().Combined