Sort by priority

list.go

@@ -4,6 +4,7 @@ import (
 	"encoding/json"
 	"fmt"
 	"os"
+	"sort"
 
 	"github.com/acarl005/stripansi"
 	todoist "github.com/sachaos/todoist/lib"
@@ -22,20 +23,6 @@ func traverseItems(item *todoist.Item, f func(item *todoist.Item, depth int), de
 	}
 }
 
-func sortItems(itemListPtr *[][]string, byIndex int) {
-	itemList := *itemListPtr
-	length := len(itemList)
-	for i := 0; i < length-1; i++ {
-		for j := 0; j < length-1-i; j++ {
-			if stripansi.Strip(itemList[j][byIndex]) > stripansi.Strip(itemList[j+1][byIndex]) {
-				tmp := itemList[j]
-				itemList[j] = itemList[j+1]
-				itemList[j+1] = tmp
-			}
-		}
-	}
-}
-
 type TaskJSON struct {
 	ID       string `json:"ID"`
 	Priority string `json:"Priority"`
@@ -87,12 +74,13 @@ func List(c *cli.Context) error {
 		jsonObjects = append(jsonObjects, obj)
 	}, 0)
 
-	// FIXME: sort JSON objects
-	// if c.Bool("priority") == true {
-	// 	// sort output by priority
-	// 	// and no need to use "else block" as items returned by API are already sorted by task id
-	// 	sortItems(&itemList, 1)
-	// }
+	if c.Bool("priority") {
+		// sort output by priority
+		// and no need to use "else block" as items returned by API are already sorted by task id
+		sort.Slice(jsonObjects, func(i, j int) bool {
+			return stripansi.Strip(jsonObjects[i].Priority) < stripansi.Strip(jsonObjects[j].Priority)
+		})
+	}
 
 	defer writer.Flush()