Issue 3738 - sl_SI.xml (Slovenian XML file)
Summary: sl_SI.xml (Slovenian XML file)
Status: CLOSED FIXED
Alias: None
Product: Internationalization
Classification: Code
Component: code (show other issues)
Version: 641
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: oc
QA Contact: issues@l10n
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-03-30 11:09 UTC by r
Modified: 2013-08-07 15:02 UTC (History)
1 user (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Slovenian XML file (sl_SI.xml) (2.58 KB, patch)
2002-03-30 11:10 UTC, r 		no flags Details | Diff
corrected version of sl_SI.xml file (2.60 KB, patch)
2002-04-03 13:30 UTC, r 		no flags Details | Diff
patch to the latest fi_FI.xml, further comments will follow (982 bytes, patch)
2002-10-10 12:05 UTC, ooo 		no flags Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description r 2002-03-30 11:09:45 UTC 
Slovenian XML file (sl_SI.xml)
Comment 1 r 2002-03-30 11:10:45 UTC 
Created attachment 1290 [details]
Slovenian XML file (sl_SI.xml)
Comment 2 ooo 2002-04-03 11:04:26 UTC 
er->bobe: I just had a glance at it, there are some issues:

The DefaultName entries 
of LangID and CountryID have to be English,
not localized, so for LangID it should be 
"Slovenian" and for
CountryID "Slovenia".

The DateSeparator is defined to be 
a point "." but some date formats
use the slash "/" instead. From what I know the "." 
would be correct,
am I right?

Should the era names really be "AD" and "BC"? The 
ICU states
"pr.n.\u0161." and "po Kr." instead.

DecimalSeparator is "," and 
Time100SecSeparator is defined to be "."
and used in time formats. Is this really 
the case or shouldn't it
also be the "," decimal separator instead?

And the 
"robert" in line 191 most certainly is not correct ;-)
Comment 3 r 2002-04-03 13:30:07 UTC 
Created attachment 1312 [details]
corrected version of sl_SI.xml file
Comment 4 ooo 2002-10-10 12:05:08 UTC 
Created attachment 3121 [details]
patch to the latest fi_FI.xml, further comments will follow
Comment 5 ooo 2002-10-10 12:22:54 UTC 
sh*t, mental absence.. of course the patch isn't fi_FI but sl_SI, so
everything is right but the name is wrong..

Changed the 100th seconds separators in the time formats to reflect
the real 100th seconds separator.

The <DefaultName> trailing blank removed is just cosmetical.

One remaining issue: the <QuotationMark> and <DoubleQuotationMark> are
wrong. A QuotationMark is a single quote (like '), your version had an
ASCII double quote there. A DoubleQuotationMark is a double quote
(like "), in your version was some other string, the UTF-8 encoded
representation was wrong. I changed both qutoation marks to their
en_US equivalents, please correct if necessary and watch the UTF-8
encoding.

For future versions we need new localized entries
<quarter#Abbreviation> that represent the short form of a quarter to
be displayed in date formats (for instance, Q1, Q2, Q3, Q4). Take a
look at i18npool/source/localedata/data/en_US.xml for examples.
Comment 6 ooo 2002-10-10 13:09:05 UTC 
committed to OOO_STABLE_1:
i18npool/source/localedata_ascii/sl_SI.xml 
1.1.2.1
i18npool/source/localedata_ascii/makefile.mk 
1.2.6.1
i18npool/source/localedata/localedata.cxx 1.3.6.3
Comment 7 ooo 2002-10-10 14:27:05 UTC 
committed to HEAD:
i18npool/source/localedata/data/sl_SI.xml 
1.1
i18npool/source/localedata/data/makefile.mk 
1.14
i18npool/source/localedata/localedata.cxx 1.17
Comment 8 ooo 2002-12-09 16:03:56 UTC 
Have to reopen to reassign, rediculous.
Comment 9 ooo 2002-12-09 16:04:22 UTC 
reassign to QA
Comment 10 ooo 2003-01-10 11:10:28 UTC 
sorry, of course this is implemented in OOo 1.0.2, not 1.0.1
Comment 11 oc 2003-01-15 10:25:38 UTC 
Reset state to fixed
Comment 12 oc 2003-03-06 14:59:18 UTC 
Verified