def new_review_request(request, template_name="reviews/new_review_request.html"): """ Displays a New Review Request form and handles the creation of a review request based on either an existing changeset or the provided information. """ if request.method == "POST": form = NewReviewRequestForm(request.POST, request.FILES) if form.is_valid(): try: review_request = form.create( user=request.user, diff_file=request.FILES["diff_path"], parent_diff_file=request.FILES.get("parent_diff_path"), ) return HttpResponseRedirect(review_request.get_absolute_url()) except (OwnershipError, ChangeSetError): pass else: form = NewReviewRequestForm() return render_to_response( template_name, RequestContext(request, {"form": form, "fields": simplejson.dumps(form.field_mapping)}) )
def new_review_request(request, template_name='reviews/new_review_request.html'): """ Displays a New Review Request form and handles the creation of a review request based on either an existing changeset or the provided information. """ if request.method == 'POST': form = NewReviewRequestForm(request.POST, request.FILES) if form.is_valid(): try: review_request = form.create( user=request.user, diff_file=request.FILES['diff_path'], parent_diff_file=request.FILES.get('parent_diff_path')) return HttpResponseRedirect(review_request.get_absolute_url()) except: # XXX - OwnershipError or ChangeSetError? # # We're preventing an exception from being thrown here so that # we can display the errors that form.create() sets in # a much nicer way in the template. Otherwise, the user would # see a useless backtrace. pass else: form = NewReviewRequestForm() return render_to_response(template_name, RequestContext(request, { 'form': form, 'fields': simplejson.dumps(form.field_mapping), }))
def new_review_request(request, template_name='reviews/new_review_request.html'): """ Displays a New Review Request form and handles the creation of a review request based on either an existing changeset or the provided information. """ if request.method == 'POST': form = NewReviewRequestForm(request.POST, request.FILES) if form.is_valid(): try: review_request = form.create( user=request.user, diff_file=request.FILES.get('diff_path'), parent_diff_file=request.FILES.get('parent_diff_path')) return HttpResponseRedirect(review_request.get_absolute_url()) except (OwnershipError, SCMError, ValueError): pass else: form = NewReviewRequestForm() return render_to_response(template_name, RequestContext(request, { 'form': form, 'fields': simplejson.dumps(form.field_mapping), }))
def new_review_request(request, local_site_name=None, template_name="reviews/new_review_request.html"): """ Displays a New Review Request form and handles the creation of a review request based on either an existing changeset or the provided information. """ if local_site_name: local_site = get_object_or_404(LocalSite, name=local_site_name) if not local_site.is_accessible_by(request.user): return _render_permission_denied(request) else: local_site = None if request.method == "POST": form = NewReviewRequestForm(request.user, local_site, request.POST, request.FILES) if form.is_valid(): try: review_request = form.create( user=request.user, diff_file=request.FILES.get("diff_path"), parent_diff_file=request.FILES.get("parent_diff_path"), local_site=local_site, ) return HttpResponseRedirect(review_request.get_absolute_url()) except (OwnershipError, SCMError, ValueError): pass else: form = NewReviewRequestForm(request.user, local_site) return render_to_response( template_name, RequestContext(request, {"form": form, "fields": simplejson.dumps(form.field_mapping)}) )
def new_review_request(request, template_name="reviews/new_review_request.html"): """ Displays a New Review Request form and handles the creation of a review request based on either an existing changeset or the provided information. """ if request.method == "POST": form = NewReviewRequestForm(request.POST, request.FILES) if form.is_valid(): try: review_request = form.create( user=request.user, diff_file=request.FILES["diff_path"], parent_diff_file=request.FILES.get("parent_diff_path"), ) return HttpResponseRedirect(review_request.get_absolute_url()) except: # XXX - OwnershipError or ChangeSetError? # # We're preventing an exception from being thrown here so that # we can display the errors that form.create() sets in # a much nicer way in the template. Otherwise, the user would # see a useless backtrace. pass else: form = NewReviewRequestForm() # Repository ID : visible fields mapping. This is so we can dynamically # show/hide the relevant fields with javascript. fields = {} for repo in Repository.objects.all(): fields[repo.id] = repo.get_scmtool().get_fields() # Turn the selected index back into an int so we can compare it properly. if "repository" in form.data: form.data["repository"] = int(form.data["repository"]) return render_to_response( template_name, RequestContext(request, {"form": form, "fields": simplejson.dumps(fields)}) )
def new_review_request(request, local_site_name=None, template_name='reviews/new_review_request.html'): """ Displays a New Review Request form and handles the creation of a review request based on either an existing changeset or the provided information. """ if local_site_name: local_site = get_object_or_404(LocalSite, name=local_site_name) if (request.user.is_anonymous() or not local_site.users.filter(pk=request.user.pk).exists()): return _render_permission_denied(requset) else: local_site = None if request.method == 'POST': form = NewReviewRequestForm(request.user, local_site, request.POST, request.FILES) if form.is_valid(): try: review_request = form.create( user=request.user, diff_file=request.FILES.get('diff_path'), parent_diff_file=request.FILES.get('parent_diff_path'), local_site=local_site) return HttpResponseRedirect(review_request.get_absolute_url()) except (OwnershipError, SCMError, ValueError): pass else: form = NewReviewRequestForm(request.user, local_site) return render_to_response(template_name, RequestContext(request, { 'form': form, 'fields': simplejson.dumps(form.field_mapping), }))
def new_review_request(request, local_site_name=None, template_name='reviews/new_review_request.html'): """ Displays a New Review Request form and handles the creation of a review request based on either an existing changeset or the provided information. """ if local_site_name: local_site = get_object_or_404(LocalSite, name=local_site_name) if not local_site.is_accessible_by(request.user): return _render_permission_denied(request) else: local_site = None if request.method == 'POST': form = NewReviewRequestForm(request.user, local_site, request.POST, request.FILES) if form.is_valid(): try: review_request = form.create( user=request.user, diff_file=request.FILES.get('diff_path'), parent_diff_file=request.FILES.get('parent_diff_path'), local_site=local_site) return HttpResponseRedirect(review_request.get_absolute_url()) except (OwnershipError, SCMError, ValueError): pass else: form = NewReviewRequestForm(request.user, local_site) return render_to_response( template_name, RequestContext(request, { 'form': form, 'fields': simplejson.dumps(form.field_mapping), }))