Bug 90798 - [SVG] Exporting image produces incorrect results (image is rotated and not filled correctly)
Summary: [SVG] Exporting image produces incorrect results (image is rotated and not fi...
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: filters and storage (show other bugs)
Version:
(earliest affected)
4.0.3.3 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, regression
Depends on:
Blocks: SVG-Open
  Show dependency treegraph
 
Reported: 2015-04-23 03:26 UTC by Radoslav
Modified: 2018-06-20 17:03 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
ODT File with the issue (45.54 KB, application/vnd.oasis.opendocument.graphics)
2015-04-23 03:26 UTC, Radoslav
Details
SVG exported that is displaying wrong (512.04 KB, image/svg+xml)
2015-04-23 03:27 UTC, Radoslav
Details
PNG version of the SVG file exported (87.15 KB, image/png)
2015-04-23 03:27 UTC, Radoslav
Details
screenshot of the file in Impress after the first bibisected transition (19.52 KB, image/png)
2015-08-20 20:56 UTC, Michael Weghorn
Details
result of export from ODT file to SVG in LO 6.1 beta 2 (533.36 KB, image/svg+xml)
2018-06-18 20:46 UTC, Roman Kuznetsov
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Radoslav 2015-04-23 03:26:41 UTC
Created attachment 115021 [details]
ODT File with the issue

Exporting a simple image (flow diagram) with custom images produces incorrect results. Images are rotated and incorrectly filled. The lines are correctly exported. See attached ODT file and the SVG (also a JPEG of the image is provided). This is not a problem with exporting to other graphic formats.

Steps to Reproduce:
1. Open ODT file provided (savefile-boxrep.odg)
2. Export to SVG

Actual Results:
The SVG image is incorrectly displayed (see savefile-boxrep.svg), the exported images are rotated and incorrectly filled

Expected Results:
The SVG image and the ODT are displayed the same

Build Date & Hardware:
Build ID: 40m0(Build:2), LinuxMint 64bit

Additional Builds and Platforms:
Also occurs on Windows 7 32bit version 4.4.2

Additional Information:
A PNG copy of how the SVG is provided (savefile-boxrep.png)
Comment 1 Radoslav 2015-04-23 03:27:21 UTC
Created attachment 115022 [details]
SVG exported that is displaying wrong
Comment 2 Radoslav 2015-04-23 03:27:52 UTC
Created attachment 115023 [details]
PNG version of the SVG file exported
Comment 3 Buovjaga 2015-04-25 14:11:52 UTC
Reproduced.
In 3.6, the freaky rotation does not happen, even though the texts are not exported.
The rotation is already happening with 4.0.

Adding bibisect request to find the rotation glitch.

Win 7 Pro 64-bit Version: 5.0.0.0.alpha1+ (x64)
Build ID: f3375fa07f27bd2ade519af3c07d69040d10eaa9
TinderBox: Win-x86_64@42, Branch:master, Time: 2015-04-22_23:38:50
Locale: fi_FI

Ubuntu 15.04 64-bit
Version 3.6.7.2 (Build ID: e183d5b)

Version 4.0.3.2 (Build ID: c6786add5a58268e11aa027c47054344040db1b)
Comment 4 Michael Weghorn 2015-08-20 20:55:25 UTC
I bibisected the change in behaviour in the rotation of the image when exporting to SVG.

I could observe two transitions while doing so.

first transition:
* before: file opens OK and rotation is OK in exported SVG file
* afterwards: only the text is displayed when opening the file with LibreOffice, no shapes are shown

second transition:
* before: only the text is displayed when opening the file with LibreOffice, no shapes are shown (s. above)
* afterwards: file opens OK, rotation in the exported SVG file is wrong (as described in this bug report)

The bibisect results for the individual transitions is listed below.

----------

bibisect result for the first transition:
 ba6eb41acb8df58f3009920f8ab8b32a3e1b764e is the first bad commit
commit ba6eb41acb8df58f3009920f8ab8b32a3e1b764e
Author: Bjoern Michaelsen <bjoern.michaelsen@canonical.com>
Date:   Tue Dec 11 01:59:31 2012 +0000

    source-hash-ae4e4a11d4300f7448cb6bd170fcb034542caddc
    
    commit ae4e4a11d4300f7448cb6bd170fcb034542caddc
    Author:     Rene Engelhard <rene@debian.org>
    AuthorDate: Tue Nov 6 21:24:32 2012 +0100
    Commit:     Rene Engelhard <rene@debian.org>
    CommitDate: Tue Nov 6 21:24:32 2012 +0100
    
        typo...
    
        Change-Id: I2c7968194afbcf74967cd16c639dce7de858a513

:100644 100644 c09b92a8ddf24b8738a7cd3a695ae1e4482d354c 6b13afd13f023cedac65a896318de4aa55dead27 M	autogen.log
:100644 100644 bcee1e11bd693642ca5a6330175b58082e5dcdd0 54d2377f5b3afd6ed668b3e24cb877c289119ec5 M	ccache.log
:100644 100644 75677cef16786d2cc95c0d8f301482278ca055c3 d18a6ebcd3b8537c3dd3dbc16e862fb6370ecb8a M	commitmsg
:100644 100644 83b4ecc0ecb3e031c804d49f45f854e95d5cc961 d482c00e5fff6081c6b87b637fd9ffaf3e1e2c6c M	dev-install.log
:100644 100644 91437b9974ffada7e049273419bb8c66c91c0539 abf45ab9609c77a5ab952aa310eafe8c2ffaf85c M	make.log
:040000 040000 45c0bab8b669778ab523c8f65a25e8c9afde604a f4bfa15b2f72d3df5675b07bcf32254a819b9d19 M	opt

---

$ git bisect log
# bad: [752769ad0d2179e17ea0a08cc9004df7b890305b] source-hash-60c64b437c6678dd1d3fa3a6fc2b7da0480890d4
# good: [65fd30f5cb4cdd37995a33420ed8273c0a29bf00] source-hash-d6cde02dbce8c28c6af836e2dc1120f8a6ef9932
git bisect start 'last42onmaster' 'oldest'
# bad: [d119104d807d798d21d6ee0fb8644b1ea4266c1e] source-hash-20f2aaced1dabbd62ea516368b85e0e881d42165
git bisect bad d119104d807d798d21d6ee0fb8644b1ea4266c1e
# good: [0a0d80b18dc905ee56faaad81c2d6839f8e0172d] source-hash-4ba8147f61fadb4e8ae7abc0ad5c9e928edf4baa
git bisect good 0a0d80b18dc905ee56faaad81c2d6839f8e0172d
# good: [cf8644ffc4c66e8567e9c4be706caac879032caf] source-hash-3cf91a21fc5089fb7f051bf8a04d2049da88179f
git bisect good cf8644ffc4c66e8567e9c4be706caac879032caf
# good: [bd5c49f162a09d173164447720e74257027c6efc] source-hash-f260c656da4457c5d87c161bdd43ad3023d07472
git bisect good bd5c49f162a09d173164447720e74257027c6efc
# bad: [99f63b2b53c0e22baac045d54f502508d7150fef] source-hash-d38a2e3ea04d354492df18aa16d2304babe87dfb
git bisect bad 99f63b2b53c0e22baac045d54f502508d7150fef
# bad: [e5973caebe5b9637f93a4da008d76b33b9d5ff6a] source-hash-683758efb22d08a4cf211a6d985148f513da2a90
git bisect bad e5973caebe5b9637f93a4da008d76b33b9d5ff6a
# good: [51b63dca7427db64929ae1885d7cf1cc7eb0ba28] source-hash-806d18ae7b8c241fe90e49d3d370306769c50a10
git bisect good 51b63dca7427db64929ae1885d7cf1cc7eb0ba28
# good: [fae90325861bbddd2af90937d29d91637c96661a] source-hash-4316e643ef345b0f673b4a03a80a4b7cb3185588
git bisect good fae90325861bbddd2af90937d29d91637c96661a
# bad: [ba6eb41acb8df58f3009920f8ab8b32a3e1b764e] source-hash-ae4e4a11d4300f7448cb6bd170fcb034542caddc
git bisect bad ba6eb41acb8df58f3009920f8ab8b32a3e1b764e
# first bad commit: [ba6eb41acb8df58f3009920f8ab8b32a3e1b764e] source-hash-ae4e4a11d4300f7448cb6bd170fcb034542caddc


----------

bibisect result for the second transition:

$ git bisect start

 a7e54955e9f49e8b59dfd8c4533785a680b1796c is the first bad commit
commit a7e54955e9f49e8b59dfd8c4533785a680b1796c
Author: Bjoern Michaelsen <bjoern.michaelsen@canonical.com>
Date:   Wed Oct 16 11:07:50 2013 +0000

    source-hash-5da10275a7475efdbfd9de14ea58cf8f4c6c1582
    
    commit 5da10275a7475efdbfd9de14ea58cf8f4c6c1582
    Author:     Stephan Bergmann <sbergman@redhat.com>
    AuthorDate: Fri Mar 1 17:09:45 2013 +0100
    Commit:     Stephan Bergmann <sbergman@redhat.com>
    CommitDate: Fri Mar 1 17:18:29 2013 +0100
    
        Related rhbz#915743: Abort UCB call from SvtMatchContext_Impl::Stop
    
        ...as otherwise the SvtMatchContext_Impl thread can continue to run for
        arbitrarily long, and the other thread calling Stop() and join() will block.
    
        However, especially the WebDAV UCP does not properly support aborting commands,
        see 260afe56fd6b2f34de8290f3cdb7d1df5b88f8a8 " neon commands cannot be aborted",
        so this is not yet enough to actually fix rhbz#915743 "thread deadlock/slow
        join in insert->hyperlink in impress."
    
        Change-Id: I0da899f824763e1b3d19bb5b38d906feb690b623

:100644 100644 fd22aadcebcf1ca20b6c2fcdb9e135deeb9b5885 8a0f14e1bb71d7ecdf8086c62e9769bb7f2d09b8 M	autogen.log
:100644 100644 5af869ab53b50329a270e7d4e2587f802bf68afb 8519bf956c5e06a85818d380070eedc0ef846790 M	ccache.log
:100644 100644 63cd7351c9d6feb098661a5783d51bb172d8a306 33abac29aad7182260562465482b493d94b78a83 M	commitmsg
:100644 100644 e9ea867065a69fa4f0fbbb5c2abb40baeeabd307 21fc5294b2cb922862b78327b6b8a3cd953f38b5 M	dev-install.log
:100644 100644 4c087a5ff52a8cef08f31417ac650666b1d9d0af c1cc87465560a589137349c81641a62968242386 M	make.log
:040000 040000 ece742cbaf9101d015210ea8da6c00ad7a4457c7 9ff9cbceea1fe6b0ad1b17fe9068b2c8e32a6cbb M	opt

---

$ git bisect log
# bad: [891878bfdd77870aa99f3f9cc6ee6169c4d1c886] source-hash-ef4b5aad82da7133e89e37e3e04791398d6fb3a6
# good: [d119104d807d798d21d6ee0fb8644b1ea4266c1e] source-hash-20f2aaced1dabbd62ea516368b85e0e881d42165
git bisect start '891878bfdd77870aa99f3f9cc6ee6169c4d1c886' 'd119104d807d798d21d6ee0fb8644b1ea4266c1e'
# bad: [1c8ce1b905c9fd78415d666f7f0d9b39bd99a431] source-hash-eceecd4a3806f64c2e8fb0a3bcdcc43e1384779f
git bisect bad 1c8ce1b905c9fd78415d666f7f0d9b39bd99a431
# good: [31bac4fcc40da1e460b132be36dfa002e52919d0] source-hash-d55155cad0926f61b5745260196b93e95471d06a
git bisect good 31bac4fcc40da1e460b132be36dfa002e52919d0
# bad: [c826604de689fbabd8b1b8ea41396694e99a23d4] source-hash-32acb98b3fb6acb4712f7195cf5ea1bd69c9c6b4
git bisect bad c826604de689fbabd8b1b8ea41396694e99a23d4
# good: [5e90d936616ff95724eaa3e3a0a7c7a9747e9b44] source-hash-ba446dd58a4ad324d242afcd5b28d3b4dff5a881
git bisect good 5e90d936616ff95724eaa3e3a0a7c7a9747e9b44
# bad: [fe956dc63cc7ed1831f0e7e9e7253ea4d8c99549] source-hash-b15f095293c6127ecaef2f0fa3a1683e72392835
git bisect bad fe956dc63cc7ed1831f0e7e9e7253ea4d8c99549
# bad: [a7e54955e9f49e8b59dfd8c4533785a680b1796c] source-hash-5da10275a7475efdbfd9de14ea58cf8f4c6c1582
git bisect bad a7e54955e9f49e8b59dfd8c4533785a680b1796c
# first bad commit: [a7e54955e9f49e8b59dfd8c4533785a680b1796c] source-hash-5da10275a7475efdbfd9de14ea58cf8f4c6c1582
Comment 5 Michael Weghorn 2015-08-20 20:56:59 UTC
Created attachment 118053 [details]
screenshot of the file in Impress after the first bibisected transition

The screenshot shows what the file looks like in Impress after the first transition as described in the previous comment. Only the text is shown in Impress; the shapes are not displayed.
Comment 6 Robinson Tryon (qubit) 2015-12-13 11:12:22 UTC Comment hidden (obsolete)
Comment 7 QA Administrators 2017-01-03 19:49:51 UTC Comment hidden (obsolete)
Comment 8 Aron Budea 2017-01-05 20:00:03 UTC
Bug still partly occurs with 5.3beta2 / Ubuntu 16.04.
The exported SVG looks okay in Firefox, but it still looks wrong after opening it in Draw.

Reverse bibisect points to the following commit that fixed the export issue:
https://cgit.freedesktop.org/libreoffice/core/commit/?id=880c6e40f694342408681929ae5c4fe12bd63332
author	Marco Cecchetti <marco.cecchetti@collabora.com>	2016-06-19 21:45:20 (GMT)
committer	Marco Cecchetti <mrcekets@gmail.com>	2016-07-01 09:42:35 (GMT)

"bccu#1307 - svg filter - added support for clip region meta action"

 8e758e0ae958d9fce493c92e2be07d732d16cfaa is the first bad commit
commit 8e758e0ae958d9fce493c92e2be07d732d16cfaa
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Wed Sep 28 05:30:27 2016 +0200

    source 880c6e40f694342408681929ae5c4fe12bd63332

# bad: [1bfd8dda84f0dd2c5662b64f382637d75b8bf227] source 6238f71ddbdc766e733b1c808a4fa7d66f7bde87
# good: [33e60eae04c889baf52713a73dc9944015408914] source 5b168b3fa568e48e795234dc5fa454bf24c9805e
git bisect start '1bfd8dda84f0dd2c5662b64f382637d75b8bf227' 'oldest'
# bad: [964789dfd0674f0447da363a8c52114097796fa3] source e473e0e1b9bc354d53908cb0ca84db06c3051fe2
git bisect bad 964789dfd0674f0447da363a8c52114097796fa3
# bad: [e076e507d65a9ac4e63569e22b7384a0eb67c5a0] source 8550366138d576123b9e66a1a7915a04026d79cd
git bisect bad e076e507d65a9ac4e63569e22b7384a0eb67c5a0
# good: [720bbf801075947ba622f09ee203a45582abbc37] source 6b3eb4788f63602c0a446ad8f56539d3b65bb302
git bisect good 720bbf801075947ba622f09ee203a45582abbc37
# good: [ff17e51d8b6816634326250d6278a7a285557316] source 822aac91426848ae36fb5a6bd5ee55db6b60de6a
git bisect good ff17e51d8b6816634326250d6278a7a285557316
# good: [1309e1628e37e05d1b1db2fd2d0cecf311a60b3a] source f3cb21a3fb0141d27b8b7435a2d938439ab7a657
git bisect good 1309e1628e37e05d1b1db2fd2d0cecf311a60b3a
# bad: [4153c997fd6c34a9824b1f40b9c0235afead2688] source 64354e6479e750e4a126abfae5d3f32a1110315e
git bisect bad 4153c997fd6c34a9824b1f40b9c0235afead2688
# bad: [a36fa8685a0f1cd59fc0f54adecbb97c323d39a3] source cc503abb860c33a54a188640a5962dbdf7052284
git bisect bad a36fa8685a0f1cd59fc0f54adecbb97c323d39a3
# bad: [7c5079e31fdf84ceba25c32457f1aea1ba091848] source 6373f9ec6d056ffaef4a209b20a6ec7bab51c7b3
git bisect bad 7c5079e31fdf84ceba25c32457f1aea1ba091848
# bad: [eff7a809142133cb7d66944e89ed448b58649d78] source 6b400cd64e6fa2b69926cbb5f2bcbf339dd2348f
git bisect bad eff7a809142133cb7d66944e89ed448b58649d78
# good: [b385cdc0666523c5cf2a2b2b708c18c05cbcb78a] source 0f95f4f99cf09b597cb9d7b575aed7ba776919dd
git bisect good b385cdc0666523c5cf2a2b2b708c18c05cbcb78a
# bad: [f28634707acff0e2292f8c710c9b25f7e9f9b288] source 415cd81d90aa5dd4f9a5d914fabd4c2371f53e4c
git bisect bad f28634707acff0e2292f8c710c9b25f7e9f9b288
# bad: [cefea17ac62b510e5642adac3d22ed5ccfb5311d] source 3a944846a8460e33fc0cb79f38823e503fb1385c
git bisect bad cefea17ac62b510e5642adac3d22ed5ccfb5311d
# bad: [8e758e0ae958d9fce493c92e2be07d732d16cfaa] source 880c6e40f694342408681929ae5c4fe12bd63332
git bisect bad 8e758e0ae958d9fce493c92e2be07d732d16cfaa
# first bad commit: [8e758e0ae958d9fce493c92e2be07d732d16cfaa] source 880c6e40f694342408681929ae5c4fe12bd63332

--------

Bibisect log of second transition (from comment 4) with bibisect-41max repo:
 8ed0ec4a7fc97f93b6a4f6baf02e471f84a364ba is the first bad commit
commit 8ed0ec4a7fc97f93b6a4f6baf02e471f84a364ba
Author: Matthew Francis <mjay.francis@gmail.com>
Date:   Fri Sep 18 10:31:10 2015 +0800

    source-hash-fcb1e4d8433b49686cd8e53c699dd5d6503ea314
    
    commit fcb1e4d8433b49686cd8e53c699dd5d6503ea314
    Author:     Michael Stahl <mstahl@redhat.com>
    AuthorDate: Wed Feb 13 15:59:38 2013 +0100
    Commit:     Michael Stahl <mstahl@redhat.com>
    CommitDate: Wed Feb 13 16:31:56 2013 +0100
    
        fdo#60471: svgio, sax: enable internal entity declarations for SVG
    
        Change-Id: I147acac61dfe0b626268fe1e600dcf641ca63a0c

# bad: [efef30bb815819507ac55773b2bbbea8d88d9015] source-hash-5da10275a7475efdbfd9de14ea58cf8f4c6c1582
# good: [067772dbb6f4af72962a2b6e7af59b67381b7e0d] source-hash-ba446dd58a4ad324d242afcd5b28d3b4dff5a881
git bisect start 'efef30bb815819507ac55773b2bbbea8d88d9015' '067772dbb6f4af72962a2b6e7af59b67381b7e0d'
# bad: [3ca600daed25e05307069562f7fcf470dfe9eb4d] source-hash-bc96a84370fb167ab2c63582c10535992f31768c
git bisect bad 3ca600daed25e05307069562f7fcf470dfe9eb4d
# good: [bf01a20f5bc14c8e69d1463d6f451c44133a7bb6] source-hash-2d18b593f36c9146bd317b9e62163f8dc078f6b8
git bisect good bf01a20f5bc14c8e69d1463d6f451c44133a7bb6
# skip: [76fbf4b413e355ffe97351fd135a0cae5350579e] source-hash-81c967e06e4247dec7ecb9ffec695ba31cb4f057
git bisect skip 76fbf4b413e355ffe97351fd135a0cae5350579e
# good: [3161ce6fb5cacc595f9b31e33a0bdbb467ce69a2] source-hash-c138a8aec8dccb97948a7d7993e6869da4079b32
git bisect good 3161ce6fb5cacc595f9b31e33a0bdbb467ce69a2
# bad: [3076c903ce9810b30e82d8b34edc56d5fb61a83a] source-hash-623253418b3fc18a8122ea1f71e86628208000ff
git bisect bad 3076c903ce9810b30e82d8b34edc56d5fb61a83a
# bad: [c0975cddb70ce36965661321f0fe81552f8fa320] source-hash-a128bea5dda1d74426dae62b032d5af52f431fe2
git bisect bad c0975cddb70ce36965661321f0fe81552f8fa320
# good: [202a0c58d0fb43c35a23187560c056874790cb3a] source-hash-90d42440eeaf7994f3ad57ebfdc8edb129c8faff
git bisect good 202a0c58d0fb43c35a23187560c056874790cb3a
# good: [1cd638090fba538f55b5820c211476a788dbd7c1] source-hash-2f4bd9d44f8bc9e50cd4b1205fa53e0a15ce0954
git bisect good 1cd638090fba538f55b5820c211476a788dbd7c1
# bad: [64901379f9198b74f0bda7589e29d858534c44a7] source-hash-a775aa57f7b1d5f0f69b8faa5277d1914455ba38
git bisect bad 64901379f9198b74f0bda7589e29d858534c44a7
# bad: [7dd0e569028420b9cf887987a3bb3c90bf1dae58] source-hash-4552bd5a7c61c4f9428ad73e19bbee5b78e510ab
git bisect bad 7dd0e569028420b9cf887987a3bb3c90bf1dae58
# bad: [8ed0ec4a7fc97f93b6a4f6baf02e471f84a364ba] source-hash-fcb1e4d8433b49686cd8e53c699dd5d6503ea314
git bisect bad 8ed0ec4a7fc97f93b6a4f6baf02e471f84a364ba
# first bad commit: [8ed0ec4a7fc97f93b6a4f6baf02e471f84a364ba] source-hash-fcb1e4d8433b49686cd8e53c699dd5d6503ea314
Comment 9 QA Administrators 2018-01-06 03:31:47 UTC Comment hidden (obsolete)
Comment 10 Roman Kuznetsov 2018-06-18 20:46:39 UTC
Created attachment 142887 [details]
result of export from ODT file to SVG in LO 6.1 beta 2

SVG looks the same as ODT. checked in Inkscape.

status -> WFM