Merge pull request #1002 from MichaelMure/changed-flag

commands: different pattern to detect changed flags

2 files changed, 22 insertions, 18 deletions

diff --git a/commands/bug/bug.go b/commands/bug/bug.go
index 4c83ffa5..55a1fa59 100644
--- a/commands/bug/bug.go
+++ b/commands/bug/bug.go
@@ -21,17 +21,18 @@ import (
 )
 
 type bugOptions struct {
-	statusQuery      []string
-	authorQuery      []string
-	metadataQuery    []string
-	participantQuery []string
-	actorQuery       []string
-	labelQuery       []string
-	titleQuery       []string
-	noQuery          []string
-	sortBy           string
-	sortDirection    string
-	outputFormat     string
+	statusQuery         []string
+	authorQuery         []string
+	metadataQuery       []string
+	participantQuery    []string
+	actorQuery          []string
+	labelQuery          []string
+	titleQuery          []string
+	noQuery             []string
+	sortBy              string
+	sortDirection       string
+	outputFormat        string
+	outputFormatChanged bool
 }
 
 func NewBugCommand(env *execenv.Env) *cobra.Command {
@@ -57,6 +58,7 @@ git bug status:open --by creation "foo bar" baz
 `,
 		PreRunE: execenv.LoadBackend(env),
 		RunE: execenv.CloseBackend(env, func(cmd *cobra.Command, args []string) error {
+			options.outputFormatChanged = cmd.Flags().Changed("format")
 			return runBug(env, options, args)
 		}),
 		ValidArgsFunction: completion.Ls(env),
@@ -93,7 +95,7 @@ git bug status:open --by creation "foo bar" baz
 	flags.StringVarP(&options.sortDirection, "direction", "d", "asc",
 		"Select the sorting direction. Valid values are [asc,desc]")
 	cmd.RegisterFlagCompletionFunc("direction", completion.From([]string{"asc", "desc"}))
-	flags.StringVarP(&options.outputFormat, "format", "f", "",
+	flags.StringVarP(&options.outputFormat, "format", "f", "default",
 		"Select the output formatting style. Valid values are [default,plain,id,json,org-mode]")
 	cmd.RegisterFlagCompletionFunc("format",
 		completion.From([]string{"default", "plain", "id", "json", "org-mode"}))
@@ -156,14 +158,15 @@ func runBug(env *execenv.Env, opts bugOptions, args []string) error {
 	}
 
 	switch opts.outputFormat {
-	case "":
+	case "default":
+		if opts.outputFormatChanged {
+			return bugsDefaultFormatter(env, excerpts)
+		}
 		if env.Out.IsTerminal() {
 			return bugsDefaultFormatter(env, excerpts)
 		} else {
 			return bugsPlainFormatter(env, excerpts)
 		}
-	case "default":
-		return bugsDefaultFormatter(env, excerpts)
 	case "id":
 		return bugsIDFormatter(env, excerpts)
 	case "plain":
diff --git a/commands/bug/bug_test.go b/commands/bug/bug_test.go
index e6fadd6f..6b0aa4e0 100644
--- a/commands/bug/bug_test.go
+++ b/commands/bug/bug_test.go
@@ -72,9 +72,10 @@ $`
 			env, _ := testenv.NewTestEnvAndBug(t)
 
 			opts := bugOptions{
-				sortDirection: "asc",
-				sortBy:        "creation",
-				outputFormat:  testcase.format,
+				sortDirection:       "asc",
+				sortBy:              "creation",
+				outputFormat:        testcase.format,
+				outputFormatChanged: true, // disable auto-detect
 			}
 
 			require.NoError(t, runBug(env, opts, []string{}))