Code review comment for lp:~bac/launchpad/bug-602773

Revision history for this message
Brad Crittenden (bac) wrote :

Edwin,

I removed the validate_person and then discovered it was doing more than we originally thought. Many tests broke with its removal as they were counting on it to check for None, etc. I have restored validate_person.

« Back to merge proposal