I found a bug in the code:

In the comment_notify_inc line 25 the code should be:
$users[0]->comment_notify = comment_notify_variable_registry_get('default_anon_mailalert');

Otherwise the default selection for anonymous user don't work fine.

CommentFileSizeAuthor
#1 comment_notify-anonymous-defaults-2403491-2.patch642 bytesjamesoakley

Comments

jamesoakley’s picture

jamesoakley
Primary language English
Location North Yorkshire, UK
commented
Version: 7.x-1.2 » 7.x-1.x-dev
Status: Active » Reviewed & tested by the community
StatusFileSize
new comment_notify-anonymous-defaults-2403491-2.patch642 bytes

Confirmed that default selection for anonymous users does not work out of the box, but that the change proposed by @sadelird fixes it

Patch attached

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Reviewed & tested by the community » Fixed

Thanks for the bug report and for turning it into a patch!

Now applied.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.