Update all flags with similar behaviour

flag_bool.go

@@ -84,7 +84,10 @@ func (f *BoolFlag) GetDefaultText() string {
 	if f.DefaultText != "" {
 		return f.DefaultText
 	}
-	return fmt.Sprintf("%v", f.defaultValue)
+	if f.defaultValueSet {
+		return fmt.Sprintf("%v", f.defaultValue)
+	}
+	return fmt.Sprintf("%v", f.Value)
 }
 
 // GetEnvVars returns the env vars for this flag
@@ -105,6 +108,7 @@ func (f *BoolFlag) RunAction(c *Context) error {
 func (f *BoolFlag) Apply(set *flag.FlagSet) error {
 	// set default value so that environment wont be able to overwrite it
 	f.defaultValue = f.Value
+	f.defaultValueSet = true
 
 	if val, source, found := flagFromEnvOrFile(f.EnvVars, f.FilePath); found {
 		if val != "" {

flag_float64.go

@@ -32,7 +32,10 @@ func (f *Float64Flag) GetDefaultText() string {
 	if f.DefaultText != "" {
 		return f.DefaultText
 	}
-	return f.GetValue()
+	if f.defaultValueSet {
+		return fmt.Sprintf("%v", f.defaultValue)
+	}
+	return fmt.Sprintf("%v", f.Value)
 }
 
 // GetEnvVars returns the env vars for this flag
@@ -42,6 +45,9 @@ func (f *Float64Flag) GetEnvVars() []string {
 
 // Apply populates the flag given the flag set and environment
 func (f *Float64Flag) Apply(set *flag.FlagSet) error {
+	f.defaultValue = f.Value
+	f.defaultValueSet = true
+
 	if val, source, found := flagFromEnvOrFile(f.EnvVars, f.FilePath); found {
 		if val != "" {
 			valFloat, err := strconv.ParseFloat(val, 64)

flag_generic.go

@@ -53,9 +53,15 @@ func (f *GenericFlag) GetDefaultText() string {
 	if f.DefaultText != "" {
 		return f.DefaultText
 	}
-	if f.defaultValue != nil {
-		return f.defaultValue.String()
+	val := f.Value
+	if f.defaultValueSet {
+		val = f.defaultValue
 	}
+
+	if val != nil {
+		return val.String()
+	}
+
 	return ""
 }
 
@@ -70,6 +76,7 @@ func (f *GenericFlag) Apply(set *flag.FlagSet) error {
 	// set default value so that environment wont be able to overwrite it
 	if f.Value != nil {
 		f.defaultValue = &stringGeneric{value: f.Value.String()}
+		f.defaultValueSet = true
 	}
 
 	if val, source, found := flagFromEnvOrFile(f.EnvVars, f.FilePath); found {

flag_path.go

@@ -33,16 +33,14 @@ func (f *PathFlag) GetDefaultText() string {
 	if f.DefaultText != "" {
 		return f.DefaultText
 	}
+	val := f.Value
 	if f.defaultValueSet {
-		if f.defaultValue == "" {
-			return f.defaultValue
-		}
-		return fmt.Sprintf("%q", f.defaultValue)
+		val = f.defaultValue
 	}
-	if f.Value == "" {
-		return f.Value
+	if val == "" {
+		return val
 	}
-	return fmt.Sprintf("%q", f.Value)
+	return fmt.Sprintf("%q", val)
 }
 
 // GetEnvVars returns the env vars for this flag

flag_string.go

@@ -31,16 +31,15 @@ func (f *StringFlag) GetDefaultText() string {
 	if f.DefaultText != "" {
 		return f.DefaultText
 	}
+	val := f.Value
 	if f.defaultValueSet {
-		if f.defaultValue == "" {
-			return f.defaultValue
-		}
-		return fmt.Sprintf("%q", f.defaultValue)
+		val = f.defaultValue
 	}
-	if f.Value == "" {
-		return f.Value
+
+	if val == "" {
+		return val
 	}
-	return fmt.Sprintf("%q", f.Value)
+	return fmt.Sprintf("%q", val)
 }
 
 // GetEnvVars returns the env vars for this flag

flag_timestamp.go

@@ -120,14 +120,13 @@ func (f *TimestampFlag) GetDefaultText() string {
 	if f.DefaultText != "" {
 		return f.DefaultText
 	}
+	val := f.Value
 	if f.defaultValueSet {
-		if f.defaultValue != nil && f.defaultValue.timestamp != nil {
-			return f.defaultValue.timestamp.String()
-		}
-	} else {
-		if f.Value != nil && f.Value.timestamp != nil {
-			return f.Value.timestamp.String()
-		}
+		val = f.defaultValue
+	}
+
+	if val != nil && val.timestamp != nil {
+		return val.timestamp.String()
 	}
 
 	return ""