Fix 500 on time tracking in timeline API (#21052)

qwerty287 · web-flow · commit be14e79e9884 · 2022-09-04T15:01:23.000+01:00
Fix #21041
diff --git a/modules/convert/issue_comment.go b/modules/convert/issue_comment.go
@@ -101,6 +101,12 @@ func ToTimelineComment(c *issues_model.Comment, doer *user_model.User) *api.Time
 	}
 
 	if c.Time != nil {
+		err = c.Time.LoadAttributes()
+		if err != nil {
+			log.Error("Time.LoadAttributes: %v", err)
+			return nil
+		}
+
 		comment.TrackedTime = ToTrackedTime(c.Time)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -101,6 +101,12 @@ func ToTimelineComment(c issues_model.Comment, doer user_model.User) *api.Time`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`if c.Time != nil {`
	`104`	`+ err = c.Time.LoadAttributes()`
	`105`	`+ if err != nil {`
	`106`	`+ log.Error("Time.LoadAttributes: %v", err)`
	`107`	`+ return nil`
	`108`	`+ }`
	`109`	`+`
`104`	`110`	`comment.TrackedTime = ToTrackedTime(c.Time)`
`105`	`111`	`}`
`106`	`112`