fix: prevent localized values from breaking app list #963

Open
ChasquiLabo wants to merge 1 commits from escuela-comun/abra:fix/817-list-i18n into main
+25 -12
View File
@@ -117,10 +117,10 @@ Use "--status/-S" flag to query all servers for the live deployment status.`),
log.Fatal(err)
}
status := i18n.G("unknown")
version := i18n.G("unknown")
chaos := i18n.G("unknown")
chaosVersion := i18n.G("unknown")
status := "unknown"
version := "unknown"
chaos := "unknown"
chaosVersion := "unknown"
if statusMeta, ok := statuses[app.StackName()]; ok {
if currentVersion, exists := statusMeta["version"]; exists {
if currentVersion != "" {
@@ -181,9 +181,9 @@ Use "--status/-S" flag to query all servers for the live deployment status.`),
if len(newUpdates) == 0 {
if version == "unknown" {
appStats.Upgrade = i18n.G("unknown")
appStats.Upgrade = "unknown"
} else {
appStats.Upgrade = i18n.G("latest")
appStats.Upgrade = "latest"
stats.LatestCount++
}
} else {
@@ -256,13 +256,12 @@ Use "--status/-S" flag to query all servers for the live deployment status.`),
}
row = append(row, []string{
appStat.Status,
chaosStatus,
appStat.Version,
appStat.Upgrade}...,
)
displayUnknown(appStat.Status),
displayUnknown(chaosStatus),
displayUnknown(appStat.Version),
displayUpgrade(appStat.Upgrade),
}...)
}
rows = append(rows, row)
}
@@ -283,6 +282,20 @@ Use "--status/-S" flag to query all servers for the live deployment status.`),
},
}
func displayUnknown(value string) string {
if value == "unknown" {
return i18n.G("unknown")
}
return value
}
func displayUpgrade(value string) string {
if value == "latest" {
return i18n.G("latest")
}
return displayUnknown(value)
}
var (
status bool
recipeFilter string