The cookie that gets saved for anonymous users to remember name and homepage should only get set for anonymous users, not everyone.

CommentFileSizeAuthor
comment-anonymous-cookie.patch1.06 KBrobertDouglass
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Stefan Nagtegaal’s picture

Stefan Nagtegaal CreditAttribution: Stefan Nagtegaal commented
Status: Needs review » Reviewed & tested by the community

Simple and straightforward patch, works as expected here..

While this is pretty simple I dare to set this RTBC.

mfer’s picture

mfer CreditAttribution: mfer commented

nice catch. looks good to me, too. should be RTBC.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)
Sepero’s picture

Sepero CreditAttribution: Sepero commented
Version: 6.x-dev » 6.9
Status: Closed (fixed) » Active

Does this patch work with the latest D6 also? Whenever anonymous users are posting to my site, they have to re-enter their Name on every comment.

mfer’s picture

mfer CreditAttribution: mfer commented
Status: Active » Closed (fixed)

This is already incorporated into Drupal 6. It is done via JavaScript. If it is not working on your site something is causing it to not work. The feature is there.