WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 3 years ago

#13893 closed defect (bug) (fixed)

Switch has two cases for 'auto-draft'

Reported by: rvenable Owned by: MarkJaquith
Milestone: 3.1 Priority: low
Severity: minor Version: 3.0
Component: General Keywords:
Focuses: Cc:

Description (last modified by nacin)

http://core.trac.wordpress.org/browser/trunk/wp-admin/includes/meta-boxes.php?rev=15190#L72

		case 'draft':
		case 'auto-draft':
			_e('Draft');
			break;
		case 'auto-draft':
			_e('Unsaved');
			break;

Attachments (2)

meta-boxes.php.diff (344 bytes) - added by nathanrice 4 years ago.
Remove the first auto-draft case
meta-boxes.php.2.diff (357 bytes) - added by nathanrice 4 years ago.
Remove the second auto-draft case

Download all attachments as: .zip

Change History (10)

comment:1 wpmuguru4 years ago

That first auto-draft should probably come out.

comment:2 nacin4 years ago

  • Description modified (diff)
  • Owner set to MarkJaquith
  • Status changed from new to assigned

comment:3 nathanrice4 years ago

Remove the first 'auto-draft' case.

nathanrice4 years ago

Remove the first auto-draft case

nathanrice4 years ago

Remove the second auto-draft case

comment:4 nacin4 years ago

I think we should remove the second case, as that's how it's been functioning since it was checked in originally.

Either that, or move it to a future milestone, because it works as is.

comment:5 nathanrice4 years ago

Sorry, had those backwards.

meta-boxes.php.2.diff removes the first auto-draft case.
meta-boxes.php.diff removes the second auto-draft case.

comment:6 westi4 years ago

  • Milestone changed from 3.0 to 3.1

Lets fix this in 3.1

It isn't actually broken just dead code.

comment:7 nacin3 years ago

  • Milestone changed from Awaiting Triage to 3.1

comment:8 nacin3 years ago

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

(In [16324]) Remove dead code. props nathanrice, fixes #13893.

Note: See TracTickets for help on using tickets.