Merge pull request #282 from EddieTheHead/master

Check permision on write to file even if .BAC0 directory exists
ChristianTremblay · Jul 28, 2021 · fef37bf · fef37bf
2 parents c363d02 + cc35bfe
commit fef37bf
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/BAC0/core/utils/notes.py b/BAC0/core/utils/notes.py
@@ -204,17 +204,17 @@ def note_and_log(cls):
     logSaveFilePath = join(logUserPath, ".BAC0")
 
     logFile = join(logSaveFilePath, "BAC0.log")
-    if not os.path.exists(logSaveFilePath):
-        try:
-            os.makedirs(logSaveFilePath)
-        except:
-            _PERMISSION_TO_WRITE = False
-    if _PERMISSION_TO_WRITE:
+    try:
+        if not os.path.exists(logSaveFilePath):
+                os.makedirs(logSaveFilePath)
         fh = FileHandler(logFile)
         fh.set_name("file_handler")
         fh.setLevel(file_level)
         fh.setFormatter(formatter)
 
+    except OSError:
+        _PERMISSION_TO_WRITE = False
+
     ch.setFormatter(formatter)
     ch2.setFormatter(formatter)
     # Add handlers the first time only...

diff --git a/BAC0/infos.py b/BAC0/infos.py
@@ -12,5 +12,5 @@
 __email__ = "christian.tremblay@servisys.com"
 __url__ = "https://github.com/ChristianTremblay/BAC0"
 __download_url__ = "https://github.com/ChristianTremblay/BAC0/archive/master.zip"
-__version__ = "21.05.28dev"
+__version__ = "21.07.27dev"
 __license__ = "LGPLv3"