evilny0/fpdb
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Factor out multiple instances of "email steffen" error message into a function

Browse Source
This commit is contained in:
Worros 2008-10-10 02:36:12 +08:00
parent 5e1362abf9
commit 0743114c75
1 changed files with 10 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
pyfpdb/fpdb_import.py
View File

@ -153,11 +153,8 @@ class Importer:
duplicates+=1
except (ValueError), fe:
errors+=1
print "Error No.",errors,", please send the hand causing this to steffen@sycamoretest.info so I can fix it."
print "Filename:",options.inputFile
print "Here is the first line so you can identify it. Please mention that the error was a ValueError:"
print hand[0]
self.printEmailErrorMessage(errors, options.inputFile, hand[0]
if (options.failOnError):
self.db.commit() #dont remove this, in case hand processing was cancelled this ties up any open ends.
self.cursor.close()
@ -165,10 +162,8 @@ class Importer:
raise
except (fpdb_simple.FpdbError), fe:
errors+=1
print "Error No.",errors,", please send the hand causing this to steffen@sycamoretest.info so I can fix it."
print "Filename:",options.inputFile
print "Here is the first line so you can identify it."
print hand[0]
self.printEmailErrorMessage(errors, options.inputFile, hand[0]
#fe.printStackTrace() #todo: get stacktrace
self.db.rollback()
@ -206,6 +201,12 @@ class Importer:
return handsId
#end def import_file_dict
def printEmailErrorMessage(self, errors, filename, line):
print "Error No.",errors,", please send the hand causing this to steffen@sycamoretest.info so I can fix it."
print "Filename:",options.inputFile
print "Here is the first line so you can identify it. Please mention that the error was a ValueError:"
print hand[0]
if __name__ == "__main__":
print "CLI for fpdb_import is currently on vacation please check in later"