From 2d905bd0481fafd22eda019369f30bb7c82eba22 Mon Sep 17 00:00:00 2001
From: Yohan Boniface <yohanboniface@free.fr>
Date: Tue, 24 Sep 2024 10:24:50 +0200
Subject: [PATCH] chore: use GeoRSStoGeoJSON as ES module

---
 package.json                                  |   2 +-
 umap/static/umap/js/modules/formatter.js      |   5 +-
 .../georsstogeojson/GeoRSSToGeoJSON.js        | 199 ++++++++++--------
 umap/templates/umap/js.html                   |   2 -
 4 files changed, 119 insertions(+), 89 deletions(-)

diff --git a/package.json b/package.json
index 6b1e0522..c5c4c835 100644
--- a/package.json
+++ b/package.json
@@ -43,7 +43,7 @@
     "colorbrewer": "^1.5.6",
     "csv2geojson": "5.1.2",
     "dompurify": "^3.0.11",
-    "georsstogeojson": "^0.1.0",
+    "georsstogeojson": "^0.2.0",
     "jsdom": "^24.0.0",
     "leaflet": "1.9.4",
     "leaflet-contextmenu": "^1.4.0",
diff --git a/umap/static/umap/js/modules/formatter.js b/umap/static/umap/js/modules/formatter.js
index 99677e0f..4f1059ed 100644
--- a/umap/static/umap/js/modules/formatter.js
+++ b/umap/static/umap/js/modules/formatter.js
@@ -1,4 +1,4 @@
-/* Uses globals for: csv2geojson, osmtogeojson, GeoRSSToGeoJSON (not available as ESM) */
+/* Uses globals for: csv2geojson, osmtogeojson (not available as ESM) */
 import { translate } from './i18n.js'
 
 export const EXPORT_FORMATS = {
@@ -115,7 +115,8 @@ export class Formatter {
   }
 
   async fromGeoRSS(str) {
-    return GeoRSSToGeoJSON(this.toDom(str))
+    const GeoRSSToGeoJSON = await import('../../vendors/georsstogeojson/GeoRSSToGeoJSON.js')
+    return GeoRSSToGeoJSON.parse(this.toDom(str))
   }
 
   toDom(x) {
diff --git a/umap/static/umap/vendors/georsstogeojson/GeoRSSToGeoJSON.js b/umap/static/umap/vendors/georsstogeojson/GeoRSSToGeoJSON.js
index 560f0bb5..4fa2e1bc 100644
--- a/umap/static/umap/vendors/georsstogeojson/GeoRSSToGeoJSON.js
+++ b/umap/static/umap/vendors/georsstogeojson/GeoRSSToGeoJSON.js
@@ -1,89 +1,120 @@
-var GeoRSSToGeoJSON = function (dom, options) {
+export function parse(dom, options) {
+  const g = {
+    type: 'FeatureCollection',
+    features: [],
+  }
 
-    function get(x, y) { return x.getElementsByTagName(y); }
-    function get1(x, y) { var n = get(x, y); return n.length ? n[0] : null; }
-    function norm(el) { if (el.normalize) { el.normalize(); } return el; }
-    function nodeVal(x) { if (x) {norm(x);} return x && x.firstChild && x.firstChild.nodeValue; }
-    function attr(x, y) { return x.getAttribute(y); }
-
-    var g = {
-        type: 'FeatureCollection',
-        features: []
-    };
-
-    function geom (node) {
-
-        function p(c) {return parseFloat(c);}
-        function r(c) {return c.reverse().map(p);}  // we have latlon we want lonlat
-        function e(f) {var _=[]; for (var i=0; i<f.length; i+=2) {_.push(r(f.slice(i, i+2)));} return _;}
-
-        var type, coordinates;
-
-        NODE = node;
-        if (get1(node, 'geo:long')) {
-            type = 'Point';
-            coordinates = [p(nodeVal(get1(node, 'geo:long'))), p(nodeVal(get1(node, 'geo:lat')))];
-        } else if (get1(node, 'long')) {
-            type = 'Point';
-            coordinates = [p(nodeVal(get1(node, 'long'))), p(nodeVal(get1(node, 'lat')))];
-        } else if (get1(node, 'georss:point')) {
-            type = 'Point';
-            coordinates = r(nodeVal(get1(node, 'georss:point')).split(' '));
-        } else if (get1(node, 'point')) {
-            type = 'Point';
-            coordinates = r(nodeVal(get1(node, 'point')).split(' '));
-        } else {
-            var line = get1(node, 'georss:line'),
-                poly = get1(node, 'georss:polygon');
-            if (line || poly) {
-                type = line ? 'LineString' : 'Polygon';
-                var tag = line ? 'georss:line' : 'georss:polygon';
-                coordinates = nodeVal(get1(node, tag)).split(' ');
-                if (coordinates.length % 2 !== 0) return;
-                coordinates = e(coordinates);
-                if (poly) {
-                    coordinates = [coordinates];
-                }
-            }
-        }
-        if (type && coordinates) {
-            return {
-                type: type,
-                coordinates: coordinates
-            };
-        }
+  const items = get(dom, 'item')
+  for (const item of Array.from(items)) {
+    const feature = processOne(item)
+    if (feature) {
+      g.features.push(feature)
     }
+  }
+  return g
+}
 
-    function processOne (node) {
-        var geometry = geom(node);
-        // TODO collect and fire errors
-        if (!geometry) return;
-        var f = {
-            type: "Feature",
-            geometry: geometry,
-            properties: {
-                title: nodeVal(get1(node, 'title')),
-                description: nodeVal(get1(node, 'description')),
-                link: nodeVal(get1(node, 'link')),
-            }
-        };
-        var media = get1(node, 'media:content'), mime;
-        if (!media) {
-            media = get1(node, 'enclosure'), mime;
-        }
-        if (media) {
-            mime = attr(media, 'type');
-            if (mime.indexOf('image') !== -1) {
-                f.properties.img = attr(media, "url");  // How not to invent a key?
-            }
-        }
-        g.features.push(f);
-    }
+function get(x, y) {
+  return x.getElementsByTagName(y)
+}
+function get1(x, y) {
+  const n = get(x, y)
+  return n.length ? n[0] : null
+}
+function norm(el) {
+  if (el.normalize) {
+    el.normalize()
+  }
+  return el
+}
+function nodeVal(x) {
+  if (x) {
+    norm(x)
+  }
+  return x?.firstChild?.nodeValue
+}
+function attr(x, y) {
+  return x.getAttribute(y)
+}
 
-    var items = get(dom, 'item');
-    for (var i = 0; i < items.length; i++) {
-        processOne(items[i]);
+function geom(node) {
+  function p(c) {
+    return parseFloat(c)
+  }
+  function r(c) {
+    return c.reverse().map(p)
+  } // we have latlon we want lonlat
+  function e(f) {
+    const _ = []
+    for (let i = 0; i < f.length; i += 2) {
+      _.push(r(f.slice(i, i + 2)))
     }
-    return g;
-};
-if (typeof module !== 'undefined') module.exports = {GeoRSSToGeoJSON: GeoRSSToGeoJSON};
+    return _
+  }
+
+  let type
+  let coordinates
+
+  if (get1(node, 'geo:long')) {
+    type = 'Point'
+    coordinates = [
+      p(nodeVal(get1(node, 'geo:long'))),
+      p(nodeVal(get1(node, 'geo:lat'))),
+    ]
+  } else if (get1(node, 'long')) {
+    type = 'Point'
+    coordinates = [p(nodeVal(get1(node, 'long'))), p(nodeVal(get1(node, 'lat')))]
+  } else if (get1(node, 'georss:point')) {
+    type = 'Point'
+    coordinates = r(nodeVal(get1(node, 'georss:point')).split(' '))
+  } else if (get1(node, 'point')) {
+    type = 'Point'
+    coordinates = r(nodeVal(get1(node, 'point')).split(' '))
+  } else {
+    const line = get1(node, 'georss:line')
+    const poly = get1(node, 'georss:polygon')
+    if (line || poly) {
+      type = line ? 'LineString' : 'Polygon'
+      const tag = line ? 'georss:line' : 'georss:polygon'
+      coordinates = nodeVal(get1(node, tag)).split(' ')
+      if (coordinates.length % 2 !== 0) return
+      coordinates = e(coordinates)
+      if (poly) {
+        coordinates = [coordinates]
+      }
+    }
+  }
+  if (type && coordinates) {
+    return {
+      type: type,
+      coordinates: coordinates,
+    }
+  }
+}
+
+function processOne(node) {
+  const geometry = geom(node)
+  // TODO collect and fire errors
+  if (!geometry) return
+  const f = {
+    type: 'Feature',
+    geometry: geometry,
+    properties: {
+      title: nodeVal(get1(node, 'title')),
+      description: nodeVal(get1(node, 'description')),
+      link: nodeVal(get1(node, 'link')),
+    },
+  }
+  let media = get1(node, 'media:content')
+  let mime
+  if (!media) {
+    media = get1(node, 'enclosure')
+  }
+  if (media) {
+    mime = attr(media, 'type')
+    if (mime.indexOf('image') !== -1) {
+      f.properties.img = attr(media, 'url') // How not to invent a key?
+    }
+  }
+  return f
+}
diff --git a/umap/templates/umap/js.html b/umap/templates/umap/js.html
index 9f36ea47..9fe0ab55 100644
--- a/umap/templates/umap/js.html
+++ b/umap/templates/umap/js.html
@@ -28,8 +28,6 @@
 <script src="{% static 'umap/vendors/contextmenu/leaflet.contextmenu.min.js' %}"
         defer></script>
 <script src="{% static 'umap/vendors/photon/leaflet.photon.js' %}" defer></script>
-<script src="{% static 'umap/vendors/georsstogeojson/GeoRSSToGeoJSON.js' %}"
-        defer></script>
 <script src="{% static 'umap/vendors/fullscreen/Leaflet.fullscreen.min.js' %}"
         defer></script>
 <script src="{% static 'umap/vendors/toolbar/leaflet.toolbar.js' %}" defer></script>