#21518 closed defect (bug) (fixed)
Remove 'hwstring_small' property from attachment metadata
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.5 | Priority: | low |
Severity: | minor | Version: | 3.4.1 |
Component: | Media | Keywords: | dev-feedback has-patch |
Focuses: | Cc: |
Description
We don't appear to use the 'hwstring_small' meta key anywhere, and as a series of html attributes, it doesn't fit in with the rest of the attachment meta keys. Thoughts on removing (and/or deprecating) it?
Attachments (2)
Change History (8)
#2
@
11 years ago
- Keywords has-patch added
Introduced in [2921], unused since [2958].
The patch also deprecates get_udims()
(unused since [6309]), which is essentially a copy of wp_shrink_dimensions()
, already deprecated in [12823]:
http://core.trac.wordpress.org/browser/tags/3.4.1/wp-admin/includes/deprecated.php#L39
#3
@
11 years ago
- Owner set to nacin
- Resolution set to fixed
- Status changed from new to closed
In [21808]:
#4
@
11 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
21518.2.patch corrects @return
values, which are reversed:
http://core.trac.wordpress.org/browser/tags/3.4.2/wp-includes/media.php#L278
Also adds @see
/@deprecated
for consistency with other functions.
#6
in reply to:
↑ 1
@
7 years ago
Replying to nacin:
Would be good to track down where it entered core and why. Sounds like it may be related to the old wp_get_attachment_thumb_file() and friends, before we had intermediate sizes.
Traced it as far as https://core.trac.wordpress.org/browser/tags/2.1/wp-admin/admin-functions.php#L2112
I'm fine with it vanishing. At worst, someone's image is no longer properly bound. Looks like it may not have been properly bound, anyway — it is based on an image downsize of 128 x 96, rather than any options.
Would be good to track down where it entered core and why. Sounds like it may be related to the old wp_get_attachment_thumb_file() and friends, before we had intermediate sizes.