X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=ui%2Ftopmenu.py;h=0c8d2257fa9c8d71e19c772639dbda7bd65f0c32;hb=a57786aed6a0db101b5304198282762ec4654a6b;hp=667711d05ff603509304dccd5d94442c62ae33b7;hpb=ea22db0e99695ec26e1481558af65e0a541572da;p=unfold.git

diff --git a/ui/topmenu.py b/ui/topmenu.py
index 667711d0..0c8d2257 100644
--- a/ui/topmenu.py
+++ b/ui/topmenu.py
@@ -1,5 +1,7 @@
 # a set of utilities to help make the global layout consistent across views
 
+from myslice.settings import logger
+
 def the_user (request):
     "retrieves logged in user's email, or empty string"
     if not request.user.is_authenticated (): 
@@ -22,7 +24,7 @@ def the_user (request):
 def topmenu_items_static (current, request):
     has_user=request.user.is_authenticated()
     result=[]
-    print request.user
+    logger.debug("request user = {}".format(request.user))
     if has_user:
         result.append({'label':'Dashboard', 'href': '/portal/dashboard/'})
         result.append({'label':'Request a slice', 'href': '/portal/slice_request/'})
@@ -58,7 +60,7 @@ def topmenu_items_static (current, request):
 
 # tmp - transition phase
 def topmenu_items (current, request):
-    print "WARNING -- please now use topmenu_items_live (label, page) toplevel_items is deprecated -- WARNING"
+    logger.warning("WARNING -- please now use topmenu_items_live (label, page) toplevel_items is deprecated -- WARNING")
     return topmenu_items_static (current, request)
 
 # integrated helper function for an animated menu
@@ -74,8 +76,11 @@ def topmenu_items_live (current, page):
     # We might use local storage instead
 
     # REGISTRY ONLY TO BE REMOVED WITH MANIFOLD-V2
-    query_pi_auths = Query.get('myslice:user').filter_by('user_hrn', '==', '$user_hrn' ).select('user_hrn','pi_authorities')
-    page.enqueue_query(query_pi_auths)
+    if request.user.is_authenticated ():
+        query_pi_auths = Query.get('myslice:user').filter_by('user_hrn', '==', '$user_hrn' ).select('user_hrn','pi_authorities')
+        page.enqueue_query(query_pi_auths)
+    else:
+        query_pi_auths = Query()
 #        # even though this plugin does not have any html materialization, the corresponding domid
 #        # must exist because it is searched at init-time to create the JS plugin
 #        # so we simply piggy-back the target button created in the topmenu