Fix: Se agrega sale_date

locale/es.po

@@ -0,0 +1,35 @@
+#
+msgid ""
+msgstr "Content-Type: text/plain; charset=utf-8\n"
+
+msgctxt "field:sale.sale,delete_lines:"
+msgid "Delete lines"
+msgstr "Lineas Eliminadas"
+
+msgctxt "field:sale.line_deleted,user:"
+msgid "User"
+msgstr "Usuario"
+
+msgctxt "field:sale.line_deleted,sale:"
+msgid "Sale"
+msgstr "Venta"
+
+msgctxt "field:sale.line_deleted,product:"
+msgid "Product"
+msgstr "Producto"
+
+msgctxt "field:sale.line_deleted,unit:"
+msgid "Unit"
+msgstr "Unidad"
+
+msgctxt "field:sale.line_deleted,quantity:"
+msgid "Quantity"
+msgstr "Cantidad"
+
+msgctxt "field:sale.line_deleted,sale_date:"
+msgid "Sale Date"
+msgstr "Fecha Venta"
+
+msgctxt "model:ir.ui.menu,name:menu_sale_line_deleted_form"
+msgid "Sale Line Deleted"
+msgstr "Linea de Venta Eliminadas"

sale.py

@@ -7,32 +7,28 @@ class Sale(metaclass=PoolMeta):
     __name__ = 'sale.sale'
 
     delete_lines = fields.One2Many(
-        'sale.line_deleted',
+        'sale.line_deleted', 'sale', "Delete lines",
-        'sale',
-        "Delete lines",
         states={
             'readonly': True
-        }
+        })
-    )
 
 
 class SaleLine(metaclass=PoolMeta):
     __name__ = 'sale.line'
 
     @classmethod
-    def delete(cls, lines):
+    def _create_sale_line_deleted_log(cls, line):
         pool = Pool()
         SaleLineDeleted = pool.get('sale.line_deleted')
-        for line in lines:
-            SaleLineDeleted.create([{
-                'sale': line.sale,
-                'quantity': line.quantity,
-                'unit': line.unit,
-                'product': line.product,
-                'user': Transaction().user
-            }])
 
-        super(SaleLine, cls).delete(lines)
+        SaleLineDeleted.create([{
+            'sale': line.sale,
+            'sale_date': line.sale_date,
+            'quantity': line.quantity,
+            'unit': line.unit,
+            'product': line.product,
+            'user': Transaction().user
+        }])
 
 
 class SaleLineDeleted(ModelSQL, ModelView):
@@ -40,26 +36,22 @@ class SaleLineDeleted(ModelSQL, ModelView):
 
     __name__ = 'sale.line_deleted'
 
+    _state = states = {
+        'readonly': True
+    }
+
     sale = fields.Many2One(
-        'sale.sale', "Sale", ondelete='CASCADE', required=True
+        'sale.sale', "Sale", ondelete='CASCADE', required=True, states=_state)
-    )
-
     quantity = fields.Float(
-        "Quantity", digits='unit',
+        "Quantity", digits='unit', states=_state)
-        # states={
+    unit = fields.Many2One(
-        #     'invisible': Eval('type') != 'line',
+        'product.uom', 'Unit', ondelete='RESTRICT', states=_state)
-        #     'required': Eval('type') == 'line',
+    product = fields.Many2One(
-        #     'readonly': Eval('sale_state') != 'draft',
+        'product.product', 'Product',
-        # }
+        ondelete='RESTRICT', states=_state)
+    sale_date = fields.Date(
+        'Sale Date',
+        states=_state
     )
-
-    unit = fields.Many2One('product.uom', 'Unit', ondelete='RESTRICT',)
-
-    product = fields.Many2One('product.product', 'Product',
-                              ondelete='RESTRICT',
-                              )
-    sale_date = fields.Date('Sale Date',)
     user = fields.Many2One(
-        'res.user',
+        'res.user', 'User', states=_state)
-        'User'
-    )

sale.xml

@@ -38,7 +38,11 @@
          parent="sale.menu_sale"
          action="act_sale_line_deleted_form"
          sequence="10"
-         id="menu_sale_form"/>
+      id="menu_sale_line_deleted_form"/>
+      <record model="ir.ui.menu-res.group" id="menu_sale_group_sale">
+            <field name="menu" ref="menu_sale_line_deleted_form"/>
+            <field name="group" ref="sale.group_sale_admin"/>
+        </record>
+
    </data>
 </tryton>
-

tests/scenario_sale_line_delete_log.rst

@@ -20,7 +20,7 @@ Imports::
 
 Activate modules::
 
-    >>> config = activate_modules('sale_line_delete_log')
+  >>> config = activate_modules('sale_line_delete_log')
 
 Initial data::
 
@@ -179,7 +179,4 @@ Delete a sale line::
   >>> sale.delete_lines[0]
   proteus.Model.get('sale.line_deleted')(1)
   >>> assert isinstance(sale.delete_lines[0], Model.get('sale.line_deleted')), "it's not instance SaleLineDeleted"
-  
+  ;
-
-
-