diff options
author | fox <[email protected]> | 2023-02-19 12:03:49 +0300 |
---|---|---|
committer | fox <[email protected]> | 2023-02-19 12:03:49 +0300 |
commit | fcfcb69a2eba083df3401497712c6a3d06c7452f (patch) | |
tree | 06a5e5eb0c216e3671c42340face0e8d0e942705 | |
parent | 7bbe71b81865e2ac5a4f0fdcc39476ab6ed3458e (diff) | |
parent | fd55e492c3a9c6dc583336d65d81e7e6d0009363 (diff) |
Merge pull request 'Handle fetch issues in 'RSSUtils::update_basic_info'.' (#101) from wn/tt-rss:bugfix/handle-failed-basic-info-fetch into master
Reviewed-on: https://dev.tt-rss.org/tt-rss/tt-rss/pulls/101
-rwxr-xr-x | classes/rssutils.php | 22 |
1 files changed, 14 insertions, 8 deletions
diff --git a/classes/rssutils.php b/classes/rssutils.php index df7727eb1..385ab31e6 100755 --- a/classes/rssutils.php +++ b/classes/rssutils.php @@ -316,14 +316,20 @@ class RSSUtils { $feed_data = trim($feed_data); - $rss = new FeedParser($feed_data); - $rss->init(); - - if (!$rss->error()) { - $basic_info = [ - 'title' => mb_substr(clean($rss->get_title()), 0, 199), - 'site_url' => mb_substr(UrlHelper::rewrite_relative($feed->feed_url, clean($rss->get_link())), 0, 245), - ]; + if ($feed_data) { + $rss = new FeedParser($feed_data); + $rss->init(); + + if (!$rss->error()) { + $basic_info = [ + 'title' => mb_substr(clean($rss->get_title()), 0, 199), + 'site_url' => mb_substr(UrlHelper::rewrite_relative($feed->feed_url, clean($rss->get_link())), 0, 245), + ]; + } else { + Debug::log(sprintf("unable to parse feed for basic info: %s", $rss->error()), Debug::LOG_VERBOSE); + } + } else { + Debug::log(sprintf("unable to fetch feed for basic info: %s [%s]", UrlHelper::$fetch_last_error, UrlHelper::$fetch_last_error_code), Debug::LOG_VERBOSE); } } |