Bug 92164 - FILESAVE XLS Excessive number of rows warning (see comment 10)
Summary: FILESAVE XLS Excessive number of rows warning (see comment 10)
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: Other All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: filter:xls
Depends on:
Blocks: XLS
  Show dependency treegraph
 
Reported: 2015-06-18 20:17 UTC by gmarco
Modified: 2023-04-25 10:43 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments
screen shot of the warning (34.86 KB, image/jpeg)
2015-06-18 20:17 UTC, gmarco
Details
xls spreadsheet (193.50 KB, application/xls)
2015-06-20 14:46 UTC, gmarco
Details
zipped user profile (233.02 KB, application/x-zip-compressed)
2015-06-20 21:31 UTC, gmarco
Details
screenshot (58.43 KB, image/jpeg)
2018-02-24 18:56 UTC, gmarco
Details
screenshot (54.06 KB, image/jpeg)
2018-02-24 18:58 UTC, gmarco
Details
last screenshot (88.58 KB, image/jpeg)
2021-04-14 10:50 UTC, gmarco
Details

Note You need to log in before you can comment on or make changes to this bug.
Description gmarco 2015-06-18 20:17:27 UTC
Created attachment 116636 [details]
screen shot of the warning

I opened a .xls file already previously open/saved in LOCALC, updating it.
Saving again the file I got the attached warning (translation: Warning saving the doc xxx: the document contains a number of rows above the limit supported in the selected format. The rows in excess were not saved).
I have files bigger than this and no warning appears.
A 1st question: which is the supported limit? I've looked for but did not find any info.
Then other observations:
- I tried to save the same still opened file as .xlsx and .ods getting no warning.
- the warning appeared after saving, so I reopened the file but I couldn't note any missing row, even making a comparison with other formats (xlsx and ods).
A 2nd question: I noticed that the size of the files on the disk is significantly different: KB 187 for xls, 61 for xlsx, 64 for ods.

