Reported by schema.module:

column notified - differences on: size, unsigned
declared: array('description' => 'TODO: please describe this field!', 'type' => 'int', 'size' => 'tiny', 'not null' => TRUE, 'default' => 0)
actual: array('description' => 'TODO: please describe this field!', 'type' => 'int', 'unsigned' => TRUE, 'size' => 'small', 'not null' => TRUE, 'default' => 0)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

greggles’s picture

Priority: Minor » Normal
Status: Active » Postponed (maintainer needs more info)

Was this a fresh install? An upgrade?

Wim Leers’s picture

Fresh install of comment_notify, after a Drupal core upgrade from D5. I'm pretty sure I have upgraded comment_notify from 1.0-beta2 to 1.0-rc1, but I'm not 100% certain. (Local dev site.)

greggles’s picture

Status: Postponed (maintainer needs more info) » Active

Ok, thanks.

nikhiljain’s picture

I'm having this problem in 6.x-1.6 version.

nikhiljain’s picture

any solution yet?

greggles’s picture

Afaik this isn't a "problem" aside from schema complaining about it so it's not a priority to me. I'd be happy to review/commit a patch someone else creates.

nikhiljain’s picture

I'm sorry for I'm not a drupal developer. So cant provide a patch. I'll wait for the next release then. But just to be clear, can this schema mismatch issue affect my site in any sense?

jasonsbytes’s picture

The module creates/updates the notified database field as type: unsigned small however the schema declaration for this field in the module is set to type: signed tiny.

Since this field appears to be a 1 or 0 flag, I changed the datatype using phpMyAdmin on the notified field and set it to tinyint and removed the unsigned attribute and that eliminated the schema complaint.

iamEAP’s picture

Status: Active » Needs review
FileSize
965 bytes

Ran into this on a D6 -> D7 upgrade.

Looks like the patch here included an update that didn't correspond to the schema: #639618: user warning: Field 'notified' doesn't have a default value

Here's a patch that brings the schema back to what's declared.

Note this patch shouldn't have any effect on the issue referenced above, as it was about a missing default value, which is in the schema now.

greggles’s picture

Title: Schema mismatch » Schema mismatch in comment_notify upgrades
Status: Needs review » Fixed

Great, fixed http://drupalcode.org/project/comment_notify.git/commit/d6aacfb

Thanks for your help, iamEAP, jasonbytes,

Status: Fixed » Closed (fixed)

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

Wim Leers’s picture

Hurray! :)