Skip to content

Commit 8a1b239

Browse files
authored
Widgets without connection has effect on layer interaction #11890 (#12483)
* fix: disconnected widget and counter filter fix * fix: not use layer filter when mapSync is off * fix non related change
1 parent 22f800f commit 8a1b239

2 files changed

Lines changed: 16 additions & 5 deletions

File tree

web/client/components/widgets/enhancers/__tests__/dependenciesToFilter-test.jsx

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -211,6 +211,17 @@ describe('widgets dependenciesToFilter enhancer', () => {
211211
/>, document.getElementById("container"));
212212
});
213213

214+
it('dependenciesToFilter with layerFilter and no mapSync', (done) => {
215+
const Sink = dependenciesToFilter(createSink(props => {
216+
expect(props).toExist();
217+
expect(props.filter).toBe(undefined);
218+
done();
219+
}));
220+
ReactDOM.render(<Sink
221+
layer={layerFilter}
222+
/>, document.getElementById("container"));
223+
});
224+
214225
it('dependenciesToFilter with empty filter parts', (done) => {
215226
const Sink = dependenciesToFilter(createSink(props => {
216227
expect(props).toExist();

web/client/components/widgets/enhancers/dependenciesToFilter.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -55,12 +55,12 @@ const createFilterProps = ({ mapSync, geomProp, dependencies = {}, filter: filte
5555
: [];
5656

5757
if (!mapSync) {
58+
const filterParts = [
59+
...(newFilterObj ? toOGCFilterParts(newFilterObj, "1.1.0", "ogc") : []),
60+
...interactionFilterParts
61+
];
5862
return {
59-
filter: !isEmpty(newFilterObj) || layerFilter || interactionFilterParts.length > 0 ? filter(and(
60-
...(layerFilter && !layerFilter.disabled ? toOGCFilterParts(layerFilter, "1.1.0", "ogc") : []),
61-
...(newFilterObj ? toOGCFilterParts(newFilterObj, "1.1.0", "ogc") : []),
62-
...interactionFilterParts
63-
)) : undefined
63+
filter: isEmpty(filterParts) ? undefined : filter(and(...filterParts))
6464
};
6565
}
6666
// merging filterObj with quickFilters coming from dependencies

0 commit comments

Comments
 (0)