Is it a possible bug?
Could you give me an explanation to questions 1 and 2?
Thanks.
Comment 1 m_a_riosv 2015-06-19 00:36:50 UTC
The limit of rows for xls is 65536, verify you haven't data neither format beyond that.
Comment 2 gmarco 2015-06-19 08:35:29 UTC
(In reply to m.a.riosv from comment #1)
> The limit of rows for xls is 65536, verify you haven't data neither format
> beyond that.

Impossible!
The file contains 3 sheets,  respectively of 24, 215, 291 rows.
I have xls files much stronger without that problem.
I think 65536 is the limit for MSExcel too, but never I had that problem.

What about the substantial difference between xls size and the others?
Comment 3 m_a_riosv 2015-06-19 13:38:02 UTC
The different size I think is in relation with the matter that ods+xlsx are compressed files while xls not.

Sorry @ gmarco, but without access to the file, doesn't seem too easy verify the issue. 

Can you attach the file after clear any information that can't be public.
Comment 4 gmarco 2015-06-19 16:07:28 UTC
(In reply to m.a.riosv from comment #3)
> Can you attach the file after clear any information that can't be public.

Sorry, but unfortunately opening/closing that file the problem no longer occurs.
That file is the same for which I received the warning after the previous save, and apparently is intact, no missing rows.
The fact remaining certain, documented, is that the warning had appeared: why remain a mystery?
Comment 5 m_a_riosv 2015-06-19 16:09:26 UTC
But don't attach the xls, the needed file is ods that when saved shows the issue.
Comment 6 gmarco 2015-06-19 17:02:00 UTC
(In reply to m.a.riosv from comment #5)
> But don't attach the xls, the needed file is ods that when saved shows the
> issue.

I think unusefull to attach anything, the file was xls, had already been modified and resaved from Calc as xls, on the last occasion, after further changes, after saving again as xls, Calc displayed the warning in question.
Now, reopening and resaving that file, the probem do not recurs any more.
Nevertheless, the file is apparently intact without missing rows.
Comment 7 m_a_riosv 2015-06-19 20:28:11 UTC
Then there is nothing to reproduce and verify.
Comment 8 gmarco 2015-06-19 21:32:11 UTC
(In reply to m.a.riosv from comment #7)
> Then there is nothing to reproduce and verify.

Hi, just tonight, changing again the xls file, the problem still occurred.
I wish to point that:
- the message appears after having completed the save (Save As)
- if I do not exit the file and try to save it again with a different name, the warning is repeated
- but if I exit the file, reopen it and resave it (Save As) the warning does no more displays.

Now I ask you: the problem exists, it is regrettably recurring but unfortunately intermittent: do you want the file anyway?
Comment 9 gmarco 2015-06-20 14:46:42 UTC
Created attachment 116681 [details]
xls spreadsheet
Comment 10 gmarco 2015-06-20 15:03:57 UTC
GOAL !!!
I managed to find a way to reproduce repeatedly the error.
The attached file named "SAVE_AS_test.xls" is the sample base.
Instructions:
- open the file (is quite complex but you should not have the need to investigate the contents or on the formulas)
- in the sheet named "scalare" select rows 1:10
- from Menu select "Insert->Rows"
- now you can also Undo, so the file is again alike when just opened
- from File select "Save As" saving with a new name (eg "SAVE_AS_testXXX.xls")
- after the saving end you should get displayed the warning (doing the above I get it repetitively)

Let me know.
Comment 11 m_a_riosv 2015-06-20 16:38:58 UTC
Following steps in comment#10 I can't reproduce the issue.

Please try resetting user profile, sometimes solves strange issues.

https://wiki.documentfoundation.org/UserProfile
Comment 12 gmarco 2015-06-20 21:29:29 UTC
(In reply to m.a.riosv from comment #11)
> Following steps in comment#10 I can't reproduce the issue.
> 
> Please try resetting user profile, sometimes solves strange issues.
> 
> https://wiki.documentfoundation.org/UserProfile

Sorry, I've resetted my user profile but nothing changed.
I attach here the zipped user folder (just resetted without any change) for your investigation.
Remember that the problem happens only if the Save As is .xls, and this also creating the file as .ods, opening it and reSaving As .xls
Comment 13 gmarco 2015-06-20 21:31:00 UTC
Created attachment 116690 [details]
zipped user profile
Comment 14 tommy27 2015-06-25 20:23:35 UTC
@gmarco @m.a.riosv
reproducible as in comment 10 under Win8.1 x64 using LibO 4.4.3.2
please tell which is the O/S you tested.

status NEW. edited summary notes.
Comment 15 gmarco 2015-06-25 21:33:48 UTC
(In reply to tommy27 from comment #14)
> @gmarco @m.a.riosv
> reproducible as in comment 10 under Win8.1 x64 using LibO 4.4.3.2
> please tell which is the O/S you tested.
> 
> status NEW. edited summary notes.

HI,
I too am working under Win8.1 x64 using LibO 4.4.3.2
Comment 16 tommy27 2015-06-25 21:53:25 UTC
warning is not present in LibO 4.2.7.2 and is present in 4.3.0.4 hence a 4.3.x regression that needs bibisecting

issue also persists in today's 5.1.0.0 daily build
Comment 17 gmarco 2015-06-26 10:51:43 UTC
(In reply to tommy27 from comment #16)
> warning is not present in LibO 4.2.7.2 and is present in 4.3.0.4 hence a
> 4.3.x regression that needs bibisecting
> 
> issue also persists in today's 5.1.0.0 daily build

Tommy, fine work, thanks.
But a curiosity: which neologism is bibisecting?
the verb does not exist, what does it mean?
Comment 18 tommy27 2015-06-26 11:10:40 UTC
(In reply to gmarco from comment #17)
> 
> But a curiosity: which neologism is bibisecting?
> the verb does not exist, what does it mean?

it's a method to identify the committ which caused a regression.
more infos here: https://wiki.documentfoundation.org/QA/Bibisect
Comment 19 Robinson Tryon (qubit) 2015-12-14 05:32:35 UTC Comment hidden (obsolete)
Comment 20 Joel Madero 2015-12-20 17:26:47 UTC
Definitely Windows only. Can't repro:

Bodhi Moksha
Version: 5.2.0.0.alpha0+
Build ID: 5df326438fd3a5613a52b4de1935426911ff1301

I don't think that this is bibisectable as the Windows bibisect package doesn't go back this far. See https://wiki.documentfoundation.org/QA/Bibisect/Windows
Comment 21 Xisco Faulí 2016-09-13 10:53:02 UTC
This regression was introduced before branch 4.4, thus it can't be bibisected with the current bibisect repositories. Changing keyword 'notBibisectable' to 'preBibisect'
Comment 22 gmarco 2017-01-07 22:40:36 UTC
Still in LO 5.2.3.3 Win10.
Tha same warning as in the past, both saving as .xls and .ods
Comment 23 gmarco 2017-02-23 21:51:44 UTC
Why this bug is assigned to myself?
Comment 24 m_a_riosv 2017-02-23 22:12:13 UTC
Because you do it, take a look to the history, top-right.
Comment 25 gmarco 2017-02-23 22:22:02 UTC
(In reply to m.a.riosv from comment #24)
> Because you do it, take a look to the history, top-right.

OK, thanks, but I do not know how and why this happened.
I can not fix the bug, so I think corrected unassign myself.
Comment 26 QA Administrators 2018-02-24 03:34:22 UTC Comment hidden (obsolete)
Comment 27 gmarco 2018-02-24 11:50:46 UTC
Sorry but the post is 3 years old and since then I do not use that file anymore, downloading the one attached in 2015 does not reflect the problem anymore.
But many conditions have changed: libo was v.4.3 and Win 8.1, now I have v.5.4 and Win 10, then a "guilty" was, now it will remain "mysterious and unpunished" and so "worksforme".
Greetings.
Comment 28 gmarco 2018-02-24 18:53:38 UTC
Unfortunately I said all that too soon, the problem is still such (I had forgotten to save by Save As).
Following the steps as per comment 10 the warning occurs both in LO5442 and in LO 5451 (see the new screenshots), the only difference is the text, a mix EN-IT, and it occurs also opening/saving an ods file as said per comment 12.
Comment 29 gmarco 2018-02-24 18:56:18 UTC
Created attachment 140119 [details]
screenshot
Comment 30 gmarco 2018-02-24 18:58:18 UTC
Created attachment 140120 [details]
screenshot
Comment 31 gmarco 2018-02-24 21:18:06 UTC
Further information: same warning also in LODev Version: 6.1.0.0.alpha0 + (x64)
Comment 32 Xisco Faulí 2018-02-26 10:44:12 UTC
Putting back to NEW...
Comment 33 Buovjaga 2019-04-14 19:13:13 UTC
Warning is already present in 4.2.0.0alpha1 (first commit of win32-4.3 bibisect repo)

What's more, the warning is present already in 3.5.0 and 3.3.0!

What's EVEN more is that the warning is also present on Linux!!@@

Arch Linux 64-bit
Version: 6.3.0.0.alpha0+
Build ID: 9030ffb1a1b282eb2c6d1773930b0de0d42df447
CPU threads: 8; OS: Linux 5.0; UI render: default; VCL: gtk3; 
Locale: fi-FI (fi_FI.UTF-8); UI-Language: en-US
Calc: threaded
Built on 13 April 2019
Comment 34 QA Administrators 2021-04-14 03:37:37 UTC Comment hidden (obsolete)
Comment 35 gmarco 2021-04-14 10:49:38 UTC
Repeating the case as per comment #10 nothing has changed (LO 7.1.1.2): see attached scr.shot.
Comment 36 gmarco 2021-04-14 10:50:57 UTC
Created attachment 171188 [details]
last screenshot
Comment 37 QA Administrators 2023-04-15 03:27:14 UTC Comment hidden (obsolete)
Comment 38 gmarco 2023-04-15 15:21:20 UTC
YES, I confirm that repeating the case as per comment #10 nothing has changed (LO 7.4.6.2)
Comment 39 ady 2023-04-15 22:12:05 UTC
(In reply to gmarco from comment #38)
> YES, I confirm that repeating the case as per comment #10 nothing has
> changed (LO 7.4.6.2)

@gmarco,

I'd like to ask for confirmation of the following test(s).

Test A (as in comment 10):
1. Open attachment 116681 [details] from comment 9.
2. Select rows _headers_ from row 1 to 10.
3. [CTRL]+[+] (> insert full rows)
4. [CTRL]+[Z] (> undo, the _entire_ rows from row 1 to 10 are still selected)
5. Menu File > Save as > testA.xls (> I get the message about too many columns for the xls file format).


Test B (as in comment 10, but with small difference):
1. Open attachment 116681 [details] from comment 9.
2. Select rows _headers_ from row 1 to 10.
3. [CTRL]+[+] (> insert full rows)
4. [CTRL]+[Z] (> undo, the _entire_ rows from row 1 to 10 are still selected)
5. [CTRL]+[HOME] (> no row is selected; only cell A1 has focus)
6. Menu File > Save as > testB.xls

With testA, I get the message about too many columns for the xls file format.

With testB, no such message.

@gmarco, Could you please confirm whether you experience the aforementioned respective behaviors with each of these two tests?
Comment 40 ady 2023-04-15 22:41:53 UTC
FWIW, I can get an even simpler test.

1. Open attachment 116681 [details] from comment 9.
2. Click on the _header_ for row 1 (> the _entire_ row 1 is selected)
3. Menu File > Save as > testB.xls (the name must change from the original).

With that, the message about too many columns for xls file format is displayed once the file is saved for the first time with a new name.

Note that no real action was performed on the file; only the entire row is selected and a new name is used (in order to trigger the check and thus the message).

When only cell A1 has focus (so, no full row is selected), there is no message.

BTW, the title subject of this report is incorrect. It is _not_ about too many rows, but about too many columns, and the only requirement is to have "too many columns selected by the time the file is saved as xls" in order to trigger the message.
Comment 41 ady 2023-04-15 22:50:46 UTC
(In reply to ady from comment #40)

Sorry, we don't need any prior attachment either.

1. Open Calc. then.

> 2. Click on the _header_ for row 1 (> the _entire_ row 1 is selected)

3. [CTRL]+[S] as xls.

The rest is of course valid.

> 
> With that, the message about too many columns for xls file format is
> displayed once the file is saved for the first time with a new name.
> 
> Note that no real action was performed on the file; only the entire row is
> selected and a new name is used (in order to trigger the check and thus the
> message).
> 
> When only cell A1 has focus (so, no full row is selected), there is no
> message.
> 
> BTW, the title subject of this report is incorrect. It is _not_ about too
> many rows, but about too many columns, and the only requirement is to have
> "too many columns selected by the time the file is saved as xls" in order to
> trigger the message.
Comment 42 gmarco 2023-04-16 14:06:05 UTC
Well, the substance doesn't change and perhaps it becomes more complicated,
selecting only line 1 and saving the file as xls this is the result:
- saving with [CTRL]+[S] everything is OK
- saving with [CTRL]+[Uppercase]+[S] the warning is (translated from IT):
"Be careful when saving the test document:
The document contains more columns than the limit supported in the selected format.
The excess columns have not been saved."
NOTE: IT SAYS columns !

but ...
selecting only column A and saving the file as xls with [CTRL]+[Uppercase]+[S] the warning is:
"Be careful when saving the test document:
The document contains more lines than the limit supported in the selected format.
The excess lines have not been saved."
NOTE: IT SAYS lines !

I confirm that the problem occurs only when saving the file with SAVE AS changing the name and every time after that it is re-saved with SAVE AS changing the name again.
Comment 43 gmarco 2023-04-16 14:31:00 UTC
I add:
same problem also opening a new sheet [CTRL]+[N] and copying only the used cells A1:P305 there, then selecting a column or a row and saving it as xls with a different name.
Comment 44 ady 2023-04-16 16:05:16 UTC
(In reply to gmarco from comment #43)
> I add:
> same problem also opening a new sheet [CTRL]+[N] and copying only the used
> cells A1:P305 there, then selecting a column or a row and saving it as xls
> with a different name.

You don't need to copy anything. When starting Calc with a new (yet to be saved for the first time) spreadsheet, and then selecting entire rows (or entire columns), the first save (as) with a new name will trigger the message.

IOW, by just _selecting_ more rows or more columns than xls supports, the xls filter attempts to save all those selected columns/rows as if all of them were part of the active range, while in fact they are only selected.

The behavior I am describing here is not not present in AOO and was not present in LO 3.3. It is indeed present in LO 4.4. I have not tested versions between them.

Maybe by solving the "selection-only" part of the problem will solve the original problem too, with simpler STR.

ATM most of the 40+ prior comments in this report seem obsolete by now.
Comment 45 gmarco 2023-04-16 20:30:29 UTC
... and then?

Just for information, as of today I have no more potentially affected xls files.
Comment 46 ady 2023-04-16 22:17:34 UTC
(In reply to gmarco from comment #45)
> ... and then?
> 
> Just for information, as of today I have no more potentially affected xls
> files.

One possibility would be to open a new bug report, with the following STR:

1. New Calc.
2. [CTRL]+[A]
3. [CTRL]+[SHIFT]+[S]
4. Use a new/different file name; select the type of file to be 97-2003 XLS; [OK]
5. In the first dialog, confirm the XLS file format (instead of the suggested ODS file format).

Actual result:
There is a warning, either about too many rows or about too many columns not supported by the (XLS) file format.

Expected result:
Avoid either of the 2 possible warnings when it is about selection _only_ and there is no real dependency on columns/rows beyond the XLS limit (i.e. beyond cell IV65536, or R256C65536).


Then this bug 92164 could be set as "See also" of the new one (not as duplicate, just in case, unless QA team decides it is what it should be).

Advantages:
* simple STR,
* covers the 2 possible warnings (i.e. both cases should be solved),
* no need to read 40+ comments (most of them obsolete by now).


Having said that, I would rather have some feedback/comment about this suggestion from someone from the QA team before doing anything of that sort.
Comment 47 gmarco 2023-04-17 07:02:28 UTC
(In reply to ady from comment #46)

Following up on my comment 45, as I said, I'm personally no longer affected by the problem (it was from 2015) and you can do what you prefer.