Make WordPress Core

Opened 13 years ago

Closed 11 years ago

Last modified 11 years ago

#18664 closed defect (bug) (maybelater)

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

Reported by: kirasong's profile kirasong Owned by: westi's profile 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 kirasong 13 years ago.
First pass at formatting for entire file

Download all attachments as: .zip

Change History (7)

@kirasong
13 years ago

First pass at formatting for entire file

#1 @scribu
13 years ago

  • Milestone changed from Awaiting Review to 3.3

+1.

#2 @westi
13 years ago

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

#3 @ryan
12 years ago

  • Milestone changed from 3.3 to Future Release

#4 follow-up: @c3mdigital
11 years 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/

#5 in reply to: ↑ 4 @nacin
11 years 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.

#6 @SergeyBiryukov
11 years ago

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