X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=portal%2Fslicerequestview.py;h=b927d4f76cdf50af8967324363824b50d82a0dea;hb=8f61aab7af867de3dc1b2ec30c53cc7f00b4734f;hp=1bcbc5dc6f2da0ae43e817602fc577d47b2ca141;hpb=70677caff8e3b3c5e8caa255f01ca1a7ec864034;p=unfold.git diff --git a/portal/slicerequestview.py b/portal/slicerequestview.py index 1bcbc5dc..b927d4f7 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, authority_check_pis #from portal.forms import SliceRequestForm from unfold.loginrequired import LoginRequiredAutoLogoutView from ui.topmenu import topmenu_items_live, the_user @@ -16,6 +16,9 @@ from myslice.theme import ThemeView import json, time, re +import activity.user +theme = ThemeView() + class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): template_name = 'slicerequest_view.html' @@ -33,12 +36,15 @@ 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) if authorities is not None: - authorities = sorted(authorities) + 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') @@ -53,14 +59,17 @@ 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) - # + platform_query = Query().get('local:platform').select('platform_id','platform','gateway_type','disabled') platform_details = execute_query(wsgi_request, platform_query) user_hrn = None - # getting user_hrn from local:account + #getting user_hrn from local:account for account_detail in account_details: for platform_detail in platform_details: if platform_detail['platform_id'] == account_detail['platform_id']: @@ -69,20 +78,23 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): if 'myslice' in platform_detail['platform']: account_config = json.loads(account_detail['config']) user_hrn = account_config.get('user_hrn','N/A') - acc_auth_cred = account_config.get('delegated_authority_credentials','N/A') + # acc_auth_cred = account_config.get('delegated_authority_credentials','N/A') # checking if pi or not - if acc_auth_cred == {} or acc_auth_cred == 'N/A': - pi = "is_not_pi" - else: - pi = "is_pi" + #if acc_auth_cred == {} or acc_auth_cred == 'N/A': + # pi = "is_not_pi" + #else: + # pi = "is_pi" + pi = authority_check_pis (wsgi_request, user_email) + print "SLICEREQUESTVIEW.PY ----- pi=",pi # Page rendering page = Page(wsgi_request) page.add_js_files ( [ "js/jquery.validate.js", "js/jquery-ui.js" ] ) - page.add_css_files ( [ "https://code.jquery.com/ui/1.10.3/themes/smoothness/jquery-ui.css" ] ) + page.add_css_files ( [ "css/jquery-ui.css" ] ) + page.expose_js_metadata() if method == 'POST': # The form has been submitted @@ -91,12 +103,19 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): current_site = Site.objects.get_current() current_site = current_site.domain - # getting the authority_hrn from the selected organization + # getting the authority_hrn from the selected organization for authority in authorities: if authority['name'] == wsgi_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', '') + + # Handle project if used + project = wsgi_request.POST.get('project', None) + if project is not None and project != '': + authority_hrn = project slice_request = { 'type' : 'slice', @@ -107,25 +126,22 @@ 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('url', ''), 'purpose' : wsgi_request.POST.get('purpose', ''), 'current_site' : current_site } # create slice_hrn based on authority_hrn and slice_name slice_name = slice_request['slice_name'] - req_slice_hrn = authority_hrn + '.' + slice_name - # comparing requested slice_hrn with the existing slice_hrn - slice_query = Query().get('slice').select('slice_hrn','parent_authority').filter_by('parent_authority','==',authority_hrn) + # 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) for _slice in slice_details_sfa: - if _slice['slice_hrn'] == req_slice_hrn: + split_list = _slice['slice_hrn'].split('.') + sfa_slice_name = split_list[-1] + if sfa_slice_name == slice_name: errors.append('Slice already exists. Please use a different slice name.') - exp_url = slice_request['exp_url'] - - if (authority_hrn is None or authority_hrn == ''): - errors.append('Please, select an organization') # What kind of slice name is valid? if (slice_name is None or slice_name == ''): @@ -133,22 +149,41 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): if (re.search(r'^[A-Za-z0-9_]*$', slice_name) == None): errors.append('Slice name may contain only letters, numbers, and underscore.') + + organization = slice_request['organization'] + if theme.theme == 'fed4fire': + if (organization is None or organization == ''): + errors.append('Selecting project is mandatory') + else: + if (organization is None or organization == ''): + errors.append('Organization is mandatory') + + slice_length= len(slice_request['slice_name']) + if slice_length >19: + errors.append('Slice name can be maximum 19 characters long') + purpose = slice_request['purpose'] if (purpose is None or purpose == ''): - errors.append('Purpose is mandatory') + errors.append('Experiment purpose is mandatory') + + 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 = {} @@ -161,9 +196,10 @@ 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, 'cc_myself': True, 'authorities': authorities, 'theme': self.theme,