Let's have a new ticket status: needs_revision?
Right now, needs_review means one of two things: either a ticket that actually needs code review of some kind, or a ticket that has received code review and is waiting for somebody to revise the code.
This makes it harder for me to figure out which tickets I actually need to review, which ones I need to bug somebody to finish up, and so on. It also sometimes makes people think that their code is waiting on me for review, when it is actually waiting on them to fix up.
(Why not just re-assign the ticket back to them, and remove it from needs_review? Because I find that it's really important to keep tickets with fixable code separate, so I can remember to bug the people to finish up the code, and so I can jump in and do the final cleanups if the aren't to big and the person who did the first version can't finish the patch.)
This should probably get a little discussion before anybody goes and implements it; I don't know whether anybody but me thinks it's a great idea. But I do think that it's a great idea!