Hardcoded strings in Gopher search alert

VERIFIED FIXED in mozilla1.0.2



17 years ago
8 years ago


(Reporter: ji, Assigned: jbetak)


({l12y, topembed+})

Windows 2000
l12y, topembed+

Firefox Tracking Flags

(Not tracked)


(Whiteboard: [adt2] [trunk patch has landed, branch patch has landed])


(2 attachments, 1 obsolete attachment)



17 years ago
The following two strings
"Enter a search term:"

are hardcoded in 

Comment 1

17 years ago
Moved from bugscape 
Keywords: l12y
Summary: Hardcoded string in Gopher search alert → Hardcoded strings in Gopher search alert
Carrying over keyword and status whiteboard marking, cc's to this bugzilla bug
from bugscape.

juraj: pls carry over your patch from bugscape to this bug, with sr=bryner.

let's get this one fixed on the 1.0 branch asap for a major embedding customer.
Severity: normal → critical
Keywords: approval, edt1.0.2, mozilla1.0.2, nsbeta1+, topembed+
Priority: -- → P1
Whiteboard: [adt2] [ETA 09/14]
Target Milestone: --- → mozilla1.0.2


17 years ago
Blocks: 154896


17 years ago
QA Contact: ruixu → ji

Comment 3

17 years ago
This fix will externalize 2 localizable strings as noted above.
Cc'ing l10n folks.  Pls add others as needed.
Comment on attachment 99170 [details] [diff] [review]
corrected trunk patch

transferring sr=bryner
Attachment #99170 - Flags: superreview+
ftang, roy could you r=?
Why do you need to proxy this? The threading rules for necko mean that 'stuff'
is always started from teh UI thread.


17 years ago
Keywords: intl
Look, I know very little about Necko - the code is straight out of the socket 
transport service. 


Bradley, did you put these strings in here? Can you help us with the patch? We 
need to land it on the trunk ASAP, the branch would have to follow very soon 


17 years ago
Component: Internationalization → Localization
Keywords: intl
Yeah, I probably have blame for not localising this.

I don't have time to test this, but if you replace the NS_WITH_PROXIED_SERVICE
with do_getService directly, then the patch is fine.

Comment 10

17 years ago
I have concerns about Necko depends on intl. (see Makefile.in)  I thought
we had problem with this during xpinstall before.  

darin: I'm not very familiar about dependency structure of Necko; 
       but is it ok to have gopher depends of intl? 

Comment 11

17 years ago
hmmmm.  I see jbetak's #8 comment and, indeed, necko is already depend on intl.
Ignore my comment above.

Comment 12

17 years ago
Comment on attachment 99170 [details] [diff] [review]
corrected trunk patch

/r=yokoyama with do_getService() instead of 
Attachment #99170 - Flags: review+
jbetak/tao:we got r & sr =, pls request drivers' approval for a trunk landing
(if needed). we'd like to get this baked for a day before taking it to the 1.0
branch. thanks!
Keywords: mozilla1.2

Comment 14

17 years ago
roy: the problem as far as i recall had to do with XPCOM depending on intl.  it
is ok for necko to depend on intl.
Created attachment 99403 [details] [diff] [review]
patch v2

this will land momentarily
Attachment #99170 - Attachment is obsolete: true
Attachment #99403 - Flags: superreview+
Attachment #99403 - Flags: review+
Comment on attachment 99403 [details] [diff] [review]
patch v2

carrying over

Trunk patch has just landed. I'll keep this open for the pending branch fix.
Whiteboard: [adt2] [ETA 09/14] → [adt2] [trunk patch has landed, branch patch ETA 09/17]
Resolving as fixed per Comment #17 From jbetak@netscape.com, so that QA can
verify as fixed on tomorrow's trunk builds. We use keywords (i.e. edt1.02 and
mozilla1.0.2) to manage 1.0 branch checkins.

ji: pls verify this as fixed on tomorrow's trunk builds. thanks!
Last Resolved: 17 years ago
Resolution: --- → FIXED
edt1.0.2+ (per verbal from saari) approval for landing on the 1.0 branch,
pending Drivers' approval. Pls land time asap, the replace "mozilla1.0.2+" with
"fixed1.0.2". thanks!
Keywords: edt1.0.2 → edt1.0.2+
Comment on attachment 99403 [details] [diff] [review]
patch v2

a=rjesup@wgate.com for 1.0 branch.  Please change mozilla1.0.2+ to fixed1.0.2
when checked in.

JaimeJr, please be more clear in your comments; it sounded like you were giving
the a= for this bug.
Attachment #99403 - Flags: approval+
Comment on attachment 99551 [details] [diff] [review]
branch patch

carrying over 

Attachment #99551 - Flags: superreview+
Attachment #99551 - Flags: review+
Attachment #99551 - Flags: approval+
branch patch has landed
Keywords: mozilla1.0.2 → fixed1.0.2
Whiteboard: [adt2] [trunk patch has landed, branch patch ETA 09/17] → [adt2] [trunk patch has landed, branch patch has landed]

Comment 24

17 years ago
Verified as fixed with 09/17 trunk build. Replaced the two new strings in
necko.properties with pseudo localized strings, the localized strings appear in
the search dialog.
ji, just a friendly heads-up: we'll need to verify this in a 1.0 branch build or
in Moz1.0.2 as well

Comment 26

17 years ago
I'll verify this on branch build tomorrow.

Comment 27

17 years ago
Verified as fixed on 09/18 branch build.
Keywords: fixed1.0.2 → verified1.0.2
You need to log in before you can comment on or make changes to this bug.