Merge pull request #189 from nodedge/fix/dats_recent_files

Resolves #183: Do not add log file to recent file if format is not su…

nodedge/dats/dats_window.py

@@ -542,7 +542,6 @@ def replaceCurve(self, oldName, newName):
                         self.plotCurves([newName])
 
     def deleteCurve(self):
-
         worksheetTabWidget = self.workbooksTabWidget.currentWidget()
         nPlotWidget = worksheetTabWidget.currentWidget()
         if not nPlotWidget.items:
@@ -715,6 +714,17 @@ def addToRecentFiles(self, filepath):
         :param filepath: absolute path and filename of the file to open.
         :type filepath: ``str``
         """
+        extension = os.path.splitext(filepath)[1]
+        if extension.lower() not in [
+            ".mf4",
+            ".mdf4",
+            ".csv",
+            ".mat",
+            ".txt",
+            ".hdf5",
+            ".h5",
+        ]:
+            return
         if filepath in self.recentFiles:
             self.recentFiles.remove(filepath)
         self.recentFiles.insert(0, filepath)

nodedge/dats/logs_list_widget.py

@@ -91,7 +91,7 @@ def openLog(self, filename) -> Optional[MDF]:
         if extension.lower() == "mf4":
             log = MDF(filename)
         elif extension.lower() in ["csv", "txt"]:
-            with open(filename, "r") as f:
+            with open(filename) as f:
                 line = f.readline()
                 separator = ","
                 if "," not in line:
@@ -212,7 +212,6 @@ def addLog(self, log, shortname, prependDate=True):
 
     def addLogs(self, logs, prependDate=True):
         for logName, log in logs.items():
-
             self.addLog(log, logName, prependDate)
 
     def onItemClicked(self, item):