Export to GitHub

ruckusing - issue #4

MySQLAdapter::change_column does not use 'default' and 'null' options


Posted on Dec 10, 2007 by Massive Camel

class.MySQLAdapter.php, line 347:

$sql = sprintf("ALTER TABLE %s CHANGE %s %s %s", $table_name, $column_name, $column_name, $this->type_to_sql($type,$options['limit'], $options['precision'], $options['scale'])); return $this->execute_ddl($sql);

Should be:

$sql = sprintf("ALTER TABLE %s CHANGE %s %s %s", $table_name, $column_name, $column_name, $this->type_to_sql($type,$options['limit'], $options['precision'], $options['scale'])); $sql .= $this->add_column_options($options); return $this->execute_ddl($sql);

Comment #1

Posted on Dec 21, 2007 by Swift Giraffe

Issue has been resolved and unit tested.

Thanks for your feedback!

Status: Verified

Labels:
Type-Defect Priority-Medium