do_action(quick_edit_custom_box) results in duplicates

[nerrad]

I noticed that there were some changes in this hook for WP3.1 - what's happening, is when I hook into the action I'm getting a double output. Trying to narrow down the cause now but thought I'd throw this up here in case there's something obvious I'm missing.

Attached is an example screenshot of my plugin in action using this hook on WP3.1

[nerrad]

never mind. disregard. pre 3.1 $column_name didn't have to be accounted for (must have been a bug?). The multiple output is for each $column.

[nacin]

3.0:

./wp-admin/includes/template.php:59:
do_action( 'quick_edit_custom_box', $column_name, $type, $taxonomy );

./wp-admin/includes/template.php:1147:
do_action( $bulk ? 'bulk_edit_custom_box' : 'quick_edit_custom_box', $column_name, $screen->post_type );

3.1:

./wp-admin/includes/class-wp-terms-list-table.php:356:
do_action( 'quick_edit_custom_box', $column_name, $tax->name );

./wp-admin/includes/class-wp-posts-list-table.php:990:
do_action( $bulk ? 'bulk_edit_custom_box' : 'quick_edit_custom_box', $column_name, $screen->post_type );

[nacin]

I see a regression. I'm not sure why this change was made.

In 3.0, inline_edit_term_row() took $type ('edit-tags', 'categories', or 'edit-link-categories') and $taxonomy.

In 3.0 we didn't account for the merger of 'edit-tags' and 'categories'. It appears people didn't rely on this argument as we hadn't seen a report. That's fine with me.

But, the taxonomy argument has now moved. That's a problem for people adding custom column data here.

We should just pass 'edit-tags' consistently and bump the argument down one. This is also helpful to distinguish between post type and taxonomy usage of the filter.

[nacin]

Blamed it to #15285. Reopening that one instead.