From 28a58cb30aa9b552dbbf17f777e0aec7d5174eb6 Mon Sep 17 00:00:00 2001 From: Yasin Date: Fri, 23 Jan 2015 17:15:26 +0100 Subject: [PATCH] Various improvements --- portal/actions.py | 48 +++++++++++-------- portal/templates/base.html | 3 ++ .../onelab/onelab_widget-topmenu.html | 3 -- 3 files changed, 31 insertions(+), 23 deletions(-) diff --git a/portal/actions.py b/portal/actions.py index 8965a9fe..c80ba652 100644 --- a/portal/actions.py +++ b/portal/actions.py @@ -80,30 +80,38 @@ def authority_check_pis(request, user_email): def authority_add_pis(request, authority_hrn,user_hrn): - - # getting pis of the authority of the user - pis = authority_get_pis (request, authority_hrn) - for pi in pis: - pi_list = pi['pi_users'] + try: + # getting pis of the authority of the user + pis = authority_get_pis (request, authority_hrn) + for pi in pis: + pi_list = pi['pi_users'] - updated_pi_list = pi_list.append(user_hrn) - query = Query.update('authority').filter_by('authority_hrn', '==', authority_hrn).set({'pi_users':pi_list}) - results = execute_admin_query(request,query) - newpis = authority_get_pis (request, authority_hrn) - return newpis + updated_pi_list = pi_list.append(user_hrn) + query = Query.update('authority').filter_by('authority_hrn', '==', authority_hrn).set({'pi_users':pi_list}) + results = execute_admin_query(request,query) + newpis = authority_get_pis (request, authority_hrn) + return newpis + except Exception,e: + print "Exception in actions.py in authority_add_pis %s" % e + return None -def authority_remove_pis(request, authority_hrn,user_hrn): - # getting pis of the authority of the user - pis = authority_get_pis (request, authority_hrn) - for pi in pis: - pi_list = pi['pi_users'] +def authority_remove_pis(request, authority_hrn,user_hrn): + try: + # getting pis of the authority of the user + pis = authority_get_pis (request, authority_hrn) + for pi in pis: + pi_list = pi['pi_users'] - updated_pi_list = pi_list.remove(user_hrn) - query = Query.update('authority').filter_by('authority_hrn', '==', authority_hrn).set({'pi_users':pi_list}) - results = execute_admin_query(request,query) - newpis = authority_get_pis (request, authority_hrn) - return newpis + updated_pi_list = pi_list.remove(user_hrn) + query = Query.update('authority').filter_by('authority_hrn', '==', authority_hrn).set({'pi_users':pi_list}) + results = execute_admin_query(request,query) + newpis = authority_get_pis (request, authority_hrn) + return newpis + except Exception,e: + print "Exception in actions.py in authority_remove_pis %s" % e + return None + def authority_get_pi_emails(request, authority_hrn): diff --git a/portal/templates/base.html b/portal/templates/base.html index decc1b38..4e2c5064 100644 --- a/portal/templates/base.html +++ b/portal/templates/base.html @@ -97,6 +97,9 @@ $(document).ready(function() { +

+{% widget '_widget-message.html' %} +

{% block container %} {% block topmenu %} {% widget "_widget-topmenu.html" %} diff --git a/portal/templates/onelab/onelab_widget-topmenu.html b/portal/templates/onelab/onelab_widget-topmenu.html index 10fbbe13..668c3d32 100644 --- a/portal/templates/onelab/onelab_widget-topmenu.html +++ b/portal/templates/onelab/onelab_widget-topmenu.html @@ -44,9 +44,6 @@ -

-{% widget '_widget-message.html' %} -