WordPress.org

Make WordPress Core

Opened 5 weeks ago

Last modified 3 weeks ago

#51014 reviewing defect (bug)

Improve the structure format on the class-wp-media-list-table file

Reported by: Laxman Prajapati Owned by: SergeyBiryukov
Milestone: 5.6 Priority: normal
Severity: normal Version: trunk
Component: Administration Keywords: good-first-bug has-patch
Focuses: coding-standards Cc:

Description

Improve the structure format of PHP code on the class-wp-media-list-table file for the understanding of which section are where closed.

Attachments (2)

51014.patch (5.7 KB) - added by Laxman Prajapati 5 weeks ago.
Created patch.
51014-2.patch (5.4 KB) - added by hansjovisyoast 3 weeks ago.

Download all attachments as: .zip

Change History (7)

@Laxman Prajapati
5 weeks ago

Created patch.

#1 @Laxman Prajapati
5 weeks ago

  • Keywords has-patch added

#2 @SergeyBiryukov
5 weeks ago

  • Component changed from General to Administration
  • Milestone changed from Awaiting Review to 5.6
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#3 @Laxman Prajapati
4 weeks ago

  • Keywords good-first-bug added

#4 @Laxman Prajapati
3 weeks ago

Any update on this?

#5 @hansjovisyoast
3 weeks ago

When reviewing we (a few colleagues: @justinahinon and @noudwordpress and myself) noticed a few code standard issues in the patch, which we addressed in 51014-2.patch:

  • The if statement on lines 272 to 276 was not correctly indented. It had one tab too many.
  • Line 425 had a trailing whitespace.

There is one code standard issue left about a strict comparison:

FILE: src/wp-admin/includes/class-wp-media-list-table.php
-------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------
 656 | WARNING | Found: ==. Use strict comparisons (=== or !==).
     |         | (WordPress.PHP.StrictComparisons.LooseComparison)
-------------------------------------------------------------------------------------------------------------------

Since this is a pre-existing issue, where fixing it could change the behavior, we decided to leave it as is.

Last edited 3 weeks ago by hansjovisyoast (previous) (diff)
Note: See TracTickets for help on using tickets.