Changeset 7103 for main/waeup.sirp/trunk/src/waeup/sirp/tests

Timestamp:

13 Nov 2011, 10:22:25 (13 years ago)

Author:

Henrik Bettermann

Message:

File name choosers generally don't know the extension of the file stored. So we have to feed the chooseName method with the extension (ext) of the file stored in the file system.

This may conflict somehow with the idea behind the file name chooser. I think you (Uli) intended to use only the attr parameter to select a proper extension. In my opinion it's better to use attr and ext.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/tests/test_imagestorage.py (modified) (1 diff)

main/waeup.sirp/trunk/src/waeup/sirp/tests/test_imagestorage.py

@@ -158 +158 @@
         self.context = context
 
-    def chooseName(self, name=None, attr=None):
+    def chooseName(self, name=None, attr=None, ext=None):
         # this name chooser returns different file ids depending on
         # the `attr` parameter, a simple string.