Opened 4 years ago
Closed 3 years ago
#11117 closed enhancement (fixed)
Add hook to allow modification of SimplePie options
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.0 |
| Component: | Feeds | Version: | 2.9 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: |
Description
I suggest an action is added to to the function fetch_feed() in wp-includes/feed.php. This would allow someone to add options to a SimplePie fetch without unnecessarily duplicating the function.
I also don't see a harm in allowing for different cache durations depending on the url. I would find that to be important context. So, I've also added the url as an argument to the wp_feed_cache_transient_lifetime filter.
Patch attached.
Attachments (3)
Change History (9)
Replying to scribu:
Should probably use do_action_ref_array() instead of do_action().
Good call, not sure how I missed that. Will patch up as array(&$feed, &$url).
- Keywords commit added
Trivial patch, could be committed. Not unlike #11237 from today.
comment:6
automattor — 3 years ago
- Resolution set to fixed
- Status changed from new to closed

Should probably use do_action_ref_array() instead of do_action().