Extend comment search to support user_id as argument

[neoxx]

Hi,

what do you think about the idea of extending _wp_get_comment_list in wp-admin/includes/template.php in a way that we can also query for user-ids?

I know, that the user_id will only be stored for comments of logged-in users, though I would need this search functionality not only, but also for ​my count-comments plugin. Currently, I implemented a link from users.php to edit-comments.php using
?s=user_display_name which obviously results in missed comments as well as false positives.

Just to give the idea a start, I've attached a patch prototype that extends handling of $_REQUEST['s']. - Or would we prefer a separate argument?

[scribu]

As part of my GSoC project, I have removed _wp_get_comment_list() in favor of get_comments(), which already has a 'user_id' argument.

[scribu]

Here is how it is currently:

​http://gsoc.trac.wordpress.org/browser/2010/scribu/wp-admin/includes/list-tables.php?rev=542#L670

[neoxx]

That's good to know. - Imho the best way would be to have the 'user_id' query-able by a $_REQUEST variable handed over to users.php (same as edit.php?author=nn for posts).

[scribu]

See #14579.

[scribu]

[15491]

[neoxx]

nice update :) but how is it now possible to query for the user-id with a $_REQUEST-parameter?

[neoxx]

updated patch for default-list-tables

[scribu]

(In [15887]) Allow filtering comments by user_id via URL. Props neoxx. Fixes #14163

[scribu]

(In [15888]) Revert accidental changes to WP_Query made in [15887]. See #14163