Bug #76641

Database compare and "NULL default NULL" is not recognized

Added by Pascal Dürsteler over 5 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Install Tool
Target version:
-
Start date:
2016-06-15
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

I have a case where a decimal field is needed, which has a default value of NULL, and can contain NULL values, to differentiate between 0.0 and NULL.

The definition is

price decimal(5,2) unsigned NULL default NULL

Database compare then shows said field under "change fields" as

CHANGE price price decimal(5,2) unsigned NULL default NULL; 
Current value: decimal(5,2) unsigned default NULL

However, using "unsigned default NULL" would not allow NULL values and will yield errors on inserting or updating records.


Related issues

Related to TYPO3 Core - Bug #76508: Comparison of NOT NULL fields without a default value does not work in "Compare Database" tool if a field is not autoincrementClosed2016-06-07

Actions
Related to TYPO3 Core - Feature #77643: Reimplement SqlSchemaMigrationService using SchemaManagerClosedMorton Jonuschat2016-08-25

Actions
#1

Updated by Gerrit Code Review about 5 years ago

  • Status changed from New to Under Review

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49593

#2

Updated by Gerrit Code Review about 5 years ago

Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49593

#3

Updated by Gerrit Code Review about 5 years ago

Patch set 14 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49593

#4

Updated by Gerrit Code Review about 5 years ago

Patch set 15 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49593

#5

Updated by Morton Jonuschat about 5 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#6

Updated by Benni Mack about 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF