[Bug 252178] No error message when trying to display emails while 'Enable Disconnected Mode' unchecked

2011-01-07 Thread Sabine Faure
https://bugs.kde.org/show_bug.cgi?id=252178


Sabine Faure sab...@kdab.net changed:

   What|Removed |Added

 Status|RESOLVED|CLOSED




--- Comment #4 from Sabine Faure sabine kdab net  2011-01-08 00:53:52 ---
Hi Tobias!

Indeed it is corrected now.

First a 'The KDE email client' message appears on a white background as well as
'Loading message...'
After about 20s a 'Message loading failed: Unable to fetch item from backend'
message appears indicating to the user that something wrong has happened while
loading.
Hopefully, that will give her a clue to check her wifi connexion so I am
closing this bug.

N900, 4:4.6~.20110106.1209.gitfde48d5-1maemo1.121147

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Kdepim-bugs mailing list
Kdepim-bugs@kde.org
https://mail.kde.org/mailman/listinfo/kdepim-bugs


[Bug 252178] No error message when trying to display emails while 'Enable Disconnected Mode' unchecked

2010-12-30 Thread Tobias Koenig
https://bugs.kde.org/show_bug.cgi?id=252178


Tobias Koenig to...@kde.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Comment #3 from Tobias Koenig tokoe kde org  2010-12-30 13:34:30 ---
commit aa74751de1b4466f6779ce9e865b801893d7de5c
branch master
Author: Tobias Koenig to...@kde.org
Date:   Thu Dec 30 13:34:04 2010 +0100

Propagate errors from ItemRetriever to FETCH handler

Errors from the ItemRetriever (e.g. calling requestItemDelivery
on a resource in offline mode) will be propagated to the FETCH handler
now, so that the caller of ItemFetchJob can show an error message
to the user.

BUG:252178

diff --git a/server/src/handler/fetchhelper.cpp
b/server/src/handler/fetchhelper.cpp
index 88edf74..d66824f 100644
--- a/server/src/handler/fetchhelper.cpp
+++ b/server/src/handler/fetchhelper.cpp
@@ -224,7 +224,9 @@ bool FetchHelper::parseStream( const QByteArray
responseIdentifier )
 retriever.setScope( mScope );
 retriever.setRetrieveParts( payloadList );
 retriever.setRetrieveFullPayload( mFullPayload );
-retriever.exec(); // There we go, retrieve the missing parts from the
resource.
+if ( !retriever.exec() ) { // There we go, retrieve the missing parts from
the resource.
+  throw HandlerException( Unable to fetch item from backend );
+}
   }

   QSqlQuery itemQuery = buildItemQuery();
diff --git a/server/src/storage/itemretriever.cpp
b/server/src/storage/itemretriever.cpp
index 6d5a030..9c84a07 100644
--- a/server/src/storage/itemretriever.cpp
+++ b/server/src/storage/itemretriever.cpp
@@ -169,10 +169,10 @@ QSqlQuery ItemRetriever::buildQuery() const
 }


-void ItemRetriever::exec()
+bool ItemRetriever::exec()
 {
   if ( mParts.isEmpty()  !mFullPayload )
-return;
+return true;

   QSqlQuery query = buildQuery();
   ItemRetrievalRequest* lastRequest = 0;
@@ -234,6 +234,7 @@ void ItemRetriever::exec()
   ItemRetrievalManager::instance()-requestItemDelivery( request );
 } catch ( const ItemRetrieverException e ) {
   akError()  e.type()  :   e.what();
+  return false;
 }
   }

@@ -247,6 +248,8 @@ void ItemRetriever::exec()
   retriever.exec();
 }
   }
+
+  return true;
 }

 QString ItemRetriever::driverName()
diff --git a/server/src/storage/itemretriever.h
b/server/src/storage/itemretriever.h
index aeb650b..866d265 100644
--- a/server/src/storage/itemretriever.h
+++ b/server/src/storage/itemretriever.h
@@ -62,7 +62,7 @@ class ItemRetriever
 void setScope( const Scope scope );
 Scope scope() const;

