Browse Source

removed bookkeeping object, it was adding unneeded complexity

pull/1002/head
Stephen Hess 7 years ago
parent
commit
7eb2c68090
  1. 17
      helper/geojsonify_place_details.js

17
helper/geojsonify_place_details.js

@ -66,28 +66,23 @@ function copyProperties( params, source, dst ) {
return; return;
} }
var property = {
name: prop.name || prop,
type: prop.type || 'default'
};
var value = null; var value = null;
if ( source.hasOwnProperty( property.name ) ) { if ( source.hasOwnProperty( prop.name ) ) {
switch (property.type) { switch (_.defaultTo(prop.type, 'default')) {
case 'string': case 'string':
value = getStringValue(source[property.name]); value = getStringValue(source[prop.name]);
break; break;
case 'array': case 'array':
value = getArrayValue(source[property.name]); value = getArrayValue(source[prop.name]);
break; break;
// default behavior is to copy property exactly as is // default behavior is to copy property exactly as is
default: default:
value = source[property.name]; value = source[prop.name];
} }
if (_.isNumber(value) || (value && !_.isEmpty(value))) { if (_.isNumber(value) || (value && !_.isEmpty(value))) {
dst[property.name] = value; dst[prop.name] = value;
} }
} }
}); });

Loading…
Cancel
Save