Make WordPress Core

Opened 4 years ago

Last modified 7 months ago

#17262 new enhancement

wp_get_attachment_thumb_file is always false — at Version 12

Reported by: lonnylot Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 3.0
Component: Media Keywords: needs-patch needs-unit-tests
Focuses: Cc:

Description (last modified by DrewAPicture)

The issue is that on line 3863 we always search for $imagedata['thumb'] and it never exists. Instead we have $imagedata['thumbnail']. This always exists.

Change History (16)

@lonnylot4 years ago

comment:1 @scribu4 years ago

  • Keywords has-patch removed

Please submit patches as svn diffs. Tutorials can be found on the trac homepage.

Last edited 4 years ago by scribu (previous) (diff)

@lonnylot4 years ago

wp-includes/post.php diff

comment:2 @lonnylot4 years ago

Sorry for the delay! Added the correct diff off of trunk rev. 18191

comment:3 @nacin4 years ago

  • Version changed from 3.2 to 3.0

So does wp_get_attachment_thumb_file() ever work? That function should probably be deprecated in favor of the various other functions now.

comment:4 @lonnylot4 years ago

As far as I can tell not unless you add a custom image type called 'thumb'

comment:5 @azaozz4 years ago

@nacin is right, deprecate +1.

comment:6 @scribu4 years ago

  • Keywords needs-patch 3.3-early added
  • Milestone changed from Awaiting Review to Future Release

comment:7 @tw21133 years ago

  • Cc michael.d.beckwith@… added

@wonderboymusic3 years ago

comment:8 @wonderboymusic3 years ago

  • Keywords has-patch added; needs-patch removed
  • Milestone changed from Future Release to 3.6

17262.2.diff updates wp_get_attachment_thumb_file() so it works in trunk. The previous patch did not have current code in mind. It's not immediately obvious to me what wp_get_attachment_thumb_file() and wp_get_attachment_thumb_url() would be deprecated in favor of.

comment:9 @nacin3 years ago

'thumb' is the old-style thumbnail, prior to proper intermediate sizes ('thumbnail' / 'medium' / 'large').

wp_get_attachment_thumb_url() does return the newer 'thumbnail' if it exists, and falls back to wp_get_attachment_thumb_file() if it does not.

You can see a bit of code history in image_downsize(): "fall back to the old thumbnail" in a code comment.

Not sure what _file() should return in a deprecated situation. _url() is basically the old-school form of wp_get_attachment_image_src( $size = 'thumbnail' ), which returns array( $src, $width, $height ). Which is kind of lame.

So, perhaps they can both be updated to work on 'thumbnail', and then left, as wp_get_attachment_thumb_url() is definitely easier, nicer, and more obvious to use than wp_get_attachment_image_src()['url'], and according to MarkJaquith, is widely used by plugins (probably for that reason).

Last edited 3 years ago by SergeyBiryukov (previous) (diff)

comment:10 @SergeyBiryukov3 years ago

#23556 was marked as a duplicate.

@JoshuaAbenazer3 years ago

This one first checks for the "fall back to the old thumbnail" and if that returns false then goes ahead and checks according to the revised array

comment:11 @SergeyBiryukov2 years ago

#10492 was marked as a duplicate.

comment:12 @DrewAPicture2 years ago

  • Description modified (diff)
Note: See TracTickets for help on using tickets.