Bug 50471 - FILESAVE: false Warning saving in xls format "contains more rows than supported"
Summary: FILESAVE: false Warning saving in xls format "contains more rows than supported"
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
3.5.3 release
Hardware: Other All
: medium enhancement
Assignee: Not Assigned
URL:
Whiteboard: BSA
Keywords: filter:xls
Depends on:
Blocks: XLS-Limitations
  Show dependency tree / graph
 
Reported: 2012-05-29 10:46 UTC by Desgeorge
Modified: 2023-04-25 10:54 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:

Attachments
a sample ods sheet 481 rows (64.05 KB, application/vnd.oasis.opendocument.spreadsheet)
2012-05-29 10:46 UTC, Desgeorge 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Desgeorge 2012-05-29 10:46:50 UTC 
Created attachment 62233 [details]
a sample ods sheet 481 rows

Problem description: 
when saving a spreadsheet in ods format, then in xls format, to distribute, I get the message saying:
"Warnig saving the document xxx:"
"the document contains more rows than supported in the selected format."
"Additional rows were not saved"
which is not true, both files have the same number of rows 481.

If I re-open the xls file, make a change and save, I get no warning.
both files saved as csv and compared are identical.
Steps to reproduce:
1. open myfile.ods 
2. make a change, save as ods
3. save as xls Excel 97

Current behavior:
Warning
Expected behavior:
Warning only if this is true ;-)
Platform (if different from the browser): 
              
Browser: Mozilla/5.0 (Windows NT 5.1; rv:12.0) Gecko/20100101 Firefox/12.0
Comment 1 Björn Michaelsen 2012-06-11 05:12:38 UTC 
confirmed by dupe/downstream bug
Comment 2 Björn Michaelsen 2012-06-11 05:14:30 UTC 
Is this a regression? Did this work in 3.4/3.3?
Comment 3 Eike Rathke 2012-06-11 05:37:13 UTC 
The attached file has all cells of columns C to IN explicitly formatted with white background down to row 1048576. Binary xls 97 file format supports only 64k rows, so the message is correct.
Comment 4 Eike Rathke 2012-06-11 06:35:07 UTC 
Thinking this over, the expected behavior might be that extraneous attributed-only rows are ignored in this case. Import of xls 97 already handles that by expanding the attributed range. Actually LibO 3.4 treated this differently and did not show the message on export.
Comment 5 Markus Mohrhard 2012-06-11 06:46:28 UTC 
(In reply to comment #4)
> Thinking this over, the expected behavior might be that extraneous
> attributed-only rows are ignored in this case. Import of xls 97 already handles
> that by expanding the attributed range. Actually LibO 3.4 treated this
> differently and did not show the message on export.

I had to change the export of attribute only rows because without the row height information which are part of the attributes we misplace cell anchored objects.
Comment 6 Eike Rathke 2012-06-11 07:27:17 UTC 
(In reply to comment #5)

Well, yes, but it should work to export the full range that is supported by the target format and truncate the extraneous rows that are only attributed. Or am I missing something?
Comment 7 Markus Mohrhard 2012-06-11 07:39:26 UTC 
That sounds sane.
Comment 8 Joel Madero 2014-11-06 20:54:08 UTC 
Looks like this was confirmed but never taken up by anyone - moving to NEW.