fix: nil pointer check for get/list projects

uselagoon · Aug 6, 2024 · 9280dc3 · 9280dc3
1 parent 29f002e
commit 9280dc3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/cmd/get.go b/cmd/get.go
@@ -102,7 +102,11 @@ var getProjectCmd = &cobra.Command{
 			returnNonEmptyString(fmt.Sprintf("%v", project.GitURL)),
 			returnNonEmptyString(fmt.Sprintf("%v", project.ProductionEnvironment)),
 			returnNonEmptyString(fmt.Sprintf("%v", productionRoute)),
-			returnNonEmptyString(fmt.Sprintf("%v/%v", devEnvironments, *project.DevelopmentEnvironmentsLimit)),
+		}
+		if project.DevelopmentEnvironmentsLimit != nil {
+			projData = append(projData, returnNonEmptyString(fmt.Sprintf("%v/%v", devEnvironments, *project.DevelopmentEnvironmentsLimit)))
+		} else {
+			projData = append(projData, returnNonEmptyString(fmt.Sprintf("%v/%v", devEnvironments, 0)))
 		}
 		projHeader := []string{"ID", "ProjectName", "GitUrl", "ProductionEnvironment", "ProductionRoute", "DevEnvironments"}
 		if wide {

diff --git a/cmd/list.go b/cmd/list.go
@@ -75,7 +75,11 @@ var listProjectsCmd = &cobra.Command{
 				returnNonEmptyString(fmt.Sprintf("%v", project.GitURL)),
 				returnNonEmptyString(fmt.Sprintf("%v", project.ProductionEnvironment)),
 				returnNonEmptyString(fmt.Sprintf("%v", productionRoute)),
-				returnNonEmptyString(fmt.Sprintf("%v/%v", devEnvironments, *project.DevelopmentEnvironmentsLimit)),
+			}
+			if project.DevelopmentEnvironmentsLimit != nil {
+				projData = append(projData, returnNonEmptyString(fmt.Sprintf("%v/%v", devEnvironments, *project.DevelopmentEnvironmentsLimit)))
+			} else {
+				projData = append(projData, returnNonEmptyString(fmt.Sprintf("%v/%v", devEnvironments, 0)))
 			}
 			// if wide {
 			// 	projData = append(projData, returnNonEmptyString(fmt.Sprintf("%v", project.AutoIdle)))