vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Merge pull request #1443 from revant/patch-1

Browse source 
Enter Recipients if communication_medium is Email
This commit is contained in:
Rushabh Mehta 2015-12-01 16:41:54 +05:30
commit d9bb0b3bd4
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
frappe/public/js/frappe/views/communication.js
View file

@ -51,7 +51,7 @@ frappe.views.CommunicationComposer = Class.extend({
var fields_before_cc = [
{fieldtype: "Section Break"},
{label:__("To"), fieldtype:"Data", reqd: 1, fieldname:"recipients"},
{label:__("To"), fieldtype:"Data", reqd: 0, fieldname:"recipients"},
{fieldtype: "Section Break", collapsible: 1, label: "CC & Standard Reply"},
{label:__("CC"), fieldtype:"Data", fieldname:"cc"},
];
@ -65,7 +65,7 @@ frappe.views.CommunicationComposer = Class.extend({
{fieldtype: "Section Break"},
{label:__("Message"), fieldtype:"Text Editor", reqd: 1,
fieldname:"content"},
{fieldtype: "Section Break"},
{fieldtype: "Section Break"},
{fieldtype: "Column Break"},
{label:__("Send As Email"), fieldtype:"Check",
fieldname:"send_email"},
@ -399,6 +399,11 @@ frappe.views.CommunicationComposer = Class.extend({
send_email: function(btn, form_values, selected_attachments, print_html, print_format) {
var me = this;
if((form_values.send_email || form_values.communication_medium === "Email") && !form_values.recipients){
msgprint(__("Enter Email Recipient(s)"));
return;
}
if(!form_values.attach_document_print) {
print_html = null;
print_format = null;