Make WordPress Core

Opened 13 years ago

Closed 9 years ago

Last modified 9 years ago

#20788 closed enhancement (fixed)

Add widget instance argument to widget_links_args filter

Reported by: ragulka's profile ragulka Owned by: wonderboymusic's profile wonderboymusic
Milestone: 4.4 Priority: normal
Severity: normal Version: 3.3
Component: Widgets Keywords: dev-feedback has-patch commit
Focuses: Cc:

Description

Using the widget_links_args filter (used in the default Links Widget class) as it is right now is a bit useless because one does not have access to the widget instance that applies the filter.

Widget instance may contain custom variables set by custom fields that may have been added to the widget form by plugins or themes using other filters.

Being able to access widget instance from within the filter function gives developers lots of possibilities to customize the widget.

My suggestion is simple: add $instance argument to the widget_links_args filter, like this (wp-includes/default-widget.php line 113):

wp_list_bookmarks(apply_filters('widget_links_args', array(
	'title_before' => $before_title, 'title_after' => $after_title,
	'category_before' => $before_widget, 'category_after' => $after_widget,
	'show_images' => $show_images, 'show_description' => $show_description,
	'show_name' => $show_name, 'show_rating' => $show_rating,
	'category' => $category, 'class' => 'linkcat widget'
), $instance ));

Attachments (3)

20788.patch (965 bytes) - added by SergeyBiryukov 13 years ago.
20788.2.diff (1.0 KB) - added by MikeHansenMe 10 years ago.
Refresh + docs
20788.diff (1.9 KB) - added by DrewAPicture 9 years ago.
Refresh + cleanup

Download all attachments as: .zip

Change History (11)

#1 @SergeyBiryukov
13 years ago

  • Keywords has-patch added

@MikeHansenMe
10 years ago

Refresh + docs

#3 @chriscct7
9 years ago

  • Version changed from 3.3.2 to 3.3

@DrewAPicture
9 years ago

Refresh + cleanup

#4 @DrewAPicture
9 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 4.4

20788.diff refreshes the patch and simplifies the actual filter call by splitting out the args array to a separate variable. Adds the new parameter + docs

Moving for commit consideration.

#5 @chriscct7
9 years ago

  • Owner set to DrewAPicture
  • Status changed from new to assigned

#6 @DrewAPicture
9 years ago

  • Owner DrewAPicture deleted

#7 @wonderboymusic
9 years ago

  • Owner set to wonderboymusic
  • Resolution set to fixed
  • Status changed from assigned to closed

In 33971:

WP_Widget_Links: pass widget instance to widget_links_args filter

Props SergeyBiryukov, MikeHansenMe, DrewAPicture.
Fixes #20788.

#8 @DrewAPicture
9 years ago

In 33978:

Docs: Use the $widget_links_args parameter in the hook doc for the widget_links_args filter, introduced when the args array was split out to a variable in [33971].

See #20788.

Note: See TracTickets for help on using tickets.