From b7e029adc7c5cf27ae2c5db768b971a959fa2449 Mon Sep 17 00:00:00 2001 From: Stefan Haller Date: Wed, 29 Mar 2023 11:42:37 +0200 Subject: [PATCH] Don't set platform defaults on OSConfig struct immediately Instead, query the platform defaults only if the config is empty. This will be necessary later to distinguish an empty config from a default config, so that we can give deprecation warnings. --- pkg/commands/oscommands/os.go | 6 ++++++ pkg/config/user_config.go | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/pkg/commands/oscommands/os.go b/pkg/commands/oscommands/os.go index 11c964ca4..f66f25b72 100644 --- a/pkg/commands/oscommands/os.go +++ b/pkg/commands/oscommands/os.go @@ -79,6 +79,9 @@ func FileType(path string) string { func (c *OSCommand) OpenFile(filename string) error { commandTemplate := c.UserConfig.OS.OpenCommand + if commandTemplate == "" { + commandTemplate = config.GetPlatformDefaultConfig().OpenCommand + } templateValues := map[string]string{ "filename": c.Quote(filename), } @@ -88,6 +91,9 @@ func (c *OSCommand) OpenFile(filename string) error { func (c *OSCommand) OpenLink(link string) error { commandTemplate := c.UserConfig.OS.OpenLinkCommand + if commandTemplate == "" { + commandTemplate = config.GetPlatformDefaultConfig().OpenLinkCommand + } templateValues := map[string]string{ "link": c.Quote(link), } diff --git a/pkg/config/user_config.go b/pkg/config/user_config.go index a7be98c9c..0c227899e 100644 --- a/pkg/config/user_config.go +++ b/pkg/config/user_config.go @@ -599,7 +599,7 @@ func GetDefaultConfig() *UserConfig { BulkMenu: "b", }, }, - OS: GetPlatformDefaultConfig(), + OS: OSConfig{}, DisableStartupPopups: false, CustomCommands: []CustomCommand(nil), Services: map[string]string(nil),