Replaced deprecated tkinter trace methods with new versions

Author: jstone-psi

pds4_tools/viewer/core.py

@@ -418,11 +418,12 @@ def _add_trace(self, variable, mode, callback, default=None):
         if default is not None:
             variable.set(default)
 
-        trace_id = variable.trace(mode, callback)
+
+        trace_id = variable.trace_add(mode, callback)
 
         # Create a callback to delete the trace, otherwise it will stay bound and prevent python
         # from clearing window memory on close
-        self._add_callback('close', variable.trace_vdelete, 'w', trace_id)
+        self._add_callback('close', variable.trace_remove, 'write', trace_id)
 
         return trace_id
 
@@ -787,11 +788,11 @@ def __init__(self, viewer):
 
         # Stores search string in search box
         self._search_text = StringVar()
-        self._add_trace(self._search_text, 'w', self._do_search)
+        self._add_trace(self._search_text, 'write', self._do_search)
 
         # Stores whether match case box is selected
         self._match_case = BooleanVar()
-        self._add_trace(self._match_case, 'w', self._do_search, default=False)
+        self._add_trace(self._match_case, 'write', self._do_search, default=False)
 
         # Stores a list of 3-valued tuples, each containing the line number, start position
         # and stop position of each result that matches the search string; and stores the index

pds4_tools/viewer/image_view.py

@@ -111,7 +111,7 @@ def __init__(self, viewer):
             var = option['type']
             self._menu_options[option['name']] = var
 
-            self._add_trace(var, 'w', option['trace'], option['default'])
+            self._add_trace(var, 'write', option['trace'], option['default'])
 
     @property
     def settings(self):
@@ -2461,7 +2461,7 @@ def __init__(self, viewer, image_structure_window):
 
         self._selected_axis = IntVar()
         self._selected_axis.set(self._structure_window.settings['selected_axis'])
-        self._add_trace(self._selected_axis, 'w',
+        self._add_trace(self._selected_axis, 'write',
                         lambda *args: self._structure_window.select_slice(axis=self._selected_axis.get()))
 
         self._sliders = []
@@ -2514,7 +2514,7 @@ def _create_axis_row(self, slider_index, axis_name, axis_sequence, axis_slice, m
 
             slider_var = IntVar()
             slider_var.set(axis_slice)
-            self._add_trace(slider_var, 'w', lambda *args: self._slider_moved(slider_index, axis_sequence))
+            self._add_trace(slider_var, 'write', lambda *args: self._slider_moved(slider_index, axis_sequence))
 
             slider_row_box = Frame(self._sliders_box, bg=self.get_bg('gray'))
             slider_row_box.pack(pady=(20, 10), expand=1, fill='x')
@@ -2593,7 +2593,7 @@ def __init__(self, viewer, image_structure_window, image):
             var = option['type']
             self._menu_options[option['name']] = var
 
-            self._add_trace(var, 'w', option['trace'], option['default'])
+            self._add_trace(var, 'write', option['trace'], option['default'])
 
         # Add the menu
         self._add_menus()

pds4_tools/viewer/label_view.py

@@ -38,12 +38,12 @@ def __init__(self, viewer, full_label, structure_label=None, initial_display='fu
         # Stores what display type is currently selected
         display_type = StringVar()
         self._menu_options['display_type'] = display_type
-        self._add_trace(display_type, 'w', self._update_label, default=initial_display)
+        self._add_trace(display_type, 'write', self._update_label, default=initial_display)
 
         # Stores whether label is being pretty printed or shown as in the file
         pretty_print = BooleanVar()
         self._menu_options['pretty_print'] = pretty_print
-        self._add_trace(pretty_print, 'w', self._update_label, default=True)
+        self._add_trace(pretty_print, 'write', self._update_label, default=True)
 
         # Draw the main window content
         self._set_heading('Label')

pds4_tools/viewer/plot_view.py

@@ -96,7 +96,7 @@ def __init__(self, viewer):
             var = option['type']
             self._menu_options[option['name']] = var
 
-            self._add_trace(var, 'w', option['trace'], option['default'])
+            self._add_trace(var, 'write', option['trace'], option['default'])
 
     @property
     def settings(self):
@@ -2469,7 +2469,7 @@ def _add_axes_tab(self):
 
         current_axis_limits = self._structure_window.menu_option('axis_limits')
         axis_limits = self._axes_options['axis_limits'] = StringVar()
-        self._add_trace(axis_limits, 'w', self._update_limits_setting, default=current_axis_limits.capitalize())
+        self._add_trace(axis_limits, 'write', self._update_limits_setting, default=current_axis_limits.capitalize())
 
         menu = OptionMenu(option_menus_box, axis_limits, *('Intelligent', 'Tight', 'Auto', 'Manual'))
         menu.config(width=15, **option_menu_params)
@@ -2480,7 +2480,7 @@ def _add_axes_tab(self):
         manual_limits_box.pack(side='top', anchor='nw', pady=10)
 
         manual_limits = BooleanVar()
-        self._add_trace(manual_limits, 'w', self._update_limits_setting, default=(current_axis_limits == 'manual'))
+        self._add_trace(manual_limits, 'write', self._update_limits_setting, default=(current_axis_limits == 'manual'))
 
         b = Checkbutton(manual_limits_box, text='Manual Limits', variable=axis_limits,
                         onvalue='Manual', offvalue='Intelligent', **text_params)

pds4_tools/viewer/summary_view.py

@@ -546,13 +546,13 @@ def _add_menus(self, quiet, lazy_load, show_headers):
 
         # Initialize menu options
         self._menu_options['quiet'] = BooleanVar()
-        self._add_trace(self._menu_options['quiet'], 'w', self._update_quiet, default=quiet)
+        self._add_trace(self._menu_options['quiet'], 'write', self._update_quiet, default=quiet)
 
         self._menu_options['lazy_load'] = BooleanVar()
-        self._add_trace(self._menu_options['lazy_load'], 'w', self._update_lazy_load, default=lazy_load)
+        self._add_trace(self._menu_options['lazy_load'], 'write', self._update_lazy_load, default=lazy_load)
 
         self._menu_options['show_headers'] = BooleanVar()
-        self._add_trace(self._menu_options['show_headers'], 'w', self._update_show_headers, default=show_headers)
+        self._add_trace(self._menu_options['show_headers'], 'write', self._update_show_headers, default=show_headers)
 
         # Add a File menu
         file_menu = self._add_menu('File', in_menu='main')

pds4_tools/viewer/table_view.py

@@ -63,7 +63,7 @@ def __init__(self, viewer):
             var = option['type']
             self._menu_options[option['name']] = var
 
-            self._add_trace(var, 'w', option['trace'], option['default'])
+            self._add_trace(var, 'write', option['trace'], option['default'])
 
         # These variables are used to store widgets, and info about them, used for displaying tabular data
         self._data_boxes = []