Fix: show message if contact doesn't have email if notification control is enabled

This commit is contained in:
Anand Doshi 2012-01-24 17:08:53 +05:30
parent f51e52c1be
commit d194e6b5cf

View File

@ -10,23 +10,22 @@ cur_frm.cscript.select_transaction = function(doc, dt, dn) {
} }
cur_frm.cscript.notify = function(doc, args) { cur_frm.cscript.notify = function(doc, args) {
if(validate_email(args['send_to'] || doc.contact_email || '')) { $c_obj('Notification Control', 'get_formatted_message', {
$c_obj('Notification Control', 'get_formatted_message', { type: args['type'],
type: args['type'], doctype: args['doctype'],
doctype: args['doctype'], contact_name: args['contact_name'] || doc.contact_display
contact_name: args['contact_name'] || doc.contact_display }, function(r, rt) {
}, function(r, rt) { if(!r.exc) {
if(!r.exc) { var res = JSON.parse(r.message);
var res = JSON.parse(r.message); var send_from = (function() {
var send_from = (function() { if(user!='Administrator') {
if(user!='Administrator') { return user;
return user; } else {
} else { return (wn.control_panel.auto_email_id || 'automail@erpnext.com');
var cp = locals['Control Panel']['Control Panel']; }
return (cp.auto_email_id || 'automail@erpnext.com'); })();
} if(res.send) {
})(); if(validate_email(args['send_to'] || doc.contact_email || '')) {
if(res.send) {
var print_heading = (doc.select_print_heading || args['type']) var print_heading = (doc.select_print_heading || args['type'])
sendmail( sendmail(
args['send_to'] || doc.contact_email, args['send_to'] || doc.contact_email,
@ -38,9 +37,11 @@ cur_frm.cscript.notify = function(doc, args) {
); );
msgprint('This ' + print_heading + ' is being sent to <b>' msgprint('This ' + print_heading + ' is being sent to <b>'
+ (args['send_to'] || doc.contact_email) + '</b><br />...'); + (args['send_to'] || doc.contact_email) + '</b><br />...');
} else {
msgprint('Invalid/Missing Email Address. ' + print_heading + ' not sent.');
} }
} }
//console.log(JSON.parse(r.message)); }
}); //console.log(JSON.parse(r.message));
} });
} }