Closed Bug 230210 Opened 21 years ago Closed 21 years ago

Missing /language.accept/ line in file res/language.properties for (Upper ) Sorbian

Categories

(Core :: Internationalization, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: milupo, Assigned: smontagu)

Details

Attachments

(1 file)

User-Agent: Mozilla/5.0 (Windows; U; Win 9x 4.90; wen-DE; rv:1.6b) Gecko/20031208 Build Identifier: Mozilla/5.0 (Windows; U; Win 9x 4.90; wen-DE; rv:1.6b) Gecko/20031208 In the file res/language.properties has been removed the line *sb.accept=true* for Sorbian because the language code (sb) wasn't valid. Unfortunately it wasn't replaced by a line with a valid code (wen for both Sorbian languages) or hsb (for Upper Sorbian, valid since 2003, September 1st). The missing line causes that you can't more create a profile directly for Sorbian. It'a always necessary to switch the from English to Sorbian by Edit-->Preferences-->Appearance-->Languages/Content. I propose to use "hsb" now although there isn't a Lower Sorbian localization yet. But maybe it will future so that you can use the corresponding code dsb for Lower Sorbian. (see: http://www.loc.gov/standards/iso639-2/codechanges.html) The collective code "wen" used for both Sorbian languages would disable a separate localization for Upper resp. Lower Sorbian. Reproducible: Always Steps to Reproduce: 1. 2. 3.
jshin, in bug 224546 you removed sb (see bug 224546 comment #15). I'm not sure what it meant that it was missing in that other file, but obviously it was working before removal from language.properties. I guess you know much more about those files and can fix the problem easily. If the fix is just to re-add the line to this one file (or similarly trivial), I'll do the patch, just say it. Thanks.
Attached patch patchSplinter Review
I added 'hsb' (Upper Sorbian) per Michael's suggestion. Andreas, adding back 'sb' is not an option. 'sb' has NEVER been the language code for Sorbian.
Comment on attachment 138859 [details] [diff] [review] patch asking for r/sr. I'll fix the misalignment in the following line (the column position of 'true') hsb.accept = true
Attachment #138859 - Flags: superreview?(blizzard)
Attachment #138859 - Flags: review?(smontagu)
Comment on attachment 138859 [details] [diff] [review] patch r=smontagu
Attachment #138859 - Flags: review?(smontagu) → review+
Attachment #138859 - Flags: superreview?(blizzard) → superreview+
fix checked into trunk
Status: UNCONFIRMED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
I'm sorry it was too late for 1.6. I'll ask for 1.6.1 approval if 1.6.1 is planned.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: