replaced 3 xml packages with xml-utils

DanielJDufour · DanielJDufour · commit d54bee2ad1d3 · 2021-04-03T17:08:24.000-04:00
diff --git a/package.json b/package.json
@@ -44,15 +44,13 @@
     "georaster-to-canvas": "0.2.0",
     "geotiff": "1.0.0-beta.13",
     "geotiff-palette": "0.0.0",
-    "simple-xml-dom": "1.0.0",
     "threads": "^1.4.0",
     "tiny-worker": "^2.3.0",
     "ts-node": "^8.8.2",
     "txml": "3.1.2",
     "underscore": "^1.8.3",
     "worker-loader": "^2.0.0",
-    "xmldom": "0.1.27",
-    "xpath": "0.0.27"
+    "xml-utils": "^0.2.0"
   },
   "devDependencies": {
     "@babel/types": "^7.9.5",
@@ -65,7 +63,6 @@
     "eslint": "^5.7.0",
     "eslint-config-google": "^0.11.0",
     "mocha": "^6.2.0",
-    "replace-in-files-cli": "^0.3.1",
     "threads-plugin": "^1.3.1",
     "webpack": "^4.12.0",
     "webpack-bundle-analyzer": "^3.6.0",
diff --git a/src/parse_metadata.js b/src/parse_metadata.js
@@ -1,33 +1,24 @@
-const xml = require('simple-xml-dom');
-const xpath = require('xpath');
+const findTagByPath = require('xml-utils/src/find-tag-by-path');
+
 
 const parseISO = metadata => {
   const results = {};
 
-  const dom = xml.parse(metadata);
-
-  const select = xpath.useNamespaces({
-    gco: 'http://www.isotc211.org/2005/gco',
-    gmd: 'http://www.isotc211.org/2005/gmd',
-  });
-
   try {
-    results.projection = parseFloat(select('string(//gmd:RS_Identifier//gmd:code//gco:CharacterString/text())', dom));
+    results.projection = parseFloat(findTagByPath(metadata, ['gmd:RS_Identifier', 'gmd:code', 'gco:CharacterString']).inner);
   } catch (error) {
     console.error(error);
   }
 
   try {
-    results.xmin = parseFloat(select('string(//gmd:westBoundLongitude//gco:Decimal/text())', dom));
-    results.xmax = parseFloat(select('string(//gmd:eastBoundLongitude//gco:Decimal/text())', dom));
-    results.ymin = parseFloat(select('string(//gmd:southBoundLatitude//gco:Decimal/text())', dom));
-    results.ymax = parseFloat(select('string(//gmd:northBoundLatitude//gco:Decimal/text())', dom));
+    results.xmin = parseFloat(findTagByPath(metadata, ['gmd:westBoundLongitude', 'gco:Decimal']).inner);
+    results.xmax = parseFloat(findTagByPath(metadata, ['gmd:eastBoundLongitude', 'gco:Decimal']).inner);
+    results.ymin = parseFloat(findTagByPath(metadata, ['gmd:southBoundLatitude', 'gco:Decimal']).inner);
+    results.ymax = parseFloat(findTagByPath(metadata, ['gmd:northBoundLatitude', 'gco:Decimal']).inner);
   } catch (error) {
     console.error(error);
   }
 
-  console.log('results:', results);
-
   return results;
 };
 
diff --git a/test/index.html b/test/index.html
@@ -1,7 +1,7 @@
 <!DOCTYPE html>
 <html>
   <head>
-    <script src="https://unpkg.com/georaster"></script>
+    <script src="../dist/georaster.browser.bundle.js"></script>
   </head>
   <body>
     <div id="container"></div>
diff --git a/test/test.js b/test/test.js
@@ -7,6 +7,7 @@ let parseMetadata = require('../src/parse_metadata.js');
 let parseISO = parseMetadata.parseISO;
 let countIn2D = require('../src/utils.js').countIn2D;
 
+
 describe('Parsing Data Object', function() {
    describe('Parsing Simple Examples', function() {
       it('should create raster correctly', function(done) {
@@ -76,7 +77,6 @@ describe('Checking Error Catching', function() {
   });
 });
 
-
 describe('Parsing Metadata', function() {
   describe('if you pass in iso xml text', function() {
     it('should parse metadata', function(done) {
@@ -93,7 +93,6 @@ describe('Parsing Metadata', function() {
   });
 });
 
-
 // Using tiff created from http://geomap.arpa.veneto.it/geoserver/wcs?crs=EPSG%3A4326&service=WCS&format=GeoTIFF&request=GetCoverage&height=329&width=368&version=1.0.0&BBox=9.679858245722988%2C13.951082737884812%2C44.183855724634675%2C47.38727409375604&Coverage=geonode%3Aatlanteil
 describe('Parsing Geonode Files', function() {
   describe('if you pass in tiff from geoserver', function() {
