|
 |
91334d |
From ffd8369b909c86653ef30593fddf67c9f5308583 Mon Sep 17 00:00:00 2001
|
|
|
91334d |
From: Stephan Bergmann <sbergman@redhat.com>
|
|
|
91334d |
Date: Mon, 5 Feb 2018 17:17:17 +0100
|
|
|
91334d |
Subject: [PATCH] rhbz#1541486: Fix localized'ness of ooSetupFactoryUIName
|
|
|
91334d |
props
|
|
|
91334d |
|
|
|
91334d |
8f44fb5ee0bd21d192820ffd359504f6d1c68a52 "Mark Base window titles for
|
|
|
91334d |
localisation" localized some uses of ooSetupFactoryUIName in
|
|
|
91334d |
officecfg/registry/data/org/openoffice/Setup.xcu, but failed to mark the prop as
|
|
|
91334d |
oor:localized="true" in officecfg/registry/schema/org/openoffice/Setup.xcs. (So
|
|
|
91334d |
some arbitrary localized version of the prop would win at runtime, potentially
|
|
|
91334d |
displaying certain window titles in wrong languages.)
|
|
|
91334d |
|
|
|
91334d |
Other uses of ooSetupFactoryUIName in Setup.xcu (for "Calc", "Impress",
|
|
|
91334d |
"Writer", etc.) have never been localized, and are probably not meant to be
|
|
|
91334d |
localized. AFAICS, irrespective of oor:localized="true" in Setup.xcs, their
|
|
|
91334d |
values not being marked with xml:lang="en-US" in Setup.xcu will mean that they
|
|
|
91334d |
will not appear in
|
|
|
91334d |
translations/source/*/officecfg/registry/data/org/openoffice.po and will not
|
|
|
91334d |
accidentally start to be translated after this commit.
|
|
|
91334d |
|
|
|
91334d |
(Theoretically, inconsistencies between oor:localized in xcs and uses of
|
|
|
91334d |
xml:lang in xcu could statically be caught at build time, e.g., with
|
|
|
91334d |
officecfg/util/sanity.xsl. However, those checks work on each XML file
|
|
|
91334d |
individually, so don't have the schema information available when looking at
|
|
|
91334d |
the data in the xcu file. And those checks are written in XSLT, which makes
|
|
|
91334d |
writing new checks rather, erm, unpleasant.)
|
|
|
91334d |
|
|
|
91334d |
Change-Id: I9c61e8ebf75d3d54f7e5b6688eac47a6ed8dc5c3
|
|
|
91334d |
Reviewed-on: https://gerrit.libreoffice.org/49252
|
|
|
91334d |
Tested-by: Jenkins <ci@libreoffice.org>
|
|
|
91334d |
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
|
91334d |
---
|
|
|
91334d |
officecfg/registry/schema/org/openoffice/Setup.xcs | 2 +-
|
|
|
91334d |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
91334d |
|
|
|
91334d |
diff --git a/officecfg/registry/schema/org/openoffice/Setup.xcs b/officecfg/registry/schema/org/openoffice/Setup.xcs
|
|
|
91334d |
index f55e3698f58f..31db1eca9af4 100644
|
|
|
91334d |
--- a/officecfg/registry/schema/org/openoffice/Setup.xcs
|
|
|
91334d |
+++ b/officecfg/registry/schema/org/openoffice/Setup.xcs
|
|
|
91334d |
@@ -149,7 +149,7 @@
|
|
|
91334d |
<desc>Specifies the short name of the factory.</desc>
|
|
|
91334d |
</info>
|
|
|
91334d |
</prop>
|
|
|
91334d |
- <prop oor:name="ooSetupFactoryUIName" oor:type="xs:string">
|
|
|
91334d |
+ <prop oor:name="ooSetupFactoryUIName" oor:type="xs:string" oor:localized="true">
|
|
|
91334d |
|
|
|
91334d |
<info>
|
|
|
91334d |
<desc>Specifies the UI name of the factory.</desc>
|
|
|
91334d |
--
|
|
|
91334d |
2.14.3
|
|
|
91334d |
|