Column length too big for column description

Encarte - April 24, 2008 - 10:34
Project:NodeReview
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:JohnForsythe
Status:closed
Description

While testing this module, after updating from 5.x-1.0 to 5.x-1.x-dev I get the following message:

user warning: Column length too big for column 'description' (max = 255); use BLOB or TEXT instead query: ALTER TABLE nodereview_axes ADD COLUMN description varchar(1024) NOT NULL default '' in /home/bric/public_html/includes/database.mysql.inc on line 172.

#1

Encarte - May 6, 2008 - 21:44
Priority:normal» critical

It is not possible to use this module with this kind of error. Besides, it causes PHP crashes on servers where the module is installed. Since I got no feedback on this issue, I'm stopping tests, uninstalling the module and tagging this issue as critical. Maybe I can test it again in some future release...

#2

JohnForsythe - May 11, 2008 - 07:01
Status:active» by design

You need to use a newer version of MySQL. This problem was solved over 3 years ago in MySQL 5.03.

http://dev.mysql.com/doc/refman/5.0/en/char.html

#3

Encarte - May 12, 2008 - 14:40

Thank you John. I've posted two new issues about this (http://drupal.org/node/257500 and http://drupal.org/node/257501).

#4

Crell - May 12, 2008 - 15:10
Status:by design» active

John, I disagree. There's no reason you need to use a huge varchar for a blob-o-text. That's what the TEXT type is for. Technically Drupal 5 itself should support even MySQL 4.0 and MySQL 3, but I don't think those even exist anymore. :-) Let's switch the field type, unless there's some specific reason why it can't be TEXT and has to be varchar.

#5

JohnForsythe - May 15, 2008 - 17:32
Assigned to:Anonymous» JohnForsythe
Status:active» fixed

ok, fixed in latest dev release.

#6

Anonymous (not verified) - May 29, 2008 - 18:29
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.