X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=portal%2Fslicerequestview.py;h=b51273f55a7ec52bf120d8765bb0603e3795b576;hb=refs%2Fheads%2Ffibre;hp=bd711d5cf17c994b0be691872d2a6290650c86dc;hpb=594732ab461c6a75a8778f41724f4224839b26a6;p=unfold.git diff --git a/portal/slicerequestview.py b/portal/slicerequestview.py index bd711d5c..b51273f5 100644 --- a/portal/slicerequestview.py +++ b/portal/slicerequestview.py @@ -7,7 +7,7 @@ from unfold.page import Page from manifold.core.query import Query from manifoldapi.manifoldapi import execute_admin_query, execute_query -from portal.actions import is_pi, create_slice, create_pending_slice +from portal.actions import is_pi, create_slice, create_pending_slice, clear_user_creds #from portal.forms import SliceRequestForm from unfold.loginrequired import LoginRequiredAutoLogoutView from ui.topmenu import topmenu_items_live, the_user @@ -16,6 +16,8 @@ from myslice.theme import ThemeView import json, time, re +import activity.user + class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): template_name = 'slicerequest_view.html' @@ -33,7 +35,9 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): errors = [] slice_name ='' purpose='' - exp_url='' + url='' + authority_hrn = None + 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) @@ -53,6 +57,9 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): if authority['authority_hrn'] == user_authority: authority_name = authority['name'] + # Handle the case when we use only hrn and not name + if authority_name is None: + 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) @@ -97,8 +104,8 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): if authority['name'] == wsgi_request.POST.get('org_name', ''): authority_hrn = authority['authority_hrn'] - # Handle the case when the template uses only hrn and not name - if not 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', '') slice_request = { @@ -110,7 +117,7 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): 'authority_hrn' : authority_hrn, 'organization' : wsgi_request.POST.get('org_name', ''), 'slice_name' : wsgi_request.POST.get('slice_name', ''), - 'exp_url' : wsgi_request.POST.get('exp_url', ''), + 'url' : wsgi_request.POST.get('exp_url', ''), 'purpose' : wsgi_request.POST.get('purpose', ''), 'current_site' : current_site } @@ -143,18 +150,22 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): if (purpose is None or purpose == ''): errors.append('Experiment purpose is mandatory') - exp_url = slice_request['exp_url'] + url = slice_request['url'] if not errors: if is_pi(wsgi_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) 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) 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 else: slice_request = {} @@ -167,7 +178,7 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): 'purpose': purpose, 'email': user_email, 'user_hrn': user_hrn, - 'exp_url': exp_url, + 'url': url, 'pi': pi, 'authority_name': authority_name, 'authority_hrn': user_authority,