and Schema rightfully complains about this, as a primary key can never be NULL.

CommentFileSizeAuthor
#2 field-revision-id-can-be-null.patch1.13 KBStevel
#1 field-revision-id-can-be-null.patch1.08 KBStevel
field-revision-id-can-be-null.patch893 bytesStevel

Comments

Stevel’s picture

Stevel commented
StatusFileSize
new field-revision-id-can-be-null.patch1.08 KB

This is only true for the field revision table of course...

Stevel’s picture

Stevel commented
StatusFileSize
new field-revision-id-can-be-null.patch1.13 KB

And there is a comment definition that doesn't really do anything, because revision_id is contained within the 'fields' array.

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Reviewed & tested by the community

That's a no-brainer. Good catch!

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

#2: field-revision-id-can-be-null.patch queued for re-testing.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.