Repair Folder should attempt to remember column settings and threading settings for the folder
Categories
(MailNews Core :: Database, defect, P3)
Tracking
(thunderbird_esr91 affected, thunderbird_esr102+ affected, thunderbird103 affected, thunderbird104 affected)
People
(Reporter: darktrojan, Assigned: benc)
References
(Depends on 1 open bug)
Details
(Keywords: dataloss, leave-open, Whiteboard: [needs automated test])
Attachments
(1 obsolete file)
If I repair a folder, the view settings are thrown out with the database. There should be an attempt made to remember which columns are visible, sort order, threading, etc..
Comment 1•5 years ago
|
||
Hmm, there are other things stored only in the MSF which are lost. IIRC, whether you displayed remote content on a message. I'd have to did out Kent's comments on the issue.
Comment 2•5 years ago
|
||
You shouldn't be losing column settings https://mzl.la/2YhhRsD . And we've had past regressions in this area - every 2-4 years. But perhaps what you are seeing is bug 550286. Perhaps you can pick up the torch there?
Reporter | ||
Comment 3•5 years ago
|
||
Quite likely this is a dupe of bug 550286. I'll leave this open for now in case I find time for it.
Comment 4•5 years ago
|
||
I should have said, you can probably find, in the compact related code, a method for keeping the settings
Updated•5 years ago
|
Comment 5•4 years ago
|
||
In testing bug 1699514 on Mac, I had to kill Thunderbird.
When I restarted, the columns in my fastmail imap account had been reset
Comment 6•4 years ago
|
||
I confirm Repair still fails, but only for imap folders.
We've fixed lost columns so many times, perhaps bug 710056 comment 145 can be a hint of what's needed?
Also, we either have no tests, or a faulty one, to ensure columns don't regress. Can this be a model? https://bugzilla.mozilla.org/attachment.cgi?id=664148&action=diff
Assignee | ||
Comment 7•4 years ago
|
||
Updated•4 years ago
|
Assignee | ||
Comment 8•4 years ago
|
||
Oops - looks like there's a stray notifyItemEvent()
left over after bug 1685484, which prevents "repair folder" from running.
This patch (Comment 7) restores "repair folder" back to it's usual folder-setting-destroying state :-)
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Comment 9•3 years ago
|
||
Comment on attachment 9222751 [details]
Bug 1568355 - Fix regression from Bug 1685484 which prevents folder repair. r?darktrojan
Revision D115574 was moved to bug 1685484. Setting attachment 9222751 [details] to obsolete.
Updated•3 years ago
|
Comment 11•2 years ago
|
||
Thomas, can you check if this still occurs, and adjust flags accordingly?
I'm pretty user it does. And with user needing to do more repairs lately than normal, we really must get this fixed.
Comment 12•2 years ago
|
||
(In reply to Wayne Mery (:wsmwk) from comment #11)
Thomas, can you check if this still occurs, and adjust flags accordingly?
I'm pretty user it does. And with user needing to do more repairs lately than normal, we really must get this fixed.
Yes, this is still happening, tested on Win10, set tracking flags accordingly:
- 102.0.2 (64-bit)
- 103.0b4 (64-bit)
- 104.0a1 (2022-07-11) (64-bit)
Repair resets to their defaults:
- visible columns (Subject, correspondents, date, and several small columns)
- sort order (date ascending)
- threading (on)
Updated•2 years ago
|
Comment 13•2 years ago
|
||
This should also be backed by tests that fail accordingly. We're way past the decade where this should "just work", always.
Updated•2 years ago
|
Comment 14•2 years ago
|
||
Repair folder is not something that should really ever be needed.
Comment 15•2 years ago
|
||
(In reply to Magnus Melin [:mkmelin] from comment #14)
Repair folder is not something that should really ever be needed.
and yet, it clearly is needed.
Comment 16•2 years ago
|
||
Of course, that's why it exists. But it's a very rare operation (=> less severity).
Comment 17•2 years ago
|
||
(In reply to Magnus Melin [:mkmelin] from comment #16)
Of course, that's why it exists. But it's a very rare operation (=> less severity).
Is the "repair folder" something else than "maintenance"? The maintenance operation runs automatically several times a day for me and losing the grouping settings every single time is very annoying (see also the discussion of the linked bug 1737216).
Comment 18•2 years ago
|
||
(In reply to Magnus Melin [:mkmelin] from comment #16)
Of course, that's why it exists. But it's a very rare operation (=> less severity).
I must categorically disagree - during the 102 cycle, it was frequently used by users trying to get out of their messed up folders. And rare does not equate directly to severity.
Comment 19•2 years ago
|
||
(In reply to Pavel Kotrč from comment #17)
(In reply to Magnus Melin [:mkmelin] from comment #16)
Of course, that's why it exists. But it's a very rare operation (=> less severity).
Is the "repair folder" something else than "maintenance"? The maintenance operation runs automatically several times a day for me and losing the grouping settings every single time is very annoying
The automatic operation is Compact. And yes, it can have failures similar to repair.
However, I think it may be more typical for the columns to be lost when .msf files are just lost outright for reasons other than compact.
Updated•2 years ago
|
Comment 20•2 years ago
|
||
I agree this sucks, but it's not clear to me that it's worth fixing so long as Mork is the message index format.
Comment 21•2 years ago
|
||
This has been a long standing Thunderbird problem, going back to more than 2-3 years ago now.
Comment 22•2 years ago
|
||
(In reply to Magnus Melin [:mkmelin] from comment #16)
Of course, that's why it exists. But it's a very rare operation (=> less severity).
Its regularly used in the support forums. Compact folder, repair folder, retest in safe mode, retest using just view -> folders -> all , compare it to the webmail folder listing and see if it occurs with another email provider is frequently the short list of suggestions when there is anything wrong with the folders.
Updated•2 years ago
|
Updated•2 years ago
|
Comment 24•2 years ago
|
||
(In reply to Andrei Hajdukewycz [:sancus] from comment #20)
I agree this sucks, but it's not clear to me that it's worth fixing so long as Mork is the message index format.
But mork is still going strong.
Description
•