X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=portal%2Fslicerequestview.py;h=449147ca1642e2bd054afdf28d2327837485cad5;hb=23d0b7e5a5f276eac75e518948fd8023c2798179;hp=1bcbc5dc6f2da0ae43e817602fc577d47b2ca141;hpb=70677caff8e3b3c5e8caa255f01ca1a7ec864034;p=myslice.git diff --git a/portal/slicerequestview.py b/portal/slicerequestview.py index 1bcbc5dc..449147ca 100644 --- a/portal/slicerequestview.py +++ b/portal/slicerequestview.py @@ -91,7 +91,8 @@ 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 + authority_hrn = '' for authority in authorities: if authority['name'] == wsgi_request.POST.get('org_name', ''): authority_hrn = authority['authority_hrn'] @@ -122,10 +123,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 +130,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):