Changeset 7105 for main/waeup.sirp/trunk/src/waeup/sirp/students

Timestamp:

13 Nov 2011, 17:50:26 (13 years ago)

Author:

Henrik Bettermann

Message:

Undo r7102 and r7103.

Location:

main/waeup.sirp/trunk/src/waeup/sirp/students

Files:

• student.py (modified) (2 diffs)
• viewlets.py (modified) (6 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/students/student.py

@@ -177 +177 @@
         return name == self.chooseName()
 
-    def chooseName(self, name=None, attr=None, ext=u''):
+    def chooseName(self, name=None, attr=None):
         """Get a valid file id for student context.
 
         *Example:*
 
-        For a student with student id ``'A123456'``,
-        with attr ``'nice_image'`` and an uploaded file named
-        `'anybasename.JPG'`` to be stored in
+        For a student with student id ``'A123456'`` and
+        with attr ``'nice_image'`` stored in
         the students container this chooser would create:
 
@@ -195 +194 @@
 
         """
-        if name and name.count('.') == 1:
-            basename, ext = os.path.splitext(name)
-            ext.lower()
         stud_id = self.context.student_id
-        marked_filename = '__%s__%s/%s/%s_%s%s' % (
-            STUDENT_FILE_STORE_NAME, stud_id[0], stud_id, attr, stud_id, ext)
+        marked_filename = '__%s__%s/%s/%s_%s.jpg' % (
+            STUDENT_FILE_STORE_NAME, stud_id[0], stud_id, attr, stud_id)
         return marked_filename
 

main/waeup.sirp/trunk/src/waeup/sirp/students/viewlets.py

@@ -198 +198 @@
         return self.view.application_url() + rel_link
 
-def handle_file_upload(upload, context, view, max_size, attr=None):
+def handle_img_upload(upload, context, view, max_size, attr=None):
     """Handle upload of applicant image.
 
@@ -212 +212 @@
     upload.seek(0) # file pointer moved when determining size
     store = getUtility(IExtFileStore)
-    file_id = IFileStoreNameChooser(
-        context).chooseName(attr=attr, name=upload.filename)
+    file_id = IFileStoreNameChooser(context).chooseName(attr=attr)
     store.createFile(file_id, upload)
     return True
@@ -253 +252 @@
         if upload:
             # We got a fresh upload
-            file_changed = handle_file_upload(
+            file_changed = handle_img_upload(
                 upload, self.context, self.view, self.mus, self.attr)
             if file_changed is False:  # False is not None!
@@ -288 +287 @@
     grok.baseclass()
     attr = None
-    ext = u'.jpg'
 
     def render(self):
@@ -294 +292 @@
         # for file storage.
         image = getUtility(IExtFileStore).getFileByContext(
-            self.context, attr=self.attr, ext=self.ext)
+            self.context, attr=self.attr)
         self.response.setHeader(
             'Content-Type', 'image/jpeg')
@@ -307 +305 @@
     grok.name('birth_certificate.jpg')
     attr = u'birth_certificate'
-    ext = u'.jpg'