#1096 (interim?) fix by testing isfile() before the remove()

svn: r9411
This commit is contained in:
James G Sack 2007-11-26 10:21:18 +00:00
parent 63a3f8479b
commit f6b5ac971e
2 changed files with 7 additions and 1 deletions

View File

@ -1,3 +1,7 @@
2007-11-26 Jim Sack <jgsack@san.rr.com>
* src/plugins/ReadPkg.py : #1096 add isfile() before remove()
Note: this may be part a larger "relative path" problem
2007-11-25 Jim Sack <jgsack@san.rr.com> 2007-11-25 Jim Sack <jgsack@san.rr.com>
* src/Utils.py : add get_empty_tempdir() and rm_tempdir() * src/Utils.py : add get_empty_tempdir() and rm_tempdir()
* src/ArgHandler.py : use get_empty_tempdir, etc * src/ArgHandler.py : use get_empty_tempdir, etc

View File

@ -82,6 +82,7 @@ def impData(database, name, cb=None, cl=0):
else: # tempdir exists and writable -- clean it up if not empty else: # tempdir exists and writable -- clean it up if not empty
files = os.listdir(tmpdir_path) ; files = os.listdir(tmpdir_path) ;
for filename in files: for filename in files:
if os.path.isfile(filename):
os.remove(os.path.join(tmpdir_path, filename)) os.remove(os.path.join(tmpdir_path, filename))
try: try:
@ -98,6 +99,7 @@ def impData(database, name, cb=None, cl=0):
importer = gramps_db_reader_factory(const.APP_GRAMPS_XML) importer = gramps_db_reader_factory(const.APP_GRAMPS_XML)
importer(database, imp_db_name, cb) importer(database, imp_db_name, cb)
# (it's not really a "tempdir" --Note this code may be in-flux?)
# Clean up tempdir after ourselves # Clean up tempdir after ourselves
# THIS HAS BEEN CHANGED, because now we want to keep images # THIS HAS BEEN CHANGED, because now we want to keep images
# stay after the import is over. Just delete the XML file. # stay after the import is over. Just delete the XML file.