Hello, Greg.

I found that recent update of my module AJAX Comments affects on Comment Notify functioning. The reason is that AJAX Comments modifies some of the comment form elements to allow safe and fast ajax submissions. One of this elements is 'nid', which converted from "value" to "hidden" FAPI element. Comment Notify threats 'nid' value only as "value" type and can't grab 'nid' parameter from the form, which completely disable comment notifications.

The solution is to check either "#value" is accessible, and if it's not, try to grab "#default_value" from the 'nid' element.

Please, take a look at the patch. Thanks.

CommentFileSizeAuthor
cn_ac.patch578 bytesneochief
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

greggles’s picture

Status: Active » Fixed

And done - http://drupal.org/cvs?commit=285288

Thanks, neochief!

Status: Fixed » Closed (fixed)

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