Posted by whathits on January 27, 2009 at 6:46am
| Project: | Admin Notify |
| Version: | 5.x-1.1 |
| Component: | User interface |
| Category: | support request |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | active |
Issue Summary
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
Comments
#1
Yep, i confirm, same error :(
#2
#3
change line 39 :
'#default_value' => variable_get('admin_notify_node_types', $admin_mail),
with this :
'#default_value' => $admin_mail,
#4
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 !
#5
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.
#6
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.