[#9742] - [com_content] multiple databases config mapping invalid for JTable::getInstance
- Closed
- 5 Aug 2016
- Medium
- Build: master
- # 9742
Steps to reproduce the issue
- Connect to another database $db = JDatabaseDriver::getInstance($databaseOptions);
- use this $db object to get a JTable instance $contentTable = JTable::getInstance('content', 'JTable', array('dbo' => $db));
- JTable::getInstance will not use $db as specified, but default database
Expected result
Use specified database
Actual result
JTable Uses same database as in /configuration.php
FIX
look for JTableContent (libraries/legacy/table/content.php) line
// Verify that the alias is unique
//$table = JTable::getInstance('Content', 'JTable', array('dbo',$this->getDbo())); //incorrect
$table = JTable::getInstance('Content', 'JTable', array('dbo'=>$this->getDbo())); //correct
Do a search for 'dbo',$this->getDbo() to find possibly more occurences of this bug which are JTableMenuType,JTableNested, JTableLanguage
System information (as much as possible)
Joomla 3.5
Additional comments
excerpt from JTable::getInstance:
$db = isset($config['dbo']) ? $config['dbo'] : JFactory::getDbo(); //assoziative array syntax
bug is obvious, isn't it?
| Labels |
Removed:
?
|
||
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Libraries |
| Status | New | ⇒ | Confirmed |
Confirmed. Can you provide a pull request?
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9742.
done, thank you, see #10369
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9742.
| Status | Confirmed | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-08-05 13:33:45 |
| Closed_By | ⇒ | brianteeman |
Closed as we had a PR and that has been merged.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9742.
Can anyone comment this really is a bug or a misunderstanding by myself? I need to inform my customers if this will be fixed in any upcoming release. Thanks