WordPress.org

Make WordPress Core

Opened 17 months ago

Last modified 6 months ago

#43406 new defect (bug)

CS: Fix violations for wp-trackback.php

Reported by: GaryJ Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Pings/Trackbacks Keywords: has-patch 2nd-opinion
Focuses: coding-standards Cc:

Description

Address code standard violations for wp-trackback.php.

Attachments (1)

wp-trackback.diff (1.0 KB) - added by GaryJ 17 months ago.

Download all attachments as: .zip

Change History (8)

@GaryJ
17 months ago

#1 @GaryJ
17 months ago

  • Focuses coding-standards added
  • Keywords has-patch added

This ticket was mentioned in Slack in #core-coding-standards by garyj. View the logs.


17 months ago

#3 @jrf
17 months ago

  • Version set to trunk

+1

Just a small question - why not have the first whitelist on the line above the variable declaration as well ?

Oh and maybe add -- renaming WP global would be BC break or something, so it's documented why the variable name is exempt.

#4 @netweb
17 months ago

  • Milestone changed from Awaiting Review to 5.0

#5 @SergeyBiryukov
17 months ago

  • Component changed from General to Pings/Trackbacks

#6 @johnbillion
9 months ago

  • Milestone changed from 5.0 to 5.1

#7 @pento
6 months ago

  • Keywords 2nd-opinion added
  • Milestone changed from 5.1 to Future Release
  • Version trunk deleted

We actually could rename $comment_post_ID, I suspect without breaking back compat.

See #23231 for an example of how it's been done in the past. This is a bit of a bigger thing that just adding a phpcs:ignore comment, though. 🙂

Note: See TracTickets for help on using tickets.