Column altering in migration from TEXT to LONGTEXT may not apply.

[janokary]

Based on this
laravel/framework#12363
and my experience here
https://laracasts.com/discuss/channels/laravel/alter-table-change-field-to-longtext

I think there is a problem when migrating a table column from TEXT to LONG TEXT

[Ocramius]

@janokary what RDBMs does this apply to? Maybe this is missing schema introspection?

[janokary]

It is allways mysql

version: 5.5.52-0+deb8u1

or

version: 5.7.11-log

or

version: 5.7.12-0ubuntu1.1

[deeky666]

@janokary I need more information. What is the Doctrine DBAL type before and what should it be after? Also length property is significant.

[ariews]

Hi all, I'm facing same issue, using Laravel 5.4, doctrine/dbal v2.5.12, php 5.6.30, mysql Ver 14.14 Distrib 5.6.33.

before: $table->text('body') alter using: $table->longText('body')->change().

The column type should be longtext, but the column type is unchanged.

[Ocramius]

@ariews can this be reproduced with DBAL-only?

[frandieguez]

Any news on getting this integrated?

[Ocramius]

@frandieguez this needs a test case first.

[frandieguez]

@Ocramius I can see that the solution presented at https://github.com/Cheezykins/dbal/commit/c119353f7832cd4da2003b0ce388f7727e3fcfa9 contains a test case. Does this solution is "good enought"?

I'm quite interested on getting this merged, even if I can help in some way.

[Ocramius]

Can't find a PR for that: can you open or dig up one?

[cbotsikas]

@Ocramius here's your PR.

[cbotsikas]

I came across this issue trying to change a column in a Laravel application from text to longtext. I traced the migration and the issue was that the change was ignored since $changedProperties was empty although the _length was different (65535 on $table1 vs 16777216 on $table2) .
The creation migration in Laravel was like this:

Schema::create('myTable', function (Blueprint $table) {
    $table->increments('id');
    $table->text('myColumn')->nullable();
});

and the change migration like this:

Schema::table('myTable', function (Blueprint $table) {
    $table->longText('myColumn')->change();
});

[cbotsikas]

I have added some comments on my PR. There's an issue with a failing test which i feel is not well defined.
Please have a look and let me know if i'm missing something..

[uphlewis]

Any movement on this? Currently experiencing this problem, and google brought me to this github issue created over 2 years ago... !

[uphlewis]

Some helpful fellow has suggested this workaround, for those who come after me:

laravel/framework#21847 (comment)

tldr; change to string column with length > 16777216, since it exceeds varchar max length the column gets converted to longtext. Hacky, but it works.

[kl83]

I found another workaround. Just change the column comment, for example:

$table->mediumText('myColumn')->comment(' ')->change(); // up
$table->text('myColumn')->comment('')->change(); // down

Works perfectly on Ubuntu 18.04, PHP 7.2, MySQL 8, Laravel 5.8

[ZsgsDesign]

I found another workaround. Just change the column comment, for example:

$table->mediumText('myColumn')->comment(' ')->change(); // up
$table->text('myColumn')->comment('')->change(); // down

Works perfectly on Ubuntu 18.04, PHP 7.2, MySQL 8, Laravel 5.8

I am using the same method to solve the problem. That works.

[lwitzel]

Ran across this after trying to diagnose the problem for the last hour. Both hacks work. Fixing $table->longText()->change() would be even better.

[jm-vasquez]

I found another workaround. Just change the column comment, for example:

$table->mediumText('myColumn')->comment(' ')->change(); // up
$table->text('myColumn')->comment('')->change(); // down

Works perfectly on Ubuntu 18.04, PHP 7.2, MySQL 8, Laravel 5.8

this works as well on ubuntu 16.04, php 7.0 and mariadb 10.2.8, Laravel 5.4

[InsaneSoftware]

As of today.. this bug is still in laravel..

I found another workaround. Just change the column comment, for example:

$table->mediumText('myColumn')->comment(' ')->change(); // up
$table->text('myColumn')->comment('')->change(); // down

Works perfectly on Ubuntu 18.04, PHP 7.2, MySQL 8, Laravel 5.8

As of today.. this bug is still in laravel and this is the fix..

[holymp2006]

I found another workaround. Just change the column comment, for example:

$table->mediumText('myColumn')->comment(' ')->change(); // up
$table->text('myColumn')->comment('')->change(); // down

Works perfectly on Ubuntu 18.04, PHP 7.2, MySQL 8, Laravel 5.8

As of today, when changing between text types (text, mediumText, or longText), this is still the fix.

[morozov]

This bug will be covered by the Doctrine\DBAL\Tests\Functional\Schema\MySQL::testLobLengthIncrementOverLimit test in #4746.

Additionally,

$column = new Column('myColumn', new TextType(), [
    'length' => 2**15, // TEXT
]);
$desiredTable = new Table('test', [$column]);

$sm = $connection->createSchemaManager();
$sm->dropAndCreateTable($desiredTable);
echo $connection->fetchNumeric('SHOW CREATE TABLE test')[1], PHP_EOL;

// CREATE TABLE `test` (
//   `myColumn` text COLLATE utf8_unicode_ci NOT NULL
// ) ENGINE=InnoDB DEFAULT CHARSET=utf8mb3 COLLATE=utf8_unicode_ci

$column->setLength(2**24); // LONGTEXT

$actualTable = $sm->listTableDetails('test');

$comparator = $sm->createComparator();
$diff = $comparator->diffTable($actualTable, $desiredTable);
$sm->alterTable($diff);
echo $connection->fetchNumeric('SHOW CREATE TABLE test')[1], PHP_EOL;

// CREATE TABLE `test` (
//   `myColumn` longtext COLLATE utf8_unicode_ci NOT NULL
// ) ENGINE=InnoDB DEFAULT CHARSET=utf8mb3 COLLATE=utf8_unicode_ci

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.