Date: prev next · Thread: first prev next last
2012 Archives by date, by thread · List index


Hi Efe,

On Thursday, 2012-08-02 00:05:29 +0300, Efe Gürkan YALAMAN wrote:

The currency for Turkish was TRL TL since 2005. After that the currency was
TRY YTL since 2009. Current currency is TRY TL.
I changed default currency to TRY TL for Turkish. There is two patches
because I could not decide whether to delete TRY YTL.

Thanks for the patches.

The first where you added another currency entry goes into the right
direction, but there's an error

Error: CurrencySymbol "YTL" flagged as usedInCompatibleFormatCodes doesn't match "TL" determined 
from format codes.
Error: Currency: more than one currency flagged as usedInCompatibleFormatCodes.

The entry used in the format codes usually stays the same, the number
formatter exchanges symbols in the format codes, but since the older
symbol is now reused it would duplicate the list of formats. I fixed
that, and since the ISO 4217 code TRY actually stays the same and only
the currency symbol changes another attribute legacyOnly="true" is
needed to correctly load documents using the old currency symbol and not
offer it in the dialog otherwise. Using legacyOnly="true" on the old TL
entry and makeing the new one default prevents duplication of the
currency formats.

Much of weird stuff happens if some organizations decide to reuse
symbols ;-)

There is also a new currency sign for Turkish Lira. The sign can be
implemented later.

I would had preferred if we could had added it now, as we're on it
anyway and for changing the symbol again later we would have to
introduce yet another entry and declare the current as legacy, but the
new Unicode code point U+20BA assigned to the symbol isn't in yet and
won't be displayed properly by systems (it is some generic "₺" currency
symbol now) so we'll have to do that later, see also
https://en.wikipedia.org/wiki/Turkish_lira_sign

I commited your patch as d09b4e30d7e87f181fa3913e81c6d38c04a8f5f8

and my follow-up changes cb7c34f3bcd4df7a67f1a9d31b3bf849af4f79e4

(sorry, the fdo machine where cgit runs on is currently down, otherwise
I'd give you browsable URLs)


Btw, please configure your git to use a correct email address, in the
patch I changed <efe@efe-HP-Ubuntu.(none)> to <efeyalaman@gmail.com>

git config --global user.email efeyalaman@gmail.com

As a convention, if a patch fixes a bug we also mention fdo#... in the
commit summary to have a patch assigned to a bug, in this case
fdo#53002, I added that in the commit.

Last but not least, apparently we don't have your license statement on
file, https://wiki.documentfoundation.org/Development/Developers

Could you please send a mail to the mailing list with a blanket
statement that you contribute this and further patches under LGPLv3+ and
MPL 1.1 licenses?

Thanks
  Eike

-- 
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GnuPG key 0x293C05FD : 997A 4C60 CE41 0149 0DB3  9E96 2F1A D073 293C 05FD

Attachment: pgpCC2w80MF_C.pgp
Description: PGP signature


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.