Merge lp:~numerigraphe-team/openobject-addons/trunk-wms-stock-latest-move into lp:~openerp-dev/openobject-addons/trunk-wms

[Lionel Sausin - Initiatives/Numérigraphe]

This branch improves the report model "latest inventory date", by showing the latest Stock Move as well.
This helps users isolate unused products that may deserve special measures like discount sales or quality control.
It also optimizes the SQL query a bit, and let users filter inactive products or not.

This branch is unchanged from the v7.0 proposal, only includes the current trunk-wms for Runbot.
It runs green: http://runbot.openerp.com/numerigraphe-team-trunk-wms-stock-latest-move-30937/logs/

[qdp (OpenERP) (qdp)]

i merged it in revision 9648 even if, to be honnest, i'm not sure the add of latest stock move date is worth the join with stock_move table. I trusted you on this. ;-)

thanks

=== modified file 'stock/report/report_stock.py'
--- stock/report/report_stock.py	2013-11-29 16:56:44 +0000
+++ stock/report/report_stock.py	2014-03-18 14:01:45 +0000
@@ -25,13 +25,15 @@
 
 class report_stock_lines_date(osv.osv):
     _name = "report.stock.lines.date"
-    _description = "Dates of Inventories"
+    _description = "Dates of Inventories and latest Moves"
     _auto = False
     _order = "date"
     _columns = {
-        'id': fields.integer('Inventory Line Id', readonly=True),
+        'id': fields.integer('Product Id', readonly=True),
         'product_id': fields.many2one('product.product', 'Product', readonly=True, select=True),
-        'date': fields.datetime('Latest Inventory Date'),
+        'date': fields.datetime('Date of latest Inventory', readonly=True),
+        'move_date': fields.datetime('Date of latest Stock Move', readonly=True),
+        "active" : fields.boolean("Active", readonly=True),
     }
     def init(self, cr):
         drop_view_if_exists(cr, 'report_stock_lines_date')
@@ -40,13 +42,16 @@
                 select
                 p.id as id,
                 p.id as product_id,
-                max(s.date) as date
+                max(s.date) as date,
+                max(m.date) as move_date,
+				p.active as active
             from
                 product_product p
-                    left outer join stock_inventory_line l on (p.id=l.product_id)
-                    left join stock_inventory s on (l.inventory_id=s.id)
-                and s.state = 'done'
-                where p.active='true'
+                    left join (
+                        stock_inventory_line l
+                        inner join stock_inventory s on (l.inventory_id=s.id and s.state = 'done')
+                    ) on (p.id=l.product_id)
+                    left join stock_move m on (m.product_id=p.id and m.state = 'done')
                 group by p.id
             )""")
 

=== modified file 'stock/report/report_stock_view.xml'
--- stock/report/report_stock_view.xml	2013-11-29 16:56:44 +0000
+++ stock/report/report_stock_view.xml	2014-03-18 14:01:45 +0000
@@ -10,6 +10,7 @@
                 <tree string="Dates of Inventories" create="false">
                     <field name="product_id"/>
                     <field name="date" />
+                    <field name="move_date"/>
                 </tree>
             </field>
         </record>
@@ -18,12 +19,13 @@
             <field name="name">report.stock.lines.date.search</field>
             <field name="model">report.stock.lines.date</field>
             <field name="arch" type="xml">
-                <search string="Dates of Inventories">
+                <search string="Dates of Inventories &amp; Moves">
                     <field name="date"/>
                     <filter icon="terp-accessories-archiver" name="stockable" string="Stockable" domain="[('product_id.type','=', 'product')]"/>
                     <filter icon="terp-accessories-archiver" string="Consumable" domain="[('product_id.type','=', 'consu')]"/>
                     <separator/>
-                    <filter icon="terp-accessories-archiver-minus" string="Non Inv" domain="[('date','=', False)]"/>
+                    <filter icon="terp-accessories-archiver-minus" string="No Inventory yet" domain="[('date','=', False)]"/>
+                    <filter icon="terp-accessories-archiver-minus" string="No Stock Move yet" domain="[('move_date','=', False)]"/>
                     <field name="product_id"/>
                 </search>
             </field>
@@ -33,22 +35,23 @@
             <field name="name">report.stock.lines.date.form</field>
             <field name="model">report.stock.lines.date</field>
             <field name="arch" type="xml">
-                <form string="Dates of Inventories" version="7.0">
+                <form string="Dates of Inventories &amp; Moves" version="7.0">
                     <group>
                         <field name="product_id"/>
                         <field name="date"/>
+                        <field name="move_date"/>
                     </group>
                 </form>
             </field>
         </record>
 
         <record model="ir.actions.act_window" id="action_stock_line_date">
-            <field name="name">Last Product Inventories</field>
+            <field name="name">Latest Inventories &amp; Moves</field>
             <field name="res_model">report.stock.lines.date</field>
             <field name="view_type">form</field>
              <field name="context">{'search_default_stockable':1}</field>
             <field name="view_mode">tree,form</field>
-            <field name="help">Display the last inventories done on your products and easily sort them with specific filtering criteria. If you do frequent and partial inventories, you need this report in order to ensure that the stock of each product is controlled at least once a year.</field>
+            <field name="help">Display the latest Inventories and Moves done on your products and easily sort them with specific filtering criteria. If you do frequent and partial inventories, you need this report in order to ensure that the stock of each product is controlled at least once a year. This also lets you find out which products have seen little move lately and may deserve special measures (discounted sale, quality control...)</field>
         </record>
 
         <menuitem parent="next_id_61" action="action_stock_line_date" id="menu_report_stock_line_date" sequence="2"/>