[bugfix] Don't throw error when parent statuses are missing (#2011) (#3088)

* [bugfix] Don't throw error when parent statuses are missing (#2011)

* Split missing parent status case from error check
This commit is contained in:
snow 2024-07-10 06:38:25 -07:00 committed by GitHub
parent ad93e57d08
commit 8f8093aea4
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -202,7 +202,7 @@ func (s *statusDB) PopulateStatus(ctx context.Context, status *gtsmodel.Status)
gtscontext.SetBarebones(ctx), gtscontext.SetBarebones(ctx),
status.InReplyToID, status.InReplyToID,
) )
if err != nil { if err != nil && !errors.Is(err, db.ErrNoEntries) {
errs.Appendf("error populating status parent: %w", err) errs.Appendf("error populating status parent: %w", err)
} }
} }
@ -561,10 +561,15 @@ func (s *statusDB) GetStatusParents(ctx context.Context, status *gtsmodel.Status
for id := status.InReplyToID; id != ""; { for id := status.InReplyToID; id != ""; {
parent, err := s.GetStatusByID(ctx, id) parent, err := s.GetStatusByID(ctx, id)
if err != nil { if err != nil && !errors.Is(err, db.ErrNoEntries) {
return nil, err return nil, err
} }
if parent == nil {
// Parent status not found (e.g. deleted)
break
}
// Append parent status to slice // Append parent status to slice
parents = append(parents, parent) parents = append(parents, parent)