remove_filter( ‘comment_author’, ‘floated_admin_avatar’ ); doesnât work

March 8, 20231 Views

In a small plugin Iâve written to add an IP address column to the comments list I want to remove the avatars.
In line 156 I tried to use remove_filter:

remove_filter( 'comment_author', 'floated_admin_avatar', 50 );

Well â¦ I know this function fails sometimes, but why does it fail here?

My workaround is:

$_GET['comment_status'] == 'spam' 
    and add_filter( 'pre_option_show_avatars', '__return_zero' );

I guess I stared too long at this code to see what went wrong. I need a hint into the right direction.

Post Views: 1

2 comments

Anonymous says:

March 8, 2023 at 9:55 pm
As usual with hooks this is issue of timing.
- your init function is hooked to admin load process, which works fine for most things;
- however in this specific case function is added to filter in constructor of WP_Comments_List_Table class, and object is created in edit-comments.php after admin loader had been processed.
In my plugin for similar stuff I am removing it at manage_edit-comments_columns hook.

Simplified example from my class:
```
static function on_load() {

    add_action( 'admin_init', array( __CLASS__, 'admin_init' ) );
}

static function admin_init() {

        add_filter( 'manage_edit-comments_columns', array( __CLASS__, 'manage_columns' ) );
}

static function manage_columns( $columns ) {

    remove_filter( 'comment_author', 'floated_admin_avatar' );

    // stuff

    return $columns;
}
```
Log in to Reply
Anonymous says:

March 8, 2023 at 9:56 pm
May be its the priority 50. Instead let it be default.

From http://codex.wordpress.org/Function_Reference/remove_filter

$priority
(int) (optional) The priority of the function (as defined when the function was originally hooked).
Default: 10

So change
```
remove_filter( 'comment_author', 'floated_admin_avatar', 50 );
```
to
```
remove_filter( 'comment_author', 'floated_admin_avatar');
```
Log in to Reply