1964161 - Compact news server/newsgroup errors with "Uncaught NS_ERROR_NOT_IMPLEMENTED: Component returned failure code: 0x80004001 (NS_ERROR_NOT_IMPLEMENTED) [nsIMsgFolder.compactAll] compactAllFoldersForAccount chrome://messenger/content/about3Pane.js:3856"

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Wayne Mery (:wsmwk)]

140.0a1 2025-05-01

1. Picked a news server (news.blueworldhosting.com)
2. File > Compact

Uncaught NS_ERROR_NOT_IMPLEMENTED: Component returned failure code: 0x80004001 (NS_ERROR_NOT_IMPLEMENTED) [nsIMsgFolder.compactAll]
compactAllFoldersForAccount chrome://messenger/content/about3Pane.js:3856
<anonymous> chrome://messenger/content/about3Pane.js:6621
doCommand chrome://messenger/content/mailCommon.js:754
triggerCommand chrome://messenger/content/about3Pane.js:834
onCommand chrome://messenger/content/about3Pane.js:818
handleEvent chrome://messenger/content/about3Pane.js:276

Comment 1

[Magnus Melin [:mkmelin]]

Right ::CompactAll() is not implemented for news, only Compact() which does not do the normal compact... - https://searchfox.org/comm-central/rev/8c6d2ec9e235c9aa3b8ea444f939d776304fbea8/mailnews/news/src/nsNewsFolder.cpp#1362
I'm not sure why it doesn't do the same compact as local folders do.

Comment 2

[Hartmut Welpmann [:welpy-cw]]

Attachment: Bug 1964161 - Disable "Compact" for newsgroup accounts. r=#thunderbird-reviewers

Comment 3

[Hartmut Welpmann [:welpy-cw]]

Attachment: Bug 1964161 - Remove unused/non-functional folder compaction code from nsNewsFolder. r=#thunderbird-reviewers

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/8db698ef5032
Disable "Compact" for newsgroup accounts. r=babolivier
https://hg.mozilla.org/comm-central/rev/06af8f98b148
Remove unused/non-functional folder compaction code from nsNewsFolder. r=babolivier