Merge pull request #16787 from uberbrady/suppress_action_date_error_in_loggable

Fixed: [RB-19645] Suppress error message about 'action_date' not existing
This commit is contained in:
snipe 2025-04-22 14:48:38 +01:00 committed by GitHub
commit ed86c90b7e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -100,13 +100,15 @@ trait Loggable
foreach ($originalValues as $key => $value) {
// TODO - action_date isn't a valid attribute of any first-class object, so we might want to remove this?
if ($key == 'action_date' && $value != $action_date) {
$changed[$key]['old'] = $value;
$changed[$key]['new'] = is_string($action_date) ? $action_date : $action_date->format('Y-m-d H:i:s');
} elseif ($value != $this->getAttributes()[$key]) {
} elseif (array_key_exists($key, $this->getAttributes()) && $value != $this->getAttributes()[$key]) {
$changed[$key]['old'] = $value;
$changed[$key]['new'] = $this->getAttributes()[$key];
}
// NOTE - if the attribute exists in $originalValues, but *not* in ->getAttributes(), it isn't added to $changed
}
if (!empty($changed)){