fix(router): show full API response in marshaled outputs after filter (

#256)
UpCloudLtd · Oct 16, 2023 · 1b09bb4 · 1b09bb4
1 parent 3eb94a9
commit 1b09bb4
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 8 deletions.
diff --git a/internal/commands/router/list.go b/internal/commands/router/list.go
@@ -50,7 +50,7 @@ func (s *listCommand) ExecuteWithoutArguments(exec commands.Executor) (output.Ou
 	if s.serviceRouters.Value() {
 		s.normalRouters = config.False
 	}
-	var filtered []upcloud.Router
+	filtered := make([]upcloud.Router, 0)
 	if s.allRouters.Value() {
 		filtered = routers.Routers
 	} else {
@@ -67,12 +67,16 @@ func (s *listCommand) ExecuteWithoutArguments(exec commands.Executor) (output.Ou
 	for i, router := range filtered {
 		rows[i] = output.TableRow{router.UUID, router.Name, router.Type}
 	}
-	return output.Table{
-		Columns: []output.TableColumn{
-			{Header: "UUID", Key: "uuid", Colour: ui.DefaultUUUIDColours},
-			{Header: "Name", Key: "name"},
-			{Header: "Type", Key: "type"},
+
+	return output.MarshaledWithHumanOutput{
+		Value: filtered,
+		Output: output.Table{
+			Columns: []output.TableColumn{
+				{Header: "UUID", Key: "uuid", Colour: ui.DefaultUUUIDColours},
+				{Header: "Name", Key: "name"},
+				{Header: "Type", Key: "type"},
+			},
+			Rows: rows,
 		},
-		Rows: rows,
 	}, nil
 }
diff --git a/internal/commands/router/show.go b/internal/commands/router/show.go
@@ -86,7 +86,10 @@ func (s *showCommand) Execute(exec commands.Executor, arg string) (output.Output
 		},
 	}
 
-	return combined, nil
+	return output.MarshaledWithHumanOutput{
+		Value:  router,
+		Output: combined,
+	}, nil
 }
 
 func getNetworks(exec commands.Executor, attached upcloud.RouterNetworkSlice) ([]upcloud.Network, error) {