Django

Code

Ticket #6071 (closed: fixed)

Opened 9 months ago

Last modified 9 months ago

Revison 6777 / 6778: maximum recursion depth exceeded

Reported by: hans.rauch@gmx.net Assigned to: trey
Milestone: Component: Forms
Version: SVN Keywords:
Cc: Triage Stage: Accepted
Has patch: 1 Needs documentation: 0
Needs tests: 0 Patch needs improvement: 0

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

trace.txt (17.3 kB) - added by hans.rauch@gmx.net on 12/01/07 03:48:55.
Traceback
utils.py (33.8 kB) - added by hans.rauch@gmx.net on 12/01/07 03:49:55.
form_system.py (7.5 kB) - added by hans.rauch@gmx.net on 12/01/07 03:50:25.
views_add.py (5.8 kB) - added by hans.rauch@gmx.net on 12/01/07 03:50:46.
6813_SafeString_Updates.patch (2.7 kB) - added by trey on 12/01/07 15:39:43.

Change History

12/01/07 03:21:20 changed by anonymous

  • needs_better_patch changed.
  • needs_tests changed.
  • needs_docs changed.

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

12/01/07 03:48:55 changed by hans.rauch@gmx.net

  • attachment trace.txt added.

Traceback

12/01/07 03:49:55 changed by hans.rauch@gmx.net

  • attachment utils.py added.

12/01/07 03:50:25 changed by hans.rauch@gmx.net

  • attachment form_system.py added.

12/01/07 03:50:46 changed by hans.rauch@gmx.net

  • attachment views_add.py added.

12/01/07 12:44:15 changed by Karen Tracey <kmtracey@gmail.com>

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.

12/01/07 15:39:17 changed by trey

  • owner changed from nobody to trey.
  • status changed from new to assigned.
  • has_patch set to 1.

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.

12/01/07 15:39:43 changed by trey

  • attachment 6813_SafeString_Updates.patch added.

12/01/07 19:24:56 changed by Simon G <dev@simon.net.nz>

  • stage changed from Unreviewed to Accepted.

12/02/07 01:26:37 changed by hans.rauch@gmx.net

  • status changed from assigned to closed.
  • resolution set to fixed.

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.

12/02/07 02:48:56 changed by Simon G <dev@simon.net.nz>

  • status changed from closed to reopened.
  • resolution deleted.

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).

12/02/07 14:17:10 changed by mtredinnick

  • status changed from reopened to closed.
  • resolution set to fixed.

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

Add/Change #6071 (Revison 6777 / 6778: maximum recursion depth exceeded)




Change Properties
Action