Bug 50044 - FILEOPEN partiular .xls shows too small row height (b/c it is defined twice - fine if RT by MSO)
Summary: FILEOPEN partiular .xls shows too small row height (b/c it is defined twice -...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
3.3.0 release
Hardware: All All
: low minor
Assignee: Not Assigned
URL:
Whiteboard: BSA
Keywords: filter:xls
Depends on:
Blocks: XLS MSO-External-Producers
  Show dependency tree / graph
 
Reported: 2012-05-17 05:29 UTC by Nicola Ruggero
Modified: 2022-09-20 16:49 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:

Attachments
document with problem (99.50 KB, application/vnd.ms-excel)
2012-05-17 05:29 UTC, Nicola Ruggero 		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 Nicola Ruggero 2012-05-17 05:29:46 UTC 
Created attachment 61744 [details]
document with problem

Problem description:
Opening the attached file I see a wrong row height. The same file opened in msoffice is shown correctly. 

Steps to reproduce:
1. open attached file

Current behavior:
shows wrong row height

Expected behavior:
show correct row height as in msoffice

Platform (if different from the browser): 
              
Browser: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; WOW64; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729)
Comment 1 Rainer Bielefeld Retired 2012-08-28 04:41:37 UTC 
[Reproducible] with reporter's sample document and "LibreOffice 3.6.1.2  German UI/Locale [Build-ID:  e29a214] on German WIN7 Home Premium (64bit): Height of several rows (all rows with contents except first one?) only 0,56mm height instead of appropriate 5mm.

Any reformatting in any cell of the row (select old character size again, Number formatting, ...) will heal the problem.

For sample document already visible with LibO 3.3.0, but ok with AOOo 3.4 

Currently only reproduced with WIN.

Although the effect is much more extreme, I am pretty sure that this one is a DUP of  "Bug 34717 - FILEOPEN FORMATTING: automatic row height is too small in particular .xls"

@reporter:
Please feel free to reopen this Bug if you find evidence that we have an independent issue here.

*** This bug has been marked as a duplicate of bug 34717 ***
Comment 2 Justin L 2022-09-14 17:22:00 UTC 
The focus of this bug report is the first 4 rows I think.
It looks like the rows are defined twice. The first time the values look sane, but the second time around they are tiny.

A1: not sure why this row was so small. Fixed when I change the font.
A2-A4: redefined from 255 to 32. These are set to manual height, so they honor that last tiny size.
Comment 3 Justin L 2022-09-14 17:47:00 UTC Comment hidden (obsolete)
Comment 4 Justin L 2022-09-14 19:44:49 UTC 
(In reply to Justin L from comment #2)
> A1: not sure why this row was so small.
Bad testing. It actually seems fine in master.
Comment 5 Justin L 2022-09-14 21:40:49 UTC 
If I first round-trip the file with MS Word 2003, it opens fine in LO.

It is calling XclImpColRowSettings::SetRowSettings twice - first for rows 1-627 and then again for rows 1-116. I don't know how to stop that because ImportExcel8::Read() is really generic. I'd guess you would have to understand xls internals pretty good to figure out where to identify and stop a duplicate.