Print JSON as a well-formed object

This prints all the bugs in a JSON array instead of one by one.
MichaelMure · Jun 18, 2020 · 87eeba4 · 87eeba4
1 parent de5565b
commit 87eeba4
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/commands/ls.go b/commands/ls.go
@@ -97,7 +97,8 @@ type JSONIdentity struct {
 }
 
 func lsJsonFormatter(backend *cache.RepoCache, bugExcerpts []*cache.BugExcerpt) error {
-	for _, b := range bugExcerpts {
+	jsonBugs := make([]JSONBug, len(bugExcerpts))
+	for i, b := range bugExcerpts {
 		jsonBug := JSONBug{
 			b.Id.String(),
 			b.Id.Human(),
@@ -155,9 +156,10 @@ func lsJsonFormatter(backend *cache.RepoCache, bugExcerpts []*cache.BugExcerpt)
 			})
 		}
 
-		jsonObject, _ := json.MarshalIndent(jsonBug, "", "    ")
-		fmt.Printf("%s\n", jsonObject)
+		jsonBugs[i] = jsonBug
 	}
+	jsonObject, _ := json.MarshalIndent(jsonBugs, "", "    ")
+	fmt.Printf("%s\n", jsonObject)
 	return nil
 }
 
@@ -203,7 +205,7 @@ func lsDefaultFormatter(backend *cache.RepoCache, bugExcerpts []*cache.BugExcerp
 	return nil
 }
 
-func lsPlainFormatter(backend *cache.RepoCache, bugExcerpts []*cache.BugExcerpt) error {
+func lsPlainFormatter(_ *cache.RepoCache, bugExcerpts []*cache.BugExcerpt) error {
 	for _, b := range bugExcerpts {
 		fmt.Printf("[%s] %s\n", b.Status, b.Title)
 	}