-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comparator fails for BLOBS #2663
Comments
@thomashohn this needs a test case, as the issue as reported is too superficial. |
Create the following table: Try to change the column to ablob medium blob - the diffColumn doesn't catch this case. |
Seems like this happens because the the |
Is there any progress here? We‘re also experiencing this issue in our contao cms instance :-( |
I created pull request #4551 to fix this issue, it includes several test cases. |
This bug will be indirectly covered by the Additionally, $connection->executeStatement('DROP TABLE IF EXISTS test');
$connection->executeStatement('CREATE TABLE test (ablob BLOB)');
$sm = $connection->createSchemaManager();
$desiredTable = $sm->listTableDetails('test');
$connection->executeStatement('ALTER TABLE test MODIFY COLUMN ablob MEDIUMBLOB');
$actualTable = $sm->listTableDetails('test');
$comparator = $sm->createComparator();
$diff = $comparator->diffTable($desiredTable, $actualTable);
var_dump($diff);
/*
class Doctrine\DBAL\Schema\TableDiff#49 (14) {
...
}
*/ |
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. |
In the TYPO3 project we use the diffColumn method in the class Doctrine\DBAL\Schema. Unfortunately this method fails to detect changes in BLOB columns. So changing a column from BLOB to another BLOB type fails - which is problematic in an MySql env.
The text was updated successfully, but these errors were encountered: