[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [kdepim/work/akonadi-ports] krss/plugin: check if deserialization
From:       Frank Osterfeld <frank.osterfeld () kdab ! com>
Date:       2011-09-17 15:00:32
Message-ID: 20110917150032.DA738A607A () git ! kde ! org
[Download RAW message or body]

Git commit 4760c445c2f46b4e45eea92bf1a0883abad85b66 by Frank Osterfeld.
Committed on 17/09/2011 at 16:59.
Pushed by osterfeld into branch 'work/akonadi-ports'.

check if deserialization actually works

M  +7    -3    krss/plugin/akonadi_serializer_rss.cpp

http://commits.kde.org/kdepim/4760c445c2f46b4e45eea92bf1a0883abad85b66

diff --git a/krss/plugin/akonadi_serializer_rss.cpp b/krss/plugin/akonadi_serializer_rss.cpp
index 6bfe699..1cae4aa 100644
--- a/krss/plugin/akonadi_serializer_rss.cpp
+++ b/krss/plugin/akonadi_serializer_rss.cpp
@@ -40,16 +40,20 @@ bool SerializerPluginRss::deserialize( Akonadi::Item& item, const QByteArray& la
         rssItem = item.payload<RssItem>();
     }
 
+    bool success = false;
     if ( label == Akonadi::Item::FullPayload ) {
-        m_serializer.deserialize( rssItem, data.readAll(), RssItemSerializer::Full );
+        success = m_serializer.deserialize( rssItem, data.readAll(), RssItemSerializer::Full );
     }
     else if ( label == KRss::Item::HeadersPart ) {
-        m_serializer.deserialize( rssItem, data.readAll(), RssItemSerializer::Headers );
+        success = m_serializer.deserialize( rssItem, data.readAll(), RssItemSerializer::Headers );
     }
     else if ( label == KRss::Item::ContentPart ) {
-        m_serializer.deserialize( rssItem, data.readAll(), RssItemSerializer::Content );
+        success = m_serializer.deserialize( rssItem, data.readAll(), RssItemSerializer::Content );
     }
 
+    if ( !success )
+        return false;
+
     item.setPayload<RssItem>( rssItem );
     return true;
 }

[prev in list] [next in list] [prev in thread] [next in thread]