Calculating total in print format

Frappe Framework Print Formats
1

I am calculating “total carton” in my sales invoice print format based on the qty entered in sales invoice and units_in_ctn field fetched from Item master. Grid is showing correct data. However when I use a variable to calculate total then it always show 0.0

{% set total_ctns = 0.0 %}
{% for item in doc.entries %}
{% set item_doc = frappe.get_doc("Item", item.item_code) %}
{% set ctns = (item.qty+item.schm)/item_doc.units_in_ctn %}
{% set total_ctns = total_ctns + ctns %}
 ---- SHOW DATA IN TABLE HERE
{%- endfor -%}

If I show this varible total_ctns in the for loop then it shows correct value, however after endfor it shows 0.00. Any help???

Inv.png657×518 31.1 KB

2

This is a Jinja scoping issue and does not allow sharing of variables in the scope.

try using doc.total_ctns

3

@rmehta means I have to create a new field in “Sales Invoice” doctype and then set the value of this field in print format? How can I set the value of this field in print format?

4

You can attach your variable to “doc” rather than declare it as global. It might fix the scope issue.

5

@rmehta any sample code related to setting value in doc variable in print format?

6

This is what I meant:

{% set doc.total_ctns = 0.0 %}

7

@rmehta I tried to create a variable total_ctns and then setting its value. However facing this issue
> self.handle_exception(exc_info, source_hint=source)

File “/home/erpnext/frappe-bench/env/local/lib/python2.7/site-packages/jinja2/environment.py”, line 742, in handle_exception
reraise(exc_type, exc_value, tb)
File “”, line 33, in template
TemplateSyntaxError: expected token ‘=’, got ‘.’

1 Like
8

mmm… I am not sure. Maybe you should do all calculations server side in custom fields and use the data in Jinja. Jinja is not optimized for processing.

9

@rmehta this are very small calculations and I think jinja is doing the job but not handling the scope of the variables correctly. Jinja is calculating the total_ctns but not able to retain the value of this varibale beyond the scope of for loop

Well let me try to find a way for it.

10

Its late for this post, but i also ran into same issue. For ref, i took the following approach.

Register a custom filter with jinja context in custom app by adding jenv_filter to hooks.py

jenv_filter = [
    'indent_refill_qty:flows.jinja_filters.indent_refill_qty',
    'indent_oneway_qty:flows.jinja_filters.indent_oneway_qty',
]

format for registering filter is filter_name:method_name

Use filters in template like following

{{ doc.indent| indent_refill_qty }}
5 Likes
11

Try this:
If the child table is called Items and you want the sum of ‘qty’ from Item rows:

{{ doc.items|sum(attribute='qty') }}

6 Likes
12

The path “flows.jinja_filters.indent_refill_qty”: can you describe what “flows”, “jinja_filters” and “indent_refill_qty” are? Is this a dotted.path.to_function syntax? I’m trying to repeat your steps in my own custom app without success.

1 Like
14

flows is my app name.
jinja_filters is the module/file in app.
indent_refill_qty is a function within the module/file.

you can also check this out Flows/jinja_filters.py at master · BhupeshGupta/Flows · GitHub

1 Like
15

Perfect! I knew that path was too long to point to a method in a doctype. Thank you!

16

So an update on this: I got it working with @BhupeshGupta’s instructions and example, but it required a bench migrate for the changes in hooks.py to get picked up in the Jinja environment. I doesn’t really make sense to me what that would need to be done since other changes to hooks.py are dynamic. I hope I save somebody some future frustration with that.

17

use array to maintain the value out of the looping scope

18

I tried to add my filter and linked it to hooks.py as you suggested in comment above. But it says: no filter named ‘’ while saving the template.

Any idea where i could have missed ?

Regards,

Parth Joshi

19

Haven’t faced this issue myself, but as per @tmatteson you need to do bench migrate. Try that.

20

bench migrate ??

Do we need to do bench migrate because of hooks.py or some other reasons ?

Regards,

Parth Joshi

21

i suggest create another field in sales invoice then used this custom script to compute the total number of cartons. and try this code

total_cartons → the field that calculates the number of cartons

frappe.ui.form.on(“Sales Invoice”, “refresh”, function(frm, cdt, cdn) {
total_cartons = 0;
$.each(frm.doc.items || [], function(i, d) {
total_cartons += flt(d.units_in_ctn);
});
frm.set_value(“total_cartons”, total_cartons);
});