Feature #44991

DB table COLLATION: disabled for ext_tables.sql but not for ext_tables_static+adt.sql

Added by Alain over 9 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Should have
Category:
Database API (Doctrine DBAL)
Target version:
Start date:
2013-01-30
Due date:
% Done:

100%

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

Description

Case I

When CHARSET is set (e.g. to utf8) for table in ext_tables.sql

CREATE TABLE tx_myext_domain_model_xyz (
  uid int(11) unsigned NOT NULL auto_increment,
  [...]
) ENGINE=MyISAM CHARSET=utf8;

it is ignored while installing the extension in EM. A check with phpmyadmin shows that the collation is set to latin1_swedisch_ci instead.

I track this issue and found out in

namespace TYPO3\CMS\Install\Sql;
SchemaMigrator::getUpdateSuggestions()

that COLLATION is not yet supported.

// Skip these special statements. TODO: collation support is currently disabled (needs more testing)

Case II

But when CHARSET is set for the same table in ext_tables_static+adt.sql (no need to insert static data),

DROP TABLE IF EXISTS tx_myext_domain_model_xyz;
CREATE TABLE tx_myext_domain_model_xyz (
  uid int(11) unsigned NOT NULL auto_increment,
  [...]
) ENGINE=MyISAM CHARSET=utf8;

then the COLLATION is set correctly to utf8_general_ci in DB.

For utf8 collation for use with TYPO3 CMS 6, are there drawbacks to only use ext_tables_static+adt.sql to hold tables ATM.


Related issues

Related to TYPO3 Core - Bug #43027: Problems on case-sensitive filesystemsClosedAndreas Wolf2012-11-16

Actions
Related to TYPO3 Core - Task #46553: Resolve issues with case-sensitive filesClosed2013-03-22

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

Actions
#1

Updated by Mathias Schreiber over 7 years ago

  • Priority changed from -- undefined -- to Should have
  • Target version set to 7.4 (Backend)
#2

Updated by Susanne Moog almost 7 years ago

  • Target version changed from 7.4 (Backend) to 7.5
#3

Updated by Benni Mack almost 7 years ago

  • Target version deleted (7.5)
#4

Updated by Morton Jonuschat about 6 years ago

  • Category changed from Extension Manager to Database API (Doctrine DBAL)
  • Assignee set to Morton Jonuschat
  • Target version set to Candidate for patchlevel
#5

Updated by Morton Jonuschat almost 6 years ago

  • Target version changed from Candidate for patchlevel to 8.4
#6

Updated by Gerrit Code Review almost 6 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

#7

Updated by Gerrit Code Review almost 6 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

#8

Updated by Gerrit Code Review almost 6 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

#9

Updated by Gerrit Code Review almost 6 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

#10

Updated by Morton Jonuschat almost 6 years ago

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

Updated by Riccardo De Contardi over 4 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF