Skip to content
Snippets Groups Projects
Verified Commit 44613036 authored by Jonathan Weth's avatar Jonathan Weth :keyboard:
Browse files

Raise 404 error if live document has no current PDF file

parent 4d233879
No related branches found
No related tags found
1 merge request!16Resolve "Provide infrastructure for live documents"
...@@ -3,7 +3,7 @@ from typing import Any, Dict, Type ...@@ -3,7 +3,7 @@ from typing import Any, Dict, Type
from django.contrib.contenttypes.models import ContentType from django.contrib.contenttypes.models import ContentType
from django.db.models import QuerySet from django.db.models import QuerySet
from django.forms import BaseModelForm, modelform_factory from django.forms import BaseModelForm, modelform_factory
from django.http import FileResponse, HttpRequest from django.http import FileResponse, HttpRequest, Http404
from django.shortcuts import get_object_or_404 from django.shortcuts import get_object_or_404
from django.urls import reverse_lazy from django.urls import reverse_lazy
from django.utils.decorators import method_decorator from django.utils.decorators import method_decorator
...@@ -229,4 +229,7 @@ class LiveDocumentShowView(PermissionRequiredMixin, SingleObjectMixin, View): ...@@ -229,4 +229,7 @@ class LiveDocumentShowView(PermissionRequiredMixin, SingleObjectMixin, View):
def get(self, request: HttpRequest, *args: Any, **kwargs: Any) -> FileResponse: def get(self, request: HttpRequest, *args: Any, **kwargs: Any) -> FileResponse:
live_document = self.get_object() live_document = self.get_object()
file = live_document.get_current_file()
if not file:
raise Http404
return FileResponse(live_document.get_current_file(), content_type="application/pdf") return FileResponse(live_document.get_current_file(), content_type="application/pdf")
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment