This is because the column 'nid' specifies a 'length' but lengths are for string based data types. You can' t specify a length for int columns but you can specify a 'size' with possible values of small, normal (default) or large.

- 'length' => 50,

see #346450: Schema API: 'length' should be ignored on non varchar and char fields

CommentFileSizeAuthor
#1 send-346450.patch564 bytesstuzza

Comments

stuzza’s picture

stuzza commented
StatusFileSize
new send-346450.patch564 bytes

Have attached a mighty one line patch addressing this.

josh waihi’s picture

josh waihi commented
Status: Active » Reviewed & tested by the community

Thanks @stuzza

stuzza’s picture

stuzza commented
Status: Reviewed & tested by the community » Active

A colleague suggested it useful to ask why 'length' => 50 was specified in the first place. In the node_revisions table nid is defined as a standard int (in node table, of course, it's a serial). I note that numerous fields in the send_schema() are 'varchar' of length 50 so I'm guessing this was a typo rather than an attempt to specify int size.

Thoughts?

Stuart

josh waihi’s picture

josh waihi commented
Status: Active » Reviewed & tested by the community

patch fixes problem, length doesn't need to be specified and replicating the nid column in the node table will keep things consistent.

jerdavis’s picture

jerdavis
Location Minneapolis, MN
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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