WordPress.org

Make WordPress Core

Opened 4 months ago

Closed 4 weeks ago

#46902 closed defect (bug) (fixed)

Coding Standard fixes for wp-admin/includes/class-walker-nav-menu-edit.php

Reported by: sudhiryadav Owned by: audrasjb
Milestone: 5.3 Priority: normal
Severity: minor Version:
Component: Menus Keywords: has-patch commit
Focuses: administration, coding-standards Cc:

Description

Coding Standard fixes for wp-admin/includes/class-walker-nav-menu-edit.php
Fix Yoda condition.

Attachments (2)

46902.patch (627 bytes) - added by sudhiryadav 4 months ago.
46902.2.diff (675 bytes) - added by audrasjb 4 weeks ago.
Patch refreshed

Download all attachments as: .zip

Change History (11)

@sudhiryadav
4 months ago

#1 @sudhiryadav
4 months ago

  • Keywords has-patch added

#2 @SergeyBiryukov
4 months ago

  • Component changed from General to Menus
  • Focuses administration coding-standards added

#3 @audrasjb
4 months ago

  • Keywords needs-testing added
  • Milestone changed from Awaiting Review to 5.3

Hi @sudhiryadav, welcome to WordPress Core Trac and thanks for your first ticket!

The patch looks good at a glance. Adding needs-testing keyword and moving it to 5.3.

Cheers,
Jb

#4 @audrasjb
3 months ago

  • Keywords needs-testing removed
  • Owner set to audrasjb
  • Severity changed from normal to minor
  • Status changed from new to accepted

#5 @audrasjb
3 months ago

  • Priority changed from normal to low

#6 @audrasjb
4 weeks ago

  • Keywords needs-refresh added
  • Priority changed from low to normal

The patch doesn't apply cleanly, adding needs-refresh keyword.

@audrasjb
4 weeks ago

Patch refreshed

#7 @audrasjb
4 weeks ago

  • Keywords commit added; needs-refresh removed

In 46902.2.diff: patch refresh. Tested on my side.
Adding commit keyword.

#8 @SergeyBiryukov
4 weeks ago

Per comment:41:ticket:47632, there's still a discussion to be had over whether we keep using Yoda Conditions in the coding standards or not.

However, since this is only violation in this file, I don't have any objections to this going in.

#9 @SergeyBiryukov
4 weeks ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 45697:

Coding Standards: Fix WPCS violations in wp-admin/includes/class-walker-nav-menu-edit.php.

Props sudhiryadav, audrasjb.
Fixes #46902.

Note: See TracTickets for help on using tickets.