WordPress.org

Make WordPress Core

#24484 closed task (blessed) (fixed)

Lose the "remove" support for media extraction functions

Reported by: markjaquith Owned by: ocean90
Milestone: 3.6 Priority: high
Severity: blocker Version: 3.6
Component: Post Formats Keywords: needs-patch
Focuses: Cc:

Attachments (4)

24484.diff (16.8 KB) - added by markjaquith 21 months ago.
Rough first pass
24484.ut.patch (2.5 KB) - added by ocean90 20 months ago.
24484.patch (2.5 KB) - added by ocean90 20 months ago.
24484.ut.2.patch (1.3 KB) - added by ocean90 20 months ago.

Download all attachments as: .zip

Change History (13)

@markjaquith21 months ago

Rough first pass

comment:1 @markjaquith21 months ago

There's a rough first pass.

comment:2 @markjaquith21 months ago

In 24400:

Lose content removal and splitting from the media extraction functions.

see #24484

comment:3 @greenshady21 months ago

  • Cc justin@… added

Is there a particular reason this feature of the media functions is being removed? As a theme author, I found this functionality to be the most promising because I could extract the media on single posts and place it outside of the post content. It definitely opened up a lot of design opportunities that were hard to achieve before. Now, these functions only seem useful for archive-type pages where I might show the media + excerpt.

comment:4 @wonderboymusic21 months ago

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

This can be added back in the plugin, when it exists

comment:5 @ocean9020 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Unit tests need to be updated.

Last edited 20 months ago by ocean90 (previous) (diff)

@ocean9020 months ago

comment:6 @ocean9020 months ago

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

comment:7 @ocean9020 months ago

  • Keywords needs-patch added
  • Resolution fixed deleted
  • Status changed from closed to reopened

We still have the remove support in get_content_url(). Should we remove it here too?

These lines should be removed:

comment:8 @markjaquith20 months ago

  • Severity changed from critical to blocker

Yes.

@ocean9020 months ago

@ocean9020 months ago

comment:9 @ocean9020 months ago

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

In 24554:

Lose content removal and splitting from get_content_url(). And remove unneeded lines from [24400].

fixes #24484.

Note: See TracTickets for help on using tickets.