A very simple patch to fix this.

CommentFileSizeAuthor
#3 816246-D6.patch1.68 KBamateescu
token.patch774 bytesmoritzz

Comments

Status: Needs review » Needs work

The last submitted patch, token.patch, failed testing.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Version: 6.x-1.12 » 6.x-1.x-dev

Anonymous users can fill out their own 'name' information which is stored there, so this would overwrite those values. It should probably check if $comment->name itself is empty, not $comment->uid. This will need to be re-patched against the latest code, 6.x-1.x and the token comment test adjusted.

amateescu’s picture

amateescu commented
Status: Needs work » Needs review
StatusFileSize
new 816246-D6.patch1.68 KB

I encountered this bug today so here's a fixed and updated patch.

bluegeek9’s picture

bluegeek9 commented
Issue summary: View changes
Status: Needs review » Closed (outdated)