Bug 128724 - EDITING: ** Expression is faulty ** selecting cells in table
Summary: EDITING: ** Expression is faulty ** selecting cells in table
Status: RESOLVED NOTABUG
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
5.2 all versions
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected
Depends on:
Blocks: Writer-Tables-Formulas
  Show dependency treegraph
 
Reported: 2019-11-11 23:42 UTC by Xisco Faulí
Modified: 2020-10-19 09:28 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
sample file (12.88 KB, application/vnd.oasis.opendocument.text)
2019-11-11 23:42 UTC, Xisco Faulí
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Xisco Faulí 2019-11-11 23:42:32 UTC
Created attachment 155720 [details]
sample file

Steps to reproduce:
1. Open attached document
2. Click on any cell in PVP TOTAL column
3. Total, IVA and total factura cells become ** Expression is faulty **

Reproduced in

Version: 6.4.0.0.alpha1+
Build ID: fea5c10e222b10d83e0081dc1d1b2e678689d250
CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: x11; 
Locale: ca-ES (ca_ES.UTF-8); UI-Language: en-US
Calc: threaded

and


Version: 5.2.0.0.alpha0+
Build ID: 3ca42d8d51174010d5e8a32b96e9b4c0b3730a53
Threads 4; Ver: 4.15; Render: default; 

but not in

Version: 5.0.0.0.alpha1+
Build ID: 0db96caf0fcce09b87621c11b584a6d81cc7df86
Locale: ca-ES (ca_ES.UTF-8)

thus, it needs to be bisected with 5.1 repo
Comment 1 Durgapriyanka 2019-11-19 15:50:22 UTC
I can reproduce this bug in

Version: 6.4.0.0.alpha1+ (x86)
Build ID: ec7374ff84c71edfbb30d6e4dc5b486b6df7107f
CPU threads: 2; OS: Windows 6.1 Service Pack 1 Build 7601; UI render: default; VCL: win; 
TinderBox: Win-x86@42, Branch:master, Time: 2019-11-10_21:37:30
Locale: en-US (en_US); UI-Language: en-US
Calc: threaded

But, not it

LibreOffice 3.3.0 
OOO330m19 (Build:6)
tag libreoffice-3.3.0.4
Comment 2 Xavier Van Wijmeersch 2019-11-20 07:10:04 UTC
Hi Xisco,

Did have a look to your sample file and i found that your formula is wrong
=suma(<E3:E6>) need to be =suma<E3:E6>
idem for formula in Total Factura

and then everything is working fine

its a WFM

Best Regards
Comment 3 Xisco Faulí 2019-11-20 09:35:13 UTC
(In reply to Xavier Van Wijmeersch from comment #2)
> Hi Xisco,
> 
> Did have a look to your sample file and i found that your formula is wrong
> =suma(<E3:E6>) need to be =suma<E3:E6>
> idem for formula in Total Factura
> 
> and then everything is working fine
> 
> its a WFM
> 
> Best Regards

Hi Xavier,
The formula might be wrong, but as said in the description, the sum is correct in 5.0.0.0.alpha1+
Comment 4 raal 2019-11-20 14:46:55 UTC
Started with
 96412245dbfda9f4eee0a7ceeb0e7801ad33ea74 is the first bad commit
commit 96412245dbfda9f4eee0a7ceeb0e7801ad33ea74
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Tue Nov 24 21:59:24 2015 -0800

    source d667e3210b12c7ce3b3727e2a0e369a520fbaaa4

author	Julien Nabet <serval2412@yahoo.fr>	2015-11-22 17:42:08 +0100
committer	Mike Kaganski <mikekaganski@hotmail.com>	2015-11-24 22:03:10 +0000
commit d667e3210b12c7ce3b3727e2a0e369a520fbaaa4 (patch)
tree dbceb51cdab2c9205e483a859a34f1a838cbc9e6
parent 994d80fd53bc264cef92017dba5cbab1241e5a5b (diff)
tdf#61228: Wrong function names in Formula Writer are considered as SUM
Comment 5 Julien Nabet 2019-11-20 15:36:18 UTC
I know that Calc can use localized function but I just wonder if it's also true in Writer.
I mean if you replace by "SUMA" by "SUM" in formula, it works.
The quoted commit fixed a bugtracker where an unknown function was always considered as "SUM".

If I try different languages from this page:
https://help.libreoffice.org/Writer/Calculating_Cell_Totals_in_Tables/de
I see "=sum" which is quoted for French and German, Portuguese but for Spanish I indeed see "=suma" but is it a doc bug or code bug?
Comment 6 Timur 2020-09-25 14:29:54 UTC
Repro 7.1+
Comment 7 Mike Kaganski 2020-10-19 08:43:43 UTC
This is not a bug, but fixed tdf#61228: before that fix, just *any garbage* as a function name (including, but not limited to, wrong spelling / expected localization of "sum") was ignored, and its argument was processed as sum. It was fixed, and all wrong formulas surfaced *correctly* and *expectedly*. This formula are wrong, and only worked previously because of that bug.