X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=portal%2Fslicerequestview.py;h=598aca3914f50e08b7aa09ed30d4f9c3ca5f0209;hb=91f1a0ed2c2b3be7377cce7182731a2339962304;hp=1bcbc5dc6f2da0ae43e817602fc577d47b2ca141;hpb=70677caff8e3b3c5e8caa255f01ca1a7ec864034;p=myslice.git diff --git a/portal/slicerequestview.py b/portal/slicerequestview.py index 1bcbc5dc..598aca39 100644 --- a/portal/slicerequestview.py +++ b/portal/slicerequestview.py @@ -34,6 +34,7 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): slice_name ='' purpose='' exp_url='' + authority_hrn = None # Retrieve the list of authorities authorities_query = Query.get('authority').select('name', 'authority_hrn') authorities = execute_admin_query(wsgi_request, authorities_query) @@ -83,6 +84,7 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): 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 @@ -91,12 +93,14 @@ 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 the template uses only hrn and not name + if authority_hrn is None: + authority_hrn = wsgi_request.POST.get('org_name', '') slice_request = { 'type' : 'slice', @@ -122,10 +126,6 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): if _slice['slice_hrn'] == req_slice_hrn: 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,11 +133,18 @@ 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 (organization is None or organization == ''): + errors.append('Organization is mandatory') + purpose = slice_request['purpose'] if (purpose is None or purpose == ''): - errors.append('Purpose is mandatory') + errors.append('Experiment purpose is mandatory') + + exp_url = slice_request['exp_url'] if not errors: if is_pi(wsgi_request, user_hrn, authority_hrn): @@ -164,6 +171,7 @@ class SliceRequestView (LoginRequiredAutoLogoutView, ThemeView): 'exp_url': exp_url, 'pi': pi, 'authority_name': authority_name, + 'authority_hrn': user_authority, 'cc_myself': True, 'authorities': authorities, 'theme': self.theme,