Bug 104771 - Graphics unit is not displayed correctly in the converted file.
Summary: Graphics unit is not displayed correctly in the converted file.
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
4.2.0.4 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, filter:doc, regression
Depends on:
Blocks:
 
Reported: 2016-12-19 06:39 UTC by pet47
Modified: 2021-04-07 08:04 UTC (History)
5 users (show)

See Also:
Crash report or crash signature:


Attachments
description of the bug (32.06 KB, application/pdf)
2016-12-19 06:47 UTC, pet47
Details
source-data (55.50 KB, application/msword)
2016-12-19 06:50 UTC, pet47
Details
Image extracted from document (20.14 KB, image/x-emf)
2017-04-27 23:01 UTC, Bartosz
Details
MS Word vs LibreOffice 7.2 master comparison (1.27 MB, image/png)
2021-04-07 07:53 UTC, Bartosz
Details

Note You need to log in before you can comment on or make changes to this bug.
Description pet47 2016-12-19 06:39:15 UTC
Description:
The doc-file with CorelDraw-unit is not displayed correctly (see attachment).

Steps to Reproduce:
- Create a drawing in the CorelDRAW.
- Paste it in MSWord-file.
- Open the file in the LibreOfficeWriter.

Actual Results:  
We see distortion.

Expected Results:
display without distortion.


Reproducible: Always

User Profile Reset: No

Additional Info:
In the attachment:
- A block diagram of the problem (diagram of bug.odg).
- File with problematic data (data.doc).


User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.79 Safari/537.36 Edge/14.14393
Comment 1 pet47 2016-12-19 06:47:27 UTC
Created attachment 129760 [details]
description of the bug
Comment 2 pet47 2016-12-19 06:50:54 UTC
Created attachment 129761 [details]
source-data
Comment 3 Telesto 2016-12-19 15:26:47 UTC
Confirming with:
Version: 5.4.0.0.alpha0+
Build ID: 53edf60c4ce6ed32f87471e018878c40b788005a
CPU Threads: 4; OS Version: Windows 6.19; UI Render: default; 
TinderBox: Win-x86@42, Branch:master, Time: 2016-12-18_06:57:59
Locale: nl-NL (nl_NL); Calc: CL

and with
Versie: 4.4.6.3 
Build ID: e8938fd3328e95dcf59dd64e7facd2c7d67c704d
Locale: nl_NL

and with
Version: 4.3.0.4
Build ID: 62ad5818884a2fc2e5780dd45466868d41009ec0

and with:
Versie: 4.2.0.4 
Build ID: 05dceb5d363845f2cf968344d7adab8dcfb2ba71

but not with:
Versie: 4.1.0.4 
Build ID: 89ea49ddacd9aa532507cbf852f2bb22b1ace28
Comment 4 Xisco Faulí 2016-12-22 16:31:10 UTC
Regression introduced by:

author	Jan Holesovsky <kendy@collabora.com>	2013-11-21 15:28:32 (GMT)
committer	Jan Holesovsky <kendy@collabora.com>	2013-11-25 13:10:46 (GMT)
commit 4ba2b2ae62276ab7b40ba5730d631ad4b3fd6ac8 (patch)
tree b17713c55073c9acb1f61bbc10bf7bf718d91865
parent 9d3c0aa1e64ae97ddc305df3873f977051f0b317 (diff)
fdo#61272: Do the mapping correctly both for WMF and EMF.
Turns out that for the WMF, we already had everything set up correctly, we
were just overwriting the right settings with default data again :-) - fix
that.

Adding Cc : to Jan Holesovsky

@Kendy: it looks like reverting the commit above fixes the problem and doesn't break fdo#61272. Could you take a look? probably just reverting the commit fixes everything...
Comment 5 Jan Holesovsky 2016-12-23 09:58:55 UTC
Xisco: Too bad I did not create a unit test at the time of the fix, we could be sure now if reverting is OK :-( [because it is possible that something else has changed making the previous fix obsolete]

Xisco: would it be possible to isolate what exactly my patch fixed?  Unfortunately I don't remember; ideally if you could try in the bibisect repo how the document from bug 61272 looked like before 4ba2b2ae62276ab7b40ba5730d631ad4b3fd6ac8 and after that (ie. what was the exact difference that my patch fixed)?

Thank you!
Comment 6 Xisco Faulí 2016-12-23 10:47:00 UTC
(In reply to Jan Holesovsky from comment #5)
> Xisco: Too bad I did not create a unit test at the time of the fix, we could
> be sure now if reverting is OK :-( [because it is possible that something
> else has changed making the previous fix obsolete]
> 
> Xisco: would it be possible to isolate what exactly my patch fixed? 
> Unfortunately I don't remember; ideally if you could try in the bibisect
> repo how the document from bug 61272 looked like before
> 4ba2b2ae62276ab7b40ba5730d631ad4b3fd6ac8 and after that (ie. what was the
> exact difference that my patch fixed)?

Hi Jan,
I've done some bisections here and the files in bug 61272 look the same even with 4ba2b2ae62276ab7b40ba5730d631ad4b3fd6ac8 or without it. However, the real fix for bug 61272 is done by 198b17dc5e182dfb2e5c930458764c7b3e6c914f. So I can't really tell what 4ba2b2ae62276ab7b40ba5730d631ad4b3fd6ac8 fixes using the files in bug 61272. Do you know how to get file n#417818 ?
Comment 7 Xisco Faulí 2017-04-13 08:39:20 UTC Comment hidden (obsolete)
Comment 8 Bartosz 2017-04-27 23:01:58 UTC
Created attachment 132922 [details]
Image extracted from document
Comment 9 QA Administrators 2018-04-28 02:31:55 UTC Comment hidden (obsolete)
Comment 10 pet47 2018-04-28 04:37:15 UTC
In LibreOffice:

Версия: 6.0.3.2 (x64)
ID сборки: 8f48d515416608e3a835360314dac7e47fd0b821
Потоков ЦП: 8; ОС:Windows 10.0; Отрисовка ИП: по умолчанию; 
Локаль: ru-RU (ru_RU); Calc: group

bug is present.
Comment 11 Xisco Faulí 2018-06-05 21:05:20 UTC Comment hidden (obsolete)
Comment 12 QA Administrators 2019-06-06 02:53:48 UTC Comment hidden (obsolete)
Comment 13 Bartosz 2021-04-07 07:53:01 UTC
Created attachment 170994 [details]
MS Word vs LibreOffice 7.2 master comparison

Unfortunately I cannot reproduce this issue:

Version: 7.2.0.0.alpha0+ / LibreOffice Community
Build ID: 615ceb107e9faf01b568b0a2440a3f09c8f88ca6
CPU threads: 8; OS: Mac OS X 10.15.7; UI render: default; VCL: osx
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded