WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#23937 closed defect (bug) (fixed)

"Restore This Revision" button redirects to /wp-admin/null

Reported by: mgibbs189 Owned by: SergeyBiryukov
Priority: normal Milestone: 3.6
Component: Revisions Version: 3.6
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

Steps to reproduce:

  1. Create a new page
  2. Make content edits several times (save each time)
  3. Click on a revision link within the Revisions meta box
  4. Click on the "Restore This Revision" button

Attachments (1)

23937.patch​ (391 bytes ) - added by ocean90 13 years ago.

Download all attachments as: .zip

Change History (10)

#1 @ocean90
13 years ago

  • Keywords has-patch added; dev-feedback removed
  • Milestone Awaiting Review → 3.6

@ocean90
13 years ago

#2 @mgibbs189
13 years ago

#23937 doesn't solve the whole problem. The restore() view doesn't seem to be getting triggered at all.

#3 @ocean90
13 years ago

Then I neeed some more infos. Which browser, OS? Any plugins enabled, SCRIPT_DEBUG is true?

Edit:
If it doesn't trigger, then you shouldn't have been redirected.

Last edited 13 years ago by ocean90 (previous) (diff)

#4 @mgibbs189
13 years ago

The patch works correctly only when SCRIPT_DEBUG is true.

No plugins, latest Chrome, WP trunk, Win7.

#5 @ocean90
13 years ago

  • Keywords commit added

The patch works correctly only when SCRIPT_DEBUG is true.

And that's okay. The minified version will be generated when the patch lands in.

#6 @mgibbs189
13 years ago

Ah, didn't realize that SCRIPT_DEBUG toggles .min -- my bad :)

#7 @mgibbs189
13 years ago

  • Resolution → fixed
  • Status new → closed

[deleted]

Last edited 13 years ago by mgibbs189 (previous) (diff)

#8 @ocean90
13 years ago

  • Resolution fixed
  • Status closed → reopened

Tickets are marked as fixed when a changeset is commited to trunk which fixes the issue.

#9 @SergeyBiryukov
13 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status reopened → closed

In 23909:

Use correct id attribute. props ocean90. fixes #23937.

Note: See TracTickets for help on using tickets.