From: Loic Baron Date: Tue, 21 Apr 2015 15:02:46 +0000 (+0200) Subject: request.session.user.pi to know if the user is_pi not to Query everytime X-Git-Tag: myslice-1.3~35^2~1^2~2 X-Git-Url: http://git.onelab.eu/?p=myslice.git;a=commitdiff_plain;h=81e73becc2f47d96de6cf9f94ba4c86f710a18ba request.session.user.pi to know if the user is_pi not to Query everytime --- diff --git a/localauth/manifoldbackend.py b/localauth/manifoldbackend.py index e52b1753..74511a10 100644 --- a/localauth/manifoldbackend.py +++ b/localauth/manifoldbackend.py @@ -7,6 +7,8 @@ from manifold.core.query import Query from myslice.settings import config, logger, DEBUG +from portal.actions import authority_check_pis + # from unfold.sessioncache import SessionCache # Name my backend 'ManifoldBackend' @@ -78,6 +80,8 @@ class ManifoldBackend: if 'lastname' in person: user.last_name = person['lastname'] + user.pi = authority_check_pis (request, user.email) + request.session['user'] = {'email':user.email,'pi':user.pi,'firstname':user.first_name,'lastname':user.last_name} return user # Required for your backend to work properly - unchanged in most scenarios diff --git a/myslice/settings.py b/myslice/settings.py index f4d0835b..86a5b07f 100644 --- a/myslice/settings.py +++ b/myslice/settings.py @@ -34,6 +34,7 @@ except: # DEBUG if config.myslice.debug : DEBUG = True + INTERNAL_IPS = ("127.0.0.1","132.227.84.195","132.227.78.191","132.227.84.191") else : DEBUG = False @@ -256,6 +257,7 @@ INSTALLED_APPS = [ # Uncomment the next line to enable admin documentation: # 'django.contrib.admindocs', 'portal', + #'debug_toolbar', ] # with django-1.7 we leave south and use native migrations # managing database migrations diff --git a/portal/accountview.py b/portal/accountview.py index 076f993f..ffed13f9 100644 --- a/portal/accountview.py +++ b/portal/accountview.py @@ -201,25 +201,6 @@ class AccountView(LoginRequiredAutoLogoutView, ThemeView): platform_list = [{'platform_no_access': t[0]} for t in itertools.izip_longest(total_platform_list)] - - ## check user is pi or not - # platform_query = Query().get('local:platform').select('platform_id','platform','gateway_type','disabled') - # account_query = Query().get('local:account').select('user_id','platform_id','auth_type','config') - # platform_details = execute_query(self.request, platform_query) - # account_details = execute_query(self.request, account_query) - # for platform_detail in platform_details: - # for account_detail in account_details: - # if platform_detail['platform_id'] == account_detail['platform_id']: - # if 'config' in account_detail and account_detail['config'] is not '': - # account_config = json.loads(account_detail['config']) - # if 'myslice' in platform_detail['platform']: - # acc_auth_cred = account_config.get('delegated_authority_credentials','N/A') - # assigning values - if acc_auth_cred == {} or acc_auth_cred == 'N/A': - pi = "is_not_pi" - else: - pi = "is_pi" - # check if the user has creds or not if acc_user_cred == {} or acc_user_cred == 'N/A': user_cred = 'no_creds' @@ -235,7 +216,6 @@ class AccountView(LoginRequiredAutoLogoutView, ThemeView): context['ref_acc'] = ref_acc_list context['platform_list'] = platform_list context['my_users'] = my_users - context['pi'] = pi context['user_cred'] = user_cred context['my_slices'] = my_slices context['my_auths'] = my_auths @@ -257,6 +237,9 @@ class AccountView(LoginRequiredAutoLogoutView, ThemeView): context['theme'] = self.theme context['section'] = "User account" # context ['firstname'] = config['firstname'] + + context['request'] = self.request + prelude_env = page.prelude_env() context.update(prelude_env) return context diff --git a/portal/contactview.py b/portal/contactview.py index 19cc1b94..f3899a0c 100644 --- a/portal/contactview.py +++ b/portal/contactview.py @@ -72,7 +72,7 @@ class ContactView (FreeAccessView, ThemeView): username = None # log user activity activity.user.contact(self.request) - return render(request,'contact_sent.html', { 'theme' : self.theme, 'username': username}) # Redirect after POST + return render(request,'contact_sent.html', { 'theme' : self.theme, 'username': username, 'request':request}) # Redirect after POST else: return self._display (request, form) @@ -82,23 +82,6 @@ class ContactView (FreeAccessView, ThemeView): def _display (self, request, form): if request.user.is_authenticated(): username = request.user.email - ## check user is pi or not - platform_query = Query().get('local:platform').select('platform_id','platform','gateway_type','disabled') - account_query = Query().get('local:account').select('user_id','platform_id','auth_type','config') - platform_details = execute_query(self.request, platform_query) - account_details = execute_query(self.request, account_query) - for platform_detail in platform_details: - for account_detail in account_details: - if platform_detail['platform_id'] == account_detail['platform_id']: - if 'config' in account_detail and account_detail['config'] is not '': - account_config = json.loads(account_detail['config']) - if 'myslice' in platform_detail['platform']: - acc_auth_cred = account_config.get('delegated_authority_credentials','N/A') - # assigning values - if acc_auth_cred == {} or acc_auth_cred == 'N/A': - pi = "is_not_pi" - else: - pi = "is_pi" else : username = None pi = "is_not_pi" @@ -107,6 +90,6 @@ class ContactView (FreeAccessView, ThemeView): 'topmenu_items': topmenu_items('Contact', request), 'theme' : self.theme, 'username': username, - 'pi': pi, - 'section': "Contact" + 'section': "Contact", + 'request': request, }) diff --git a/portal/homeview.py b/portal/homeview.py index 1dc87c18..42d8eaee 100644 --- a/portal/homeview.py +++ b/portal/homeview.py @@ -135,7 +135,7 @@ class HomeView (FreeAccessView, ThemeView): # log user activity activity.user.login(self.request, "error") env['state'] = "Your username and/or password were incorrect." - + env['request'] = request return render_to_response(self.template,env, context_instance=RequestContext(request)) def get (self, request, state=None): @@ -204,12 +204,11 @@ class HomeView (FreeAccessView, ThemeView): env['theme'] = self.theme env['section'] = "Dashboard" - env['username']=the_user(request) env['topmenu_items'] = topmenu_items(None, request) + env['request'] = request if state: env['state'] = state elif not env['username']: env['state'] = None # use one or two columns for the layout - not logged in users will see the login prompt - return render_to_response(self.template, env, context_instance=RequestContext(request)) diff --git a/portal/institution.py b/portal/institution.py index 332acbcf..f0a3b2db 100644 --- a/portal/institution.py +++ b/portal/institution.py @@ -33,6 +33,7 @@ class InstitutionView (LoginRequiredAutoLogoutView, ThemeView): def post (self,request): env = self.default_env() env['theme'] = self.theme + env['request'] = request return render_to_response(self.template, env, context_instance=RequestContext(request)) def get (self, request, authority_hrn=None, state=None): @@ -85,6 +86,6 @@ class InstitutionView (LoginRequiredAutoLogoutView, ThemeView): # use one or two columns for the layout - not logged in users will see the login prompt env['layout_1_or_2']="layout-unfold2.html" if not env['username'] else "layout-unfold1.html" - + env['request'] = request return render_to_response(self.template, env, context_instance=RequestContext(request)) diff --git a/portal/slicerequestview.py b/portal/slicerequestview.py index f0643e22..6d7801d5 100644 --- a/portal/slicerequestview.py +++ b/portal/slicerequestview.py @@ -3,6 +3,8 @@ import time import re from django.shortcuts import render +from django.shortcuts import render_to_response +from django.template import RequestContext from django.contrib.sites.models import Site from unfold.page import Page @@ -32,9 +34,14 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): def get (self, request): return self.get_or_post (request, 'GET') - def get_or_post (self, wsgi_request, method): + def get_or_post (self, request, method): """ """ + from django.conf import settings + print "_"*80 + print settings.TEMPLATE_CONTEXT_PROCESSORS + print "_"*80 + errors = [] slice_name ='' purpose='' @@ -43,14 +50,14 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): authority_name = None # Retrieve the list of authorities authorities_query = Query.get('authority').select('name', 'authority_hrn') - authorities = execute_admin_query(wsgi_request, authorities_query) + authorities = execute_admin_query(request, authorities_query) if authorities is not None: authorities = sorted(authorities, key=lambda k: k['authority_hrn']) authorities = sorted(authorities, key=lambda k: k['name']) # Get user_email (XXX Would deserve to be simplified) user_query = Query().get('local:user').select('email','config') - user_details = execute_query(wsgi_request, user_query) + user_details = execute_query(request, user_query) user_email = user_details[0].get('email') # getting user_hrn for user_detail in user_details: @@ -66,10 +73,10 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): authority_name = user_authority account_query = Query().get('local:account').select('user_id','platform_id','auth_type','config') - account_details = execute_query(wsgi_request, account_query) + account_details = execute_query(request, account_query) platform_query = Query().get('local:platform').select('platform_id','platform','gateway_type','disabled') - platform_details = execute_query(wsgi_request, platform_query) + platform_details = execute_query(request, platform_query) user_hrn = None #getting user_hrn from local:account for account_detail in account_details: @@ -89,11 +96,11 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): #else: # pi = "is_pi" - pi = authority_check_pis (wsgi_request, user_email) + pi = authority_check_pis (request, user_email) logger.debug("SLICEREQUESTVIEW.PY ----- pi= {}".format(pi)) # Page rendering - page = Page(wsgi_request) + page = Page(request) page.add_js_files ( [ "js/jquery.validate.js", "js/jquery-ui.js" ] ) page.add_css_files ( [ "css/jquery-ui.css" ] ) page.expose_js_metadata() @@ -107,15 +114,15 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): # getting the authority_hrn from the selected organization for authority in authorities: - if authority['name'] == wsgi_request.POST.get('org_name', ''): + if authority['name'] == request.POST.get('org_name', ''): authority_hrn = authority['authority_hrn'] # Handle the case when we use only hrn and not name if authority_hrn is None: - authority_hrn = wsgi_request.POST.get('org_name', '') + authority_hrn = request.POST.get('org_name', '') # Handle project if used - project = wsgi_request.POST.get('project', None) + project = request.POST.get('project', None) if project is not None and project != '': authority_hrn = project @@ -126,10 +133,10 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): 'email' : user_email, 'timestamp' : time.time(), 'authority_hrn' : authority_hrn, - 'organization' : wsgi_request.POST.get('org_name', ''), - 'slice_name' : wsgi_request.POST.get('slice_name', ''), - 'url' : wsgi_request.POST.get('url', ''), - 'purpose' : wsgi_request.POST.get('purpose', ''), + 'organization' : request.POST.get('org_name', ''), + 'slice_name' : request.POST.get('slice_name', ''), + 'url' : request.POST.get('url', ''), + 'purpose' : request.POST.get('purpose', ''), 'current_site' : current_site } @@ -137,7 +144,7 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): slice_name = slice_request['slice_name'] # slice name is unique among all authorities slice_query = Query().get('myslice:slice').select('slice_hrn') - slice_details_sfa = execute_admin_query(wsgi_request, slice_query) + slice_details_sfa = execute_admin_query(request, slice_query) for _slice in slice_details_sfa: split_list = _slice['slice_hrn'].split('.') sfa_slice_name = split_list[-1] @@ -173,25 +180,25 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): url = slice_request['url'] if not errors: - if is_pi(wsgi_request, user_hrn, authority_hrn): + if is_pi(request, user_hrn, authority_hrn): # PIs can directly create slices in their own authority... - create_slice(wsgi_request, slice_request) - clear_user_creds(wsgi_request, user_email) + create_slice(request, slice_request) + clear_user_creds(request, user_email) self.template_name = 'slice-request-done-view.html' else: - # Otherwise a wsgi_request is sent to the PI - create_pending_slice(wsgi_request, slice_request, user_email) + # Otherwise a request is sent to the PI + create_pending_slice(request, slice_request, user_email) self.template_name = 'slice-request-ack-view.html' # log user activity - activity.user.slice(wsgi_request) - - return render(wsgi_request, self.template, {'theme': self.theme}) # Redirect after POST + activity.user.slice(request) + return render_to_response(self.template, {'theme': self.theme, 'request':request}, context_instance=RequestContext(request)) + #return render(request, self.template, {'theme': self.theme}) # Redirect after POST else: slice_request = {} template_env = { - 'username': wsgi_request.user.email, + 'username': request.user.email, 'topmenu_items': topmenu_items_live('Request a slice', page), 'errors': errors, 'slice_name': slice_name, @@ -205,8 +212,11 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): 'cc_myself': True, 'authorities': authorities, 'theme': self.theme, - 'section': "Slice request" + 'section': "Slice request", + 'request': request, } template_env.update(slice_request) template_env.update(page.prelude_env()) - return render(wsgi_request, self.template, template_env) + + return render_to_response(self.template,template_env, context_instance=RequestContext(request)) + #return render(request, self.template, template_env) diff --git a/portal/sliceresourceview.py b/portal/sliceresourceview.py index 2451600a..17b9096d 100644 --- a/portal/sliceresourceview.py +++ b/portal/sliceresourceview.py @@ -413,27 +413,11 @@ class SliceResourceView (LoginRequiredView, ThemeView): # username = request.user, # ) # - ## check user is pi or not - platform_query = Query().get('local:platform').select('platform_id','platform','gateway_type','disabled') - account_query = Query().get('local:account').select('user_id','platform_id','auth_type','config') - platform_details = execute_query(self.request, platform_query) - account_details = execute_query(self.request, account_query) - - # XXX When session has expired, this is None and thus not iterable - for platform_detail in platform_details: - for account_detail in account_details: - if platform_detail['platform_id'] == account_detail['platform_id']: - if 'config' in account_detail and account_detail['config'] is not '': - account_config = json.loads(account_detail['config']) - if 'myslice' in platform_detail['platform']: - acc_auth_cred = account_config.get('delegated_authority_credentials','N/A') - # assigning values - if acc_auth_cred == {} or acc_auth_cred == 'N/A': - pi = "is_not_pi" - else: - pi = "is_pi" - + template_env = {} + + template_env['request'] = self.request + template_env['list_resources'] = list_resources.render(self.request) template_env['list_reserved_resources'] = list_reserved_resources.render(self.request) template_env['list_reserved_leases'] = list_reserved_leases.render(self.request) @@ -457,11 +441,10 @@ class SliceResourceView (LoginRequiredView, ThemeView): # template_env['vms_list'] = univbrisvtamplugin.render(self.request) # template_env['vm_form'] = univbrisvtamform.render(self.request) -# template_env['pending_resources'] = pending_resources.render(self.request) + # template_env['pending_resources'] = pending_resources.render(self.request) # template_env['sla_dialog'] = '' # sla_dialog.render(self.request) template_env["theme"] = self.theme template_env["username"] = request.user - template_env["pi"] = pi template_env["slice"] = slicename template_env["section"] = "resources" template_env["msg"] = msg diff --git a/portal/slicetabexperiment.py b/portal/slicetabexperiment.py index 138754a0..cc20df2a 100644 --- a/portal/slicetabexperiment.py +++ b/portal/slicetabexperiment.py @@ -23,6 +23,8 @@ from sfa.planetlab.plxrn import hash_loginbase import urllib2,json class ExperimentView (FreeAccessView, ThemeView): + # parent View is portal/sliceview.py + template_name = 'slice-tab-experiment.html' def get (self, request, slicename, state=None): @@ -99,6 +101,16 @@ class ExperimentView (FreeAccessView, ThemeView): for user in all_users: if user['email'] == username: iot_login = user['login'] - - return render_to_response(self.template, { 'theme' : self.theme,'slicename':slicename, 'ple_slicename':ple_slicename, 'username':username, 'ple_resources':ple_resource_list, 'nitos_resources': nitos_resource_list, 'nitos_paris_resources':nitos_paris_resource_list, 'iotlab_resources':iotlab_resource_list, 'iot_login':iot_login }, context_instance=RequestContext(request)) + env = { 'theme' : self.theme, + 'slicename':slicename, + 'ple_slicename':ple_slicename, + 'username':username, + 'ple_resources':ple_resource_list, + 'nitos_resources': nitos_resource_list, + 'nitos_paris_resources':nitos_paris_resource_list, + 'iotlab_resources':iotlab_resource_list, + 'iot_login':iot_login, + 'request':self.request, + } + return render_to_response(self.template, env, context_instance=RequestContext(request)) diff --git a/portal/sliceview.py b/portal/sliceview.py index 7bf139d1..b5fe92fa 100644 --- a/portal/sliceview.py +++ b/portal/sliceview.py @@ -20,27 +20,10 @@ class SliceView (LoginRequiredView, ThemeView): template_name = "slice-view.html" def get(self, request, slicename): - ## check user is pi or not - platform_query = Query().get('local:platform').select('platform_id','platform','gateway_type','disabled') - account_query = Query().get('local:account').select('user_id','platform_id','auth_type','config') - platform_details = execute_query(self.request, platform_query) - account_details = execute_query(self.request, account_query) - for platform_detail in platform_details: - for account_detail in account_details: - if platform_detail['platform_id'] == account_detail['platform_id']: - if 'config' in account_detail and account_detail['config'] is not '': - account_config = json.loads(account_detail['config']) - if 'myslice' in platform_detail['platform']: - acc_auth_cred = account_config.get('delegated_authority_credentials','N/A') - # assigning values - if acc_auth_cred == {} or acc_auth_cred == 'N/A': - pi = "is_not_pi" - else: - pi = "is_pi" + return render_to_response(self.template, {"slice" : slicename, "theme" : self.theme, "username" : request.user, - "pi" : pi, - "section" : "Slice {}".format(slicename) }, + "section" : "Slice {}".format(slicename),'request':request }, context_instance=RequestContext(request)) diff --git a/portal/supportview.py b/portal/supportview.py index 80653e82..23bcf763 100644 --- a/portal/supportview.py +++ b/portal/supportview.py @@ -31,24 +31,6 @@ class SupportView (FreeAccessView, ThemeView): if request.user.is_authenticated(): env['person'] = self.request.user - ## check user is pi or not - platform_query = Query().get('local:platform').select('platform_id','platform','gateway_type','disabled') - account_query = Query().get('local:account').select('user_id','platform_id','auth_type','config') - platform_details = execute_query(self.request, platform_query) - account_details = execute_query(self.request, account_query) - for platform_detail in platform_details: - for account_detail in account_details: - if platform_detail['platform_id'] == account_detail['platform_id']: - if 'config' in account_detail and account_detail['config'] is not '': - account_config = json.loads(account_detail['config']) - if 'myslice' in platform_detail['platform']: - acc_auth_cred = account_config.get('delegated_authority_credentials','N/A') - # assigning values - if acc_auth_cred == {} or acc_auth_cred == 'N/A': - pi = "is_not_pi" - else: - pi = "is_pi" - env['pi'] = pi else: env['person'] = None @@ -60,6 +42,6 @@ class SupportView (FreeAccessView, ThemeView): if state: env['state'] = state elif not env['username']: env['state'] = None # use one or two columns for the layout - not logged in users will see the login prompt - + env['request'] = request return render_to_response(self.template, env, context_instance=RequestContext(request)) diff --git a/portal/templates/_widget-topmenu.html b/portal/templates/_widget-topmenu.html index 668c3d32..320642f5 100644 --- a/portal/templates/_widget-topmenu.html +++ b/portal/templates/_widget-topmenu.html @@ -19,9 +19,9 @@ - {%if 'is_pi' in pi %} + {% if request.session.user.pi %} - {%endif%} + {%endif%}
  • SUPPORT
    • diff --git a/portal/templates/base.html b/portal/templates/base.html index c83c2bdc..4b5999ee 100644 --- a/portal/templates/base.html +++ b/portal/templates/base.html @@ -76,7 +76,9 @@ $(document).ready(function() { slices.push("no slice"); }else{ slices = user.slices; - //drawSlices(slices); + {% if theme != "fed4fire" %} + drawSlices(slices); + {% endif %} } {% if theme == "fed4fire" %} p = myslice.get_projects(); diff --git a/portal/templates/onelab/onelab_account-view.html b/portal/templates/onelab/onelab_account-view.html index e0629490..ffa882f9 100644 --- a/portal/templates/onelab/onelab_account-view.html +++ b/portal/templates/onelab/onelab_account-view.html @@ -130,7 +130,7 @@ - +