Project

General

Profile

Actions

Bug #89438

closed

"Upgrade Wizard" - "Database Analyzer" - Loop

Added by Sven Wappler over 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Install Tool
Target version:
-
Start date:
2019-10-17
Due date:
% Done:

0%

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

Description

Currently, the Upgrade Wizard is not compatible with the Database Analyzer. The reason for this is the \TYPO3\CMS\Install\Updates\ExtensionManagerTables Task. It checks all tables with the SchemaMigrator and requires a generated database schema out of the TCA model.

$defaultTcaSchema = GeneralUtility::makeInstance(DefaultTcaSchema::class);
$tables = $defaultTcaSchema->enrich($tables);

Many extension authors do not use unsigned in their sql files for int fields. This leads to a very large number of fields being changed by ExtensionManagerTables, although it should only change 2 tables.
Using unsigned int field seems to be the new standard.

If you execute the Database Analyzer after this task, these fields are displayed incorrectly and it is suggested to restore them to their original non-unsigned state. This is because the Database Analyzer uses the sql files of the extensions as a basis before comparing them with the SchemaMigrator. An "enrich" is then no longer necessary.

Possible solution:
- Dissuade the ExtensionManagerTables Task from checking other tables. Because it should not be responsible for other modifications.

Then the system can regain 100% integrity.


Files

loop.png (209 KB) loop.png Update Loop Sven Wappler, 2019-10-17 13:22

Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #91665: Upgrade wizard "ExtensionManagerTables" should not declare itself neccesary in composer and/or offline modeClosed2020-06-17

Actions
Related to TYPO3 Core - Feature #92457: Simplify TER connectionClosedBenni Mack2020-10-01

Actions
Actions

Also available in: Atom PDF