Closed Bug 344785 Opened 18 years ago Closed 18 years ago

truncated replacement tokens in pipnss.properties

Categories

(Mozilla Localizations :: ko / Korean, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dveditz, Assigned: channy)

Details

(Keywords: fixed1.8.0.7, fixed1.8.1)

Attachments

(1 file)

in pipnss.properties the initial replacement token is mangled in the strings

  nick_template
  nick_template_with_num

Both start with "%1$\uC758", missing the 's' type specifier, which displays as a literal "%1$" (or perhaps results in an error leading to a blank string, I haven't looked at error handling in the text formatting code). Should be "%1$s\uC758" unless the \uC758 was a mistaken translation of the character s
Status: NEW → ASSIGNED
I confirmed this bug. But, it will be changed Firefox 2 beta 2. 
Axel, can I changed it 1.8.0 branch?
Please create a patch and request approval1.8.0.6.
Attached patch string changesSplinter Review
Attachment #229431 - Flags: approval1.8.0.6?
Axel, please review this patch.
Comment on attachment 229431 [details] [diff] [review]
string changes

From a review point of view, this is fine. Please keep the approval flag open and wait with landing until we opened the development on 1.5.0.6.
Attachment #229431 - Flags: review+
Comment on attachment 229431 [details] [diff] [review]
string changes

approved for 1.8.0 branch, a=dveditz for drivers
Attachment #229431 - Flags: approval1.8.0.7? → approval1.8.0.7+
Needs fixing in 1.8.1 and trunk too, right?
(In reply to comment #7)
> Needs fixing in 1.8.1 and trunk too, right?
> 

I did it. Thanks.
Ok, on July 23, along with a bunch of other stuff. I missed it because I was looking for a reference to this bug number. Still needed on the 1.8.0 branch it looks like, right?
Keywords: fixed1.8.1
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Looks like this got checked in on Aug 23
Keywords: fixed1.8.0.7
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: