Ticket #7947 (closed defect (bug): fixed)
Wrong "icon_dir" action
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 2.7 |
| Component: | General | Version: | |
| Severity: | trivial | Keywords: | icon_dir, icon_dir_uri |
| Cc: |
Description
When I checked for "icon_dir" action I figure out there are 2 action :
- icon_dir : absolut path to the icon set directory
- icon_dir_uri : uri to the icon set directory
(see a good example at lines 2612 and 2613 in wp-includes/post.php)
In file wp-includes/media.php on line 507 the wrong of this 2 actions is called. It should be icon_dir_uri instead of icon_dir in order to keep it logical:
$icon_dir = apply_filters( 'icon_dir_uri', includes_url('images/crystal') );
Otherwise if you want to hook icon_dir, you have to check if you are dealing with a path or a uri.
Change History
Note: See
TracTickets for help on using
tickets.

(In [9280]) icon_dir_uri filter instead of icon_dir. Props jmini. fixes #7947