Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#23937 closed defect (bug) (fixed)

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

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


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 3 years ago.

Download all attachments as: .zip

Change History (10)

#1 @ocean90
3 years ago

  • Keywords has-patch added; dev-feedback removed
  • Milestone changed from Awaiting Review to 3.6

3 years ago

#2 @mgibbs189
3 years ago

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

#3 @ocean90
3 years ago

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

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

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

#4 @mgibbs189
3 years ago

The patch works correctly only when SCRIPT_DEBUG is true.

No plugins, latest Chrome, WP trunk, Win7.

#5 @ocean90
3 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
3 years ago

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

#7 @mgibbs189
3 years ago

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


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

#8 @ocean90
3 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

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

#9 @SergeyBiryukov
3 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from reopened to closed

In 23909:

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

Note: See TracTickets for help on using tickets.