[#7696] - JModelAdmin class - method publish -
- Closed
- 16 Aug 2015
- Medium
- Build: master
- # 7696
Hi,
Test environment: Joomla! 3.4.3 (13.1.0), PHP 5.5.6, MySQL 5.6.14
JModelAdmin class - method publish() includes the following code:
JPluginHelper::importPlugin($this->events_map['change_state']);( libraries\legacy\model\admin.php )
which then calls this event: PlgContentJoomla class method onContentChangeState
( plugins\content\joomla\joomla.php )
which throws an error for 3pd extensions which do not use Joomla!'s categories table:
Error:
Unknown column 'a.state' in 'field list' SQL=SELECT a.id,a.state AS state, c.published AS cat_state,a.access, c.access AS cat_access FROM jos_3pd_extension AS a LEFT JOIN jos_categories AS c ON c.id = a.catid WHERE a.id = 3
I didn't find any option to disable running this event for 3pd extensions (maybe my knowledge on this event handler is not sufficient)
Thank you, Jan
zero-24
- | Category | ⇒ | Libraries |
| Labels |
Added:
?
|
||
The event "onContentChangeState" dont use context, triggering this event call all the method
onContentChangeState
EDIT: not found where a.state used... maybe another error from your plg
EDIT2: JTable take care about state fields: see
Hi, thank you for the info, yes, I see it now, the "onContentChangeState" does not call this SQL, it is caused by SmartSearch plugins ( FinderIndexerAdapter class category or item changestate method).
I will try to find what is wrong there (relationship between extension: component and its finder (smart search plugin) and component which does not use standard Joomla! category feature), unfortunately for now I cannot say what exactly is causing this error, so maybe this issue should be marked as unconfirmed.
Thank you, Jan
The smart search plugins should only trigger on their own extensions. So unless you use a context from another extensions, then they shouldn't do anything.
so maybe this issue should be marked as unconfirmed.
I'm just going to close it for now. You can always open a new one or request this one to be reopened.
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-08-16 18:24:06 |
| Closed_By | ⇒ | Bakual |
I don't see where that plugin method does run a query using "a.state". Are you sure this one is the source?
Also we use different state fields in core already. Eg
com_newsfeedusespublishedandcom_contentusesstate, so there should be a way to fix that somehow as it seems to work for both those components.