Changeset 12447 for main/waeup.kofa/trunk/src/waeup/kofa/documents

Timestamp:

12 Jan 2015, 09:24:58 (10 years ago)

Author:

Henrik Bettermann

Message:

Remove redundant code and set download_filename for pdf files.

Location:

main/waeup.kofa/trunk/src/waeup/kofa/documents

Files:

• fileviewlets.py (modified) (3 diffs)
• tests/test_browser.py (modified) (3 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/documents/fileviewlets.py

@@ -44 +44 @@
     tab_redirect = '#tab2'
 
+    @property
+    def download_filename(self):
+        return u"%s.pdf" % self.context.document_id
+
 
 class PDFScanDisplay(FileDisplay):
@@ -56 +60 @@
     download_name = u'sample.pdf'
 
+    @property
+    def download_filename(self):
+        return u"%s.pdf" % self.context.document_id
+
 
 class PDFScanImage(UtilityView, Image):
@@ -66 +74 @@
     download_name = u'sample.pdf'
 
+    @property
+    def download_filename(self):
+        return self.context.document_id
+
     def update(self):
         if self.context.state != PUBLISHED:

main/waeup.kofa/trunk/src/waeup/kofa/documents/tests/test_browser.py

@@ -148 +148 @@
             name='upload_pdfscanmanageupload').click()
         self.assertTrue(
-            'href="http://localhost/app/documents/DOC1/sample.pdf">PDF File</a>'
+            'href="http://localhost/app/documents/DOC1/sample.pdf">DOC1.pdf</a>'
             in self.browser.contents)
         # The file can be found in the file system
@@ -159 +159 @@
         # Browsing the link shows a real pdf only if the document
         # has been published
-        self.browser.getLink("PDF File").click()
+        self.browser.getLink("DOC1.pdf").click()
         self.assertTrue(
             'The document requested has not yet been published'
@@ -167 +167 @@
         self.assertEqual(
             self.browser.headers['content-type'], 'application/pdf')
+        # The name of the downloaded file will be different
+        self.assertEqual(
+            self.browser.headers['Content-Disposition'],
+            'attachment; filename="DOC1.pdf')
 
         # Transitions can be performed