Summary: | Default Array Constant separator doesn't works with language using comma as decimal separator | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | GerardF <gerard.fargeot> |
Component: | Calc | Assignee: | Kohei Yoshida <kohei> |
Status: | CLOSED FIXED | ||
Severity: | critical | CC: | kendy, kohei, sophi |
Priority: | medium | ||
Version: | 3.3.0 RC1 | ||
Hardware: | x86 (IA32) | ||
OS: | Windows (All) | ||
Whiteboard: | target:3.3 | ||
Crash report or crash signature: | Regression By: |
Description
GerardF
2010-12-22 00:27:32 UTC
Kohei: Is there anything else needed than the help change, please? Hello I confirm with XP SP3 (In reply to comment #1) > Kohei: Is there anything else needed than the help change, please? Avoid the Err 512 would be greatly appreciated ... :) Best regards I'll take this. I bet somewhere in the interpreter there is a hard-coded '.'. Ok. I fixed it on master. The formula compiler was checking for a number followed by a '.', and throwing an error because we don't support unquoted numbers as sheet names (e.g. 100.A1 instead of '100'.A1 where the 100 is a sheet name). And there, the '.' was hard-coded. http://cgit.freedesktop.org/libreoffice/calc/commit/?id=98764831e3b6c02d7630c61a6c389ce4318787bd I removed this check since we already automatically quote numerical sheet names anyway, so this check is totally unnecessary not to mention it breaks in-line arrays. Plus, the sheet-to-ref separator is not always a '.'; in Excel A1 and R1C1 mode, '!' is used as the separator, so this check makes no sense. FYI, Kendy has ported this fix to 3.3. Verified - Closed - Sophie |