#8307: ImageFile doesn't use the width_field and height_field for cache
--------------------------------------------------+-------------------------
          Reporter:  [EMAIL PROTECTED]  |         Owner:  jacob                 
            Status:  assigned                     |     Milestone:  1.0         
          
         Component:  File uploads/storage         |       Version:  SVN         
          
        Resolution:                               |      Keywords:  imagefile 
height width
             Stage:  Accepted                     |     Has_patch:  1           
          
        Needs_docs:  1                            |   Needs_tests:  0           
          
Needs_better_patch:  0                            |  
--------------------------------------------------+-------------------------
Comment (by [EMAIL PROTECTED]):

 About your worries, the width and height values are updated every time the
 imagefield save method is fired, they could only be out of sync from a bug
 or a wrong use of the FileStorage mechanism (like setting the ._name field
 attr manually).
 Also I have been using the patch I submit and has been working fine for
 me.
 Without this patch the people that use remote filestorage backends (like
 S3, nirvanix or mogilefs), suffer a big performance hit for having to load
 the entire file every time a Model with an imagefield gets loaded. That
 means in this scenario a lot of more traffic, less performance and also a
 monetary cost.

 English is not my primary language, sorry if something is not clear.
 PD: im attaching another patch that could be more safe.

-- 
Ticket URL: <http://code.djangoproject.com/ticket/8307#comment:7>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to