[5.2] Refactored JDatabaseDriver in some params

[pe7er]

Refactored JDatabaseDriver to Joomla\Database\DatabaseDriver

Summary of Changes

Refactored some params to use Joomla\Database\DatabaseDriver instead

Testing Instructions

Code review

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

[Hackwar]

Are you sure that it should be DatabaseDriver and not DatabaseDriverInterface?

[richard67]

I have tested this item ✅ successfully on c855172

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44750.}

[richard67]

I have not tested this item.

Reverting my test result as I hadn't seen @Hackwar 's comment.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44750.}

[pe7er]

Are you sure that it should be DatabaseDriver and not DatabaseDriverInterface?

No, I'm not. I chose to replace JDatabaseDriver with DatabaseDriver because
plugins/behaviour/compat/src/classmap/classmap.php states

JLoader::registerAlias('JDatabaseDriver', '\Joomla\Database\DatabaseDriver', '6.0');

If it should be DatabaseDriverInterface, could you please correct it?

[laoneo]

Yes please change it to the interface as the model doesn't get the concrete class injected.

[pe7er]

Yes please change it to the interface as the model doesn't get the concrete class injected.

Thanks!

I've refactored \JDatabaseDriver to DatabaseInterface