Custom script in Stock Entry for Price List

ERPNext Stock/Inventory
6

Hi @airzoink,

Did you added custom field on stock entry detail doctype? If yes then just goto Custom script > select doctype as stock entry > paste above script under Script field > save it.

On stock entry, select the price list first and then item code, one more thing you have to set the name of the field instead of fieldname in which you want to set the rate

7

@rohit_w
What i have done:

  • Added “Price List Rate” data field type in Stock Entry Detail
  • Added “Selling Price List” link field type in Stock Entry
  • Added your code as a Custom Script under Stock Entry DocType and changing the ‘fieldname’ to ‘selling_price_list’

I must be doing something not right since i don’t see the changes.

8

Fieldtype should be of type currency not data

use below code

frappe.ui.form.on("Stock Entry Detail", {
	item_code: function(frm, cdt, cdn) {
		child 
		me.frm.call({
			method: "frappe.client.get_value",
			args: {
				doctype: "Item Price",
				fieldname: "price_list_rate",
				filters: { item_code: child.item_code, price_list : frm.doc.selling_price_list},
			},
			callback: function(r, rt) {
				if(r.message) {
					frappe.mode.set_value(cdt, cdn, 'price_list_rate', r.message.price_list_rate)
				}
			}
		});
	}
})

where,
fieldname is price_list_rate
price list is selling_price_list

Thanks

9

Made changes as you described. But price rate is still not displayed.

s2.JPG1243×679 49 KB

sample.JPG1158×760 35.3 KB

10

Hi @airzoink,

share browser’s console log

11

Browser’s Console Log

VM1511:509 Uncaught ReferenceError: child is not defined(…)item_code @ VM1511:509(anonymous function) @ form.min.js:254(anonymous function) @ form.min.js:254map @ jquery.min.js:2trigger @ form.min.js:254(anonymous function) @ form.min.js:33(anonymous function) @ desk.min.js:548each @ jquery.min.js:2run @ desk.min.js:548trigger @ desk.min.js:548set_value @ desk.min.js:547set_model_value @ desk.min.js:384set @ desk.min.js:383callback @ form.min.js:261callback @ desk.min.js:133200 @ desk.min.js:135(anonymous function) @ desk.min.js:140i @ jquery.min.js:2fireWith @ jquery.min.js:2z @ jquery.min.js:4(anonymous function) @ jquery.min.js:4

12

Hi @airzoink

oops got the issue

frappe.ui.form.on("Stock Entry Detail", {
	item_code: function(frm, cdt, cdn) {
		child = locals[cdt][cdn];
		me.frm.call({
			method: "frappe.client.get_value",
			args: {
				doctype: "Item Price",
				fieldname: "price_list_rate",
				filters: { item_code: child.item_code, price_list : frm.doc.selling_price_list},
			},
			callback: function(r, rt) {
				if(r.message) {
					frappe.mode.set_value(cdt, cdn, 'price_list_rate', r.message.price_list_rate)
				}
			}
		});
	}
})
1 Like
13

Applied. New error

VM2241:519 Uncaught TypeError: Cannot read property ‘set_value’ of undefined(…)callback @ VM2241:519opts.callback @ form.min.js:176callback @ desk.min.js:133200 @ desk.min.js:135(anonymous function) @ desk.min.js:140i @ jquery.min.js:2fireWith @ jquery.min.js:2z @ jquery.min.js:4(anonymous function) @ jquery.min.js:4

14

Hi @airzoink

frappe.model.set_value(cdt, cdn, 'price_list_rate', r.message.price_list_rate)
15

Success! :grinning:

16

@rohit_w
By the way can the script only show selling rates and not buying rate?

17

Hi @airzoink,

It shows the rate of the selected price list, so don’t select the buying price list or use get_query for price list to show only selling price list.

18

@rohit_w

Can you give me an example on how to apply get_query?

Thanks!

19

Hi @airzoink

check

20

@rohit_w

Based on the original script you have provided. Where do i insert these 7 lines of code?

If i edit it based on what i understand would it look like this?

frm.fields_dict.selling_price_list.get_query = function() {
return {
filters:{
‘selling’: ‘Yes’
}
}
}

Sorry i’m really a beginner at this. But i’m trying to understand bit by bit. :slight_smile:

21

Hi @airzoink

  • Were you successful with this customization?

If the answer is yes: -Do you mind showing me how you achieved it?

Thanks in advance

22

Not working :frowning:

23

@Muzzammil_Hussain
Can you please provide more details

24

sure let me add

25

you are missing this :
var child = locals[cdt][cdn];