#11981 closed defect (bug) (fixed)
Bad default data for icon_dir filter in in post-template.php
| Reported by: | chrisjean | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.1 |
| Component: | Media | Version: | 3.0 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
Description
Apparently an old batch of code was never updated in the get_attachment_icon_src function found in wp-includes/post-template.php. The default value for the icon_dir filter is get_template_directory()/images rather than ABSPATH.WPINC/images/crystal. This causes the returned icon path for mime type icons to not match the paired URL if the default directory is used.
The included patch has a fix that makes this filter's default value match the default value found in media.php and post.php.
The patch was generated off of 3.0-alpha.
Attachments (1)
Change History (5)
#3
@
16 years ago
- Resolution → fixed
- Status new → closed
Patch is stale. Trunk is somehow fixed. The only bad default describben here I could find is located in get_attachment_icon_src(), depreceted since 2.5 or so.
I close as fixed. If the problem still exists, please feel free to reopen and provide a fresh patch. I could not find this issue any longer.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Related: #6751