Opened 17 years ago
Closed 16 years ago
#8495 closed defect (bug) (fixed)
Dashboard RSS errors not handled well
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.8 | Priority: | normal |
Severity: | normal | Version: | 2.7 |
Component: | Feeds | Keywords: | has-patch |
Focuses: | Cc: |
Description
The Dashboard RSS items do not handle errors very well.
For example, Upon an error occuring in fetching the Feed, The Feed URL will be set to 'Error: could not find an RSS or ATOM feed at that URL.' with no feedback to the UI that an error has occured (other than that its blank).
It appears that the main cause for the error in my case, was the magpie fetch time limit of 2 seconds being too slow for my connection, increasing it to 10 allows it to fetch the entire RSS feed, but that is hopefully only a issue on home-hosted installs.
dump of the widget_options:
'dashboard_secondary' => array 'title' => string 'Other WordPress News' (length=20) 'url' => string 'Error: could not find an RSS or ATOM feed at that URL.' (length=54) 'link' => string '' (length=0) 'items' => int 15 'error' => string 'Error in RSS 0' (length=14) 'show_summary' => int 1 'show_author' => int 1 'show_date' => int 1
Attachments (4)
Change History (14)
#2
@
16 years ago
There are two distinct problems here:
1) The RSS feeds aren't updating properly. Possibly a timeout issue.
2) When there *is* a problem, the actual URL setting is overwritten with an error message, after which the feed will *never* work unless the URL is manually reset.
#3
@
16 years ago
the same annoyance applies to feed widgets. Personally, I ended up adding a define to lengthen the timeout, but it's nonetheless annoying. At the very least, the url shouldn't get overwritten.
#4
@
16 years ago
- Component changed from UI to Feeds
The main issue here is feed functionality, not UI, so I'm changing the component to feeds and will start a separate ticket about error messaging under UI.
#6
@
16 years ago
- Owner set to DD32
- Status changed from new to assigned
The move to SimplePie + [10697] will probably have made a difference, I'll have to test this out again later and see if anything needs doing to fix this ticket
This really should be fixed. The big problem is that if there is an error *once*, the URL for the feed is overwritten with the error message. The error should *display*, but certainly not overwrite the actual feed URL setting. The way it is, a one-time error becomes permanent.