From a33356c8bea1f30560d61229e6583a44b78564a5 Mon Sep 17 00:00:00 2001 From: Tony Mack Date: Thu, 23 Jun 2011 19:34:41 -0400 Subject: [PATCH] dont include the location tag if longitude or latitude is null --- sfa/rspecs/pg_rspec.py | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/sfa/rspecs/pg_rspec.py b/sfa/rspecs/pg_rspec.py index 0752974e..cc73047d 100755 --- a/sfa/rspecs/pg_rspec.py +++ b/sfa/rspecs/pg_rspec.py @@ -121,14 +121,15 @@ class PGRSpec(RSpec): node_type_tag = etree.SubElement(node_tag, 'hardware_type', name='plab-pc') node_type_tag = etree.SubElement(node_tag, 'hardware_type', name='pc') available_tag = etree.SubElement(node_tag, 'available', now='true') - location_tag = etree.SubElement(node_tag, 'location', country="us") + + # only add location tag if longitude and latitude are not null if 'site' in node: - if 'longitude' in node['site']: - location_tag.set('longitude', str(node['site']['longitude'])) - if 'latitude' in node['site']: - location_tag.set('latitude', str(node['site']['latitude'])) - #if 'interfaces' in node: - + longitude = node['site'].get('longitude', None) + latitude = node['site'].get('latitude', None) + if longitude and latitude: + location_tag = etree.SubElement(node_tag, 'location', country="us", \ + longitude=longitude, latitude=latitude) + def add_slivers(self, slivers, sliver_urn=None, no_dupes=False): slivers = self._process_slivers(slivers) -- 2.47.0