diff --git a/appview/db/pulls.go b/appview/db/pulls.go --- a/appview/db/pulls.go +++ b/appview/db/pulls.go @@ -92,7 +92,7 @@ _, 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 } @@ -101,7 +101,7 @@ if err != nil { return "", err } - return pull.PullAt(), err + return pull.AtUri(), err } func NextPullId(e Execer, repoAt syntax.ATURI) (int, error) { @@ -214,12 +214,12 @@ 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 { diff --git a/appview/models/pull.go b/appview/models/pull.go --- a/appview/models/pull.go +++ b/appview/models/pull.go @@ -167,7 +167,7 @@ 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)) } diff --git a/appview/pulls/pulls.go b/appview/pulls/pulls.go --- a/appview/pulls/pulls.go +++ b/appview/pulls/pulls.go @@ -192,7 +192,7 @@ 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.") @@ -200,7 +200,7 @@ 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( @@ -752,7 +752,7 @@ Rkey: tid.TID(), Record: &lexutil.LexiconTypeDecoder{ Val: &tangled.RepoPullComment{ - Pull: pull.PullAt().String(), + Pull: pull.AtUri().String(), Body: body, CreatedAt: createdAt, }, @@ -1839,7 +1839,7 @@ } defer tx.Rollback() - pullAt := pull.PullAt() + pullAt := pull.AtUri() newRoundNumber := len(pull.Submissions) newPatch := patch newSourceRev := sourceRev @@ -2036,7 +2036,7 @@ } // resubmit the new pull - pullAt := op.PullAt() + pullAt := op.AtUri() newRoundNumber := len(op.Submissions) newPatch := np.LatestPatch() combinedPatch := np.LatestSubmission().Combined diff --git a/appview/notify/db/db.go b/appview/notify/db/db.go --- a/appview/notify/db/db.go +++ b/appview/notify/db/db.go @@ -203,7 +203,7 @@ 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) @@ -249,7 +249,7 @@ 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) @@ -353,7 +353,7 @@ } entityType := "pull" - entityId := pull.PullAt().String() + entityId := pull.AtUri().String() repoId := &repo.Id var issueId *int64 var eventType models.NotificationType diff --git a/appview/pages/templates/repo/pulls/pull.html b/appview/pages/templates/repo/pulls/pull.html --- a/appview/pages/templates/repo/pulls/pull.html +++ b/appview/pages/templates/repo/pulls/pull.html @@ -18,7 +18,7 @@ {{ template "repo/fragments/labelPanel" (dict "RepoInfo" $.RepoInfo "Defs" $.LabelDefs - "Subject" $.Pull.PullAt + "Subject" $.Pull.AtUri "State" $.Pull.Labels) }} {{ template "repo/fragments/participants" $.Pull.Participants }} diff --git a/appview/pages/templates/repo/pulls/fragments/pullHeader.html b/appview/pages/templates/repo/pulls/fragments/pullHeader.html --- a/appview/pages/templates/repo/pulls/fragments/pullHeader.html +++ b/appview/pages/templates/repo/pulls/fragments/pullHeader.html @@ -75,7 +75,7 @@ "Kind" $kind "Count" $reactionData.Count "IsReacted" (index $.UserReacted $kind) - "ThreadAt" $.Pull.PullAt + "ThreadAt" $.Pull.AtUri "Users" $reactionData.Users) }} {{ end }}