-void exec();
+bool exec();

   private:
 /** Convenience method which returns the database driver name */

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Kdepim-bugs mailing list
Kdepim-bugs@kde.org
https://mail.kde.org/mailman/listinfo/kdepim-bugs


[Bug 252178] No error message when trying to display emails while 'Enable Disconnected Mode' unchecked

2010-12-29 Thread Tobias Koenig
https://bugs.kde.org/show_bug.cgi?id=252178


Tobias Koenig to...@kde.org changed:

   What|Removed |Added

 CC||to...@kde.org,
   ||vkra...@kde.org




--- Comment #1 from Tobias Koenig tokoe kde org  2010-12-29 13:40:06 ---
Hej Sabine,

I have changed the message viewer already to show status information if some
error condition happens. Attached is a patch which forwards the item retrieval
errors (caused when calling requestItemDelivery on the IMAP resource in
offline-mode) to the Akonadi::ItemFetchJob object, which itself will show the
error as status information in the message viewer.

@Volker is the patch Ok to commit or do you see any unwanted side effects,
since the complete ItemFetchJob fails now whenever requestItemDelivery fails?

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Kdepim-bugs mailing list
Kdepim-bugs@kde.org
https://mail.kde.org/mailman/listinfo/kdepim-bugs


[Bug 252178] No error message when trying to display emails while 'Enable Disconnected Mode' unchecked

2010-12-29 Thread Tobias Koenig
https://bugs.kde.org/show_bug.cgi?id=252178





--- Comment #2 from Tobias Koenig tokoe kde org  2010-12-29 13:42:12 ---
Created an attachment (id=55347)
 -- (http://bugs.kde.org/attachment.cgi?id=55347)
Patch to fix error propagation in Akonadi server

Patch that forwards errors of requestItemDelivery() to the FETCH handler

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Kdepim-bugs mailing list
Kdepim-bugs@kde.org
https://mail.kde.org/mailman/listinfo/kdepim-bugs


[Bug 252178] No error message when trying to display emails while 'Enable Disconnected Mode' unchecked

2010-12-03 Thread Volker Krause
https://bugs.kde.org/show_bug.cgi?id=252178


Volker Krause vkra...@kde.org changed:

   What|Removed |Added

 Blocks|252225  |




-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Kdepim-bugs mailing list
Kdepim-bugs@kde.org
https://mail.kde.org/mailman/listinfo/kdepim-bugs


[Bug 252178] No error message when trying to display emails while 'Enable Disconnected Mode' unchecked

2010-09-24 Thread Bjoern Ricks
https://bugs.kde.org/show_bug.cgi?id=252178


Bjoern Ricks bjoern.ri...@intevation.de changed:

   What|Removed |Added

 Blocks||252225




-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Kdepim-bugs mailing list
Kdepim-bugs@kde.org
https://mail.kde.org/mailman/listinfo/kdepim-bugs


[Bug 252178] No error message when trying to display emails while 'Enable Disconnected Mode' unchecked

2010-09-23 Thread Sabine Faure
https://bugs.kde.org/show_bug.cgi?id=252178


Sabine Faure sab...@kdab.net changed:

   What|Removed |Added

   Keywords||komo3




-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Kdepim-bugs mailing list
Kdepim-bugs@kde.org
https://mail.kde.org/mailman/listinfo/kdepim-bugs


[Bug 252178] No error message when trying to display emails while 'Enable Disconnected Mode' unchecked

2010-09-23 Thread Sabine Faure
https://bugs.kde.org/show_bug.cgi?id=252178


Sabine Faure sab...@kdab.net changed:

   What|Removed |Added

 CC||k...@lazs.de




-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Kdepim-bugs mailing list
Kdepim-bugs@kde.org
https://mail.kde.org/mailman/listinfo/kdepim-bugs