mrp_advanced_operations: Progressbar is wrong

[Daniel Stenlöv]

The progressbar does give a faulty value.

If you have more then 1 compomponent that gets produced it will show the progress as 100% for each finished product. So if you produce for example 4 products in one operation it will show 400%.

There is a secondary problem with the progress and that is if you cancel half of the productions it will still show 400%. It should however show 50% since half of the production is done.

[Daniel Stenlöv (daniel-stenlov-xclude)]

The error seems to be because of two things. First the production number seems to be wrong. The total production produced are based on the final product, not final product + the bi product. So the totalt number of products to be produced should be the total quantity of the final product and the bi products.

The secondary error seems to be that it calculated the product to be finished even if the product is cancelled. This could be fixed with an easy check if the product is in done state like this:

---------- CUT ----------

                if ((finish_move.product_id.id == finish_move.operation_id.product_id.id) or (finish_move.product_id.id == real_operation.product.id) and (finish_move.state == 'done')):
                    qty += finish_move.product_qty

---------- CUT ----------

[Daniel Stenlöv (daniel-stenlov-xclude)]

Partly fixed in lp:~xclude/avanzosc/mrp-6.1 revision 4.