From 20a2bcd2f7fd62f7df0d791e181009b81c18a1fe Mon Sep 17 00:00:00 2001
From: Nicolas Martinelli <nim@odoo.com>
Date: Fri, 11 Sep 2015 11:06:37 +0200
Subject: [PATCH] [FIX] lunch: previous orders

For some reason, the computed Many2many or One2many field is not
received through the onchange RPC call when creating a new order, but
only once the order is saved. This makes the widget displaying previous
orders kind of useless. We use this workaround with the default value
in order to be sure to send the value in the default_get call.

We also display previous orders and the agressive alerts in edit mode.
---
 addons/lunch/models/lunch.py       | 25 ++++++++++++++-----------
 addons/lunch/views/lunch_views.xml |  4 ++--
 2 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/addons/lunch/models/lunch.py b/addons/lunch/models/lunch.py
index 85059464886..6390af8e4cc 100644
--- a/addons/lunch/models/lunch.py
+++ b/addons/lunch/models/lunch.py
@@ -16,6 +16,15 @@ class LunchOrder(models.Model):
     _description = 'Lunch Order'
     _order = 'date desc'
 
+    def _default_previous_order_ids(self):
+        prev_order = self.env['lunch.order.line'].search([('user_id', '=', self.env.uid)], limit=20, order='id desc')
+        # If we return return prev_order.ids, we will have duplicates (identical orders).
+        # Therefore, this following part removes duplicates based on product_id and note.
+        return {
+            (order.product_id, order.note): order.id
+            for order in prev_order
+        }.values()
+
     user_id = fields.Many2one('res.users', 'User', required=True, readonly=True,
                               states={'new': [('readonly', False)]},
                               default=lambda self: self.env.uid)
@@ -32,8 +41,8 @@ class LunchOrder(models.Model):
                              'Status', readonly=True, index=True, copy=False, default='new',
                              compute='_compute_order_state')
     alerts = fields.Text(compute='_compute_alerts_get', string="Alerts")
-    previous_order_ids = fields.One2many(comodel_name='lunch.order.line',
-                                         compute='_compute_get_previous_order_ids')
+    previous_order_ids = fields.Many2many('lunch.order.line', compute='_compute_previous_order_ids',
+                                          default=lambda self: self._default_previous_order_ids())
     company_id = fields.Many2one('res.company', related='user_id.company_id', store=True)
     currency_id = fields.Many2one('res.currency', related='company_id.currency_id', readonly=True, store=True)
     cash_move_balance = fields.Monetary(compute='_compute_cash_move_balance', multi='cash_move_balance')
@@ -64,15 +73,9 @@ class LunchOrder(models.Model):
         if self.state == 'new':
             self.alerts = alert_msg and '\n'.join(alert_msg) or False
 
-    @api.depends('state')
-    def _compute_get_previous_order_ids(self):
-        prev_order = self.env['lunch.order.line'].search([('user_id', '=', self.env.uid)], limit=20, order='id desc')
-        # If we return return prev_order.ids, we will have duplicates (identical orders).
-        # Therefore, this following part removes duplicates based on product_id and note.
-        self.previous_order_ids = {
-            (order.product_id, order.note): order.id
-            for order in prev_order
-        }.values()
+    @api.depends('user_id')
+    def _compute_previous_order_ids(self):
+        self.previous_order_ids = self._default_previous_order_ids()
 
     @api.onchange('total')
     def _onchange_total(self):
diff --git a/addons/lunch/views/lunch_views.xml b/addons/lunch/views/lunch_views.xml
index d772141149b..2133a0280b4 100644
--- a/addons/lunch/views/lunch_views.xml
+++ b/addons/lunch/views/lunch_views.xml
@@ -394,10 +394,10 @@
                                 </div>
                             </group>
                         </group>
-                        <div attrs="{'invisible': ['|',('state','!=','new'),('alerts','=',False)]}" class="oe_inline alert alert-danger">
+                        <div attrs="{'invisible': ['|',('state','!=','new'),('alerts','=',False)]}" class="oe_inline alert alert-danger oe_edit_only">
                             <field name='alerts'/>
                         </div>
-                        <field name='previous_order_ids' widget='previous_order'/>
+                        <field name='previous_order_ids' widget='previous_order' class="oe_edit_only"/>
                         <separator string='Select your order'/>
                         <field name='order_line_ids' nolabel='1'>
                             <tree string='List' editable='bottom'>