<div class="gmail_quote">2012/8/2 Eike Rathke <span dir="ltr"><<a href="mailto:erack@redhat.com" target="_blank">erack@redhat.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi Efe,<br>
<div class="im"><br>
On Thursday, 2012-08-02 00:05:29 +0300, Efe Gürkan YALAMAN wrote:<br>
<br>
</div>Thanks for the patches.<br>
<br>
The first where you added another currency entry goes into the right<br>
direction, but there's an error<br>
<br>
Error: CurrencySymbol "YTL" flagged as usedInCompatibleFormatCodes doesn't match "TL" determined from format codes.<br>
Error: Currency: more than one currency flagged as usedInCompatibleFormatCodes.<br>
<br>
The entry used in the format codes usually stays the same, the number<br>
formatter exchanges symbols in the format codes, but since the older<br>
symbol is now reused it would duplicate the list of formats. I fixed<br>
that, and since the ISO 4217 code TRY actually stays the same and only<br>
the currency symbol changes another attribute legacyOnly="true" is<br>
needed to correctly load documents using the old currency symbol and not<br>
offer it in the dialog otherwise. Using legacyOnly="true" on the old TL<br>
entry and makeing the new one default prevents duplication of the<br>
currency formats.<br></blockquote><div> </div><div>That part is little bit challenging for me. <br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Much of weird stuff happens if some organizations decide to reuse<br>


symbols ;-)<br><br></blockquote><div>Yes i recognized it. :-) <br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
> There is also a new currency sign for Turkish Lira. The sign can be<br>
> implemented later.<br>
<br>
</div>I would had preferred if we could had added it now, as we're on it<br>
anyway and for changing the symbol again later we would have to<br>
introduce yet another entry and declare the current as legacy, but the<br>
new Unicode code point U+20BA assigned to the symbol isn't in yet and<br>
won't be displayed properly by systems (it is some generic "₺" currency<br>
symbol now) so we'll have to do that later, see also<br>
<a href="https://en.wikipedia.org/wiki/Turkish_lira_sign" target="_blank">https://en.wikipedia.org/wiki/Turkish_lira_sign</a><br></blockquote><div><br>By the way new currency sign is just a character inside of a font.<br>

<a href="http://www.tcmb.gov.tr/yeni/eng/">http://www.tcmb.gov.tr/yeni/eng/</a><br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I commited your patch as d09b4e30d7e87f181fa3913e81c6d38c04a8f5f8<br>
<br>
and my follow-up changes cb7c34f3bcd4df7a67f1a9d31b3bf849af4f79e4<br>
<br>
(sorry, the fdo machine where cgit runs on is currently down, otherwise<br>
I'd give you browsable URLs)<br></blockquote><div><br>Thank you.<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Btw, please configure your git to use a correct email address, in the<br>
patch I changed <efe@efe-HP-Ubuntu.(none)> to <<a href="mailto:efeyalaman@gmail.com">efeyalaman@gmail.com</a>><br>
<br>
git config --global user.email <a href="mailto:efeyalaman@gmail.com">efeyalaman@gmail.com</a><br></blockquote><div><br>I recognized after i sending patches. sorry.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br>
As a convention, if a patch fixes a bug we also mention fdo#... in the<br>
commit summary to have a patch assigned to a bug, in this case<br>
fdo#53002, I added that in the commit.<br> 
<br></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Last but not least, apparently we don't have your license statement on<br>
file, <a href="https://wiki.documentfoundation.org/Development/Developers" target="_blank">https://wiki.documentfoundation.org/Development/Developers</a><br>
<br>
Could you please send a mail to the mailing list with a blanket<br>
statement that you contribute this and further patches under LGPLv3+ and<br>
MPL 1.1 licenses?<br></blockquote><div><br>I will.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks<br></blockquote><div> </div><div>It was challenging for me as a 1. grade computer engineering student. I learned a lot from you.<br><br>Thank you again. <br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<span class="HOEnZb"><font color="#888888">  Eike<br>
<br>
--<br>
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.<br>
GnuPG key 0x293C05FD : 997A 4C60 CE41 0149 0DB3  9E96 2F1A D073 293C 05FD<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>Efe Gürkan YALAMAN<br><a href="http://about.me/efegurkan" target="_blank">http://about.me/efegurkan</a><br><br>