WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#24052 closed defect (bug) (fixed)

Don't remove extra content media when removing the first audio / video

Reported by: wonderboymusic's profile wonderboymusic Owned by: markjaquith's profile markjaquith
Milestone: 3.6 Priority: normal
Severity: normal Version: 3.6
Component: Post Formats Keywords: has-patch
Focuses: Cc:

Description

Only remove ONE piece of media from $post->post_content in setup_postdata(). Patching...

Attachments (3)

24052.diff​ (3.2 KB) - added by wonderboymusic 13 years ago.
24052.2.diff​ (3.2 KB) - added by wonderboymusic 13 years ago.
References are amazing
24052.3.diff​ (3.9 KB) - added by wonderboymusic 13 years ago.

Download all attachments as: .zip

Change History (8)

@wonderboymusic
13 years ago

#1 @wonderboymusic
13 years ago

  • Keywords has-patch added; needs-patch removed

#2 @Ipstenu
13 years ago

  • Cc ipstenu@… added

@wonderboymusic
13 years ago

References are amazing

@wonderboymusic
13 years ago

#3 @Ipstenu
13 years ago

Patch works in trunk :)

#4 @DrewAPicture
13 years ago

  • Version set to trunk

I almost feel like we could do without the extra parameters and accomplish the same thing with a couple of filters instead.

Your patch works in trunk for me as well, by the way.

#5 @markjaquith
13 years ago

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

In 23984:

Only remove ONE piece of media from the content in setup_postdata().

props wonderboymusic. fixes #24052.

Note: See TracTickets for help on using tickets.