Roundup Tracker - Issues

Issue 604875

classification
enhance instance loading diagnostics
Type: Severity: normal
Components: None Versions:
process
Status: closed later
:
: richard : richard, stefan
Priority: normal :

Created on 2002-09-05 03:53 by stefan, last changed 2002-09-05 03:53 by stefan.

Messages
msg324 Author: [hidden] (stefan) Date: 2002-09-05 03:53
if dbinit.py contains errors (which is quite possible
as that file is supposed to be edited by the tracker
admin),
'roundup-admin initialise' will generate a bogus error
saying 
 " 'module' object has no attribute 'init' "

It would be far more useful to catch load errors and report
correctly what went wrong, instead of initializing the
'module' with a dummy placeholder object.
msg325 Author: [hidden] (richard) Date: 2002-09-05 04:02
Logged In: YES 
user_id=6405

Yeah, the try/except in the instance __init__ has always been evil. I 
guess it's time to actually try to come up with a better solution. 
msg326 Author: [hidden] (stefan) Date: 2002-09-05 04:06
Logged In: YES 
user_id=764

what errors can occure here ? Why are you catching exceptions
in the first place, instead of just letting the caller deal
with them ?

If you want the convenience of a clear error message (and no
backtrace), you should probably wrap the imp.load_package()
in a try block, and then issue a sensible error message and
abort...
msg327 Author: [hidden] (richard) Date: 2002-09-05 04:19
Logged In: YES 
user_id=6405

I'm going to move this discussion to the mailing list where it's easier. 
History
Date User Action Args
2002-09-05 03:53:50stefancreate