Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move message to folder also expunges messages marked as deleted #1505

Closed
rcubetrac opened this issue Apr 21, 2008 · 6 comments
Closed

Move message to folder also expunges messages marked as deleted #1505

rcubetrac opened this issue Apr 21, 2008 · 6 comments

Comments

@rcubetrac
Copy link

Reported by r on 21 Apr 2008 21:21 UTC as Trac ticket #1485002

Steps to reproduce:

  1. In main.inc.php set $rcmail_config[= TRUE;[BR]

  2. In main.inc.php set $rcmail_config[= '';[BR]

  3. Log into RC, open any folder and mark a message to be deleted[Drag (move) a message from the same folder above into any other folder[[BR]([BR]]

4))]

At this point, a few things happen:

  1. The message marked as deleted in step 3 was actually expunged from the IMAP server during the move operation in step 4, but still appears in the message list[Clicking on "Compact" to get rid of the "phantom" message marked as deleted in step 3 fails to remove it from the message list[[BR]([BR]]

2))]

  1. Clicking on the message marked as deleted in step 3 results in a "Could not load message from server" error, because it no longer actually exists on the server[[BR]]

To recover from this condition, one must click on a folder, click on the "E-mail" icon in the upper-right toolbar, or reload the page.

I believe that when 'flag_for_deletion' is set in the config file, the behavior of moving messages from one folder to another should mimic the behavior of marking messages for deletion - that is, copy the message to the new folder, but only mark the message in the original folder as deleted. Thus there is no "surprise" expunge during a message move which tramples messages already marked for deletion by the user, and avoids the resulting UI weirdness.

Migrated-From: http://trac.roundcube.net/ticket/1485002

@rcubetrac
Copy link
Author

Comment by @thomascube on 23 Apr 2008 12:07 UTC

(fixed formatting)

@rcubetrac
Copy link
Author

Comment by @till on 6 May 2008 14:33 UTC

Err... you want to delete a message and then move it to another folder?

Maybe you should move it and be done with it?

@rcubetrac
Copy link
Author

Comment by r on 8 May 2008 14:16 UTC

Sorry I wasn't clearer. The message in step 3 and the message in step 4 are different messages. For example, if I mark message #1 for deletion, and then move message #2 to a different folder, both message #1 and message #2 are expunged from the IMAP server during the move operation. Message #1 however is not removed from RoundCube's message list, while message #2 is. Subsequent attempts to remove message #1 from the message list by "Compact"ing the folder fail. The main problem is that after a move operation, RoundCube's message list does not reflect the fact that messages marked for deletion prior to the move operation were expunged along with the moved message. Should I provide screenshots?

@rcubetrac
Copy link
Author

Milestone changed by @alecpl on 10 May 2008 20:05 UTC

later => 0.1.2

@rcubetrac
Copy link
Author

Comment by @alecpl on 11 Jun 2008 19:26 UTC

Fixed in d87fc29.

@rcubetrac
Copy link
Author

Status changed by @alecpl on 11 Jun 2008 19:26 UTC

new => closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant