[L12y] PLS remove locale name (en-US) from the localMsgs.properties

VERIFIED FIXED

Status

defect
P3
normal
VERIFIED FIXED
20 years ago
11 years ago

People

(Reporter: tao, Assigned: tao)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Assignee

Description

20 years ago
Hi, Scott:

This is probably a residue from our out-dated localizability strategy.
Now, we plan to treat property files as *.dtd and place them under
locale/[locale-name] directories. Therefore, we do not need to embed
locale name in the filename anymore.

PLS bounce back to me if you'd like me to fix it. Let's check this in
today.

Thanks.
Assignee

Updated

20 years ago
QA Contact: lchiang → tao

Comment 2

20 years ago
So we have a new plan for this eh? =)

So localMsgs.properties should be the name of the file and it stays in the same
directory?

If you have cycles to do this today then I'll re-assign this to you as it would
get done faster that way. I probably won't get to it until after M11 tree
closes.
Assignee

Updated

20 years ago
Assignee: mscott → tao
Assignee

Comment 3

20 years ago
Yes, right and the same directory, too.

I am taking it over. But, would you kindly let me know where it is used; just
in case that I miss one.

Thanks
Assignee

Comment 4

20 years ago
One more

/mailnews/compose/resources/locale/en-US/composeMsgs_en.properties

Comment 5

20 years ago
there's one under the imap directory too. Did you get that one as well?

If you change the names and the projects, I can run through when your done and
verify that mailnews is finding the new properties files (i.e. taht we still get
the strings out correctly).
Assignee

Updated

20 years ago
Status: NEW → RESOLVED
Last Resolved: 20 years ago
QA Contact: tao → mscott
Resolution: --- → FIXED
Assignee

Comment 6

20 years ago
Hi, Scott:


Would you verify this for me?


Thanks.

Comment 7

20 years ago
Tao, I've verified your changes and have also started to remove the old files
from the repository. Thanks for doing this!
Assignee

Updated

20 years ago
Status: RESOLVED → VERIFIED
Assignee

Comment 8

20 years ago
Per mscott's comment, marking it verified.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.