change admin property names to <place_type>_gid from <place_type>_id

helper/geojsonify.js

@@ -4,6 +4,7 @@ var GeoJSON = require('geojson'),
     labelGenerator = require('./labelGenerator'),
     logger = require('pelias-logger').get('api'),
     type_mapping = require('./type_mapping'),
+    Document = require('pelias-model').Document,
     _ = require('lodash');
 
 // Properties to be copied
@@ -14,28 +15,28 @@ var DETAILS_PROPS = [
   'confidence',
   'distance',
   'country',
-  'country_id',
+  'country_gid',
   'country_a',
   'macroregion',
-  'macroregion_id',
+  'macroregion_gid',
   'macroregion_a',
   'region',
-  'region_id',
+  'region_gid',
   'region_a',
   'macrocounty',
-  'macrocounty_id',
+  'macrocounty_gid',
   'macrocounty_a',
   'county',
-  'county_id',
+  'county_gid',
   'county_a',
   'localadmin',
-  'localadmin_id',
+  'localadmin_gid',
   'localadmin_a',
   'locality',
-  'locality_id',
+  'locality_gid',
   'locality_a',
   'neighbourhood',
-  'neighbourhood_id',
+  'neighbourhood_gid',
   'bounding_box'
 ];
 
@@ -208,7 +209,8 @@ function copyProperties( source, props, dst ) {
  * @param {object} src
  */
 function makeGid(src) {
-  return lookupSource(src) + ':' + lookupLayer(src) + ':' + src._id;
+  var doc = new Document(lookupSource(src), lookupLayer(src), src._id);
+  return doc.getGid();
 }
 
 /**

helper/placeTypes.js

@@ -0,0 +1,11 @@
+module.exports = [
+  'country',
+  'macroregion',
+  'region',
+  'macrocounty',
+  'county',
+  'localadmin',
+  'locality',
+  'borough',
+  'neighbourhood'
+];

middleware/normalizeParentIds.js

@@ -1,17 +1,7 @@
 var logger = require('pelias-logger').get('api');
 var Document = require('pelias-model').Document;
 
-var placeTypes = [
-  'country',
-  'macroregion',
-  'region',
-  'macrocounty',
-  'county',
-  'localadmin',
-  'locality',
-  'neighbourhood',
-  'borough'
-];
+var placeTypes = require('../helper/placeTypes');
 
 /**
  * Convert WOF integer ids to Pelias formatted ids that can be used by the /place endpoint.
@@ -42,7 +32,7 @@ function normalizeParentIds(place) {
   if (place) {
     placeTypes.forEach(function (placeType) {
       if (place[placeType] && place[placeType].length > 0 && place[placeType][0]) {
-        place[placeType + '_id'] = [ makeNewId(placeType, place[placeType + '_id']) ];
+        place[placeType + '_gid'] = [ makeNewId(placeType, place[placeType + '_gid']) ];
       }
     });
   }

middleware/renamePlacenames.js

@@ -1,16 +1,6 @@
 var _ = require('lodash');
 
-/**
- - P is a preferred English name
- - Q is a preferred name (in other languages)
- - V is a well-known (but unofficial) variant for the place
- (e.g. "New York City" for New York)
- - S is either a synonym or a colloquial name for the place
- (e.g. "Big Apple" for New York), or a version of the name which
- is stripped of accent characters.
- - A is an abbreviation or code for the place (e.g. "NYC" for New
- York)
- */
+var PARENT_PROPS = require('../helper/placeTypes');
 
 var ADDRESS_PROPS = {
   'number': 'housenumber',
@@ -18,32 +8,6 @@ var ADDRESS_PROPS = {
   'street': 'street'
 };
 
-var PARENT_PROPS = [
-  'country',
-  'country_id',
-  'country_a',
-  'macroregion',
-  'macroregion_id',
-  'macroregion_a',
-  'region',
-  'region_id',
-  'region_a',
-  'macrocounty',
-  'macrocounty_id',
-  'macrocounty_a',
-  'county',
-  'county_id',
-  'county_a',
-  'localadmin',
-  'localadmin_id',
-  'localadmin_a',
-  'locality',
-  'locality_id',
-  'locality_a',
-  'neighbourhood',
-  'neighbourhood_id'
-];
-
 
 function setup() {
   return renamePlacenames;
@@ -74,6 +38,8 @@ function renameOneRecord(place) {
   if (place.parent) {
     PARENT_PROPS.forEach(function (prop) {
       place[prop] = place.parent[prop];
+      place[prop + '_a'] = place.parent[prop + '_a'];
+      place[prop + '_gid'] = place.parent[prop + '_id'];
     });
   }
 

test/unit/helper/geojsonify.js

@@ -17,6 +17,8 @@ module.exports.tests.earth = function(test, common) {
   var earth = [{
     '_type': 'geoname',
     '_id': '6295630',
+    'source': 'whosonfirst',
+    'layer': 'continent',
     'name': {
       'default': 'Earth'
     },
@@ -240,7 +242,7 @@ module.exports.tests.search = function(test, common) {
         'country': [
           'United States'
         ],
-        'country_id': [
+        'country_gid': [
           '85633793'
         ],
         'country_a': [
@@ -249,7 +251,7 @@ module.exports.tests.search = function(test, common) {
         'macroregion': [
           'MacroRegion Name'
         ],
-        'macroregion_id': [
+        'macroregion_gid': [
           'MacroRegion Id'
         ],
         'macroregion_a': [
@@ -258,7 +260,7 @@ module.exports.tests.search = function(test, common) {
         'region': [
           'New York'
         ],
-        'region_id': [
+        'region_gid': [
           '85688543'
         ],
         'region_a': [
@@ -267,7 +269,7 @@ module.exports.tests.search = function(test, common) {
         'macrocounty': [
           'MacroCounty Name'
         ],
-        'macrocounty_id': [
+        'macrocounty_gid': [
           'MacroCounty Id'
         ],
         'macrocounty_a': [
@@ -276,7 +278,7 @@ module.exports.tests.search = function(test, common) {
         'county': [
           'Kings County'
         ],
-        'county_id': [
+        'county_gid': [
           '102082361'
         ],
         'county_a': [
@@ -285,20 +287,20 @@ module.exports.tests.search = function(test, common) {
         'localadmin': [
           'Brooklyn'
         ],
-        'localadmin_id': [
+        'localadmin_gid': [
           '404521211'
         ],
         'localadmin_a': [
           null
         ],
-        'locality_id': [
+        'locality_gid': [
           '85977539'
         ],
         'locality_a': [
           null
         ],
         'neighbourhood': [],
-        'neighbourhood_id': []
+        'neighbourhood_gid': []
       }
     ];
 
@@ -316,23 +318,23 @@ module.exports.tests.search = function(test, common) {
             'name': 'East New York',
             'confidence': 0.888,
             'country': 'United States',
-            'country_id': '85633793',
+            'country_gid': '85633793',
             'country_a': 'USA',
             'macroregion': 'MacroRegion Name',
-            'macroregion_id': 'MacroRegion Id',
+            'macroregion_gid': 'MacroRegion Id',
             'macroregion_a': 'MacroRegion Abbreviation',
             'region': 'New York',
-            'region_id': '85688543',
+            'region_gid': '85688543',
             'region_a': 'NY',
             'macrocounty': 'MacroCounty Name',
-            'macrocounty_id': 'MacroCounty Id',
+            'macrocounty_gid': 'MacroCounty Id',
             'macrocounty_a': 'MacroCounty Abbreviation',
             'county': 'Kings County',
-            'county_id': '102082361',
+            'county_gid': '102082361',
             'localadmin': 'Brooklyn',
-            'localadmin_id': '404521211',
+            'localadmin_gid': '404521211',
             'locality': 'New York',
-            'locality_id': '85977539',
+            'locality_gid': '85977539',
             'bounding_box': {
               'min_lat': 40.6514712164,
               'max_lat': 40.6737320588,

test/unit/middleware/normalizeParentIds.js

@@ -13,28 +13,28 @@ module.exports.tests.interface = function(test, common) {
           'country_a': ['USA']
         },
         'country': ['United States'],
-        'country_id': ['85633793'],
+        'country_gid': ['85633793'],
         'country_a': ['USA'],
         'macroregion': ['MacroRegion Name'],
-        'macroregion_id': ['foobar'],
+        'macroregion_gid': ['foobar'],
         'macroregion_a': ['MacroRegion Abbreviation'],
         'region': ['New York'],
-        'region_id': ['85688543'],
+        'region_gid': ['85688543'],
         'region_a': ['NY'],
         'macrocounty': ['MacroCounty Name'],
-        'macrocounty_id': ['~~~~~'],
+        'macrocounty_gid': ['~~~~~'],
         'macrocounty_a': ['MacroCounty Abbreviation'],
         'county': ['Kings County'],
-        'county_id': ['102082361'],
+        'county_gid': ['102082361'],
         'county_a': [null],
         'localadmin': ['Brooklyn'],
-        'localadmin_id': ['404521211'],
+        'localadmin_gid': ['404521211'],
         'localadmin_a': [null],
         'locality': ['Some Locality'],
-        'locality_id': ['85977539'],
+        'locality_gid': ['85977539'],
         'locality_a': [null],
         'neighbourhood': [],
-        'neighbourhood_id': []
+        'neighbourhood_gid': []
       }]
     };
 
@@ -46,28 +46,28 @@ module.exports.tests.interface = function(test, common) {
           'country_a': ['USA']
         },
         'country': ['United States'],
-        'country_id': ['whosonfirst:country:85633793'],
+        'country_gid': ['whosonfirst:country:85633793'],
         'country_a': ['USA'],
         'macroregion': ['MacroRegion Name'],
-        'macroregion_id': ['whosonfirst:macroregion:foobar'],
+        'macroregion_gid': ['whosonfirst:macroregion:foobar'],
         'macroregion_a': ['MacroRegion Abbreviation'],
         'region': ['New York'],
-        'region_id': ['whosonfirst:region:85688543'],
+        'region_gid': ['whosonfirst:region:85688543'],
         'region_a': ['NY'],
         'macrocounty': ['MacroCounty Name'],
-        'macrocounty_id': ['whosonfirst:macrocounty:~~~~~'],
+        'macrocounty_gid': ['whosonfirst:macrocounty:~~~~~'],
         'macrocounty_a': ['MacroCounty Abbreviation'],
         'county': ['Kings County'],
-        'county_id': ['whosonfirst:county:102082361'],
+        'county_gid': ['whosonfirst:county:102082361'],
         'county_a': [null],
         'localadmin': ['Brooklyn'],
-        'localadmin_id': ['whosonfirst:localadmin:404521211'],
+        'localadmin_gid': ['whosonfirst:localadmin:404521211'],
         'localadmin_a': [null],
         'locality': ['Some Locality'],
-        'locality_id': ['whosonfirst:locality:85977539'],
+        'locality_gid': ['whosonfirst:locality:85977539'],
         'locality_a': [null],
         'neighbourhood': [],
-        'neighbourhood_id': []
+        'neighbourhood_gid': []
       }]
     };