Bug #49758
checkTheDatabase hook missing in refactored Install-Tool
| Status: | Closed | Start date: | 2013-07-07 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assigned To: | - | % Done: | 0% |
|
| Category: | Install Tool | Spent time: | - | |
| Target version: | 6.2.0 | |||
| TYPO3 Version: | 6.2 | Is Regression: | No | |
| PHP Version: | Sprint Focus: | |||
| Complexity: |
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
History
#1 Updated by Christian Kuhn almost 2 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.
#2 Updated by Andreas Kiessling over 1 year 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!
#3 Updated by Wouter Wolters about 1 year ago
- Status changed from New to Closed
New way to handle this! See comment above