Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#13876 closed defect (bug) (fixed)

Favourite Actions for Editing a post shows 'Edit Post'

Reported by: dd32's profile dd32 Owned by: ryan's profile ryan
Milestone: 3.0 Priority: normal
Severity: normal Version: 3.0
Component: Posts, Post Types Keywords: has-patch needs-testing
Focuses: Cc:


Under 2.9, when editing a post, The default Favourite Action would display "New Post". Under 3.0 it displays "Edit Post"

furthermore, 3.0 reads "Edit Post" when on the New Post screen, whereas 2.9 reads "Edit Posts"

I can track this town to $current_Screen being 'post' not 'edit-{$post_type}' as favorite_actions expects.

Attachments (2)

13876.diff (1.6 KB) - added by duck_ 14 years ago.
13876.2.diff (2.9 KB) - added by ryan 14 years ago.

Download all attachments as: .zip

Change History (9)

#1 @nacin
14 years ago

  • Keywords needs-patch added

#2 follow-up: @dd32
14 years ago

I attempted changing it in set_current_screen() only to find this set of code in post.php overriding it.. I'm just too lost in the screen code and what uses it right now to patch it.

14 years ago

#3 @duck_
14 years ago

  • Keywords has-patch needs-testing added; needs-patch removed
  • Version set to 3.0

Didn't go for changing the $current_screen since quite a few things rely on it, so here's a potential fix with minimal changes.

#4 in reply to: ↑ 2 @nacin
14 years ago

Replying to dd32:

I'm just too lost in the screen code and what uses it right now to patch it.

Thinking ryan would be the best person to review all this.

#5 @nacin
14 years ago

  • Owner set to ryan
  • Status changed from new to assigned

14 years ago

#6 @ryan
14 years ago

Patch adds an action field to the screen that is set to add for -new.php and -add.php screens. This is safer than changing the edit and add ids to be distinct.

#7 @ryan
14 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.