os.path.isdir() -> .exists() for check for "comments" file.
authorW. Trevor King <wking@drexel.edu>
Mon, 21 Sep 2009 13:52:28 +0000 (09:52 -0400)
committerW. Trevor King <wking@drexel.edu>
Mon, 21 Sep 2009 13:52:28 +0000 (09:52 -0400)
"comments" should always be a directory if it exists, but if it is
something else, be should fail rather than return an empty comment
list.

This fix was inspired by some browsing around in response to Thomas
Habets' version of a fix for missing "bugs" directory.

libbe/comment.py

index 19c556ffbb012eb2e9b622b2e4307e635cdba23a..41bc7e6c1ea629277e8836186ad570ab676ea22a 100644 (file)
@@ -128,7 +128,7 @@ def loadComments(bug, load_full=False):
     if bug.sync_with_disk == False:
         raise DiskAccessRequired("load comments")
     path = bug.get_path("comments")
-    if not os.path.isdir(path):
+    if not os.path.exists(path):
         return Comment(bug, uuid=INVALID_UUID)
     comments = []
     for uuid in os.listdir(path):