I am getting an error when trying to enter an email address for notification in the module configuration:

Home › Administer › Site configuration ›
Admin Notification
warning: preg_match() expects parameter 2 to be string, array given in /home/moving/public_html/includes/bootstrap.inc on line 679.

Any help would be appreciated,

Chris

CommentFileSizeAuthor
#6 admin_notify.zip1.44 KBvolkiz

Comments

FreeFox’s picture

FreeFox commented

Yep, i confirm, same error :(

whathits’s picture

whathits commented
Priority: Normal » Critical
volkiz’s picture

volkiz commented

change line 39 :
'#default_value' => variable_get('admin_notify_node_types', $admin_mail),

with this :
'#default_value' => $admin_mail,

volkiz’s picture

volkiz commented

and line 121:
$admin_email = variable_get('email', 'admin@contrastsofnewzealand.co.nz');

to

$admin_email = variable_get('admin_notify_email', '');

SO THIS MODULE CAN ACTUALLY WORK !

mpeal’s picture

mpeal commented

I got the same error message as Chris (whathits), and the two edits by volkiz removed the error.
One problem remains: the email address field keeps getting filled with the default email address set in Site Information, and attempts to change it in admin_notify are ignored.

volkiz’s picture

volkiz commented
StatusFileSize
new admin_notify.zip1.44 KB

yes sorry , i forgot one thing to update
change line #34
$admin_mail = variable_get('site_mail', ini_get('sendmail_from'));

to:
$admin_mail = variable_get('admin_notify_email','');

fully patched module attached on this comment.