From a1abbd9e05c80584d831b1d12c27c5f7d731cece Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Petr=20Mr=C3=A1zek?= Date: Sat, 28 May 2016 19:54:17 +0200 Subject: NOISSUE refactor *Download into more, smaller pieces * Download is now Download. * Download uses Sink subclasses to process various events. * Validators can be used to further customize the Sink behaviour. --- api/logic/news/NewsChecker.cpp | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) (limited to 'api/logic/news/NewsChecker.cpp') diff --git a/api/logic/news/NewsChecker.cpp b/api/logic/news/NewsChecker.cpp index be4aa1d1..7cc87953 100644 --- a/api/logic/news/NewsChecker.cpp +++ b/api/logic/news/NewsChecker.cpp @@ -37,7 +37,7 @@ void NewsChecker::reloadNews() qDebug() << "Reloading news."; NetJob* job = new NetJob("News RSS Feed"); - job->addNetAction(ByteArrayDownload::make(m_feedUrl)); + job->addNetAction(Net::Download::makeByteArray(m_feedUrl, &newsData)); QObject::connect(job, &NetJob::succeeded, this, &NewsChecker::rssDownloadFinished); QObject::connect(job, &NetJob::failed, this, &NewsChecker::rssDownloadFailed); m_newsNetJob.reset(job); @@ -49,13 +49,7 @@ void NewsChecker::rssDownloadFinished() // Parse the XML file and process the RSS feed entries. qDebug() << "Finished loading RSS feed."; - QByteArray data; - { - ByteArrayDownloadPtr dl = std::dynamic_pointer_cast(m_newsNetJob->first()); - data = dl->m_data; - m_newsNetJob.reset(); - } - + m_newsNetJob.reset(); QDomDocument doc; { // Stuff to store error info in. @@ -64,12 +58,14 @@ void NewsChecker::rssDownloadFinished() int errorCol = -1; // Parse the XML. - if (!doc.setContent(data, false, &errorMsg, &errorLine, &errorCol)) + if (!doc.setContent(newsData, false, &errorMsg, &errorLine, &errorCol)) { QString fullErrorMsg = QString("Error parsing RSS feed XML. %s at %d:%d.").arg(errorMsg, errorLine, errorCol); fail(fullErrorMsg); + newsData.clear(); return; } + newsData.clear(); } // If the parsing succeeded, read it. -- cgit v1.2.3