Changeset 12444 for main/waeup.ikoba/trunk/src/waeup/ikoba/documents

Timestamp:

11 Jan 2015, 22:43:35 (10 years ago)

Author:

Henrik Bettermann

Message:

We need to provide unique filenames. Let's use the document id for the filename and not the download_name of the respective viewlet.

Location:

main/waeup.ikoba/trunk/src/waeup/ikoba/documents

Files:

• document.py (modified) (1 diff)
• fileviewlets.py (modified) (3 diffs)
• tests/test_browser.py (modified) (4 diffs)

main/waeup.ikoba/trunk/src/waeup/ikoba/documents/document.py

@@ -146 +146 @@
     form_fields_interface = IPDFDocument
 
-    filenames = ('scan.pdf',)
+    filenames = ('file.pdf',)
 
 PDFDocument = attrs_to_fields(PDFDocument)

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

@@ -40 +40 @@
     title = _(u'PDF File')
     mus = 1024 * 200
-    download_name = u'sample.pdf'
+    download_name = u'file.pdf'
     tab_redirect = '#tab2'
 
@@ -53 +53 @@
     label = _(u'PDF Scan')
     title = _(u'PDF Scan')
-    download_name = u'sample.pdf'
+    download_name = u'file.pdf'
 
 
@@ -59 +59 @@
     """Scan document.
     """
-    ##: Needs to be protected if not published
-    grok.name('sample.pdf')
+    grok.name('file.pdf')
     grok.context(PDFDocument)
     grok.require('waeup.Public')
-    download_name = u'sample.pdf'
+    download_name = u'file.pdf'
 
     def update(self):

main/waeup.ikoba/trunk/src/waeup/ikoba/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/file.pdf">DOC1.pdf</a>'
             in self.browser.contents)
         # The file can be found in the file system
         file = getUtility(IExtFileStore).getFileByContext(
-            document, attr='sample.pdf')
+            document, attr='file.pdf')
         file_content = file.read()
         pdf.seek(0)
@@ -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'
             in self.browser.contents)
         IWorkflowState(document).setState(PUBLISHED)
-        self.browser.open(self.container_path + '/DOC1/sample.pdf')
+        self.browser.open(self.container_path + '/DOC1/file.pdf')
         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
@@ -186 +190 @@
         # File has been removed too
         file = getUtility(IExtFileStore).getFileByContext(
-            document, attr='sample.pdf')
+            document, attr='file.pdf')
         self.assertTrue(file is None)
 
@@ -204 +208 @@
             % document.document_id in logcontent)
         self.assertTrue(
-            'INFO - zope.mgr - documents.browser.DocumentManageFormPage - %s - uploaded: sample.pdf (my_sample_scan.pdf)'
+            'INFO - zope.mgr - documents.browser.DocumentManageFormPage - %s - uploaded: file.pdf (my_sample_scan.pdf)'
             % document.document_id in logcontent)
         self.assertTrue(