Setting a folder's charset doesn't happen until after the program is restarted

VERIFIED FIXED in mozilla0.8

Status

MailNews Core
Backend
P1
major
VERIFIED FIXED
18 years ago
10 years ago

People

(Reporter: scottputterman, Assigned: nhottanscp)

Tracking

({intl})

Trunk
mozilla0.8
x86
Windows NT

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [nsbeta1+])

(Reporter)

Description

18 years ago
nsMsgDBFolder::SetCharSet is only setting the db but not changing the current 
value.  This means that things like the charset menu won't take effect until 
after the program is restarted.
(Reporter)

Comment 1

18 years ago
nominating for beta2.

A better explanation:  Whenever we try to change a folder's charset, whether 
through a property dialog or through the charset menu, it's not happening in the 
current session.  It only happens the next time you start up.
Keywords: nsbeta2

Comment 2

18 years ago
Putting on [nsbeta2-] radar.  
Whiteboard: [nsbeta2-]

Comment 3

18 years ago
Target nominated bugs to M16.  Please update as necessary.
Target Milestone: --- → M16
(Reporter)

Updated

18 years ago
Status: NEW → ASSIGNED
Target Milestone: M16 → M17
(Reporter)

Comment 4

18 years ago
nominating as nsbeta3 correctness
Keywords: correctness, nsbeta3
Target Milestone: M17 → M18
(Assignee)

Comment 5

18 years ago
Folder charset UI is out for 6.0 (bug 32714) and I commented out the code to set 
a folder charset by charset menu. We can mark this as "Future" unless we enable 
the folder property dialog in 6.0.
(Reporter)

Comment 6

18 years ago
ok.  then moving to future milestone and removing nomination.  Thanks for updating.
Keywords: correctness, nsbeta3
Target Milestone: M18 → Future
(Assignee)

Comment 7

18 years ago
adding intl keyword
Keywords: intl

Updated

18 years ago
Severity: normal → major
Keywords: nsbeta1
(Reporter)

Comment 8

18 years ago
marking nsbeta1+ and moving to mozilla0.8.
Keywords: nsbeta2
Priority: P3 → P1
Whiteboard: [nsbeta2-] → [nsbeta1+]
Target Milestone: Future → mozilla0.8
(Reporter)

Comment 9

18 years ago
reassigning to naving.
Assignee: putterman → naving
Status: ASSIGNED → NEW
(Assignee)

Comment 10

18 years ago
I found that in nsMsgDBFolder.cpp,
mCharset is not updated to a new charset in nsMsgDBFolder::SetCharset.
I fixed that in my patch for bug 39756, so reassign to me.
I will verify it when I finish folder charset UI.
Assignee: naving → nhotta
QA Contact: lchiang → nhotta
(Assignee)

Comment 11

18 years ago
checked in
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Assignee)

Comment 12

18 years ago
Verified, used win32 build ID 2001011204.
With the UI "View" -> "Foldre Character Coding...", I can set a folder charset.
The change is effective without restarting the app.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.