From c725745d14de28611d895459d43bd5996f8703b5 Mon Sep 17 00:00:00 2001
From: xmo-odoo <xmo@odoo.com>
Date: Mon, 9 Oct 2017 10:56:37 +0200
Subject: [PATCH] [FIX] sale: action methods with no return values

Returns apparently lost during salepocalypse, leads to errors when
calling them over XML-RPC (cannot marshal None).

Fixes #19889
---
 addons/sale/sale.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/addons/sale/sale.py b/addons/sale/sale.py
index 7b81e631213..020219a7e8c 100644
--- a/addons/sale/sale.py
+++ b/addons/sale/sale.py
@@ -360,11 +360,11 @@ class SaleOrder(models.Model):
             'state': 'draft',
             'procurement_group_id': False,
         })
-        orders.mapped('order_line').mapped('procurement_ids').write({'sale_line_id': False})
+        return orders.mapped('order_line').mapped('procurement_ids').write({'sale_line_id': False})
 
     @api.multi
     def action_cancel(self):
-        self.write({'state': 'cancel'})
+        return self.write({'state': 'cancel'})
 
     @api.multi
     def action_quotation_send(self):
@@ -413,7 +413,7 @@ class SaleOrder(models.Model):
 
     @api.multi
     def action_done(self):
-        self.write({'state': 'done'})
+        return self.write({'state': 'done'})
 
     @api.model
     def _prepare_procurement_group(self):