Closed Bug 573278 Opened 10 years ago Closed 10 years ago

Port |Bug 522252 - make archiving of read-only folders do a copy, not a move| to SeaMonkey

Categories

(SeaMonkey :: MailNews: Backend, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1a2

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch patch (obsolete) — Splinter Review
From bug 522252:
"Archiving of news messages attempts to do a move to the archive folder, which
only half succeeds (the copy part succeeds, the delete after fails). This
leaves the destination folder locked. We should make archive of newsgroups or
other folders that don't allow deleting of messages to do a copy, not a move."

I think this could be a MOA case, thus requesting r+sr at the same time.
Attachment #452500 - Flags: superreview?(mnyromyr)
Attachment #452500 - Flags: review?(mnyromyr)
While we're here we could just as well add some comments about why and how we're implementing functions from the listeners used, and group what belongs together. The choice is yours. :-)
Assignee: nobody → jh
Status: NEW → ASSIGNED
Attachment #452504 - Flags: superreview?(mnyromyr)
Attachment #452504 - Flags: review?(mnyromyr)
Comment on attachment 452504 [details] [diff] [review]
patch with comment updates [Checkin: comment 3]

r/moa=me
Attachment #452504 - Flags: superreview?(mnyromyr)
Attachment #452504 - Flags: superreview+
Attachment #452504 - Flags: review?(mnyromyr)
Attachment #452504 - Flags: review+
Attachment #452500 - Flags: superreview?(mnyromyr)
Attachment #452500 - Flags: review?(mnyromyr)
Attachment #452500 - Attachment is obsolete: true
Comment on attachment 452504 [details] [diff] [review]
patch with comment updates [Checkin: comment 3]

http://hg.mozilla.org/comm-central/rev/42d7e1d26ef4
Attachment #452504 - Attachment description: patch with comment updates → patch with comment updates [Checkin: comment 3]
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a2
You need to log in before you can comment on or make changes to this bug.