From c0f49e52c856f9b335c366d1802e939f28c3e313 Mon Sep 17 00:00:00 2001
From: Sergi Almacellas Abellana <sergi@koolpi.com>
Date: Tue, 20 Sep 2016 09:49:57 +0200
Subject: [PATCH] Merge recursive results in DomainParser.parse_clause

We should use jQuery.merge instead of Array.concat, as concat creates a new
Array (which was not correctly returned) and jQuery.merge merges the contents
of both arrays in the first array.

issue5883
review26651003
---
 src/common.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/common.js b/src/common.js
index fc7da30b50..fa8f6e3df1 100644
--- a/src/common.js
+++ b/src/common.js
@@ -1561,7 +1561,7 @@
             var result = [];
             tokens.forEach(function(clause) {
                 if (this.is_generator(clause)) {
-                    result.concat(this.parse_clause(clause));
+                    jQuery.merge(result, this.parse_clause(clause));
                 } else if ((clause == 'OR') || (clause == 'AND')) {
                     result.push(clause);
                 } else if ((clause.length == 1) &&