Bug #105310
closed
\TYPO3\CMS\Core\Database\Schema\Parser\Parser translates CHAR to VARCHAR
100%
Description
Given the following ext_tables.sql
CREATE TABLE table (
column CHAR(2),
);
the schema parser detects a char-type but it translates it into a doctrine/dbal type string without fixed set to true. So, this will be translated to
CREATE TABLE table (
column VARCHAR(2),
);
This would be fixed if the constructor of \TYPO3\CMS\Core\Database\Schema\Parser\AST\DataType\CharDataType sets its fixed property to true.
Alternatively, fixed could be part of the options passed into the constructor.
Updated by Alexander Schnitzler 3 months ago
- Related to Bug #103141: TCA type=uuid does not use native uuid type for postgresql added
Updated by Stefan Bürk 30 days ago
The problem with fixed CHAR is, that PostgreSQL behaves here
differently than MariaDB, MySQL and SQLite.
Postgres fills up the retrieved value with spaces to PAD the
value to the fixed length, the others will not do this.
Cross database queries like it have to be provided by TYPO3
itself MUST handle this difference(s) correctly and should
be avoid as much as possible using VARCHAR instead and have
the same behaviour accross all supported TYPO3 database vendors.
Updated by Gerrit Code Review 30 days ago
- Status changed from New to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 30 days ago
Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Stefan Bürk 25 days ago
- Related to Task #105867: Use VARCHAR instead of CHAR in core ext_tables.sql added
Updated by Gerrit Code Review 25 days ago
Patch set 3 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 20 days ago
Patch set 4 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 19 days ago
Patch set 5 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 16 days ago
Patch set 6 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 16 days ago
Patch set 7 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 16 days ago
Patch set 8 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 16 days ago
Patch set 9 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 16 days ago
Patch set 10 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 15 days ago
Patch set 11 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 15 days ago
Patch set 12 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 15 days ago
Patch set 13 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 15 days ago
Patch set 14 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87584
Updated by Gerrit Code Review 15 days ago
Patch set 1 for branch 13.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87656
Updated by Stefan Bürk 15 days ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset ba10a6ed70de823c29dfe6c4cba71aa474c49390.