Opened 17 years ago
Closed 17 years ago
#6071 closed (fixed)
Revison 6777 / 6778: maximum recursion depth exceeded
| Reported by: | 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)
Change History (12)
comment:1 by , 17 years ago
by , 17 years ago
by , 17 years ago
| Attachment: | form_system.py added |
|---|
by , 17 years ago
| Attachment: | views_add.py added |
|---|
comment:2 by , 17 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 , 17 years ago
| Has patch: | set |
|---|---|
| Owner: | changed from to |
| Status: | new → assigned |
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 , 17 years ago
| Attachment: | 6813_SafeString_Updates.patch added |
|---|
comment:4 by , 17 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:5 by , 17 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
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 , 17 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
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 , 17 years ago
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
Need to see your relevant source code, and a full traceback that includes line numbers within your source code.