Comment 0 for bug 358296

Henning Eggers (henninge) wrote :

Looking at IRevision and its implementation Revsion I found that more fields are marked as "notNull=True" in the implementation then are marked as "required=True" in the interface. I would have thought that theese should match.

IRevision required=True:
date_created, revision_date, karma_allocated

Revision notNull=True:
date_created, log_body, revision_author, revision_id, revision_date, karma_allocated

I stumbled onto this when I wanted to access revision_author.person and wondered if I'd needed to check revision_author for None first.