A very simple patch to fix this.

CommentFileSizeAuthor
#3 816246-D6.patch1.68 KBamateescu
token.patch774 bytesmoritzz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

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

Dave Reid’s picture

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

Status: Needs work » Needs review
FileSize
1.68 KB

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