[wp-trac] [WordPress Trac] #56244: Use a consistent parameter name for functions accepting a comment ID or object

WordPress Trac noreply at wordpress.org
Thu Jul 21 04:59:20 UTC 2022


#56244: Use a consistent parameter name for functions accepting a comment ID or
object
----------------------------+-------------------------------------
 Reporter:  SergeyBiryukov  |       Owner:  SergeyBiryukov
     Type:  enhancement     |      Status:  accepted
 Priority:  normal          |   Milestone:  6.1
Component:  Comments        |     Version:
 Severity:  normal          |  Resolution:
 Keywords:                  |     Focuses:  docs, coding-standards
----------------------------+-------------------------------------

Comment (by peterwilsoncc):

 I'll try to do some testing of re-normalization after the filter.

 With the patch applied `isset( $commentdata['user_ID'] )` will always
 return `true` after the filter, which wasn't necessarily the case before.
 My reading of the code is that once the filter runs, `user_ID` will take
 priority over `user_id` which seems the opposite of what you want here.

 Determining whether a plugin has modified the user ID seems complicated,
 sadly.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/56244#comment:16>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list