fields length fix for distributed themes

GEOLYTIX · Mar 4, 2024 · 508d0ba · 508d0ba
1 parent 70f7217
commit 508d0ba
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 5 deletions.
diff --git a/lib/layer/featureFields.mjs b/lib/layer/featureFields.mjs
@@ -35,7 +35,7 @@ featureFields.process(layer) will process the fields required for a theme and re
 export function process(layer) {
 
   // Check if the distribution method is defined in the distribution object.
-  if (Object.hasOwnProperty(distribution, layer.style?.theme?.distribution)) {
+  if (Object.hasOwn(distribution, layer.style?.theme?.distribution)) {
 
     // Call the corresponding distribution function.
     distribution[layer.style.theme.distribution](layer);

diff --git a/lib/layer/format/mvt.mjs b/lib/layer/format/mvt.mjs
@@ -202,9 +202,6 @@ function changeEndLoad(layer) {
     layer.style.label?.field
   ].flat().filter(field => !!field))]
 
-  // Do not query WKT without fields.
-  if (!layer.params.fields.length) return;
-
   const bounds = layer.mapview.getBounds()
 
   // Assign current viewport if queryparam is truthy.

diff --git a/mod/workspace/templates/wkt.js b/mod/workspace/templates/wkt.js
@@ -20,7 +20,7 @@ module.exports = _ => {
   return `
     SELECT
       \${qID} AS id
-      ${fields && `, ${fields.join(', ')}` || ''}
+      ${fields.length && `, ${fields.join(', ')}` || ''}
       FROM \${table}
       WHERE TRUE ${where} \${filter};`
 }