Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#32191 closed defect (bug) (fixed)

Edit Post screen messages don't run preview link through filter

Reported by: wonderboymusic's profile wonderboymusic Owned by: pareshradadiya's profile pareshradadiya
Milestone: 4.3 Priority: normal
Severity: normal Version:
Component: Posts, Post Types Keywords: good-first-bug has-patch
Focuses: Cc:

Description

Should have apply_filters( 'preview_post_link', .... ) applied.

Attachments (3)

32191.diff (4.2 KB) - added by pareshradadiya 9 years ago.
Proposed patch
32191.2.diff (2.8 KB) - added by pareshradadiya 9 years ago.
Patch refreshed
32191.3.diff (3.1 KB) - added by pareshradadiya 9 years ago.
Duplicate hooks documented

Download all attachments as: .zip

Change History (12)

@pareshradadiya
9 years ago

Proposed patch

#1 @pareshradadiya
9 years ago

  • Keywords has-patch added; needs-patch removed

#2 @wonderboymusic
9 years ago

  • Keywords needs-refresh added

@pareshradadiya thanks, but not every link, just those wrapped in add_query_arg( 'preview', 'true', ... )

@pareshradadiya
9 years ago

Patch refreshed

#3 @pareshradadiya
9 years ago

  • Keywords needs-refresh removed

#4 @SergeyBiryukov
9 years ago

The new instances of the filter need a duplicate hook comment.

#5 @iseulde
9 years ago

  • Keywords needs-docs added

@pareshradadiya
9 years ago

Duplicate hooks documented

#6 @pareshradadiya
9 years ago

  • Keywords needs-docs removed

#7 @DrewAPicture
9 years ago

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

#8 @seanchayes
9 years ago

FWIW I downloaded and tested this with success.

#9 @wonderboymusic
9 years ago

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

In 32870:

Edit Post screen messages need to run preview URLs through the preview_post_link filter.

Props pareshradadiya.
Fixes #32191.

Note: See TracTickets for help on using tickets.