Playback of offline IMAP copy operations fails silently

VERIFIED FIXED in mozilla0.9.1

Status

defect
P1
normal
VERIFIED FIXED
18 years ago
15 years ago

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Tracking

Trunk
mozilla0.9.1
x86
Windows 2000

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [nsbeta1+] patch; r=naving; sr=sspitzer; a=?)

Attachments

(1 attachment)

Assignee

Description

18 years ago
1. go offline
2. copy an imap message from one folder to another
3. Go online

the copy operation does not take place (fails silently because apparently the
imap server doesn't complain about a failed copy)

I'll attach a patch shortly. It's a one line change. Move works fine; it's just
copy that fails. The change is only in the code that plays back offline imap
copy operations so it won't break anything.
Assignee

Comment 1

18 years ago
nominating
Status: NEW → ASSIGNED
Keywords: nsbeta1
Assignee

Comment 2

18 years ago
Posted patch proposed fixSplinter Review
Assignee

Comment 3

18 years ago
CC'ing Seth and Navin for code reviews. The bug was that we were using the
destination folder as the folder to call ReplayOfflineMoveCopy on, and we want
to use the source folder, like the routine above it does, 
nsImapOfflineSync::ProcessMoveOperation().

Comment 4

18 years ago
r=naving

Comment 5

18 years ago
Putting on 0.9.1 radar. If someone thinks they copied something, they might
delete the message.
Priority: -- → P1
Whiteboard: [nsbeta1+]
Target Milestone: --- → mozilla0.9.1

Updated

18 years ago
Whiteboard: [nsbeta1+] → [nsbeta1+] patch; r=naving; sr=sspitzer; a=?

Comment 7

18 years ago
a= asa@mozilla.org for checkin to 0.9.1 and the trunk.
(on behalf of drivers)
Assignee

Comment 8

18 years ago
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED

Comment 9

18 years ago
Verified on commerical branch builds
2001-06-05-11-0.9.1/ -windows
2001-06-05-04-0.9.1/ -linux
2001-06-05-03-0.9.1/ -mac

Copying messages (both downloaded/non-downloaded) works in both ways:
 -right click on the message and selecting the 'copy to' option
 -Message|Copy Message option

But a bug I did see is that when copying a dowloaded message to another
folder: in the new folder, it looses it's italic(downloaded) mode.
will open a new bug on that.

adding keyword vtrunk

marking as verified
Status: RESOLVED → VERIFIED
Keywords: vtrunk

Comment 10

18 years ago
Verified using commercial trunk builds
2001061504 -win nt 4.0
2001061508 -linux 2.2, red hat 7.0
2001061508 -mac os 9.0.4

Removing keyword vtrunk
Keywords: vtrunk
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.