From 9330f4e82d6f943d91547474210957b416f47e2e Mon Sep 17 00:00:00 2001
From: Florent THOMAS <mailinglist@tdeo.fr>
Date: Sun, 1 Apr 2018 19:21:52 +0200
Subject: [PATCH] [FIX] contract_sale_generation: it doesn't create sales
 (#141)

* Change the method called in the view
* Complete the create_invoice method
* Bump version + authoring
* Correct bad call of method
  Small Documentation
* Add super call in python test
* FIX bad field names causing bad quantities in sale.order.line
---
 contract_sale_generation/README.rst             |  1 +
 contract_sale_generation/__manifest__.py        |  2 +-
 .../models/account_analytic_account.py          | 17 ++++++++++++++---
 .../views/account_analytic_account_view.xml     |  2 +-
 4 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/contract_sale_generation/README.rst b/contract_sale_generation/README.rst
index dda50022..e7ccfec9 100644
--- a/contract_sale_generation/README.rst
+++ b/contract_sale_generation/README.rst
@@ -40,6 +40,7 @@ Contributors
 ------------
 
 * Angel Moya <angel.moya@pesol.es>
+* Florent THOMAS <florent.thomas@mind-and-go.com>
 
 Maintainer
 ----------
diff --git a/contract_sale_generation/__manifest__.py b/contract_sale_generation/__manifest__.py
index 285b3013..f20c1a7a 100644
--- a/contract_sale_generation/__manifest__.py
+++ b/contract_sale_generation/__manifest__.py
@@ -6,7 +6,7 @@
 
 {
     'name': 'Contracts Management - Recurring Sales',
-    'version': '10.0.2.0.0',
+    'version': '10.0.3.0.0',
     'category': 'Contract Management',
     'license': 'AGPL-3',
     'author': "PESOL, "
diff --git a/contract_sale_generation/models/account_analytic_account.py b/contract_sale_generation/models/account_analytic_account.py
index 1079d3b8..c07d6b0d 100644
--- a/contract_sale_generation/models/account_analytic_account.py
+++ b/contract_sale_generation/models/account_analytic_account.py
@@ -21,10 +21,11 @@ class AccountAnalyticAccount(models.Model):
         sale_line = self.env['sale.order.line'].new({
             'order_id': order_id,
             'product_id': line.product_id.id,
-            'proudct_uom_qty': line.quantity,
-            'proudct_uom_id': line.uom_id.id,
+            'product_qty': line.quantity,
+            'product_uom_qty': line.quantity,
+            'product_uom': line.uom_id.id,
         })
-        # Get other invoice line values from product onchange
+        # Get other sale line values from product onchange
         sale_line.product_id_change()
         sale_line_vals = sale_line._convert_to_write(sale_line._cache)
         # Insert markers
@@ -66,6 +67,11 @@ class AccountAnalyticAccount(models.Model):
 
     @api.multi
     def _create_invoice(self):
+        """
+        Create invoices
+        @param self: single record of account.invoice
+        @return: MUST return an invoice recordset
+        """
         self.ensure_one()
         if self.type == 'invoice':
             return super(AccountAnalyticAccount, self)._create_invoice()
@@ -74,6 +80,11 @@ class AccountAnalyticAccount(models.Model):
 
     @api.multi
     def _create_sale(self):
+        """
+        Create Sale orders
+        @param self: single record of sale.order
+        @return: MUST return a sale.order recordset
+        """
         self.ensure_one()
         if self.type == 'sale':
             sale_vals = self._prepare_sale()
diff --git a/contract_sale_generation/views/account_analytic_account_view.xml b/contract_sale_generation/views/account_analytic_account_view.xml
index 91578633..47a4ccec 100644
--- a/contract_sale_generation/views/account_analytic_account_view.xml
+++ b/contract_sale_generation/views/account_analytic_account_view.xml
@@ -14,7 +14,7 @@
                 <attribute name="attrs">{'invisible': ['|',('recurring_invoices','!=',True),('type','!=','invoice')]}</attribute>
             </xpath> 
             <xpath expr="//button[@name='recurring_create_invoice']" position="before">
-                <button name="recurring_create_invoice"
+                <button name="recurring_create_sale"
                             type="object"
                             attrs="{'invisible': ['|',('recurring_invoices','!=',True),('type','!=','sale')]}"
                             string="Create sales"