FIX: reset "back" bar when selecting a variable in the list

Author: gdementen

larray_editor/arraywidget.py

@@ -341,6 +341,20 @@ def add_back(self, data, data_adapter):
         if not self.isVisible():
             self.show()
 
+    def clear(self):
+        for adapter in self._back_data_adapters[::-1]:
+            self._close_adapter(adapter)
+
+        self._back_data_adapters = []
+        self._back_data = []
+        self.hide()
+
+    @staticmethod
+    def _close_adapter(adapter):
+        clsname = type(adapter).__name__
+        logger.debug(f"closing data adapter ({clsname})")
+        adapter.close()
+
     def on_clicked(self):
         if not len(self._back_data):
             logger.warn("Back button has no target to go to")
@@ -1878,14 +1892,13 @@ def close(self):
         logger.debug("ArrayEditorWidget.close()")
         if self.data_adapter is not None:
             self._close_adapter(self.data_adapter)
-        for adapter in self.back_button_bar._back_data_adapters[::-1]:
-            self._close_adapter(adapter)
+        self.back_button_bar.clear()
         super().close()
 
     @staticmethod
     def _close_adapter(adapter):
         clsname = type(adapter).__name__
-        logger.debug(f"closing old data adapter ({clsname})")
+        logger.debug(f"closing data adapter ({clsname})")
         adapter.close()
 
     def set_data_adapter(self, data_adapter: AbstractAdapter, frac_digits):

larray_editor/editor.py

@@ -1070,7 +1070,9 @@ def set_current_array(self, array, expr_text):
             self.sql_widget.update_completer_options(self.data, selected=array)
         # FIXME: we should never store the current_array but current_adapter instead
         self.current_array = array
-        self.array_widget.set_data(array)
+        array_widget = self.array_widget
+        array_widget.back_button_bar.clear()
+        array_widget.set_data(array)
         self.current_expr_text = expr_text
         self.update_title()