Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LibraryRegistry doesn't support DI #15

Closed
dkd-friedrich opened this issue Oct 29, 2021 · 1 comment
Closed

LibraryRegistry doesn't support DI #15

dkd-friedrich opened this issue Oct 29, 2021 · 1 comment

Comments

@dkd-friedrich
Copy link

Version(s) affected:

1.1.0, TYPO3 10.4

Description

To display a list of registered chart libraries in the extensions settings, LibraryRegistry is loading ext_localconf.php of all loaded extensions. Unfortunately by just loading the configurations via ExtensionManagementUtility the dependency injection (DI) is not supported and if classes used in ext_localconf.php depend on that, an ArgumentCountError occurs in the extension configuration module.

How to reproduce

Prepare a TYPO3 10.4 installation and install charts 1.1.0 and evoweb/extender 7.x, then try to open extension configuration module.

Possible Solution

Adapt required bootstrapping in LibraryRegistry.

Additional context

Details about the exception to expect when trying to reproduce:

(1/1) ArgumentCountError

Too few arguments to function Evoweb\Extender\Utility\ClassLoader::__construct(), 0 passed in /var/www/html/public/typo3/sysext/core/Classes/Utility/GeneralUtility.php on line 3481 and exactly 2 expected
This was referenced Jan 15, 2022
@hoogi91
Copy link
Owner

hoogi91 commented Jan 15, 2022

Hey @dkd-friedrich, the problem should be resolved with release 1.1.1

@hoogi91 hoogi91 closed this as completed Jan 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants