[ENHANCEMENT] Make it possible to override show.autoclip (#3082)

* [ENHANCEMENT] Make it possible to override `show.autoclip`

With this change is now possible to use `--alsoclip=false` on the command line to override the `show.autoclip` set in the config file.

Signed-off-by: Fabio Alessandro Locati <[email protected]>

* Improve code legibility

Signed-off-by: Fabio Alessandro Locati <[email protected]>

* Improve logic

Signed-off-by: Fabio Alessandro Locati <[email protected]>

---------

Signed-off-by: Fabio Alessandro Locati <[email protected]>

Author: Fale

internal/action/show.go

@@ -49,6 +49,7 @@ func showParseArgs(c *cli.Context) context.Context {
 		ctx = WithRevision(ctx, c.String("revision"))
 	}
 
+	ctx = WithAlsoClip(ctx, config.Bool(ctx, "show.autoclip"))
 	if c.IsSet("alsoclip") {
 		ctx = WithAlsoClip(ctx, c.Bool("alsoclip"))
 	}
@@ -214,7 +215,7 @@ func (s *Action) showHandleOutput(ctx context.Context, name string, sec gopass.S
 		}
 	}
 
-	if (IsClip(ctx) || config.Bool(ctx, "show.autoclip")) && pw != "" {
+	if (IsClip(ctx) || IsAlsoClip(ctx)) && pw != "" {
 		if err := clipboard.CopyTo(ctx, name, []byte(pw), config.AsInt(s.cfg.Get("core.cliptimeout"))); err != nil {
 			return err
 		}
@@ -274,7 +275,7 @@ func (s *Action) showGetContent(ctx context.Context, sec gopass.Secret) (string,
 	// everything but the first line.
 	if config.Bool(ctx, "show.safecontent") && !ctxutil.IsForce(ctx) && ctxutil.IsShowParsing(ctx) {
 		body := showSafeContent(sec)
-		if IsAlsoClip(ctx) || config.Bool(ctx, "show.autoclip") {
+		if IsAlsoClip(ctx) {
 			return pw, body, nil
 		}