diff --git a/frappe/public/js/frappe/form/footer/timeline.js b/frappe/public/js/frappe/form/footer/timeline.js index eb9d35398f..6459a4b30a 100644 --- a/frappe/public/js/frappe/form/footer/timeline.js +++ b/frappe/public/js/frappe/form/footer/timeline.js @@ -562,33 +562,32 @@ frappe.ui.form.Timeline = class Timeline { build_version_comments(docinfo, out) { var me = this; docinfo.versions.forEach(function(version) { - if(!version.data) return; + if (!version.data) return; var data = JSON.parse(version.data); // comment - if(data.comment) { + if (data.comment) { out.push(me.get_version_comment(version, data.comment, data.comment_type)); return; } // value changed in parent - if(data.changed && data.changed.length) { - var parts = []; - data.changed.every(function(p) { - if(p[0]==='docstatus') { - if(p[2]==1) { + if (data.changed && data.changed.length) { + const parts = []; + data.changed.every(function (p) { + if (p[0] === 'docstatus') { + if (p[2] == 1) { out.push(me.get_version_comment(version, __('submitted this document'))); - } else if (p[2]==2) { + } else if (p[2] == 2) { out.push(me.get_version_comment(version, __('cancelled this document'))); } } else { - - var df = frappe.meta.get_docfield(me.frm.doctype, p[0], me.frm.docname); - - if(df && !df.hidden) { - var field_display_status = frappe.perm.get_field_display_status(df, null, + p = p.map(frappe.utils.escape_html); + const df = frappe.meta.get_docfield(me.frm.doctype, p[0], me.frm.docname); + if (df && !df.hidden) { + const field_display_status = frappe.perm.get_field_display_status(df, null, me.frm.perm); - if(field_display_status === 'Read' || field_display_status === 'Write') { + if (field_display_status === 'Read' || field_display_status === 'Write') { parts.push(__('{0} from {1} to {2}', [ __(df.label), (frappe.ellipsis(frappe.utils.html2text(p[1]), 40) || '""').bold(), @@ -599,9 +598,8 @@ frappe.ui.form.Timeline = class Timeline { } return parts.length < 3; }); - if(parts.length) { - parts = parts.map(frappe.utils.escape_html); - out.push(me.get_version_comment(version, __("changed value of {0}", [parts.join(', ').bold()]))); + if (parts.length) { + out.push(me.get_version_comment(version, __('changed value of {0}', [parts.join(', ')]))); } }