[REVIEW 3-6] Prevent exporting raw field names in pivot table to ODS
Kohei Yoshida
kohei.yoshida at gmail.com
Fri Aug 31 10:27:19 PDT 2012
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
More information about the LibreOffice
mailing list