Re: [REVIEW 3-6] Prevent exporting raw field names in pivot table to ODS

2012-08-31 Thread Eike Rathke
Hi Kohei,

On Friday, 2012-08-31 13:27:19 -0400, Kohei Yoshida wrote:

> https://gerrit.libreoffice.org/gitweb?p=core.git;a=commitdiff;h=fe0ef1b2f13d1c2cc8e4f777911c158390183053

cherry-picked to 3-6 with my sign-off
https://gerrit.libreoffice.org/gitweb?p=core.git;a=commit;h=6781ee7039adc7248d887589b1369dda4dc45791

  Eike

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


pgpnEq4FcpwTh.pgp
Description: PGP signature
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[REVIEW 3-6] Prevent exporting raw field names in pivot table to ODS

2012-08-31 Thread Kohei Yoshida

Hi there,

I'd like

https://gerrit.libreoffice.org/gitweb?p=core.git;a=commitdiff;h=fe0ef1b2f13d1c2cc8e4f777911c158390183053

cherry-picked to 3-6 to prevent us from exporting any internal field 
names in pivot table to ODS.


Background

Our pivot table code internally appends '*' character to a duplicate 
field name, because we require that each field name be unique.  The 
decision was not mine but it had been done this way historically.


Unfortunately, this scheme has a big drawback that, if the real field 
name ends with '*', it would cause a catastrophic result.  To make 
matters worse, the document that caused a crash earlier in this bug report


https://bugs.freedesktop.org/show_bug.cgi?id=54303

contained a field whose name was just '*'!

During the 3.6 cycle, I had spent a great deal to isolate the parts of 
the code that adds and removes these '*' from the real field name, with 
the intention of changing this suffix character from '*' to something 
else (probably one that's outside the printable range).


So, it's very important that we prevent these characters from being 
exported into ODS.


Kohei

--
Kohei Yoshida, LibreOffice hacker, Calc
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice