X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=portal%2Fprojectrequestview.py;h=2757b94a1462bc5c75ad817985c493ec58bc89f6;hb=refs%2Fheads%2Fonelab;hp=8a48428263c80cb013f056b93b6e14e2f41bb1ac;hpb=708e4ed85f2ef9d5b17097cb71a5879b244ebfe9;p=myslice.git diff --git a/portal/projectrequestview.py b/portal/projectrequestview.py index 8a484282..2757b94a 100644 --- a/portal/projectrequestview.py +++ b/portal/projectrequestview.py @@ -1,3 +1,8 @@ +import requests +import json +import time +import re + from django.shortcuts import render from django.contrib.sites.models import Site @@ -6,24 +11,15 @@ from manifoldapi.manifoldapi import execute_admin_query, execute_query from unfold.loginrequired import LoginRequiredAutoLogoutView -from portal.actions import create_pending_project, create_pending_join, sfa_add_authority, authority_add_pis, is_pi +from portal.actions import create_pending_project, create_pending_join, sfa_add_authority, authority_add_pis, is_pi, getAuthorities from portal.models import PendingProject, PendingJoin from myslice.theme import ThemeView - -import json, time, re +from myslice.settings import logger class ProjectRequestView(LoginRequiredAutoLogoutView, ThemeView): template_name = 'projectrequest_view.html' - def getAuthorities(self, request): - authorities_query = Query.get('authority').select('name', 'authority_hrn') - 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']) - return authorities - def getUserAuthority(self, request): # Get user_email (XXX Would deserve to be simplified) user_query = Query().get('local:user').select('email','config') @@ -77,23 +73,30 @@ class ProjectRequestView(LoginRequiredAutoLogoutView, ThemeView): user_email = self.getUserEmail(wsgi_request) - authorities = self.getAuthorities(wsgi_request) + authorities = getAuthorities(wsgi_request) user_authority = self.getUserAuthority(wsgi_request) # getting the org from authority - for authority in authorities: - if authority['authority_hrn'] == user_authority: - authority_name = authority['name'] + #for authority in authorities: + # if authority['authority_hrn'] == user_authority: + # authority_name = authority['name'] if method == 'POST' : - + + project_name = wsgi_request.POST.get('project_name', '') + if not project_name or len(project_name) == 0 : + errors.append('Project name can\'t be empty') + + # accept only lowercase names + project_name = project_name.lower() + if 'join' in wsgi_request.POST: post = { 'user_hrn' : user_hrn, 'email' : user_email, - 'project_name' : wsgi_request.POST.get('project_name', ''), - 'authority_hrn' : wsgi_request.POST.get('project_name', ''), + 'project_name' : project_name, + 'authority_hrn' : project_name, } else: @@ -101,30 +104,37 @@ class ProjectRequestView(LoginRequiredAutoLogoutView, ThemeView): 'user_hrn' : user_hrn, 'email' : user_email, 'authority_hrn' : wsgi_request.POST.get('authority_name', ''), - 'project_name' : wsgi_request.POST.get('project_name', ''), + 'project_name' : project_name, 'purpose' : wsgi_request.POST.get('purpose', ''), + 'url' : wsgi_request.POST.get('url', ''), } - if (post['authority_hrn'] is None or post['authority_hrn'] == ''): - errors.append('Organization is mandatory') + # for new projects max project_name length is 10 + if (len(post['project_name']) >10): + errors.append('Project name can be maximum 10 characters long') + + #if (post['authority_hrn'] is None or post['authority_hrn'] == ''): + # errors.append('Organization is mandatory') - if (post['purpose'] is None or post['purpose'] == ''): + if post['purpose'] is None or post['purpose'] == '': errors.append('Project purpose is mandatory') - if (re.search(r'^[A-Za-z0-9_]*$', post['project_name']) == None): + if re.search(r'^[A-Za-z0-9_]*$', post['project_name']) is None: errors.append('Project name may contain only letters, numbers, and underscore.') # What kind of project name is valid? - if (post['project_name'] is None or post['project_name'] == ''): - errors.append('Project name is mandatory') + if post['project_name'] is None or post['project_name'] == '': + errors.append('Project name is mandatory') if not errors: - print "is_pi on auth_hrn = ", user_authority - if is_pi(wsgi_request, user_hrn, user_authority): + logger.info("is_pi on auth_hrn = {}".format(user_authority)) + if is_pi(wsgi_request, user_hrn, post['authority_hrn']): # PIs can directly create/join project in their own authority... if 'join' in wsgi_request.POST: + # join existing project authority_add_pis(wsgi_request, post['project_name'], user_hrn) else: + # Create project hrn = post['authority_hrn'] + '.' + post['project_name'] sfa_add_authority(wsgi_request, {'authority_hrn':hrn}) authority_add_pis(wsgi_request, hrn, user_hrn) @@ -154,167 +164,3 @@ class ProjectRequestView(LoginRequiredAutoLogoutView, ThemeView): 'pending_join_projects': pending_join_projects, } return render(wsgi_request, self.template, env) - - - - """ - """ - errors = [] - slice_name ='' - purpose='' - 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, 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_email = user_details[0].get('email') - # getting user_hrn - for user_detail in user_details: - user_config = json.loads(user_detail['config']) - user_authority = user_config.get('authority','N/A') - # getting the org from authority - for authority in authorities: - 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 - for account_detail in account_details: - for platform_detail in platform_details: - if platform_detail['platform_id'] == account_detail['platform_id']: - # taking user_hrn only from myslice account - # NOTE: we should later handle accounts filter_by auth_type= managed OR user - 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') - - - # checking if pi or not - if acc_auth_cred == {} or acc_auth_cred == 'N/A': - pi = "is_not_pi" - else: - pi = "is_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.expose_js_metadata() - - if method == 'POST': - # The form has been submitted - - # get the domain url -# current_site = Site.objects.get_current() -# current_site = current_site.domain - - # 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', '') - - slice_request = { - 'type' : 'slice', - 'id' : None, - 'user_hrn' : user_hrn, - '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', ''), - '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('myslice:slice').select('slice_hrn','parent_authority').filter_by('parent_authority','==',authority_hrn) - slice_details_sfa = execute_admin_query(wsgi_request, slice_query) - for _slice in slice_details_sfa: - if _slice['slice_hrn'] == req_slice_hrn: - errors.append('Slice already exists. Please use a different slice name.') - - - # What kind of slice name is valid? - if (slice_name is None or slice_name == ''): - errors.append('Slice name is mandatory') - - 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 (organization is None or organization == ''): - errors.append('Organization is mandatory') - - - - purpose = slice_request['purpose'] - if (purpose is None or purpose == ''): - 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 = {} - - template_env = { - 'username': wsgi_request.user.email, - 'errors': errors, - 'slice_name': slice_name, - 'purpose': purpose, - 'email': user_email, - 'user_hrn': user_hrn, - 'url': url, - 'pi': pi, - 'authority_name': authority_name, - 'authority_hrn': user_authority, - 'cc_myself': True, - 'authorities': authorities, - 'theme': self.theme, - 'section': "Slice request" - } - template_env.update(slice_request) - template_env.update(page.prelude_env()) - return render(wsgi_request, self.template, template_env)