Opened 16 years ago

Closed 16 years ago

Last modified 13 years ago

#8680 closed (fixed)

gis may break Python help() modules command

Reported by: Karen Tracey Owned by: nobody
Component: GIS Version: dev
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

As reported here: http://groups.google.com/group/django-users/browse_thread/thread/47529751d7a82fee?hl=en#

This is similar to #5743 and perhaps could use a similar approach to fix?

Attachments (1)

8680.patch (766 bytes ) - added by gkelly 16 years ago.
Patch to raise different exception that allows Python's help to finish.

Download all attachments as: .zip

Change History (5)

by gkelly, 16 years ago

Attachment: 8680.patch added

Patch to raise different exception that allows Python's help to finish.

comment:1 by Malcolm Tredinnick, 16 years ago

Resolution: fixed
Status: newclosed

(In [8727]) Fixed #8680 -- Changed a couple of GEOExceptions to Python's RuntimeError so
that help() works. Without this change, even if you didn't use the gis stuff,
just using Python 2.5's help() system would crash when searching for model
names.

comment:2 by Malcolm Tredinnick, 16 years ago

As noted in #5743, our policy is not to break help() for everybody just because we're installed, so this had to be fixed. There's a full explanation in the earlier tickets (where people were playing "why is Malcolm so dumb he can't understand this" and using sock puppets to teach me the way forwards).

Justin: I somewhat arbitrarily chose RuntimeError (the choices are RuntimeError and ImportError), but if you want to make a subclass of GEOSException and RuntimeError because you really want to be able to catch the former, go for it.

comment:3 by Malcolm Tredinnick, 16 years ago

(In [8728]) Changed [8727] to use ImportError, which seems to actually work. Not sure what
I was smoking when I tested this 10 minutes ago. Refs #8680. Thanks to gkelly
for cleaning up after me.

comment:4 by Jacob, 13 years ago

milestone: 1.0

Milestone 1.0 deleted

Note: See TracTickets for help on using tickets.
Back to Top