CakeDC/migrations — pull request #69 by airdrummingfool (2012-06-05 19:52:25+00:00)
Title: 'time' field causes errors

Commit message: Remove the 'length' attribute if we are altering a field to a time field.

I couldn't reopen issue #10, but this is directly related to it. CakePHP now supports time fields in their schema (since [at least 1.3](http://book.cakephp.org/1.3/view/1003/Data-Type-Associations-by-Database)), but Migrations tries to give the time field a length.

The error looks something like this:
```Error: SQLSTATE[42000]: Syntax error or access violation: 1064
You have an error in your SQL syntax;
check the manual that corresponds to your MySQL server version for the right syntax
to use near '(11) NOT NULL' at line 2```

And is caused by an SQL query such as:
```ALTER TABLE `dev`.`times`
CHANGE `start_time` `start_time` time(11) NOT NULL;```

The fix for removing the length when altering the 'time' field is the same as existing code that removes the length when altering a 'date' field.