IRevision and Revision do not match required and notNull.

[Henning Eggers]

Looking at IRevision and its implementation Revsion I found that more fields are marked as "notNull=True" in the implementation than are marked as "required=True" in the interface. I would have thought that these 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.

[Tim Penhey (thumper)]

Trivial fix for someone.

[Matthew Revell (matthew.revell)]

Retargeting to 3.0, which is the new name for 2.2.9. Haven't renamed the milestone as the 3.0 milestone is in a different series.