diff --git a/appview/pulls/pulls.go b/appview/pulls/pulls.go --- a/appview/pulls/pulls.go +++ b/appview/pulls/pulls.go @@ -1093,7 +1093,7 @@ defer tx.Rollback() // We've already checked earlier if it's diff-based and title is empty, // so if it's still empty now, it's intentionally skipped owing to format-patch. - if title == "" { + if title == "" || body == "" { formatPatches, err := patchutil.ExtractPatches(patch) if err != nil { s.pages.Notice(w, "pull", fmt.Sprintf("Failed to extract patches: %v", err)) @@ -1104,8 +1104,12 @@ s.pages.Notice(w, "pull", "No patches found in the supplied format-patch.") return } - title = formatPatches[0].Title - body = formatPatches[0].Body + if title == "" { + title = formatPatches[0].Title + } + if body == "" { + body = formatPatches[0].Body + } } rkey := tid.TID()