@sergeytolkachyov Could not say that I understand these container stuffs, however, base on what I see from core code and by reading comments from maintainers on other similar PRs in the past, you should use Factory::getContainer()->get(DatabaseInterface::class) instead of using Factory::getContainer()->get('DatabaseDriver') as how you are doing here. See https://github.com/joomla/joomla-cms/blob/4.2-dev/modules/mod_finder/src/Helper/FinderHelper.php#L82 for an example

@sergeytolkachyov Could not say that I understand these container stuffs, however, base on what I see from core code and by reading comments from maintainers on other similar PRs in the past, you should use Factory::getContainer()->get(DatabaseInterface::class) instead of using Factory::getContainer()->get('DatabaseDriver') as how you are doing here. See https://github.com/joomla/joomla-cms/blob/4.2-dev/modules/mod_finder/src/Helper/FinderHelper.php#L82 for an example

Thank you, I changed it

I have tested this item ✅ successfully on d9dbca3

Both code review and run some code to test the methods which are affected by this change.

If someone wants to do real test, add this line of code below to index.php file of your template, then access to backend and see data for the phpass lib displayed:

$lib = \Joomla\CMS\Helper\LibraryHelper::getLibrary('phpass');
var_dump($lib);

I also tested saveParams method, too. So maybe you just need to do a code review should be enough.

I have tested this item ✅ successfully on d9dbca3

RTC this time with the issue tracker.

Thanks!