Feeling Lucky
PR-5.2-dev
[#43517] - [5.2] Replacing getDbo() with call to container
- Pending
- Medium
- Build: 5.2-dev
- # 43517
- Diff
- Hackwar:5.2-database-container
User tests: Successful: Unsuccessful:
Summary of Changes
The call to Factory::getDbo() is deprecated. This PR replaces that with calls to the DI container.
Testing Instructions
Codereview
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_admin com_associations com_banners com_categories com_config com_contact com_content com_contenthistory com_fields com_finder com_installer com_languages com_login com_menus com_modules com_newsfeeds com_plugins com_redirect com_templates |
| Labels |
Added:
PR-5.2-dev
|
||
@Hackwar I think unit tests are failing at places where the isMariaDb method of the MySQL drivers are used because this method is not part of the database interface.
Update: It seems unit tests are failing for a different reason. The
isMariaDbmethod we use only in fileinstallation/src/Helper/DatabaseHelper.php.