From 374d44a1ef7f9daffe0d461d21b80fabeda2b967 Mon Sep 17 00:00:00 2001
From: Anand Doshi <anand@erpnext.com>
Date: Tue, 23 Jul 2013 11:48:25 +0530
Subject: [PATCH] [minor] [fix] clear value and run trigger of attach_files:
 field on removing attachment, if that attachment is being used by a given
 field

---
 public/js/wn/form/attachments.js | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/public/js/wn/form/attachments.js b/public/js/wn/form/attachments.js
index 9ae34e9e21..7985fc2e7c 100644
--- a/public/js/wn/form/attachments.js
+++ b/public/js/wn/form/attachments.js
@@ -182,11 +182,14 @@ wn.ui.form.Attachments = Class.extend({
 		for(var i=0; i<this.frm.fields.length; i++) {
 			if(this.frm.fields[i].df.options=="attach_files:") {
 				var fieldname = this.frm.fields[i].df.fieldname;
-				refresh_field(fieldname);
-				if(this.frm.doc[fieldname]!=undefined && !inList(this.frm.fields[i].df.options.split("\n"), this.frm.doc[fieldname])) {
-					this.frm.cscript.on_remove_attachment && this.frm.cscript.on_remove_attachment(this.frm.doc);
+				var selected_option = this.frm.fields[i].$input.find("option:selected").val();
+				
+				if(this.frm.doc[fieldname]!=null && selected_option!==this.frm.doc[fieldname]) {
+					this.frm.script_manager.trigger(fieldname);
 					this.frm.set_value(fieldname, "");
 				}
+				
+				this.frm.fields[i].refresh();
 			}
 		}
 	}