Bug 151308

Summary:	[PATCH] Leave X messages on server or Keep Y Mb on server broken
Product:	[Applications] kmail	Reporter:	JW <cyb3rj>
Component:	pop3	Assignee:	Will Stephenson <wstephenson>
Status:	RESOLVED FIXED
Severity:	normal	CC:	mcguire, wstephenson
Priority:	NOR
Version:	SVN trunk (KDE 4)
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	Fix broken logic that determines which mails to keep Fix broken logic that determines which mails to keep

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