fix: only fetch membership expiry if not already set in member.js (#31259)
This commit is contained in:
@@ -149,6 +149,7 @@ def allow_regional(fn):
|
|||||||
return caller
|
return caller
|
||||||
|
|
||||||
|
|
||||||
|
@frappe.whitelist()
|
||||||
def get_last_membership(member):
|
def get_last_membership(member):
|
||||||
"""Returns last membership if exists"""
|
"""Returns last membership if exists"""
|
||||||
last_membership = frappe.get_all(
|
last_membership = frappe.get_all(
|
||||||
|
|||||||
@@ -44,21 +44,18 @@ frappe.ui.form.on('Member', {
|
|||||||
frappe.contacts.clear_address_and_contact(frm);
|
frappe.contacts.clear_address_and_contact(frm);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (!frm.doc.membership_expiry_date && !frm.doc.__islocal) {
|
||||||
frappe.call({
|
frappe.call({
|
||||||
method:"frappe.client.get_value",
|
method: "erpnext.get_last_membership",
|
||||||
args: {
|
args: {
|
||||||
'doctype':"Membership",
|
member: frm.doc.member
|
||||||
'filters':{'member': frm.doc.name},
|
|
||||||
'fieldname':[
|
|
||||||
'to_date'
|
|
||||||
]
|
|
||||||
},
|
},
|
||||||
callback: function(data) {
|
callback: function(data) {
|
||||||
if (data.message) {
|
if (data.message) {
|
||||||
frappe.model.set_value(frm.doctype,frm.docname,
|
frappe.model.set_value(frm.doctype, frm.docname, "membership_expiry_date", data.message.to_date);
|
||||||
"membership_expiry_date", data.message.to_date);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
}
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user