Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#13893 closed defect (bug) (fixed)

Switch has two cases for 'auto-draft'

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

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 16 years ago.
Remove the first auto-draft case
meta-boxes.php.2.diff​ (357 bytes ) - added by nathanrice 16 years ago.
Remove the second auto-draft case

Download all attachments as: .zip

Change History (10)

#1 @wpmuguru
16 years ago

That first auto-draft should probably come out.

#2 @nacin
16 years ago

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

#3 @nathanrice
16 years ago

Remove the first 'auto-draft' case.

@nathanrice
16 years ago

Remove the first auto-draft case

@nathanrice
16 years ago

Remove the second auto-draft case

#4 @nacin
16 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.

#5 @nathanrice
16 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.

#6 @westi
16 years ago

  • Milestone 3.0 → 3.1

Lets fix this in 3.1

It isn't actually broken just dead code.

#7 @nacin
16 years ago

  • Milestone Awaiting Triage → 3.1

#8 @nacin
16 years ago

  • Resolution → fixed
  • Status assigned → closed

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

Note: See TracTickets for help on using tickets.