User tests: Successful: Unsuccessful:
Status | New | ⇒ | Pending |
Category | ⇒ | Administration |
Labels |
Added:
?
|
I have tested this item
Perhaps move this to the plugin? And add component check?
@alikon
@SharkyKZ is right. This should depend on the plugin.
I propose this as a solution.
diff --git a/plugins/actionlog/joomla/joomla.php b/plugins/actionlog/joomla/joomla.php
index 0fb7e27..2d06e50 100644
--- a/plugins/actionlog/joomla/joomla.php
+++ b/plugins/actionlog/joomla/joomla.php
@@ -78,5 +78,12 @@
}
- list(, $contentType) = explode('.', $params->type_alias);
+ if ($option === 'com_content')
+ {
+ $contentType = 'article';
+ }
+ else
+ {
+ list(, $contentType) = explode('.', $params->type_alias);
+ }
if ($isNew)
@@ -191,5 +198,12 @@
}
- list(, $contentType) = explode('.', $params->type_alias);
+ if ($option === 'com_content')
+ {
+ $contentType = 'article';
+ }
+ else
+ {
+ list(, $contentType) = explode('.', $params->type_alias);
+ }
switch ($value)
@@ -470,4 +484,9 @@
$option = $this->app->input->getCmd('option');
+ if ($table->get('module') != null)
+ {
+ $option = 'com_modules';
+ }
+
if (!$this->checkLoggable($option))
{
Oh, I see it contains the check for com_modules which is already taken care in another PR.
I will merge that one now.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-10-09 15:23:07 |
Closed_By | ⇒ | Quy |
Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/22554
Closed_Date | 2018-10-09 15:23:07 | ⇒ | 2018-10-09 15:23:08 |
Closed_By | Quy | ⇒ | joomla-cms-bot |
3 lines of code 2 @Quy yellow card collected?
?