Changeset 6598

Timestamp:

12 Aug 2011, 15:23:23 (13 years ago)

Author:

uli

Message:

Workaround for freshly uploaded images not being displayed in
otherwise erraneous applicants forms.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/applicants/browser.py (modified) (3 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/applicants/browser.py

@@ -54 +54 @@
     ThumbnailWidget, EncodingImageFileWidget,
     )
+from waeup.sirp.image.image import createWAeUPImageFile
 from waeup.sirp.interfaces import IWAeUPObject, ILocalRolesAssignable
 from waeup.sirp.permissions import get_users_with_local_roles
@@ -708 +709 @@
         super(EditApplicantFull, self).update()
         self.wf_info = IWorkflowInfo(self.context)
+        upload = self.request.form.get('form.passport', None)
+        if upload:
+            # We got a fresh upload
+            image = createWAeUPImageFile(upload.filename, upload)
+            # This would normally be totally illegal. We set context
+            # data without the complete form data being
+            # verified. Normally data is set in methods like `save`
+            # only, which is only called when all form data is correct
+            # (and if not, the context is not touched). With images
+            # and their uploads the problem then is, that the uploaded
+            # image won't be shown in a partially erranous form
+            # because the new uploaded image is still not part of the
+            # context obeject and the image widget cannot work around
+            # that (it has no access to the new data). For this reason
+            # we set the image here and not in 'save()' or other
+            # methods.
+            self.context.passport = image
+            upload.seek(0)
+            self.passport_changed = True
         return
 
@@ -769 +789 @@
     def update(self):
         if self.context.locked:
-            #import pdb; pdb.set_trace()
             self.emitLockMessage()
             return
         datepicker.need() # Enable jQuery datepicker in date fields.
+        upload = self.request.form.get('form.passport', None)
+        if upload:
+            # We got a fresh upload
+            image = createWAeUPImageFile(upload.filename, upload)
+            # This would normally be totally illegal. We set context
+            # data without the complete form data being
+            # verified. Normally data is set in methods like `save`
+            # only, which is only called when all form data is correct
+            # (and if not, the context is not touched). With images
+            # and their uploads the problem then is, that the uploaded
+            # image won't be shown in a partially erranous form
+            # because the new uploaded image is still not part of the
+            # context obeject and the image widget cannot work around
+            # that (it has no access to the new data). For this reason
+            # we set the image here and not in 'save()' or other
+            # methods.
+            self.context.passport = image
+            upload.seek(0)
+            self.passport_changed = True
         super(EditApplicantStudent, self).update()
         return