Merge pull request #3949 from tundebabzy/issue-erpnext-10403

Tags auto-deleting (frappe/erpnext#10403)

frappe/public/js/frappe/form/sidebar.js

@@ -87,7 +87,7 @@ frappe.ui.form.Sidebar = Class.extend({
 			parent: this.sidebar.find(".tag-area"),
 			frm: this.frm,
 			on_change: function(user_tags) {
-				me.frm.doc._user_tags = user_tags;
+				me.frm.doc._user_tags += ("," + user_tags);
 			}
 		});
 	},

frappe/public/js/frappe/ui/tags.js

@@ -25,7 +25,7 @@ frappe.ui.TagEditor = Class.extend({
 						method: 'frappe.desk.tags.add_tag',
 						args: me.get_args(tag),
 						callback: function(r) {
-							var user_tags = me.user_tags.split(",");
+							var user_tags = me.user_tags ? me.user_tags.split(",") : [];
 							user_tags.push(tag)
 							me.user_tags = user_tags.join(",");
 							me.on_change && me.on_change(me.user_tags);