Opened 18 years ago
Closed 18 years ago
#3736 closed (fixed)
[patch] [4724] uses MySQLdb.__version__ instead of Database.__version__
Reported by: | Matt Riggott | Owned by: | Adrian Holovaty |
---|---|---|---|
Component: | Database layer (models, ORM) | Version: | dev |
Severity: | Keywords: | ||
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
In [4724], MySQLdb is imported as Database. If the required version of MySQLdb isn't found it raises an error:
raise ImportError, "MySQLdb-1.2.1p2 or newer is required; you have %s" % MySQLdb.__version__
However, because the name 'MySQL' doesn't exist, a NameError is raised instead. It should be:
raise ImportError, "MySQLdb-1.2.1p2 or newer is required; you have %s" % Database.__version__
Patch is attached.
Attachments (1)
Change History (4)
by , 18 years ago
Attachment: | mysql_base.diff added |
---|
comment:1 by , 18 years ago
Summary: | [4724] uses MySQLdb.__version__ instead of Database.__version__ → [patch] [4724] uses MySQLdb.__version__ instead of Database.__version__ |
---|
comment:2 by , 18 years ago
Triage Stage: | Unreviewed → Ready for checkin |
---|
comment:3 by , 18 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Fixed with another patch in [4751].