Skip to content

Commit 082108f

Browse files
committed
query builder fix geo facet
1 parent 4a74856 commit 082108f

File tree

3 files changed

+11
-5
lines changed

3 files changed

+11
-5
lines changed

.gitignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,4 @@
11
node_modules
2+
tmp
3+
.project
4+
.settings

lib/query-builder.js

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -669,6 +669,7 @@ function geoQuery(variant, args, query, next) {
669669
seekingRegion = false;
670670
} else if (seekingRegion && arg instanceof Array && arg.length === 2) {
671671
query.point = [latlon.call(null, arg)];
672+
seekingRegion = false;
672673
} else {
673674
if (constraintName === undefined) {
674675
constraintName = arg.constraintName;
@@ -681,11 +682,13 @@ function geoQuery(variant, args, query, next) {
681682
}
682683

683684
var wrapper = {};
684-
if (constraintName === undefined) {
685-
wrapper[variant+'-query'] = query;
686-
} else {
685+
if (!valcheck.isUndefined(constraintName)) {
687686
wrapper.name = constraintName;
688687
wrapper[variant] = query;
688+
} else if (seekingRegion) {
689+
wrapper[variant] = query;
690+
} else {
691+
wrapper[variant+'-query'] = query;
689692
}
690693
return wrapper;
691694
}

test-basic/query-builder.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -523,14 +523,14 @@ describe('query-builder', function() {
523523
assert.deepEqual(
524524
q.geoElement('parent', 'element',
525525
q.heatmap(2, 3, q.southWestNorthEast(1.1, 2.2, 3.3, 4.4))),
526-
{'geo-elem-query':{parent:{name:'parent'},
526+
{'geo-elem':{parent:{name:'parent'},
527527
element:{name:'element'},
528528
heatmap:{latdivs:2, londivs:3, s:1.1, w:2.2, n:3.3, e:4.4}}}
529529
);
530530
assert.deepEqual(
531531
q.geoElement('parent', 'element',
532532
q.heatmap(2, 3, 1.1, 2.2, 3.3, 4.4)),
533-
{'geo-elem-query':{parent:{name:'parent'},
533+
{'geo-elem':{parent:{name:'parent'},
534534
element:{name:'element'},
535535
heatmap:{latdivs:2, londivs:3, s:1.1, w:2.2, n:3.3, e:4.4}}}
536536
);

0 commit comments

Comments
 (0)