socallinuxexpo / scalereg

Registration software for the Southern California Linux Expo
Other
3 stars 3 forks source link

cash payment allows the creation of attendees without names or email addresses #27

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Login to staff portal
2. Select Cash Payment from services list.
3. Do not fill in any fields, and click register button.

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

This used to return an error indicating a required field was missing.
Its looks like the django behavior changed during one of the 1.x upgrades.
request.POST now returns an empty string for empty form fields rather than not 
including them in the list at all.

Original issue reported on code.google.com by i...@fonz.net on 14 Aug 2012 at 9:18

GoogleCodeExporter commented 9 years ago
I only have 1.2 and 1.4 here, so I don't know if this changed in 1.3 or 1.4. 
This is a rather subtle change and a potential source for a lot of bugs, but I 
can't find it in the release notes nor can I find a bug for it on the Django 
bug tracker.

I think we need to fix it in tons of places. I guess we can take out this bug 
and bug 28 in one shot if we do that. Arg, what a pain.

Original comment by zhang...@gmail.com on 4 Sep 2012 at 7:13