diff options
| author | Pierre Masereel <pim@odoo.com> | 2016-08-25 10:38:55 +0200 |
|---|---|---|
| committer | Pierre Masereel <pim@odoo.com> | 2016-08-26 10:57:13 +0200 |
| commit | 770c9c4607f8480ec128b91c9320b068a4df9030 (patch) | |
| tree | 2c891ab3a89f5f103de847e7cf64b66fdf74dec1 /addons/lunch/models | |
| parent | ba564c42c1b7dfd640be280881cd484a803162a3 (diff) | |
[FIX] lunch: remove required on user_id
* Since the user_id field is required, it was impossible to delete
user when he had 'lunch.order' or 'lunch.cashmove'.
We removed required arguments on 'user_id' fields in model and set
"required='1'" on the fields in the view.
* There was a 'ondelete="cascade"' on 'order_line_ids' which does
nothing because it's not supported on One2many fields.
Diffstat (limited to 'addons/lunch/models')
| -rw-r--r-- | addons/lunch/models/lunch.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/addons/lunch/models/lunch.py b/addons/lunch/models/lunch.py index 385edc97a82..7cc750fb349 100644 --- a/addons/lunch/models/lunch.py +++ b/addons/lunch/models/lunch.py @@ -26,14 +26,14 @@ class LunchOrder(models.Model): for order in prev_order }.values() - user_id = fields.Many2one('res.users', 'User', required=True, readonly=True, + user_id = fields.Many2one('res.users', 'User', readonly=True, states={'new': [('readonly', False)]}, default=lambda self: self.env.uid) date = fields.Date('Date', required=True, readonly=True, states={'new': [('readonly', False)]}, default=fields.Date.context_today) order_line_ids = fields.One2many('lunch.order.line', 'order_id', 'Products', - ondelete="cascade", readonly=True, copy=True, + readonly=True, copy=True, states={'new': [('readonly', False)], False: [('readonly', False)]}) total = fields.Float(compute='_compute_total', string="Total", store=True) state = fields.Selection([('new', 'New'), @@ -220,7 +220,7 @@ class LunchCashMove(models.Model): _name = 'lunch.cashmove' _description = 'lunch cashmove' - user_id = fields.Many2one('res.users', 'User', required=True, + user_id = fields.Many2one('res.users', 'User', default=lambda self: self.env.uid) date = fields.Date('Date', required=True, default=fields.Date.context_today) amount = fields.Float('Amount', required=True, help='Can be positive (payment) or negative (order or payment if user wants to get his money back)') |
