How to override method in frappe?

Frappe Framework Customization
,
3

@rmehta The complete feature will be enable bold fields in pos.
or
For this current fix, we just need to update below code. and loop data (data is local storage customer list here)
Please advise on it.

def add_customer(data):
	customer_doc = frappe.new_doc('Customer')
	customer_doc.customer_name = data.get('full_name') or data.get('customer')
	customer_doc.customer_pos_id = data.get('customer_pos_id')
	customer_doc.customer_type = 'Company'
	customer_doc.vehicle_no = data.get('vehicle_no')
	customer_doc.customer_group = get_customer_group(data)
	customer_doc.territory = get_territory(data)
	customer_doc.flags.ignore_mandatory = True
	customer_doc.save(ignore_permissions = True)
	frappe.db.commit()
return customer_doc.name
4

In custom_app module_name.py

import frappe
from erpnext.accounts.doctype.sales_invoice.sales_invoice import SalesInvoice

def shoutout(self):
	print("Yay!")

def before_cancel(self):
	self.shoutout()
	self.update_time_sheet(None)

def build_my_thing():
	SalesInvoice.shoutout = shoutout
	SalesInvoice.before_cancel = before_cancel

:hammer: python console :hammer:

In [1]: from custom_app.module_name import build_my_thing

In [2]: build_my_thing()

In [3]: sales_invoice = frappe.get_doc("Sales Invoice", "SINV-00165")

In [4]: sales_invoice.shoutout()
Yay!

In [5]: sales_invoice.cancel()
Yay!

more:

17 Likes
5

@revant_one
Thanks for the solution. I have marked this as solution as its working fine for overriding class method.

However this will not work in my case, as I need to override method which doesn’t belong to any class.

https://github.com/frappe/erpnext/blob/develop/erpnext/accounts/doctype/sales_invoice/pos.py#L372

1 Like
6

@revant_one I have been looking for such a method for a while. This is great pointer.
Quick Question - How is this loaded in the ERPNext context? Would you call build_my_thing() function on load of the core doctype? I’m no able to grasp when the bench would load the overridden function…

7

call build_my_thing just before you need to do overridden action.
onload, before frappe.get_doc or frappe.new_doc.

also, just like classes same thing is possible with modules.
Try it in bench console

8

Thanks @revant_one I tried using befoe_insert and onload. It works, but works inconsistently. Sometimes the custom method is called, but then other times, original method is called…doctype needs to be reloaded to call the custom method again…I dont know when it happens/ any thoughts?
Thanks.

9

Continued here

12

This thread is really popular. So I am posting this here.

You can now override a standard DocType class via hooks:

https://github.com/frappe/frappe/pull/11527

13 Likes
13

Is this different to this existing hook?

override_whitelisted_methods
14

That only works for whitelisted methods, which are those methods designated accessible to javascript.

15

@netchampfaris what about the method not belong to a class?

Like this file frappe/frappe/utils/data.py

1 Like
16

Other form that you can use to override methods in an application is call build_my_thing function at the end of hooks.py file. Specially userful when the method is in a class controller. We are testing that solution but apparently it works.

2 Likes
17

how to override the python method (not a whitelisted method and also not a method inside the class) which is in the erpnext app @rmehta @netchampfaris @revant_one @Mohammed_Redha @peterg

18

Did you check below before tagging to so many people ? What you have done so far and what are you trying to do ?

19

@sanjay yes I tried, I can able to override the whitelisted erpnext method from custom app via hooks.py and also a method inside the class, but i could not able to override the python method

20

Were you able to achieve it? I’m using override_doctype_class of hooks but it’s not working…

21

Hi @netchampfaris I need to use override_doctype_class hook but not sure if I’m missing something because it’s not working for me. The method metioned in the override_doctype_class is not reached. Could you please help?

22

I think override_doctype_class hook is part of v13 and above.

1 Like
23

@revant_one @netchampfaris Is there any way to override the method not belong to a class?

2 Likes