Fix trashed document previews
Signed-off-by: Roberto Rosario <roberto.rosario.gonzalez@gmail.com>
This commit is contained in:
@@ -165,7 +165,7 @@ class APIDocumentPageImageView(generics.RetrieveAPIView):
|
|||||||
|
|
||||||
AccessControlList.objects.check_access(
|
AccessControlList.objects.check_access(
|
||||||
obj=document, permissions=(permission_required,),
|
obj=document, permissions=(permission_required,),
|
||||||
user=self.request.user
|
user=self.request.user, manager=Document.passthrough
|
||||||
)
|
)
|
||||||
return document
|
return document
|
||||||
|
|
||||||
|
|||||||
@@ -627,6 +627,34 @@ class TrashedDocumentAPITestCase(DocumentTestMixin, BaseAPITestCase):
|
|||||||
response.data['uuid'], force_text(self.test_document.uuid)
|
response.data['uuid'], force_text(self.test_document.uuid)
|
||||||
)
|
)
|
||||||
|
|
||||||
|
def _request_trashed_document_image_view(self):
|
||||||
|
latest_version = self.test_document.latest_version
|
||||||
|
|
||||||
|
return self.get(
|
||||||
|
viewname='rest_api:documentpage-image', kwargs={
|
||||||
|
'pk': latest_version.document.pk,
|
||||||
|
'version_pk': latest_version.pk,
|
||||||
|
'page_pk': latest_version.pages.first().pk
|
||||||
|
}
|
||||||
|
)
|
||||||
|
|
||||||
|
def test_trashed_document_image_view_no_permission(self):
|
||||||
|
self.upload_document()
|
||||||
|
self.test_document.delete()
|
||||||
|
|
||||||
|
response = self._request_trashed_document_image_view()
|
||||||
|
self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
|
||||||
|
|
||||||
|
def test_trashed_document_image_view_with_access(self):
|
||||||
|
self.upload_document()
|
||||||
|
self.test_document.delete()
|
||||||
|
self.grant_access(
|
||||||
|
obj=self.test_document, permission=permission_document_view
|
||||||
|
)
|
||||||
|
|
||||||
|
response = self._request_trashed_document_image_view()
|
||||||
|
self.assertEqual(response.status_code, status.HTTP_200_OK)
|
||||||
|
|
||||||
def _request_trashed_document_list_view(self):
|
def _request_trashed_document_list_view(self):
|
||||||
return self.get(
|
return self.get(
|
||||||
viewname='rest_api:trasheddocument-list'
|
viewname='rest_api:trasheddocument-list'
|
||||||
|
|||||||
Reference in New Issue
Block a user