Double message print custom script

1

image

image
image766×300 5.76 KB

how can i make only one print message? tnx in advance :blush:

2

:pray::pray::pray::pray:

3

@mycarlo try to add break; after the msgprint.

 break;
4

it works, but the checkbox is checked…
image

5

I want the checkbox is not check when not equal like the picture below…but I want only one msgprint

image
image711×335 12.5 KB

6

@mycarlo put it before the break .

7

what do you mean?

8

@mycarlo the frm.set_value… replace it just before the break .

9

image
image735×348 12.5 KB

same issue the checkbox is checked

10

frappe.ui.form.on(“Disposed Item”, {
check(frm){

	 for(var a=0, len1=frm.doc.items.length; a < len1;a++){
	
		 if(frm.doc.items[a].qty1 == frm.doc.items[a].qty){
		continue;
		}
		else{
		frappe.msgprint("Some Items are not equal to your Witdrawal");			
		break;	
		}				
		
	 }

}

});

this is my code

11

up ???

12

image
image766×602 22.7 KB

13

@Thukten_Dendup sir :smiling_face:

14

Since you are looping within for loop, it will print you multiple messages. The only option to print a single message is by introducing a flag and moving the msgprint outside for loop.

var flag = 0;
for(var a=0, len1=frm.doc.items.length; a < len1;a++){
if(frm.doc.items[a].qty1 == frm.doc.items[a].qty){
flag = 1;
}
}
if(flag){
frappe.msgprint(“Some Items are not equal to your Witdrawal”);
}

15

this is my code sir but not working…my trigger is checkbox

frappe.ui.form.on(“Disposed Item”, {
check(frm){
var flag =0;
for(var a=0; a<frm.doc.items.length;a++){

		 if(frm.doc.items[a].qty1 != frm.doc.items[a].qty){
	 var flag = 1;
		}
		}
		if(flag){
		msgprint('Items are not equal to your Witdrawal ' + 'row #'   + frm.doc.items[a].idx + '- '+frm.doc.items[a].item_code );
		frm.set_value("check",'');
		}				
					
				
}

});