pelias-openstreetmap 7.3.0 → 7.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pelias-openstreetmap",
-  "version": "7.3.0",
+  "version": "7.5.0",
   "engines": {
     "node": ">=10.0.0"
   },

package/schema/address_karlsruhe.js

@@ -10,7 +10,7 @@
   @ref: http://wiki.openstreetmap.org/wiki/Karlsruhe_Schema
 **/
 
-var KARLSRUHE_SCHEMA = {
+const KARLSRUHE_SCHEMA = {
   'addr:housename':     'name',
   'addr:housenumber':   'number',
   'addr:street':        'street',

package/schema/address_naptan.js

@@ -10,7 +10,7 @@
   @ref: http://wiki.openstreetmap.org/wiki/NaPTAN
 **/
 
-var NAPTAN_SCHEMA = {
+const NAPTAN_SCHEMA = {
   'naptan:Street': 'street'
 };
 

package/schema/address_osm.js

@@ -10,7 +10,7 @@
   @ref: http://wiki.openstreetmap.org/wiki/Key:postal_code
 **/
 
-var OSM_SCHEMA = {
+const OSM_SCHEMA = {
   'postal_code': 'zip'
 };
 

package/schema/address_tiger.js

@@ -10,7 +10,7 @@
   @ref: http://wiki.openstreetmap.org/wiki/TIGER_to_OSM_Attribute_Map
 **/
 
-var TIGER_SCHEMA = {
+const TIGER_SCHEMA = {
   'tiger:zip_left': 'zip',
   'tiger:zip_right': 'zip'
 };

package/schema/name_osm.js

@@ -14,16 +14,13 @@
   When multiple keys have the value 'default' then they are considered
   as aliases of the default field.
 
-  The '_primary' property defined below defines which of those aliases
-  is considered the 'primary default name' for label generation.
-
   No values other than 'default' should be specified more than once.
 
   @ref: http://wiki.openstreetmap.org/wiki/Key:name
   @ref: http://wiki.openstreetmap.org/wiki/Names
 **/
 
-var OSM_NAMING_SCHEMA = {
+const OSM_NAMING_SCHEMA = {
   'name':             'default',
   'loc_name':         'default',
   'alt_name':         'default',
@@ -38,14 +35,4 @@ var OSM_NAMING_SCHEMA = {
   // 'sorting_name':     'sorting'
 };
 
-// this property is considered the 'primary name'
-// for label generation, the others are considered
-// secondary or 'aliases'.
-Object.defineProperty(OSM_NAMING_SCHEMA, '_primary', {
-  value: 'name',
-  enumerable: false,
-  configurable: false,
-  writable: false
-});
-
 module.exports = OSM_NAMING_SCHEMA;

package/stream/address_extractor.js

@@ -23,11 +23,12 @@
   not searchable.
 **/
 
-var through = require('through2');
-var isObject = require('is-object');
-var extend = require('extend');
-var peliasLogger = require( 'pelias-logger' ).get( 'openstreetmap' );
-var Document = require('pelias-model').Document;
+const through = require('through2');
+const isObject = require('is-object');
+const extend = require('extend');
+const peliasLogger = require( 'pelias-logger' ).get( 'openstreetmap' );
+const Document = require('pelias-model').Document;
+const parseSemicolonDelimitedValues = require('../util/parseSemicolonDelimitedValues');
 
 function hasValidAddress( doc ){
   if( !isObject( doc ) ){ return false; }
@@ -42,21 +43,20 @@ function hasValidAddress( doc ){
 module.exports = function(){
 
   var stream = through.obj( function( doc, enc, next ) {
-    var isNamedPoi = !!doc.getName('default');
-    var isAddress = hasValidAddress( doc );
+    const isNamedPoi = !!doc.getName('default');
+    const isAddress = hasValidAddress( doc );
 
     // accept semi-colon delimited house numbers
     // ref: https://github.com/pelias/openstreetmap/issues/21
-    var streetNumbers = (doc.getAddress('number') || '').split(';').map(Function.prototype.call, String.prototype.trim);
+    const streetNumbers = parseSemicolonDelimitedValues(doc.getAddress('number'));
 
     // create a new record for street addresses
     if( isAddress ){
-      var record;
-
       streetNumbers.forEach( function( streetno, i ){
+        let record;
 
         try {
-          var newid = [ doc.getSourceId() ];
+          const newid = [ doc.getSourceId() ];
           if( i > 0 ){
             newid.push( i );
             peliasLogger.debug('[address_extractor] found multiple house numbers: ', streetNumbers);
@@ -116,7 +116,7 @@ module.exports = function(){
 };
 
 // properties to map from the osm record to the pelias doc
-var addrProps = [ 'name', 'number', 'street', 'zip' ];
+const addrProps = [ 'name', 'number', 'street', 'zip' ];
 
 // call document setters and ignore non-fatal errors
 function setProperties( record, doc ){

package/stream/tag_mapper.js

@@ -7,10 +7,11 @@
 const _ = require('lodash');
 const through = require('through2');
 const peliasLogger = require('pelias-logger').get('openstreetmap');
+const parseSemicolonDelimitedValues = require('../util/parseSemicolonDelimitedValues');
 
-var LOCALIZED_NAME_KEYS = require('../config/localized_name_keys');
-var NAME_SCHEMA = require('../schema/name_osm');
-var ADDRESS_SCHEMA = _.merge( {},
+const LOCALIZED_NAME_KEYS = require('../config/localized_name_keys');
+const NAME_SCHEMA = require('../schema/name_osm');
+const ADDRESS_SCHEMA = _.merge( {},
   require('../schema/address_tiger'),
   require('../schema/address_osm'),
   require('../schema/address_naptan'),
@@ -19,7 +20,7 @@ var ADDRESS_SCHEMA = _.merge( {},
 
 module.exports = function(){
 
-  var stream = through.obj( function( doc, enc, next ) {
+  const stream = through.obj( function( doc, enc, next ) {
 
     try {
 
@@ -35,34 +36,39 @@ module.exports = function(){
 
         // Map localized names which begin with 'name:'
         // @ref: http://wiki.openstreetmap.org/wiki/Namespace#Language_code_suffix
-        var suffix = getNameSuffix( key );
-        if( suffix ){
-          var val1 = trim( value );
-          if( val1 ){
-            doc.setName( suffix, val1 );
-          }
+        const langCode = getNameSuffix( key );
+        if( langCode ){
+          const langValues = parseSemicolonDelimitedValues( value );
+          langValues.forEach(( langValue, i ) => {
+            if ( i === 0 ) {
+              doc.setName( langCode, langValue );
+            } else {
+              doc.setNameAlias( langCode, langValue );
+            }
+          });
         }
 
         // Map name data from our name mapping schema
         else if( _.has(NAME_SCHEMA, key) ){
-          var val2 = trim( value );
-          if( val2 ){
-            if( key === NAME_SCHEMA._primary ){
-              doc.setName( NAME_SCHEMA[key], val2 );
-            } else if ( 'default' === NAME_SCHEMA[key] ) {
-              doc.setNameAlias( NAME_SCHEMA[key], val2 );
-            } else {
-              doc.setName( NAME_SCHEMA[key], val2 );
+          const nameValues = parseSemicolonDelimitedValues( cleanString( value ) );
+          nameValues.forEach(( nameValue, i ) => {
+            // For the primary name key 'name', ensure it is the first value
+            if( 'name' === key && i === 0 ){
+              doc.setName(NAME_SCHEMA[key], nameValue);
+              return;
             }
-          }
+
+            // Otherwise set as an alias
+            doc.setNameAlias( NAME_SCHEMA[key], nameValue );
+          });
         }
 
         // Map address data from our address mapping schema
         else if( _.has(ADDRESS_SCHEMA, key) ){
-          var val3 = trim( value );
-          if( val3 ){
-            let label = ADDRESS_SCHEMA[key];
-            doc.setAddress(label, normalizeAddressField(label, val3));
+          const addrValue = cleanString( value );
+          if( addrValue ){
+            const label = ADDRESS_SCHEMA[key];
+            doc.setAddress(label, normalizeAddressField(label, addrValue));
           }
         }
       });
@@ -71,21 +77,22 @@ module.exports = function(){
       // other names which we could use as the default.
       if( !doc.getName('default') ){
 
-        var defaultName =
-          _.get(tags, 'official_name') ||
-          _.get(tags, 'int_name') ||
-          _.get(tags, 'nat_name') ||
-          _.get(tags, 'reg_name') ||
-          doc.getName('en');
+        const defaultName = [
+          ...parseSemicolonDelimitedValues(_.get(tags, 'official_name')),
+          ...parseSemicolonDelimitedValues(_.get(tags, 'int_name')),
+          ...parseSemicolonDelimitedValues(_.get(tags, 'nat_name')),
+          ...parseSemicolonDelimitedValues(_.get(tags, 'reg_name')),
+          ...parseSemicolonDelimitedValues(doc.getName('en'))
+        ].filter(Boolean);
 
         // use one of the preferred name tags listed above
-        if ( defaultName ){
-          doc.setName('default', defaultName);
+        if ( defaultName.length ){
+          doc.setName('default', defaultName[0]);
         }
 
         // else try to use an available two-letter language name tag
         else {
-          var keys = Object.keys(doc.name).filter(n => n.length === 2);
+          const keys = Object.keys(doc.name).filter(n => n.length === 2);
 
           // unambiguous (there is only a single two-letter name tag)
           if ( keys.length === 1 ){
@@ -101,7 +108,7 @@ module.exports = function(){
       // Import airport codes as aliases
       if( tags.hasOwnProperty('aerodrome') || tags.hasOwnProperty('aeroway') ){
         if( tags.hasOwnProperty('iata') ){
-          var iata = trim( tags.iata );
+          const iata = cleanString( tags.iata );
           if( iata ){
             doc.setNameAlias( 'default', iata );
             doc.setNameAlias( 'default', `${iata} Airport` );
@@ -127,7 +134,7 @@ module.exports = function(){
 };
 
 // Clean string of leading/trailing junk chars
-function trim( str ){
+function cleanString( str ){
   return _.trim( str, '#$%^*<>-=_{};:",./?\t\n\' ' );
 }
 

package/util/parseSemicolonDelimitedValues.js

@@ -0,0 +1,14 @@
+const _ = require('lodash');
+
+// Split multi-value OSM tags into an Array
+// https://wiki.openstreetmap.org/wiki/Talk:Semi-colon_value_separator
+function parseSemicolonDelimitedValues(value) {
+  return (_.isString(value) ? value : '')
+    .split(';')
+    // Historical Landmark: former site of The Most Clever Line of JavaScript
+    // https://blog.bloomca.me/2017/11/08/the-most-clever-line-of-javascript.html
+    .map(v => v.trim())
+    .filter(v => v.length);
+}
+
+module.exports = parseSemicolonDelimitedValues;