Merge pull request #12013 from glassez/fix-rss

Don't access download handler after it's finished
This commit is contained in:
Vladimir Golovnev 2020-02-15 07:02:59 +03:00 committed by GitHub
commit 362305f339
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -124,7 +124,7 @@ void Feed::markAsRead()
void Feed::refresh()
{
if (isLoading())
if (m_downloadHandler)
m_downloadHandler->cancel();
// NOTE: Should we allow manually refreshing for disabled session?
@ -193,6 +193,8 @@ bool Feed::hasError() const
void Feed::handleDownloadFinished(const Net::DownloadResult &result)
{
m_downloadHandler = nullptr; // will be deleted by DownloadManager later
if (result.status == Net::DownloadStatus::Success) {
LogMsg(tr("RSS feed at '%1' is successfully downloaded. Starting to parse it.")
.arg(result.url));