WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 13 years ago

Last modified 13 years ago

#18664 closed defect (bug) (maybelater)

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

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

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 15 years ago.
First pass at formatting for entire file

Download all attachments as: .zip

Change History (7)

@kirasong
15 years ago

First pass at formatting for entire file

#1 @scribu
15 years ago

  • Milestone Awaiting Review → 3.3

+1.

#2 @westi
15 years ago

  • Owner set to westi
  • Priority normal → lowest
  • Status new → reviewing

#3 @ryan
15 years ago

  • Milestone 3.3 → Future Release

#4 follow-up: @c3mdigital
13 years ago

  • Resolution → maybelater
  • Status reviewing → closed

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

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

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