Bug 151308 - [PATCH] Leave X messages on server or Keep Y Mb on server broken
Summary: [PATCH] Leave X messages on server or Keep Y Mb on server broken
Status: RESOLVED FIXED
Alias: None
Product: kmail
Classification: Applications
Component: pop3 (show other bugs)
Version: SVN trunk (KDE 4)
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Will Stephenson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-10-25 03:17 UTC by JW
Modified: 2008-07-18 17:23 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Fix broken logic that determines which mails to keep (2.63 KB, patch)
2008-07-18 10:56 UTC, Will Stephenson
Details
Fix broken logic that determines which mails to keep (2.17 KB, patch)
2008-07-18 13:47 UTC, Will Stephenson
Details

Note You need to log in before you can comment on or make changes to this bug.
Description JW 2007-10-25 03:17:19 UTC
Version:           1.2.4 (using KDE KDE 3.5.8)
Installed from:    Ubuntu Packages
OS:                Linux

I setup a pop mail account to leave mail on the server.  It did not.

Checked
- store POP password
- Leave fetched messages on the server
- Keep only the last 100 messages
- Include in manual mail check
- Enable interval mail checking (20 minutes)

Unchecked
- Leave message on the server for X days
- Keep only the last X MB
- Filter messages if they are greater than X bytes

Destination folder was a subfolder of local inbox (inbox/cyb3rj).

Using TLS encryption for secure mail download.
Authentication Method: Clear text


What other information would help out to diagnose this?
Comment 1 francis 2008-07-12 19:21:05 UTC
I 've got a similar problem with opensuse11.0 - kmail 1.9.51 - kde 4.0.4:::

In account-settings there is the  possibility to leave messages on the server
when checking mail, but:

If on a pop-account the 
 "leave fetched messages on the server" (1) is enabled
not specifying any of the other possibilities
- "leave messages on the server for 'x' days" (2)
- "keep only the last 'x' messages" (3)
- "keep only the last 'x' MB" (4), 
then the messages remain on the server

The following combination (1) + (2) works as well

But the combinations (1) + (3) or (1) + (4) don't work at all: all messages are
removed from the server.
Comment 2 Will Stephenson 2008-07-18 10:48:50 UTC
https://bugzilla.novell.com/show_bug.cgi?id=408686
Comment 3 Will Stephenson 2008-07-18 10:56:49 UTC
Created attachment 26230 [details]
Fix broken logic that determines which mails to keep
Comment 4 Will Stephenson 2008-07-18 10:58:45 UTC
Thomas: does the patch look correct to you?
Comment 5 Will Stephenson 2008-07-18 13:47:59 UTC
Created attachment 26233 [details]
Fix broken logic that determines which mails to keep

Replaces version with debug left in
Comment 6 Thomas McGuire 2008-07-18 16:34:24 UTC
SVN commit 834272 by tmcguire:

Fix "leave on server only the last x messages" and "leave on server only the last x MB" not working
when "leave on server for x days" is not checked for POP3 accounts.

Many thanks to Will Stephenson who came up with the patch.
BUG:151308


 M  +37 -15    popaccount.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=834272
Comment 7 Thomas McGuire 2008-07-18 17:23:10 UTC
SVN commit 834291 by tmcguire:

Backport r834272 by tmcguire to the 4.1 branch:

Fix "leave on server only the last x messages" and "leave on server only the last x MB" not working
when "leave on server for x days" is not checked for POP3 accounts.

Many thanks to Will Stephenson who came up with the patch.
CCBUG:151308


 M  +37 -15    popaccount.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=834291