Bug 167182 - In the help page of the Fourier function, the argument of the example is wrong
Summary: In the help page of the Fourier function, the argument of the example is wrong
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Documentation (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: All All
: medium normal
Assignee: Julien Nabet
URL:
Whiteboard: target:26.2.0 target:25.8.0.0.beta2
Keywords:
Depends on:
Blocks:
 
Reported: 2025-06-24 07:21 UTC by nobu
Modified: 2025-07-11 07:10 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description nobu 2025-06-24 07:21:50 UTC
Description:
In the help page of the Fourier function, the argument of the example is wrong.
LibreOffice 25.2 Help

Steps to Reproduce:
1. Open Help Page of FOURIER Function.
   https://help.libreoffice.org/latest/en-US/text/scalc/01/func_fourier.html#bm_id581556228060864
2. See Syntax.
   https://help.libreoffice.org/latest/en-US/text/scalc/01/func_fourier.html#func_head_syntax

   =FOURIER(Array; GroupedByColumns [; Inverse [; Polar [; MinimumMagnitude]]])
                                       ^^^^^^^    ^^^^^
3. See Examples.
   https://help.libreoffice.org/latest/en-US/text/scalc/01/func_fourier.html#func_head_example

        Col A            : Col B
Row1 :	Group By Columns : TRUE
Row2 :  Polar            : FALSE
        ^^^^^
Row3 :  Inverse          : FALSE
        ^^^^^^^
Formula : {=FOURIER(B6:C40,B1,B2,B3,0)}

Actual Results:
4. Description of Cell [B2] : Cell [A2] is Polor
   Description of Cell [B3] : Cell [A3] is Inverse
   The order is wrong.

Expected Results:
4. Description of Cell [B2] : Cell [A2] is Inverse
   Description of Cell [B3] : Cell [A3] is Polor


Reproducible: Always


User Profile Reset: No

Additional Info:

Both are FALSE, so there is no difference in the results.
Comment 1 Julien Nabet 2025-06-25 18:30:27 UTC
Olivier: I've just submitted this patch:
https://gerrit.libreoffice.org/c/help/+/187000
Comment 2 Commit Notification 2025-06-25 19:49:24 UTC
Julien Nabet committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/help/commit/48572394a6d89fe4472d3adce8b4b5ac35574b07

tdf#167182: Fix Fourier swapped arguments
Comment 3 Julien Nabet 2025-06-25 20:00:13 UTC
Cherry-pick for 25.8 here:
https://gerrit.libreoffice.org/c/help/+/187006

Of course, after this, you got to wait for website to be sync.
Comment 4 Commit Notification 2025-06-25 20:02:29 UTC
Julien Nabet committed a patch related to this issue.
It has been pushed to "libreoffice-25-8":

https://git.libreoffice.org/help/commit/125a11f9479c1234eeb2a9136a5f914b90c0aef3

tdf#167182: Fix Fourier swapped arguments
Comment 5 nobu 2025-06-30 02:25:39 UTC
Reporting bugs just for this is a pain.
If possible, please correct the delimiter.
I didn't notice because it is just right in my country's locale, but it doesn't seem to follow the rules of the help.

Formula : {=FOURIER(B6:C40,B1,B2,B3,0)}
                          ^  ^  ^  ^
, -> ;
Formula : {=FOURIER(B6:C40;B1;B2;B3;0)}
Comment 6 Julien Nabet 2025-06-30 12:00:43 UTC
(In reply to nobu from comment #5)
> Reporting bugs just for this is a pain.
> If possible, please correct the delimiter.
> I didn't notice because it is just right in my country's locale, but it
> doesn't seem to follow the rules of the help.
> 
> Formula : {=FOURIER(B6:C40,B1,B2,B3,0)}
>                           ^  ^  ^  ^
> , -> ;
> Formula : {=FOURIER(B6:C40;B1;B2;B3;0)}

Olivier: do you confirm?

I mean, there are other locations, eg:
source/text/scalc/01/func_lookup.xhp:44:    <paragraph role="paragraph" id="par_id931750273347370"><input>=LOOKUP(46,A2:B34)</input> returns "Sweden", the country which code is 46. Since the result vector is not defined, the function returns the value in the last column of the search array.</paragraph>
source/text/scalc/01/func_lookup.xhp:45:    <paragraph role="paragraph" id="par_id631750273351969"><input>=LOOKUP(55,A2:A34,D2:D34)</input> returns "BR/BRA", the ISO code for country code 55 (Brazil).</paragraph>
source/text/scalc/01/func_lookup.xhp:46:    <paragraph role="paragraph" id="par_id1001750273540349"><input>=LOOKUP(7,A2:B34)
Comment 7 Julien Nabet 2025-07-10 20:58:37 UTC
(In reply to nobu from comment #5)
> Reporting bugs just for this is a pain.
> If possible, please correct the delimiter.
> I didn't notice because it is just right in my country's locale, but it
> doesn't seem to follow the rules of the help.
> 
> Formula : {=FOURIER(B6:C40,B1,B2,B3,0)}
>                           ^  ^  ^  ^
> , -> ;
> Formula : {=FOURIER(B6:C40;B1;B2;B3;0)}

Ok patch here:
https://gerrit.libreoffice.org/c/help/+/187668
Comment 8 Commit Notification 2025-07-11 07:10:19 UTC
Julien Nabet committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/help/commit/9a0bfd12f63f9648dcefec48b7bd93a92953d49d

Related tdf#167182: change argument separator in Fourier from "," to ";"