From 90dd27e5c8bf5001c38770b1ce15cdcd32763a16 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Wed, 17 Sep 2014 21:45:54 +0200
Subject: [PATCH] Use context to check model and field access

issue4081
review8481002
---
 sale.py  | 16 ++++++----------
 stock.py |  3 +--
 2 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/sale.py b/sale.py
index 406ec91fe6..9dc2dab620 100644
--- a/sale.py
+++ b/sale.py
@@ -160,12 +160,11 @@ class Sale:
                 with Transaction().set_context(
                         shipment.get_carrier_context()):
                     cost, currency_id = self.carrier.get_sale_price()
-                with Transaction().set_user(0, set_context=True):
-                    Shipment.write([shipment], {
-                            'carrier': self.carrier.id,
-                            'cost': cost,
-                            'cost_currency': currency_id,
-                            })
+                Shipment.write([shipment], {
+                        'carrier': self.carrier.id,
+                        'cost': cost,
+                        'cost_currency': currency_id,
+                        })
         return shipments
 
     def _get_invoice_line_sale_line(self, invoice_type):
@@ -185,8 +184,6 @@ class Sale:
         if (invoice
                 and invoice_type == 'out_invoice'
                 and self.shipment_cost_method == 'shipment'):
-            with Transaction().set_user(0, set_context=True):
-                invoice = Invoice(invoice.id)
             for shipment in self.shipments:
                 if (shipment.state == 'done'
                         and shipment.carrier
@@ -200,8 +197,7 @@ class Sale:
                     Shipment.write([shipment], {
                             'cost_invoice_line': invoice_line.id,
                             })
-            with Transaction().set_user(0, set_context=True):
-                Invoice.update_taxes([invoice])
+            Invoice.update_taxes([invoice])
         return invoice
 
 
diff --git a/stock.py b/stock.py
index 6d83c88ce2..a83a864535 100644
--- a/stock.py
+++ b/stock.py
@@ -88,8 +88,7 @@ class ShipmentOut:
 
         if not self.cost:
             return {}
-        with Transaction().set_user(0, set_context=True):
-            invoice_line = InvoiceLine()
+        invoice_line = InvoiceLine()
         product = self.carrier.carrier_product
         invoice_line.type = 'line'