According to API docs
http://api.drupal.org/api/function/hook_field_schema/7
this hook MUST be defined in .install for it to be detected during installation and upgrade.

By now, it is in field_example.module

CommentFileSizeAuthor
#3 939740-field-example-schema-D7.patch2.25 KBdave reid
#2 939740-field-example-schema-D7.patch0 bytesdave reid

Comments

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented

This was a recent change to core, and it's "best practice" more than "must be". But thanks for pointing it out! Patches are welcome.

-Randy

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 939740-field-example-schema-D7.patch0 bytes
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 939740-field-example-schema-D7.patch2.25 KB

Try that again...

Status: Needs review » Needs work

The last submitted patch, 939740-field-example-schema-D7.patch, failed testing.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Odd, I cannot duplicate these failures yet.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Needs review

#3: 939740-field-example-schema-D7.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 939740-field-example-schema-D7.patch, failed testing.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Needs work » Reviewed & tested by the community

Passes now. I'll commit.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD: http://drupal.org/cvs?commit=436222

Thanks!

Status: Fixed » Closed (fixed)

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