preprocess_comment filter does not contain old user_ID field for user_id, instead it has new user_id field

[cookiesowns]

in wp-includes/comment.php, line 1680, the old $commentdata['user_ID'] field is not being set.

As a result, all old filters using the old field name in preprocess_comment filter will not have the proper parameter set.

I'm not sure what to do in this case, but we should update the PHPDOC above on line 1656 as $user_ID technically doesn't exist anymore..

Should we set both fields when passing through to preprocess_comment or?

[cookiesowns]

It also appears the documentation for preprocess_comment is still using user_ID instead of user_id in the example as well.

​https://codex.wordpress.org/Plugin_API/Filter_Reference/preprocess_comment

[SergeyBiryukov]

Hi @cookiesowns, welcome to Trac!

This code has not changed since [28915], see #23231.

Could you provide the steps to reproduce the issue on a clean install?

[johnbillion]

Culprit: [35435].

The workaround is to switch to the user_id field instead of user_ID.

[cookiesowns]

Indeed.

Given this code here...

<?php

function filter_comments( $commentdata ) {

  // This will fail due to new change

  if($commentdata['user_ID']) {

    // Do something with user
  }

}


add_filter( 'preprocess_comment', 'filter_comments' );

That code above will break with the new code.

<?php

function filter_comments( $commentdata ) {

  // This is how we're supposed to filter comments now apparently..

  if($commentdata['user_id']) {

    // Do something with user
  }

}


add_filter( 'preprocess_comment', 'filter_comments' );

Documentation has not yet been changed for this simple syntax change.

I'm thinking due to backwards compatibility, we might want to send both user_ID and user_id ? Or should the documentation just be changed to account for this?

[johnbillion]

34997.diff​ fixes this and adds a test.

[johnbillion]

In 36038:

Comments: When a comment is submitted, ensure the user_ID element in the array that's passed to the preprocess_comment filter gets populated.

Fixes #34997

[johnbillion]

In 36039:

Comments: When a comment is submitted, ensure the user_ID element in the array that's passed to the preprocess_comment filter gets populated.

Merges [36038] to the 4.4 branch.

Fixes #34997

[cookiesowns]

Replying to johnbillion:

In 36039:

Comments: When a comment is submitted, ensure the user_ID element in the array that's passed to the preprocess_comment filter gets populated.

Merges [36038] to the 4.4 branch.

Fixes #34997

@johnbillion

I see in your test, you're looking for both user_ID and user_id. However in the patch for comment.php, you've changed user_id, back to user_ID.

Shouldn't this break the test?

Another thing is, what do we do for users that have already worked-around this change by looking for user_id, instead of user_ID?

[johnbillion]

Replying to cookiesowns:

I see in your test, you're looking for both user_ID and user_id. However in the patch for comment.php, you've changed user_id, back to user_ID.

Shouldn't this break the test?

The wp_new_comment() function populates the `user_id` element when `user_ID` is passed. Hence, both elements are present in the $commentdata array which gets passed to the preprocess_comment filter.