Opened 9 years ago

Closed 9 years ago

#6071 closed (fixed)

Revison 6777 / 6778: maximum recursion depth exceeded

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

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@… 9 years ago.
Traceback
utils.py (33.8 KB) - added by hans.rauch@… 9 years ago.
form_system.py (7.5 KB) - added by hans.rauch@… 9 years ago.
views_add.py (5.8 KB) - added by hans.rauch@… 9 years ago.
6813_SafeString_Updates.patch (2.7 KB) - added by Trey 9 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 9 years ago by anonymous

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

Changed 9 years ago by hans.rauch@…

Attachment: trace.txt added

Traceback

Changed 9 years ago by hans.rauch@…

Attachment: utils.py added

Changed 9 years ago by hans.rauch@…

Attachment: form_system.py added

Changed 9 years ago by hans.rauch@…

Attachment: views_add.py added

comment:2 Changed 9 years ago by Karen Tracey <kmtracey@…>

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 Changed 9 years ago by Trey

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.

Changed 9 years ago by Trey

comment:4 Changed 9 years ago by Simon G <dev@…>

Triage Stage: UnreviewedAccepted

comment:5 Changed 9 years ago by hans.rauch@…

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 Changed 9 years ago by Simon G <dev@…>

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 Changed 9 years ago by Malcolm Tredinnick

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