FIX: fixed the editor being unstable after loading the demo data (closes #232)

The problem is that the path is given as a Path object, which is passed as is to QSettings (for saving the recent files list) and in the status.
The first case prevents the editor from even starting *when using several Python versions on the same machine* (I suspect an incompatible pickle),
the second case displayed annoying errors.

Author: gdementen

larray_editor/editor.py

@@ -1,6 +1,9 @@
 import os
 import re
 import sys
+from pathlib import Path
+from typing import Union
+
 
 # Python3.8 switched from a Selector to a Proactor based event loop for asyncio but they do not offer the same
 # features, which breaks Tornado and all projects depending on it, including Jupyter consoles
@@ -773,8 +776,9 @@ def set_current_array(self, array, name):
         self.current_array_name = name
         self.update_title()
 
-    def set_current_file(self, filepath):
-        self.recent_data_files.add(filepath)
+    def set_current_file(self, filepath: Union[str, Path]):
+        if filepath is not None:
+            self.recent_data_files.add(filepath)
         self.current_file = filepath
         self.update_title()
 
@@ -1010,12 +1014,10 @@ def save_script(self):
                 else:
                     self._save_script(filepath, lines, overwrite)
 
-
-    #=============================#
-    #  METHODS TO SAVE/LOAD DATA  #
-    #=============================#
-
-    def _open_file(self, filepath):
+    # ============================= #
+    #  METHODS TO SAVE/LOAD DATA    #
+    # ============================= #
+    def _open_file(self, filepath: Union[str, Path]):
         session = la.Session()
         # a list => .csv files. Possibly a single .csv file.
         if isinstance(filepath, (list, tuple)):

larray_editor/utils.py

@@ -2,6 +2,9 @@
 import sys
 import math
 import logging
+from pathlib import Path
+from typing import Union
+
 from larray.util.misc import Product
 
 import numpy as np
@@ -605,13 +608,16 @@ def files(self, files):
     def actions(self):
         return self._actions
 
-    def add(self, filepath):
-        if filepath is not None:
-            recent_files = self.files
-            if filepath in recent_files:
-                recent_files.remove(filepath)
-            recent_files = [filepath] + recent_files
-            self.files = recent_files
+    def add(self, filepath: Union[str, Path]):
+        if filepath is None:
+            return
+        elif isinstance(filepath, Path):
+            filepath = str(filepath)
+        recent_files = self.files
+        if filepath in recent_files:
+            recent_files.remove(filepath)
+        recent_files = [filepath] + recent_files
+        self.files = recent_files
 
     def clear(self):
         self.files = []
@@ -624,10 +630,12 @@ def _update_actions(self):
 
             # zip will iterate up to the shortest of the two
             for filepath, action in zip(recent_files, self.actions):
+                if isinstance(filepath, Path):
+                    filepath = str(filepath)
                 action.setText(os.path.basename(filepath))
                 action.setStatusTip(filepath)
                 action.setData(filepath)
                 action.setVisible(True)
-            # if we have less recent recent files than actions, hide the remaining actions
+            # if we have less recent files than actions, hide the remaining actions
             for action in self.actions[len(recent_files):]:
                 action.setVisible(False)