Issue 116371 - PDF-Export: background color of hatching not exported
Summary: PDF-Export: background color of hatching not exported
Status: CLOSED FIXED
Alias: None
Product: gsl
Classification: Code
Component: code (show other issues)
Version: OOO330m8
Hardware: PC All
: P3 Trivial (vote)
Target Milestone: OOo 3.3
Assignee: h.ilter
QA Contact: issues@gsl
URL:
Keywords: regression
Depends on:
Blocks: 111112
  Show dependency tree
 
Reported: 2011-01-09 08:07 UTC by maand
Modified: 2017-05-20 10:30 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
The testfile with the backgrounded hatching (10.85 KB, application/vnd.oasis.opendocument.graphics)
2011-01-09 08:10 UTC, maand
no flags Details
PDF-Export with OOo 3.2.1 (14.18 KB, application/pdf)
2011-01-09 08:13 UTC, maand
no flags Details
PDF-Export with OOo 3.3.0 rc 8 (13.99 KB, text/plain)
2011-01-09 08:14 UTC, maand
no flags Details
PDF-Export with OOo 3.3.0 rc 8 (13.99 KB, application/pdf)
2011-01-09 08:14 UTC, maand
no flags Details
PDF-Export with LibreOffice 3.3.0-rc2 (14.95 KB, application/pdf)
2011-01-09 08:15 UTC, maand
no flags Details
Proposed fix for missing background fill for hatched objects (1.74 KB, text/plain)
2011-01-13 14:40 UTC, Armin Le Grand
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description maand 2011-01-09 08:07:52 UTC
A user at the de-users-list made me aware of this issue. He created a sample
file, that I'll attach to this issue.
Description: He created a odg-file with a hatching in the background and set the
hatching to hatch 4. He choose a background color for this hatching and set it
to 20% grey. The odg-file contains also some text in the foreground.
If you export this file with OOo 3.2.1 to PDF you got the background color
exported. If you do the same with OOo 3.3.0-rc8 and LibreOffice 3.3.0-rc2 the
background color is not visible in the PDF-File. This is a regression to OOo
3.2.1 and should be fixed before the release.
Comment 1 maand 2011-01-09 08:10:44 UTC
Created attachment 75506 [details]
The testfile with the backgrounded hatching
Comment 2 maand 2011-01-09 08:13:42 UTC
Created attachment 75507 [details]
PDF-Export with OOo 3.2.1
Comment 3 maand 2011-01-09 08:14:27 UTC
Created attachment 75508 [details]
PDF-Export with OOo 3.3.0 rc 8
Comment 4 maand 2011-01-09 08:14:49 UTC
Created attachment 75509 [details]
PDF-Export with OOo 3.3.0 rc 8
Comment 5 maand 2011-01-09 08:15:42 UTC
Created attachment 75510 [details]
PDF-Export with LibreOffice 3.3.0-rc2
Comment 6 maand 2011-01-09 08:29:00 UTC
I made my tests on openSuSE 11.3-x64 with the vanilla versions of OOo-x64 and
LibreOffice-x64.
Comment 7 Armin Le Grand 2011-01-13 14:39:17 UTC
AW: Happened with fixing #i111954#, is in the version for 7 months. Problem is
that when converting the primitives to metafile format (since all exports still
use metafile format) the evtl. background fill of a hatch-filled objects was not
taken into account. This is because before #i111954# the primitive decomposition
was used which does everything correct; that fix calls OutputDevice::DrawHatch
again and thus backgrund fill was gone.

Adding proposed fix as patch. Fix is compatible and not critical, it just
creates the missing filled PolyPolygon when needed.
Comment 8 Armin Le Grand 2011-01-13 14:40:10 UTC
Created attachment 75555 [details]
Proposed fix for missing background fill for hatched objects
Comment 9 Armin Le Grand 2011-01-13 14:40:39 UTC
AW->PL: Please have a look at the patch and review.
Comment 10 philipp.lohmann 2011-01-13 14:48:09 UTC
patch seems to work fine
Comment 11 philipp.lohmann 2011-01-13 14:58:34 UTC
committed in CWS calc65
Comment 12 mdxonefour 2011-01-13 15:01:41 UTC
adjusting target to 3.3
Comment 13 philipp.lohmann 2011-01-13 17:12:39 UTC
please verify in CWS calc65
Comment 14 h.ilter 2011-01-14 09:27:12 UTC
Verified with cws cal65 = OK