WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 8 months ago

Last modified 8 months ago

#18664 closed defect (bug) (maybelater)

Formatting/Cleanup for wp-admin/includes/class-wp-posts-list-table.php

Reported by: DH-Shredder Owned by: westi
Milestone: Priority: lowest
Severity: trivial Version:
Component: Formatting Keywords: has-patch
Focuses: Cc:

Description

wp-admin/includes/class-wp-posts-list-table.php is extremely hard to read.

Created this ticket to move conversation/patches for cleanup out of #17958

Attachments (1)

18664.diff (12.8 KB) - added by DH-Shredder 3 years ago.
First pass at formatting for entire file

Download all attachments as: .zip

Change History (7)

DH-Shredder3 years ago

First pass at formatting for entire file

comment:1 scribu3 years ago

  • Milestone changed from Awaiting Review to 3.3

+1.

comment:2 westi3 years ago

  • Owner set to westi
  • Priority changed from normal to lowest
  • Status changed from new to reviewing

comment:3 ryan3 years ago

  • Milestone changed from 3.3 to Future Release

comment:4 follow-up: c3mdigital8 months ago

  • Resolution set to maybelater
  • Status changed from reviewing to closed

Are we just refactoring this for readability? http://make.wordpress.org/core/2011/03/23/code-refactoring/

comment:5 in reply to: ↑ 4 nacin8 months ago

Replying to c3mdigital:

Are we just refactoring this for readability?

Yes, but this file could really benefit from it. That said, this patch is very old and would need a possibly major refresh.

comment:6 SergeyBiryukov8 months ago

  • Milestone Future Release deleted
Note: See TracTickets for help on using tickets.