From ecd65187307569def23c996a7c1ef16886617db8 Mon Sep 17 00:00:00 2001 From: A_D Date: Wed, 19 May 2021 18:19:57 +0200 Subject: [PATCH] Fixed showing unset shipyard provider as False The code around the dropdown menu in prefs for shipyard providers was a bit too clever for its own good. This brings it in line with the behaviour of the other two --- prefs.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/prefs.py b/prefs.py index 65a018a1..385780d3 100644 --- a/prefs.py +++ b/prefs.py @@ -535,10 +535,9 @@ class PreferencesDialog(tk.Toplevel): ) with row as cur_row: + shipyard_provider = config.get_str('shipyard_provider') self.shipyard_provider = tk.StringVar( - value=str( - config.get_str('shipyard_provider') in plug.provides('shipyard_url') - and config.get_str('shipyard_provider', default='EDSY')) + value=str(shipyard_provider if shipyard_provider in plug.provides('shipyard_url') else 'EDSY') ) # Setting to decide which ship outfitting website to link to - either E:D Shipyard or Coriolis nb.Label(config_frame, text=_('Shipyard')).grid(padx=self.PADX, pady=2*self.PADY, sticky=tk.W, row=cur_row)