Project

General

Profile

Actions

Bug #14676

closed

MySQL table type hardcoded to MyISAM

Added by Karsten Dambekalns over 19 years ago. Updated over 18 years ago.

Status:
Closed
Priority:
Should have
Category:
Install Tool
Target version:
-
Start date:
2005-04-17
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
3.8.0rc1
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

The table type for tables created through TYPO3 is hardcoded to MyISAM. This makes it impossible to set the table type to a different one by changing the default type in MySQL or in the ext_tables.sql file.

Proposed change: Remove the type unless it is given in the ext_tables.sql file (or whatever source for the table definition is used). This way it can be overridden when needed. Additionally nothing will change for the majority of setups, as MyISAM is the default table type in MySQL anyway.

Has to be changed in getUpdateSuggestions() in t3lib/class.t3lib_install.php.
(issue imported from #M987)


Files

t3lib_install.php.diff (1.55 KB) t3lib_install.php.diff Administrator Admin, 2005-05-17 12:27

Related issues 1 (0 open1 closed)

Has duplicate TYPO3 Core - Feature #14334: Extension Manager should allow MySQL tables with type InnoDBClosedKarsten Dambekalns2004-09-21

Actions
Actions #1

Updated by Karsten Dambekalns over 19 years ago

Could you think about this and give me some feedback? if you think this needs a lot more testing, I'll wait until after 3.8.0, otherwise I'll work on this before.

Actions #2

Updated by Robert Lemke over 19 years ago

I don't have a opinion on that, for me it seems like a change that doesn't hurt, but maybe I'm wrong ...

Actions #3

Updated by Michael Stucki over 19 years ago

Yes that should be changed. I just checked the MySQL reference for that:
http://dev.mysql.com/doc/mysql/en/storage-engines.html

Yes MyISAM is the default.

Actions #4

Updated by Karsten Dambekalns over 19 years ago

The attached diff parses the ext_tables.sql for a table type definition and uses it if present. If not present, no type is forced at all, thus allowing the DB admin to set his default table type to his liking.

It works here, but please have a look as well. Thanks!

Actions #5

Updated by Michael Stucki over 19 years ago

Looks good to me.

Actions #6

Updated by Karsten Dambekalns over 19 years ago

Applied the changes to CVS.

Actions

Also available in: Atom PDF