Opened 3 years ago
Closed 2 years ago
#15491 closed defect (bug) (fixed)
embed_oembed_html filter needs post ID
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.1 |
| Component: | Media | Version: | 3.1 |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
If you want to use the embed_oembed_html filter in the context of a particular post, currently you have to hope that the global request variable stars align in your favor. That is, you have to hope that the shortcode callback---in which the filter is used---is being called in the admin when $_GET['post'] is defined. Because it's a shortcode callback, that's not something you can have great confidence in.
Patch passes the post ID to the filter.
Attachments (1)
Change History (2)
comment:1
markjaquith — 2 years ago
- Resolution set to fixed
- Status changed from new to closed
Note: See
TracTickets for help on using
tickets.

(In [16740]) Pass post ID to embed_oembed_html filter to avoid global variable incantations. props filosofo. fixes #15491