fix: reload Diagnostic Report after workflow action to resolve unsaved state

Author: Irfan778899

healthcare/healthcare/doctype/diagnostic_report/diagnostic_report.js

@@ -12,7 +12,7 @@ frappe.ui.form.on("Diagnostic Report", {
 	},
 	before_save: function (frm) {
 		if (!frm.is_new() && frm.is_dirty()) {
-			this.diagnostic_report.save_action("save");
+			return this.diagnostic_report.save_action("save");
 		}
 	},
 	after_workflow_action: function (frm) {
@@ -21,6 +21,9 @@ frappe.ui.form.on("Diagnostic Report", {
 			args: {
 				docname: frm.doc.name,
 			},
+			callback: function () {
+				frm.reload_doc();
+			},
 		});
 	},
 });
@@ -89,18 +92,18 @@ var generate_pdf_with_print_format = function (frm) {
 
 		const w = window.open(
 			"/api/method/frappe.utils.print_format.download_multi_pdf?" +
-				"doctype=" +
-				encodeURIComponent(frm.doc.doctype) +
-				"&name=" +
-				encodeURIComponent(doc_names) +
-				"&format=" +
-				encodeURIComponent(print_format) +
-				"&no_letterhead=" +
-				(with_letterhead ? "0" : "1") +
-				"&letterhead=" +
-				encodeURIComponent(letterhead) +
-				"&options=" +
-				encodeURIComponent(pdf_options),
+			"doctype=" +
+			encodeURIComponent(frm.doc.doctype) +
+			"&name=" +
+			encodeURIComponent(doc_names) +
+			"&format=" +
+			encodeURIComponent(print_format) +
+			"&no_letterhead=" +
+			(with_letterhead ? "0" : "1") +
+			"&letterhead=" +
+			encodeURIComponent(letterhead) +
+			"&options=" +
+			encodeURIComponent(pdf_options),
 		);
 
 		if (!w) {

healthcare/public/js/diagnostic_report_controller.js

@@ -69,7 +69,7 @@ healthcare.Diagnostic.DiagnosticReport = class DiagnosticReport {
 	save_action(func) {
 		var me = this;
 		if (func == "save") {
-			frappe.call({
+			return frappe.call({
 				method: "healthcare.healthcare.doctype.observation.observation.record_observation_result",
 				args: {
 					values: this.result,

healthcare/public/js/observation_widget.js

@@ -207,8 +207,11 @@ healthcare.ObservationWidget = class {
 					options: options,
 					read_only: 1 ? obs_data.status == "Approved" : 0,
 					change: s => {
-						me.frm.dirty();
-						me.set_result_n_name(obs_data.name);
+						let current_val = me[obs_data.name].get_value("result");
+						if ((current_val || "") != (default_input || "")) {
+							me.frm.dirty();
+							me.set_result_n_name(obs_data.name);
+						}
 					},
 					default: default_input,
 					hidden: 1 ? obs_data.observation_category == "Imaging" : 0,