Task #67297

Move MySQL / DBMS field type translation into DBAL Specifics

Added by Morton Jonuschat over 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Database API (Doctrine DBAL)
Target version:
-
Start date:
2015-06-04
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
5.5
Tags:
Complexity:
Sprint Focus:

Description

Provide a default implementation in the abstract class that mimics the current behavior and allow overrides for specific DBMS that can resemble MySQL closer

  • MySQLActualType()
  • MySQLMetaType

Related issues

Related to TYPO3 Core - Epic #64459: Make TYPO3 run on PostgreSQL Closed 2015-05-21
Related to TYPO3 Core - Bug #67288: Field types don't get properly translated to MySQL types Closed 2015-06-03
Related to TYPO3 Core - Bug #67290: Translation of MySQL field types to other databases is incorrect Closed 2015-06-04
Related to TYPO3 Core - Bug #67362: PHP Fatal error: Cannot use 'Null' as class name Closed 2015-06-09

Associated revisions

Revision b1059192 (diff)
Added by Morton Jonuschat over 4 years ago

[TASK] dbal: Move MySQL / DBMS field type translation into DBMS Specifics

The function used by the DBAL provide a very coarse and generic translations
between the ADdb meta types and the MySQL native types. This results in
ambiguities that can't properly be resolved and leads to changes in the
install tool schema migration that can never be resolved.

The default DBMS specifics provided resemble the current output of the
functions and don't result in any changes to the DBMS schema.

Resolves: #67297
Related: #67290
Related: #67288
Releases: master
Change-Id: Id26a897c7f43520edd188e0bd2ddb158507c8049
Reviewed-on: http://review.typo3.org/39954
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>
Reviewed-by: Andreas Fernandez <>
Tested-by: Andreas Fernandez <>

Revision 9295cc1f (diff)
Added by Andreas Fernandez over 4 years ago

[BUGFIX] DBAL: Rename null specific

Rename the null specific for PHP 7 compliance.

Resolves: #67362
Related: #67297
Releases: master
Change-Id: I4fe56c5815f84a7e8db8abd3a4eddded4faf9044
Reviewed-on: http://review.typo3.org/40123
Reviewed-by: Mathias Brodala <>
Tested-by: Mathias Brodala <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

History

#1 Updated by Gerrit Code Review over 4 years ago

  • Status changed from New 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/39954

#2 Updated by Gerrit Code Review over 4 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/39954

#3 Updated by Gerrit Code Review over 4 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/39954

#4 Updated by Gerrit Code Review over 4 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/39954

#5 Updated by Gerrit Code Review over 4 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/39954

#6 Updated by Morton Jonuschat over 4 years ago

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

#7 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF