LotusGuild/tinker_tickets
4
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Include ticket_id in comment-edit audit log so it appears on the timeline (#87)

Browse Source 
AuditLogModel::getTicketTimeline() requires, for entity_type='comment'
rows, that details.ticket_id match the ticket being viewed.
logCommentCreate() and delete-comment's audit call both correctly
include it; update_comment.php's audit call only set
comment_text_preview, so an edited comment's audit row was written
(visible in the admin's global Audit Log) but never matched the
timeline's join condition — a comment edit left no trace on the
ticket's own history, while deleting the same comment would be
visible.

Added ticket_id to the details array, using $comment['ticket_id']
already loaded earlier in the file for the access check. Verified
against real MariaDB: the fixed shape now correctly appears in
getTicketTimeline()'s results.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Lhz7pGMaoTfL5sdYS5XiKv
This commit is contained in:
jared
2026-09-11 14:05:01 -04:00
co-authored by Claude Sonnet 5
parent 9702aafacd
commit 09cea2b388
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
api/update_comment.php
+4 -1
View File
@@ -104,7 +104,10 @@ try {
'update', 'update',
'comment', 'comment',
(string)$commentId, (string)$commentId,
['comment_text_preview' => substr($commentText, 0, 100)] [
'ticket_id' => $comment['ticket_id'] ?? null,
'comment_text_preview' => substr($commentText, 0, 100),
]
); );
} }