Make WordPress Core

Changes between Initial Version and Version 5 of Ticket #32117


Ignore:
Timestamp:
10/10/2015 05:58:03 AM (9 years ago)
Author:
SergeyBiryukov
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #32117 – Description

    initial v5  
    2828This is at least confusing, make it difficult to get the URL of the file - each size need to be then requested separately by wp_get_attachment_image_src function.
    2929
    30 IMO optimal solution would be to use full path in both $metadata['file'] and $metadata['sizes'][$size]['file'] so the same name would have the same structure. But i do not know if it wouldn't have some compatibility issues.
     30IMO optimal solution would be to use full path in both `$metadata['file']` and `$metadata['sizes'][$size]['file']` so the same name would have the same structure. But i do not know if it wouldn't have some compatibility issues.
    3131
    3232Less optimal imo would be to add there also the path - it can be there only once in the top level, as all sizes are currently always in the same folder, but i think this could lock us from possible changes / plugin modifications etc. E.g. I think that it would be great, if it would be possible (and even default) to have size name, as a folder, so that we would have thumbnails in uploads/thumbnail, medium size in uploads/medium ... - this would highly reduce the number of images in one folder in default settings and would reduce the problems with displaying them on most systems. And also if we would want to delete some defined size, we could simply delete one folder and save space.
    3333
    34 So the second optimal would be to show the path in sizes[$size] subarray, e.g.
     34So the second optimal would be to show the path in `sizes[$size]` subarray, e.g.
    3535
    3636{{{