allow to upload/download Files with unknown mime type

product/ERP5/mixin/downloadable.py

@@ -86,16 +86,17 @@ class DownloadableMixin:
     output_format = None
     if not format:
       # Guess the format from original mimetype
-      mimetypes_registry = getToolByName(self.getPortalObject(),
-                                                          'mimetypes_registry')
-      mimetype_object_list = mimetypes_registry.lookup(mime)
-      for mimetype_object in mimetype_object_list:
-        if mimetype_object.extensions:
-          output_format = mimetype_object.extensions[0]
-          break
-        elif mimetype_object.globs:
-          output_format = mimetype_object.globs.strip('*.')
-          break
+      if mime:
+        mimetypes_registry = getToolByName(self.getPortalObject(),
+                                                            'mimetypes_registry')
+        mimetype_object_list = mimetypes_registry.lookup(mime)
+        for mimetype_object in mimetype_object_list:
+          if mimetype_object.extensions:
+            output_format = mimetype_object.extensions[0]
+            break
+          elif mimetype_object.globs:
+            output_format = mimetype_object.globs.strip('*.')
+            break
     else:
       output_format = format
 

product/ERP5OOo/tests/testDms.py

@@ -2616,6 +2616,19 @@ return 1
     self.assertEqual('archived', document_nolang_005.getValidationState())
     self.assertEqual('shared_alive', document_nolang_006.getValidationState())
 
+  def testFileWithNotDefinedMimeType(self):
+    upload_file = makeFileUpload('TEST-001-en.dummy')
+    kw = dict(file=upload_file, synchronous_metadata_discovery=True,
+              portal_type='File')
+    document = self.portal.Base_contribute(**kw)
+    document.setReference('TEST')
+    request = self.app.REQUEST
+    download_file = document.index_html(REQUEST=request, format=None)
+    self.assertEquals(download_file, 'foo\n')
+    document_format = None
+    self.assertEquals('TEST-001-en.dummy', document.getStandardFilename(
+                      document_format))
+
 class TestDocumentWithSecurity(TestDocumentMixin):
 
   username = 'yusei'

product/ERP5OOo/tests/test_document/TEST-001-en.dummy

+foo