Load pages faster with scale of images uploaded to OSQA

I am thinking that pages containing images would load faster, if the images were scaled to a maximum 640x853 dimensions. I have a patch that does this. If you spot any problems please point them out.

Additionally, I would like the image appearing in the question or answer, to be a link to the original image. How do I achieve this? Where is it img= ... code rendered so that I can add an href?

diff -r 0e1e1886daf3 forum/views/writers.py
--- a/forum/views/writers.py    Tue Aug 09 09:41:43 2011 +0200
+++ b/forum/views/writers.py    Tue Aug 09 10:16:50 2011 +0200
@@ -20,6 +20,8 @@

from vars import PENDING_SUBMISSION_SESSION_ATTR

+from PIL import Image
+
 @csrf_exempt
 def upload(request):#ajax upload file to a question or answer
     class FileTypeNotAllow(Exception):
@@ -31,6 +33,24 @@

xml_template = "<result><msg><![CDATA[%s]]></msg><error><![CDATA[%s]]></error><file_url>%s</file_url></result>"

+    def scale_dimensions(width, height, longest_width, longest_height):
+        if width > height:
+            if width > longest_width:
+                ratio = longest_width*1./width
+                return (int(width*ratio), int(height*ratio))
+            elif height > longest_height:
+                ratio = longest_height*1./height
+                return (int(width*ratio), int(height*ratio))
+        else:
+            if height > longest_height:
+                ratio = longest_height*1./height
+                return (int(width*ratio), int(height*ratio))
+            elif width > longest_width:
+                ratio = longest_width*1./width
+                return (int(width*ratio), int(height*ratio))
+
+        return (width, height)
+
     try:
         f = request.FILES['file-upload']
         # check upload permission
@@ -39,7 +59,9 @@

# check file type
         try:
-            file_name_suffix = os.path.splitext(f.name)[1].lower()
+            (file_name_root, file_name_ext) = os.path.splitext(f.name)
+            file_name_suffix = file_name_ext.lower()
+            file_name_small = file_name_root + '.s' + file_name_ext
         except KeyError:
             raise FileTypeNotAllow()

@@ -55,6 +77,17 @@
         if size > float(settings.ALLOW_MAX_FILE_SIZE) * 1024 * 1024:
             storage.delete(new_file_name)
             raise FileSizeNotAllow()
+        
+        imageImage = Image.open(storage.open(new_file_name))
+        (width, height) = imageImage.size
+        (width_new, height_new) = scale_dimensions(width, height, longest_width=640, longest_height=853)
+        if ( width_new != width or height_new != height ) :
+                (width, height) = (width_new, height_new)
+       resizedImage = imageImage.resize((width, height))
+       new_file_name = storage.open("_".join(file_name_small.split()), 'wb')
+       resizedImage.save(new_file_name)
+       new_file_name.close()
+       new_file_name = storage.get_valid_name("_".join(file_name_small.split()))

result = xml_template % ('Good', '', str(settings.UPFILES_ALIAS) + new_file_name)
     except

UploadPermissionNotAuthorized: