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

Merge pull request #1368 from nabinhait/link_field_trigger

Browse source 
[fix] Don't trigger link fields on new doc after mapping
This commit is contained in:
Nabin Hait 2015-10-26 17:42:18 +05:30
commit 2d58df2e05
2 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
frappe/public/js/frappe/model/create_new.js
View file

@ -227,7 +227,10 @@ $.extend(frappe.model, {
freeze: true,
callback: function(r) {
if(!r.exc) {
var doc = frappe.model.sync(r.message);
frappe.model.sync(r.message);
var doc = frappe.model.get_doc(r.message.doctype, r.message.name);
doc.__mapped = true;
frappe.set_route("Form", r.message.doctype, r.message.name);
}
}

2
frappe/public/js/legacy/form.js
View file

@ -548,7 +548,7 @@ _f.Frm.prototype.setnewdoc = function() {
_f.Frm.prototype.trigger_link_fields = function() {
// trigger link fields which have default values set
if (this.is_new()) {
if (this.is_new() && !this.doc.__mapped) {
$.each(this.fields_dict, function(fieldname, field) {
if (field.df.fieldtype=="Link" && this.doc[fieldname]) {
// triggers add fetch, sets value in model and runs triggers