Bug #67300

dbal admin_get_tables() returns additional DBMS specific info for DEFAULT

Added by Morton Jonuschat about 1 month ago. Updated about 1 month ago.

Status:Resolved Start date:2015-06-05
Priority:Should have Due date:
Assigned To:Morton Jonuschat % Done:

100%
Category:DBAL Spent time: -
Target version:-
TYPO3 Version:7 Is Regression:No
PHP Version:5.6 Sprint Focus:
Complexity:

Description

MySQL:

default ''

DBAL (PostgreSQL):

default '\'\'::character varying'

This is wrong in two ways:

1) The correct value to show would be default ''::character varying
2) The ::character varying part is PostgreSQL specific/internal

Related issues

related to Core - Epic #64459: Make TYPO3 run on PostgreSQL New 2015-01-23
related to Core - Bug #67329: auto_increment extra missing from admin_get_fields() Resolved 2015-06-06

Associated revisions

Revision 7876d5ff
Added by Morton Jonuschat about 1 month ago

[TASK] dbal: convert DBMS specific key/default/extra values to MySQL SQL

Allow DBMS specific transformations of field information to MySQL SQL
dialect. Provide the database compare in the install tool with more
details about the current database schema to avoid changes that can not
be implemented due to syntax differences.

Resolves: #67300
Resolves: #67329
Releases: master
Change-Id: Id415ba6246963fd0da97b5825fbe4fc9ac959641
Reviewed-on: http://review.typo3.org/40156
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>
Reviewed-by: Andreas Fernandez <>
Tested-by: Andreas Fernandez <>

History

#1 Updated by Morton Jonuschat about 1 month ago

  • Assigned To set to Morton Jonuschat

#2 Updated by Gerrit Code Review about 1 month 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/40156

#3 Updated by Gerrit Code Review about 1 month 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/40156

#4 Updated by Gerrit Code Review about 1 month 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/40156

#5 Updated by Gerrit Code Review about 1 month 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/40156

#6 Updated by Gerrit Code Review about 1 month 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/40156

#7 Updated by Gerrit Code Review about 1 month 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/40156

#8 Updated by Gerrit Code Review about 1 month 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/40156

#9 Updated by Morton Jonuschat about 1 month ago

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

Also available in: Atom PDF