Fix order listing bug

In typical Shopify fashion they now require an explicit status to be
provided. They claim it defaults to open but without an explicit
status nothing is returned, even when an ID is given!

We also add the -s option to specify a status filter

Author: sshaw

cmd/orders/orders.go

@@ -16,6 +16,7 @@ var Cmd cli.Command
 type listOrdersOptions struct {
 	Ids []int64 `url:"ids,comma,omitempty"`
 	CreatedAtMin time.Time `url:"created_at_min,omitempty"`
+	Status string `url:"status,omitempty"`
 }
 
 func userAgentAction(c *cli.Context) error {
@@ -48,8 +49,7 @@ func userAgentAction(c *cli.Context) error {
 }
 
 func listAction(c *cli.Context) error {
-	var options listOrdersOptions
-
+	options := listOrdersOptions{Status: "open"}
 
 	if c.NArg() > 0 {
 		for i := 0; i < c.NArg(); i++ {
@@ -63,6 +63,10 @@ func listAction(c *cli.Context) error {
 
 	}
 
+	if len(c.String("status")) > 0 {
+		options.Status = c.String("status")
+	}
+
 	orders, err := cmd.NewShopifyClient(c).Order.List(options)
 	if err != nil {
 		return fmt.Errorf("Cannot list orders: %s", err)
@@ -139,10 +143,19 @@ func printLineItems(lines []shopify.LineItem) {
 }
 
 func init() {
+	ordersFlags := []cli.Flag{
+		&cli.StringFlag{
+			Name:    "status",
+			Aliases: []string{"s"},
+			Usage:   "Orders status to filter, defaults to 'open'",
+		},
+	}
+
 	Cmd = cli.Command{
 		Name:  "orders",
 		Aliases: []string{"o"},
 		Usage:   "Information about orders",
+
 		Subcommands: []*cli.Command{
 			{
 				Name: "useragent",
@@ -154,7 +167,7 @@ func init() {
 			{
 				Name: "ls",
 				Usage:   "List the shop's orders or the orders given by the specified IDs",
-				Flags: cmd.Flags,
+				Flags: append(cmd.Flags, ordersFlags...),
 				Action: listAction,
 			},
 		},