WordPress.org

Make WordPress Core

#24052 closed defect (bug) (fixed)

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

Reported by: wonderboymusic Owned by: 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 12 months ago.
24052.2.diff (3.2 KB) - added by wonderboymusic 12 months ago.
References are amazing
24052.3.diff (3.9 KB) - added by wonderboymusic 12 months ago.

Download all attachments as: .zip

Change History (8)

wonderboymusic12 months ago

comment:1 wonderboymusic12 months ago

  • Keywords has-patch added; needs-patch removed

comment:2 Ipstenu12 months ago

  • Cc ipstenu@… added

wonderboymusic12 months ago

References are amazing

wonderboymusic12 months ago

comment:3 Ipstenu12 months ago

Patch works in trunk :)

comment:4 DrewAPicture12 months 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.

comment:5 markjaquith12 months 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.