My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
  Advanced search   Search tips   Subscriptions
Issue 182: Implement pick list for Sent and Trash folders (IMAP)
2 people starred this issue and may be notified of changes. Back to list
Status:  Fixed
Closed:  Mar 2009

Sign in to add a comment
Reported by troyengel, Jan 4, 2009
What version of K-9 are you using?


Is your email account a POP account or an IMAP account?

IMAP - Dovecot 1.0.14 server with the namespace INBOX configured like so:

# Courier-IMAP friendly
namespace private {
  separator = .
  prefix = INBOX.
  inbox = yes

Please provide any additional information below.

The IMAP server uses a namespace prefix (INBOX.); I leave the namespace
field blank during account creation and K-9 finds the folders like
INBOX.Sent and INBOX.Trash. However, the Sent and Trash are kind of risky
as I've had top level (on device?) folders created called "Sent" and
"Trash", sent mail is not actually copied up to my IMAP server. I've tried
filling in the namepsace field with INBOX but it doesn't seem to make a
difference. Am I supposed to put INBOX.Sent, INBOX.Trash? I tried that in a
previous version and it didn't work either (yet to try with 0.91).

The Sent and Trash folder fields should be a picklist of folders from your
IMAP server so we can easily and without error specify which folders to
use; right now I can't even tell where things are going to go or what's
going to happen with the defaults - I fear sending any email from my device
as I must have those Sent emails saved on the server. The whole situation
is kind of sketchy - clients such as the iPhone and Thunderbird allow
picking the folder off the server so no problems are encountered, K-9
should do the same.

Jan 12, 2009
I was having the same problem, deleted items were marked deleted on my server but not
moved to the trash. I had to modify Sent/Trash in Message Composition settings to
INBOX.Trash and INBOX.Sent

Agree, this should be inherited from the prefix settings.
Jan 12, 2009
Project Member #2
Please see comment 10 for  Issue 198 :  Text is:
ImapStore fixed at trunk revision 294 to use Imap prefix during "copy to Trash"

Expect fix to be included in K-9 build .95
The Sent folder appears to work properly, too, utilizing the IMAP prefix. 

Please retest once version .95 or later is available.

However, I agree that a picklist would be a good approach for selecting the Sent and
Trash folders.

Jan 12, 2009
#3 troyengel
@danapple0 - with 0.95, will the automatic NAMESPACE be used given by the server, or
do we have to manually configure the Prefix setting in K-9? And if we have to enter
it, to we include the "." (or "/" or whatever separator the server uses)? The proper
way to set things up is a little confusing at the moment.

Jan 12, 2009
Project Member #4
The change I made merely makes K-9 use the prefix to find the server-side trash folder the same way it 
finds other folders.  I didn't address anything about how prefixes or separators are handled in general. 

I normally don't use a prefix, but when I did for testing this chamge, I entered it without the separator 
(which is . on my system) and it worked well.  So, unless somebody else makes further changes, enter the 
prefix manually and do not enter the separator.
Jan 12, 2009
Project Member #5
I coded the current prefix handling. The separator is pulled from the server at
runtime, as the server provides this with the list of mailboxes.
Jan 17, 2009
#6 troyengel
I upgraded to 0.95 just now, I don't see anything different - did the new code make it in?

With no prefix entered: sent email is saved to top level Sent (not INBOX.Sent), deletes are the same with 
the trash folder, top level. Empty Trash command empties top level Trash not INBOX.Trash.

With manual INBOX entered: cannot select any folder except main Inbox, new top level folder k9mail-
errors with messages about 'INBOX.INBOX.Sent cannot select folder' - seems totally broken.

Jan 18, 2009
Project Member #7
Please attach the contents of one of those messages from K9mail-errors to this issue.
 Also, after changing the prefix, please be sure to execute a Refresh while in the
folder/message list.  Hit the Menu button, then Refresh.
Jan 20, 2009
#8 troyengel
I deleted my IMAP account and recreated everything from scratch and now it seems to
work if you enter INBOX in the prefix, my Sents are saved to INBOX.Sent and deleted
go to INBOX.Trash. Hooray! Something was amiss in the saved config or cache on the
device I think. I no longer get the k9mail-errors folder or any error messages.

Leaving the Prefix field blank doesn't work though, the app is not picking up the
NAMESPACE field as presented by the IMAP server and automatically using it like it
should. Here is a small bit of debug info from the Thunderbird connection: 1 capability * CAPABILITY IMAP4rev1 SASL-IR SORT

Hope this helps some.

Jan 20, 2009
Project Member #9
OK, looks like the defect that was mentioned as supporting this enhancement request
was fixed in 0.95.  However, the original request for a pick-list for the Sent and
Trash folder is a valid enhancement request, so changing this issue to a
Type-Enhancement, and leaving it open.

It sounds like another request would be for K-9 to use the NAMESPACE capability, when
available.  troyengel, you can submit another issue, as an enhancement, for K-9 to
support the NAMESPACE capability.

Labels: -Type-Defect Type-Enhancement
Jan 20, 2009
Project Member #10
In terms of adding a pick-list: K-9 will automatically create the Trash folder (or
whatever folder is chosen for trash purposes) when deleting a message, if the folder
does not exist.  This is valuable functionality.  If a pick-list is used for
selecting the folder to use for trash purposes, it is important that the ability to
choose a non-existent folder is not completely lost.  Perhaps creation of the new
folder can be done immediately, establishing before attempting to delete a message
that the folder can be created and already exists.

Jan 20, 2009
#11 troyengel
Gotcha, new issue for NAMESPACE K-9 support started:

Mar 20, 2009
Project Member #12
This will be my next item on the list. If everything goes as planned it should be
finished within this weekend.
Mar 20, 2009
Project Member #13
I case you missed my comment on  Issue 182 :

Look at the way the Move/Copy functions use the ChooseFolder Activity for selecting
an existing folder.  This should be completely reusable for your purposes.
Mar 20, 2009
Project Member #14
I have seen it, although I think you mixed up the issue numbers. :D
Anyway thanks for the hint.
Mar 21, 2009
Project Member #15
And another successful midnight coding session.
Here's my patch for this issue, I hope I didn't miss anything:
- Select server folder in incoming settings
- Removed folder settings from composing settings
- small change to ChooseFolder-activity

You can, of course, commit it to the repository, if you deem it worthy.
But I would prefer to do that myself... ;)
13.7 KB   View   Download
Mar 23, 2009
Project Member #16
(No comment was entered for this change.)
Status: Started
Owner: dumbfile
Labels: Usability
Mar 24, 2009
Project Member #17
Fix checked in as r418
Status: Fixed
Sign in to add a comment

Powered by Google Project Hosting