wip

Author: tobiaszehntner

stock_move_value_report/report/stock_move_line_value_report.xml

@@ -59,12 +59,12 @@
                     </td>
                     <td class="text-end">
                         <t
-                            t-set="valuation_layers"
-                            t-value="move_line.move_id.stock_valuation_layer_ids"
+                            t-set="move"
+                            t-value="move_line.move_id"
                         />
                         <t
                             t-set="cost"
-                            t-value="sum(valuation_layers.mapped(lambda svl: svl.unit_cost if svl.quantity > 0 else -svl.unit_cost))"
+                            t-value="sum(move.mapped(lambda m: m.value if m.quantity > 0 else -m.value))"
                         />
                         <span
                             t-esc="cost"
@@ -98,12 +98,12 @@
                                 <t t-set="total" t-value="0" />
                                 <t t-foreach="move_lines" t-as="move_line">
                                     <t
-                                        t-set="valuation_layers"
-                                        t-value="move_line.move_id.stock_valuation_layer_ids"
+                                        t-set="move"
+                                        t-value="move_line.move_id"
                                     />
                                     <t
                                         t-set="cost"
-                                        t-value="sum(valuation_layers.mapped(lambda svl: svl.unit_cost if svl.quantity > 0 else -svl.unit_cost))"
+                                        t-value="sum(move.mapped(lambda m: m.value if m.quantity > 0 else -m.value))"
                                     />
                                     <t
                                         t-set="total"
@@ -134,7 +134,7 @@
                     <t t-set="ml_from_inventories" />
                     <t
                         t-set="ml_from_scraps"
-                        t-value="move_lines_ids.filtered('move_id.scrapped')"
+                        t-value="move_lines_ids.filtered('move_id.scrap_id')"
                     />
                     <!-- ML From Pickings -->
                     <t t-if="ml_from_pickings">
@@ -147,7 +147,7 @@
                                 t-set="picking_move_lines"
                                 t-value="ml_from_pickings.filtered(lambda x: x.picking_id == picking)"
                             />
-                            <h4><span>[<t t-esc="picking.date" />]</span> Picking <t
+                            <h4><span>[<t t-esc="picking.date_done" />]</span> Picking <t
                                 t-esc="picking.name"
                             />: <t t-esc="picking.location_id.name" /> &#8594; <t
                                 t-esc="picking.location_dest_id.name"

stock_move_value_report/report/stock_picking_value_report.xml

@@ -91,7 +91,7 @@
                                     <span t-field="picking.state" />
                                 </td>
                                 <td class="text-center">
-                                    <span t-field="picking.date" />
+                                    <span t-field="picking.date_done" />
                                 </td>
                                 <td class="text-center" name="td_sched_date">
                                     <span t-field="picking.scheduled_date" />

stock_move_value_report/tests/test_stock_move_value_report.py

@@ -14,23 +14,45 @@ class Test(BaseCommon):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
-        cls.moves = cls.env["stock.move"].search([], limit=100)
+        cls.pickings = cls.env["stock.picking"].search([], limit=100)
+        cls.moves = cls.pickings.move_ids
         cls.move_lines = cls.moves.move_line_ids
-        cls.company = cls.env.company
+        picking = cls.pickings[0]
+        product = picking.move_ids[0].product_id
+        cls.scrap = cls.env['stock.scrap'].create({
+            'product_id': product.id,
+            'product_uom_id': product.uom_id.id,
+        })
 
-    def _render_report(self, reportname, records):
+    def _render_report(self, report_name, records):
+        """
+        Ensure rendering passes without errors
+        """
         context = {
             **self.env.context,
             "active_ids": records.ids,
             "active_model": "stock.move"
         }
         self.env["ir.actions.report"].with_context(**context)._render_qweb_pdf(
-            reportname, records.ids, data={"context": context}
+            report_name, records.ids, data={"context": context}
         )
 
     def test_report_stock_move_line_value(self):
-        """
-        Ensure
-        """
-        reportname = "stock_move_value_report.report_stock_move_line_value"
-        self._render_report(reportname, self.move_lines)
+        report_name = "stock_move_value_report.report_stock_move_line_value"
+        self.assertTrue(bool(self.move_lines))
+        self._render_report(report_name, self.move_lines)
+
+    def test_report_stock_move_value(self):
+        report_name = "stock_move_value_report.report_stock_move_value"
+        self.assertTrue(bool(self.moves))
+        self._render_report(report_name, self.moves)
+
+    def test_report_stock_picking_value(self):
+        report_name = "stock_move_value_report.report_stock_picking_value"
+        self.assertTrue(bool(self.pickings))
+        self._render_report(report_name, self.pickings)
+
+    def test_report_stock_scrap_value(self):
+        report_name = "stock_move_value_report.report_stock_scrap_value"
+        self.assertTrue(bool(self.scrap))
+        self._render_report(report_name, self.scrap)