[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>

1 files 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;