Inconsistent 'the_title' filter

[jfarthing84]

In most places, 'the_title' filter passes two arguments, the title being filtered and the post ID. This is the desirable situation.

However, there are a few places in core code that do not pass the second argument while utilizing 'the_title' filter. This inconsistency is causing notices from my plugin because it expects the second argument in a callback attached to the filter.

+1 on this - my plugin is experiencing exactly the same warnings due to this inconsistency. Of the 14 instances of the_title filter being applied currently in trunk, two of them don't pass the ID along, which the patch fixes.

For some reason previous_post() and next_post() pass the whole $post object instead of $post->ID:
http://core.trac.wordpress.org/browser/tags/3.2.1/wp-includes/deprecated.php#L152

Though they are both deprecated, I wonder if that should be fixed too.

Related changeset: [2227]

Adds missing ID filter args, amends args in deprecated functions using post object to post ID

Well, next_post() is trying to pass in $nextpost as the second argument - and that doesn't appear to be defined anywhere in that function. I assume it's supposed to be $post (and then as per your comment, $post->ID). Patch attached in case this is decided to be fixed as well, though there will be an obvious backwards compatibility concern.

Just got rid of the pointless $arc_title variable and used $arcresult->post_title instead.

In [18907]:

Always pass the post ID as the second argument to the the_title filter. props GaryJ. fixes #16688