From 988e201d2ec20b693096dff3318cc62ce40ccb45 Mon Sep 17 00:00:00 2001
From: RomainLibert <rli@odoo.com>
Date: Mon, 9 Sep 2019 14:04:12 +0000
Subject: [PATCH] [FIX] lunch: allow location_id to be NULL

In case the last location of a user was NULL in databse, lunch was
crashing because lunch_kanban was adding an undefined leaf to the load
domain.

closes odoo/odoo#36571

Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
---
 .../lunch/static/src/js/lunch_kanban_model.js | 22 +++++++++++--------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/addons/lunch/static/src/js/lunch_kanban_model.js b/addons/lunch/static/src/js/lunch_kanban_model.js
index 1e17b489e5f..d94812a6c3c 100644
--- a/addons/lunch/static/src/js/lunch_kanban_model.js
+++ b/addons/lunch/static/src/js/lunch_kanban_model.js
@@ -33,7 +33,7 @@ var LunchKanbanModel = KanbanModel.extend({
     getLocationDomain: function () {
         var self = this;
         return this._initUserLocation().then(function () {
-            return [self._buildLocationDomainLeaf()];
+            return self._buildLocationDomainLeaf() ? [self._buildLocationDomainLeaf()]: [];
         });
     },
     load: function () {
@@ -62,15 +62,19 @@ var LunchKanbanModel = KanbanModel.extend({
     //--------------------------------------------------------------------------
 
     _addOrUpdate: function (domain, subDomain) {
-        var key = subDomain[0];
-        var index = _.findIndex(domain, function (val) {
-            return val[0] === key;
-        });
+        if (subDomain && subDomain.length) {
+            var key = subDomain[0];
+            var index = _.findIndex(domain, function (val) {
+                return val[0] === key;
+            });
 
-        if (index < 0) {
-            domain.push(subDomain);
-        } else {
-            domain[index] = subDomain;
+            if (index < 0) {
+                domain.push(subDomain);
+            } else {
+                domain[index] = subDomain;
+            }
+
+            return domain;
         }
 
         return domain;