| Summary: | FILEOPEN DOCX with embeddings: Image is blurred | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Xisco Faulí <xiscofauli> |
| Component: | Writer | Assignee: | Armin Le Grand <Armin.Le.Grand> |
| Status: | VERIFIED FIXED | ||
| Severity: | normal | CC: | Armin.Le.Grand, aron.budea, telesto, timur, xiscofauli |
| Priority: | medium | Keywords: | bibisected, bisected, regression |
| Version: | 6.2.0.0.alpha0+ | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: |
https://bugs.documentfoundation.org/show_bug.cgi?id=118662 https://bugs.documentfoundation.org/show_bug.cgi?id=121022 |
||
| Whiteboard: | target:6.3.0 target:6.2.0.2 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 104468, 112976 | ||
| Attachments: | comparison MSO 2010 and LibreOffice 6.2 | ||
|
Description
Xisco Faulí
2018-08-09 17:22:08 UTC
Regression introduced by: https://cgit.freedesktop.org/libreoffice/core/commit/?id=cbc992e7370ab006ea7c0f8520896845f79f7749 author Armin Le Grand <Armin.Le.Grand@cib.de> 2018-07-19 19:26:14 +0200 committer Armin Le Grand <Armin.Le.Grand@cib.de> 2018-07-20 20:29:11 +0200 commit cbc992e7370ab006ea7c0f8520896845f79f7749 (patch) tree 71000129a35f40d680279134efae7fac1c4e6a52 parent 133da6ed83b278b9e6059c5c1a3d49f9f402792e (diff) tdf#118662 Cleanup old hack with cloned SdrCaptionObj Bisected with: bibisect-linux64-6.2 Adding Cc: to Armin Le Grand attachment 53915 [details] from bug 33590 is also affected by this commit.... and attachment 95137 [details] from bug 75557 and attachment 100957 [details] from bug 79969 *** Bug 119529 has been marked as a duplicate of this bug. *** and attachment 144474 [details] from Bug 119529 Repro 6.3+. DOCX is 2007 but regardless, same is if saved in newer MSO. attachment 4931 image is PNG, with binary object oleObject1.bin inside DOCX. attachment 144474 [details] image is EMF. Also with oleObject1.bin. attachment 53915 [details] image is WMF. oleObject1.bin. attachment 95137 [details] image is EMF. oleObject1.bin. attachment 100957 [details] image is EMF. With embedding Microsoft_Excel_Binary_Worksheet1.xlsb. Also reproduced with attachment 124792 [details] from bug 99631 Could reproduce. Looks as if I was to ambitioned/nice in SvxShape::GetBitmap by also changing the way the Bitmap gets constructed. It's UNO API and thus results have to be in 100th_mm - theoretically. So I first tried to adapt the else-part that creates a Bitmap by scaling the basegfx::B2DRange aRange from the MapTwip when Writer is running to Map100thMM, theoretically correct. But does not scale the Primitives involved... Note: This may also be necessary for the if-part where a Metafile is created... Made this work for the common case ('else' part). Also added a shortcut if the object in question already is a SdrGrafObj, has a Bitmap graphic and that is requested. NOT for Metafile (see comments there). Always better to use the original graphic - using convertPrimitive2DSequenceToBitmapEx may (and as we saw does) scale the result. This is okay if not a SdrGrafObj/Bitmap already.
Looks good, preparing to commit solution...
Solution on 'https://gerrit.libreoffice.org/#/c/65505/'... Okay, should be fine Armin Le Grand committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/df57d50e00820d59b2b7104b6e59883405d0f183%5E%21 tdf#119180 Use 100th_mm in UNO API implementation It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Verified in Version: 6.3.0.0.alpha0+ Build ID: 7d63c700c36afd27850346e42b92768f084f5d4d CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); UI-Language: en-US Calc: threaded @Armin, thanks for fixing this!! Cherry-picked to 6-2: https://gerrit.libreoffice.org/#/c/65523/ Armin Le Grand committed a patch related to this issue. It has been pushed to "libreoffice-6-2": https://git.libreoffice.org/core/+/e42e3f3f28691de2ed8132564066bc3007eac651%5E%21 tdf#119180 Use 100th_mm in UNO API implementation It will be available in 6.2.0.2. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. |