Project

General

Profile

Actions

Feature #67301

closed

Don't suggest database alterations for functionally equivalent field types with DBAL

Added by Morton Jonuschat almost 9 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Category:
Install Tool
Target version:
-
Start date:
2015-06-05
Due date:
% Done:

100%

Estimated time:
PHP Version:
5.6
Tags:
Complexity:
Sprint Focus:

Description

When using the InstallTool with the DBAL extension a lot of database alterations will be suggested that can never be resolved due to MySQL specific field types.
When the extension DBAL is loaded accept field types that only differ in their storage space requirements as long as they are functionally equivalent. Example:

TINYTEXT -> VARCHAR is OK
TINYINT -> SMALLINT, INTEGER, BIGINT is OK
INTEGER -> BIGINT is OK
MEDIUMINT -> INTEGER is OK
MEDIUMINT -> SMALLINT is NOT OK

Especially integer types which have lower min/max values should still be migrated.

Also ignore length information for INT fields as this information is purely for display purposes and doesn't change anything in regard to the values that can be stored in the field. Also the display width is ignored when the value exceeds it.


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Epic #64459: Make TYPO3 run on PostgreSQLClosed2015-05-21

Actions
Blocked by TYPO3 Core - Bug #67445: ALTER TABLE ADD KEY / DROP KEY unsupportedClosedMorton Jonuschat2015-06-14

Actions
Actions #1

Updated by Morton Jonuschat almost 9 years ago

  • Assignee set to Morton Jonuschat
Actions #2

Updated by Morton Jonuschat almost 9 years ago

  • Status changed from New to In Progress

Waiting for related patches to get review/merged as this depends on a lot of other bugfixes

Actions #3

Updated by Gerrit Code Review almost 9 years ago

  • Status changed from In Progress 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/40455

Actions #4

Updated by Gerrit Code Review almost 9 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/40455

Actions #5

Updated by Gerrit Code Review almost 9 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/40455

Actions #6

Updated by Gerrit Code Review almost 9 years ago

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

Actions #7

Updated by Gerrit Code Review almost 9 years ago

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

Actions #8

Updated by Gerrit Code Review almost 9 years ago

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

Actions #9

Updated by Gerrit Code Review almost 9 years ago

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

Actions #10

Updated by Morton Jonuschat almost 9 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF