Use named exports from filter module

examples/vector-wfs-getfeature.js

@@ -1,6 +1,10 @@
 import Map from '../src/ol/Map.js';
 import View from '../src/ol/View.js';
-import _ol_format_filter_ from '../src/ol/format/filter.js';
+import {
+  equalTo as equalToFilter,
+  like as likeFilter,
+  and as andFilter
+} from '../src/ol/format/filter.js';
 import WFS from '../src/ol/format/WFS.js';
 import GeoJSON from '../src/ol/format/GeoJSON.js';
 import TileLayer from '../src/ol/layer/Tile.js';
@@ -46,9 +50,9 @@ const featureRequest = new WFS().writeGetFeature({
   featurePrefix: 'osm',
   featureTypes: ['water_areas'],
   outputFormat: 'application/json',
-  filter: _ol_format_filter_.and(
+  filter: andFilter(
-    _ol_format_filter_.like('name', 'Mississippi*'),
+    likeFilter('name', 'Mississippi*'),
-    _ol_format_filter_.equalTo('waterway', 'riverbank')
+    equalToFilter('waterway', 'riverbank')
   )
 });
 

src/ol/format/WFS.js

@@ -6,7 +6,7 @@ import {assert} from '../asserts.js';
 import GML2 from '../format/GML2.js';
 import GML3 from '../format/GML3.js';
 import GMLBase from '../format/GMLBase.js';
-import _ol_format_filter_ from '../format/filter.js';
+import {and as andFilter, bbox as bboxFilter} from '../format/filter.js';
 import XMLFeature from '../format/XMLFeature.js';
 import XSD from '../format/XSD.js';
 import Geometry from '../geom/Geometry.js';
@@ -959,11 +959,11 @@ WFS.prototype.writeGetFeature = function(options) {
     if (options.bbox) {
       assert(options.geometryName,
         12); // `options.geometryName` must also be provided when `options.bbox` is set
-      const bbox = _ol_format_filter_.bbox(
+      const bbox = bboxFilter(
         /** @type {string} */ (options.geometryName), options.bbox, options.srsName);
       if (filter) {
         // if bbox and filter are both set, combine the two into a single filter
-        filter = _ol_format_filter_.and(filter, bbox);
+        filter = andFilter(filter, bbox);
       } else {
         filter = bbox;
       }

test/spec/ol/format/wfs.test.js

@@ -1,7 +1,24 @@
 import Feature from '../../../../src/ol/Feature.js';
 import GML2 from '../../../../src/ol/format/GML2.js';
 import WFS from '../../../../src/ol/format/WFS.js';
-import _ol_format_filter_ from '../../../../src/ol/format/filter.js';
+import {
+  and as andFilter,
+  bbox as bboxFilter,
+  between as betweenFilter,
+  contains as containsFilter,
+  during as duringFilter,
+  equalTo as equalToFilter,
+  greaterThan as greaterThanFilter,
+  greaterThanOrEqualTo as greaterThanOrEqualToFilter,
+  intersects as intersectsFilter,
+  isNull as isNullFilter,
+  lessThan as lessThanFilter,
+  lessThanOrEqualTo as lessThanOrEqualToFilter,
+  like as likeFilter,
+  not as notFilter,
+  or as orFilter,
+  within as withinFilter
+} from '../../../../src/ol/format/filter.js';
 import LineString from '../../../../src/ol/geom/LineString.js';
 import MultiLineString from '../../../../src/ol/geom/MultiLineString.js';
 import MultiPoint from '../../../../src/ol/geom/MultiPoint.js';
@@ -306,7 +323,7 @@ describe('ol.format.WFS', function() {
         featureNS: 'http://www.openplans.org/topp',
         featurePrefix: 'topp',
         featureTypes: ['states'],
-        filter: _ol_format_filter_.equalTo('name', 'New York', false)
+        filter: equalToFilter('name', 'New York', false)
       });
       expect(serialized.firstElementChild).to.xmleql(_ol_xml_.parse(text));
     });
@@ -334,9 +351,9 @@ describe('ol.format.WFS', function() {
         featureNS: 'http://www.openplans.org/topp',
         featurePrefix: 'topp',
         featureTypes: ['states'],
-        filter: _ol_format_filter_.or(
+        filter: orFilter(
-          _ol_format_filter_.equalTo('name', 'New York'),
+          equalToFilter('name', 'New York'),
-          _ol_format_filter_.equalTo('area', 1234))
+          equalToFilter('area', 1234))
       });
       expect(serialized.firstElementChild).to.xmleql(_ol_xml_.parse(text));
     });
