felsen / django-survey

Automatically exported from code.google.com/p/django-survey
0 stars 0 forks source link

a number of survey url patterns using '\d' instead of '\d+' ... more then 10 questions or 10 choices and bad stuff.. #32

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

Create more then 10 questions, or more then 10 choices.

What is the expected output? What do you see instead?

You instead get a 500 error because likely the 'reverse()' pattern for the 
question or choice no 
longer matches.

What version of the product are you using? On what operating system?

Subversion version 92.

Please provide any additional information below.

The problem is the several url patterns that use "\d" instead of "\d+" to find 
a question or choice 
id, for example:

url(r'^choice/add/(?P<question_id>\d)/$', choice_add,   name='choice-add'),

Please find attached a patch that fixes this.

Original issue reported on code.google.com by scan...@gmail.com on 28 Oct 2008 at 11:01

Attachments:

GoogleCodeExporter commented 9 years ago
Hello,
good catch and thank you for the patch.
This is fixed in r93.
--yml

Original comment by yann.ma...@gmail.com on 30 Oct 2008 at 12:59