Bug #64616
closedCompare Database does not remove 'NOT NULL'
100%
Description
The compare database wizard does not detect if fields to not have the NOT NULL
clause anymore (if they had such a clause previously).
MariaDB [dummy]> SHOW FIELDS FROM sys_file_processedfile; +-------------------+--------------+------+-----+---------+----------------+ | Field | Type | Null | Key | Default | Extra | +-------------------+--------------+------+-----+---------+----------------+ | uid | int(11) | NO | PRI | NULL | auto_increment | | pid | int(11) | NO | MUL | 0 | | | tstamp | int(11) | NO | | 0 | | | crdate | int(11) | NO | | 0 | | | cruser_id | int(11) | NO | | 0 | | | deleted | tinyint(4) | NO | | 0 | | | storage | int(11) | NO | MUL | 0 | | | original | int(11) | NO | MUL | 0 | | | identifier | varchar(512) | YES | | | | | name | tinytext | YES | | NULL | | | configuration | text | YES | | NULL | | | context | varchar(200) | NO | | | | | checksum | varchar(255) | NO | | | | | is_processed | varchar(200) | NO | | | | | extension | varchar(255) | NO | | | | | mime_type | varchar(255) | NO | | | | | sha1 | tinytext | YES | | NULL | | | size | int(11) | NO | | 0 | | | width | int(11) | NO | | 0 | | | height | int(11) | NO | | 0 | | | originalfilesha1 | char(40) | YES | | | | | task_type | varchar(200) | NO | | | | | configurationsha1 | char(40) | YES | | | | +-------------------+--------------+------+-----+---------+----------------+
Notice the NO
for NULL
for the fields width
and height
. The corresponding SQL file (from core sysext):
CREATE TABLE sys_file_processedfile ( uid int(11) NOT NULL auto_increment, tstamp int(11) DEFAULT '0' NOT NULL, crdate int(11) DEFAULT '0' NOT NULL, storage int(11) DEFAULT '0' NOT NULL, original int(11) DEFAULT '0' NOT NULL, identifier varchar(512) DEFAULT '' NOT NULL, name tinytext, configuration text, configurationsha1 char(40) DEFAULT '' NOT NULL, originalfilesha1 char(40) DEFAULT '' NOT NULL, task_type varchar(200) DEFAULT '' NOT NULL, checksum varchar(255) DEFAULT '' NOT NULL, width int(11) DEFAULT '0', height int(11) DEFAULT '0', PRIMARY KEY (uid), KEY combined_1 (original,task_type,configurationsha1), KEY identifier (storage,identifier(199)) );
Notice that it does not have a NOT NULL
for the width
and height
fields.
The database compare does not detect this.
Updated by Philipp Gampe almost 10 years ago
- Subject changed from Compare Database does remove 'NOT NULL' to Compare Database does not remove 'NOT NULL'
Updated by Stephan Großberndt almost 10 years ago
I had that issue some time before too. I know its cause, the according parameter is not set when the function to check the fields is called. I did not know if this is on purpose so I did not file a bug report.
Updated by Philipp Gampe almost 10 years ago
DBAL needs to xclass anyway?!? Therefore I suggest to improve the behavior for MySQL/MariaDB Databases and fix this for 95% of our users.
If you know the cause, please push a patch. Then we will at least be able to discuss this.
Updated by Stephan Großberndt almost 10 years ago
The problem is in TYPO3\CMS\Install\Service\SqlSchemaMigrationService
/** * Compares two arrays with field information and returns information about fields that are MISSING and fields that have CHANGED. * FDsrc and FDcomp can be switched if you want the list of stuff to remove rather than update. * * @param array $FDsrc Field definitions, source (from getFieldDefinitions_fileContent()) * @param array $FDcomp Field definitions, comparison. (from getFieldDefinitions_database()) * @param string $onlyTableList Table names (in list) which is the ONLY one observed. * @param bool $ignoreNotNullWhenComparing If set, this function ignores NOT NULL statements of the SQL file field definition when comparing current field definition from database with field definition from SQL file. This way, NOT NULL statements will be executed when the field is initially created, but the SQL parser will never complain about missing NOT NULL statements afterwards. * @return array Returns an array with 1) all elements from $FDsrc that is not in $FDcomp (in key 'extra') and 2) all elements from $FDsrc that is different from the ones in $FDcomp */ public function getDatabaseExtra($FDsrc, $FDcomp, $onlyTableList = '', $ignoreNotNullWhenComparing = TRUE) {
Why would you have the SQL parser not complain about missing NOT NULL after creating the table?
Either all (?) calls to getDatabaseExtra have to contain $ignoreNotNullWhenComparing = FALSE or the default signature for getDatabaseExtra should be changed to $ignoreNotNullWhenComparing = FALSE.
Updated by Gerrit Code Review almost 10 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36400
Updated by Gerrit Code Review almost 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36400
Updated by Gerrit Code Review almost 10 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36400
Updated by Gerrit Code Review almost 10 years ago
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36525
Updated by Stephan Großberndt almost 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 47bdb4f81566de9e12997d3c3c31694658baae65.
Updated by Anja Leichsenring almost 9 years ago
- Sprint Focus deleted (
On Location Sprint)