Opened 16 years ago

Closed 16 years ago

#6071 closed (fixed)

Revison 6777 / 6778: maximum recursion depth exceeded

Reported by: hans.rauch@… Owned by: Trey
Component: Forms Version: dev
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Until revison 6777 my Django-program works fine. If I update to version 6778, I'll get the following error:

Request Method: GET
Request URL: http://dms.bildung.hessen.de/wir_ueber_uns/intern/index.html/edit/
Exception Type: RuntimeError
Exception Value: maximum recursion depth exceeded
Exception Location: /usr/local/lib64/python2.5/site-packages/django/utils/safestring.py in add, line 73
Python Executable: /usr/local/bin/python
Python Version: 2.5.1

The error occurs, if I try to show a form (newforms).

Attachments (5)

trace.txt (17.3 KB ) - added by hans.rauch@… 16 years ago.
Traceback
utils.py (33.8 KB ) - added by hans.rauch@… 16 years ago.
form_system.py (7.5 KB ) - added by hans.rauch@… 16 years ago.
views_add.py (5.8 KB ) - added by hans.rauch@… 16 years ago.
6813_SafeString_Updates.patch (2.7 KB ) - added by Trey 16 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 by anonymous, 16 years ago

Need to see your relevant source code, and a full traceback that includes line numbers within your source code.

by hans.rauch@…, 16 years ago

Attachment: trace.txt added

Traceback

by hans.rauch@…, 16 years ago

Attachment: utils.py added

by hans.rauch@…, 16 years ago

Attachment: form_system.py added

by hans.rauch@…, 16 years ago

Attachment: views_add.py added

comment:2 by Karen Tracey <kmtracey@…>, 16 years ago

The traceback seems to have been truncated at 200 lines. Could you also attach one that shows just the traceback, not all the local vars (that is, post what you see after clicking "Switch to copy-and-paste view")? What's visible in the first one doesn't show how the code gets into deep recursion.

comment:3 by Trey, 16 years ago

Has patch: set
Owner: changed from nobody to Trey
Status: newassigned

There were some additional SafeString issues earlier today. This patch includes updated tests and new SafeString operations that should be nicer when adding safe strings together as well as plain strings to safe strings.

by Trey, 16 years ago

comment:4 by Simon G <dev@…>, 16 years ago

Triage Stage: UnreviewedAccepted

comment:5 by hans.rauch@…, 16 years ago

Resolution: fixed
Status: assignedclosed

Thanks! The patch 6813_SafeString_Updates.patch is working well.

By the way: I couldn't send you the 'Switch to copy-and-paste view', since it was empty.

comment:6 by Simon G <dev@…>, 16 years ago

Resolution: fixed
Status: closedreopened

Glad to hear it works for you Hans, but we'll keep this opened until it's merged into Django's trunk (and is truly fixed).

comment:7 by Malcolm Tredinnick, 16 years ago

Resolution: fixed
Status: reopenedclosed

(In [6845]) Fixed #6071 -- Fixed another infinite recursion problem in SafeString and
SafeUnicode. Thanks, Trey Long.

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