Changes between Initial Version and Version 1 of Ticket #51780, comment 22
- Timestamp:
- 02/24/2023 12:09:29 AM (4 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #51780, comment 22
initial v1 5 5 However I don't see a reason why plugin authors cannot use `image_get_intermediate_size()` for this. That's probably currently used anyways. 6 6 7 Another serious objection here is that this patch slows down `get_attached_file()` as it always does another DB request , no matter what file is attached.7 Another serious objection here is that this patch slows down `get_attached_file()` as it always does another DB request looking for image meta, no matter what file is attached. 8 8 9 9 In my opinion this patch should be reverted and moved to 6.3 or future release allowing for a better review whether this is really needed, and for better patch (new function) that doesn't reduce performance of get_attached_file()`.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)