From: Tony Mack <tmack@paris.CS.Princeton.EDU>
Date: Fri, 22 Jun 2012 00:52:54 +0000 (-0400)
Subject: cannot assume all SliverStatus results have the pl_login field
X-Git-Tag: sfa-2.1-12~44
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=749f786e7f33f0fc7315bb1e83578366e0aedf8f;p=sfa.git

cannot assume all SliverStatus results have the pl_login field
---

diff --git a/sfa/managers/slice_manager.py b/sfa/managers/slice_manager.py
index 75dbec41..445dbebd 100644
--- a/sfa/managers/slice_manager.py
+++ b/sfa/managers/slice_manager.py
@@ -365,7 +365,11 @@ class SliceManager:
     
         # mmh, it is expected that all results carry the same urn
         overall['geni_urn'] = results[0]['geni_urn']
-        overall['pl_login'] = results[0]['pl_login']
+        overall['pl_login'] = None
+        for result in results:
+            if result.get('pl_login'):
+                overall['pl_login'] = result['pl_login']
+                break
         # append all geni_resources
         overall['geni_resources'] = \
             reduce (lambda x,y: x+y, [ result['geni_resources'] for result in results] , [])