chg: [audit log naming] renamed action to request_action to avoid reserved keyword usage
parent
7b52d29320
commit
fe8e217d61
|
@ -106,7 +106,7 @@ class AuditLogBehavior extends Behavior
|
||||||
$modelTitle = $this->old[$modelTitleField];
|
$modelTitle = $this->old[$modelTitleField];
|
||||||
}
|
}
|
||||||
$this->auditLogs()->insert([
|
$this->auditLogs()->insert([
|
||||||
'action' => $action,
|
'request_action' => $action,
|
||||||
'model' => $entity->getSource(),
|
'model' => $entity->getSource(),
|
||||||
'model_id' => $id,
|
'model_id' => $id,
|
||||||
'model_title' => $modelTitle,
|
'model_title' => $modelTitle,
|
||||||
|
@ -130,7 +130,7 @@ class AuditLogBehavior extends Behavior
|
||||||
}
|
}
|
||||||
|
|
||||||
$logEntity = $this->auditLogs()->newEntity([
|
$logEntity = $this->auditLogs()->newEntity([
|
||||||
'action' => $entity->getConstant('ACTION_DELETE'),
|
'request_action' => $entity->getConstant('ACTION_DELETE'),
|
||||||
'model' => $entity->getSource(),
|
'model' => $entity->getSource(),
|
||||||
'model_id' => $this->old->id,
|
'model_id' => $this->old->id,
|
||||||
'model_title' => $modelTitle,
|
'model_title' => $modelTitle,
|
||||||
|
|
|
@ -47,9 +47,9 @@ class AuditLog extends AppModel
|
||||||
*/
|
*/
|
||||||
public function generateUserFriendlyTitle($auditLog)
|
public function generateUserFriendlyTitle($auditLog)
|
||||||
{
|
{
|
||||||
if (in_array($auditLog['action'], [self::ACTION_TAG, self::ACTION_TAG_LOCAL, self::ACTION_REMOVE_TAG, self::ACTION_REMOVE_TAG_LOCAL], true)) {
|
if (in_array($auditLog['request_action'], [self::ACTION_TAG, self::ACTION_TAG_LOCAL, self::ACTION_REMOVE_TAG, self::ACTION_REMOVE_TAG_LOCAL], true)) {
|
||||||
$attached = ($auditLog['action'] === self::ACTION_TAG || $auditLog['action'] === self::ACTION_TAG_LOCAL);
|
$attached = ($auditLog['request_action'] === self::ACTION_TAG || $auditLog['request_action'] === self::ACTION_TAG_LOCAL);
|
||||||
$local = ($auditLog['action'] === self::ACTION_TAG_LOCAL || $auditLog['action'] === self::ACTION_REMOVE_TAG_LOCAL) ? __('local') : __('global');
|
$local = ($auditLog['request_action'] === self::ACTION_TAG_LOCAL || $auditLog['request_action'] === self::ACTION_REMOVE_TAG_LOCAL) ? __('local') : __('global');
|
||||||
if ($attached) {
|
if ($attached) {
|
||||||
return __('Attached %s tag "%s" to %s #%s', $local, $auditLog['model_title'], strtolower($auditLog['model']), $auditLog['model_id']);
|
return __('Attached %s tag "%s" to %s #%s', $local, $auditLog['model_title'], strtolower($auditLog['model']), $auditLog['model_id']);
|
||||||
} else {
|
} else {
|
||||||
|
|
Loading…
Reference in New Issue