@@ -376,14 +393,14 @@ describe('ol.format.WFS', function() {
         featureNS: 'http://www.openplans.org/topp',
         featurePrefix: 'topp',
         featureTypes: ['states'],
-        filter: _ol_format_filter_.or(
+        filter: orFilter(
-          _ol_format_filter_.and(
+          andFilter(
-            _ol_format_filter_.greaterThan('area', 100),
+            greaterThanFilter('area', 100),
-            _ol_format_filter_.greaterThanOrEqualTo('pop', 20000)
+            greaterThanOrEqualToFilter('pop', 20000)
           ),
-          _ol_format_filter_.and(
+          andFilter(
-            _ol_format_filter_.lessThan('area', 100),
+            lessThanFilter('area', 100),
-            _ol_format_filter_.lessThanOrEqualTo('pop', 20000)
+            lessThanOrEqualToFilter('pop', 20000)
           )
         )
       });
@@ -408,7 +425,7 @@ describe('ol.format.WFS', function() {
         featureNS: 'http://www.openplans.org/topp',
         featurePrefix: 'topp',
         featureTypes: ['states'],
-        filter: _ol_format_filter_.between('area', 100, 1000)
+        filter: betweenFilter('area', 100, 1000)
       });
       expect(serialized.firstElementChild).to.xmleql(_ol_xml_.parse(text));
     });
@@ -429,7 +446,7 @@ describe('ol.format.WFS', function() {
         featureNS: 'http://www.openplans.org/topp',
         featurePrefix: 'topp',
         featureTypes: ['states'],
-        filter: _ol_format_filter_.isNull('area')
+        filter: isNullFilter('area')
       });
       expect(serialized.firstElementChild).to.xmleql(_ol_xml_.parse(text));
     });
@@ -451,7 +468,7 @@ describe('ol.format.WFS', function() {
         featureNS: 'http://www.openplans.org/topp',
         featurePrefix: 'topp',
         featureTypes: ['states'],
-        filter: _ol_format_filter_.like('name', 'New*')
+        filter: likeFilter('name', 'New*')
       });
       expect(serialized.firstElementChild).to.xmleql(_ol_xml_.parse(text));
     });
@@ -473,7 +490,7 @@ describe('ol.format.WFS', function() {
         featureNS: 'http://www.openplans.org/topp',
         featurePrefix: 'topp',
         featureTypes: ['states'],
-        filter: _ol_format_filter_.like('name', 'New*', '*', '.', '!', false)
+        filter: likeFilter('name', 'New*', '*', '.', '!', false)
       });
       expect(serialized.firstElementChild).to.xmleql(_ol_xml_.parse(text));
     });
@@ -497,7 +514,7 @@ describe('ol.format.WFS', function() {
         featureNS: 'http://www.openplans.org/topp',
         featurePrefix: 'topp',
         featureTypes: ['states'],
-        filter: _ol_format_filter_.not(_ol_format_filter_.equalTo('name', 'New York'))
+        filter: notFilter(equalToFilter('name', 'New York'))
       });
       expect(serialized.firstElementChild).to.xmleql(_ol_xml_.parse(text));
     });
@@ -533,10 +550,10 @@ describe('ol.format.WFS', function() {
         featureNS: 'http://www.openplans.org/topp',
         featurePrefix: 'topp',
         featureTypes: ['states'],
-        filter: _ol_format_filter_.and(
+        filter: andFilter(
-          _ol_format_filter_.equalTo('name', 'New York'),
+          equalToFilter('name', 'New York'),
-          _ol_format_filter_.bbox('the_geom', [1, 2, 3, 4], 'urn:ogc:def:crs:EPSG::4326'),
+          bboxFilter('the_geom', [1, 2, 3, 4], 'urn:ogc:def:crs:EPSG::4326'),
-          _ol_format_filter_.greaterThan('population', 2000000)
+          greaterThanFilter('population', 2000000)
         )
       });
       expect(serialized.firstElementChild).to.xmleql(_ol_xml_.parse(text));
@@ -565,7 +582,7 @@ describe('ol.format.WFS', function() {
       const serialized = new WFS().writeGetFeature({
         srsName: 'EPSG:4326',
         featureTypes: ['area'],
-        filter: _ol_format_filter_.contains(
+        filter: containsFilter(
           'the_geom',
           new Polygon([[
             [10, 20],
@@ -602,7 +619,7 @@ describe('ol.format.WFS', function() {
       const serialized = new WFS().writeGetFeature({
         srsName: 'EPSG:4326',
         featureTypes: ['area'],
-        filter: _ol_format_filter_.intersects(
+        filter: intersectsFilter(
           'the_geom',
           new Polygon([[
             [10, 20],
@@ -639,7 +656,7 @@ describe('ol.format.WFS', function() {
       const serialized = new WFS().writeGetFeature({
         srsName: 'EPSG:4326',
         featureTypes: ['area'],
-        filter: _ol_format_filter_.within(
+        filter: withinFilter(
           'the_geom',
           new Polygon([[
             [10, 20],
@@ -679,7 +696,7 @@ describe('ol.format.WFS', function() {
       const serialized = new WFS().writeGetFeature({
         srsName: 'EPSG:4326',
         featureTypes: ['states'],
-        filter: _ol_format_filter_.during('date_prop', '2010-01-20T00:00:00Z', '2012-12-31T00:00:00Z')
+        filter: duringFilter('date_prop', '2010-01-20T00:00:00Z', '2012-12-31T00:00:00Z')
       });
       expect(serialized.firstElementChild).to.xmleql(_ol_xml_.parse(text));
     });
@@ -1298,9 +1315,9 @@ describe('ol.format.WFS', function() {
           '  </And>' +
           '</Filter>';
       const serialized = WFS.writeFilter(
-        _ol_format_filter_.and(
+        andFilter(
-          _ol_format_filter_.like('name', 'Mississippi*'),
+          likeFilter('name', 'Mississippi*'),
-          _ol_format_filter_.equalTo('waterway', 'riverbank')
+          equalToFilter('waterway', 'riverbank')
         )
       );
       expect(serialized).to.xmleql(_ol_xml_.parse(text));