[Tracker-discuss] [issue336] 'problem encountered' message when giving invalid nosy entry

[Ezio Melotti]

Ezio Melotti <ezio.melotti at gmail.com> added the comment:

I've looked into this problem and this is what I figured out:
1) Roundup checks that the names are invalid;
2) It generates an error message;
3) It tries to render the page with the error message on the top in a red box;
4) It keeps displaying the rest of the page with invalid names still in the nosy;
5) It tries to sort the names by their id;
6) An invalid query is sent to the database and this "blocks" the db;
7) At the next query the db is still blocked an error is raised;

The easy fix is check that the id used in the query at 6) is numeric and raise an error if it's not.

----------
assignedto:  -> ezio.melotti
nosy: +ezio.melotti
status: unread -> in-progress

_______________________________________________________
PSF Meta Tracker <metatracker at psf.upfronthosting.co.za>
<http://psf.upfronthosting.co.za/roundup/meta/issue336>
_______________________________________________________