don't panic on non-fully populated status edits, it may not always be possible (#4006)

1 files changed, 7 insertions, 5 deletions

diff --git a/internal/gtsmodel/status.go b/internal/gtsmodel/status.go
index cdeccbebb..62458c2db 100644
--- a/internal/gtsmodel/status.go
+++ b/internal/gtsmodel/status.go
@@ -21,6 +21,7 @@ import (
 	"slices"
 	"time"
 
+	"github.com/superseriousbusiness/gotosocial/internal/log"
 	"github.com/superseriousbusiness/gotosocial/internal/util/xslices"
 )
 
@@ -272,14 +273,15 @@ func (s *Status) IsLocalOnly() bool {
 	return s.Federated == nil || !*s.Federated
 }
 
-// AllAttachmentIDs gathers ALL media attachment IDs from both the
-// receiving Status{}, and any historical Status{}.Edits. Note that
-// this function will panic if Status{}.Edits is not populated.
+// AllAttachmentIDs gathers ALL media attachment IDs from both
+// the receiving Status{}, and any historical Status{}.Edits.
 func (s *Status) AllAttachmentIDs() []string {
 	var total int
 
-	if len(s.EditIDs) != len(s.Edits) {
-		panic("status edits not populated")
+	// Check if this is being correctly
+	// called on fully populated status.
+	if !s.EditsPopulated() {
+		log.Warnf(nil, "status edits not populated for %s", s.URI)
 	}
 
 	// Get count of attachment IDs.