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

fix: Email Dialog use_html field hidden after re-opening

Browse source
This commit is contained in:
Alex Leach 2025-11-26 16:22:43 +00:00
parent 9b75fa5487
commit a1cb7430e8
No known key found for this signature in database
GPG key ID: CBB1F1542760286C
1 changed files with 16 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -108,27 +108,12 @@ frappe.views.CommunicationComposer = class {
fieldtype: "Link",
options: "Email Template",
fieldname: "email_template",
onchange: function () {
onchange: async function () {
const email_template = this.value;
if (!email_template) {
return me.hide_use_html_field();
}
frappe.db
.get_value("Email Template", email_template, "use_html")
.then((r) => {
// Show or hide "Use HTML" based on the Email Template's use_html value
if (r.message?.use_html === 1) {
// Show the field.
me.dialog.fields_dict.use_html.toggle(true);
} else {
me.hide_use_html_field();
}
})
.catch((e) => {
console.error("Failed to load template", e);
me.hide_use_html_field();
});
await me.check_email_template_html(email_template);
},
},
{
@ -143,6 +128,7 @@ frappe.views.CommunicationComposer = class {
default: 0,
hidden: 1,
onchange: function (e) {
if (!e) return;
if (e.target.checked) {
me.dialog.set_value("html_content", me.dialog.get_value("content"));
} else {
@ -299,6 +285,17 @@ frappe.views.CommunicationComposer = class {
this.dialog.set_value("print_language", lang);
}
async check_email_template_html(email_template) {
const r = await frappe.db.get_value("Email Template", email_template, "use_html");
// Show or hide "Use HTML" based on the Email Template's use_html value
if (r.message?.use_html === 1) {
// Show the field.
this.dialog.fields_dict.use_html.toggle(true);
} else {
this.hide_use_html_field();
}
}
hide_use_html_field() {
this.dialog.fields_dict.use_html.set_input(false); // reset the value
this.dialog.fields_dict.use_html.toggle(false); // hide the field
@ -469,7 +466,7 @@ frappe.views.CommunicationComposer = class {
let content = content_field.get_value() || "";
content_field.set_value(`${reply.message}<br>${content}`);
content_field.set_value(reply.message + content);
subject_field.set_value(reply.subject);
}
@ -578,6 +575,7 @@ frappe.views.CommunicationComposer = class {
if (last_edited.email_template) {
const template_field = this.dialog.fields_dict.email_template;
await template_field.set_model_value(last_edited.email_template);
await this.check_email_template_html(last_edited.email_template);
delete last_edited.email_template;
}