don't put unsent messages in the "sent" folder

I can confirm this bug

On Nov 5, 2010 4:55 PM, "Daniel Fore" <email address hidden> wrote:

Public bug reported:

If a message fails to sent (msmtp not found, timeout, etc) it should
stay in the outbox and not go into the sent folder.

** Affects: postler
    Importance: Undecided
        Status: New

--
don't put unsent messages in the "sent" folder
https://bugs.launchpad.net/bugs/671663
You received this bug notification because you are a member of Postler
Dev Team, which is the registrant for Postler.

Status in Postler Desktop Mail Client: New

Bug description:
If a message fails to sent (msmtp not found, timeout, etc) it should stay in
the outbox and not go into the sent folder.

This patch makes the composer store messages in the Queue folder first, and the DBus service is expected to move it once it is done. However, it isn't stable and seems to crash in the IO channel callback. Additional eyes would be helpful to find the cause.

I have started to wrap the process launching in a small class, so it becomes stateful and captures error output. And I'm adding a Sent signal to the service so we can asynchronously wait for however long needed.

I pushed the mentioned refactoring, the composer shows a progressbar now while waiting for a message to be sent or an error message if needed. Receiving is prepared for error message display.