Project

General

Profile

Actions

Bug #49758

closed

checkTheDatabase hook missing in refactored Install-Tool

Added by Andreas Kiessling almost 11 years ago. Updated almost 10 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Install Tool
Target version:
Start date:
2013-07-07
Due date:
% Done:

0%

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

Description

I just updated one of my dev setups to current master and one extension that used the checkTheDatabase hook to dynamically create tables and fields is throwing an exception about the missing interface \TYPO3\CMS\Install\CheckTheDatabaseHookInterface. Up to 6.1, \TYPO3\CMS\Install\Installer evaluated

$GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['ext/install/mod/class.tx_install.php']['checkTheDatabase']

but this code was apparently not migrated in https://review.typo3.org/#/c/21468/.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #49162: Rewrite install toolClosed2011-10-30

Actions
Actions #1

Updated by Christian Kuhn over 10 years ago

  • Is Regression set to No

Yeah, the hook should probably be re-implemented.

I think there was also some interface for the hook and this will be different from the previous version because it got tx_install as one parameter, and this class does not exist anymore ... this is a breaking change and we can not circumvent this.

Actions #2

Updated by Andreas Kiessling about 10 years ago

Seems like this can now be done using the signal "tablesDefinitionIsBeingBuilt" to generate the sql setup and the hook "extTablesInclusion-PostProcessing" to set up TCA. Will try to convert as it seems, that the hook won't reappear in 6.2
Thanks to Thomas Maroschik for the pointer!

Actions #3

Updated by Wouter Wolters almost 10 years ago

  • Status changed from New to Closed

New way to handle this! See comment above

Actions

Also available in: Atom PDF